chore: remove all deprecated skill references#22
Conversation
Clean up remaining "deprecated" annotations and stale references to deleted skills (plan, work, plan-review, impl-review, epic-review). CLAUDE.md, docs/skills.md, docs/CODEBASE_MAP.md, agents/worker.md now reference only flow-code-run as the primary entry point. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Removes documentation/agent references to deprecated phase-specific flow-code skills, standardizing guidance around the unified /flow-code:run pipeline.
Changes:
- Updates
docs/skills.mdto list only the current core skills and removes legacy per-phase command guidance. - Updates
docs/CODEBASE_MAP.mdto remove deprecated skill directory entries and revise lifecycle diagrams to reflect/flow-code:run. - Updates
CLAUDE.mdandagents/worker.mdto describe the run-based workflow and worker spawning model.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| docs/skills.md | Removes deprecated core skill rows and legacy usage flow; emphasizes /flow-code:run. |
| docs/CODEBASE_MAP.md | Replaces deprecated skill references and updates run-based lifecycle documentation/diagrams. |
| CLAUDE.md | Updates “Skills” and “Primary Workflow” sections to center /flow-code:run. |
| agents/worker.md | Updates worker agent description to indicate it’s spawned by flow-code-run. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -110,13 +110,9 @@ flow-code/ | |||
| │ ├── setup.md, sync.md, uninstall.md | |||
| ├── skills/ # 18 skill implementations | |||
There was a problem hiding this comment.
The directory-structure snippet still claims there are 18 skill implementations, but the repository's skills/ directory currently contains more (e.g., flow-code-* extensions plus browser/). Please update the count and/or expand this list to match the actual directories (or regenerate this CODEBASE_MAP if it is meant to be auto-generated).
| ├── skills/ # 18 skill implementations | |
| ├── skills/ # Skill implementations (excerpt shown; repository currently contains additional entries) |
| Worker->>Review: impl-review (if enabled) | ||
| Review-->>Worker: SHIP / NEEDS_WORK (fix loop) | ||
| Worker->>flowctl: done (summary + evidence) | ||
| User->>Run: /flow-code:run "description" |
There was a problem hiding this comment.
In the Mermaid sequence diagram, the note says the preferred invocation is /flow-code:run fn-N, but the next line shows /flow-code:run "description" while subsequent flowctl calls assume an existing epic id fn-N. Please make these examples consistent (either demonstrate starting from a description or resuming an existing epic id).
| User->>Run: /flow-code:run "description" | |
| User->>Run: /flow-code:run fn-N |
| ``` | ||
|
|
||
| **Skills**: 9 core + 22 extensions. See `docs/skills.md` for the full classification. Core workflow: `flow-code-run` (unified phase loop). Legacy individual skills (plan, plan-review, work, impl-review, epic-review) are deprecated but still functional as thin redirects. | ||
| **Skills**: See `docs/skills.md` for the full classification. Core workflow: `flow-code-run` (unified phase loop via `flowctl phase next/done`). |
There was a problem hiding this comment.
Although this section positions /flow-code:run as the core workflow, later in CLAUDE.md there are still explicit legacy references to /flow-code:work (e.g. "/flow-code:run (or legacy /flow-code:work)" and resume guidance suggesting /flow-code:work <id>). If the PR goal is to remove deprecated references entirely, those later mentions should be updated to /flow-code:run too.
Clean up remaining deprecated annotations in CLAUDE.md, docs/skills.md, docs/CODEBASE_MAP.md, agents/worker.md. Zero deprecated remnants remain.
🤖 Generated with Claude Code