@vibe-agent-toolkit/cli 0.1.39-rc.8 → 0.1.39

package/dist/utils/config-loader.d.ts

@@ -5,6 +5,19 @@
  * - VAT_TEST_CONFIG: Override config file path for testing (absolute path)
  */
 import { type ProjectConfig } from '@vibe-agent-toolkit/resources';
+/**
+ * A `vibe-agent-toolkit.config.yaml` exists but failed to parse or validate.
+ *
+ * Distinct from "no config found" (which surfaces as `undefined`): a *broken*
+ * config is a hard error the user must fix, not something to silently treat as
+ * absent. Commands that resolve a skill through a config (`vat skill review`,
+ * `vat skill test`) should surface this; a bulk linter (`vat audit`) may catch
+ * it and fall back to config-free validation.
+ */
+export declare class ConfigLoadError extends Error {
+    readonly projectRoot: string;
+    constructor(projectRoot: string, cause: unknown);
+}
 /**
  * Load and validate project configuration
  *
@@ -41,10 +54,17 @@ export declare function resetLoadedConfigCache(): void;
 /**
  * Cached variant of {@link loadConfig} keyed by `projectRoot`.
  *
- * Both successful loads and parse failures are cached (failures as `null`)
- * so a broken config doesn't re-parse on every skill in the same scan. A
- * test that edits a broken config into a good one between calls must invoke
+ * Returns the parsed config, or `undefined` when no config file exists. A config
+ * file that exists but fails to parse/validate throws {@link ConfigLoadError} —
+ * a broken config is a hard error, NOT silently treated as "no config" (that
+ * conflation silently downgraded `vat skill review` and would let `vat skill
+ * test` stage the wrong artifact). Both outcomes are cached (the error too) so a
+ * broken config re-throws without re-parsing on every skill in a scan. A test
+ * that edits a broken config into a good one between calls must invoke
  * {@link resetLoadedConfigCache} (audit's `resetAuditCaches()` does this).
+ *
+ * Callers that intentionally tolerate a broken config (e.g. `vat audit`, a bulk
+ * linter that should still validate the skill itself) catch `ConfigLoadError`.
  */
 export declare function loadConfigCached(projectRoot: string): ProjectConfig | undefined;
 //# sourceMappingURL=config-loader.d.ts.map

package/dist/utils/config-loader.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"config-loader.d.ts","sourceRoot":"","sources":["../../src/utils/config-loader.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAKH,OAAO,EAAuB,KAAK,aAAa,EAAE,MAAM,+BAA+B,CAAC;AAMxF;;;;;;;;;;;;;;;;;;;;GAoBG;AACH,wBAAgB,UAAU,CAAC,WAAW,EAAE,MAAM,GAAG,aAAa,GAAG,SAAS,CA8BzE;AAED;;GAEG;AACH,wBAAgB,YAAY,CAAC,UAAU,EAAE,MAAM,GAAG,MAAM,CAEvD;AAgBD;;;;;GAKG;AACH,wBAAgB,sBAAsB,IAAI,IAAI,CAE7C;AAED;;;;;;;GAOG;AACH,wBAAgB,gBAAgB,CAAC,WAAW,EAAE,MAAM,GAAG,aAAa,GAAG,SAAS,CAY/E"}
+{"version":3,"file":"config-loader.d.ts","sourceRoot":"","sources":["../../src/utils/config-loader.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAKH,OAAO,EAAuB,KAAK,aAAa,EAAE,MAAM,+BAA+B,CAAC;AAMxF;;;;;;;;GAQG;AACH,qBAAa,eAAgB,SAAQ,KAAK;IACxC,QAAQ,CAAC,WAAW,EAAE,MAAM,CAAC;gBACjB,WAAW,EAAE,MAAM,EAAE,KAAK,EAAE,OAAO;CAMhD;AAED;;;;;;;;;;;;;;;;;;;;GAoBG;AACH,wBAAgB,UAAU,CAAC,WAAW,EAAE,MAAM,GAAG,aAAa,GAAG,SAAS,CA8BzE;AAED;;GAEG;AACH,wBAAgB,YAAY,CAAC,UAAU,EAAE,MAAM,GAAG,MAAM,CAEvD;AAuBD;;;;;GAKG;AACH,wBAAgB,sBAAsB,IAAI,IAAI,CAG7C;AAED;;;;;;;;;;;;;;GAcG;AACH,wBAAgB,gBAAgB,CAAC,WAAW,EAAE,MAAM,GAAG,aAAa,GAAG,SAAS,CAgB/E"}

package/dist/utils/config-loader.js

@@ -10,6 +10,25 @@ import { ProjectConfigSchema } from '@vibe-agent-toolkit/resources';
 import { safePath } from '@vibe-agent-toolkit/utils';
 import * as yaml from 'yaml';
 const CONFIG_FILENAME = 'vibe-agent-toolkit.config.yaml';
+/**
+ * A `vibe-agent-toolkit.config.yaml` exists but failed to parse or validate.
+ *
+ * Distinct from "no config found" (which surfaces as `undefined`): a *broken*
+ * config is a hard error the user must fix, not something to silently treat as
+ * absent. Commands that resolve a skill through a config (`vat skill review`,
+ * `vat skill test`) should surface this; a bulk linter (`vat audit`) may catch
+ * it and fall back to config-free validation.
+ */
+export class ConfigLoadError extends Error {
+    projectRoot;
+    constructor(projectRoot, cause) {
+        super(cause instanceof Error ? cause.message : String(cause));
+        this.name = 'ConfigLoadError';
+        this.projectRoot = projectRoot;
+        if (cause instanceof Error)
+            this.cause = cause;
+    }
+}
 /**
  * Load and validate project configuration
  *
@@ -75,6 +94,12 @@ export function getConfigDir(configPath) {
  * See spec docs/superpowers/specs/2026-05-17-root-model-and-leading-slash-design.md §8.
  */
 const loadedConfigCache = new Map();
+/**
+ * Companion cache for {@link loadConfigCached}: a broken config's
+ * {@link ConfigLoadError}, keyed by `projectRoot`, so a broken config re-throws
+ * the same error on every skill in a scan without re-parsing.
+ */
+const loadErrorCache = new Map();
 /**
  * Reset the cache used by {@link loadConfigCached}.
  *
@@ -83,16 +108,27 @@ const loadedConfigCache = new Map();
  */
 export function resetLoadedConfigCache() {
     loadedConfigCache.clear();
+    loadErrorCache.clear();
 }
 /**
  * Cached variant of {@link loadConfig} keyed by `projectRoot`.
  *
- * Both successful loads and parse failures are cached (failures as `null`)
- * so a broken config doesn't re-parse on every skill in the same scan. A
- * test that edits a broken config into a good one between calls must invoke
+ * Returns the parsed config, or `undefined` when no config file exists. A config
+ * file that exists but fails to parse/validate throws {@link ConfigLoadError} —
+ * a broken config is a hard error, NOT silently treated as "no config" (that
+ * conflation silently downgraded `vat skill review` and would let `vat skill
+ * test` stage the wrong artifact). Both outcomes are cached (the error too) so a
+ * broken config re-throws without re-parsing on every skill in a scan. A test
+ * that edits a broken config into a good one between calls must invoke
  * {@link resetLoadedConfigCache} (audit's `resetAuditCaches()` does this).
+ *
+ * Callers that intentionally tolerate a broken config (e.g. `vat audit`, a bulk
+ * linter that should still validate the skill itself) catch `ConfigLoadError`.
  */
 export function loadConfigCached(projectRoot) {
+    const cachedError = loadErrorCache.get(projectRoot);
+    if (cachedError !== undefined)
+        throw cachedError;
     const cached = loadedConfigCache.get(projectRoot);
     if (cached !== undefined)
         return cached ?? undefined;
@@ -101,10 +137,12 @@ export function loadConfigCached(projectRoot) {
         loadedConfigCache.set(projectRoot, config ?? null);
         return config;
     }
-    catch {
-        // Cache failures as null so a broken config doesn't re-parse per skill.
-        loadedConfigCache.set(projectRoot, null);
-        return undefined;
+    catch (err) {
+        // loadConfig returns undefined when the file is ABSENT and only throws when
+        // it EXISTS but is broken — so reaching here means a genuinely broken config.
+        const configErr = err instanceof ConfigLoadError ? err : new ConfigLoadError(projectRoot, err);
+        loadErrorCache.set(projectRoot, configErr);
+        throw configErr;
     }
 }
 //# sourceMappingURL=config-loader.js.map

package/dist/utils/config-loader.js.map

@@ -1 +1 @@
-{"version":3,"file":"config-loader.js","sourceRoot":"","sources":["../../src/utils/config-loader.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,EAAE,YAAY,EAAE,UAAU,EAAE,MAAM,SAAS,CAAC;AACnD,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAEpC,OAAO,EAAE,mBAAmB,EAAsB,MAAM,+BAA+B,CAAC;AACxF,OAAO,EAAE,QAAQ,EAAE,MAAM,2BAA2B,CAAC;AACrD,OAAO,KAAK,IAAI,MAAM,MAAM,CAAC;AAE7B,MAAM,eAAe,GAAG,gCAAgC,CAAC;AAEzD;;;;;;;;;;;;;;;;;;;;GAoBG;AACH,MAAM,UAAU,UAAU,CAAC,WAAmB;IAC5C,sEAAsE;IACtE,MAAM,UAAU,GAAG,OAAO,CAAC,GAAG,CAAC,iBAAiB,CAAC,IAAI,QAAQ,CAAC,IAAI,CAAC,WAAW,EAAE,eAAe,CAAC,CAAC;IAEjG,uIAAuI;IACvI,IAAI,CAAC,UAAU,CAAC,UAAU,CAAC,EAAE,CAAC;QAC5B,OAAO,SAAS,CAAC;IACnB,CAAC;IAED,IAAI,CAAC;QACH,uHAAuH;QACvH,MAAM,OAAO,GAAG,YAAY,CAAC,UAAU,EAAE,OAAO,CAAC,CAAC;QAClD,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;QAEnC,wDAAwD;QACxD,MAAM,MAAM,GAAG,mBAAmB,CAAC,SAAS,CAAC,MAAM,CAAC,CAAC;QAErD,IAAI,CAAC,MAAM,CAAC,OAAO,EAAE,CAAC;YACpB,MAAM,IAAI,KAAK,CACb,+BAA+B,MAAM,CAAC,KAAK,CAAC,OAAO,EAAE,CACtD,CAAC;QACJ,CAAC;QAED,OAAO,MAAM,CAAC,IAAI,CAAC;IACrB,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,IAAI,KAAK,YAAY,KAAK,EAAE,CAAC;YAC3B,MAAM,IAAI,KAAK,CAAC,0BAA0B,KAAK,CAAC,OAAO,EAAE,CAAC,CAAC;QAC7D,CAAC;QACD,MAAM,KAAK,CAAC;IACd,CAAC;AACH,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,YAAY,CAAC,UAAkB;IAC7C,OAAO,OAAO,CAAC,UAAU,CAAC,CAAC;AAC7B,CAAC;AAED;;;;;;;;;;;GAWG;AACH,MAAM,iBAAiB,GAAsC,IAAI,GAAG,EAAE,CAAC;AAEvE;;;;;GAKG;AACH,MAAM,UAAU,sBAAsB;IACpC,iBAAiB,CAAC,KAAK,EAAE,CAAC;AAC5B,CAAC;AAED;;;;;;;GAOG;AACH,MAAM,UAAU,gBAAgB,CAAC,WAAmB;IAClD,MAAM,MAAM,GAAG,iBAAiB,CAAC,GAAG,CAAC,WAAW,CAAC,CAAC;IAClD,IAAI,MAAM,KAAK,SAAS;QAAE,OAAO,MAAM,IAAI,SAAS,CAAC;IACrD,IAAI,CAAC;QACH,MAAM,MAAM,GAAG,UAAU,CAAC,WAAW,CAAC,CAAC;QACvC,iBAAiB,CAAC,GAAG,CAAC,WAAW,EAAE,MAAM,IAAI,IAAI,CAAC,CAAC;QACnD,OAAO,MAAM,CAAC;IAChB,CAAC;IAAC,MAAM,CAAC;QACP,wEAAwE;QACxE,iBAAiB,CAAC,GAAG,CAAC,WAAW,EAAE,IAAI,CAAC,CAAC;QACzC,OAAO,SAAS,CAAC;IACnB,CAAC;AACH,CAAC"}
+{"version":3,"file":"config-loader.js","sourceRoot":"","sources":["../../src/utils/config-loader.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,EAAE,YAAY,EAAE,UAAU,EAAE,MAAM,SAAS,CAAC;AACnD,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAEpC,OAAO,EAAE,mBAAmB,EAAsB,MAAM,+BAA+B,CAAC;AACxF,OAAO,EAAE,QAAQ,EAAE,MAAM,2BAA2B,CAAC;AACrD,OAAO,KAAK,IAAI,MAAM,MAAM,CAAC;AAE7B,MAAM,eAAe,GAAG,gCAAgC,CAAC;AAEzD;;;;;;;;GAQG;AACH,MAAM,OAAO,eAAgB,SAAQ,KAAK;IAC/B,WAAW,CAAS;IAC7B,YAAY,WAAmB,EAAE,KAAc;QAC7C,KAAK,CAAC,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC;QAC9D,IAAI,CAAC,IAAI,GAAG,iBAAiB,CAAC;QAC9B,IAAI,CAAC,WAAW,GAAG,WAAW,CAAC;QAC/B,IAAI,KAAK,YAAY,KAAK;YAAE,IAAI,CAAC,KAAK,GAAG,KAAK,CAAC;IACjD,CAAC;CACF;AAED;;;;;;;;;;;;;;;;;;;;GAoBG;AACH,MAAM,UAAU,UAAU,CAAC,WAAmB;IAC5C,sEAAsE;IACtE,MAAM,UAAU,GAAG,OAAO,CAAC,GAAG,CAAC,iBAAiB,CAAC,IAAI,QAAQ,CAAC,IAAI,CAAC,WAAW,EAAE,eAAe,CAAC,CAAC;IAEjG,uIAAuI;IACvI,IAAI,CAAC,UAAU,CAAC,UAAU,CAAC,EAAE,CAAC;QAC5B,OAAO,SAAS,CAAC;IACnB,CAAC;IAED,IAAI,CAAC;QACH,uHAAuH;QACvH,MAAM,OAAO,GAAG,YAAY,CAAC,UAAU,EAAE,OAAO,CAAC,CAAC;QAClD,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;QAEnC,wDAAwD;QACxD,MAAM,MAAM,GAAG,mBAAmB,CAAC,SAAS,CAAC,MAAM,CAAC,CAAC;QAErD,IAAI,CAAC,MAAM,CAAC,OAAO,EAAE,CAAC;YACpB,MAAM,IAAI,KAAK,CACb,+BAA+B,MAAM,CAAC,KAAK,CAAC,OAAO,EAAE,CACtD,CAAC;QACJ,CAAC;QAED,OAAO,MAAM,CAAC,IAAI,CAAC;IACrB,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,IAAI,KAAK,YAAY,KAAK,EAAE,CAAC;YAC3B,MAAM,IAAI,KAAK,CAAC,0BAA0B,KAAK,CAAC,OAAO,EAAE,CAAC,CAAC;QAC7D,CAAC;QACD,MAAM,KAAK,CAAC;IACd,CAAC;AACH,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,YAAY,CAAC,UAAkB;IAC7C,OAAO,OAAO,CAAC,UAAU,CAAC,CAAC;AAC7B,CAAC;AAED;;;;;;;;;;;GAWG;AACH,MAAM,iBAAiB,GAAsC,IAAI,GAAG,EAAE,CAAC;AAEvE;;;;GAIG;AACH,MAAM,cAAc,GAAiC,IAAI,GAAG,EAAE,CAAC;AAE/D;;;;;GAKG;AACH,MAAM,UAAU,sBAAsB;IACpC,iBAAiB,CAAC,KAAK,EAAE,CAAC;IAC1B,cAAc,CAAC,KAAK,EAAE,CAAC;AACzB,CAAC;AAED;;;;;;;;;;;;;;GAcG;AACH,MAAM,UAAU,gBAAgB,CAAC,WAAmB;IAClD,MAAM,WAAW,GAAG,cAAc,CAAC,GAAG,CAAC,WAAW,CAAC,CAAC;IACpD,IAAI,WAAW,KAAK,SAAS;QAAE,MAAM,WAAW,CAAC;IACjD,MAAM,MAAM,GAAG,iBAAiB,CAAC,GAAG,CAAC,WAAW,CAAC,CAAC;IAClD,IAAI,MAAM,KAAK,SAAS;QAAE,OAAO,MAAM,IAAI,SAAS,CAAC;IACrD,IAAI,CAAC;QACH,MAAM,MAAM,GAAG,UAAU,CAAC,WAAW,CAAC,CAAC;QACvC,iBAAiB,CAAC,GAAG,CAAC,WAAW,EAAE,MAAM,IAAI,IAAI,CAAC,CAAC;QACnD,OAAO,MAAM,CAAC;IAChB,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,4EAA4E;QAC5E,8EAA8E;QAC9E,MAAM,SAAS,GAAG,GAAG,YAAY,eAAe,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,eAAe,CAAC,WAAW,EAAE,GAAG,CAAC,CAAC;QAC/F,cAAc,CAAC,GAAG,CAAC,WAAW,EAAE,SAAS,CAAC,CAAC;QAC3C,MAAM,SAAS,CAAC;IAClB,CAAC;AACH,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vibe-agent-toolkit/cli",
-  "version": "0.1.39-rc.8",
+  "version": "0.1.39",
   "description": "Command-line interface for vibe-agent-toolkit",
   "type": "module",
   "bin": {
@@ -41,16 +41,16 @@
   "//dependencies-note": "DO NOT add example agent packages or adopter packages here. Adopter packages (like vat-development-agents) depend on the CLI binary at build time — declaring them as deps creates a circular build dependency. They are built in a second turbo pass via package#task overrides in turbo.json.",
   "dependencies": {
     "@anthropic-ai/sdk": "^0.71.2",
-    "@vibe-agent-toolkit/agent-config": "0.1.39-rc.8",
-    "@vibe-agent-toolkit/agent-schema": "0.1.39-rc.8",
-    "@vibe-agent-toolkit/agent-skills": "0.1.39-rc.8",
-    "@vibe-agent-toolkit/claude-marketplace": "0.1.39-rc.8",
-    "@vibe-agent-toolkit/discovery": "0.1.39-rc.8",
-    "@vibe-agent-toolkit/gateway-mcp": "0.1.39-rc.8",
-    "@vibe-agent-toolkit/rag": "0.1.39-rc.8",
-    "@vibe-agent-toolkit/rag-lancedb": "0.1.39-rc.8",
-    "@vibe-agent-toolkit/resources": "0.1.39-rc.8",
-    "@vibe-agent-toolkit/utils": "0.1.39-rc.8",
+    "@vibe-agent-toolkit/agent-config": "0.1.39",
+    "@vibe-agent-toolkit/agent-schema": "0.1.39",
+    "@vibe-agent-toolkit/agent-skills": "0.1.39",
+    "@vibe-agent-toolkit/claude-marketplace": "0.1.39",
+    "@vibe-agent-toolkit/discovery": "0.1.39",
+    "@vibe-agent-toolkit/gateway-mcp": "0.1.39",
+    "@vibe-agent-toolkit/rag": "0.1.39",
+    "@vibe-agent-toolkit/rag-lancedb": "0.1.39",
+    "@vibe-agent-toolkit/resources": "0.1.39",
+    "@vibe-agent-toolkit/utils": "0.1.39",
     "adm-zip": "^0.5.16",
     "commander": "^12.1.0",
     "picomatch": "^4.0.3",