@open-agent-toolkit/cli 0.1.24 → 0.1.26

package/assets/docs/reference/cli-reference.md

@@ -58,7 +58,7 @@ Notable commands introduced in the current CLI surface:
 - `oat project status --field <path>` - print one arbitrary dot-path field from the same status payload, e.g. `project.workflowMode` or `project.timestamps.stateUpdated`. Missing/null fields print `null`; object and array fields print compact JSON.
 - `oat project status --project-path <path>` - read from a repo-relative or absolute project path instead of `.oat/config.local.json`'s active project pointer. Combine it with `--field` or `--shell` when a skill has already resolved the target project path.
 - `oat project status --shell NAME=path ...` - print shell-safe assignments for one or more fields from one status read, e.g. `WORKFLOW_MODE='quick'`. This is the preferred multi-field read API for skills. See [Writing Skills → Reading project state](../contributing/skills.md#reading-project-state) for examples and the `npx`-backed `oat` shim contract.
-- `oat review latest --json` - find the newest review artifact by `oat_generated_at`, scanning the active or specified project's `reviews/` and `reviews/archived/` directories plus ad-hoc review locations. Same-time candidates use target priority, then lifecycle recency (`final` > higher phase/task > lower phase/task). The JSON contract returns `path`, `scope`, `generatedAt`, and `kind` (`project` or `adhoc`), with `null` values when no review exists.
+- `oat review latest --json` - find the newest review artifact by `oat_generated_at`, scanning the active or specified project's `reviews/` and `reviews/archived/` directories plus ad-hoc review locations. Same-time candidates use target priority, then lifecycle recency (`final` > higher phase/task > lower phase/task). The JSON contract returns `path`, `scope`, `generatedAt`, `kind` (`project` or `adhoc`), `archived`, and `actionable`, with `null` values when no review exists. Archived project reviews remain discoverable as history but return `actionable: false`.
 - `oat project list --json` - summary state for tracked projects under the configured projects root
 - `oat project complete-state <project-path>` - apply the canonical completed-state mutation to a project's `state.md`; used by `oat-project-complete` during lifecycle closeout
 - `oat project archive [project-path]` - archive a tracked project through the same local move, summary export, and optional S3 upload path used by completion. When omitted, the project path falls back to the active project.

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

@@ -28,7 +28,7 @@ oat review latest --json
 oat review latest --project .oat/projects/shared/example --json
 ```
 
-The resolver orders candidates by `oat_generated_at` frontmatter, not filesystem mtime. With an active or explicit project, it scans the project's `reviews/` directory first, then `reviews/archived/`, then ad-hoc review locations (`.oat/repo/reviews/` and `.oat/projects/local/orphan-reviews/`). When candidates share the same generated time, active project reviews outrank archived and ad-hoc reviews, then lifecycle recency breaks remaining ties (`final` > higher phase/task scope > lower phase/task scope). The JSON response contains `path`, `scope`, `generatedAt`, and `kind` (`project` or `adhoc`). If no review exists, those fields are `null`.
+The resolver orders candidates by `oat_generated_at` frontmatter, not filesystem mtime. With an active or explicit project, it scans the project's `reviews/` directory first, then `reviews/archived/`, then ad-hoc review locations (`.oat/repo/reviews/` and `.oat/projects/local/orphan-reviews/`). When candidates share the same generated time, active project reviews outrank archived and ad-hoc reviews, then lifecycle recency breaks remaining ties (`final` > higher phase/task scope > lower phase/task scope). The JSON response contains `path`, `scope`, `generatedAt`, `kind` (`project` or `adhoc`), `archived`, and `actionable`. Archived project reviews remain discoverable as history but return `actionable: false`; active top-level project reviews return `actionable: true`. If no review exists, those fields are `null`.
 
 `oat-project-review-receive` uses this resolver when it is invoked from natural language and needs to offer the latest project review, or route an ad-hoc result to `oat-review-receive`.
 

package/assets/public-package-versions.json

@@ -1,6 +1,6 @@
 {
-  "cli": "0.1.24",
-  "docs-config": "0.1.24",
-  "docs-theme": "0.1.24",
-  "docs-transforms": "0.1.24"
+  "cli": "0.1.26",
+  "docs-config": "0.1.26",
+  "docs-theme": "0.1.26",
+  "docs-transforms": "0.1.26"
 }

package/assets/skills/oat-project-next/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: oat-project-next
-version: 1.0.4
+version: 1.0.5
 description: Use when continuing work on the active OAT project. Reads project state, determines the next lifecycle action, and invokes the appropriate skill automatically.
 disable-model-invocation: true
 user-invocable: true
@@ -283,9 +283,9 @@ Parse plan.md Reviews table for a row with `Scope="final"` and `Type="code"`:
   → Route to `oat-project-review-provide` (with scope hint: "code final")
   → Announce: "Review fixes implemented — triggering re-review"
 
-- If row exists with other non-passed status (e.g., `"received"`, `"fixes_added"`):
-  → Route to `oat-project-review-receive` (review findings not yet converted to tasks)
-  → Announce: "Final review in progress — continuing review cycle"
+- If row exists with other non-passed status (e.g., `"received"`, `"fixes_added"`) and Step 5.2 did not find an active top-level review artifact:
+  → Route to `oat-project-review-provide` (with scope hint: "code final")
+  → Announce: "Final review is not passed and no active review artifact exists — rerunning final review"
 
 - If row exists with `Status="passed"`:
   → Continue to 5.4

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

@@ -1,6 +1,6 @@
 ---
 name: oat-project-review-receive
-version: 1.5.2
+version: 1.5.3
 description: Use when the user explicitly asks to receive review findings for an OAT project — e.g. "receive review", "process review", "process the project review", or confirms a previously offered review-receive step. Do NOT auto-invoke merely because a review file exists. Resolves the latest review and offers before acting.
 disable-model-invocation: false
 user-invocable: true
@@ -105,7 +105,8 @@ Selection rules:
 - Use `oat review latest` as the first-choice resolver. It scans project reviews (`reviews/` and `reviews/archived/`) plus ad-hoc review locations and orders candidates by `oat_generated_at` frontmatter rather than filesystem mtime.
 - Read the JSON result:
   - `path: null` means no review target was found.
-  - `kind: "project"` means this skill can process the target when the path is an active top-level project review.
+  - `kind: "project"` and `actionable: true` means this skill can process the target when the path is an active top-level project review.
+  - `archived: true` or `actionable: false` means the target is historical/non-actionable for project review-receive.
   - `kind: "adhoc"` means route to `oat-review-receive` after offering that handoff to the user.
 - Only process active project review artifacts in the top level of `"$PROJECT_PATH/reviews/"`.
 - Treat archived project artifacts as history only; do not receive them automatically. If `oat review latest` returns an archived project review, tell the user no active project review is waiting and offer to run `oat-project-review-provide`.

package/dist/commands/local/expand.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"expand.d.ts","sourceRoot":"","sources":["../../../src/commands/local/expand.ts"],"names":[],"mappings":"AAuEA,wBAAgB,aAAa,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO,CAEnD;AAED,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,GAAG,OAAO,CAEzE;AAED,MAAM,WAAW,YAAY;IAC3B,QAAQ,EAAE,MAAM,EAAE,CAAC;IACnB,YAAY,EAAE,MAAM,EAAE,CAAC;CACxB;AAED,wBAAsB,gBAAgB,CACpC,IAAI,EAAE,MAAM,EACZ,UAAU,EAAE,MAAM,EAAE,GACnB,OAAO,CAAC,YAAY,CAAC,CA0BvB"}
+{"version":3,"file":"expand.d.ts","sourceRoot":"","sources":["../../../src/commands/local/expand.ts"],"names":[],"mappings":"AAqHA,wBAAgB,aAAa,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO,CAEnD;AAED,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,GAAG,OAAO,CAEzE;AAED,MAAM,WAAW,YAAY;IAC3B,QAAQ,EAAE,MAAM,EAAE,CAAC;IACnB,YAAY,EAAE,MAAM,EAAE,CAAC;CACxB;AAED,wBAAsB,gBAAgB,CACpC,IAAI,EAAE,MAAM,EACZ,UAAU,EAAE,MAAM,EAAE,GACnB,OAAO,CAAC,YAAY,CAAC,CA4BvB"}

package/dist/commands/local/expand.js

@@ -1,6 +1,31 @@
 import { readdir } from 'node:fs/promises';
 import { join } from 'node:path';
 const GLOB_CHARS = /[*?[]/;
+const PRUNED_CANDIDATE_DIRECTORIES = new Set([
+    '.cache',
+    '.git',
+    '.gradle',
+    '.next',
+    '.nuxt',
+    '.output',
+    '.parcel-cache',
+    '.pnpm-store',
+    '.pytest_cache',
+    '.ruff_cache',
+    '.turbo',
+    '.venv',
+    '.vite',
+    '.worktrees',
+    '.yarn',
+    '__pycache__',
+    'build',
+    'coverage',
+    'dist',
+    'node_modules',
+    'out',
+    'target',
+    'tmp',
+]);
 function normalizePattern(path) {
     return path.replaceAll('\\', '/').replace(/^\.\//, '').replace(/\/+$/, '');
 }
@@ -40,15 +65,30 @@ function globToRegExp(pattern) {
     regex += '$';
     return new RegExp(regex);
 }
+function isOatStatePath(path) {
+    return path === '.oat' || path.startsWith('.oat/');
+}
+function shouldPruneCandidateDirectory(path, name) {
+    return !isOatStatePath(path) && PRUNED_CANDIDATE_DIRECTORIES.has(name);
+}
 async function collectRelativePaths(root, current = '') {
-    const dirPath = current === '' ? root : join(root, current);
-    const entries = await readdir(dirPath, { withFileTypes: true });
     const paths = [];
-    for (const entry of entries) {
-        const relativePath = current === '' ? entry.name : `${current}/${entry.name}`;
-        paths.push(relativePath);
-        if (entry.isDirectory()) {
-            paths.push(...(await collectRelativePaths(root, relativePath)));
+    const pendingDirectories = [current];
+    while (pendingDirectories.length > 0) {
+        const directory = pendingDirectories.pop();
+        const dirPath = directory === '' ? root : join(root, directory);
+        const entries = await readdir(dirPath, { withFileTypes: true });
+        for (const entry of entries) {
+            const relativePath = directory === '' ? entry.name : `${directory}/${entry.name}`;
+            const shouldPrune = entry.isDirectory() &&
+                shouldPruneCandidateDirectory(relativePath, entry.name);
+            if (shouldPrune) {
+                continue;
+            }
+            paths.push(relativePath);
+            if (entry.isDirectory()) {
+                pendingDirectories.push(relativePath);
+            }
         }
     }
     return paths;
@@ -76,7 +116,9 @@ export async function expandLocalPaths(root, localPaths) {
             missingGlobs.push(localPath);
         }
         else {
-            resolved.push(...matches.sort());
+            for (const match of matches.sort()) {
+                resolved.push(match);
+            }
         }
     }
     return { resolved, missingGlobs };

package/dist/commands/review/latest.d.ts

@@ -7,6 +7,8 @@ export interface LatestReview {
     scope: string;
     generatedAt: string;
     kind: ReviewLatestKind;
+    archived: boolean;
+    actionable: boolean;
 }
 export interface FindLatestReviewOptions {
     repoRoot: string;

package/dist/commands/review/latest.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"latest.d.ts","sourceRoot":"","sources":["../../../src/commands/review/latest.ts"],"names":[],"mappings":"AAGA,OAAO,EAEL,KAAK,cAAc,EACnB,KAAK,aAAa,EACnB,MAAM,sBAAsB,CAAC;AAM9B,OAAO,EAAsB,KAAK,cAAc,EAAE,MAAM,oBAAoB,CAAC;AAE7E,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAEpC,MAAM,MAAM,gBAAgB,GAAG,SAAS,GAAG,OAAO,CAAC;AAEnD,MAAM,WAAW,YAAY;IAC3B,IAAI,EAAE,MAAM,CAAC;IACb,KAAK,EAAE,MAAM,CAAC;IACd,WAAW,EAAE,MAAM,CAAC;IACpB,IAAI,EAAE,gBAAgB,CAAC;CACxB;AAQD,MAAM,WAAW,uBAAuB;IACtC,QAAQ,EAAE,MAAM,CAAC;IACjB,WAAW,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;CAC7B;AAED,UAAU,wBAAwB;IAChC,mBAAmB,EAAE,CAAC,OAAO,EAAE,aAAa,KAAK,cAAc,CAAC;IAChE,kBAAkB,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,MAAM,CAAC,CAAC;IACrD,kBAAkB,EAAE,CAAC,QAAQ,EAAE,MAAM,KAAK,OAAO,CAAC,cAAc,CAAC,CAAC;CACnE;AA6HD,wBAAsB,gBAAgB,CACpC,OAAO,EAAE,uBAAuB,GAC/B,OAAO,CAAC,YAAY,GAAG,IAAI,CAAC,CAmE9B;AA+CD,wBAAgB,yBAAyB,CACvC,SAAS,GAAE,OAAO,CAAC,wBAAwB,CAAM,GAChD,OAAO,CAkBT"}
+{"version":3,"file":"latest.d.ts","sourceRoot":"","sources":["../../../src/commands/review/latest.ts"],"names":[],"mappings":"AAGA,OAAO,EAEL,KAAK,cAAc,EACnB,KAAK,aAAa,EACnB,MAAM,sBAAsB,CAAC;AAM9B,OAAO,EAAsB,KAAK,cAAc,EAAE,MAAM,oBAAoB,CAAC;AAE7E,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAEpC,MAAM,MAAM,gBAAgB,GAAG,SAAS,GAAG,OAAO,CAAC;AAEnD,MAAM,WAAW,YAAY;IAC3B,IAAI,EAAE,MAAM,CAAC;IACb,KAAK,EAAE,MAAM,CAAC;IACd,WAAW,EAAE,MAAM,CAAC;IACpB,IAAI,EAAE,gBAAgB,CAAC;IACvB,QAAQ,EAAE,OAAO,CAAC;IAClB,UAAU,EAAE,OAAO,CAAC;CACrB;AAQD,MAAM,WAAW,uBAAuB;IACtC,QAAQ,EAAE,MAAM,CAAC;IACjB,WAAW,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;CAC7B;AAED,UAAU,wBAAwB;IAChC,mBAAmB,EAAE,CAAC,OAAO,EAAE,aAAa,KAAK,cAAc,CAAC;IAChE,kBAAkB,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,MAAM,CAAC,CAAC;IACrD,kBAAkB,EAAE,CAAC,QAAQ,EAAE,MAAM,KAAK,OAAO,CAAC,cAAc,CAAC,CAAC;CACnE;AAmID,wBAAsB,gBAAgB,CACpC,OAAO,EAAE,uBAAuB,GAC/B,OAAO,CAAC,YAAY,GAAG,IAAI,CAAC,CAqF9B;AA+CD,wBAAgB,yBAAyB,CACvC,SAAS,GAAE,OAAO,CAAC,wBAAwB,CAAM,GAChD,OAAO,CAkBT"}

package/dist/commands/review/latest.js

@@ -16,6 +16,8 @@ const EMPTY_RESULT = {
     scope: null,
     generatedAt: null,
     kind: null,
+    archived: null,
+    actionable: null,
 };
 function normalizeRepoRelativePath(repoRoot, pathValue) {
     const rawPath = pathValue.trim();
@@ -60,7 +62,7 @@ async function listMarkdownFiles(repoRoot, relativeDir) {
         throw error;
     }
 }
-async function readReviewCandidate(repoRoot, relativePath, kind, priority) {
+async function readReviewCandidate(repoRoot, relativePath, kind, priority, archived, actionable) {
     const content = await readFile(join(repoRoot, relativePath), 'utf8');
     const frontmatter = getFrontmatterBlock(content);
     if (!frontmatter) {
@@ -80,6 +82,8 @@ async function readReviewCandidate(repoRoot, relativePath, kind, priority) {
         scope,
         generatedAt,
         kind,
+        archived,
+        actionable,
         generatedTime,
         lifecycleRank: reviewLifecycleRank(scope),
         priority,
@@ -107,20 +111,32 @@ export async function findLatestReview(options) {
             dir: `${projectPath}/reviews`,
             kind: 'project',
             priority: 0,
+            archived: false,
+            actionable: true,
         }, {
             dir: `${projectPath}/reviews/archived`,
             kind: 'project',
             priority: 1,
+            archived: true,
+            actionable: false,
         });
     }
-    scanTargets.push({ dir: '.oat/repo/reviews', kind: 'adhoc', priority: 2 }, {
+    scanTargets.push({
+        dir: '.oat/repo/reviews',
+        kind: 'adhoc',
+        priority: 2,
+        archived: false,
+        actionable: true,
+    }, {
         dir: '.oat/projects/local/orphan-reviews',
         kind: 'adhoc',
         priority: 3,
+        archived: false,
+        actionable: true,
     });
     const candidates = (await Promise.all(scanTargets.map(async (target) => {
         const files = await listMarkdownFiles(options.repoRoot, target.dir);
-        return Promise.all(files.map((file) => readReviewCandidate(options.repoRoot, file, target.kind, target.priority)));
+        return Promise.all(files.map((file) => readReviewCandidate(options.repoRoot, file, target.kind, target.priority, target.archived, target.actionable)));
     })))
         .flat()
         .filter((candidate) => candidate !== null)
@@ -134,6 +150,8 @@ export async function findLatestReview(options) {
         scope: latest.scope,
         generatedAt: latest.generatedAt,
         kind: latest.kind,
+        archived: latest.archived,
+        actionable: latest.actionable,
     };
 }
 async function resolveProjectPath(repoRoot, explicitProject, dependencies) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@open-agent-toolkit/cli",
-  "version": "0.1.24",
+  "version": "0.1.26",
   "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.24"
+    "@open-agent-toolkit/control-plane": "0.1.26"
   },
   "devDependencies": {
     "@types/node": "^22.10.0",