@open-agent-toolkit/cli 0.3.11 → 0.3.12

package/assets/bundle-metadata.json

@@ -1,4 +1,4 @@
 {
   "schemaVersion": 1,
-  "oatVersion": "0.3.11"
+  "oatVersion": "0.3.12"
 }

package/assets/docs/reference/docs-index-contract.md

@@ -63,6 +63,7 @@ Fumadocs generated behavior:
 - `pages` follows the order of the directory's `## Contents` links. The root list starts with `index`; a child directory appears by its folder name, and its own `index.md` becomes that folder's landing page. A folder whose `meta.json` keeps `root: true` also lists `index` first, because Fumadocs gives a root folder no implicit landing page.
 - Navigation is strict: there is no `"..."` rest entry. A page or folder that no `## Contents` map lists stays out of the sidebar, and the command reports it by path in its human output and in `--json` (`unlisted`). Add the page to its directory's `## Contents` map to show it.
 - A `## Contents` link to a page in another directory is written as a Fumadocs link entry such as `[Writing Skills](/contributing/skills)`, so each page belongs to exactly one folder.
+- A page or folder whose name Fumadocs would read as a `pages` directive, such as `!hidden`, `z...a`, `...name`, or `---`, is written in the local-path form `./!hidden` so it stays a page entry.
 - The folder `title` comes from the `index.md` frontmatter `title`, falling back to its first `#` heading.
 - Keys nav sync does not own, such as `icon` or `defaultOpen`, are kept.
 - Existing files are compared by meaning, not bytes, so a second run with no docs changes writes nothing, even after a formatter rewrites the files.

package/assets/docs/workflows/skills/recon.md

@@ -304,7 +304,10 @@ scoped to it or global to its review, a coverage finding that names it, or a
 non-affirming disposition) is listed under Review Downgrades with the review's
 own text, even when it is not a key claim, so a `complete` packet cannot hide a
 downgraded claim. A claim that a required review left without a disposition stays
-`unresolved` and is listed there as not reviewed. A review's
+`unresolved`. Any claim below `verified` that an incorporated assurance review,
+core or thorough, listed in its brief but left without a disposition is listed
+there as not reviewed by that review; a claim outside a review's brief is never
+reported as its omission. A review's
 `unresolvedIssues` entry may name the claim IDs it affects or apply to the
 whole review; a legacy plain-string entry is read as review-wide. A material
 coverage gap does not fail publication by itself: the claims it affects stay

package/assets/public-package-versions.json

@@ -1,6 +1,6 @@
 {
-  "cli": "0.3.11",
-  "docs-config": "0.3.11",
-  "docs-theme": "0.3.11",
-  "docs-transforms": "0.3.11"
+  "cli": "0.3.12",
+  "docs-config": "0.3.12",
+  "docs-theme": "0.3.12",
+  "docs-transforms": "0.3.12"
 }

package/assets/skills/recon/SKILL.md

@@ -6,7 +6,7 @@ disable-model-invocation: true
 user-invocable: true
 allowed-tools: Read, Write, Glob, Grep, Bash, AskUserQuestion, Agent, mcp__*
 metadata:
-  version: 1.1.6
+  version: 1.1.7
 ---
 
 # Recon

package/assets/skills/recon/scripts/render-packet.mjs

@@ -9,9 +9,9 @@ import { isDirectExecution } from './lib/cli-entry.mjs';
 import {
   affirmingDispositionByReviewKind,
   classifyUnresolvedIssue,
-  requiredReviewKinds,
   unresolvedIssuesBlockClaim,
 } from './lib/contracts.mjs';
+import { reviewBriefEntries } from './lib/review-binding.mjs';
 import {
   assertSafeExistingPath,
   assertSafeOutputPath,
@@ -121,17 +121,34 @@ function intendedRoutingSection(manifest, routing) {
   ];
 }
 
