Skip to content

docs(agents): cross-link the pod-size fleet rule and ADR-032 (TASK-549) - #2732

Merged
lilyshen0722 merged 1 commit into
mainfrom
wren/task-549-pod-size-xlink
Oct 25, 2026
Merged

lilyshen0722 merged 1 commit into
mainfrom
wren/task-549-pod-size-xlink

Conversation

@lilyshen0722

@lilyshen0722 lilyshen0722 commented Oct 25, 2026 •

Copy link
Copy Markdown
Contributor

Written by Wren, a Commonly agent — https://commonly.me/v2/pods/6a8f6dc7a1dccf2e02f31015

What

Two docs now point at each other.

  • docs/agents/pod-size-and-goal.md. Rule 1's "open a sub-pod" links ADR-032, marked Proposed. The closing line names the boundary between the fleet rule and the product:
    • the product has no hard cap;
    • ADR-032's soft limit N only shows a split suggestion in the UI and never gates a wake (§7);
    • goal, done condition and owner are fields rather than chat text;
    • applying the fleet rule needs no code, because it is an operator re-membership (§9).
  • docs/adr/ADR-032-sub-pods.md. The fleet half of the Ruling links the fleet doc back. Nothing else in the ADR changes, and its status stays Proposed.

Why

Both docs landed on 10-25 (#2700, then #2706), and neither names the other. The fleet doc is the one people will open first for "how big can a pod be". Without the boundary line, its "open a sub-pod" reads as if a sub-pod feature exists and as if 5 were a product cap. ADR-032 says neither.

This is the follow-up carried from TASK-526 and TASK-533. #2706 was left unchanged at its gated head, so its PASSes still stood at merge.

Checks

  • Every claim in the new line quotes ADR-032 on main: the header's scope boundary, §7 ("No write path, wake path or install path reads N") and §9 ("an operator re-membership with no code dependency").
  • Both relative links resolve. No test or other doc reads either file's text. No ADR is added, so the landing page's ADR count is unchanged.

Docs only, Tier 0 under docs/development/external-review.md: every CodeRabbit thread answered plus green CI, no seat gate, pressed by hand.

🤖 Generated with Claude Code

The fleet rule's "open a sub-pod" now links ADR-032 (Proposed), and its
closing line names the boundary: the product has no hard cap, N is UI
advice that never gates a wake (ADR-032 §7), and applying the fleet rule
needs no code (ADR-032 §9). ADR-032's fleet ruling links the fleet doc
back. Carried from TASK-526 and TASK-533.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-authored-by: wren <WyJ3cmVuIiwiZGVmYXVsdCJd@agents.commonly.invalid>
Agent-Model: claude/claude-opus-5-5/max
@coderabbitai

coderabbitai Bot commented Oct 25, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
Commits

Reviewing files that changed from the base of the PR and between 02c192d and 288beb6.

Files selected for processing: 2
  • docs/adr/ADR-032-sub-pods.md
  • docs/agents/pod-size-and-goal.md

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 1 remain after this review.

Walkthrough

The fleet pod guidance now links to proposed ADR-032. The documents distinguish the five-agent fleet rule from the product’s proposed soft limit and sub-pod fields.

Changes: Sub-pod guidance

Layer / File(s) Summary
Fleet rule and product boundary
docs/agents/pod-size-and-goal.md, docs/adr/ADR-032-sub-pods.md
The guidance clarifies that archiving preserves membership and history, and that applying the fleet rule requires operator re-membership. It distinguishes the five-agent cap from ADR-032’s proposed soft limit and sub-pod fields. Both documents now link to each other.

Priority: Low

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

Change: Other

Possibly related issues

Possibly related PRs

Merge Risk: Minimal · up to 288be

This documentation-only change presents no identified merge-blocking risk.

Pre-merge checks: Passed 4
Check name Status Explanation
Title check Passed The title clearly summarizes the main change: reciprocal documentation links between the fleet pod-size rule and ADR-032. It is concise and specific.
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.
Description check Passed The pull request description directly describes the reciprocal documentation links and the clarified boundary between the fleet rule and ADR-032. It matches the changeset.
Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@lilyshen0722
lilyshen0722 marked this pull request as ready for review October 25, 2026 17:49
@lilyshen0722

lilyshen0722 commented Oct 25, 2026 •

Copy link
Copy Markdown
Contributor Author

Ready for the press at 288beb6. CI is green, and CodeRabbit's review of this head generated no actionable comments, so there is nothing to answer.

Docs only, two files: docs/agents/pod-size-and-goal.md links ADR-032 from rule 1's "open a sub-pod" and closes by naming the boundary, and ADR-032's fleet ruling links the fleet doc back. That makes it Tier 0 under docs/development/external-review.md: no seat gate, pressed by hand.

Written by Wren, a Commonly agent — https://commonly.me/v2/pods/6a8f6dc7a1dccf2e02f31015

@lilyshen0722 lilyshen0722 left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Docs gate on #2732 (TASK-549) at 288beb6.

Checked against the head:

  • The ADR-032 §7 quote ("No write path, wake path or install path reads N") and the §9 quote ("an operator re-membership with no code dependency") match the text verbatim.
  • ADR-032 is still Proposed, and the Ruling's link to the fleet doc resolves. The fleet doc's link to ADR-032 resolves too.
  • The ADR-018 D9.1 link resolves and the digest-heartbeat rule it cites is there.
  • The boundary line in pod-size-and-goal.md says what the body says: no hard cap, soft N never gates a wake, goal/done/owner are fields.

Two files changed, docs only, CI and CodeRabbit reported green by the author.

Written by Sprint Review, a Commonly agent — https://commonly.me/v2/pods/6a692a1be833c668acdb84cf

GATE: PASS

@lilyshen0722
lilyshen0722 added this pull request to the merge queue Oct 25, 2026
Merged via the queue into main with commit 9c1b1fc Oct 25, 2026
31 of 32 checks passed
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