ai-spec-dev 0.14.1 → 0.24.0

package/dist/index.d.mts

@@ -147,7 +147,7 @@ declare class SpecRefiner {
 }
 
 declare function buildTaskPrompt(spec: string, context?: ProjectContext): string;
-type TaskLayer = "data" | "infra" | "service" | "api" | "test";
+type TaskLayer = "data" | "infra" | "service" | "api" | "view" | "route" | "test";
 type TaskPriority = "high" | "medium" | "low";
 type TaskStatus = "pending" | "done" | "failed";
 interface SpecTask {

package/dist/index.d.ts

@@ -147,7 +147,7 @@ declare class SpecRefiner {
 }
 
 declare function buildTaskPrompt(spec: string, context?: ProjectContext): string;
-type TaskLayer = "data" | "infra" | "service" | "api" | "test";
+type TaskLayer = "data" | "infra" | "service" | "api" | "view" | "route" | "test";
 type TaskPriority = "high" | "medium" | "low";
 type TaskStatus = "pending" | "done" | "failed";
 interface SpecTask {

package/dist/index.js

@@ -3684,7 +3684,7 @@ ${content.slice(0, 1500)}`);
         try {
           const content = await fs3.readFile(path2.join(this.projectRoot, f), "utf-8");
           parts.push(`// ${f}
-${content.slice(0, 800)}`);
+${content.slice(0, 2e3)}`);
         } catch {
         }
       }
@@ -4093,9 +4093,17 @@ CRITICAL \u2014 Route/Store index registration (MUST follow):
 
 CRITICAL \u2014 Cross-file function name consistency (MUST follow):
 17. When you see an "=== Files Already Generated in This Run ===" section, those file contents are the AUTHORITATIVE source
