docs(agents): re-anchor Trap 3 now that #1761 has landed (TASK-066 follow-up) - #1773
Merged
Merged
Conversation
#1761 merged as c9eb7eb, and on cbc3fe3 the range Trap 3 cited when it was written (daemon-supervisor.js:357-359) now falls inside the adopt loop. The record-first trio is :408-410, so the citation moves with the code and the corollary's "once #1761 lands" hedge becomes a plain statement. The header gains a dated re-anchor note that names what was re-measured and what was not. TASK-066 follow-up; #1762 merged as cbc3fe3.
lilyshen0722
force-pushed
the
kai/daemon-doc-trap3-anchor
branch
from
September 19, 2026 17:00
8c5f5ec to
6fec5b3
Compare
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
1 participant
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.
Vera's follow-up on #1762, recorded on TASK-066 with trigger
#1762 merged— fired now that it has landed atcbc3fe38.Trap 3 called #1761 "unmerged" and said main
ac544d5c"still has the inverted order atdaemon-supervisor.js:357-359". #1761 merged asc9eb7eb8, and oncbc3fe38that range is inside the adopt loop (:350–365), so this is a re-anchor, not a tense flip: the record-first trio is:408–410, and the corollary's "scope this half to once #1761 lands" hedge becomes a plain statement — the status path reads the record first and falls back torow.runtime?.…only when there is no record at all.Also added a dated re-anchor note in the header saying what was re-measured (
:408–410and the corollary) and what was not — the rest of the field map is still a read ofac544d5c, and saying so is cheaper than implying the whole doc moved with it.Docs only ⇒ no release-ladder slot. Swept the diff for the class (#1762's trap): the only remaining "unmerged" wording is the explicitly historical "at the time of writing", immediately followed by the merge SHA.