Conversation
Add the multi-line comment layout rules (one sentence per line, never wrap, marker-agnostic '-' sub-bullets for parallel sub-topics only) with two worked examples, adopted from ESPHome-Config's local section. Fold the redundant line-length bullet into the sentence-per-line rule.
) Classify all 18 AGENTS.md sections (16 verbatim incl one operational-scoped, 2 intent), declare each fidelity in files.json, add spec/section-model.md as the ground-truth classification + review-weight principle, generalize the Branching Model SHA sentence, and update fidelity-model.md's rationale. ~90% of AGENTS.md is now byte-checked fleet-wide.
Contributor
There was a problem hiding this comment.
Pull request overview
Codifies governance around how AGENTS.md is carried across the fleet by introducing a per-section fidelity model, and updates the canonical comment layout rules so multi-line comments converge on a consistent structure.
Changes:
- Add
spec/section-model.mdas the ground-truth classification ofAGENTS.mdsections (verbatim vs intent, plusappliesTo). - Expand
spec/files.jsonto declare per-section fidelity forAGENTS.md, shifting most sections to verbatim enforcement. - Update governance docs (
spec/fidelity-model.md,AGENTS.md) to reference the section model and to codify multi-line comment layout (one sentence per line + sub-topic bullets with worked examples).
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
spec/section-model.md |
Defines the per-section fidelity categories and the canonical classification table for AGENTS.md. |
spec/files.json |
Declares AGENTS.md section fidelity explicitly (mostly verbatim, with repo-specific sections kept intent). |
spec/fidelity-model.md |
Updates the fidelity model rationale to point to the new section model for AGENTS.md section classification. |
AGENTS.md |
Generalizes Branching Model language (removes repo-history SHA examples) and codifies structured multi-line comment layout with examples. |
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
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.
Two governance PRs from the comment-style investigation and the verbatim-carry codification:
spec/section-model.mdis the ground-truth classification + the principle that section-structure changes carry fleet-wide review weight. Branching Model generalized (SHAs removed) to carry verbatim.Fleet impact: the audit now byte-checks ~90% of AGENTS.md across the fleet - the intended convergence pressure, burned down via the AGENTS refreshes already in motion.
Merge commit, no release fires (human merge never auto-publishes).