ai-spec-dev 0.28.1 → 0.29.1

package/dist/cli/index.mjs

@@ -21,8 +21,7 @@ __export(codegen_prompt_exports, {
   getCodeGenSystemPrompt: () => getCodeGenSystemPrompt,
   reviewArchitectureSystemPrompt: () => reviewArchitectureSystemPrompt,
   reviewImpactComplexitySystemPrompt: () => reviewImpactComplexitySystemPrompt,
-  reviewImplementationSystemPrompt: () => reviewImplementationSystemPrompt,
-  reviewSystemPrompt: () => reviewSystemPrompt
+  reviewImplementationSystemPrompt: () => reviewImplementationSystemPrompt
 });
 function getCodeGenSystemPrompt(repoType) {
   switch (repoType) {
@@ -40,7 +39,7 @@ function getCodeGenSystemPrompt(repoType) {
       return codeGenSystemPrompt;
   }
 }
-var codeGenSystemPrompt, codeGenGoSystemPrompt, codeGenPythonSystemPrompt, codeGenJavaSystemPrompt, codeGenRustSystemPrompt, codeGenPhpSystemPrompt, reviewSystemPrompt, reviewArchitectureSystemPrompt, reviewImplementationSystemPrompt, reviewImpactComplexitySystemPrompt;
+var codeGenSystemPrompt, codeGenGoSystemPrompt, codeGenPythonSystemPrompt, codeGenJavaSystemPrompt, codeGenRustSystemPrompt, codeGenPhpSystemPrompt, reviewArchitectureSystemPrompt, reviewImplementationSystemPrompt, reviewImpactComplexitySystemPrompt;
 var init_codegen_prompt = __esm({
   "prompts/codegen.prompt.ts"() {
     "use strict";
@@ -190,23 +189,6 @@ CRITICAL \u2014 File Reuse Rules:
 10. Register new routes in the EXISTING routes/api.php (or routes/web.php) \u2014 never create a parallel routes file.
 11. Add new Eloquent model methods to the EXISTING Model class \u2014 never create a parallel model file.
 12. Add new service methods to the EXISTING service class if one already covers the same resource.`;
-    reviewSystemPrompt = `You are a Senior Software Architect conducting a thorough code review. Your review should be structured, constructive, and specific.
-
-Always format your review with these exact sections:
-
-## \u2705 \u4F18\u70B9 (What's Good)
-List specific things done well.
-
-## \u26A0\uFE0F \u95EE\u9898 (Issues Found)
-List bugs, security issues, or spec deviations with line references.
-
-## \u{1F4A1} \u6539\u8FDB\u5EFA\u8BAE (Suggestions)
-Actionable improvements that would elevate code quality.
-
-## \u{1F4CA} \u603B\u4F53\u8BC4\u4EF7 (Overall Assessment)
-Score: X/10 \u2014 One-paragraph summary.
-
-Be specific. Reference actual code, not vague principles.`;
     reviewArchitectureSystemPrompt = `You are a Senior Software Architect reviewing the HIGH-LEVEL design of a code change.
 
 Focus ONLY on:
@@ -7026,7 +7008,9 @@ ${content.slice(0, 3e3)}`;
     for (const entry of recent) {
       const bar = "\u2588".repeat(entry.score) + "\u2591".repeat(10 - entry.score);
       const color = entry.score >= 8 ? chalk9.green : entry.score >= 6 ? chalk9.yellow : chalk9.red;
-      console.log(`  ${entry.date}  [${color(bar)}] ${color(entry.score + "/10")}  ${path10.basename(entry.specFile)}`);
+      const impactTag = entry.impactLevel ? chalk9.gray(` \u5F71\u54CD:${entry.impactLevel === "\u9AD8" ? chalk9.red(entry.impactLevel) : entry.impactLevel === "\u4E2D" ? chalk9.yellow(entry.impactLevel) : chalk9.green(entry.impactLevel)}`) : "";
+      const complexityTag = entry.complexityLevel ? chalk9.gray(` \u590D\u6742\u5EA6:${entry.complexityLevel === "\u9AD8" ? chalk9.red(entry.complexityLevel) : entry.complexityLevel === "\u4E2D" ? chalk9.yellow(entry.complexityLevel) : chalk9.green(entry.complexityLevel)}`) : "";
+      console.log(`  ${entry.date}  [${color(bar)}] ${color(entry.score + "/10")}${impactTag}${complexityTag}  ${path10.basename(entry.specFile)}`);
     }
     console.log(chalk9.cyan("\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500"));
   }
@@ -10040,9 +10024,11 @@ program.command("create").description("Generate a feature spec and kick off code
   });
   setActiveLogger(runLogger);
   console.log(chalk19.blue("[1/6] Loading project context..."));
+  runLogger.stageStart("context_load");
   const loader = new ContextLoader(currentDir);
   const context = await loader.loadProjectContext();
   const { type: detectedRepoType } = await detectRepoType(currentDir);
+  runLogger.stageEnd("context_load", { techStack: context.techStack, repoType: detectedRepoType });
   console.log(chalk19.gray(`  Tech stack  : ${context.techStack.join(", ") || "unknown"} [${detectedRepoType}]`));
   console.log(chalk19.gray(`  Dependencies: ${context.dependencies.length} packages`));
   console.log(chalk19.gray(`  API files   : ${context.apiStructure.length} files`));
@@ -10073,6 +10059,7 @@ program.command("create").description("Generate a feature spec and kick off code
   const specProvider = createProvider(specProviderName, specApiKey, specModelName);
   let initialSpec;
   let initialTasks = [];
+  runLogger.stageStart("spec_gen", { provider: specProviderName, model: specModelName });
   try {
     if (opts.skipTasks) {
       const generator = new SpecGenerator(specProvider);
@@ -10089,7 +10076,9 @@ program.command("create").description("Generate a feature spec and kick off code
         console.log(chalk19.yellow("  \u26A0 Tasks not parsed from response \u2014 will retry separately after refinement."));
       }
     }
+    runLogger.stageEnd("spec_gen", { taskCount: initialTasks.length });
   } catch (err) {
+    runLogger.stageFail("spec_gen", err.message);
     console.error(chalk19.red("  \u2718 Spec generation failed:"), err);
     process.exit(1);
   }
@@ -10099,8 +10088,10 @@ program.command("create").description("Generate a feature spec and kick off code
     finalSpec = initialSpec;
   } else {
     console.log(chalk19.blue("\n[3/6] Interactive spec refinement..."));
+    runLogger.stageStart("spec_refine");
     const refiner = new SpecRefiner(specProvider);
     finalSpec = await refiner.refineLoop(initialSpec);
+    runLogger.stageEnd("spec_refine");
   }
   const featureSlug = slugify(idea);
   const minScore = config2.minSpecScore ?? 0;
@@ -10109,14 +10100,17 @@ program.command("create").description("Generate a feature spec and kick off code
     if (!opts.auto) {
       console.log(chalk19.blue("\n[3.4/6] Spec quality assessment..."));
     }
+    runLogger.stageStart("spec_assess");
     const assessment = await assessSpec(specProvider, finalSpec, context.constitution ?? void 0);
     if (assessment) {
+      runLogger.stageEnd("spec_assess", { overallScore: assessment.overallScore });
       if (!opts.auto) printSpecAssessment(assessment);
       if (minScore > 0 && assessment.overallScore < minScore) {
         if (opts.force) {
           console.log(chalk19.yellow(`
   \u26A0 Score gate: ${assessment.overallScore}/10 < minimum ${minScore}/10 \u2014 bypassed with --force.`));
         } else {
+          runLogger.stageFail("spec_assess", `Score gate: ${assessment.overallScore} < ${minScore}`);
           console.log(chalk19.red(`
   \u2718 Spec quality gate failed: overallScore ${assessment.overallScore}/10 < minimum ${minScore}/10`));
           if (!opts.auto) {
@@ -10128,8 +10122,11 @@ program.command("create").description("Generate a feature spec and kick off code
           process.exit(1);
         }
       }
-    } else if (!opts.auto) {
-      console.log(chalk19.gray("  (Assessment skipped \u2014 AI call failed or timed out)"));
+    } else {
+      runLogger.stageEnd("spec_assess", { skipped: true });
+      if (!opts.auto) {
+        console.log(chalk19.gray("  (Assessment skipped \u2014 AI call failed or timed out)"));
+      }
     }
   }
   if (!opts.auto) {
@@ -10187,17 +10184,21 @@ program.command("create").description("Generate a feature spec and kick off code
   } else {
     console.log(chalk19.blue("\n[DSL] Extracting structured DSL from spec..."));
     console.log(chalk19.gray(`  Provider: ${specProviderName}/${specModelName}`));
+    runLogger.stageStart("dsl_extract");
     try {
       const isFrontend = isFrontendDeps(context.dependencies);
       if (isFrontend) console.log(chalk19.gray("  Frontend project detected \u2014 using ComponentSpec extractor"));
       const dslExtractor = new DslExtractor(specProvider);
       extractedDsl = await dslExtractor.extract(finalSpec, { auto: opts.auto, isFrontend });
       if (extractedDsl) {
+        runLogger.stageEnd("dsl_extract", { endpoints: extractedDsl.endpoints?.length ?? 0, models: extractedDsl.models?.length ?? 0 });
         console.log(chalk19.green("  \u2714 DSL extracted and validated."));
       } else {
+        runLogger.stageEnd("dsl_extract", { skipped: true });
         console.log(chalk19.yellow("  \u26A0 DSL skipped \u2014 codegen will use Spec + Tasks only."));
       }
     } catch (err) {
+      runLogger.stageFail("dsl_extract", err.message);
       console.log(chalk19.yellow(`  \u26A0 DSL extraction error: ${err.message} \u2014 continuing without DSL.`));
     }
   }
@@ -10255,6 +10256,7 @@ program.command("create").description("Generate a feature spec and kick off code
     const testGen = new TestGenerator(codegenProvider);
     generatedTestFiles = await testGen.generateTdd(extractedDsl, workingDir);
   }
+  runLogger.stageStart("codegen", { mode: codegenMode, provider: codegenProviderName, model: codegenModelName });
   const codegen = new CodeGenerator(codegenProvider, codegenMode);
   const generatedFiles = await codegen.generateCode(specFile, workingDir, context, {
     auto: opts.auto,
@@ -10262,6 +10264,7 @@ program.command("create").description("Generate a feature spec and kick off code
     dslFilePath: savedDslFile ?? void 0,
     repoType: detectedRepoType
   });
+  runLogger.stageEnd("codegen", { filesGenerated: generatedFiles.length });
   if (opts.tdd) {
     console.log(chalk19.gray("\n[7/9] TDD mode \u2014 test files already written pre-implementation."));
   } else if (opts.skipTests) {
@@ -10271,8 +10274,10 @@ program.command("create").description("Generate a feature spec and kick off code
   } else {
     console.log(chalk19.blue(`
 [7/9] Test skeleton generation...`));
+    runLogger.stageStart("test_gen");
     const testGen = new TestGenerator(codegenProvider);
     generatedTestFiles = await testGen.generate(extractedDsl, workingDir);
+    runLogger.stageEnd("test_gen", { filesGenerated: generatedTestFiles.length });
   }
   if (opts.skipErrorFeedback) {
     console.log(chalk19.gray("[8/9] Skipping error feedback (--skip-error-feedback)."));
@@ -10280,14 +10285,17 @@ program.command("create").description("Generate a feature spec and kick off code
     if (opts.tdd) {
       console.log(chalk19.cyan("[8/9] TDD mode \u2014 error feedback loop driving implementation to pass tests..."));
     }
+    runLogger.stageStart("error_feedback");
     await runErrorFeedback(codegenProvider, workingDir, extractedDsl, {
       maxCycles: opts.tdd ? 3 : 2
       // TDD gets one extra cycle
     });
+    runLogger.stageEnd("error_feedback");
   }
   let reviewResult = "";
   if (!opts.skipReview) {
     console.log(chalk19.blue("\n[9/9] Automated code review (3-pass: architecture + implementation + impact/complexity)..."));
+    runLogger.stageStart("review");
     const reviewer = new CodeReviewer(specProvider, currentDir);
     const savedSpec = await fs24.readFile(specFile, "utf-8");
     if (codegenMode === "api" && generatedFiles.length > 0) {
@@ -10301,6 +10309,7 @@ program.command("create").description("Generate a feature spec and kick off code
         process.chdir(originalDir);
       }
     }
+    runLogger.stageEnd("review");
     await accumulateReviewKnowledge(specProvider, currentDir, reviewResult);
   }
   runLogger.finish();
@@ -11149,6 +11158,12 @@ program.command("update").description("Update an existing spec with a change req
   const provider = createProvider(providerName, apiKey, modelName);
   console.log(chalk19.blue("\n\u2500\u2500\u2500 ai-spec update \u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500"));
   console.log(chalk19.gray(`  Provider: ${providerName}/${modelName}`));
+  const updateRunId = generateRunId();
+  const updateSnapshot = new RunSnapshot(currentDir, updateRunId);
+  setActiveSnapshot(updateSnapshot);
+  const updateLogger = new RunLogger(currentDir, updateRunId, { provider: providerName, model: modelName });
+  setActiveLogger(updateLogger);
+  console.log(chalk19.gray(`  Run ID: ${updateRunId}`));
   let specPath = opts.spec ?? null;
   if (!specPath) {
     const specsDir = path23.join(currentDir, "specs");
@@ -11207,6 +11222,7 @@ ${context.constitution}
 === DSL Context ===
 ${JSON.stringify(result.updatedDsl, null, 2).slice(0, 3e3)}
 ` : "";
+    updateLogger.stageStart("update_codegen");
     for (const affected of result.affectedFiles) {
       const fullPath = path23.join(currentDir, affected.file);
       let existing = "";
@@ -11231,12 +11247,29 @@ ${existing || "Create from scratch."}`;
         const raw = await codegenProvider.generate(codePrompt, _getPrompt(repoType));
         const content = raw.replace(/^```\w*\n?/gm, "").replace(/\n?```$/gm, "").trim();
         await fs24.ensureDir(path23.dirname(fullPath));
+        await updateSnapshot.snapshotFile(fullPath);
         await fs24.writeFile(fullPath, content, "utf-8");
+        updateLogger.fileWritten(affected.file);
         console.log(chalk19.green("\u2714"));
       } catch (err) {
+        updateLogger.stageFail("update_codegen", `${affected.file}: ${err.message}`);
         console.log(chalk19.red(`\u2718 ${err.message}`));
       }
     }
+    updateLogger.stageEnd("update_codegen", { filesUpdated: result.affectedFiles.length });
+    const updatedSpecContent = await fs24.readFile(result.newSpecPath, "utf-8").catch(() => "");
+    if (updatedSpecContent) {
+      const updateReviewer = new CodeReviewer(provider, currentDir);
+      const reviewResult = await updateReviewer.reviewCode(updatedSpecContent, result.newSpecPath).catch(() => "");
+      if (reviewResult && reviewResult !== "No changes") {
+        await accumulateReviewKnowledge(provider, currentDir, reviewResult);
+      }
+    }
+  }
+  updateLogger.finish();
+  updateLogger.printSummary();
+  if (updateSnapshot.fileCount > 0) {
+    console.log(chalk19.gray(`  To undo changes: ai-spec restore ${updateRunId}`));
   }
   if (!opts.codegen && result.affectedFiles.length > 0) {
     console.log(chalk19.blue("\n  Next steps:"));