@open-agent-toolkit/cli 0.1.48 → 0.1.51

package/dist/commands/config/index.js

@@ -5,12 +5,14 @@ import { resolveProjectsRoot } from '../shared/oat-paths.js';
 import { confirmAction, } from '../shared/shared.prompts.js';
 import { readGlobalOptions } from '../shared/shared.utils.js';
 import { compileDispatchCeilingPreset } from '../../config/dispatch-ceiling-preset.js';
+import { normalizeDispatchMatrix, validateDispatchRouteTarget, walkDispatchMatrix, } from '../../config/dispatch-matrix.js';
 import { dispatchPolicyModeDescription, dispatchPolicyPolicyDescription, managedDispatchPolicyValueList, } from '../../config/dispatch-policy-options.js';
-import { VALID_DISPATCH_POLICY_MODES, VALID_MANAGED_DISPATCH_POLICIES, isCodexMaterializedRouteTarget, isWorkflowDispatchCandidateLadder, isWorkflowDispatchFallbackRoute, readOatConfig, readOatLocalConfig, readUserConfig, validateDispatchRouteTarget, writeOatConfig, writeOatLocalConfig, writeUserConfig, } from '../../config/oat-config.js';
+import { VALID_DISPATCH_POLICY_MODES, VALID_MANAGED_DISPATCH_POLICIES, readOatConfig, readOatLocalConfig, readUserConfig, writeOatConfig, writeOatLocalConfig, writeUserConfig, } from '../../config/oat-config.js';
 import { resolveEffectiveConfig, } from '../../config/resolve.js';
 import { resolveAssetsRoot } from '../../fs/assets.js';
 import { resolveProjectRoot } from '../../fs/paths.js';
 import { normalizeMatrixCellAvailability, validateMatrixCell, } from '../../providers/identity/availability.js';
+import { createDispatchValidationPassContext, validateDispatchMatrixRefs, } from '../../providers/identity/dispatch-validation.js';
 import { Command } from 'commander';
 import { createConfigDumpCommand } from './dump.js';
 const DISPATCH_CEILING_PROVIDER_KEY_PREFIX = 'workflow.dispatchCeiling.providers.';
