Audit and shrink AGENTS.md (#341) - #376
Merged
Merged
Conversation
obj-p
force-pushed
the
341-agents-md-audit
branch
from
July 11, 2026 12:32
f8d9ebb to
9c9642e
Compare
Cut ~40% (6.9k→4.0k est. tokens), weighting toward build/test/merge workflow and test craft, and dropping content that duplicated README. Fix four stale claims: - SwiftPM: Package.swift only publishes PreviewsSetupKit, so `swift build`/`swift test` are false positives. Replace the "same coverage / swift test fallback" advice (and the stray `.build`/`swift build` mentions) with Bazel build+test+lint reality. - .mcp.json is at the repo root, not a "parent directory". - Move the load-bearing bullets out of "CI-specific concerns" into Test architecture (per-target socket dir #283, bidirectional MCP-ping liveness, MCPTestServer pthread watchdog) and drop the dead CI-runner bullets. Keep the window-server / CGMainDisplayID hint. - Consolidate the twice-described merge gate into one checklist. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
obj-p
force-pushed
the
341-agents-md-audit
branch
from
July 11, 2026 12:38
9c9642e to
d07b48d
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
1 participant
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.
Closes #341.
Shrinks AGENTS.md from ~6.9k to ~4.0k estimated tokens (42% cut,
wc -c / 4), weighting the remaining content toward the build/test/merge workflow and test craft, and dropping the ~half that duplicated README (install, CLI usage,--json, traits, variants, multi-preview, setup plugin, touch injection → now a pointer to README.md).Four stale claims fixed
Package.swiftpublishes onlyPreviewsSetupKit, soswift build/swift testcompile a near-empty package and pass without exercising changed code — a false positive. Replaced the "same coverage /swift testfallback" advice, and swept the strayswift build(daemon-staleness) and.build/...(worktree) mentions, with the real Bazel build+test+lint reality.mcp__previewsmcp__*and the integration-test skill reach).CGMainDisplayID()==0hang hint as a real debugging lever./simplify+/code-reviewper chunk →lint:check→ full local suite (no CI) → classify failures ours-vs-preexisting → stage explicit paths → un-draft → squash-merge.Scope / verification
bazel run //tools/lint:checkclean (exit 0; remaining warnings are pre-existing in untouched source files)..mcp.jsonretained (keep/remove is being arbitrated separately); this PR only fixes its documented location.🤖 Generated with Claude Code