@anthropologies/claudestory 0.1.32 → 0.1.33

package/dist/cli.js

@@ -5021,6 +5021,7 @@ var init_session_types = __esm({
       "WRITE_TESTS",
       "TEST",
       "CODE_REVIEW",
+      "BUILD",
       "VERIFY",
       "FINALIZE",
       "COMPACT",
@@ -5046,6 +5047,7 @@ var init_session_types = __esm({
       "WRITE_TESTS",
       "TEST",
       "CODE_REVIEW",
+      "BUILD",
       "VERIFY",
       "FINALIZE",
       "COMPACT",
@@ -5196,6 +5198,7 @@ var init_session_types = __esm({
       }).nullable().default(null),
       testRetryCount: z9.number().default(0),
       writeTestsRetryCount: z9.number().default(0),
+      buildRetryCount: z9.number().default(0),
       verifyRetryCount: z9.number().default(0),
       verifyAutoDetected: z9.boolean().default(false),
       // T-128: Resolved recipe (frozen at session start, survives compact/resume)
@@ -5914,6 +5917,12 @@ function resolveRecipe(recipeName, projectOverrides) {
       pipeline.splice(codeReviewIdx + 1, 0, "VERIFY");
     }
   }
+  if (stages2.BUILD?.enabled) {
+    const finalizeIdx = pipeline.indexOf("FINALIZE");
+    if (finalizeIdx !== -1 && !pipeline.includes("BUILD")) {
+      pipeline.splice(finalizeIdx, 0, "BUILD");
+    }
+  }
   if (stages2.TEST?.enabled) {
     const implementIdx = pipeline.indexOf("IMPLEMENT");
     if (implementIdx !== -1 && !pipeline.includes("TEST")) {
@@ -6526,7 +6535,8 @@ var init_implement = __esm({
           ].join("\n"),
           reminders: [
             "Follow the plan exactly. Do NOT deviate without re-planning.",
-            "Do NOT ask the user for confirmation."
+            "Do NOT ask the user for confirmation.",
+            "If you discover pre-existing bugs, failing tests not caused by your changes, or other out-of-scope problems, file them as issues using claudestory_issue_create. Do not fix them inline."
           ],
           transitionedFrom: ctx.state.previousState ?? void 0
         };
@@ -6804,12 +6814,16 @@ var init_code_review = __esm({
             "",
             `Capture the diff with: ${diffCommand}`,
             "",
-            "**IMPORTANT:** Pass the FULL unified diff output to the reviewer. Do NOT summarize, compress, or truncate the diff.",
+            "**IMPORTANT:** Pass the FULL unified diff to the reviewer. For diffs over ~500 lines, use file-scoped chunks (`git diff <mergebase> -- <filepath>`) across separate calls (pass the same session_id). Do NOT summarize or truncate any individual chunk.",
             "",
             `Run a code review using **${reviewer}**.`,
             "When done, report verdict and findings."
           ].join("\n"),
-          reminders: [diffReminder, "Do NOT compress or summarize the diff."],
+          reminders: [
+            diffReminder,
+            "Do NOT compress or summarize the diff.",
+            "If the reviewer flags pre-existing issues unrelated to your changes, file them as issues using claudestory_issue_create with severity and impact. Do not fix them in this ticket."
+          ],
           transitionedFrom: ctx.state.previousState ?? void 0
         };
       }
@@ -6924,6 +6938,95 @@ var init_code_review = __esm({
   }
 });
 
+// src/autonomous/stages/build.ts
+var MAX_BUILD_RETRIES, BuildStage;
+var init_build = __esm({
+  "src/autonomous/stages/build.ts"() {
+    "use strict";
+    init_esm_shims();
+    MAX_BUILD_RETRIES = 2;
+    BuildStage = class {
+      id = "BUILD";
+      skip(ctx) {
+        const buildConfig = ctx.recipe.stages?.BUILD;
+        return !buildConfig?.enabled;
+      }
+      async enter(ctx) {
+        const buildConfig = ctx.recipe.stages?.BUILD;
+        const command = buildConfig?.command ?? "npm run build";
+        const retryCount = ctx.state.buildRetryCount ?? 0;
+        return {
+          instruction: [
+            `# Build${retryCount > 0 ? ` (retry ${retryCount}/${MAX_BUILD_RETRIES})` : ""}`,
+            "",
+            `Run the build: \`${command}\``,
+            "",
+            "Report the results with:",
+            "```json",
+            `{ "sessionId": "${ctx.state.sessionId}", "action": "report", "report": { "completedAction": "build_done", "notes": "<exit code and summary>" } }`,
+            "```",
+            "",
+            "Include the exit code (0 = pass, non-0 = fail) and any error output in notes."
+          ].join("\n"),
+          reminders: ["Run the FULL build, not a partial or dev-mode build."],
+          transitionedFrom: ctx.state.previousState ?? void 0
+        };
+      }
+      async report(ctx, report) {
+        const notes = report.notes ?? "";
+        const retryCount = ctx.state.buildRetryCount ?? 0;
+        const exitCodeMatch = notes.match(/exit\s*(?:code[:\s]*)?\s*(\d+)/i);
+        if (!exitCodeMatch) {
+          const nextRetry = retryCount + 1;
+          if (nextRetry > MAX_BUILD_RETRIES) {
+            ctx.writeState({ buildRetryCount: 0 });
+            ctx.appendEvent("build_parse_exhausted", { retryCount: nextRetry });
+            return {
+              action: "advance",
+              result: {
+                instruction: `Could not parse build exit code after ${MAX_BUILD_RETRIES} retries. Proceeding, but build status is unknown.`,
+                reminders: ["Mention unknown build status in the commit message."]
+              }
+            };
+          }
+          ctx.writeState({ buildRetryCount: nextRetry });
+          return { action: "retry", instruction: 'Could not parse exit code from notes. Include "exit code: 0" (or non-zero) in your notes.' };
+        }
+        const exitCode = parseInt(exitCodeMatch[1], 10);
+        if (exitCode === 0) {
+          ctx.writeState({ buildRetryCount: 0 });
+          ctx.appendEvent("build_passed", { retryCount, notes: notes.slice(0, 200) });
+          return { action: "advance" };
+        }
+        if (retryCount < MAX_BUILD_RETRIES) {
+          ctx.writeState({ buildRetryCount: retryCount + 1 });
+          ctx.appendEvent("build_failed_retry", { retryCount: retryCount + 1, notes: notes.slice(0, 200) });
+          return {
+            action: "back",
+            target: "IMPLEMENT",
+            reason: `Build failed (attempt ${retryCount + 1}/${MAX_BUILD_RETRIES}). Fix the build errors.`
+          };
+        }
+        ctx.writeState({ buildRetryCount: 0 });
+        ctx.appendEvent("build_failed_exhausted", { retryCount, notes: notes.slice(0, 200) });
+        return {
+          action: "advance",
+          result: {
+            instruction: [
+              "# Build Failed - Proceeding",
+              "",
+              `Build failed after ${MAX_BUILD_RETRIES} retries. Proceeding but build errors remain.`,
+              "",
+              "Document the build failure in the commit message."
+            ].join("\n"),
+            reminders: ["Mention build failure in the commit message."]
+          }
+        };
+      }
+    };
+  }
+});
+
 // src/autonomous/stages/verify.ts
 function exhaustionAction2(ctx) {
   ctx.writeState({ verifyRetryCount: 0 });
@@ -7733,6 +7836,7 @@ var init_stages = __esm({
     init_write_tests();
     init_test();
     init_code_review();
+    init_build();
     init_verify();
     init_finalize();
     init_complete();
@@ -7746,6 +7850,7 @@ var init_stages = __esm({
     registerStage(new WriteTestsStage());
     registerStage(new TestStage());
     registerStage(new CodeReviewStage());
+    registerStage(new BuildStage());
     registerStage(new VerifyStage());
     registerStage(new FinalizeStage());
     registerStage(new CompleteStage());
@@ -10258,7 +10363,7 @@ var init_mcp = __esm({
     init_init();
     ENV_VAR2 = "CLAUDESTORY_PROJECT_ROOT";
     CONFIG_PATH2 = ".story/config.json";
-    version = "0.1.32";
+    version = "0.1.33";
     main().catch((err) => {
       process.stderr.write(`Fatal: ${err instanceof Error ? err.message : String(err)}
 `);
@@ -13542,7 +13647,7 @@ async function runCli() {
     registerConfigCommand: registerConfigCommand2,
     registerSessionCommand: registerSessionCommand2
   } = await Promise.resolve().then(() => (init_register(), register_exports));
-  const version2 = "0.1.32";
+  const version2 = "0.1.33";
   class HandledError extends Error {
     constructor() {
       super("HANDLED_ERROR");

package/dist/index.d.ts

@@ -1334,15 +1334,40 @@ declare const SnapshotV1Schema: z.ZodObject<{
         file: z.ZodString;
         message: z.ZodString;
     }, "strip", z.ZodTypeAny, {
-        message: string;
         type: string;
+        message: string;
         file: string;
     }, {
-        message: string;
         type: string;
+        message: string;
         file: string;
     }>, "many">>;
 }, "strip", z.ZodTypeAny, {
+    version: 1;
+    config: {
+        version: number;
+        type: string;
+        language: string;
+        project: string;
+        features: {
+            issues: boolean;
+            tickets: boolean;
+            handovers: boolean;
+            roadmap: boolean;
+            reviews: boolean;
+        } & {
+            [k: string]: unknown;
+        };
+        schemaVersion?: number | undefined;
+        recipe?: string | undefined;
+        recipeOverrides?: {
+            maxTicketsPerSession?: number | undefined;
+            compactThreshold?: string | undefined;
+            reviewBackends?: string[] | undefined;
+        } | undefined;
+    } & {
+        [k: string]: unknown;
+    };
     issues: z.objectOutputType<{
         id: z.ZodString;
         title: z.ZodString;
@@ -1379,8 +1404,8 @@ declare const SnapshotV1Schema: z.ZodObject<{
         claimedBySession: z.ZodOptional<z.ZodNullable<z.ZodString>>;
     }, z.ZodTypeAny, "passthrough">[];
     roadmap: {
-        title: string;
         date: string;
+        title: string;
         phases: z.objectOutputType<{
             id: z.ZodString;
             label: z.ZodString;
@@ -1398,7 +1423,6 @@ declare const SnapshotV1Schema: z.ZodObject<{
     } & {
         [k: string]: unknown;
     };
-    version: 1;
     project: string;
     notes: z.objectOutputType<{
         id: z.ZodString;
@@ -1424,11 +1448,19 @@ declare const SnapshotV1Schema: z.ZodObject<{
         status: z.ZodEnum<["active", "deprecated", "superseded"]>;
     }, z.ZodTypeAny, "passthrough">[];
     createdAt: string;
-    config: {
+    handoverFilenames: string[];
+    warnings?: {
         type: string;
+        message: string;
+        file: string;
+    }[] | undefined;
+}, {
+    version: 1;
+    config: {
         version: number;
-        project: string;
+        type: string;
         language: string;
+        project: string;
         features: {
             issues: boolean;
             tickets: boolean;
@@ -1448,13 +1480,6 @@ declare const SnapshotV1Schema: z.ZodObject<{
     } & {
         [k: string]: unknown;
     };
-    handoverFilenames: string[];
-    warnings?: {
-        message: string;
-        type: string;
-        file: string;
-    }[] | undefined;
-}, {
     issues: z.objectInputType<{
         id: z.ZodString;
         title: z.ZodString;
@@ -1491,8 +1516,8 @@ declare const SnapshotV1Schema: z.ZodObject<{
         claimedBySession: z.ZodOptional<z.ZodNullable<z.ZodString>>;
     }, z.ZodTypeAny, "passthrough">[];
     roadmap: {
-        title: string;
         date: string;
+        title: string;
         phases: z.objectInputType<{
             id: z.ZodString;
             label: z.ZodString;
@@ -1510,33 +1535,8 @@ declare const SnapshotV1Schema: z.ZodObject<{
     } & {
         [k: string]: unknown;
     };
-    version: 1;
     project: string;
     createdAt: string;
-    config: {
-        type: string;
-        version: number;
-        project: string;
-        language: string;
-        features: {
-            issues: boolean;
-            tickets: boolean;
-            handovers: boolean;
-            roadmap: boolean;
-            reviews: boolean;
-        } & {
-            [k: string]: unknown;
-        };
-        schemaVersion?: number | undefined;
-        recipe?: string | undefined;
-        recipeOverrides?: {
-            maxTicketsPerSession?: number | undefined;
-            compactThreshold?: string | undefined;
-            reviewBackends?: string[] | undefined;
-        } | undefined;
-    } & {
-        [k: string]: unknown;
-    };
     notes?: z.objectInputType<{
         id: z.ZodString;
         title: z.ZodNullable<z.ZodString>;
@@ -1561,8 +1561,8 @@ declare const SnapshotV1Schema: z.ZodObject<{
         status: z.ZodEnum<["active", "deprecated", "superseded"]>;
     }, z.ZodTypeAny, "passthrough">[] | undefined;
     warnings?: {
-        message: string;
         type: string;
+        message: string;
         file: string;
     }[] | undefined;
     handoverFilenames?: string[] | undefined;

package/dist/mcp.js

@@ -3488,6 +3488,7 @@ var init_session_types = __esm({
       "WRITE_TESTS",
       "TEST",
       "CODE_REVIEW",
+      "BUILD",
       "VERIFY",
       "FINALIZE",
       "COMPACT",
@@ -3638,6 +3639,7 @@ var init_session_types = __esm({
       }).nullable().default(null),
       testRetryCount: z9.number().default(0),
       writeTestsRetryCount: z9.number().default(0),
+      buildRetryCount: z9.number().default(0),
       verifyRetryCount: z9.number().default(0),
       verifyAutoDetected: z9.boolean().default(false),
       // T-128: Resolved recipe (frozen at session start, survives compact/resume)
@@ -5523,6 +5525,12 @@ function resolveRecipe(recipeName, projectOverrides) {
       pipeline.splice(codeReviewIdx + 1, 0, "VERIFY");
     }
   }
+  if (stages2.BUILD?.enabled) {
+    const finalizeIdx = pipeline.indexOf("FINALIZE");
+    if (finalizeIdx !== -1 && !pipeline.includes("BUILD")) {
+      pipeline.splice(finalizeIdx, 0, "BUILD");
+    }
+  }
   if (stages2.TEST?.enabled) {
     const implementIdx = pipeline.indexOf("IMPLEMENT");
     if (implementIdx !== -1 && !pipeline.includes("TEST")) {
@@ -6079,7 +6087,8 @@ var ImplementStage = class {
       ].join("\n"),
       reminders: [
         "Follow the plan exactly. Do NOT deviate without re-planning.",
-        "Do NOT ask the user for confirmation."
+        "Do NOT ask the user for confirmation.",
+        "If you discover pre-existing bugs, failing tests not caused by your changes, or other out-of-scope problems, file them as issues using claudestory_issue_create. Do not fix them inline."
       ],
       transitionedFrom: ctx.state.previousState ?? void 0
     };
@@ -6338,12 +6347,16 @@ var CodeReviewStage = class {
         "",
         `Capture the diff with: ${diffCommand}`,
         "",
-        "**IMPORTANT:** Pass the FULL unified diff output to the reviewer. Do NOT summarize, compress, or truncate the diff.",
+        "**IMPORTANT:** Pass the FULL unified diff to the reviewer. For diffs over ~500 lines, use file-scoped chunks (`git diff <mergebase> -- <filepath>`) across separate calls (pass the same session_id). Do NOT summarize or truncate any individual chunk.",
         "",
         `Run a code review using **${reviewer}**.`,
         "When done, report verdict and findings."
       ].join("\n"),
-      reminders: [diffReminder, "Do NOT compress or summarize the diff."],
+      reminders: [
+        diffReminder,
+        "Do NOT compress or summarize the diff.",
+        "If the reviewer flags pre-existing issues unrelated to your changes, file them as issues using claudestory_issue_create with severity and impact. Do not fix them in this ticket."
+      ],
       transitionedFrom: ctx.state.previousState ?? void 0
     };
   }
@@ -6456,6 +6469,89 @@ var CodeReviewStage = class {
   }
 };
 
+// src/autonomous/stages/build.ts
+init_esm_shims();
+var MAX_BUILD_RETRIES = 2;
+var BuildStage = class {
+  id = "BUILD";
+  skip(ctx) {
+    const buildConfig = ctx.recipe.stages?.BUILD;
+    return !buildConfig?.enabled;
+  }
+  async enter(ctx) {
+    const buildConfig = ctx.recipe.stages?.BUILD;
+    const command = buildConfig?.command ?? "npm run build";
+    const retryCount = ctx.state.buildRetryCount ?? 0;
+    return {
+      instruction: [
+        `# Build${retryCount > 0 ? ` (retry ${retryCount}/${MAX_BUILD_RETRIES})` : ""}`,
+        "",
+        `Run the build: \`${command}\``,
+        "",
+        "Report the results with:",
+        "```json",
+        `{ "sessionId": "${ctx.state.sessionId}", "action": "report", "report": { "completedAction": "build_done", "notes": "<exit code and summary>" } }`,
+        "```",
+        "",
+        "Include the exit code (0 = pass, non-0 = fail) and any error output in notes."
+      ].join("\n"),
+      reminders: ["Run the FULL build, not a partial or dev-mode build."],
+      transitionedFrom: ctx.state.previousState ?? void 0
+    };
+  }
+  async report(ctx, report) {
+    const notes = report.notes ?? "";
+    const retryCount = ctx.state.buildRetryCount ?? 0;
+    const exitCodeMatch = notes.match(/exit\s*(?:code[:\s]*)?\s*(\d+)/i);
+    if (!exitCodeMatch) {
+      const nextRetry = retryCount + 1;
+      if (nextRetry > MAX_BUILD_RETRIES) {
+        ctx.writeState({ buildRetryCount: 0 });
+        ctx.appendEvent("build_parse_exhausted", { retryCount: nextRetry });
+        return {
+          action: "advance",
+          result: {
+            instruction: `Could not parse build exit code after ${MAX_BUILD_RETRIES} retries. Proceeding, but build status is unknown.`,
+            reminders: ["Mention unknown build status in the commit message."]
+          }
+        };
+      }
+      ctx.writeState({ buildRetryCount: nextRetry });
+      return { action: "retry", instruction: 'Could not parse exit code from notes. Include "exit code: 0" (or non-zero) in your notes.' };
+    }
+    const exitCode = parseInt(exitCodeMatch[1], 10);
+    if (exitCode === 0) {
+      ctx.writeState({ buildRetryCount: 0 });
+      ctx.appendEvent("build_passed", { retryCount, notes: notes.slice(0, 200) });
+      return { action: "advance" };
+    }
+    if (retryCount < MAX_BUILD_RETRIES) {
+      ctx.writeState({ buildRetryCount: retryCount + 1 });
+      ctx.appendEvent("build_failed_retry", { retryCount: retryCount + 1, notes: notes.slice(0, 200) });
+      return {
+        action: "back",
+        target: "IMPLEMENT",
+        reason: `Build failed (attempt ${retryCount + 1}/${MAX_BUILD_RETRIES}). Fix the build errors.`
+      };
+    }
+    ctx.writeState({ buildRetryCount: 0 });
+    ctx.appendEvent("build_failed_exhausted", { retryCount, notes: notes.slice(0, 200) });
+    return {
+      action: "advance",
+      result: {
+        instruction: [
+          "# Build Failed - Proceeding",
+          "",
+          `Build failed after ${MAX_BUILD_RETRIES} retries. Proceeding but build errors remain.`,
+          "",
+          "Document the build failure in the commit message."
+        ].join("\n"),
+        reminders: ["Mention build failure in the commit message."]
+      }
+    };
+  }
+};
+
 // src/autonomous/stages/verify.ts
 init_esm_shims();
 var MAX_VERIFY_RETRIES = 3;
@@ -7219,6 +7315,7 @@ registerStage(new ImplementStage());
 registerStage(new WriteTestsStage());
 registerStage(new TestStage());
 registerStage(new CodeReviewStage());
+registerStage(new BuildStage());
 registerStage(new VerifyStage());
 registerStage(new FinalizeStage());
 registerStage(new CompleteStage());
@@ -9414,7 +9511,7 @@ async function ensureGitignoreEntries(gitignorePath, entries) {
 // src/mcp/index.ts
 var ENV_VAR2 = "CLAUDESTORY_PROJECT_ROOT";
 var CONFIG_PATH2 = ".story/config.json";
-var version = "0.1.32";
+var version = "0.1.33";
 function tryDiscoverRoot() {
   const envRoot = process.env[ENV_VAR2];
   if (envRoot) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@anthropologies/claudestory",
-  "version": "0.1.32",
+  "version": "0.1.33",
   "license": "UNLICENSED",
   "description": "Cross-session context persistence for AI coding projects. Tracks tickets, issues, roadmap, and handovers so every session builds on the last.",
   "keywords": [