@@ -563,6 +565,8 @@ const DEFAULT_DEPENDENCIES = {
     readFile: (path) => readFileDefault(path, 'utf8'),
     confirmAction,
     validateMatrixCell,
+    createDispatchValidationPassContext,
+    validateDispatchMatrixRefs,
     processEnv: process.env,
 };
 function isConfigKey(value) {
@@ -765,130 +769,62 @@ function parseDispatchMatrixRecommendation(raw) {
     if (!isRecord(parsed.providers)) {
         throw new Error('Dispatch matrix recommendation asset is missing providers.');
     }
+    const normalized = normalizeDispatchMatrix(parsed.providers, {
+        pathPrefix: 'workflow.dispatchCeiling.providers',
+        compatibilityMode: 'layered-config',
+    });
     return {
         version,
-        providers: parsed.providers,
+        providers: normalized.providers,
+        issues: normalized.issues,
     };
 }
-function isRouteTarget(entry) {
-    return isRecord(entry);
-}
-function formatRouteTargetValue(entry) {
-    return [entry.model, entry.effort].filter(Boolean).join('/');
+function collectDispatchMatrixCellRefs(providers) {
+    return walkDispatchMatrix(providers, {
+        source: 'repo-config',
+        pathPrefix: 'workflow.dispatchCeiling.providers',
+    });
 }
-function addDispatchMatrixCellRefs(refs, provider, path, cell) {
-    if (typeof cell === 'string') {
-        refs.push({ provider, value: cell, path });
-        return;
-    }
-    const addEntry = (entry, entryPath) => {
-        if (typeof entry === 'string') {
-            refs.push({ provider, value: entry, path: entryPath });
-            return;
-        }
-        if (!isRouteTarget(entry)) {
-            return;
-        }
-        const targetProvider = entry.harness ?? provider;
-        if (isCodexMaterializedRouteTarget(provider, entry)) {
-            if (entry.model && entry.effort) {
-                refs.push({
-                    provider: targetProvider,
-                    value: formatRouteTargetValue(entry),
-                    path: entryPath,
-                    target: entry,
-                });
-                return;
-            }
-        }
-        if (entry.model) {
-            refs.push({
-                provider: targetProvider,
-                value: entry.model,
-                path: `${entryPath}.model`,
-            });
-        }
-        if (entry.effort) {
-            refs.push({
-                provider: targetProvider,
-                value: entry.effort,
-                path: `${entryPath}.effort`,
-            });
-        }
-    };
-    if (isWorkflowDispatchCandidateLadder(cell)) {
-        for (const [candidateIndex, candidate] of cell.candidates.entries()) {
-            const candidatePath = `${path}.candidates[${candidateIndex}]`;
-            if (isWorkflowDispatchFallbackRoute(candidate)) {
-                for (const [routeIndex, entry] of candidate.route.entries()) {
-                    addEntry(entry, `${candidatePath}.route[${routeIndex}]`);
-                }
-                continue;
-            }
-            addEntry(candidate, candidatePath);
+function collectDispatchMatrixTargetValidationErrors(refs, issues) {
+    const errors = issues.map((issue) => {
+        const relativePath = issue.path.startsWith(DISPATCH_CEILING_PROVIDER_KEY_PREFIX)
+            ? issue.path.slice(DISPATCH_CEILING_PROVIDER_KEY_PREFIX.length)
+            : '';
+        const provider = relativePath.split(/[.[]/, 1)[0] ?? '';
+        const closedValues = closedDispatchProviderValues(provider);
+        if (typeof issue.value === 'string' && closedValues) {
+            return `Invalid value for ${issue.path}: expected one of ${closedValues.join(' | ')}, got '${issue.value}'`;
         }
-        return;
-    }
-    for (const [index, entry] of cell.entries()) {
-        addEntry(entry, `${path}[${index}]`);
-    }
-}
-function collectDispatchMatrixCellRefs(providers) {
-    const refs = [];
-    for (const [provider, providerValue] of Object.entries(providers)) {
-        const providerPath = `workflow.dispatchCeiling.providers.${provider}`;
-        if (typeof providerValue === 'string') {
-            refs.push({ provider, value: providerValue, path: providerPath });
+        return `${issue.path}: malformed dispatch matrix entry (${issue.kind}).`;
+    });
+    for (const ref of refs) {
+        if (ref.target === null) {
             continue;
         }
-        for (const [tier, cell] of Object.entries(providerValue)) {
-            if (cell === undefined) {
-                continue;
-            }
-            addDispatchMatrixCellRefs(refs, provider, `${providerPath}.${tier}`, cell);
+        const validation = validateDispatchRouteTarget(ref.provider, ref.target);
+        if (!validation.valid) {
+            errors.push(`${ref.path}: ${validation.reason}`);
         }
     }
-    return refs;
+    return errors;
 }
-function collectDispatchMatrixTargetValidationErrors(providers) {
-    const errors = [];
-    for (const [provider, providerValue] of Object.entries(providers)) {
-        if (typeof providerValue === 'string') {
-            continue;
-        }
-        const providerPath = `workflow.dispatchCeiling.providers.${provider}`;
-        for (const [tier, cell] of Object.entries(providerValue)) {
-            if (cell === undefined || typeof cell === 'string') {
-                continue;
-            }
-            const validateEntry = (entry, entryPath) => {
-                if (!isRouteTarget(entry)) {
-                    return;
-                }
-                const validation = validateDispatchRouteTarget(provider, entry);
-                if (!validation.valid) {
-                    errors.push(`${entryPath}: ${validation.reason}`);
-                }
-            };
-            if (isWorkflowDispatchCandidateLadder(cell)) {
-                for (const [candidateIndex, candidate] of cell.candidates.entries()) {
-                    const candidatePath = `${providerPath}.${tier}.candidates[${candidateIndex}]`;
-                    if (isWorkflowDispatchFallbackRoute(candidate)) {
-                        for (const [routeIndex, entry] of candidate.route.entries()) {
-                            validateEntry(entry, `${candidatePath}.route[${routeIndex}]`);
-                        }
-                        continue;
-                    }
-                    validateEntry(candidate, candidatePath);
-                }
-                continue;
-            }
-            for (const [index, entry] of cell.entries()) {
-                validateEntry(entry, `${providerPath}.${tier}[${index}]`);
-            }
-        }
+function toAvailabilityRef(ref) {
+    if (ref.value !== null) {
+        return { provider: ref.provider, value: ref.value, path: ref.path };
     }
-    return errors;
+    if (ref.target === null) {
+        return null;
+    }
+    const value = ref.target.model ?? ref.target.effort;
+    if (!value) {
+        return null;
+    }
+    return {
+        provider: ref.target.harness ?? ref.provider,
+        value,
+        path: ref.path,
+        target: ref.target,
+    };
 }
 function applyWorkflowValue(workflow, key, value) {
     const subKey = key.slice('workflow.'.length);
@@ -1225,28 +1161,34 @@ async function loadDispatchMatrixRecommendation(dependencies) {
     return parseDispatchMatrixRecommendation(await dependencies.readFile(assetPath));
 }
 async function validateRecommendationCells(repoRoot, recommendation, dependencies, warn) {
-    const targetErrors = collectDispatchMatrixTargetValidationErrors(recommendation.providers);
+    const refs = collectDispatchMatrixCellRefs(recommendation.providers);
+    const targetErrors = collectDispatchMatrixTargetValidationErrors(refs, recommendation.issues);
     if (targetErrors.length > 0) {
         throw new Error(targetErrors.join('\n'));
     }
-    for (const ref of collectDispatchMatrixCellRefs(recommendation.providers)) {
+    const availabilityRefs = refs.filter((matrixRef) => {
+        const ref = toAvailabilityRef(matrixRef);
+        if (ref === null) {
+            return false;
+        }
         const closedValues = closedDispatchProviderValues(ref.provider);
         if (!ref.target && closedValues && !closedValues.includes(ref.value)) {
             throw new Error(`Invalid value for ${ref.path}: expected one of ${closedValues.join(' | ')}, got '${ref.value}'`);
         }
-        let availability;
-        try {
-            availability = await dependencies.validateMatrixCell(ref.provider, ref.value, {
-                cwd: repoRoot,
-                env: dependencies.processEnv,
-                detailed: true,
-                ...(ref.target ? { target: ref.target } : {}),
-            });
-        }
-        catch {
-            availability = 'unvalidated';
-        }
-        const warning = matrixCellAvailabilityWarning(ref.path, ref.value, availability);
+        return true;
+    });
+    const pass = dependencies.createDispatchValidationPassContext({
+        cwd: repoRoot,
+        env: dependencies.processEnv,
+        validateMatrixCell: dependencies.validateMatrixCell,
+    });
+    const results = await dependencies.validateDispatchMatrixRefs(availabilityRefs, pass);
+    for (const result of results) {
+        const ref = toAvailabilityRef(result.ref);
+        const warning = matrixCellAvailabilityWarning(ref.path, ref.value, {
+            availability: result.status,
+            ...(result.diagnostic ? { message: result.diagnostic } : {}),
+        });
         if (warning) {
             warn(warning);
         }

package/dist/commands/doctor/index.d.ts

@@ -3,6 +3,7 @@ import { type PjmDoctorOptions } from '../pjm/doctor.js';
 import { type OatConfig, type OatLocalConfig, type UserConfig } from '../../config/oat-config.js';
 import { type Manifest } from '../../manifest/index.js';
 import { type MatrixCellAvailabilityResponse, type ValidateMatrixCellOptions } from '../../providers/identity/availability.js';
+import { createDispatchValidationPassContext, validateDispatchMatrixRefs } from '../../providers/identity/dispatch-validation.js';
 import type { ConcreteScope } from '../../shared/types.js';
 import { type DoctorCheck } from '../../ui/output.js';
 import { Command } from 'commander';
@@ -23,6 +24,8 @@ interface DoctorDependencies {
     readOatLocalConfig: (repoRoot: string) => Promise<OatLocalConfig>;
     readUserConfig: (userConfigDir: string) => Promise<UserConfig>;
     validateMatrixCell: (provider: string, value: string, options: ValidateMatrixCellOptions) => Promise<MatrixCellAvailabilityResponse>;
+    createDispatchValidationPassContext: typeof createDispatchValidationPassContext;
+    validateDispatchMatrixRefs: typeof validateDispatchMatrixRefs;
     processEnv: NodeJS.ProcessEnv;
     runPjmDoctorChecks: (repoRoot: string, options?: PjmDoctorOptions) => Promise<DoctorCheck[]>;
     checkSkillVersions: (scopeRoot: string, assetsRoot: string, pathExists: (path: string) => Promise<boolean>) => Promise<SkillVersionReport>;

package/dist/commands/doctor/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/commands/doctor/index.ts"],"names":[],"mappings":"AAWA,OAAO,EAEL,KAAK,cAAc,EACnB,KAAK,aAAa,EACnB,MAAM,sBAAsB,CAAC;AAE9B,OAAO,EAGL,KAAK,gBAAgB,EACtB,MAAM,sBAAsB,CAAC;AAO9B,OAAO,EAEL,KAAK,SAAS,EACd,KAAK,cAAc,EAMnB,KAAK,UAAU,EAKhB,MAAM,oBAAoB,CAAC;AAI5B,OAAO,EAAgB,KAAK,QAAQ,EAAE,MAAM,iBAAiB,CAAC;AAO9D,OAAO,EAIL,KAAK,8BAA8B,EACnC,KAAK,yBAAyB,EAC/B,MAAM,kCAAkC,CAAC;AAC1C,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,eAAe,CAAC;AACnD,OAAO,EAAE,KAAK,WAAW,EAAuB,MAAM,YAAY,CAAC;AACnE,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAEpC,UAAU,kBAAkB;IAC1B,mBAAmB,EAAE,CAAC,OAAO,EAAE,aAAa,KAAK,cAAc,CAAC;IAChE,gBAAgB,EAAE,CAChB,KAAK,EAAE,aAAa,EACpB,OAAO,EAAE,cAAc,KACpB,OAAO,CAAC,MAAM,CAAC,CAAC;IACrB,UAAU,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,OAAO,CAAC,OAAO,CAAC,CAAC;IAC/C,YAAY,EAAE,CAAC,YAAY,EAAE,MAAM,KAAK,OAAO,CAAC,QAAQ,CAAC,CAAC;IAC1D,mBAAmB,EAAE,CAAC,SAAS,EAAE,MAAM,KAAK,OAAO,CAAC,OAAO,CAAC,CAAC;IAC7D,cAAc,EAAE,CACd,SAAS,EAAE,MAAM,KACd,OAAO,CACV,KAAK,CAAC;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,QAAQ,EAAE,OAAO,CAAC;QAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAAA;KAAE,CAAC,CACnE,CAAC;IACF,QAAQ,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,OAAO,CAAC,MAAM,CAAC,CAAC;IAC5C,iBAAiB,EAAE,MAAM,OAAO,CAAC,MAAM,CAAC,CAAC;IACzC,aAAa,EAAE,CAAC,QAAQ,EAAE,MAAM,KAAK,OAAO,CAAC,SAAS,CAAC,CAAC;IACxD,kBAAkB,EAAE,CAAC,QAAQ,EAAE,MAAM,KAAK,OAAO,CAAC,cAAc,CAAC,CAAC;IAClE,cAAc,EAAE,CAAC,aAAa,EAAE,MAAM,KAAK,OAAO,CAAC,UAAU,CAAC,CAAC;IAC/D,kBAAkB,EAAE,CAClB,QAAQ,EAAE,MAAM,EAChB,KAAK,EAAE,MAAM,EACb,OAAO,EAAE,yBAAyB,KAC/B,OAAO,CAAC,8BAA8B,CAAC,CAAC;IAC7C,UAAU,EAAE,MAAM,CAAC,UAAU,CAAC;IAC9B,kBAAkB,EAAE,CAClB,QAAQ,EAAE,MAAM,EAChB,OAAO,CAAC,EAAE,gBAAgB,KACvB,OAAO,CAAC,WAAW,EAAE,CAAC,CAAC;IAC5B,kBAAkB,EAAE,CAClB,SAAS,EAAE,MAAM,EACjB,UAAU,EAAE,MAAM,EAClB,UAAU,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,OAAO,CAAC,OAAO,CAAC,KAC3C,OAAO,CAAC,kBAAkB,CAAC,CAAC;CAClC;AAED,UAAU,oBAAoB;IAC5B,KAAK,EAAE,MAAM,CAAC;IACd,gBAAgB,EAAE,MAAM,GAAG,IAAI,CAAC;IAChC,cAAc,EAAE,MAAM,GAAG,IAAI,CAAC;CAC/B;AAED,UAAU,kBAAkB;IAC1B,mBAAmB,EAAE,MAAM,CAAC;IAC5B,0BAA0B,EAAE,MAAM,CAAC;IACnC,cAAc,EAAE,oBAAoB,EAAE,CAAC;CACxC;AA0uBD,wBAAgB,mBAAmB,CACjC,SAAS,GAAE,OAAO,CAAC,kBAAkB,CAAM,GAC1C,OAAO,CAcT"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/commands/doctor/index.ts"],"names":[],"mappings":"AAWA,OAAO,EAEL,KAAK,cAAc,EACnB,KAAK,aAAa,EACnB,MAAM,sBAAsB,CAAC;AAE9B,OAAO,EAGL,KAAK,gBAAgB,EACtB,MAAM,sBAAsB,CAAC;AAY9B,OAAO,EAEL,KAAK,SAAS,EACd,KAAK,cAAc,EAGnB,KAAK,UAAU,EAChB,MAAM,oBAAoB,CAAC;AAI5B,OAAO,EAAgB,KAAK,QAAQ,EAAE,MAAM,iBAAiB,CAAC;AAO9D,OAAO,EAGL,KAAK,8BAA8B,EACnC,KAAK,yBAAyB,EAC/B,MAAM,kCAAkC,CAAC;AAC1C,OAAO,EACL,mCAAmC,EACnC,0BAA0B,EAC3B,MAAM,yCAAyC,CAAC;AACjD,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,eAAe,CAAC;AACnD,OAAO,EAAE,KAAK,WAAW,EAAuB,MAAM,YAAY,CAAC;AACnE,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAEpC,UAAU,kBAAkB;IAC1B,mBAAmB,EAAE,CAAC,OAAO,EAAE,aAAa,KAAK,cAAc,CAAC;IAChE,gBAAgB,EAAE,CAChB,KAAK,EAAE,aAAa,EACpB,OAAO,EAAE,cAAc,KACpB,OAAO,CAAC,MAAM,CAAC,CAAC;IACrB,UAAU,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,OAAO,CAAC,OAAO,CAAC,CAAC;IAC/C,YAAY,EAAE,CAAC,YAAY,EAAE,MAAM,KAAK,OAAO,CAAC,QAAQ,CAAC,CAAC;IAC1D,mBAAmB,EAAE,CAAC,SAAS,EAAE,MAAM,KAAK,OAAO,CAAC,OAAO,CAAC,CAAC;IAC7D,cAAc,EAAE,CACd,SAAS,EAAE,MAAM,KACd,OAAO,CACV,KAAK,CAAC;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,QAAQ,EAAE,OAAO,CAAC;QAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAAA;KAAE,CAAC,CACnE,CAAC;IACF,QAAQ,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,OAAO,CAAC,MAAM,CAAC,CAAC;IAC5C,iBAAiB,EAAE,MAAM,OAAO,CAAC,MAAM,CAAC,CAAC;IACzC,aAAa,EAAE,CAAC,QAAQ,EAAE,MAAM,KAAK,OAAO,CAAC,SAAS,CAAC,CAAC;IACxD,kBAAkB,EAAE,CAAC,QAAQ,EAAE,MAAM,KAAK,OAAO,CAAC,cAAc,CAAC,CAAC;IAClE,cAAc,EAAE,CAAC,aAAa,EAAE,MAAM,KAAK,OAAO,CAAC,UAAU,CAAC,CAAC;IAC/D,kBAAkB,EAAE,CAClB,QAAQ,EAAE,MAAM,EAChB,KAAK,EAAE,MAAM,EACb,OAAO,EAAE,yBAAyB,KAC/B,OAAO,CAAC,8BAA8B,CAAC,CAAC;IAC7C,mCAAmC,EAAE,OAAO,mCAAmC,CAAC;IAChF,0BAA0B,EAAE,OAAO,0BAA0B,CAAC;IAC9D,UAAU,EAAE,MAAM,CAAC,UAAU,CAAC;IAC9B,kBAAkB,EAAE,CAClB,QAAQ,EAAE,MAAM,EAChB,OAAO,CAAC,EAAE,gBAAgB,KACvB,OAAO,CAAC,WAAW,EAAE,CAAC,CAAC;IAC5B,kBAAkB,EAAE,CAClB,SAAS,EAAE,MAAM,EACjB,UAAU,EAAE,MAAM,EAClB,UAAU,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,OAAO,CAAC,OAAO,CAAC,KAC3C,OAAO,CAAC,kBAAkB,CAAC,CAAC;CAClC;AAED,UAAU,oBAAoB;IAC5B,KAAK,EAAE,MAAM,CAAC;IACd,gBAAgB,EAAE,MAAM,GAAG,IAAI,CAAC;IAChC,cAAc,EAAE,MAAM,GAAG,IAAI,CAAC;CAC/B;AAED,UAAU,kBAAkB;IAC1B,mBAAmB,EAAE,MAAM,CAAC;IAC5B,0BAA0B,EAAE,MAAM,CAAC;IACnC,cAAc,EAAE,oBAAoB,EAAE,CAAC;CACxC;AA8wBD,wBAAgB,mBAAmB,CACjC,SAAS,GAAE,OAAO,CAAC,kBAAkB,CAAM,GAC1C,OAAO,CAcT"}