feat: map confirmed findings to the code that owns them - #90
Merged
Merged
Conversation
Moves itd-3 from drafts to planned with its acceptance criteria, mechanism claim, five stamped scope conditions, and grounds; mints and links its spec, spc-2609152218094570, which designs the host-delegated `map` command, the `refs.jsonl` record family, and `review -kind refs`. Terminal-session anchoring is a different problem and is filed as its own draft, itd-2609152113364815, with its three known obstacles as open questions. The decomposition calibration note records the routing and grades it against the maintainer's confirmation. One decision-log line records the planning choices. Assisted-by: Claude:claude-fable-5-1
Ships itd-3 (spec spc-2609152218094570). `testimony map -session DIR -repo DIR` emits a host-delegated mapping request over each confirmed finding whose `ui` carries a selector or route: the anchor, the finding's record, its event window, and the repository's path. `map -ingest FILE` is the validation boundary into a new `refs.jsonl` record family: every reference must name a currently confirmed anchored finding, this session, a role in the closed set, and a repo-relative path that is an existing regular file under `-repo` (containment checked through symlinked parents, no symlink at the leaf), with any line within the file's length; the checked path is what is recorded, an answer is capped at 1000 references before any is checked, and unknown fields such as `confidence` are refused. `review -kind refs` appends accept / reject decisions with no edit path, showing the source around each line when `-repo` is given to the walk. `map -render` renders one issue draft per mapped finding with a CLI-derived title and steps, the quote, and every reference with its current status; review does not gate it and nothing is filed. A session with no confirmed anchored finding is refused with the by-status tally, including the anchored count, and writes nothing. The repository is opened read-only and never written; the emitted request is the one artefact naming an absolute local path, so `map` refuses an `-out` inside the session directory. Docs, sample session, and CI smoke follow; the spec is closed and the intent shipped in the same change, with the fidelity audit ingested. Assisted-by: Claude:claude-fable-5-1
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.
Ships intent itd-3, codebase mapping, against spec spc-2609152218094570. Stacked on #89 (the planning record); merge that first.
testimony map -session DIR -repo DIRemits a host-delegated mapping request: a versioned rubric, the repository path, and, for each confirmed finding whoseuicarries a selector or route, its record and event window. Web anchors only; terminal findings are itd-2609152113364815.map -ingest FILEis the validation boundary into a newrefs.jsonlrecord family: every reference must name a currently confirmed anchored finding, this session, a role inowner|handler|route|test, and a repo-relative path that is an existing regular file under-repo(containment checked through symlinked parents, no symlink at the leaf), with any line within the file's length. Unknown fields such asconfidenceare refused. Transactional; empty answers and re-ingest over decisions are refused. The repository is opened read-only and never written.review -kind refsappendsaccepted|rejecteddecisions (no edit path); with-repothe walk shows the source around each line through the same containment rule.map -renderrenders one issue draft per mapped finding: CLI-derived title and steps from the event window, the quote, and every reference with its current status. Review does not gate the render; nothing is filed anywhere.cli.md,session-directory.md, a new how-to, README, agent file, context; sample session gainsrefs.jsonlpointing into a fixture tree; CI smoke covers render and emit. Spec closed and intent shipped in the same change; fidelity audit ingested.Assisted-by: Claude:claude-fable-5-1