@open-agent-toolkit/cli 0.1.52 → 0.1.53

package/assets/skills/oat-repo-improve/references/plan-template.md

@@ -0,0 +1,199 @@
+# Handoff Plan Template
+
+Every plan is written for an executor model that has **zero context**: it has not seen the advisor session, the audit, the other plans, or any prior conversation. It may be a smaller/cheaper model. Assume it is competent at following explicit instructions and weak at filling gaps, recovering from ambiguity, or knowing when to stop.
+
+Three properties make a plan executable by a weaker model:
+
+1. **Self-contained context** — everything needed is in the file: paths, code excerpts, conventions, commands.
+2. **Verification gates** — every step ends with a command and its expected result. The executor never has to _judge_ whether it succeeded.
+3. **Hard boundaries and escape hatches** — explicit out-of-scope list, and "STOP and report" conditions instead of letting the model improvise when reality doesn't match the plan.
+
+File naming: `plans/NNN-short-slug.md`, numbered in recommended execution order.
+
+---
+
+## Template
+
+```markdown
+# Plan NNN: <Imperative title — what will be true after this plan>
+
+> **Executor instructions**: Follow this plan step by step. Run every
+> verification command and confirm the expected result before moving to the
+> next step. If anything in the "STOP conditions" section occurs, stop and
+> report — do not improvise. When done, update the status row for this plan
+> in `plans/README.md` — unless a reviewer dispatched you and told you they
+> maintain the index.
+>
+> **Drift check (run first)**: `git diff --stat <planned-at SHA>..HEAD -- <in-scope paths>`
+> If any in-scope file changed since this plan was written, compare the
+> "Current state" excerpts against the live code before proceeding; on a
+> mismatch, treat it as a STOP condition.
+
+## Status
+
+- **Priority**: P1 | P2 | P3
+- **Effort**: S | M | L
+- **Risk**: LOW | MED | HIGH
+- **Depends on**: plans/NNN-\*.md (or "none")
+- **Category**: bug | security | perf | tests | tech-debt | migration | dx | docs | direction
+- **Planned at**: commit `<short SHA>`, <YYYY-MM-DD>
+- **Issue**: <GitHub issue URL — only when published via `--issues`; omit otherwise>
+
+## Why this matters
+
+2–5 sentences. The problem, its concrete cost, and what improves when this
+lands. Written so the executor (and a human reviewer) understands the intent —
+intent is what lets a correct judgment call happen when a detail is off.
+
+## Current state
+
+The facts the executor needs, inlined — never "as discussed" or "see audit":
+
+- The relevant files, each with one line on its role:
+  - `src/orders/api.ts` — order-list endpoint; contains the N+1 (lines 130–160)
+- Excerpts of the code as it exists today (short, with `file:line` markers),
+  enough that the executor can confirm it's looking at the right thing.
+- The repo conventions that apply here, with a pointer to one exemplar file:
+  "Error handling follows the Result pattern — see `src/lib/result.ts` and its
+  use in `src/users/api.ts:40-60`. Match it."
+- Any documented vocabulary or design constraints the plan must honor, inlined
+  from the intent/design docs found in recon: the relevant `CONTEXT.md` terms
+  the executor should use in names and comments, the `DESIGN.md` tokens/components
+  to reuse, or the ADR whose decision this work must stay consistent with. Quote
+  the specific lines — the executor has not read those docs.
+
+## Commands you will need
+
+| Purpose   | Command                 | Expected on success |
+| --------- | ----------------------- | ------------------- |
+| Install   | `pnpm install`          | exit 0              |
+| Typecheck | `pnpm typecheck`        | exit 0, no errors   |
+| Tests     | `pnpm test -- <filter>` | all pass            |
+| Lint      | `pnpm lint`             | exit 0              |
+
+(Exact commands from this repo — verified during recon, not guessed.)
+
+## Suggested executor toolkit
+
+(Optional — include only when relevant skills/tools plausibly exist in the
+executor's environment. Skip the section otherwise.)
+
+- Skills the executor should invoke if available, and for what:
+  "use `vercel-react-best-practices` when writing the memoization in step 3".
+- Reference docs worth reading before starting, by path or URL.
+
+## Scope
+
+**In scope** (the only files you should modify):
+
+- `src/orders/api.ts`
+- `src/orders/api.test.ts` (create)
+
+**Out of scope** (do NOT touch, even though they look related):
+
+- `src/orders/legacy-api.ts` — deprecated path, scheduled for deletion;
+  changing it wastes effort and risks the v1 clients still pinned to it.
+- Any change to the public response shape — clients depend on it.
+
+## Git workflow
+
+(Filled from recon — match the repo's observed conventions.)
+
+- Branch: `advisor/NNN-<slug>` (or the repo's branch-naming convention if one is evident)
+- Commit per step or per logical unit; message style: <match repo, e.g. conventional commits — include an example from `git log`>
+- Do NOT push or open a PR unless the operator instructed it.
+
+## Steps
+
+### Step 1: <imperative title>
+
+What to do, precisely. Reference exact files/symbols. Include the target code
+shape when it's load-bearing (the pattern to produce, not necessarily every
+line).
+
+**Verify**: `<command>` → <expected output>
+
+### Step 2: ...
+
+(Each step small enough to verify independently. Order steps so the codebase
+is never broken between steps when possible — e.g. add new path, switch
+callers, then remove old path.)
+
+## Test plan
+
+- New tests to write, in which file, covering which cases (list them:
+  happy path, the specific bug/regression this plan fixes, named edge cases).
+- Which existing test to use as the structural pattern:
+  "model after `src/users/api.test.ts`".
+- Verification: `<test command>` → all pass, including N new tests.
+
+## Done criteria
+
+Machine-checkable. ALL must hold:
+
+- [ ] `pnpm typecheck` exits 0
+- [ ] `pnpm test` exits 0; new tests for <X> exist and pass
+- [ ] `grep -rn "<old pattern>" src/` returns no matches
+- [ ] No files outside the in-scope list are modified (`git status`)
+- [ ] `plans/README.md` status row updated
+
+## STOP conditions
+
+Stop and report back (do not improvise) if:
+
+- The code at the locations in "Current state" doesn't match the excerpts
+  (the codebase has drifted since this plan was written).
+- A step's verification fails twice after a reasonable fix attempt.
+- The fix appears to require touching an out-of-scope file.
+- You discover the assumption "<key assumption>" is false.
+
+## Maintenance notes
+
+For the human/agent who owns this code after the change lands:
+
+- What future changes will interact with this (e.g. "if pagination is added
+  to this endpoint, the batching in step 2 must be revisited").
+- What a reviewer should scrutinize in the PR.
+- Any follow-up explicitly deferred out of this plan (and why).
+```
+
+---
+
+## Index file: `plans/README.md`
+
+Written once by the advisor after all plans, updated by executors:
+
+```markdown
+# Implementation Plans
+
+Generated by the oat-repo-improve skill on <date>. Execute in the order below unless
+dependencies say otherwise. Each executor: read the plan fully before starting,
+honor its STOP conditions, and update your row when done.
+
+## Execution order & status
+
+| Plan | Title | Priority | Effort | Depends on | Status |
+| ---- | ----- | -------- | ------ | ---------- | ------ |
+| 001  | ...   | P1       | S      | —          | TODO   |
+| 002  | ...   | P1       | M      | 001        | TODO   |
+
+Status values: TODO | IN PROGRESS | DONE | BLOCKED (with one-line reason) | REJECTED (with one-line rationale — finding fixed independently or approach abandoned)
+
+## Dependency notes
+
+- 002 requires 001 because <reason>.
+
+## Findings considered and rejected
+
+- <finding>: not worth doing because <one line>. (So nobody re-audits it.)
+```
+
+## Quality bar — check before finishing each plan
+
+- Could a model that has never seen this repo execute this with only the plan file and the repo? If any step requires knowledge from the advisor session, inline that knowledge.
+- Is every verification a command with an expected result, not a judgment ("make sure it works")?
+- Does every step name exact files and symbols, not "the relevant module"?
+- Are the STOP conditions specific to this plan's actual risks, not boilerplate?
+- Would a reviewer reading only "Why this matters" + "Done criteria" understand what they're approving?
+- No secret values anywhere in the file — locations and credential types only.
+- "Planned at" SHA is filled in and the in-scope paths in the drift check match the Scope section.

package/dist/commands/init/tools/shared/skill-manifest.d.ts

@@ -11,7 +11,7 @@ export interface PackMetadata {
 }
 export declare const PACK_METADATA: Record<string, PackMetadata>;
 export declare function resolvePackDefaultScope(packName: string): 'user' | 'project';
-export declare const WORKFLOW_SKILLS: readonly ["oat-project-capture", "oat-project-clear-active", "oat-project-complete", "oat-project-design", "oat-project-discover", "oat-project-document", "oat-project-implement", "oat-project-import-plan", "oat-project-new", "oat-project-next", "oat-project-open", "oat-project-plan", "oat-project-plan-writing", "oat-project-pr-final", "oat-project-pr-progress", "oat-project-progress", "oat-project-promote-spec-driven", "oat-project-quick-start", "oat-project-reconcile", "oat-project-revise", "oat-project-review-provide", "oat-project-review-provide-remote", "oat-project-review-receive", "oat-project-review-receive-remote", "oat-project-spec", "oat-project-split", "oat-project-summary", "oat-repo-knowledge-index", "oat-worktree-bootstrap", "oat-worktree-bootstrap-auto", "oat-wrap-up"];
+export declare const WORKFLOW_SKILLS: readonly ["oat-project-capture", "oat-project-clear-active", "oat-project-complete", "oat-project-design", "oat-project-dispatch-subagents", "oat-project-discover", "oat-project-document", "oat-project-implement", "oat-project-import-plan", "oat-project-new", "oat-project-next", "oat-project-open", "oat-project-plan", "oat-project-plan-writing", "oat-project-pr-final", "oat-project-pr-progress", "oat-project-progress", "oat-project-promote-spec-driven", "oat-project-quick-start", "oat-project-reconcile", "oat-project-revise", "oat-project-review-provide", "oat-project-review-provide-remote", "oat-project-review-receive", "oat-project-review-receive-remote", "oat-project-spec", "oat-project-split", "oat-project-summary", "oat-repo-knowledge-index", "oat-worktree-bootstrap", "oat-worktree-bootstrap-auto", "oat-wrap-up"];
 export declare const WORKFLOW_AGENTS: readonly ["oat-codebase-mapper.md", "oat-phase-implementer.md", "oat-reviewer.md"];
 export declare const WORKFLOW_TEMPLATES: readonly ["state.md", "discovery.md", "spec.md", "design.md", "plan.md", "implementation.md", "summary.md"];
 export declare const WORKFLOW_SCRIPTS: readonly ["generate-oat-state.sh", "generate-thin-index.sh", "resolve-tracking.sh"];
@@ -19,7 +19,7 @@ export declare const IDEA_SKILLS: readonly ["oat-idea-new", "oat-idea-ideate", "
 export declare const CORE_SKILLS: readonly ["oat-docs", "oat-doctor"];
 export declare const DOCS_SKILLS: readonly ["authoring-docs", "oat-agent-instructions-analyze", "oat-agent-instructions-apply", "oat-docs-analyze", "oat-docs-apply", "oat-docs-authoring", "oat-docs-bootstrap"];
 export declare const DOCS_SCRIPTS: readonly ["resolve-tracking.sh"];
-export declare const UTILITY_SKILLS: readonly ["create-agnostic-skill", "oat-repo-maintainability-review", "oat-review-provide", "oat-review-provide-remote", "oat-review-receive", "oat-review-receive-remote"];
+export declare const UTILITY_SKILLS: readonly ["create-agnostic-skill", "oat-dispatch-subagents", "oat-repo-improve", "oat-repo-maintainability-review", "oat-review-provide", "oat-review-provide-remote", "oat-review-receive", "oat-review-receive-remote"];
 export declare const PROJECT_MANAGEMENT_SKILLS: readonly ["oat-pjm-add-backlog-item", "oat-pjm-decision", "oat-pjm-update-repo-reference", "oat-pjm-review-backlog"];
 export declare const PROJECT_MANAGEMENT_TEMPLATES: readonly ["backlog-item.md", "roadmap.md", "current-state.md", "decision.md", "repo-agents.md", "pjm-agents.md", "reference-agents.md"];
 export declare const PROJECT_MANAGEMENT_SCRIPTS: readonly [];

package/dist/commands/init/tools/shared/skill-manifest.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"skill-manifest.d.ts","sourceRoot":"","sources":["../../../../../src/commands/init/tools/shared/skill-manifest.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAgBH,MAAM,WAAW,YAAY;IAC3B,IAAI,EAAE,MAAM,CAAC;IACb,YAAY,EAAE,MAAM,GAAG,SAAS,CAAC;CAClC;AAED,eAAO,MAAM,aAAa,EAAE,MAAM,CAAC,MAAM,EAAE,YAAY,CAKtD,CAAC;AAEF,wBAAgB,uBAAuB,CAAC,QAAQ,EAAE,MAAM,GAAG,MAAM,GAAG,SAAS,CAE5E;AAID,eAAO,MAAM,eAAe,6xBAgClB,CAAC;AAEX,eAAO,MAAM,eAAe,oFAIlB,CAAC;AAEX,eAAO,MAAM,kBAAkB,6GAQrB,CAAC;AAEX,eAAO,MAAM,gBAAgB,qFAInB,CAAC;AAIX,eAAO,MAAM,WAAW,2FAKd,CAAC;AAIX,eAAO,MAAM,WAAW,qCAAsC,CAAC;AAI/D,eAAO,MAAM,WAAW,iLAQd,CAAC;AAEX,eAAO,MAAM,YAAY,kCAAmC,CAAC;AAI7D,eAAO,MAAM,cAAc,6KAOjB,CAAC;AAIX,eAAO,MAAM,yBAAyB,sHAK5B,CAAC;AAEX,eAAO,MAAM,4BAA4B,yIAQ/B,CAAC;AAEX,eAAO,MAAM,0BAA0B,aAAc,CAAC;AAItD,eAAO,MAAM,iBAAiB,6BAA8B,CAAC;AAI7D,eAAO,MAAM,eAAe,2EAMlB,CAAC;AAEX,eAAO,MAAM,eAAe,qCAAsC,CAAC"}
+{"version":3,"file":"skill-manifest.d.ts","sourceRoot":"","sources":["../../../../../src/commands/init/tools/shared/skill-manifest.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAgBH,MAAM,WAAW,YAAY;IAC3B,IAAI,EAAE,MAAM,CAAC;IACb,YAAY,EAAE,MAAM,GAAG,SAAS,CAAC;CAClC;AAED,eAAO,MAAM,aAAa,EAAE,MAAM,CAAC,MAAM,EAAE,YAAY,CAKtD,CAAC;AAEF,wBAAgB,uBAAuB,CAAC,QAAQ,EAAE,MAAM,GAAG,MAAM,GAAG,SAAS,CAE5E;AAID,eAAO,MAAM,eAAe,+zBAiClB,CAAC;AAEX,eAAO,MAAM,eAAe,oFAIlB,CAAC;AAEX,eAAO,MAAM,kBAAkB,6GAQrB,CAAC;AAEX,eAAO,MAAM,gBAAgB,qFAInB,CAAC;AAIX,eAAO,MAAM,WAAW,2FAKd,CAAC;AAIX,eAAO,MAAM,WAAW,qCAAsC,CAAC;AAI/D,eAAO,MAAM,WAAW,iLAQd,CAAC;AAEX,eAAO,MAAM,YAAY,kCAAmC,CAAC;AAI7D,eAAO,MAAM,cAAc,2NASjB,CAAC;AAIX,eAAO,MAAM,yBAAyB,sHAK5B,CAAC;AAEX,eAAO,MAAM,4BAA4B,yIAQ/B,CAAC;AAEX,eAAO,MAAM,0BAA0B,aAAc,CAAC;AAItD,eAAO,MAAM,iBAAiB,6BAA8B,CAAC;AAI7D,eAAO,MAAM,eAAe,2EAMlB,CAAC;AAEX,eAAO,MAAM,eAAe,qCAAsC,CAAC"}

package/dist/commands/init/tools/shared/skill-manifest.js

@@ -20,6 +20,7 @@ export const WORKFLOW_SKILLS = [
     'oat-project-clear-active',
     'oat-project-complete',
     'oat-project-design',
+    'oat-project-dispatch-subagents',
     'oat-project-discover',
     'oat-project-document',
     'oat-project-implement',
@@ -90,6 +91,8 @@ export const DOCS_SCRIPTS = ['resolve-tracking.sh'];
 // ── Utility pack ───────────────────────────────────────────────────
 export const UTILITY_SKILLS = [
     'create-agnostic-skill',
+    'oat-dispatch-subagents',
+    'oat-repo-improve',
     'oat-repo-maintainability-review',
     'oat-review-provide',
     'oat-review-provide-remote',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@open-agent-toolkit/cli",
-  "version": "0.1.52",
+  "version": "0.1.53",
   "private": false,
   "description": "Open Agent Toolkit CLI",
   "homepage": "https://github.com/voxmedia/open-agent-toolkit/tree/main/packages/cli",
@@ -34,7 +34,7 @@
     "ora": "^9.0.0",
     "yaml": "2.8.2",
     "zod": "^3.25.76",
-    "@open-agent-toolkit/control-plane": "0.1.52"
+    "@open-agent-toolkit/control-plane": "0.1.53"
   },
   "devDependencies": {
     "@types/node": "^22.10.0",