docs(handoff): rank 15 merged upstream as ZacxDev/cairn#7 β and NOT marked done, because merged is not live - #1419
Merged
Conversation
β¦ and deliberately not marked done, because merged is not live `cairn recall`'s digest told readers to run `--ref <name>` / `--limit N` and the client exited 2 with `unrecognized arguments`, so a reader following the output it had just been shown fell back to the raw module. The fix is not the flags. `main()` derived mode/limit/page AND enforced the flag-conflict rules inline, while the client reaches the module as a LIBRARY and never through `main()` β so offering the flags meant open-coding both at a second site. `recall_selection()` and `reject_recall_flags()` are extracted and called by both, so `main()` got shorter instead of the wrapper growing a copy. All four flags wired, not the two the item named: fixing only `--ref` leaves the class open, the same enumeration-vs-derivation shape rank 12 closed. Evidence is recorded split rather than totalled. Two tests are RED at base on their own assertions; a third fails at base with AttributeError and is NOT counted as regression evidence. Four flags measured live against the real store, because a passing suite and a working client are different claims. A defect the change introduced is recorded rather than quietly fixed: exposing `--limit` made `recall()`'s ValueError reachable from the command line for the first time, and `--limit 0` printed a traceback at rc 1. Guarded, watched before and after, reachable by measurement. Also recorded: the two failures the FULL suite found that a four-test subset did not β the mutation battery refusing orphaned anchors after the guards moved, and a TEXT ledger tripping on a comment quoting the callee's name. The second is a false red, so the comment moved and the guard did not; that ledger cannot tell a call from a comment, which is noted and not fixed. π΄ The rank is NOT marked done. Its closing condition named a merged devrc PR and the fix landed in the OSS client, so `readlink -f ~/.local/bin/cairn` still resolves to devrc/scripts/cairn and `--ref` is still broken on this host. The condition is re-pointed at "#7 merged AND #1406 landed AND the pin bumped past 059ec17", so it is checkable again rather than quietly unsatisfiable. Rank 3 is untouched here: the session that owns it had already recorded #1406 across twelve mentions, and duplicating that would be the drift this doc keeps paying for. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MCjWicwrMAwjGTh3uXsSNT Claude-Session-Id: 054db069-07b1-4abb-962a-f677e633829f
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
docs(handoff): rank 15 fixed and MERGED upstream as ZacxDev/cairn#7 β and deliberately not marked done, because merged is not live
cairn recall's digest told readers to run--ref <name>/--limit Nand theclient exited 2 with
unrecognized arguments, so a reader following the outputit had just been shown fell back to the raw module.
The fix is not the flags.
main()derived mode/limit/page AND enforced theflag-conflict rules inline, while the client reaches the module as a LIBRARY and
never through
main()β so offering the flags meant open-coding both at a secondsite.
recall_selection()andreject_recall_flags()are extracted and calledby both, so
main()got shorter instead of the wrapper growing a copy. All fourflags wired, not the two the item named: fixing only
--refleaves the classopen, the same enumeration-vs-derivation shape rank 12 closed.
Evidence is recorded split rather than totalled. Two tests are RED at base on
their own assertions; a third fails at base with AttributeError and is NOT
counted as regression evidence. Four flags measured live against the real store,
because a passing suite and a working client are different claims.
A defect the change introduced is recorded rather than quietly fixed: exposing
--limitmaderecall()'s ValueError reachable from the command line for thefirst time, and
--limit 0printed a traceback at rc 1. Guarded, watched beforeand after, reachable by measurement.
Also recorded: the two failures the FULL suite found that a four-test subset did
not β the mutation battery refusing orphaned anchors after the guards moved, and
a TEXT ledger tripping on a comment quoting the callee's name. The second is a
false red, so the comment moved and the guard did not; that ledger cannot tell a
call from a comment, which is noted and not fixed.
π΄ The rank is NOT marked done. Its closing condition named a merged devrc PR and
the fix landed in the OSS client, so
readlink -f ~/.local/bin/cairnstillresolves to devrc/scripts/cairn and
--refis still broken on this host. Thecondition is re-pointed at "#7 merged AND #1406 landed AND the pin bumped past
059ec17", so it is checkable again rather than quietly unsatisfiable.
Rank 3 is untouched here: the session that owns it had already recorded #1406
across twelve mentions, and duplicating that would be the drift this doc keeps
paying for.
π€ Generated with Claude Code
https://claude.ai/code/session_01MCjWicwrMAwjGTh3uXsSNT