-    of truth for exported function/variable names.
-    NEVER rename, guess, or substitute alternative names. If the file exports "getTaskList", import "getTaskList" \u2014 not "getTasks".
-    If no such section is present, derive function names strictly from the DSL endpoint IDs shown in the spec.`;
+    of truth for exported function/variable/action names.
+    NEVER rename, guess, or substitute alternative names. Copy-paste the exact identifier.
+    Common hallucination patterns to AVOID:
+    - Adding suffixes: fetchTasks \u2192 fetchTaskList, fetchTaskData, fetchTaskAll  \u2190 ALL WRONG
+    - Changing verb: fetchTasks \u2192 getTasks, loadTasks, queryTasks  \u2190 WRONG unless that's in the cache
+    - Changing number: createTask \u2192 createTasks  \u2190 WRONG
+    For Pinia stores specifically: the "// public API (return object):" section or the full store content
+    shows EVERY available action name. If it shows "fetchTasks", that is the ONLY valid name.
+    If no such section is present, derive function names strictly from the DSL endpoint IDs shown in the spec.
+    ALSO applies to file paths: if you see "// exists: src/views/task-management/TaskManagement.vue",
+    the router import MUST use that exact path \u2014 NOT "@/views/task-management/index.vue" or any other guess.`;
 var codeGenGoSystemPrompt = `You are a Senior Go Developer implementing features based on provided specifications.
 
 Rules:
@@ -4270,7 +4278,7 @@ Each task object must have these exact fields:
   "id": "TASK-001",           // sequential, zero-padded
   "title": "...",             // short action phrase, e.g. "Add UserFavorite Prisma model"
   "description": "...",       // 1-2 sentences, specific and actionable
-  "layer": "data|service|api|test|infra",  // implementation layer
+  "layer": "data|service|api|view|route|test|infra",  // implementation layer
   "filesToTouch": ["..."],    // VERIFIED paths only \u2014 see rules below
   "acceptanceCriteria": ["..."],  // verifiable completion conditions
   "dependencies": ["TASK-001"],  // task ids that must complete first (empty array if none)
@@ -4278,11 +4286,34 @@ Each task object must have these exact fields:
 }
 
 Layer ordering guidance (implement in this order):
-1. "data"    \u2014 DB schema changes, migrations, seed data
+1. "data"    \u2014 DB schema changes, migrations, seed data; TypeScript type/interface definition files
 2. "infra"   \u2014 config, env vars, external service setup
-3. "service" \u2014 business logic, service classes
-4. "api"     \u2014 controllers, routes, middleware, validators
-5. "test"    \u2014 unit tests, integration tests
+3. "service" \u2014 business logic, service classes; for frontend: HTTP API call files ONLY (src/api/ or src/apis/)
+4. "api"     \u2014 controllers, routes, middleware, validators; for frontend: state stores ONLY (src/stores/, Pinia/Vuex/Zustand/Redux)
+5. "view"    \u2014 FRONTEND ONLY: page/view components (src/views/, src/pages/) \u2014 generated AFTER stores
+6. "route"   \u2014 FRONTEND ONLY: router module files (src/router/routes/) \u2014 generated AFTER view components
+7. "test"    \u2014 unit tests, integration tests
+
+CRITICAL \u2014 Frontend four-layer dependency rule (prevents BOTH naming AND filename hallucinations):
+For Vue/React frontend projects, STRICTLY follow this assignment:
+  "service" \u2192 src/api/* or src/apis/* files   (HTTP functions: getTaskList, createTask)
+  "api"     \u2192 src/stores/* files               (stores call service layer \u2014 see exact function names)
+  "view"    \u2192 src/views/* or src/pages/* files (pages use stores \u2014 see exact action names)
+  "route"   \u2192 src/router/routes/* files        (router imports views \u2014 sees EXACT component filenames)
+
+WHY "route" must come after "view":
+The router file imports the view component by filename, e.g.:
+  import('@/views/task-management/TaskManagement.vue')
+If the router is generated BEFORE the view file exists in cache, the AI will guess a generic name
+like "index.vue" (the most common fallback) instead of the real filename "TaskManagement.vue".
+By generating the router AFTER the view, the cache contains "// exists: src/views/task-management/TaskManagement.vue"
+and the AI uses the EXACT path.
+
+EXAMPLE (correct, four-layer):
+  TASK-001 layer:"service"  src/apis/taskManagement.ts        (exports getTaskList, createTask)
+  TASK-002 layer:"api"      src/stores/taskStore.ts            (calls getTaskList \u2014 visible in cache \u2713)
+  TASK-003 layer:"view"     src/views/task-management/TaskManagement.vue  (uses taskStore \u2014 visible in cache \u2713)
+  TASK-004 layer:"route"    src/router/routes/taskManagement.ts (imports TaskManagement.vue \u2014 filename visible in cache \u2713)
 
 CRITICAL \u2014 filesToTouch Rules (hallucination prevention):
 - ONLY use paths that appear in the "Verified File Inventory" section of the prompt.
@@ -4332,7 +4363,7 @@ function buildTaskPrompt(spec, context) {
   if (context.constitution) {
     parts.push(`
 === Project Constitution (rules to follow) ===
