Skip to content

Unify command/skill API to /docflow:<verb> and cut per-session tokens - #2

Merged
MedAdemBHA merged 1 commit into
mainfrom
docflow-usability-pass
Jun 16, 2026
Merged

Unify command/skill API to /docflow:<verb> and cut per-session tokens#2
MedAdemBHA merged 1 commit into
mainfrom
docflow-usability-pass

Conversation

@MedAdemBHA

@MedAdemBHA MedAdemBHA commented Jun 16, 2026

Copy link
Copy Markdown
Owner

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

    • Replaced /docflow:docflow-* and docs-* 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.
    • Merged each command/skill pair into one skill that also triggers from plain English.
    • Lowered SessionStart defaults: INDEX 60→30 lines; changelog 38→20 lines.
    • Shortened skill descriptions to one-line triggers; rewrote README intro with a “start here” and glossary.
    • Updated docs/tests and bumped manifests to 0.3.0; helper script names unchanged.
  • Migration

    • Replace old commands with new names (e.g. /docflow:docflow-check/docflow:check, /docflow:docflow-validate/docflow:validate).
    • Run /reload-plugins after updating the local plugin.
    • Update any docs or automation that reference the old command names; scripts/docflow-*.sh remain the same.

Written for commit efd8481. Summary will update on new commits.

Review in cubic

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>

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Comment thread skills/doctor/SKILL.md
- `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.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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>

@MedAdemBHA
MedAdemBHA merged commit 7a8db18 into main Jun 16, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant