Skip to content

docs(skill): cross-reference the fleet skill and surface session children - #1796

Open
tarekrached wants to merge 1 commit into
asheshgoplani:mainfrom
tarekrached:docs/skill-fleet-crossref
Open

docs(skill): cross-reference the fleet skill and surface session children#1796
tarekrached wants to merge 1 commit into
asheshgoplani:mainfrom
tarekrached:docs/skill-fleet-crossref

Conversation

@tarekrached

Copy link
Copy Markdown
Contributor

What problem does this solve?

The main agent-deck skill never mentions the sibling fleet skill or the session children command. An agent that loads only the main skill (the common case — it's the one the plugin triggers on "agent-deck", "session", "sub-agent") has no idea non-blocking child supervision exists, so it re-invents it every time: ad-hoc polling loops over session show / session output, or blocking --wait calls where a fleet peek was the right tool.

Why this change

Docs-only, 6 added lines in skills/agent-deck/SKILL.md:

  1. Add agent-deck session children --json to the Essential Commands table (it's the primary supervision read for any parent session, and the only Essential-Commands-tier command that was missing).
  2. Add a "Fanning out several children?" pointer in the Sub-Agent Launch section directing the multi-child case to the fleet skill.
  3. List the fleet skill in References alongside session-share (which was already cross-referenced).

The alternative — duplicating fleet content into the main skill — would create two copies to keep in sync; a pointer is enough for the agent to load the right skill.

User impact

Agents driving agent-deck stop rediscovering child supervision from --help. No behavior change; no code touched.

Evidence

On main, the gap is verifiable directly:

$ grep -c "fleet\|session children" skills/agent-deck/SKILL.md
0

while the sibling skill and the command both exist (skills/fleet/SKILL.md; cmd/agent-deck/session_children_follow.go).

Real-usage provenance: mining my own Claude Code transcripts (sessions from 2026-07-22, where a conductor-style session supervised agent-deck children) showed the agent working out session children --json from scratch mid-session rather than knowing it from the loaded skill.

Docs-only diff — no test changes; revert-check not applicable.

AI disclosure

  • AI-authored (a model wrote most of it)

Model(s), if AI helped: claude-fable-5

Prompt / session log (optional):

What actually bothered you

My human asked: "I'm surprised how much is being rediscovered every time agent-deck gets invoked by an agent."

Checklist

  • Targeted diff: one problem, no unrelated changes
  • Tests added or updated for new behavior — docs-only change, no behavior to test
  • Test suite passes sandboxed: HOME=$(mktemp -d) XDG_CONFIG_HOME= XDG_DATA_HOME= XDG_CACHE_HOME= go test ./... — not run: docs-only diff, no Go changes (self-check: go-checks skipped)
  • If this touches a hot path (list, status, session output, startup, tmux layer): before/after timing evidence included
  • CHANGELOG.md untouched (entries are added at landing)
  • AI-assisted? Disclosed above, with validation evidence, and I can answer questions about the code
  • "Allow edits from maintainers" is enabled

…dren

The main agent-deck skill never mentions the sibling fleet skill or
`session children`, so an agent that loads only this skill re-invents
child supervision (ad-hoc polling loops over session show/output)
every time it fans out work. Add the command to Essential Commands
and point the Sub-Agent Launch section at the fleet skill for the
multi-child case.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@github-actions github-actions Bot added the intake:clean PR/issue passed the intake contract label Jul 30, 2026
@coderabbitai

coderabbitai Bot commented Jul 30, 2026

Copy link
Copy Markdown

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • skills/agent-deck/SKILL.md is excluded by !**/*.md

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: b4ca107f-64ff-41de-82fa-65c7ec863f8e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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 github-actions Bot added the ai-authored Primarily authored by an AI agent label Jul 30, 2026
@github-actions

Copy link
Copy Markdown

👋 Thanks for the contribution — intake looks complete.

Your PR body carries everything the maintainer's validation pipeline reads first: the problem, the reasoning, the human intent behind it, and an AI-disclosure. It will be applied, built, and tested against main, and you'll get a structured result within about a day. Merges are always human.

gate marker read: ai=authored model=claude-fable-5 intent=yes

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

Labels

ai-authored Primarily authored by an AI agent intake:clean PR/issue passed the intake contract

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant