@open-agent-toolkit/cli 0.1.33 → 0.1.35

package/dist/config/resolve.js

@@ -1,4 +1,4 @@
-import { readOatConfig, readOatLocalConfig, readUserConfig, } from './oat-config.js';
+import { BUILTIN_EXEC_TARGETS, readOatConfig, readOatLocalConfig, readUserConfig, } from './oat-config.js';
 const DEFAULT_DEPENDENCIES = {
     readOatConfig,
     readOatLocalConfig,
@@ -139,6 +139,105 @@ export async function resolveEffectiveConfig(repoRoot, userConfigDir, env = proc
         resolved,
     };
 }
+export function resolveGate(effective, skillName) {
+    for (const skills of [
+        effective.local.workflow?.gates?.skills,
+        effective.shared.workflow?.gates?.skills,
+        effective.user.workflow?.gates?.skills,
+    ]) {
+        if (!skills || !hasOwn(skills, skillName)) {
+            continue;
+        }
+        return skills[skillName] ?? null;
+    }
+    return null;
+}
+export function resolveExecTargets(effective) {
+    const targets = cloneExecTargetRegistry(BUILTIN_EXEC_TARGETS);
+    for (const execTargets of [
+        effective.user.workflow?.gates?.execTargets,
+        effective.shared.workflow?.gates?.execTargets,
+        effective.local.workflow?.gates?.execTargets,
+    ]) {
+        mergeExecTargetLayer(targets, execTargets);
+    }
+    return targets;
+}
+function mergeExecTargetLayer(targets, layer) {
+    if (!layer) {
+        return;
+    }
+    for (const [id, override] of Object.entries(layer)) {
+        if (override === null) {
+            delete targets[id];
+            continue;
+        }
+        const existing = targets[id];
+        if (existing) {
+            targets[id] = cloneExecTarget({
+                runtime: override.runtime ?? existing.runtime,
+                baseCommand: override.baseCommand ?? existing.baseCommand,
+                hostDetectionCommand: override.hostDetectionCommand ?? existing.hostDetectionCommand,
+                availabilityCommand: override.availabilityCommand ?? existing.availabilityCommand,
+                priority: override.priority ?? existing.priority,
+            });
+            continue;
+        }
+        const completeTarget = toCompleteExecTarget(override);
+        if (completeTarget) {
+            targets[id] = completeTarget;
+        }
+    }
+}
+function cloneExecTargetRegistry(registry) {
+    return Object.fromEntries(Object.entries(registry).map(([id, target]) => [
+        id,
+        cloneExecTarget(target),
+    ]));
+}
+function cloneExecTarget(target) {
+    const next = {
+        runtime: target.runtime,
+        baseCommand: [...target.baseCommand],
+        priority: target.priority,
+    };
+    if (target.hostDetectionCommand) {
+        next.hostDetectionCommand = [...target.hostDetectionCommand];
+    }
+    if (target.availabilityCommand) {
+        next.availabilityCommand = [...target.availabilityCommand];
+    }
+    return next;
+}
+function toCompleteExecTarget(target) {
+    const runtime = typeof target.runtime === 'string' ? target.runtime.trim() : '';
+    if (!runtime || !isValidArgv(target.baseCommand)) {
+        return null;
+    }
+    const completeTarget = {
+        runtime,
+        baseCommand: [...target.baseCommand],
+        priority: typeof target.priority === 'number' && Number.isFinite(target.priority)
+            ? target.priority
+            : 0,
+    };
+    if (isValidArgv(target.hostDetectionCommand)) {
+        completeTarget.hostDetectionCommand = [...target.hostDetectionCommand];
+    }
+    if (isValidArgv(target.availabilityCommand)) {
+        completeTarget.availabilityCommand = [...target.availabilityCommand];
+    }
+    return completeTarget;
+}
+function isValidArgv(value) {
+    if (!Array.isArray(value) ||
+        value.length === 0 ||
+        !value.every((part) => typeof part === 'string')) {
+        return false;
+    }
+    const [executable] = value;
+    return executable !== undefined && executable.trim().length > 0;
+}
 function flattenConfig(value, prefix = '') {
     if (!isRecord(value)) {
         return {};
@@ -168,6 +267,9 @@ function resolveEnvOverride(key, env) {
 function isRecord(value) {
     return typeof value === 'object' && value !== null && !Array.isArray(value);
 }
+function hasOwn(value, key) {
+    return Object.prototype.hasOwnProperty.call(value, key);
+}
 function isResolvedValue(value) {
     return value !== undefined && value !== null;
 }

package/dist/validation/skills.d.ts

@@ -1,6 +1,7 @@
 export interface ValidationFinding {
     file: string;
     message: string;
+    severity?: 'error' | 'warning';
 }
 export interface ValidateOatSkillsResult {
     validatedSkillCount: number;
@@ -15,6 +16,7 @@ export interface ValidateChangedSkillVersionBumpsResult {
 }
 export interface ValidateOatSkillsOptions {
     baseRef?: string;
+    gateSkillNames?: readonly string[];
 }
 export type ExecFileResult = {
     stdout: string;

package/dist/validation/skills.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"skills.d.ts","sourceRoot":"","sources":["../../src/validation/skills.ts"],"names":[],"mappings":"AAOA,MAAM,WAAW,iBAAiB;IAChC,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,WAAW,uBAAuB;IACtC,mBAAmB,EAAE,MAAM,CAAC;IAC5B,QAAQ,EAAE,iBAAiB,EAAE,CAAC;CAC/B;AAED,MAAM,WAAW,uCAAuC;IACtD,OAAO,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,WAAW,sCAAsC;IACrD,mBAAmB,EAAE,MAAM,CAAC;IAC5B,QAAQ,EAAE,iBAAiB,EAAE,CAAC;CAC/B;AAED,MAAM,WAAW,wBAAwB;IACvC,OAAO,CAAC,EAAE,MAAM,CAAC;CAClB;AAED,MAAM,MAAM,cAAc,GAAG;IAC3B,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;CAChB,CAAC;AAEF,MAAM,MAAM,YAAY,GAAG,CACzB,IAAI,EAAE,MAAM,EACZ,IAAI,EAAE,MAAM,EAAE,EACd,OAAO,CAAC,EAAE;IAAE,GAAG,CAAC,EAAE,MAAM,CAAC;IAAC,GAAG,CAAC,EAAE,MAAM,CAAC,UAAU,CAAA;CAAE,KAChD,OAAO,CAAC,cAAc,CAAC,CAAC;AAE7B,UAAU,6BAA6B;IACrC,WAAW,CAAC,EAAE,YAAY,CAAC;IAC3B,GAAG,CAAC,EAAE,MAAM,CAAC,UAAU,CAAC;CACzB;AAmPD,wBAAsB,gCAAgC,CACpD,QAAQ,EAAE,MAAM,EAChB,OAAO,EAAE,uCAAuC,EAChD,YAAY,GAAE,6BAAkC,GAC/C,OAAO,CAAC,sCAAsC,CAAC,CAoBjD;AAED,wBAAsB,iBAAiB,CACrC,QAAQ,EAAE,MAAM,EAChB,OAAO,GAAE,wBAA6B,EACtC,YAAY,GAAE,6BAAkC,GAC/C,OAAO,CAAC,uBAAuB,CAAC,CA6HlC"}
+{"version":3,"file":"skills.d.ts","sourceRoot":"","sources":["../../src/validation/skills.ts"],"names":[],"mappings":"AAOA,MAAM,WAAW,iBAAiB;IAChC,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,EAAE,MAAM,CAAC;IAChB,QAAQ,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;CAChC;AAED,MAAM,WAAW,uBAAuB;IACtC,mBAAmB,EAAE,MAAM,CAAC;IAC5B,QAAQ,EAAE,iBAAiB,EAAE,CAAC;CAC/B;AAED,MAAM,WAAW,uCAAuC;IACtD,OAAO,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,WAAW,sCAAsC;IACrD,mBAAmB,EAAE,MAAM,CAAC;IAC5B,QAAQ,EAAE,iBAAiB,EAAE,CAAC;CAC/B;AAED,MAAM,WAAW,wBAAwB;IACvC,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,cAAc,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;CACpC;AAED,MAAM,MAAM,cAAc,GAAG;IAC3B,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;CAChB,CAAC;AAEF,MAAM,MAAM,YAAY,GAAG,CACzB,IAAI,EAAE,MAAM,EACZ,IAAI,EAAE,MAAM,EAAE,EACd,OAAO,CAAC,EAAE;IAAE,GAAG,CAAC,EAAE,MAAM,CAAC;IAAC,GAAG,CAAC,EAAE,MAAM,CAAC,UAAU,CAAA;CAAE,KAChD,OAAO,CAAC,cAAc,CAAC,CAAC;AAE7B,UAAU,6BAA6B;IACrC,WAAW,CAAC,EAAE,YAAY,CAAC;IAC3B,GAAG,CAAC,EAAE,MAAM,CAAC,UAAU,CAAC;CACzB;AAuSD,wBAAsB,gCAAgC,CACpD,QAAQ,EAAE,MAAM,EAChB,OAAO,EAAE,uCAAuC,EAChD,YAAY,GAAE,6BAAkC,GAC/C,OAAO,CAAC,sCAAsC,CAAC,CAoBjD;AAED,wBAAsB,iBAAiB,CACrC,QAAQ,EAAE,MAAM,EAChB,OAAO,GAAE,wBAA6B,EACtC,YAAY,GAAE,6BAAkC,GAC/C,OAAO,CAAC,uBAAuB,CAAC,CAmIlC"}

package/dist/validation/skills.js

@@ -22,6 +22,10 @@ function getFrontmatterScalar(frontmatter, key) {
     const match = frontmatter.match(re);
     return match?.[1]?.trim() ?? null;
 }
+function hasTrueFrontmatterValue(frontmatter, key) {
+    return (frontmatterHasKey(frontmatter, key) &&
+        getFrontmatterScalar(frontmatter, key) === 'true');
+}
 function isValidSemver(value) {
     return /^\d+\.\d+\.\d+$/.test(value);
 }
@@ -87,6 +91,40 @@ function validateQuickStartSemantics(skillPath, content, findings) {
         });
     }
 }
+function normalizeGateSkillNames(gateSkillNames) {
+    if (!gateSkillNames) {
+        return [];
+    }
+    return [
+        ...new Set(gateSkillNames.map((name) => name.trim()).filter(Boolean)),
+    ].sort();
+}
+async function collectGateabilityFindings(skillsRoot, gateSkillNames, findings) {
+    for (const skillName of normalizeGateSkillNames(gateSkillNames)) {
+        const skillPath = join(skillsRoot, skillName, 'SKILL.md');
+        let content;
+        try {
+            content = await readFile(skillPath, 'utf8');
+        }
+        catch {
+            findings.push({
+                file: skillPath,
+                message: `Configured gate targets unknown skill: ${skillName}`,
+                severity: 'warning',
+            });
+            continue;
+        }
+        const frontmatter = getFrontmatterBlock(content);
+        if (frontmatter === null ||
+            !hasTrueFrontmatterValue(frontmatter, 'oat_gateable')) {
+            findings.push({
+                file: skillPath,
+                message: 'Configured gate targets skill without oat_gateable: true',
+                severity: 'warning',
+            });
+        }
+    }
+}
 async function listChangedSkillFiles(repoRoot, baseRef, dependencies) {
     const execFile = dependencies.gitExecFile ?? execFileAsync;
     const { stdout } = await execFile('git', [
@@ -261,6 +299,7 @@ export async function validateOatSkills(repoRoot, options = {}, dependencies = {
             validateQuickStartSemantics(skillPath, content, findings);
         }
     }
+    await collectGateabilityFindings(skillsRoot, options.gateSkillNames, findings);
     if (options.baseRef) {
         const changedSkillFiles = await listChangedSkillFiles(repoRoot, options.baseRef, dependencies);
         await collectChangedSkillVersionBumpFindings(repoRoot, options.baseRef, changedSkillFiles, findings, dependencies);

package/package.json

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