check -q printed nothing about a failed file: each one was logged at info level and -q shows errors only, so the exit status was the only sign the check failed.
check now logs each failed file (MISSING, SIZE_MISMATCH, HASH_MISMATCH, MODE_MISMATCH, ERROR) at error level, and also each file the manifest does not list (EXTRA) when --no-extra-files makes it a failure. Without --no-extra-files such a file stays a warning, which -q hides, as #103 settled. The closing N failed summary stays at info level.
Not visible in the diff: error-level lines carry the error symbol (red on a terminal) instead of the info bullet, so a failed file now looks like an error with or without -q.
`check -q` printed nothing about a failed file: each one was logged at info level and `-q` shows errors only, so the exit status was the only sign the check failed.
`check` now logs each failed file (`MISSING`, `SIZE_MISMATCH`, `HASH_MISMATCH`, `MODE_MISMATCH`, `ERROR`) at error level, and also each file the manifest does not list (`EXTRA`) when `--no-extra-files` makes it a failure. Without `--no-extra-files` such a file stays a warning, which `-q` hides, as https://git.eeqj.de/sneak/mfer/issues/103 settled. The closing `N failed` summary stays at info level.
Not visible in the diff: error-level lines carry the error symbol (red on a terminal) instead of the info bullet, so a failed file now looks like an error with or without `-q`.
Fixes https://git.eeqj.de/sneak/mfer/issues/195
Model: opus-5-5
check logged each failed file at info level, and -q shows errors only,
so `check -q` reported a failure through its exit status alone. Each
failed file, and each file the manifest does not list under
--no-extra-files, is now logged at error level. A file the manifest
does not list stays a warning without --no-extra-files, and the closing
summary stays at info level.
Model: opus-5-5
Judgement call: failed files now show the error symbol instead of the info bullet. That comes with logging them at error level, as #195 asks, and matches the README's "displaying error".
Judgement call: the new assertion in TestCheckWarnsAboutUnlistedFiles covers the --no-extra-files case the issue names, so it is in scope.
Model: opus-5-5
Review passed.
- Judgement call: failed files now show the error symbol instead of the info bullet. That comes with logging them at error level, as https://git.eeqj.de/sneak/mfer/issues/195 asks, and matches the README's "displaying error".
- Judgement call: the new assertion in `TestCheckWarnsAboutUnlistedFiles` covers the `--no-extra-files` case the issue names, so it is in scope.
Model: opus-5-5
clawbot
merged commit f68cc284ac into next2026-10-13 16:46:14 +02:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
check -qprinted nothing about a failed file: each one was logged at info level and-qshows errors only, so the exit status was the only sign the check failed.checknow logs each failed file (MISSING,SIZE_MISMATCH,HASH_MISMATCH,MODE_MISMATCH,ERROR) at error level, and also each file the manifest does not list (EXTRA) when--no-extra-filesmakes it a failure. Without--no-extra-filessuch a file stays a warning, which-qhides, as #103 settled. The closingN failedsummary stays at info level.Not visible in the diff: error-level lines carry the error symbol (red on a terminal) instead of the info bullet, so a failed file now looks like an error with or without
-q.Fixes #195
Model: opus-5-5
Review passed.
TestCheckWarnsAboutUnlistedFilescovers the--no-extra-filescase the issue names, so it is in scope.Model: opus-5-5