Skip to content

docs: correct stale fleet, CI coverage and skill references; add the merge rule - #14532

Merged
teamleaderleo merged 4 commits into
mainfrom
docaudit-claude-md-accuracy
Sep 25, 2026
Merged

teamleaderleo merged 4 commits into
mainfrom
docaudit-claude-md-accuracy

Conversation

@teamleaderleo

@teamleaderleo teamleaderleo commented Sep 25, 2026 •

Copy link
Copy Markdown
Collaborator

Why

CLAUDE.md (also AGENTS.md) is loaded by every agent session, and the fleet and CI policy moved a lot this week. Several lines now give commands that fail (cmux-ci build cmux without --tag) or numbers that were retired (120/250 GiB). This PR only corrects facts and adds the merge rule agreed in cmuxterm-hq today; cuts and moves follow in a second PR.

Changes

Location Class Evidence Change
CLAUDE.md fleet intro stale scripts/ci/pr_runner_pool.py docstring: owned glaeda-* pools take compile admission, app-host (GUI) shards and side lanes; nightly mini lane removed in #14243 Say owned minis take PR CI jobs with Blacksmith overflow; drop "dev-build-only" and "every GUI test stays on existing lanes"
CLAUDE.md submit recipe stale cmux-ci build cmux --help: "--tag REQUIRED for CMUX"; hq build-fleet/AGENT-BUILDS.md: submitter comes from the personal client token, a differing --submitter is dropped Add --tag, remove SUBMITTER/--submitter
CLAUDE.md disk floors stale hq AGENT-BUILDS.md and hq CLAUDE.md: 80 GiB CMUX, 180 GiB cold Chromium, 120 GiB validated warm Chromium 120/250/200 to 80/180/120
CLAUDE.md standalone reload missing scripts/lib/dev-backend-origin.sh: tagged reload refuses without a cmuxterm-hq-created checkout unless CMUX_DEV_BACKEND_MODE=local Add the env var
CLAUDE.md "Choosing CI coverage" stale / missing choose_ci_suite.py: edited suites run in one changed-suites batch (#14136, #14182), app-source diffs add reached suites (#14418), over-budget edited suites take seven shards; cmuxUITests/ is run by no PR job; ci.yml suite-coverage accepts no-full-ci Describe current routing; mention no-full-ci. Shard-layout rule (#14435) left out: not merged
CLAUDE.md "First pass, then dogfood" missing Agreed in cmuxterm-hq (hq CLAUDE.md "Merging": "Merge fast, not blind"); main_regression_attribution.py (#14436) comments on suspect PRs Add the merge rule, including the merge receipt and merged-unverified label from #14519 (merged)
CLAUDE.md pitfalls, typing paths stale TerminalSurface.forceRefresh now in Packages/macOS/CmuxTerminal/.../TerminalSurface+ForceRefresh.swift Fix location
CLAUDE.md pitfalls, macOS repro hosts stale docs/ci-runners.md "Retired: Tart VM fleet": AWS tart-cmux-aws-m4pro-* hosts removed (#14427) Point at the blacksmith-6vcpu-macos-15 pool
CLAUDE.md pitfalls, localization stale (wording) Localizable.xcstrings carries 20 languages, but scripts/localization_catalog.py LOCALES requires 9 Say the nine are what the checker requires for new strings
skills/cmux-debugging/SKILL.md stale TerminalWindowPortal.swift:133 guard is allowsPortalPointerHitTesting in performHitTest; no isPointerEvent left; forceRefresh moved (above) Fix guard name and path
skills/cmux-debugging/references/runtime-pitfalls.md stale Same as the AWS row Same fix
skills/cmux-architecture SKILL.md + references/package-boundaries.md stale project.pbxproj has no cmux-unit target; cmux-unit is a scheme running cmuxTests, which carries 41 package deps cmux-unit target to cmuxTests
skills/cmux-review/SKILL.md stale CLI/CMUXCLI+Vault.swift: checkpoint --agent <id> --session <id> [--name]; selector throws without them Add the required flags
skills/cmux-cua/SKILL.md stale ComputerUseRuntimeService moved to Packages/macOS/CmuxComputerUse (#13132) Add the package to the code-location list

CLAUDE.md: 29,348 to 30,433 characters (the cut PR follows).

Flagged, not changed: cmux-review says "Use before opening a PR" while CLAUDE.md makes second-model review opt-in only; ListAgents in "Parallel sessions" step 3 is not a tool every Claude Code build has; hq CLAUDE.md still says only dev builds run on the minis, which pr_runner_pool.py contradicts; hq lets scoped runtime fixes merge after a merge directive without re-dogfood, while this file says rebuild the tag and re-notify (left as is); scripts/cmux-debug-cli.sh only looks in DerivedData, not the HQ cache app that publish-hq restores to.

🤖 Generated with Claude Code

teamleaderleo and others added 2 commits September 25, 2026 07:01
…ge rule

- Fleet: owned minis take PR CI jobs via the pool picker; `--tag` is required
  and `--submitter` comes from the client token; disk floors are 80/180/120 GiB.
- Standalone reload needs CMUX_DEV_BACKEND_MODE=local outside an hq checkout.
- CI coverage: reached suites (#14418), no PR job runs cmuxUITests/, no-full-ci.
- Pitfalls: TerminalSurface.forceRefresh moved to CmuxTerminal; AWS Tart hosts
  retired (#14427).
- Merge rule: main is nightly, wait for the judging checks, note what was not
  verified, main-regression comments are fix-forward asks.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…flags, CmuxComputerUse package)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 25, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 5 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used all 10 included reviews currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Repository: manaflow-ai/cmux/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 51fa9b10-1d16-41d9-bc57-624a40074ab2

📥 Commits

Reviewing files that changed from the base of the PR and between 6e0bd98 and ac0d752.

📒 Files selected for processing (7)
  • CLAUDE.md
  • skills/cmux-architecture/SKILL.md
  • skills/cmux-architecture/references/package-boundaries.md
  • skills/cmux-cua/SKILL.md
  • skills/cmux-debugging/SKILL.md
  • skills/cmux-debugging/references/runtime-pitfalls.md
  • skills/cmux-review/SKILL.md

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown
Contributor

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

teamleaderleo and others added 2 commits September 25, 2026 07:09
…tive

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@teamleaderleo

teamleaderleo commented Sep 25, 2026 •

Copy link
Copy Markdown
Collaborator Author

Docs-only diff (CLAUDE.md and skills Markdown): no compile or app-host suite judges it. verify-local.py --affected passed and a review subagent re-checked every table row. Auto-merge is on; it waits only for the required ci-status.

@teamleaderleo
teamleaderleo enabled auto-merge (squash) September 25, 2026 11:31
@teamleaderleo
teamleaderleo merged commit 515b3b1 into main Sep 25, 2026
57 checks passed
@teamleaderleo
teamleaderleo deleted the docaudit-claude-md-accuracy branch September 25, 2026 11:45
@github-actions

Copy link
Copy Markdown
Contributor

Merge receipt for ac0d7523f5: every check was green at merge (12 verified; 14 skipped by policy). Full suite runs on main after merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant