Add step gate checks, crew briefing template, and imperative language#5
Conversation
- Add verification gate after Step 3: confirm sailing orders, squadron, and task assignments before proceeding to execution - Add crew briefing template (admiralty-templates/crew-briefing.md) for dispatching context to teammates who don't inherit the lead's conversation - Strengthen reference directives from "Use/Consult" to "You MUST read" to improve instruction adherence in longer missions Closes Aspegio#3 (items 1, 3, 5) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
harrymunro
left a comment
There was a problem hiding this comment.
Thanks @EmanuelFaria — scope matches exactly what we agreed in #3 and the content is solid. A few things to address before merging:
Crew briefing placement (main issue)
The briefing instruction is currently at the end of Step 2 (Form The Squadron), but the template fields reference information that doesn't exist until Step 3 — task, deliverable, action station tier, file ownership, dependencies. If the admiral follows this instruction during Step 2, they won't have the battle plan information to fill in the briefing.
The crew briefing reference should move to after Step 3 or the start of Step 4, when agents are actually dispatched with their full assignments.
Gate check scope
The issue mentioned "a one-line gate at each major transition" but the PR only adds one after Step 3. That's the most critical transition point so it may be sufficient — but worth considering whether you intended gates elsewhere too.
Minor style nit on template index
The new entry in admiralty-templates.md says "Spawn teammate" which breaks the pattern of the other entries that are step-based ("Issue Sailing Orders", "Draft Battle Plan", etc.). Something like "Brief crew for dispatch" would fit better.
Everything else — imperative language changes, gate check wording, and the crew briefing template itself — looks great. Happy to merge once the placement issue is sorted.
Address review feedback from Aspegio#5: - Crew briefing instruction moved from Step 2 to after Step 3, since briefing fields (task, deliverable, action station, file ownership) reference battle plan information that doesn't exist until Step 3. - Rename "Spawn teammate" to "Brief crew for dispatch" in the template index to match the step-based naming convention. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
EmanuelFaria
left a comment
There was a problem hiding this comment.
Thanks for the thorough review — all three points addressed in 527ee65:
-
Crew briefing placement: Moved from Step 2 to after Step 3 (after the gate check), so all battle plan fields are available when the admiral fills in the briefing.
-
Gate check scope: The Step 3→4 gate is intentionally the only one. It's the critical transition where the admiral shifts from planning to execution — if anything is missing at this point, it cascades into every downstream step. The other transitions are lower-risk: Step 1→2 (sailing orders → squadron) is naturally sequential (you can't form a squadron without orders), and Step 5→6 (action stations → stand down) is enforced by the verification evidence requirements in Step 5. Adding more gates would add friction without proportional value. Happy to revisit if operational experience suggests otherwise.
-
Template index: "Spawn teammate" → "Brief crew for dispatch" ✓
|
Thanks @EmanuelFaria - merged! |
|
This is my first contribution ever! I'm so pleased you found it useful! |
Summary
Implements the three accepted items from #3:
admiralty-templates/crew-briefing.mdfor dispatching context to teammates. Complementary to the ship manifest: the manifest is the admiral's planning view, the briefing is what the teammate receives on spawn. ~500 tokens — enough to work independently, not so much it wastes context.Files changed
.claude/skills/nelson/SKILL.md— Gate check, crew briefing note, imperative phrasing.claude/skills/nelson/references/admiralty-templates.md— Index entry for crew-briefing.md.claude/skills/nelson/references/admiralty-templates/crew-briefing.md— NEW templateTest plan
/nelsonCloses #3
🤖 Generated with Claude Code