docs(agents): trim section 2 layout tree to a thin pointer surface - #1844
Open
notno wants to merge 2 commits into
Open
docs(agents): trim section 2 layout tree to a thin pointer surface#1844notno wants to merge 2 commits into
notno wants to merge 2 commits into
Conversation
3 tasks
Replace the exhaustive annotated layout tree with a compact top-level summary plus explicit owner pointers (docs/configuration.md for layout and config schemas, producing script headers for artifact fields). Normative tree annotations without another owner survive as section 2 prose: the state/ dot-file never-edit rule, the procevent supervision-required presence rule, the symlink edit-the-original rule, and the read-script-headers-first rule. AGENTS.md drops from 63,377 to 53,563 chars.
notno
force-pushed
the
fm/agentsmd-trim-a18
branch
from
August 7, 2026 03:13
25269cb to
d0a699b
Compare
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.
Intent
Trim AGENTS.md section 2's annotated layout tree to a thin pointer surface, per the firstmate-coding-guidelines skill and AGENTS.md's own "Maintaining this file" rules.
Goal and scope:
Accepted decision (supersedes the original stretch target):
Safety-boundary sweep (required in the PR description):
Constraints honored:
What Changed
config/knob,data/file, andstate/artifact with per-line explanations) with a compact top-level directory summary plus explicit pointers:docs/configuration.mdfor layout entries andconfig/knob semantics, and each producing script's header for its owndata/andstate/artifacts. AGENTS.md shrinks from 63,377 to 53,563 chars; this still exceeds Claude Code's ~40k large-memory warning threshold, and further reduction is deliberately deferred to its own considered effort rather than a passing cut here.state/dot-file never-edit rule (watcher, wake-queue, auto-arm, away-mode, and sub-supervisor internals), the rule that a registeredstate/procevent/source keeps supervision required by its presence alone, the read-each-bin/-script-header-before-first-use rule, and a one-line reinforcement of theCLAUDE.md/.claude/skillssymlink edit-the-originals rule (CONTRIBUTING.md remains the owner). Every other normative tree line was verified as already covered by kept section 2 prose, another AGENTS.md section (e.g..envpresence-gating in section 14,skills/not loaded in section 12, the gitignored captain-private list in section 1), or the owning doc or script header.CLAUDE.mdremains a symlink so only AGENTS.md is edited. The Test gate confirmed the exact size delta, the presence of all four relocated rules, that all pointer targets exist, and thatbin/fm-doc-audience-check.shpasses on the result.Risk Assessment
✅ Low: Docs-only trim of one AGENTS.md section that matches the authoritative intent exactly: the final size (53,563 chars) is as accepted, all four relocated normative rules appear in the new prose, and every dropped annotation's claimed owner (other AGENTS.md sections, docs/configuration.md, or the producing script headers) was verified to exist and cover it.
Testing
Verified the section-2 trim end-to-end against the intent: exact before/after char counts (63,377 → 53,563), a single-hunk section-2-only diff with symlinks untouched, all four relocated safety rules present in the new prose, the sweep's coverage claims confirmed in their owning sections, every new pointer target existing on disk, no em/en dashes in added lines, and the intent-named bin/fm-doc-audience-check.sh acceptance check passing; the change is a plain-Markdown agent-context surface, so the evidence is a CLI transcript of the new section rather than a screenshot.
Evidence: Section-2 trim validation transcript (sizes, audience check, new section 2 surface)
base 345de4e: 63377 chars → head 224b13e: 53563 chars (matches intent; still above ~40k floor per accepted decision) fm-doc-audience-check: ok surfaces=65 local_links=205 (exit 0) Added lines: 0 em/en dashes; CLAUDE.md and .claude/skills remain symlinks; only AGENTS.md changed (1 file, +12/-67) All four relocated rules present: state/ dot-file never-edit, procevent presence-keeps-supervision, read-bin-headers-first, edit-the-originals symlink rulePipeline
Updates from git push no-mistakes
✅ **intent** - passed
✅ No issues found.
✅ **Rebase** - passed
✅ No issues found.
✅ **Review** - passed
✅ No issues found.
✅ **Test** - passed
✅ No issues found.
git diff --stat 345de4e..224b13e— confirmed only AGENTS.md changed (12 insertions, 67 deletions, one hunk in section 2)wc -c AGENTS.mdvsgit show 345de4e:AGENTS.md | wc -c— verified exact 63,377 → 53,563 char reduction stated in the intentbin/fm-doc-audience-check.sh— the acceptance check named in the intent; passed withok surfaces=65 local_links=205, exit 0Manual diff review confirming all four deliberately relocated normative rules appear in section 2 prose: state/ dot-file never-edit rule, procevent presence-keeps-supervision rule, read-bin-script-headers-first rule, and the CLAUDE.md/.claude-skills edit-the-originals lineSpot-checked the safety-sweep coverage claims:.envpresence-gating (section 14 line 468),skills/not loaded by firstmate (section 12 line 440), captain-private gitignored list (section 1 line 43), status-lines-as-wake-events and captain.md-canonical prose retained in section 2ls -ld CLAUDE.md .claude/skills— confirmed both remain symlinks to the edited originals;ls docs/configuration.md bin/fm-doc-audience-check.sh bin/fm-crew-state.sh— all pointer targets existgit diff 345de4e..224b13e | grep '^+' | grep -P '\x{2014}|\x{2013}'— no em/en dashes in any added lineGrep for orphaned cross-references to the removed annotated tree — none foundgit status --porcelain— worktree left clean, no transient test artifacts✅ **Document** - passed
✅ No issues found.
✅ **Push** - passed
✅ No issues found.