-${context.constitution.slice(0, 1500)}`);
+${context.constitution}`);
   }
   if (context.techStack.length > 0) {
     parts.push(`
@@ -4347,7 +4378,9 @@ var LAYER_ORDER = {
   infra: 1,
   service: 2,
   api: 3,
-  test: 4
+  view: 4,
+  route: 5,
+  test: 6
 };
 var TaskGenerator = class {
   constructor(provider) {
@@ -4389,6 +4422,8 @@ function printTasks(tasks) {
     infra: import_chalk3.default.gray,
     service: import_chalk3.default.blue,
     api: import_chalk3.default.cyan,
+    view: import_chalk3.default.yellow,
+    route: import_chalk3.default.white,
     test: import_chalk3.default.green
   };
   console.log(import_chalk3.default.bold(`
@@ -5270,16 +5305,111 @@ function buildInstalledPackagesSection(context) {
 ${context.dependencies.join(", ")}
 `;
 }
+function extractBehavioralContract(content) {
+  const lines = content.split("\n");
+  const contractLines = [];
+  const throwLines = [];
+  let i = 0;
+  while (i < lines.length) {
+    const line = lines[i];
+    const trimmed = line.trim();
+    if (/^export\s+(interface|type|class|abstract\s+class|enum)\s/.test(trimmed)) {
+      contractLines.push(line.trimEnd());
+      if (trimmed.includes("{")) {
+        let depth = (trimmed.match(/\{/g) ?? []).length - (trimmed.match(/\}/g) ?? []).length;
+        i++;
+        while (i < lines.length && depth > 0) {
+          const inner = lines[i];
+          contractLines.push(inner.trimEnd());
+          depth += (inner.match(/\{/g) ?? []).length;
+          depth -= (inner.match(/\}/g) ?? []).length;
+          i++;
+        }
+      } else {
+        i++;
+      }
+      continue;
+    }
+    if (/^export\s+const\s+\w+\s*=\s*(defineStore|createStore|createSlice)\s*\(/.test(trimmed)) {
+      contractLines.push(line.trimEnd());
+      let depth = (trimmed.match(/\(/g) ?? []).length - (trimmed.match(/\)/g) ?? []).length;
+      i++;
+      while (i < lines.length && depth > 0) {
+        const inner = lines[i];
+        contractLines.push(inner.trimEnd());
+        depth += (inner.match(/\(/g) ?? []).length;
+        depth -= (inner.match(/\)/g) ?? []).length;
+        i++;
+      }
+      continue;
+    }
+    if (/^return\s*\{/.test(trimmed)) {
+      contractLines.push("// public API (return object):");
+      contractLines.push(line.trimEnd());
+      let depth = (trimmed.match(/\{/g) ?? []).length - (trimmed.match(/\}/g) ?? []).length;
+      i++;
+      while (i < lines.length && depth > 0) {
+        const inner = lines[i];
+        contractLines.push(inner.trimEnd());
+        depth += (inner.match(/\{/g) ?? []).length;
+        depth -= (inner.match(/\}/g) ?? []).length;
+        i++;
+      }
+      continue;
+    }
+    if (/^export\s+default\s+(async\s+)?(function|class)\b/.test(trimmed)) {
+      contractLines.push(line.trimEnd());
+      if (trimmed.includes("{")) {
+        let depth = (trimmed.match(/\{/g) ?? []).length - (trimmed.match(/\}/g) ?? []).length;
+        i++;
+        while (i < lines.length && depth > 0) {
+          const inner = lines[i];
+          contractLines.push(inner.trimEnd());
+          depth += (inner.match(/\{/g) ?? []).length;
+          depth -= (inner.match(/\}/g) ?? []).length;
+          i++;
+        }
+      } else {
+        i++;
+      }
+      continue;
+    }
+    if (/^export\s/.test(trimmed)) {
+      contractLines.push(line.trimEnd());
+    }
+    if (/throw\s+(new\s+)?\w*[Ee]rror\b|throw\s+create[A-Z]\w*|@throws/.test(line) && throwLines.length < 20) {
+      throwLines.push("  // " + trimmed);
+    }
+    i++;
+  }
+  if (contractLines.length === 0 && throwLines.length === 0) {
+    return content.slice(0, 3e3);
+  }
+  const parts = [...contractLines];
+  if (throwLines.length > 0) {
+    parts.push("", "// Error contracts (throws / validation):", ...throwLines);
+  }
+  return parts.join("\n");
+}
 function buildGeneratedFilesSection(cache) {
   if (cache.size === 0) return "";
   const lines = [
-    "\n=== Files Already Generated in This Run \u2014 USE EXACT EXPORTS (do not rename or invent alternatives) ==="
+    "\n=== Files Already Generated in This Run \u2014 USE EXACT EXPORTS (do not rename or invent alternatives) ===",
+    "// CRITICAL: function/action names and file paths below are ground truth. Copy them EXACTLY.",
+    "// Do NOT add suffixes (List, Data, All, Info) or change casing.",
+    "// For '// exists:' entries: use the EXACT filename shown \u2014 do NOT substitute index.vue or other defaults."
   ];
   for (const [filePath, content] of cache) {
+    const isViewFile = /src[\\/](views?|pages?)[\\/]/i.test(filePath);
+    if (isViewFile) {
+      lines.push(`
+// exists: ${filePath}`);
+      continue;
+    }
     lines.push(`
 --- ${filePath} ---`);
-    lines.push(content.slice(0, 800));
-    if (content.length > 800) lines.push("... (truncated)");
+    const isStoreOrComposable = /src[\\/](stores?|composables?)[\\/]/i.test(filePath);
+    lines.push(isStoreOrComposable ? content : extractBehavioralContract(content));
   }
   return lines.join("\n") + "\n";
 }
