@wundam/orchex 1.0.0-rc.27 → 1.0.0-rc.28

package/dist/artifacts.js

@@ -420,6 +420,10 @@ export async function writeArtifact(projectDir, streamId, artifact) {
     await fs.writeFile(artifactPath, JSON.stringify(artifact, null, 2), 'utf-8');
     return artifactPath;
 }
+// Artifact-application trust boundary. Plan-level ownership is enforced
+// upstream by validatePlan (src/intelligence/plan-contract.ts); this
+// function guards the different trust boundary where LLM-generated
+// artifacts try to apply their output.
 /**
  * Check whether all file operations fall within the stream's owns patterns.
  *

package/dist/cloud-executor.js

@@ -1,5 +1,17 @@
 import { extractArtifact } from './artifacts.js';
 import { validateOwnership } from './utils/ownership-validator.js';
+function parseProviderErrorBody(body) {
+    try {
+        const parsed = JSON.parse(body);
+        const message = parsed?.error?.message ?? parsed?.message;
+        if (typeof message === 'string')
+            return message;
+    }
+    catch {
+        // Not JSON — return null
+    }
+    return null;
+}
 /**
  * Returns actionable error messages for cloud API failures.
  * Status-code-specific guidance tells users exactly how to fix each error.
@@ -18,6 +30,16 @@ function formatCloudError(status, body) {
             const modelHint = modelMatch ? ` "${modelMatch[1]}"` : '';
             return `Model${modelHint} not found (404)${detail}.\n\nRun \`orchex config --model <model>\` to use a different model.`;
         }
+        case 400: {
+            const providerMsg = parseProviderErrorBody(body);
+            if (providerMsg && /credit balance|insufficient|billing/i.test(providerMsg)) {
+                return `LLM provider billing error (400): ${providerMsg}\n\nTop up credits at your provider's billing page, or switch providers with \`orchex config --provider <provider>\`.`;
+            }
+            if (providerMsg) {
+                return `LLM provider error (400): ${providerMsg}`;
+            }
+            return `Cloud API error: 400${detail}`;
+        }
         case 429:
             return `Quota exceeded (429)${detail}.\n\nYou've used all cloud runs for this period. Upgrade at https://orchex.dev/pricing`;
         default:
@@ -180,11 +202,18 @@ export class CloudExecutor {
                         };
                     }
                     if (job.status === 'failed') {
+                        const rawError = job.error ?? 'Cloud execution failed';
+                        const providerMsg = parseProviderErrorBody(rawError);
+                        const formattedError = providerMsg
+                            ? (/credit balance|insufficient|billing/i.test(providerMsg)
+                                ? `LLM provider billing error: ${providerMsg}\n\nTop up credits at your provider's billing page, or switch providers with \`orchex config --provider <provider>\`.`
+                                : `LLM provider error: ${providerMsg}`)
+                            : rawError;
                         return {
                             success: false,
                             rawResponse: job.output ?? '',
                             tokensUsed: job.tokensUsed ?? { input: 0, output: 0 },
-                            error: job.error ?? 'Cloud execution failed',
+                            error: formattedError,
                         };
                     }
                     // Job still pending/running - wait with adaptive interval

package/dist/index.js

@@ -6,6 +6,7 @@ import { ORCHEX_INSTRUCTIONS } from './mcp-instructions.js';
 import { registerResources } from './mcp-resources.js';
 import { broadcaster } from './execution-broadcaster.js';
 import { loadConfig, saveConfig, maskConfigForDisplay, resolveApiUrl, PRODUCTION_URL, LLMProviderSchema } from './config.js';
+import { analyzeError } from './intelligence/index.js';
 import { buildVerificationMessage, buildStatusMessage, parseLoginApiResponse, } from './login-helpers.js';
 /** Opens browser cross-platform. Errors are silently ignored — URL already printed to terminal. */
 function openBrowser(url) {
@@ -619,6 +620,17 @@ async function handleRunCommand(args) {
     }
     // Init and execute
     console.log(`\nInitializing orchestration: "${plan.title}"\n`);
+    // Plan-contract boundary (Session 13): fail fast on invalid streams before
+    // they reach the legacy 5-layer validation. Additive — existing layers stay
+    // in place as defense-in-depth until Session 14 consolidates them.
+    {
+        const { validatePlan, formatPlanValidationErrors } = await import('./intelligence/index.js');
+        const validation = validatePlan(streamDefs, projectDir);
+        if (!validation.ok) {
+            console.error('Plan validation failed:\n' + formatPlanValidationErrors(validation.errors));
+            process.exit(1);
+        }
+    }
     await initOrchestration(projectDir, plan.title, streamDefs);
     const allResponses = [];
     let executionError;
@@ -638,9 +650,13 @@ async function handleRunCommand(args) {
                         process.stdout.write(`  ✓ ${event.data.streamId} — complete${dur}\n`);
                         break;
                     }
-                    case 'stream_failed':
-                        process.stdout.write(`  ✗ ${event.data.streamId} — failed: ${String(event.data.error).slice(0, 100)}\n`);
+                    case 'stream_failed': {
+                        const errStr = String(event.data.error);
+                        const analysis = analyzeError(errStr);
+                        const categoryTag = analysis.category !== 'unknown' ? ` [${analysis.category}]` : '';
+                        process.stdout.write(`  ✗ ${event.data.streamId} — failed${categoryTag}: ${errStr.slice(0, 200)}\n`);
                         break;
+                    }
                     case 'stream_rate_limited':
                         process.stdout.write(`  ⏳ ${event.data.streamId} — rate limited (retry in ${Math.round((event.data.retryAfterMs ?? 0) / 1000)}s)\n`);
                         break;

package/dist/intelligence/index.d.ts

@@ -1,10 +1,12 @@
-export { gatherProjectContext, generatePlan } from './auto-planner.js';
+export { gatherProjectContext, generatePlan, extractSpecPaths } from './auto-planner.js';
 export { parsePlanDocument, getSectionsAtLevel, isUnpopulatedTemplate } from './plan-parser.js';
 export { extractDeliverables, processDeliverables, formatDeliverablesReport } from './deliverable-extractor.js';
 export type { Deliverable } from './deliverable-extractor.js';
 export { buildDependencyGraph, formatDependencyReport } from './dependency-inferrer.js';
 export { generateStreams, formatStreamsForReview, toInitFormat, extractPrerequisites } from './stream-generator.js';
-export { formatPlanPreview, formatPlanPreviewText, type ModelDecision } from './plan-preview.js';
+export { validatePlan, formatPlanValidationErrors, StreamDefinitionSchema as PlanContractStreamSchema } from './plan-contract.js';
+export type { PlanValidationError, ValidatePlanResult, StreamDefinition as PlanContractStream } from './plan-contract.js';
+export { formatPlanPreview, formatPlanPreviewText, generatePlanPreview, type ModelDecision, type PlanPreview, type GeneratePlanPreviewInput, type GeneratePlanPreviewResult, } from './plan-preview.js';
 export { detectSequentialEdits, autoFixSequentialEdits } from './sequential-diagnostics.js';
 export type { SequentialEditDiagnostic } from './sequential-diagnostics.js';
 export { createDiagnostics, detectOwnershipConflicts } from './diagnostics.js';