fix(instruction-placement): three surfaces still say path-scoped rules load only on read #6199

Activity

1. kyle-sexton

   added

   needs-triageNot yet classified. Floor until a type and one priority tier are set.Not yet classified. Floor until a type and one priority tier are set.

   on Oct 4, 2026
2. github-actions commented on Oct 4, 2026

   

   github-actionsbot

   on Oct 4, 2026 – with GitHub Actions

   Contributor

   More actions

   Thanks for the report. This describes a documentation-accuracy follow-up to #6101: three surfaces in the instruction-placement plugin still describe path-scoped rules as triggering only on read, without the Read/Write/Edit-since-2.1.288 qualifier that #6101 established. I checked the current repository state and confirmed the wording is still present in scripts/render-index.sh's header comment, skills/realign/SKILL.md, and evals/adherence-results.md (the adherence-experiment.sh reference appears to be a general "read" usage rather than the stale claim, worth double-checking during the fix). The acceptance criteria and scope (align wording with verified-mechanics.md, re-render the index preamble, bump version and CHANGELOG) are clear. I didn't find any duplicate open issues. No gaps that need clarification from here.

   _{Automated triage. A maintainer reviews it.}
3. github-actions

   added

   good first issueGood entry point for new contributors.Good entry point for new contributors.

   priority: lowNice-to-have, cosmetic, or speculative; opportunistic.Nice-to-have, cosmetic, or speculative; opportunistic.

   on Oct 4, 2026
4. kyle-sexton commented on Oct 4, 2026

   

   kyle-sexton

   on Oct 4, 2026

   ContributorAuthor

   More actions

   This was generated by AI during triage.

   Verified on origin/main 79abd80 (after #6201 merged): the plugin's verified-mechanics reference now says a path-scoped rule loads when Read, Write or Edit targets a matching file on 2.1.288 and later, Read only before. The surfaces below still say read only.
   Basis: verified, origin/main 79abd80: plugins/instruction-placement/scripts/render-index.sh:5 (header comment) and :259 (rendered preamble, "enters context automatically when Claude reads a file it covers"); plugins/instruction-placement/skills/realign/SKILL.md:211 ("the rule fires on read"); plugins/instruction-placement/evals/adherence-results.md:56; plugins/instruction-placement/evals/adherence-experiment.sh:71 (a copy of the preamble); this repository's AGENTS.md carries the same rendered preamble.

   Agent Brief

   Type: Bug
   Summary: Bring the instruction-placement plugin's remaining "loads on read" wording in line with its corrected loading-mechanics reference (Read, Write or Edit from Claude Code 2.1.288; Read only before).

   Current behavior:
   The index renderer's header comment, the preamble it renders into a repository's always-loaded index, the realign skill's gotcha about index regeneration, and the adherence experiment's script and results write-up all describe path-scoped rules as entering context only when Claude reads a covered file, with no version qualifier.

   Desired behavior:

   • Each of those surfaces describes the trigger as the plugin's loading-mechanics reference now does: Read, Write or Edit of a matching file on 2.1.288 and later, Read only before.
   • The rendered index preamble is updated, and this repository's own always-loaded index is re-rendered in the same change so the plugin's index-matches-disk check stays green.
   • The adherence experiment files record a measured past run: add the qualifier where they state the mechanism, and do not rewrite any measured result or the version it was measured on.
   • The creation-deny verdict is unchanged; lifting it is feat(instruction-placement): decide the write-trigger deny, path-scoped skill routing, and rule YAML validation (split from #6101) #6193's decision.

   Key interfaces:

   • The index renderer's preamble text (consumer repositories pick it up on their next re-render).
   • The realign skill's gotchas.

   Acceptance criteria:

   • None of the listed surfaces says path-scoped rules load only on read without the 2.1.288 qualifier.
   • This repository's rendered index matches the renderer's output (the instruction-placement check passes).
   • No measured result in the adherence write-up changes.
   • The plugin is version-bumped with a CHANGELOG entry.

   Out of scope:

   • The write-trigger / creation deny and anything else feat(instruction-placement): decide the write-trigger deny, path-scoped skill routing, and rule YAML validation (split from #6101) #6193 decides.
   • Re-running or re-measuring the adherence experiment.

   Routing: delegable. Work class scoped (bug fix): wording plus one re-render, bounded to one plugin and this repository's index. Priority low (documentation accuracy, no correctness gate depends on it).
5. kyle-sexton

   added

   agent-readyFully specified and briefed; eligible for autonomous pickup from the frontier.Fully specified and briefed; eligible for autonomous pickup from the frontier.

   work-class: scopedA briefed fix or small feature; blast radius bounded by the brief, tests exist.A briefed fix or small feature; blast radius bounded by the brief, tests exist.

   status: readyTriaged, unblocked, and fully specified; eligible to pick up.Triaged, unblocked, and fully specified; eligible to pick up.

   and removed

   good first issueGood entry point for new contributors.Good entry point for new contributors.

   needs-triageNot yet classified. Floor until a type and one priority tier are set.Not yet classified. Floor until a type and one priority tier are set.

   on Oct 4, 2026
6. kyle-sexton commented on Oct 4, 2026

   

   kyle-sexton

   on Oct 4, 2026

   ContributorAuthor

   More actions

   This was generated by AI during work-loop.

   Ratify autonomous dispatch of this C3 bug-fix item?

   Classification: work-class: scoped (C3), bug-fix-shaped (confirmed at triage). Intended dispatch: the work-loop worker lane, through /work-items:work, to a draft PR the lane never merges. First drain is not complete and no item-level ratification exists, so it is queued here. Ratify through /work-items:attend-queue.

   Lane: work-loop, instance melo-worker-1, cycle 2.
7. kyle-sexton

   added

   needs-humanHuman-in-the-loop required; autonomous sessions must not resolve items carrying this.Human-in-the-loop required; autonomous sessions must not resolve items carrying this.

   and removed

   agent-readyFully specified and briefed; eligible for autonomous pickup from the frontier.Fully specified and briefed; eligible for autonomous pickup from the frontier.

   on Oct 4, 2026
8. kyle-sexton

   added 4 commits that reference this issue on Oct 10, 2026

   fix(instruction-placement): drop the Read-only AGENTS.md attach claim

   ...

   ef61402

   chore(instruction-placement): note the AGENTS.md attach fix in the fr…

   ...

   14e6521

   fix(instruction-placement): stop implying Write/Edit triggers for AGE…

   ...

   27795df

   fix(instruction-placement): finish the read-only trigger wording sweep

   ...

   2ee5711
9. kyle-sexton

   closed this as completedin #6780on Oct 10, 2026