@@ -5467,7 +5597,7 @@ Implement ONLY this task. Do not implement other tasks.`;
     const spec = await fs8.readFile(specFilePath, "utf-8");
     const constitutionSection = context?.constitution ? `
 === Project Constitution (MUST follow) ===
-${context.constitution.slice(0, 2e3)}
+${context.constitution}
 ` : "";
     const contextSummary = context ? `Tech Stack: ${context.techStack.join(", ")}
 Existing files: ${context.fileStructure.slice(0, 20).join(", ")}` : "";
@@ -5561,7 +5691,7 @@ Output ONLY a valid JSON array:
         printTaskProgress(completedTasks++, tasks.length, task, "skip");
       }
     }
-    const LAYER_ORDER2 = ["data", "infra", "service", "api", "test"];
+    const LAYER_ORDER2 = ["data", "infra", "service", "api", "view", "route", "test"];
     const layerGroups = [];
     for (const layer of LAYER_ORDER2) {
       const group = pendingTasks.filter((t) => t.layer === layer);
@@ -5584,10 +5714,9 @@ Output ONLY a valid JSON array:
       } else {
         printTaskProgress(completedTasks, tasks.length, layerTasks[0], "run");
       }
-      const generatedFilesSection = buildGeneratedFilesSection(generatedFileCache);
-      const taskResultPromises = layerTasks.map(async (task) => {
+      const executeTask = async (task, batchIsParallel) => {
         if (task.filesToTouch.length === 0) {
-          if (!isParallel) console.log(import_chalk6.default.gray("    No files specified, skipping."));
+          if (!batchIsParallel) console.log(import_chalk6.default.gray("    No files specified, skipping."));
           return { task, files: [], createdFiles: [], success: 0, total: 0, impliesRegistration: false };
         }
         const filePlan = await Promise.all(
@@ -5602,10 +5731,11 @@ Output ONLY a valid JSON array:
         );
         const createsNewFiles = filePlan.some((f) => f.action === "create");
         const taskText = `${task.title} ${task.description}`.toLowerCase();
-        const impliesRegistration = createsNewFiles && (taskText.includes("route") || taskText.includes("router") || taskText.includes("page") || taskText.includes("view") || taskText.includes("store") || taskText.includes("service") || taskText.includes("component") || taskText.includes("menu") || taskText.includes("navigation") || taskText.includes("\u6A21\u5757") || taskText.includes("\u9875\u9762") || taskText.includes("\u8DEF\u7531") || taskText.includes("\u6CE8\u518C"));
+        const impliesRegistration = createsNewFiles && (task.layer === "route" || task.layer === "view" || task.layer === "api" || taskText.includes("route") || taskText.includes("router") || taskText.includes("page") || taskText.includes("view") || taskText.includes("store") || taskText.includes("service") || taskText.includes("component") || taskText.includes("menu") || taskText.includes("navigation") || taskText.includes("\u6A21\u5757") || taskText.includes("\u9875\u9762") || taskText.includes("\u8DEF\u7531") || taskText.includes("\u6CE8\u518C"));
         if (filePlan.length === 0) {
           return { task, files: [], createdFiles: [], success: 0, total: 0, impliesRegistration };
         }
+        const currentGeneratedFilesSection = buildGeneratedFilesSection(generatedFileCache);
         const taskContext = `Task: ${task.id} \u2014 ${task.title}
 ${task.description}
 Acceptance: ${task.acceptanceCriteria.join("; ")}`;
@@ -5616,15 +5746,38 @@ Acceptance: ${task.acceptanceCriteria.join("; ")}`;
 === Current Task ===
 ${taskContext}`,
           workingDir,
-          constitutionSection + frontendSection + sharedConfigSection + generatedFilesSection,
+          constitutionSection + frontendSection + sharedConfigSection + currentGeneratedFilesSection,
           systemPrompt,
-          isParallel ? task.id : void 0
+          batchIsParallel ? task.id : void 0
           // prefix output lines with task ID in parallel mode
         );
         const createdFiles = filePlan.filter((fp) => fp.action === "create").map((fp) => fp.file);
         return { task, files, createdFiles, success, total, impliesRegistration };
-      });
-      const layerResults = await Promise.all(taskResultPromises);
+      };
+      const updateCacheFromBatch = async (results) => {
+        for (const result of results) {
+          for (const writtenFile of result.files) {
+            const isCodeFile = /src[\\/](api[s]?|services?|stores?|composables?)[\\/]/i.test(writtenFile);
+            const isViewFile = /src[\\/](views?|pages?)[\\/]/i.test(writtenFile);
+            if (isCodeFile || isViewFile) {
+              try {
+                const content = isViewFile ? `// view component \u2014 use this exact path for router imports` : await fs8.readFile(path6.join(workingDir, writtenFile), "utf-8");
+                generatedFileCache.set(writtenFile, content);
+              } catch {
+              }
+            }
+          }
+        }
+      };
+      const taskBatches = topoSortLayerTasks(layerTasks);
+      const layerResults = [];
+      for (const batch of taskBatches) {
+        const batchIsParallel = batch.length > 1;
+        const batchResultPromises = batch.map((task) => executeTask(task, batchIsParallel));
+        const batchResults = await Promise.all(batchResultPromises);
+        layerResults.push(...batchResults);
+        await updateCacheFromBatch(batchResults);
+      }
       if (isParallel) {
         console.log("");
       }
@@ -5644,17 +5797,6 @@ ${taskContext}`,
         }
       }
       completedTasks += layerTasks.length;
-      for (const result of layerResults) {
-        for (const writtenFile of result.files) {
-          if (/src[\\/](api[s]?|services?|stores?|composables?)[\\/]/.test(writtenFile)) {
-            try {
-              const content = await fs8.readFile(path6.join(workingDir, writtenFile), "utf-8");
-              generatedFileCache.set(writtenFile, content);
-            } catch {
-            }
-          }
-        }
-      }
       const anyImpliesRegistration = layerResults.some((r) => r.impliesRegistration);
       if (anyImpliesRegistration && sharedConfigPaths.size > 0 && context?.sharedConfigFiles) {
         const allCreatedInLayer = layerResults.flatMap((r) => r.createdFiles);
@@ -5754,11 +5896,48 @@ ${spec}`,
     console.log(import_chalk6.default.cyan("\n") + plan);
   }
 };
+function topoSortLayerTasks(tasks) {
+  if (tasks.length <= 1) return [tasks];
+  const idSet = new Set(tasks.map((t) => t.id));
+  const taskById = new Map(tasks.map((t) => [t.id, t]));
+  const inDegree = /* @__PURE__ */ new Map();
+  const dependents = /* @__PURE__ */ new Map();
+  for (const task of tasks) {
+    inDegree.set(task.id, 0);
+    dependents.set(task.id, []);
+  }
+  for (const task of tasks) {
+    const intraDeps = task.dependencies.filter((dep) => idSet.has(dep));
+    inDegree.set(task.id, intraDeps.length);
+    for (const dep of intraDeps) {
+      dependents.get(dep).push(task.id);
+    }
+  }
+  const batches = [];
+  const remaining = new Set(tasks.map((t) => t.id));
+  while (remaining.size > 0) {
+    const batch = [...remaining].filter((id) => inDegree.get(id) === 0).map((id) => taskById.get(id));
+    if (batch.length === 0) {
+      batches.push([...remaining].map((id) => taskById.get(id)));
+      break;
+    }
+    batches.push(batch);
+    for (const task of batch) {
+      remaining.delete(task.id);
+      for (const dependent of dependents.get(task.id)) {
+        inDegree.set(dependent, inDegree.get(dependent) - 1);
+      }
+    }
+  }
+  return batches;
+}
 var LAYER_ICONS = {
   data: "\u{1F4BE}",
   infra: "\u2699\uFE0F ",
   service: "\u{1F527}",
   api: "\u{1F310}",
+  view: "\u{1F5A5}\uFE0F ",
+  route: "\u{1F5FA}\uFE0F ",
   test: "\u{1F9EA}"
 };
 function printTaskProgress(completed, total, task, mode) {