@nathapp/nax 0.32.2 → 0.34.0

package/src/prd/index.ts

@@ -107,6 +107,7 @@ export function getNextStory(prd: PRD, currentStoryId?: string | null, maxRetrie
         s.status !== "blocked" &&
         s.status !== "failed" &&
         s.status !== "paused" &&
+        s.status !== "decomposed" &&
         s.dependencies.every((dep) => completedIds.has(dep)),
     ) ?? null
   );
@@ -132,11 +133,12 @@ export function countStories(prd: PRD): {
   skipped: number;
   blocked: number;
   paused: number;
+  decomposed: number;
 } {
   return {
     total: prd.userStories.length,
     passed: prd.userStories.filter((s) => s.passes || s.status === "passed").length,
-    failed: prd.userStories.filter((s) => s.status === "failed").length,
+    failed: prd.userStories.filter((s) => s.status === "failed" || s.status === "regression-failed").length,
     pending: prd.userStories.filter(
       (s) =>
         !s.passes &&
@@ -144,11 +146,14 @@ export function countStories(prd: PRD): {
         s.status !== "failed" &&
         s.status !== "skipped" &&
         s.status !== "blocked" &&
-        s.status !== "paused",
+        s.status !== "paused" &&
+        s.status !== "regression-failed" &&
+        s.status !== "decomposed",
     ).length,
     skipped: prd.userStories.filter((s) => s.status === "skipped").length,
     blocked: prd.userStories.filter((s) => s.status === "blocked").length,
     paused: prd.userStories.filter((s) => s.status === "paused").length,
+    decomposed: prd.userStories.filter((s) => s.status === "decomposed").length,
   };
 }
 

package/src/prd/types.ts

@@ -9,7 +9,16 @@ import type { ModelTier } from "../config";
 import type { FailureCategory } from "../tdd/types";
 
 /** User story status */
-export type StoryStatus = "pending" | "in-progress" | "passed" | "failed" | "skipped" | "blocked" | "paused";
+export type StoryStatus =
+  | "pending"
+  | "in-progress"
+  | "passed"
+  | "failed"
+  | "skipped"
+  | "blocked"
+  | "paused"
+  | "regression-failed"
+  | "decomposed";
 
 /** Verification stage where failure occurred */
 export type VerificationStage = "verify" | "review" | "regression" | "rectification" | "agent-session" | "escalation";
@@ -38,6 +47,8 @@ export interface StructuredFailure {
   summary: string;
   /** Parsed test failures (if applicable) */
   testFailures?: TestFailureContext[];
+  /** Structured review findings from plugin reviewers (e.g., semgrep, eslint) */
+  reviewFindings?: import("../plugins/types").ReviewFinding[];
   /** ISO timestamp when failure was recorded */
   timestamp: string;
 }
@@ -150,7 +161,10 @@ export function isStalled(prd: PRD): boolean {
 
   const blockedIds = new Set(
     prd.userStories
-      .filter((s) => s.status === "blocked" || s.status === "failed" || s.status === "paused")
+      .filter(
+        (s) =>
+          s.status === "blocked" || s.status === "failed" || s.status === "paused" || s.status === "regression-failed",
+      )
       .map((s) => s.id),
   );
 
@@ -159,6 +173,7 @@ export function isStalled(prd: PRD): boolean {
       s.status === "blocked" ||
       s.status === "failed" ||
       s.status === "paused" ||
+      s.status === "regression-failed" ||
       s.dependencies.some((dep) => blockedIds.has(dep)),
   );
 }

package/src/review/orchestrator.ts

@@ -72,6 +72,7 @@ export class ReviewOrchestrator {
               passed: result.passed,
               output: result.output,
               exitCode: result.exitCode,
+              findings: result.findings,
             });
             if (!result.passed) {
               builtIn.pluginReviewers = pluginResults;

package/src/review/types.ts

@@ -35,6 +35,8 @@ export interface PluginReviewerResult {
   exitCode?: number;
   /** Error message if reviewer threw an exception */
   error?: string;
+  /** Structured findings from the reviewer (optional) */
+  findings?: import("../plugins/types").ReviewFinding[];
 }
 
 /** Review phase result */

package/src/tdd/isolation.ts

@@ -55,7 +55,7 @@ function matchesAllowedPath(filePath: string, allowedPaths: string[]): boolean {
   return allowedPaths.some((pattern) => {
     // Simple glob matching: ** = any directory, * = any filename segment
     const regexPattern = pattern.replace(/\*\*/g, ".*").replace(/\*/g, "[^/]*").replace(/\//g, "\\/");
-    const regex = new RegExp(`^${regexPattern}$`);
+    const regex = new RegExp(`^${regexPattern}$`); // nosemgrep: detect-non-literal-regexp — pattern from PRD scope config, not user input
     return regex.test(filePath);
   });
 }