@open-agent-toolkit/cli 0.1.35 → 0.1.37

package/dist/commands/gate/review-verdict.js

@@ -0,0 +1,180 @@
+import { readFile } from 'node:fs/promises';
+import { getFrontmatterBlock } from '../shared/frontmatter.js';
+import YAML from 'yaml';
+const SEVERITIES = [
+    'critical',
+    'important',
+    'medium',
+    'minor',
+];
+const FRONTMATTER_COUNT_KEYS = {
+    critical: ['oat_review_critical_count', 'critical'],
+    important: ['oat_review_important_count', 'important'],
+    medium: ['oat_review_medium_count', 'medium'],
+    minor: ['oat_review_minor_count', 'minor'],
+};
+function normalizeReviewType(value) {
+    return value === 'code' || value === 'artifact' ? value : 'unknown';
+}
+function stringOrNull(value) {
+    return typeof value === 'string' && value.trim() ? value.trim() : null;
+}
+function parseCountValue(value) {
+    const numberValue = typeof value === 'number'
+        ? value
+        : typeof value === 'string'
+            ? Number(value.trim())
+            : Number.NaN;
+    if (!Number.isInteger(numberValue) || numberValue < 0) {
+        return null;
+    }
+    return numberValue;
+}
+function readFrontmatterCounts(frontmatter) {
+    const nestedCounts = typeof frontmatter['oat_review_counts'] === 'object' &&
+        frontmatter['oat_review_counts'] !== null &&
+        !Array.isArray(frontmatter['oat_review_counts'])
+        ? frontmatter['oat_review_counts']
+        : null;
+    const counts = {
+        critical: 0,
+        important: 0,
+        medium: 0,
+        minor: 0,
+    };
+    for (const severity of SEVERITIES) {
+        const candidateValues = [
+            ...FRONTMATTER_COUNT_KEYS[severity].map((key) => frontmatter[key]),
+            ...(nestedCounts
+                ? FRONTMATTER_COUNT_KEYS[severity].map((key) => nestedCounts[key])
+                : []),
+        ].filter((value) => value !== undefined && value !== null);
+        if (candidateValues.length === 0) {
+            return null;
+        }
+        const parsedCount = candidateValues.reduce((parsed, value) => {
+            if (parsed !== null) {
+                return parsed;
+            }
+            return parseCountValue(value);
+        }, null);
+        if (parsedCount === null) {
+            return null;
+        }
+        counts[severity] = parsedCount;
+    }
+    return counts;
+}
+function sectionContentIsEmpty(content) {
+    const cleaned = content
+        .split('\n')
+        .map((line) => line.trim())
+        .filter((line) => line.length > 0 && !/^<!--.*-->$/.test(line))
+        .join('\n')
+        .trim();
+    return cleaned.length === 0 || /^none\.?$/i.test(cleaned);
+}
+function countFindingsInSection(content) {
+    if (sectionContentIsEmpty(content)) {
+        return 0;
+    }
+    return content
+        .split('\n')
+        .filter((line) => /^([-*+]\s+\S|\d+\.\s+\S)/.test(line)).length;
+}
+function normalizeHeading(value) {
+    const normalized = value.trim().toLowerCase();
+    if (SEVERITIES.includes(normalized)) {
+        return normalized;
+    }
+    return null;
+}
+function parseFindingsSummaryCounts(content) {
+    const match = content.match(/^Findings:\s*(\d+)\s+critical,\s*(\d+)\s+important,\s*(\d+)\s+medium,\s*(\d+)\s+minor\s*$/im);
+    if (!match) {
+        return null;
+    }
+    return {
+        critical: Number.parseInt(match[1] ?? '0', 10),
+        important: Number.parseInt(match[2] ?? '0', 10),
+        medium: Number.parseInt(match[3] ?? '0', 10),
+        minor: Number.parseInt(match[4] ?? '0', 10),
+    };
+}
+function parseFindingsSectionCounts(content, artifactPath) {
+    const headingMatches = [...content.matchAll(/^#{3,6}\s+(.+?)\s*#*\s*$/gm)];
+    const severityHeadings = headingMatches
+        .map((match) => ({
+        severity: normalizeHeading(match[1] ?? ''),
+        index: match.index ?? 0,
+        headingLength: match[0].length,
+    }))
+        .filter((heading) => heading.severity !== null);
+    if (severityHeadings.length === 0) {
+        return null;
+    }
+    const seenSeverities = new Set(severityHeadings.map((heading) => heading.severity));
+    const missingSeverities = SEVERITIES.filter((severity) => !seenSeverities.has(severity));
+    if (missingSeverities.length > 0) {
+        throw new Error(`Review artifact at ${artifactPath} has an incomplete Findings section; expected headings for Critical, Important, Medium, and Minor. Missing: ${missingSeverities.join(', ')}.`);
+    }
+    const counts = {
+        critical: 0,
+        important: 0,
+        medium: 0,
+        minor: 0,
+    };
+    for (const [offset, heading] of severityHeadings.entries()) {
+        const nextHeading = severityHeadings[offset + 1];
+        const sectionStart = heading.index + heading.headingLength;
+        const sectionEnd = nextHeading?.index ?? content.length;
+        counts[heading.severity] = countFindingsInSection(content.slice(sectionStart, sectionEnd));
+    }
+    return counts;
+}
+function parseFrontmatterObject(frontmatter, artifactPath) {
+    try {
+        const parsed = YAML.parse(frontmatter);
+        if (typeof parsed === 'object' &&
+            parsed !== null &&
+            !Array.isArray(parsed)) {
+            return parsed;
+        }
+        throw new Error('frontmatter must be a YAML object');
+    }
+    catch (error) {
+        const detail = error instanceof Error ? error.message : String(error);
+        throw new Error(`Unable to parse review artifact frontmatter at ${artifactPath}: ${detail}`, { cause: error });
+    }
+}
+function hasBlockingFindings(counts) {
+    return counts.critical > 0 || counts.important > 0;
+}
+export async function parseReviewGateVerdict(artifactPath) {
+    let content;
+    try {
+        content = await readFile(artifactPath, 'utf8');
+    }
+    catch (error) {
+        const detail = error instanceof Error ? error.message : String(error);
+        throw new Error(`Unable to read review artifact at ${artifactPath}: ${detail}`, { cause: error });
+    }
+    const frontmatterBlock = getFrontmatterBlock(content);
+    const frontmatter = frontmatterBlock
+        ? parseFrontmatterObject(frontmatterBlock, artifactPath)
+        : {};
+    const counts = readFrontmatterCounts(frontmatter) ??
+        parseFindingsSummaryCounts(content) ??
+        parseFindingsSectionCounts(content, artifactPath);
+    if (!counts) {
+        throw new Error(`Review artifact at ${artifactPath} does not contain recognizable review findings or explicit verdict counts.`);
+    }
+    return {
+        artifactPath,
+        reviewType: normalizeReviewType(frontmatter['oat_review_type']),
+        scope: stringOrNull(frontmatter['oat_review_scope']),
+        invocation: stringOrNull(frontmatter['oat_review_invocation']),
+        counts,
+        blocking: hasBlockingFindings(counts),
+    };
+}

package/dist/config/oat-config.js

@@ -45,7 +45,7 @@ export const BUILTIN_EXEC_TARGETS = {
     },
     'cursor-default': {
         runtime: 'cursor',
-        baseCommand: ['cursor-agent', '-p', '--force'],
+        baseCommand: ['cursor-agent', '-p'],
         hostDetectionCommand: ['sh', '-c', 'test -n "$CURSOR_AGENT"'],
         availabilityCommand: ['cursor-agent', '--version'],
         priority: 70,

package/package.json

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