fix: skills reference swamp-troubleshooting for understanding internals

[stack72]

Summary

• All 10 swamp-* skills now reference swamp-troubleshooting in their "When to Use Other Skills" table, so Claude knows to fetch source code when CLI --help doesn't provide enough detail about internals (auth flow, extension push, init process, etc.)
• Broadened swamp-troubleshooting scope beyond just error debugging — it now triggers on understanding signals ("how does", "where is", "internals") and includes a new "When CLI Help Isn't Enough" section with specific areas to investigate
• Moved verbose Model Outputs section from swamp-model/SKILL.md to references/outputs.md to stay under the 500-line skill-creator limit (580 → 495 lines)
• Registered outputs.md in skill_assets.ts so it's included in repo init and repo upgrade

Closes #804

Test plan

• deno check passes on skill_assets.ts
• Verify each modified SKILL.md has the new troubleshooting row in its table
• Verify swamp-troubleshooting/SKILL.md description includes understanding signals
• Verify swamp-model/SKILL.md is under 500 lines
• Verify references/outputs.md contains the moved content
• repo init and repo upgrade copy the new outputs.md file

🤖 Generated with Claude Code

Co-authored-by: Blake Irvin blakeirvin@me.com

[github-actions]

Adversarial Review

This PR is almost entirely documentation restructuring (skill markdown files) with one code change: adding two entries to the BUNDLED_SKILLS array in src/infrastructure/assets/skill_assets.ts.

Critical / High

None.

Medium

None.

Low

1. .claude/skills/swamp-data/SKILL.md:368 — The GC dry-run output shape example was trimmed from two items to one. The removed item showed a different reason value ("lifetime:1h") which demonstrated that multiple reason formats exist. Minor documentation loss — users seeing only "lifetime:ephemeral" might not realize duration-based reasons also appear. Not a code issue.

Verdict

PASS — This is a straightforward documentation refactor: content moved from inline SKILL.md sections to references/ files, cross-skill troubleshooting links added, and the two new reference files registered in skill_assets.ts. The code change is a static array append with no logic. All referenced files exist on disk. No correctness, security, or resource management concerns.

[github-actions]

Review: Approve

No blocking issues found. This is a well-structured documentation improvement.

What was reviewed

• All 13 skill markdown files for consistent troubleshooting cross-references
• New reference files (outputs.md, checks.md) for content accuracy vs originals
• skill_assets.ts registration entries follow existing patterns
• swamp-model/SKILL.md is 495 lines (under the 500-line skill-creator limit)
• swamp-troubleshooting description correctly broadened for understanding signals

Observations

• Content in the extracted reference files is a faithful move — no information lost
• The swamp-data/SKILL.md changes go beyond just adding the troubleshooting row — the rename workflow and GC sections were improved with step-by-step workflows and safety guidance, which is a nice improvement
• All "When to Use Other Skills" tables are consistently formatted across skills
• No code logic changes, no security concerns, no test changes needed

Looks good to merge.