@open-agent-toolkit/cli 0.0.29 → 0.0.31

package/assets/docs/workflows/projects/reviews.md

@@ -53,8 +53,8 @@ Status progression in `plan.md` Reviews table:
 
 - Critical/Important: address before pass.
 - Medium: address by default; defer only with explicit approval and recorded rationale/disposition.
-- Minor (non-final scopes): auto-deferred by default with rationale; do not block review completion.
-- Minor (final scope): not auto-deferred; require explicit user disposition (defer vs convert), and explain each minor in plain language before asking.
+- Minor (non-final scopes): do not auto-defer by default. Convert when likely future cleanup or when the fix is trivial; defer only with recorded rationale.
+- Minor (final scope): not auto-deferred; require explicit user disposition (defer vs convert), and recommend convert when likely future cleanup or trivial to fix.
 
 ## Auto-review at checkpoints
 

package/assets/public-package-versions.json

@@ -1,6 +1,6 @@
 {
-  "cli": "0.0.29",
-  "docs-config": "0.0.29",
-  "docs-theme": "0.0.29",
-  "docs-transforms": "0.0.29"
+  "cli": "0.0.31",
+  "docs-config": "0.0.31",
+  "docs-theme": "0.0.31",
+  "docs-transforms": "0.0.31"
 }

package/assets/skills/oat-project-review-receive/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: oat-project-review-receive
-version: 1.3.0
+version: 1.4.0
 description: Use when review findings from oat-project-review-provide need closure. Converts review artifacts into actionable plan tasks.
 disable-model-invocation: true
 user-invocable: true
@@ -175,7 +175,7 @@ For each finding, build a structured register entry:
 - For non-final scopes:
   - Mark the review as `passed` in the plan.md Reviews table (if plan.md exists)
   - No fix tasks are added
-  - Minor findings may be auto-deferred by default (Step 9)
+  - Minor findings still follow the Step 9 recommendation/disposition rules before routing onward
   - Route user to normal next action
 - For `final` scope:
   - Do not mark `passed` until both gates are complete:
@@ -502,9 +502,13 @@ If any Medium is proposed for deferral:
 Minor findings handling is scope-aware:
 
 - If `scope != final`:
-  - Minor findings are auto-deferred by default.
-  - Record them in implementation.md under "Deferred Findings".
-  - Do not block review completion on minor disposition.
+  - Minor findings are not auto-deferred just because they are non-functional.
+  - Default to converting a Minor finding to a task when either of these is true:
+    - it is likely future cleanup (better than even chance that the team will need to address it later), or
+    - the fix scope is `Negligible` or `Minor`.
+  - Only propose deferral when the finding is genuinely low-probability cleanup, blocked by another change, duplicated elsewhere, explicitly out of scope, or fixing now would create disproportionate churn/risk.
+  - If deferred, record rationale in implementation.md under "Deferred Findings".
+  - Do not block review completion on minor disposition once each finding has been converted or explicitly deferred with rationale.
 
 - If `scope == final`:
   - Minor findings are NOT auto-deferred silently.
@@ -512,6 +516,9 @@ Minor findings handling is scope-aware:
     - what the issue is,
     - potential user/maintainer impact,
     - why fixing now vs deferring is reasonable.
+  - Recommendation default:
+    - recommend `convert` when the finding is likely future cleanup or the fix scope is `Negligible`/`Minor`;
+    - recommend `defer` only when the finding is unlikely to matter soon, blocked, duplicated, or high-churn to address now.
   - Keep explanations concise (1-3 sentences per minor) and include file/line when available.
   - Ask user explicitly:
 

package/assets/skills/oat-review-receive/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: oat-review-receive
-version: 1.2.0
+version: 1.3.0
 description: Use when processing review findings outside project context. Converts local review artifacts into actionable task lists.
 disable-model-invocation: true
 user-invocable: true
@@ -42,6 +42,7 @@ If you catch yourself:
 - Editing project lifecycle docs in ad-hoc mode -> STOP and revert to task-list output only.
 - Triaging without presenting a findings overview first -> STOP and show overview before disposition prompts.
 - Skipping Medium finding rationale when proposing deferral -> STOP and collect explicit rationale.
+- Auto-deferring Minor findings without checking future cleanup likelihood or fix cost -> STOP and re-evaluate the recommendation.
 
 **Recovery:**
 
@@ -181,11 +182,12 @@ Default suggestions:
 - Critical -> `convert`
 - Important -> `convert`
 - Medium -> `convert` (propose `defer` only with concrete rationale)
-- Minor -> `defer`
+- Minor -> `convert` when future cleanup is likely or the fix is trivial; otherwise `defer` with concrete rationale
 
 Rules:
 
 - Require explicit rationale for `defer` or `dismiss`.
+- Do not recommend `defer` for a Minor finding solely because it does not impact current functionality. If there is a better-than-even chance the team will need to clean it up later, or the fix is `Negligible`/`Minor`, recommend `convert`.
 - Do not silently skip findings.
 
 ### Step 5: Generate Task List Output

package/dist/app/create-program.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"create-program.d.ts","sourceRoot":"","sources":["../../src/app/create-program.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,OAAO,EAAU,MAAM,WAAW,CAAC;AAQ5C,wBAAgB,aAAa,IAAI,OAAO,CAavC"}
+{"version":3,"file":"create-program.d.ts","sourceRoot":"","sources":["../../src/app/create-program.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,OAAO,EAAU,MAAM,WAAW,CAAC;AAO5C,wBAAgB,aAAa,IAAI,OAAO,CAavC"}

package/dist/app/create-program.js

@@ -1,13 +1,13 @@
+import { OAT_VERSION } from '../shared/oat-version.js';
 import { Command, Option } from 'commander';
 const PROGRAM_NAME = 'oat';
 const PROGRAM_DESCRIPTION = 'Open Agent Toolkit CLI for provider interoperability';
-const PROGRAM_VERSION = '0.0.24';
 const SCOPE_CHOICES = ['project', 'user', 'all'];
 export function createProgram() {
     return new Command()
         .name(PROGRAM_NAME)
         .description(PROGRAM_DESCRIPTION)
-        .version(PROGRAM_VERSION)
+        .version(OAT_VERSION)
         .option('--json', 'Output a single JSON document')
         .option('--verbose', 'Enable verbose debug output')
         .addOption(new Option('--scope <scope>', 'Limit execution scope')

package/dist/commands/docs/init/scaffold.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"scaffold.d.ts","sourceRoot":"","sources":["../../../../src/commands/docs/init/scaffold.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,sBAAsB,EAAE,MAAM,oBAAoB,CAAC;AAGjE,OAAO,KAAK,EAGV,uBAAuB,EAExB,MAAM,mBAAmB,CAAC;AA+E3B,MAAM,WAAW,sBAAuB,SAAQ,uBAAuB;IACrE,UAAU,EAAE,MAAM,CAAC;CACpB;AAED,MAAM,WAAW,qBAAqB;IACpC,OAAO,EAAE,MAAM,CAAC;IAChB,YAAY,EAAE,MAAM,EAAE,CAAC;IACvB,mBAAmB,EAAE,sBAAsB,CAAC;CAC7C;AAED,MAAM,WAAW,aAAa;IAC5B,SAAS,EAAE,OAAO,CAAC;IACnB,aAAa,EAAE,GAAG,CAAC,MAAM,CAAC,CAAC;IAC3B,kBAAkB,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;CAC5C;AAWD,wBAAsB,eAAe,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC,CAOxE;AAgDD,wBAAsB,oBAAoB,CACxC,QAAQ,EAAE,MAAM,EAChB,UAAU,EAAE,MAAM,GACjB,OAAO,CAAC,aAAa,CAAC,CAaxB;AA4ID,wBAAsB,eAAe,CACnC,OAAO,EAAE,sBAAsB,GAC9B,OAAO,CAAC,qBAAqB,CAAC,CAoChC"}
+{"version":3,"file":"scaffold.d.ts","sourceRoot":"","sources":["../../../../src/commands/docs/init/scaffold.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,sBAAsB,EAAE,MAAM,oBAAoB,CAAC;AAIjE,OAAO,KAAK,EAGV,uBAAuB,EAExB,MAAM,mBAAmB,CAAC;AA+E3B,MAAM,WAAW,sBAAuB,SAAQ,uBAAuB;IACrE,UAAU,EAAE,MAAM,CAAC;CACpB;AAED,MAAM,WAAW,qBAAqB;IACpC,OAAO,EAAE,MAAM,CAAC;IAChB,YAAY,EAAE,MAAM,EAAE,CAAC;IACvB,mBAAmB,EAAE,sBAAsB,CAAC;CAC7C;AAED,MAAM,WAAW,aAAa;IAC5B,SAAS,EAAE,OAAO,CAAC;IACnB,aAAa,EAAE,GAAG,CAAC,MAAM,CAAC,CAAC;IAC3B,kBAAkB,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;CAC5C;AAWD,wBAAsB,eAAe,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC,CAOxE;AAgDD,wBAAsB,oBAAoB,CACxC,QAAQ,EAAE,MAAM,EAChB,UAAU,EAAE,MAAM,GACjB,OAAO,CAAC,aAAa,CAAC,CAaxB;AA4ID,wBAAsB,eAAe,CACnC,OAAO,EAAE,sBAAsB,GAC9B,OAAO,CAAC,qBAAqB,CAAC,CAoChC"}

package/dist/commands/docs/init/scaffold.js

@@ -1,6 +1,7 @@
 import { readdir, readFile, writeFile } from 'node:fs/promises';
 import { basename, dirname, join } from 'node:path';
 import { dirExists, ensureDir, fileExists } from '../../../fs/io.js';
+import { OAT_VERSION } from '../../../shared/oat-version.js';
 import { getTemplateDir } from './resolve-options.js';
 const MKDOCS_TEMPLATE_FILES = [
     { source: '.gitignore', destination: '.gitignore' },
@@ -71,7 +72,7 @@ const OAT_DEP_PACKAGES = [
     'docs-theme',
     'docs-transforms',
 ];
-const DEFAULT_OAT_PUBLISHED_VERSION = '0.0.24';
+const DEFAULT_OAT_PUBLISHED_VERSION = OAT_VERSION;
 const PUBLIC_PACKAGE_VERSIONS_FILE = 'public-package-versions.json';
 export async function detectIsOatRepo(repoRoot) {
     for (const pkg of OAT_DEP_PACKAGES) {
@@ -81,6 +82,16 @@ export async function detectIsOatRepo(repoRoot) {
     }
     return true;
 }
+async function readBundledOatPackageVersions(assetsRoot) {
+    try {
+        const content = await readFile(join(assetsRoot, PUBLIC_PACKAGE_VERSIONS_FILE), 'utf8');
+        const parsed = JSON.parse(content);
+        return Object.fromEntries(OAT_DEP_PACKAGES.flatMap((name) => typeof parsed[name] === 'string' ? [[name, parsed[name]]] : []));
+    }
+    catch {
+        return {};
+    }
+}
 async function readCliVersion(assetsRoot) {
     const cliRoot = dirname(assetsRoot);
     try {
@@ -92,16 +103,6 @@ async function readCliVersion(assetsRoot) {
         return DEFAULT_OAT_PUBLISHED_VERSION;
     }
 }
-async function readBundledOatPackageVersions(assetsRoot) {
-    try {
-        const content = await readFile(join(assetsRoot, PUBLIC_PACKAGE_VERSIONS_FILE), 'utf8');
-        const parsed = JSON.parse(content);
-        return Object.fromEntries(OAT_DEP_PACKAGES.flatMap((name) => typeof parsed[name] === 'string' ? [[name, parsed[name]]] : []));
-    }
-    catch {
-        return {};
-    }
-}
 function buildFallbackOatPackageVersions(version) {
     return Object.fromEntries(OAT_DEP_PACKAGES.map((name) => [name, version]));
 }

package/dist/index.d.ts

@@ -1,4 +1,5 @@
 #!/usr/bin/env node
 export declare function normalizeArgv(argv: string[]): string[];
 export declare function main(argv?: string[]): Promise<void>;
+export declare function isEntrypoint(argv?: string[], entrypointUrl?: string): boolean;
 //# sourceMappingURL=index.d.ts.map

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":";AAUA,wBAAgB,aAAa,CAAC,IAAI,EAAE,MAAM,EAAE,GAAG,MAAM,EAAE,CAatD;AAED,wBAAsB,IAAI,CAAC,IAAI,GAAE,MAAM,EAAiB,GAAG,OAAO,CAAC,IAAI,CAAC,CAIvE"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":";AAWA,wBAAgB,aAAa,CAAC,IAAI,EAAE,MAAM,EAAE,GAAG,MAAM,EAAE,CAatD;AAED,wBAAsB,IAAI,CAAC,IAAI,GAAE,MAAM,EAAiB,GAAG,OAAO,CAAC,IAAI,CAAC,CAIvE;AAED,wBAAgB,YAAY,CAC1B,IAAI,GAAE,MAAM,EAAiB,EAC7B,aAAa,GAAE,MAAwB,GACtC,OAAO,CAaT"}

package/dist/index.js

@@ -1,6 +1,7 @@
 #!/usr/bin/env node
+import { realpathSync } from 'node:fs';
 import { resolve } from 'node:path';
-import { pathToFileURL } from 'node:url';
+import { fileURLToPath, pathToFileURL } from 'node:url';
 import { createProgram } from './app/create-program.js';
 import { registerCommands } from './commands/index.js';
 import { CliError } from './errors/index.js';
@@ -23,11 +24,18 @@ export async function main(argv = process.argv) {
     registerCommands(program);
     await program.parseAsync(normalizeArgv(argv));
 }
-function isEntrypoint() {
-    if (!process.argv[1]) {
+export function isEntrypoint(argv = process.argv, entrypointUrl = import.meta.url) {
+    if (!argv[1]) {
         return false;
     }
-    return import.meta.url === pathToFileURL(resolve(process.argv[1])).href;
+    const entrypointPath = fileURLToPath(entrypointUrl);
+    const argvPath = resolve(argv[1]);
+    try {
+        return realpathSync(entrypointPath) === realpathSync(argvPath);
+    }
+    catch {
+        return entrypointUrl === pathToFileURL(argvPath).href;
+    }
 }
 if (isEntrypoint()) {
     void main().catch((error) => {

package/dist/manifest/manager.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"manager.d.ts","sourceRoot":"","sources":["../../src/manifest/manager.ts"],"names":[],"mappings":"AAMA,OAAO,EACL,KAAK,QAAQ,EACb,KAAK,aAAa,EAEnB,MAAM,kBAAkB,CAAC;AAe1B,wBAAgB,mBAAmB,IAAI,QAAQ,CAO9C;AAED,wBAAsB,YAAY,CAAC,YAAY,EAAE,MAAM,GAAG,OAAO,CAAC,QAAQ,CAAC,CA6C1E;AAED,wBAAsB,YAAY,CAChC,YAAY,EAAE,MAAM,EACpB,QAAQ,EAAE,QAAQ,GACjB,OAAO,CAAC,IAAI,CAAC,CAQf;AAED,wBAAgB,SAAS,CACvB,QAAQ,EAAE,QAAQ,EAClB,aAAa,EAAE,MAAM,EACrB,QAAQ,EAAE,MAAM,GACf,aAAa,GAAG,SAAS,CAK3B;AAED,wBAAgB,QAAQ,CAAC,QAAQ,EAAE,QAAQ,EAAE,KAAK,EAAE,aAAa,GAAG,QAAQ,CAc3E;AAED,wBAAgB,WAAW,CACzB,QAAQ,EAAE,QAAQ,EAClB,aAAa,EAAE,MAAM,EACrB,QAAQ,EAAE,MAAM,GACf,QAAQ,CAeV"}
+{"version":3,"file":"manager.d.ts","sourceRoot":"","sources":["../../src/manifest/manager.ts"],"names":[],"mappings":"AAOA,OAAO,EACL,KAAK,QAAQ,EACb,KAAK,aAAa,EAEnB,MAAM,kBAAkB,CAAC;AAa1B,wBAAgB,mBAAmB,IAAI,QAAQ,CAO9C;AAED,wBAAsB,YAAY,CAAC,YAAY,EAAE,MAAM,GAAG,OAAO,CAAC,QAAQ,CAAC,CA6C1E;AAED,wBAAsB,YAAY,CAChC,YAAY,EAAE,MAAM,EACpB,QAAQ,EAAE,QAAQ,GACjB,OAAO,CAAC,IAAI,CAAC,CAQf;AAED,wBAAgB,SAAS,CACvB,QAAQ,EAAE,QAAQ,EAClB,aAAa,EAAE,MAAM,EACrB,QAAQ,EAAE,MAAM,GACf,aAAa,GAAG,SAAS,CAK3B;AAED,wBAAgB,QAAQ,CAAC,QAAQ,EAAE,QAAQ,EAAE,KAAK,EAAE,aAAa,GAAG,QAAQ,CAc3E;AAED,wBAAgB,WAAW,CACzB,QAAQ,EAAE,QAAQ,EAClB,aAAa,EAAE,MAAM,EACrB,QAAQ,EAAE,MAAM,GACf,QAAQ,CAeV"}

package/dist/manifest/manager.js

@@ -2,8 +2,8 @@ import { randomUUID } from 'node:crypto';
 import { mkdir, readFile, rename, writeFile } from 'node:fs/promises';
 import { dirname } from 'node:path';
 import { CliError } from '../errors/index.js';
+import { OAT_VERSION } from '../shared/oat-version.js';
 import { ManifestSchema, } from './manifest.types.js';
-const OAT_VERSION = '0.0.24';
 function formatIssuePath(path) {
     if (path.length === 0) {
         return '(root)';

package/dist/shared/oat-version.d.ts

@@ -0,0 +1,2 @@
+export declare const OAT_VERSION: string;
+//# sourceMappingURL=oat-version.d.ts.map

package/dist/shared/oat-version.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"oat-version.d.ts","sourceRoot":"","sources":["../../src/shared/oat-version.ts"],"names":[],"mappings":"AAUA,eAAO,MAAM,WAAW,QAAiC,CAAC"}

package/dist/shared/oat-version.js

@@ -0,0 +1,3 @@
+import { readFileSync } from 'node:fs';
+const packageJson = JSON.parse(readFileSync(new URL('../../package.json', import.meta.url), 'utf8'));
+export const OAT_VERSION = packageJson.version ?? '0.0.0';

package/package.json

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