Repository navigation
fix(planning): pin interview source-text links to a commit SHA #7752
Description
Parent
Refs #5795
Depends on
- none: on the frontier now
Outcome
A source-text link on an interview question keeps pointing at the quoted lines after main moves. Links carry a commit SHA instead of main. Today 9 of 85 repo source anchors in one session pointed at lines that no longer hold the quote.
Owner decisions recorded on #5795 (2026-10-12): "Source anchors: link source quotes to a commit SHA, or re-check anchors against main when a question is posted?" → Link to a commit SHA (permalink).
Key interfaces
- Whichever layer builds the source URL: the interview skill's question-authoring rules (
surface.md) and theadd/reviseops inround.py, or the page if it builds the link. This item finds the layer and fixes it there.
Done when
- A source link produced through the interview flow names a commit SHA, not a branch (test at the layer that builds it).
- If the session writes the URL: the skill states the permalink rule, and
add/revisewarns on ablob/<branch>source link (test). - The planning surface test suite passes.
Out of scope
- Linking repo paths in recommendations and commitments (sibling item, which builds on this one).
Activity
kyle-sexton commentedon Oct 17, 2026
Ratified in the owner's batch of decomposed children (2026-10-17).
kyle-sexton commentedon Oct 18, 2026
Agent Brief
Type: Bug
Summary: Pin interview source-text links to a commit SHA and warn on a branch link.
Current behavior:
The session writes source-text links into a question's facts, and they point at
blob/main, so after main moves some links no longer land on the quoted lines.Desired behavior:
The interview surface rules for a Source block require a link pinned to a commit SHA. The
addandreviseops warn, without refusing, on a source link of the formblob/<branch>, naming the question.Key interfaces:
round.pyaddandrevise: a warning on a branch source link.Acceptance criteria:
test_round.pygains cases assertingaddandrevisewarn on ablob/mainsource link and do not warn on a SHA link, andscripts/run-test-file.sh plugins/planning/surface/test_round.pyexits 0.scripts/check-changelog-fragments.shexits 0.Out of scope:
This was generated by AI during backlog shaping. Held, not admitted: this brief is drafted and checked, but the issue lane would stop on it today. It is admitted, by adding
agent-ready, once the lane can run the skill-quality check scripts (Automation Lanes' PR #7912). Held under the owner's 2026-10-18 lane-narrowing course correction.melodic-automation-lanes commentedon Oct 18, 2026
The work-resolve-issue run that
agent-readystarted failed before it reported, or the token broker refused it because its workflow files differ from main's. work-unblock removed and addedagent-readyback to start a fresh run on main's workflow files. It does this once per version of those files, so if this run fails the same way, a person looks at it.