-// The required reviews that gave `claimId` no disposition at all. Such a
-// claim stays below `verified`, like an uncertain one, and is listed as not
-// reviewed.
+const assuranceReviewKinds = Object.keys(affirmingDispositionByReviewKind);
+
+// The claim IDs a review was briefed on: the entries of the immutable brief
+// its exact reference resolves to, read the way the validator reads them.
+function briefedClaimIds(review, artifacts) {
+  const brief = artifacts.find(
+    ({ reference, value }) =>
+      value.kind === 'recon.review-brief' &&
+      reference.path === review.brief?.path &&
+      reference.digest === review.brief?.digest,
+  )?.value;
+  return new Set(
+    reviewBriefEntries(brief, review.reviewKind).map((entry) => entry.id),
+  );
+}
+
+// Every incorporated assurance review, core or thorough, whose brief listed
+// `claimId` but which gave it no disposition. Such a claim is listed as not
+// reviewed. A claim outside a review's brief was never that review's to
+// dispose of, so it is not an omission.
 function omittedReviews(claimId, reviews) {
-  return requiredReviewKinds
-    .map((kind) => reviews.find((review) => review.reviewKind === kind))
+  return reviews
     .filter(
-      (review) =>
-        review &&
-        !(review.dispositions ?? []).some((item) => item.claimId === claimId),
-    );
+      ({ value, briefed }) =>
+        briefed.has(claimId) &&
+        !(value.dispositions ?? []).some((item) => item.claimId === claimId),
+    )
+    .map(({ value }) => value);
 }
 
 function issueText(entry) {
@@ -140,8 +157,8 @@ function issueText(entry) {
 
 // Every claim an incorporated review kept below `verified` (an unresolved
 // issue that applies to it, a coverage finding that names it, a
-// non-affirming disposition, or a required review that left it without a
-// disposition), with the review's own words. Key-claim status
+// non-affirming disposition, or an assurance review that left a briefed claim
+// without a disposition), with the review's own words. Key-claim status
 // alone would let a `complete` packet hide a downgraded non-key claim.
 function reviewDowngradeLines(validatedRun) {
   const { ledger, artifacts, assuranceReviewIds } = validatedRun;
@@ -152,6 +169,16 @@ function reviewDowngradeLines(validatedRun) {
       (value) =>
         value.kind === 'recon.review-result' && assurance.has(value.id),
     );
+  // Claim-bearing assurance reviews in table order, each with its brief
+  // membership.
+  const briefedReviews = reviews
+    .filter((review) => assuranceReviewKinds.includes(review.reviewKind))
+    .map((value) => ({ value, briefed: briefedClaimIds(value, artifacts) }))
+    .sort(
+      (left, right) =>
+        assuranceReviewKinds.indexOf(left.value.reviewKind) -
+        assuranceReviewKinds.indexOf(right.value.reviewKind),
+    );
   const lines = [];
   for (const claim of ledger.claims) {
     if (claim.status === 'verified') continue;
@@ -182,7 +209,7 @@ function reviewDowngradeLines(validatedRun) {
         }
       }
     }
-    for (const review of omittedReviews(claim.id, reviews)) {
+    for (const review of omittedReviews(claim.id, briefedReviews)) {
       reasons.push(
         `${review.reviewKind} review: not reviewed (no disposition)`,
       );

package/dist/commands/docs/nav/fumadocs.d.ts

@@ -35,6 +35,13 @@ export interface SyncFumadocsNavigationResult {
 export declare function stripInlineMarkdown(text: string): string;
 /** Folder title: frontmatter `title`, else the first H1 outside code fences. */
 export declare function resolveFolderTitle(markdown: string): string | undefined;
+/**
+ * The `pages` item for a page or folder in the same directory. A name the
+ * builder would read as a directive (`!hidden`, `z...a`, `---`, ...) is
+ * written in the local-path form `./name`, which its path join resolves to
+ * the same file; every other name is written as is.
+ */
+export declare function toFumadocsPageItem(name: string): string;
 /**
  * Write one strict `meta.json` per docs directory that has an index.md,
  * derived from that index's Contents map, and report every page the

package/dist/commands/docs/nav/fumadocs.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"fumadocs.d.ts","sourceRoot":"","sources":["../../../../src/commands/docs/nav/fumadocs.ts"],"names":[],"mappings":"AAYA;;;GAGG;AACH,MAAM,WAAW,YAAY;IAC3B,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,KAAK,EAAE,MAAM,EAAE,CAAC;CACjB;AAED,MAAM,WAAW,gBAAgB;IAC/B,wDAAwD;IACxD,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,YAAY,CAAC;IACnB,OAAO,EAAE,OAAO,CAAC;CAClB;AAED,MAAM,WAAW,4BAA4B;IAC3C,SAAS,EAAE,gBAAgB,EAAE,CAAC;IAC9B,mEAAmE;IACnE,OAAO,EAAE,MAAM,EAAE,CAAC;IAClB;;;OAGG;IACH,KAAK,EAAE,MAAM,EAAE,CAAC;IAChB;;;OAGG;IACH,QAAQ,EAAE,MAAM,EAAE,CAAC;CACpB;AAiDD;;;;GAIG;AACH,wBAAgB,mBAAmB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CAQxD;AAED,gFAAgF;AAChF,wBAAgB,kBAAkB,CAAC,QAAQ,EAAE,MAAM,GAAG,MAAM,GAAG,SAAS,CAoCvE;AAoMD;;;;;;;GAOG;AACH,wBAAsB,sBAAsB,CAAC,OAAO,EAAE;IACpD,QAAQ,EAAE,MAAM,CAAC;IACjB,6CAA6C;IAC7C,KAAK,CAAC,EAAE,OAAO,CAAC;CACjB,GAAG,OAAO,CAAC,4BAA4B,CAAC,CAkExC"}
+{"version":3,"file":"fumadocs.d.ts","sourceRoot":"","sources":["../../../../src/commands/docs/nav/fumadocs.ts"],"names":[],"mappings":"AAYA;;;GAGG;AACH,MAAM,WAAW,YAAY;IAC3B,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,KAAK,EAAE,MAAM,EAAE,CAAC;CACjB;AAED,MAAM,WAAW,gBAAgB;IAC/B,wDAAwD;IACxD,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,YAAY,CAAC;IACnB,OAAO,EAAE,OAAO,CAAC;CAClB;AAED,MAAM,WAAW,4BAA4B;IAC3C,SAAS,EAAE,gBAAgB,EAAE,CAAC;IAC9B,mEAAmE;IACnE,OAAO,EAAE,MAAM,EAAE,CAAC;IAClB;;;OAGG;IACH,KAAK,EAAE,MAAM,EAAE,CAAC;IAChB;;;OAGG;IACH,QAAQ,EAAE,MAAM,EAAE,CAAC;CACpB;AAyDD;;;;GAIG;AACH,wBAAgB,mBAAmB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CAQxD;AAED,gFAAgF;AAChF,wBAAgB,kBAAkB,CAAC,QAAQ,EAAE,MAAM,GAAG,MAAM,GAAG,SAAS,CAoCvE;AA8BD;;;;;GAKG;AACH,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CAEvD;AAoLD;;;;;;;GAOG;AACH,wBAAsB,sBAAsB,CAAC,OAAO,EAAE;IACpD,QAAQ,EAAE,MAAM,CAAC;IACjB,6CAA6C;IAC7C,KAAK,CAAC,EAAE,OAAO,CAAC;CACjB,GAAG,OAAO,CAAC,4BAA4B,CAAC,CAkExC"}

package/dist/commands/docs/nav/fumadocs.js

@@ -6,6 +6,13 @@ import { fileExists } from '../../../fs/io.js';
 import { parseIndexContents, resolveEntryTarget } from './contents.js';
 const PAGE_EXTENSIONS = ['.md', '.mdx'];
 const GROUP_FOLDER = /^\(.+\)$/;
+// The `pages` item grammar of the fumadocs-core 16.10.2 page-tree builder
+// (`resolveFolderItem`): exact rest entries, separators, links, and the
+// exclude and extract prefixes are checked before an item is read as a path.
+const FUMADOCS_REST_ITEMS = new Set(['...', 'z...a']);
+const FUMADOCS_SEPARATOR = /^---(?:\[(?<icon>[^\]]+)])?(?<name>.+)---|^---$/;
+const FUMADOCS_LINK = /^(?<external>external:)?(?:\[(?<icon>[^\]]+)])?\[(?<name>[^\]]+)]\((?<url>[^)]+)\)$/;
+const FUMADOCS_PREFIXES = ['!', '...'];
 const H1 = /^#\s+(.+?)\s*#*\s*$/;
 const FENCE = /^\s*(```|~~~)/;
 function parentDir(path) {
@@ -105,6 +112,22 @@ function pageUrl(pagePath) {
     }
     return `/${slugs.join('/')}`;
 }
+/** Whether the Fumadocs page-tree builder reads `item` as a directive, not a path. */
+function isFumadocsDirective(item) {
+    return (FUMADOCS_REST_ITEMS.has(item) ||
+        FUMADOCS_SEPARATOR.test(item) ||
+        FUMADOCS_LINK.test(item) ||
+        FUMADOCS_PREFIXES.some((prefix) => item.startsWith(prefix)));
+}
+/**
+ * The `pages` item for a page or folder in the same directory. A name the
+ * builder would read as a directive (`!hidden`, `z...a`, `---`, ...) is
+ * written in the local-path form `./name`, which its path join resolves to
+ * the same file; every other name is written as is.
+ */
+export function toFumadocsPageItem(name) {
+    return isFumadocsDirective(name) ? `./${name}` : name;
+}
 function hrefFragment(href) {
     const hashIndex = href.indexOf('#');
     return hashIndex >= 0 ? href.slice(hashIndex) : '';
@@ -150,10 +173,10 @@ async function buildFolderPages(docsRoot, relativeDir, markdown, implicitIndex)
         }
         else if (target.kind === 'section' &&
             parentDir(targetDir) === relativeDir) {
-            item = basename(targetDir);
+            item = toFumadocsPageItem(basename(targetDir));
         }
         else if (target.kind === 'page' && targetDir === relativeDir) {
-            item = basename(target.path, extname(target.path));
+            item = toFumadocsPageItem(basename(target.path, extname(target.path)));
         }
         else {
             item = `[${entry.title}](${pageUrl(target.path)}${hrefFragment(entry.href)})`;
@@ -209,7 +232,9 @@ function collectUnlisted(inventory, metas, implicitIndexDirs) {
             reachablePages.add(index);
         }
         for (const item of meta.pages) {
-            if (item.startsWith('[')) {
+            // Read items the way the loader does: a directive (including a link
+            // entry) names no local page, and `./name` joins to `name`.
+            if (isFumadocsDirective(item)) {
                 continue;
             }
             const itemPath = posix.join(relativeDir, item);

package/package.json

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