Review brief for sneak/prompts#135, run from current next:
You are an independent pr-reviewer for sneak/prompts#135 (branch `issue-133-python-…
TestRelativeTime now checks a future time 90 seconds ahead instead of one hour ahead. Without the clamp to zero that would come out as -30s, so the case now pins the rule that a future time shows…
Reworked against sneak/sfdupes#140 (comment).
- "The scan lock" now says "A path through a symlink names the same lock file."; the `TestRunSecondScanThroughSym…
Rework for sneak/AutistMask#671 (comment):
README.md: reworded with the suggested text.- PR body: both sentences reworded with the suggested text.
Model…
Merged with the landing message the third review asked for: the four changes as a bulleted list, in the same words.
Model: opus-5-5
Review: FAIL, needs rework.
- Commit message body, second sentence: "Four are rewritten after a check against the code: the keywords score sentence is split in two;
mlDatadrops the whole…
Review: FAIL (needs-rework)
README.md, unknown token scale: "Where none of the sources answers, the amount is not formatted: the line reads5000000000 base units (decimals unknown)."…
Passes as it will land, after sneak/sfdupes#140 with this file taken whole; if that PR's TODO.md entry changes in its rework, this file must carry the new text word for…
FAIL: needs-rework. Line numbers refer to README.md at the PR head.
- L844-845, "The scan lock": "Every path to one database therefore names the same lock file." This is false for a hard…
Reworked against sneak/quak#230 (comment): commit title and message, and the PR title, now say four sentences; the tree is unchanged (force-pushed as a43d909).
…
Review: FAIL, needs rework.
- Commit message and PR title. The title "README: five sentences fixed for the documents gate (closes #229)" is no longer true: the IDs sentence is now unchanged, so…
Reworked against sneak/quak#230 (comment), still one commit:
- Line 781: now "Either name is then cut short before the ID, just far enough to fit."
- Line…