Re-land Standard v1.46 §5.7 + v1.47 §17 syncs on main (recovers #33/#34) - #38
Conversation
spacecraft-standard-constitution: adds §5.7 (AGENTS.md is the authority, CLAUDE.md is an @AGENTS.md import plus Claude-only content), the §16 checklist bullet, the spacecraft-agentic-cli cross-reference row, and the v1.46 changelog entry. Masthead and frontmatter to v1.46 / 2026-08-08; description measures 988 chars under the §5.6 cap. spacecraft-agentic-cli: inverts the rule this skill previously taught. references/agents-md-authoring.md §1 and §3 said "CLAUDE.md MUST be a strict superset of AGENTS.md. Start by symlinking or copying" — a rule that produced duplication by construction and that every new scaffold reproduced. Both assets/ templates rewritten to the import model; CLAUDE.md length budget cut from 100-250 to 5-40 lines. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
The maintainer-local CLAUDE.md overlay (444 lines) becomes the tracked, authoritative AGENTS.md; the old 45-line AGENTS.md digest is folded into it so the hard rules survive in one place. CLAUDE.md is now the @AGENTS.md import plus two Claude-Code-only notes. Both leave .gitignore. The file was read end to end and reviewed for sensitive content first: the Home Manager fan-out section describes `~/`-relative and /nix/store paths only, with no credentials, hostnames, or personal filesystem paths. construct-cli's byte-identical pair is collapsed the same way. Steelbore Standard v1.46 §5.7. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Mirror the Standard's §17.1/§17.2 rewrite into the constitution skill. Every tracked track now gets its own row, its own 20-cell bar, and its own percentage, in a fixed order: milestone rows M0..Mn ascending, then MVP, then TODO, then PLAN, then PRD. TODO and PLAN are new tracks — the old three-line format could not report against a plan document or a task list. Only applicable rows are emitted; padding with fabricated 0% rows is forbidden. §17.2 defines two cell styles. Milestone/TODO/PLAN/PRD rows use U+2588/U+2591 with tight brackets; the MVP row keeps U+25B0/U+25B1 with one space inside each bracket, which is what holds the columns aligned. Cell count is specified for the first time: percentage scaled to twenty cells and rounded, saturating only at exactly 0% and 100%. §16's §17 checklist bullet rewritten to match. Also updates the reporting example in spacecraft-accessibility-support's audit reference, which carried the old three-line format. Frontmatter description measures 988 rendered chars, unchanged — only the version number moved.
…-v147-sync # Conflicts: # construct-cli/AGENTS.md
…re-landed v1.46/v1.47 stack
The check-skill-refs gate rejects document-version pins
('Standard v1.46') in prose — cite the section instead. This pin came in
with the §5.7 sync and is why #34's checks were unstable.
Rebuilds the bundles for the three skills whose sources this branch
changes (spacecraft-agentic-cli, spacecraft-standard-constitution,
spacecraft-accessibility-support) — the original stack shipped without
rebuilding them. Full-catalogue drift sweep is clean on this branch.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Lines 412 to 413 in 077a4cd
Sign this commit before publishing it
For this exact reviewed object, git cat-file -p f0295d90a54cfd801b45434005fda0822525d655 contains no gpgsig header, so the commit cannot receive the required Verified status and is blocked from shipping under the repository's non-negotiable signing rule. Recreate this commit with signing enabled before it reaches a controlled remote.
AGENTS.md reference: AGENTS.md:L412-L413
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
|
||
| - `microsoft-rust-guidelines` is `user-invocable: false` on purpose — it is the | ||
| mandatory auto-load Rust base and is hidden from the `/` menu by design. See | ||
| the editing rules in `AGENTS.md`; do not "fix" its absence from the menu. |
There was a problem hiding this comment.
Because this file imports AGENTS.md, Claude receives this user-invocable: false instruction twice: the same rule already appears in AGENTS.md. Future edits can therefore recreate the silent drift that this change is intended to prevent; keep the rule in exactly one file.
AGENTS.md reference: AGENTS.md:L400-L405
Useful? React with 👍 / 👎.
| → /nix/store/<hash>-home-manager-files/.claude/skills/<skill> | ||
| → /nix/store/<hash>-hm_<skill> | ||
| → /spacecraft-software/construct/<skill> |
There was a problem hiding this comment.
Once this formerly local context is tracked and distributed, these lines publish the maintainer host's Nix-store chain and /spacecraft-software/construct checkout location to every clone. Besides misleading agents on hosts where that path does not exist, this directly conflicts with the newly added §5.7 prohibition on personal filesystem paths and host topology; describe the Home Manager relationship generically or move host-specific diagnostics out of the context file.
AGENTS.md reference: AGENTS.md:L47-L52
Useful? React with 👍 / 👎.
| ## Bundling (.zip and .skill) | ||
|
|
||
| Each skill ships as two bundles at the repo root: `<name>.zip` and | ||
| `<name>.skill`. They contain only `SKILL.md`, `LICENSE`, `CREDITS.md`, and | ||
| `references/` (plus `assets/` where present) — never tooling, generator |
There was a problem hiding this comment.
This expansion makes AGENTS.md 461 lines, exceeding the newly shipped authoring reference's 400-line hard cap, largely by duplicating bundling, workflow, and drift-sweep procedures already present in CONTRIBUTING.md. Since agents load this file every session, this adds permanent token overhead and leaves two editable copies of the procedure; retain only the project-specific invariants and link to the existing contributor procedure as the header promises.
AGENTS.md reference: AGENTS.md:L5-L7
Useful? React with 👍 / 👎.
| - The percentage is right-aligned so its `%` sign lands in the same column on every row — two spaces after the closing bracket on the tight-bracket rows, one on the `MVP` row. That difference is exactly what the padded brackets buy. | ||
| - The separator never drops below one space. At 100% the `MVP` row keeps its single space and its percentage therefore sits one column right — the only value at which the two row types do not align, and preferable to a bracket abutting a digit. |
There was a problem hiding this comment.
When any non-MVP row reaches 100%, the mandated two spaces after column 28 followed by 100% puts its percent sign in column 34 rather than the column-33 position used at 0–99%. Thus the stated fixed separator contradicts the immediately preceding requirement that every % align, while the text incorrectly identifies MVP as the only 100% exception; either shrink the tight-row separator for three-digit percentages or document the exception for all row types.
Useful? React with 👍 / 👎.
1 participant
Why this PR exists
#34 was a stacked PR: its base was
feat/agents-md-context-files(#33's branch), notmain. Squash-merging it landed commit21af4f7onto that base branch, and the subsequent branch deletions meant the v1.46/v1.47 content never reachedmain. This PR re-lands the full stack (ebcaa65+a637c57+099c3c5) onmain, merged against today's v1.1.0 diagnostics work.Content
agents-md-authoring.md+ both templates, constitution SKILL.md).construct-cli/AGENTS.md— kept the v1.1.0 CLI-Standard citation (construct-cli: severity-tagged diagnostics — CLI Standard v1.1.0 compliance #36) and the new §5.7 import-based layout.spacecraft-agentic-cli/SKILL.mdauto-merged cleanly, retaining both CLI Standard v1.1.0 — diagnostic severity ladder, machine envelope, unified rendering #35's severity-floor/hint-is-a-field additions and the v1.46 context-file content.agents-md-authoring.mdcarried aStandard v1.46document-version pin — the exact pattern thecheck-skill-refsCI gate rejects (why Sync §5.7 Agent Context Files — Standard v1.46 #33/Sync §17 labelled-row progress block — Standard v1.47 #34 ran unstable). Reworded to cite the section.spacecraft-agentic-cli,spacecraft-standard-constitution,spacecraft-accessibility-support) — the original stack shipped without rebuilding, which was the drift flagged earlier.Verification
Description-cap check,
check-skill-refs(46 skills, no pins), and the full-catalogue bundle drift sweep are all clean on this branch.Closes the recovery loop for #33 (closed as contained in this stack) and #34 (merged to its base branch only).
🤖 Generated with Claude Code