substrate-ai 0.2.29 → 0.2.31

package/dist/run-CMagiY2Y.js

@@ -0,0 +1,8 @@
+import { registerRunCommand, runRunAction } from "./run-BxXwD2xY.js";
+import "./logger-D2fS2ccL.js";
+import "./config-migrator-DSi8KhQC.js";
+import "./helpers-RL22dYtn.js";
+import "./decisions-Dq4cAA2L.js";
+import "./operational-Bovj4fS-.js";
+
+export { runRunAction };

package/dist/{upgrade-DzpjKYlD.js → upgrade-CImByfkk.js}

@@ -1,4 +1,5 @@
-import "./version-manager-impl-CtzNu7YZ.js";
-import { isGlobalInstall, registerUpgradeCommand, runUpgradeCommand } from "./upgrade-CJ0JFQ2c.js";
+import "./config-migrator-DSi8KhQC.js";
+import "./version-manager-impl-CizNmmLT.js";
+import { isGlobalInstall, registerUpgradeCommand, runUpgradeCommand } from "./upgrade-Cvwtnwl4.js";
 
 export { isGlobalInstall, registerUpgradeCommand, runUpgradeCommand };

package/dist/{upgrade-CJ0JFQ2c.js → upgrade-Cvwtnwl4.js}

@@ -1,4 +1,4 @@
-import { createVersionManager } from "./version-manager-impl-CtzNu7YZ.js";
+import { createVersionManager } from "./version-manager-impl-CizNmmLT.js";
 import { execSync, spawn } from "child_process";
 import * as readline from "readline";
 
@@ -123,4 +123,4 @@ function registerUpgradeCommand(program) {
 
 //#endregion
 export { isGlobalInstall, registerUpgradeCommand, runUpgradeCommand };
-//# sourceMappingURL=upgrade-CJ0JFQ2c.js.map
+//# sourceMappingURL=upgrade-Cvwtnwl4.js.map

package/dist/{version-manager-impl-CtzNu7YZ.js → version-manager-impl-CizNmmLT.js}

@@ -1,252 +1,13 @@
+import { SUPPORTED_CONFIG_FORMAT_VERSIONS, SUPPORTED_TASK_GRAPH_VERSIONS, defaultConfigMigrator } from "./config-migrator-DSi8KhQC.js";
 import { createRequire } from "module";
 import { fileURLToPath } from "url";
 import path, { dirname, resolve } from "path";
 import { mkdirSync, readFileSync, writeFileSync } from "fs";
-import { z } from "zod";
 import os from "os";
 import * as semver$1 from "semver";
 import * as semver from "semver";
 import https from "https";
 
-//#region src/modules/config/config-schema.ts
-/** Subscription routing modes */
-const SubscriptionRoutingSchema = z.enum([
-	"auto",
-	"subscription",
-	"api",
-	"disabled"
-]);
-/** Rate limit configuration for a provider */
-const RateLimitSchema = z.object({
-	tokens: z.number().int().positive(),
-	window_seconds: z.number().int().positive()
-}).strict();
-/** Per-provider configuration */
-const ProviderConfigSchema = z.object({
-	enabled: z.boolean(),
-	cli_path: z.string().optional(),
-	subscription_routing: SubscriptionRoutingSchema,
-	max_concurrent: z.number().int().min(1).max(32),
-	rate_limit: RateLimitSchema.optional(),
-	api_key_env: z.string().optional(),
-	api_billing: z.boolean()
-}).strict();
-/** Map of all known providers */
-const ProvidersSchema = z.object({
-	claude: ProviderConfigSchema.optional(),
-	codex: ProviderConfigSchema.optional(),
-	gemini: ProviderConfigSchema.optional()
-}).strict();
-const LogLevelSchema = z.enum([
-	"trace",
-	"debug",
-	"info",
-	"warn",
-	"error",
-	"fatal"
-]);
-const GlobalSettingsSchema = z.object({
-	log_level: LogLevelSchema,
-	max_concurrent_tasks: z.number().int().min(1).max(64),
-	budget_cap_tokens: z.number().int().min(0),
-	budget_cap_usd: z.number().min(0),
-	workspace_dir: z.string().optional(),
-	update_check: z.boolean().optional()
-}).strict();
-const CostTrackerConfigSchema = z.object({
-	enabled: z.boolean(),
-	token_rates_provider: z.enum(["builtin", "custom"]),
-	track_planning_costs: z.boolean(),
-	savings_reporting: z.boolean()
-}).strict();
-const BudgetConfigSchema = z.object({
-	default_task_budget_usd: z.number().min(0),
-	default_session_budget_usd: z.number().min(0),
-	planning_costs_count_against_budget: z.boolean(),
-	warning_threshold_percent: z.number().min(0).max(100)
-}).strict();
-const RoutingRuleSchema = z.object({
-	task_type: z.string(),
-	preferred_provider: z.string(),
-	fallback_providers: z.array(z.string())
-}).strict();
-const RoutingPolicySchema = z.object({
-	default_provider: z.string(),
-	rules: z.array(RoutingRuleSchema)
-}).strict();
-/**
-* Per-workflow token ceiling overrides.
-* Keys match the workflow type names used in prompts and events.
-* Values must be positive integers.
-*/
-const TokenCeilingsSchema = z.object({
-	"create-story": z.number().int().positive("create-story token ceiling must be a positive integer").optional(),
-	"dev-story": z.number().int().positive("dev-story token ceiling must be a positive integer").optional(),
-	"code-review": z.number().int().positive("code-review token ceiling must be a positive integer").optional(),
-	"test-plan": z.number().int().positive("test-plan token ceiling must be a positive integer").optional(),
-	"test-expansion": z.number().int().positive("test-expansion token ceiling must be a positive integer").optional()
-});
-/** Current supported config format version */
-const CURRENT_CONFIG_FORMAT_VERSION = "1";
-/** Current supported task graph version */
-const CURRENT_TASK_GRAPH_VERSION = "1";
-/** All config format versions this toolkit can read and validate */
-const SUPPORTED_CONFIG_FORMAT_VERSIONS = ["1"];
-/** All task graph format versions this toolkit can read and validate */
-const SUPPORTED_TASK_GRAPH_VERSIONS = ["1"];
-const SubstrateConfigSchema = z.object({
-	config_format_version: z.enum(["1"]),
-	task_graph_version: z.enum(["1"]).optional(),
-	global: GlobalSettingsSchema,
-	providers: ProvidersSchema,
-	cost_tracker: CostTrackerConfigSchema.optional(),
-	budget: BudgetConfigSchema.optional(),
-	token_ceilings: TokenCeilingsSchema.optional()
-}).strict();
-const PartialProviderConfigSchema = ProviderConfigSchema.partial();
-const PartialGlobalSettingsSchema = GlobalSettingsSchema.partial();
-const PartialSubstrateConfigSchema = z.object({
-	config_format_version: z.enum(["1"]).optional(),
-	task_graph_version: z.enum(["1"]).optional(),
-	global: PartialGlobalSettingsSchema.optional(),
-	providers: z.object({
-		claude: PartialProviderConfigSchema.optional(),
-		codex: PartialProviderConfigSchema.optional(),
-		gemini: PartialProviderConfigSchema.optional()
-	}).partial().optional(),
-	cost_tracker: CostTrackerConfigSchema.partial().optional(),
-	budget: BudgetConfigSchema.partial().optional(),
-	token_ceilings: TokenCeilingsSchema.optional()
-}).strict();
-
-//#endregion
-//#region src/modules/config/config-migrator.ts
-/**
-* ConfigMigrator manages a registry of migration functions and applies them
-* sequentially to upgrade config documents from one format version to another.
-*/
-var ConfigMigrator = class {
-	migrations = new Map();
-	/**
-	* Register a migration function for the given version key.
-	*
-	* @param key - Migration key in format "N->M" (e.g. "1->2")
-	* @param fn - Migration function that receives the raw config and returns the migrated config
-	*/
-	register(key, fn) {
-		this.migrations.set(key, fn);
-	}
-	/**
-	* Check whether a sequential migration path exists from fromVersion to toVersion.
-	*
-	* @param fromVersion - Starting version string
-	* @param toVersion - Target version string
-	* @returns true if every step in the path is registered
-	*/
-	canMigrate(fromVersion, toVersion) {
-		if (fromVersion === toVersion) return true;
-		const from = parseInt(fromVersion, 10);
-		const to = parseInt(toVersion, 10);
-		if (isNaN(from) || isNaN(to) || from >= to) return false;
-		for (let v = from; v < to; v++) {
-			const key = `${String(v)}->${String(v + 1)}`;
-			if (!this.migrations.has(key)) return false;
-		}
-		return true;
-	}
-	/**
-	* Apply sequential migrations from fromVersion to toVersion.
-	*
-	* If fromVersion === toVersion, returns a no-op success result.
-	* If any intermediate migration is missing, returns success:false.
-	*
-	* When filePath is provided and migration is needed, a backup is written to
-	* `${filePath}.bak.v${fromVersion}` before any transformations are applied.
-	*
-	* @param config - Raw config object to migrate
-	* @param fromVersion - Starting format version string
-	* @param toVersion - Target format version string
-	* @param filePath - Optional path to the source config file for backup creation
-	* @returns Object containing the (possibly migrated) config and a MigrationResult
-	*/
-	migrate(config, fromVersion, toVersion, filePath) {
-		if (fromVersion === toVersion) return {
-			config,
-			result: {
-				success: true,
-				fromVersion,
-				toVersion,
-				migratedKeys: [],
-				manualStepsRequired: [],
-				backupPath: null
-			}
-		};
-		const from = parseInt(fromVersion, 10);
-		const to = parseInt(toVersion, 10);
-		if (isNaN(from) || isNaN(to) || from >= to) return {
-			config,
-			result: {
-				success: false,
-				fromVersion,
-				toVersion,
-				migratedKeys: [],
-				manualStepsRequired: [`Cannot migrate from version "${fromVersion}" to "${toVersion}": invalid version range.`],
-				backupPath: null
-			}
-		};
-		for (let v = from; v < to; v++) {
-			const key = `${String(v)}->${String(v + 1)}`;
-			if (!this.migrations.has(key)) return {
-				config,
-				result: {
-					success: false,
-					fromVersion,
-					toVersion,
-					migratedKeys: [],
-					manualStepsRequired: [`Missing migration step: "${key}". Cannot automatically migrate from version "${fromVersion}" to "${toVersion}". Please upgrade the toolkit: npm install -g substrate@latest`],
-					backupPath: null
-				}
-			};
-		}
-		let backupPath = null;
-		if (filePath !== void 0) {
-			backupPath = `${filePath}.bak.v${fromVersion}`;
-			const originalContent = readFileSync(filePath, "utf-8");
-			writeFileSync(backupPath, originalContent, "utf-8");
-		}
-		let current = config;
-		const migratedKeys = [];
-		for (let v = from; v < to; v++) {
-			const key = `${String(v)}->${String(v + 1)}`;
-			const fn = this.migrations.get(key);
-			const before = JSON.stringify(current);
-			current = fn(current);
-			const after = JSON.stringify(current);
-			if (current !== null && typeof current === "object" && !Array.isArray(current)) {
-				const beforeObj = JSON.parse(before);
-				const afterObj = JSON.parse(after);
-				for (const k of Object.keys(afterObj)) if (JSON.stringify(afterObj[k]) !== JSON.stringify(beforeObj[k])) {
-					if (!migratedKeys.includes(k)) migratedKeys.push(k);
-				}
-			}
-		}
-		return {
-			config: current,
-			result: {
-				success: true,
-				fromVersion,
-				toVersion,
-				migratedKeys,
-				manualStepsRequired: [],
-				backupPath
-			}
-		};
-	}
-};
-/** Singleton instance for use throughout the toolkit */
-const defaultConfigMigrator = new ConfigMigrator();
-
-//#endregion
 //#region src/modules/version-manager/update-checker.ts
 /**
 * Thrown when an update check fails due to network error, timeout, or bad response.
@@ -607,5 +368,5 @@ function createVersionManager(deps = {}) {
 }
 
 //#endregion
-export { CURRENT_CONFIG_FORMAT_VERSION, CURRENT_TASK_GRAPH_VERSION, PartialSubstrateConfigSchema, SUPPORTED_CONFIG_FORMAT_VERSIONS, SubstrateConfigSchema, VersionManagerImpl, createVersionManager, defaultConfigMigrator };
-//# sourceMappingURL=version-manager-impl-CtzNu7YZ.js.map
+export { VersionManagerImpl, createVersionManager };
+//# sourceMappingURL=version-manager-impl-CizNmmLT.js.map

package/dist/version-manager-impl-aL5IemIm.js

@@ -0,0 +1,4 @@
+import "./config-migrator-DSi8KhQC.js";
+import { VersionManagerImpl, createVersionManager } from "./version-manager-impl-CizNmmLT.js";
+
+export { createVersionManager };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "substrate-ai",
-  "version": "0.2.29",
+  "version": "0.2.31",
   "description": "Substrate — multi-agent orchestration daemon for AI coding agents",
   "type": "module",
   "license": "MIT",

package/packs/bmad/manifest.yaml

@@ -221,6 +221,8 @@ prompts:
   refine-artifact: prompts/refine-artifact.md
   # Readiness check prompt (Story 16-6)
   readiness-check: prompts/readiness-check.md
+  # Test plan prompt (Story 25-7)
+  test-plan: prompts/test-plan.md
 
 constraints:
   create-story: constraints/create-story.yaml

package/packs/bmad/prompts/code-review.md

@@ -100,6 +100,11 @@ ac_checklist:
 **IMPORTANT**: `ac_checklist` must contain one entry for every AC found in the story. If the story has no parseable ACs (e.g. a refactoring story), `ac_checklist` may be an empty array.
 
 **Verdict rules:**
-- `SHIP_IT` — zero blocker/major issues (minor issues acceptable)
-- `NEEDS_MINOR_FIXES` — minor issues only, or 1-2 major with no blockers
+- `SHIP_IT` — zero issues of any kind
+- `LGTM_WITH_NOTES` — zero correctness/logic/security issues; only advisory or style observations that do not need to be fixed before shipping. Use this when you have optional suggestions but the code is production-ready as-is. Include your suggestions in the `notes` field.
+- `NEEDS_MINOR_FIXES` — one or more minor issues that should be fixed, or 1-2 major issues with no blockers
 - `NEEDS_MAJOR_REWORK` — any blocker issue, or 3+ major issues
+
+**LGTM_WITH_NOTES vs NEEDS_MINOR_FIXES:**
+- Use `LGTM_WITH_NOTES` when: all findings are purely advisory (naming preferences, optional refactors, docs suggestions) and the code ships safely without any changes
+- Use `NEEDS_MINOR_FIXES` when: any finding represents a real gap that should be corrected before the story is considered done (missing error handling, incomplete AC coverage, confusing logic)

package/packs/bmad/prompts/create-story.md

@@ -35,6 +35,26 @@ Using the context above, write a complete, implementation-ready story file for s
    - Status must be: `ready-for-dev`
    - Dev Agent Record section must be present but left blank (to be filled by dev agent)
 
+## Interface Contracts Guidance
+
+**Identify cross-story dependencies** when the story creates or consumes shared schemas, interfaces, or message contracts.
+
+If the story exports (creates) or imports (consumes from another story) any TypeScript interfaces, Zod schemas, message queue contracts, or API types that are shared across module boundaries, add an `## Interface Contracts` section to the story file.
+
+Use this exact format for each item:
+
+```markdown
+## Interface Contracts
+
+- **Export**: SchemaName @ src/path/to/file.ts (queue: some-queue-name)
+- **Import**: SchemaName @ src/path/to/file.ts (from story 25-X)
+```
+
+- `Export` = this story creates/defines the schema that other stories will consume
+- `Import` = this story consumes a schema defined by another story
+- The transport annotation `(queue: ...)` or `(api: ...)` or `(from story X-Y)` is optional but recommended when applicable
+- **The `## Interface Contracts` section is optional** — omit it entirely if the story has no cross-story schema dependencies
+
 ## Scope Cap Guidance
 
 **Aim for 6-7 acceptance criteria and 7-8 tasks per story.**

package/packs/bmad/prompts/test-plan.md

@@ -2,24 +2,30 @@
 
 ## Mission
 
-Analyze the story's Acceptance Criteria and tasks. Produce a concrete test plan listing the test files to create, the test categories to cover (unit/integration/e2e), and a brief note on AC coverage.
+Analyze the story's Acceptance Criteria and tasks. Produce a concrete test plan listing the test files to create, the test categories to cover (unit/integration/e2e), and comprehensive coverage notes that include: which ACs each test covers, dependencies to mock, and error conditions to assert.
 
 ## Story Content
 
 {{story_content}}
 
+## Architecture Constraints
+
+{{architecture_constraints}}
+
 ## Instructions
 
 1. Read the Acceptance Criteria (AC1, AC2, etc.) and Tasks in the story above.
 2. Identify the source files that will need tests (from Dev Notes, Key File Paths, and Tasks).
 3. For each AC, determine which test file and test function will validate it.
-4. Produce a concise test plan — one or two test files is typical for small stories.
+4. Identify all external dependencies (modules, services, fs, db) that must be mocked or stubbed.
+5. Identify error conditions and edge cases that must be asserted (not just the happy path).
+6. Produce a concise test plan — one or two test files is typical for small stories.
 
 **Rules:**
 - List only test files that will be NEW (not existing ones you'd extend unless necessary).
 - Use the project's test path convention: `src/modules/<module>/__tests__/<file>.test.ts`
 - Test categories: `unit` for isolated function tests, `integration` for multi-module tests, `e2e` for full pipeline tests.
-- Keep `coverage_notes` brief — one sentence per AC is sufficient.
+- `coverage_notes` must include: (a) which test file covers each AC, (b) dependencies to mock (e.g., `vi.mock('node:fs/promises')`), and (c) error conditions to assert (e.g., missing file, schema validation failure, timeout).
 
 ## Output Contract
 
@@ -31,7 +37,7 @@ test_files:
 test_categories:
   - unit
   - integration
-coverage_notes: "AC1 covered by foo.test.ts describe('runFoo'). AC2 covered by..."
+coverage_notes: "AC1: foo.test.ts describe('runFoo') covers the happy path. AC2: same file covers error path (rejects on ENOENT). Mocks needed: vi.mock('node:fs/promises'), vi.mock('../db.js'). Error conditions: file not found returns failed result, schema parse error returns failed with details, timeout triggers fallback."
 
 If you cannot produce a plan (e.g., story content is missing or unreadable), emit:
 

package/dist/run-DFFZVzZK.js

@@ -1,7 +0,0 @@
-import { registerRunCommand, runRunAction } from "./run-IeDyGCak.js";
-import "./logger-D2fS2ccL.js";
-import "./helpers-DljGJnFF.js";
-import "./decisions-Dq4cAA2L.js";
-import "./operational-CnMlvWqc.js";
-
-export { runRunAction };

package/dist/version-manager-impl-BDfiGXWX.js

@@ -1,3 +0,0 @@
-import { VersionManagerImpl, createVersionManager } from "./version-manager-impl-CtzNu7YZ.js";
-
-export { createVersionManager };