Unify command/skill API to /docflow:<verb> and cut per-session tokens - #2
Merged
Conversation
Dedup the six command/skill pairs (doctor, check, init, adopt, repair, validate) into single skills that work both as typed commands and natural-language triggers, and rename every entry to a bare verb under the docflow: namespace. Drops the /docflow:docflow- stutter and the parallel docs-* vocabulary; scan/feature-plan/product-spec stay as commands. Trim every skill description to a one-line trigger and lower SessionStart hook injection defaults (INDEX 60->30, changelog 38->20 lines) to cut per-session token cost. Rewrite the README intro for non-developers (plain-language overview, "start here" path, jargon glossary). Helper script names and doctor's internal recommendation= status tokens are unchanged. Update in-tool printed commands, tests (substring asserts), dogfood docs, and bump manifests 0.2.4 -> 0.3.0 (breaking public API). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
1 issue found across 39 files
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="skills/doctor/SKILL.md">
<violation number="1" location="skills/doctor/SKILL.md:23">
P2: `validation: fail` is listed as a recommendation token in 'Interpret the recommendation', but the doctor script never produces it as a recommendation — it's just a status reported under 'Detected'. Agents routing on recommendation tokens will never see this value.</violation>
</file>
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
| - `docflow-init` = no meaningful docs found; scaffolding is safe → run `/docflow:init`. | ||
| - `docflow-adopt` = docs/README already exist; preserve them → run `/docflow:adopt`. | ||
| - `docflow-repair` = docflow exists; regenerate map/check links → run `/docflow:repair`. | ||
| - `validation: fail` = docs have blockers; run `/docflow:validate` for details. |
There was a problem hiding this comment.
P2: validation: fail is listed as a recommendation token in 'Interpret the recommendation', but the doctor script never produces it as a recommendation — it's just a status reported under 'Detected'. Agents routing on recommendation tokens will never see this value.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At skills/doctor/SKILL.md, line 23:
<comment>`validation: fail` is listed as a recommendation token in 'Interpret the recommendation', but the doctor script never produces it as a recommendation — it's just a status reported under 'Detected'. Agents routing on recommendation tokens will never see this value.</comment>
<file context>
@@ -0,0 +1,33 @@
+- `docflow-init` = no meaningful docs found; scaffolding is safe → run `/docflow:init`.
+- `docflow-adopt` = docs/README already exist; preserve them → run `/docflow:adopt`.
+- `docflow-repair` = docflow exists; regenerate map/check links → run `/docflow:repair`.
+- `validation: fail` = docs have blockers; run `/docflow:validate` for details.
+
+## Report
</file context>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Dedup the six command/skill pairs (doctor, check, init, adopt, repair, validate) into single skills that work both as typed commands and natural-language triggers, and rename every entry to a bare verb under the docflow: namespace. Drops the /docflow:docflow- stutter and the parallel docs-* vocabulary; scan/feature-plan/product-spec stay as commands.
Trim every skill description to a one-line trigger and lower SessionStart hook injection defaults (INDEX 60->30, changelog 38->20 lines) to cut per-session token cost. Rewrite the README intro for non-developers (plain-language overview, "start here" path, jargon glossary).
Helper script names and doctor's internal recommendation= status tokens are unchanged. Update in-tool printed commands, tests (substring asserts), dogfood docs, and bump manifests 0.2.4 -> 0.3.0 (breaking public API).
Summary by cubic
Unified the command/skill API under
/docflow:<verb>and cut per-session tokens for faster, cheaper sessions. This is a breaking rename with a simpler vocabulary and a clearer README.Refactors
/docflow:docflow-*anddocs-*with/docflow:<verb>(e.g./docflow:doctor,/docflow:check,/docflow:init,/docflow:adopt,/docflow:repair,/docflow:validate); kept/docflow:scan,/docflow:feature-plan,/docflow:product-spec.INDEX60→30 lines; changelog 38→20 lines.0.3.0; helper script names unchanged.Migration
/docflow:docflow-check→/docflow:check,/docflow:docflow-validate→/docflow:validate)./reload-pluginsafter updating the local plugin.scripts/docflow-*.shremain the same.Written for commit efd8481. Summary will update on new commits.