Skip to content

docs: define shared fleet capacity - #10312

Merged
azooz2003-bit merged 2 commits into
mainfrom
feat-agent-messaging-policy
Aug 18, 2026
Merged

azooz2003-bit merged 2 commits into
mainfrom
feat-agent-messaging-policy

Conversation

@azooz2003-bit

@azooz2003-bit azooz2003-bit commented Aug 18, 2026 •

Copy link
Copy Markdown
Collaborator

Document the shared Mac fleet policy for cmux agents:

  • Every healthy Mac fleet slot is general-purpose for builds, archives, tests, profiling, simulator and UI verification, and other resource-intensive work.
  • Agents should use shared lease state and slot-isolated paths instead of waiting for an AWS-only builder.

AGENTS.md is the repository symlink to CLAUDE.md. Verification: git diff --check; wc -c CLAUDE.md.

Summary by CodeRabbit

  • Documentation
    • Clarified that healthy shared Mac fleet slots support resource-intensive workloads.
    • Specified that capacity decisions should be based on shared lease state rather than workload labels or builder assumptions.

@coderabbitai

coderabbitai Bot commented Aug 18, 2026 •

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 96db0289-f15d-4143-9a1d-8564255dbb9d

📥 Commits

Reviewing files that changed from the base of the PR and between 5255262 and 6e0d983.

📒 Files selected for processing (1)
  • CLAUDE.md
💤 Files with no reviewable changes (1)
  • CLAUDE.md

Included review availability: Your plan provides up to 10 included reviews per hour; 8 remain after this review.


📝 Walkthrough

Walkthrough

CLAUDE.md adds guidance for shared Mac fleet capacity. Healthy fleet slots support resource-intensive workloads. Capacity checks must use shared lease state and fleet-provided isolated paths.

Changes

Shared Mac fleet capacity guidance

Layer / File(s) Summary
Shared capacity guidance
CLAUDE.md
Documents general-purpose Mac fleet slot usage, supported workloads, shared lease state, and fleet-provided isolated paths.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 6e0d9

This is a localized documentation-only policy update with no actionable merge-blocking risk remaining after normal checks and review.

🚥 Pre-merge checks | ✅ 24 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description covers the fleet policy and verification, but omits the no-messaging rule and most required template sections. Add the required Summary, Testing, Demo Video, Review Trigger, and Checklist sections, and document the no-iMessage or SMS rule.
✅ Passed checks (24 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Cmux Swift Actor Isolation ✅ Passed The PR diff changes only CLAUDE.md and contains no Swift paths or Swift code, so it introduces no Swift 6 actor isolation issue.
Cmux Swift Blocking Runtime ✅ Passed The diff changes only CLAUDE.md documentation and modifies no Swift or production runtime code, so it introduces no blocking or timing-based synchronization.
Cmux Browser Automation Off-Main ✅ Passed The PR net diff changes only CLAUDE.md; no browser socket implementation, routing, WebKit/AppKit access, or policy tests changed.
Cmux Expensive Synchronous Load ✅ Passed The diff changes only CLAUDE.md, deleting four documentation lines; it adds no production Swift code or synchronous agent-history load.
Cmux Cache Substitution Correctness ✅ Passed The PR changes only CLAUDE.md documentation; the diff contains no Swift, TypeScript, or JavaScript production changes and no cache substitution.
Cmux No Hacky Sleeps ✅ Passed The parent-to-HEAD diff changes only four lines in CLAUDE.md, with no TypeScript, JavaScript, shell, or runtime changes and no timing primitives.
Cmux Algorithmic Complexity ✅ Passed The PR diff against its base changes only CLAUDE.md documentation; it introduces no Swift, TypeScript, JavaScript, shell, or runtime algorithmic behavior.
Cmux Swift Concurrency ✅ Passed The commit changes only CLAUDE.md and no Swift or source-like files; it introduces no legacy Swift async pattern covered by the concurrency rule.
Cmux Swift @Concurrent ✅ Passed The pull request diff changes only CLAUDE.md and contains no Swift files, Swift declarations, or call-site changes to assess under the @concurrent rule.
Cmux Swift Package Boundaries ✅ Passed The PR changes only CLAUDE.md and introduces no production Swift or SwiftPM changes, so the package-boundary check is not applicable.
Cmux Swiftpm Lockfiles ✅ Passed The PR range changes only CLAUDE.md. No Package.swift, Package.resolved, .gitignore, workflow, or Xcode project paths changed, so the SwiftPM lockfile conditions do not apply.
Cmux Swift Logging ✅ Passed The HEAD-parent diff changes only CLAUDE.md documentation and contains no Swift paths or logging statements; the Swift logging check is not triggered.
Cmux User-Facing Error Privacy ✅ Passed The PR changes only CLAUDE.md with four documentation lines; no user-facing error, alert, command output, or API error body is added, and the rule explicitly allows docs.
Cmux Full Internationalization ✅ Passed The PR changes only CLAUDE.md operational agent guidance; no production user-facing text, Swift localization, catalog, or web locale data is introduced.
Cmux Swiftui State Layout ✅ Passed The introduced diff changes only CLAUDE.md documentation and contains no Swift, SwiftUI, state, layout, or render-time behavior changes.
Cmux Architecture Rethink ✅ Passed The commit changes only CLAUDE.md and contains no Swift or Swift architecture changes, so the custom check is inapplicable.
Cmux Swift Auxiliary Window Close Shortcuts ✅ Passed The PR diff changes only CLAUDE.md and adds no Swift or standalone window code, so the auxiliary-window close-shortcut rule is not applicable.
Cmux Source Artifacts ✅ Passed The cumulative diff changes only CLAUDE.md, adding hand-written fleet guidance; no local, generated, cached, temporary, or build artifacts enter source control.
Cmux No Test Or Debug Seam In Production Source ✅ Passed The PR range changes only CLAUDE.md (+4 lines); it changes no Swift file under a production Sources/ path, so this check's failure conditions do not apply.
Cmux No Ambient Global State ✅ Passed The PR range changes only CLAUDE.md (+4 lines); it contains no Swift or production code changes, so the ambient-global-state rule is not applicable.
Title check ✅ Passed The title clearly summarizes the main documentation change about shared fleet capacity.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat-agent-messaging-policy

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.

@azooz2003-bit azooz2003-bit changed the title docs: define shared fleet and messaging policy docs: define shared fleet capacity Aug 18, 2026
@azooz2003-bit
azooz2003-bit merged commit 029d652 into main Aug 18, 2026
7 checks passed
@azooz2003-bit
azooz2003-bit deleted the feat-agent-messaging-policy branch August 18, 2026 19:34
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