standards: per-concern standards layout #7583

Activity

1. 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.

   on Oct 16, 2026
2. kyle-sexton

   added a parent issue on Oct 16, 2026

   • Epic: absorb Lauren Tan's and Matt Pocock's agent workflows (poteto-review) #7555
3. kyle-sexton

   marked this as blocked on Oct 16, 2026

   • config-cascade: settings resolution and convention home #7571
4. kyle-sexton

   added the

   Task

   issue type on Oct 16, 2026
5. kyle-sexton

   set

   Work status

   to

   needs-spec

   on Oct 16, 2026
6. kyle-sexton

   marked this as blocking 2 issues on Oct 16, 2026

   • review: spec-first fanout and ratchet move to toolchain #7584
   • architecture: boundaries and conventions lenses #7590
7. github-actions commented on Oct 16, 2026

   

   github-actionsBot

   on Oct 16, 2026 – with GitHub Actions

   Last edited by melodic-automation-lanes

   Contributor

   More actions

   Proposed work class: work-class: structural
   Basis: automated intake triage; the lanes App applied this class.

   Size: Large
   Complexity: Medium
   Uncertainty: Known
   Flags: none
   Blocked by: none

   Decision recorded. This issue's - [ ] criteria are already in its body, written by a listed human. The owner amended D97/Q150 on 2026-10-16 to allow one pointer line in AGENTS.md. Nothing new is decided here. [...]
   [quote truncated; the full decision is at the link]

   #7583 (comment)

   The decision settles the open questions: standards go into per-concern files, AGENTS.md keeps one pointer line, and the issue closes through its sub-issues (per-concern-split, review-md). Nothing is left to ask.

   I found no open duplicates. The dependency #7571 isn't in the open-issue list I was given, so I couldn't confirm its state. Part of the work is in melodic-software/standards, which is a separate repository.

   _{Automated triage, applied by the lanes App.}
8. kyle-sexton

   added

   needs-specNeeds a Brief or plan before anyone can build it.Needs a Brief or plan before anyone can build it.

   on Oct 16, 2026
9. kyle-sexton commented on Oct 16, 2026

   

   kyle-sexton

   on Oct 16, 2026

   MemberAuthor

   More actions

   What the article adds to the standards layout, and the owner's 2026-10-16 decisions. Source: https://claude.dev/blog/solving-code-review-at-anthropic (2026-10-15); ids are digest unit and claim.

   Owner decisions, 2026-10-16.

   • D97/Q150 (GUIDE 97) is amended to allow one pointer line in CLAUDE.md/AGENTS.md to the per-concern review files. Reason: local /code-review reads CLAUDE.md, and its docs say of REVIEW.md: "It doesn't read REVIEW.md" (https://code.claude.com/docs/en/code-review-local). Without a pointer, a local review sees none of this repository's review rules. The line is a pointer, not a rule, so Q150's "no review rules inside AGENTS.md" holds.
   • The rule list now under AGENTS.md ## Code Review Rules moves into the per-concern files, leaving only the pointer line.

   The gap is narrow: under Q500 (GUIDE 75) a repository with a CI lane runs no local review, and the CI lanes follow REVIEW.md citations (GUIDE 122). It affects ad hoc local sessions and consumers set to review_venue: local.

   A suggestion from the ADR review consensus, not an owner decision: load the per-concern files by path scope, so each enters context only when a file it covers is touched.

   This thread already puts the nit cap in its per-concern file (Q570). The article's REVIEW.md traits it lacks (04 C18-C20), which belong in those files:

   • Keep it short.
   • Say what failure looks like in this repository (for docs "it lies", for config "it misroutes").
   • List paths to skip.
   • Ask for a file and line as evidence instead of a guess from a name.
   • Ask for a summary that starts with a count and says plainly when there is nothing to fix.

   The article also checks separately whether the reviewer follows REVIEW.md (04 C17).

   Background only: managed Code Review is not used here (GUIDE 122, Q594). Q570's generated REVIEW.md stays an opt-in for consumers on that service.

   Against the originating sources. This thread rejects Matt's single standards file and Lauren's rule table in the instruction file. The article agrees with pointing out of a short CLAUDE.md (04 C14); its reviewer also flags code that breaks a CLAUDE.md convention (04 C15). The pointer line meets 04 C15 without putting rules in the instruction file.

   Related: #7576 (PR template), #7573 (codify destinations), #7638 (whether rules take effect).
10. kyle-sexton

    marked this as blocked on Oct 18, 2026

    • config-cascade: resolver and reader implement the config ADR (overlay, policy floors, root classification, --ref) #7938
11. kyle-sexton

    unmarked this as blocked on Oct 18, 2026

    • config-cascade: settings resolution and convention home #7571
12. kyle-sexton

    removed

    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.

    on Oct 24, 2026
13. kyle-sexton commented on Oct 24, 2026

    

    kyle-sexton

    on Oct 24, 2026

    MemberAuthor

    More actions

    Released for re-triage under the owner's 2026-10-24 rule: an issue stops for a person only for an intent or vision question, a security call, a host or attended run, or spend outside prepaid credit. The hold label came off and intake triage runs again now; it re-applies a hold, with a numbered question, when one of those applies.

    To hold this issue again, add needs-human (or status: needs-decision) and say why in a comment.
14. github-actions

    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 24, 2026
15. 13 remaining items

    Load more
16. melodic-automation-lanes

    added

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

    on Oct 26, 2026
17. melodic-automation-lanes commented on Oct 26, 2026

    

    melodic-automation-lanesBot

    on Oct 26, 2026

    Last edited by melodic-automation-lanes

    Contributor

    More actions

    Split at intake (owner Q93). This issue is Size Large, so its done-when is split into 2 sub-issues, each holding its task-list lines as written.

    • Part 1: task-list lines 1, 4: standards: per-concern standards layout - part 1 of 2 #10231
    • Part 2: task-list lines 2, 3, after part 1: standards: per-concern standards layout - part 2 of 2 #10232

    _{Automated split by the lanes App. Admission waits on this issue while a sub-issue is open; work-unblock closes it as completed once every sub-issue is closed by a merge or a listed human.}
18. melodic-automation-lanes

    added sub-issues on Oct 26, 2026

    • standards: per-concern standards layout - part 1 of 2 #10231
    • standards: per-concern standards layout - part 2 of 2 #10232
19. melodic-automation-lanes

    unmarked this as blocking 2 issues on Oct 26, 2026

    • review: spec-first fanout and ratchet move to toolchain #7584
    • architecture: boundaries and conventions lenses #7590
20. melodic-automation-lanes

    added

    admission-deferredAdmitted but waiting for WIP room; the hourly work-unblock sweep labels it agent-ready.Admitted but waiting for WIP room; the hourly work-unblock sweep labels it agent-ready.

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

    and removed

    admission-deferredAdmitted but waiting for WIP room; the hourly work-unblock sweep labels it agent-ready.Admitted but waiting for WIP room; the hourly work-unblock sweep labels it agent-ready.

    on Nov 1, 2026
21. kyle-sexton commented on Nov 1, 2026

    

    kyle-sexton

    on Nov 1, 2026

    MemberAuthor

    More actions

    Issue Triage session: removing agent-ready from this split parent; it waits on its open sub-issues #10231 and #10232, and lane runs on it no-op (run 41293781163).
22. kyle-sexton

    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 Nov 1, 2026
23. kyle-sexton commented on Nov 1, 2026

    

    kyle-sexton

    on Nov 1, 2026

    MemberAuthor

    More actions

    Issue Triage session, amending this issue's Decisions line "REVIEW.md syncs from melodic-software/standards, so the change there is another-repository work (owner stop)": the standards change is agent work, not an owner stop. GitHub Support ruled it 2026-11-01 ~15:5xZ; no owner words or owner-decisions row back the "owner stop" note. Automation Lanes files the melodic-software/standards PR (one repository-neutral REVIEW.md pointer to a reviewed repository's docs/review/README.md); its scope is in the #10231 record. The body is left as filed.
24. kyle-sexton

    added a sub-issue on Nov 2, 2026

    • docs(review): state the nit cap in its own per-concern review file #12198
25. melodic-automation-lanes commented on Nov 2, 2026

    

    melodic-automation-lanesBot

    on Nov 2, 2026

    Contributor

    More actions

    Every sub-issue of this split is closed as completed by a listed human or a merge (#10231, #10232, #12198), so the lanes App closes it as completed.
26. melodic-automation-lanes

    closed this as completedon Nov 2, 2026