@nathapp/nax 0.46.3 → 0.48.0

package/src/review/orchestrator.ts

@@ -75,6 +75,7 @@ export class ReviewOrchestrator {
     executionConfig: NaxConfig["execution"],
     plugins?: PluginRegistry,
     storyGitRef?: string,
+    scopePrefix?: string,
   ): Promise<OrchestratorReviewResult> {
     const logger = getSafeLogger();
 
@@ -95,14 +96,17 @@ export class ReviewOrchestrator {
         // Use the story's start ref if available to capture auto-committed changes
         const baseRef = storyGitRef ?? executionConfig?.storyGitRef;
         const changedFiles = await getChangedFiles(workdir, baseRef);
+        const scopedFiles = scopePrefix
+          ? changedFiles.filter((f) => f === scopePrefix || f.startsWith(`${scopePrefix}/`))
+          : changedFiles;
         const pluginResults: ReviewResult["pluginReviewers"] = [];
 
         for (const reviewer of reviewers) {
           logger?.info("review", `Running plugin reviewer: ${reviewer.name}`, {
-            changedFiles: changedFiles.length,
+            changedFiles: scopedFiles.length,
           });
           try {
-            const result = await reviewer.check(workdir, changedFiles);
+            const result = await reviewer.check(workdir, scopedFiles);
             // Always log the result so skips/passes are visible in the log
             logger?.info("review", `Plugin reviewer result: ${reviewer.name}`, {
               passed: result.passed,

package/src/verification/smart-runner.ts

@@ -194,7 +194,27 @@ export function buildSmartTestCommand(testFiles: string[], baseCommand: string):
   return newParts.join(" ");
 }
 
-export async function getChangedSourceFiles(workdir: string, baseRef?: string): Promise<string[]> {
+/**
+ * Get TypeScript source files changed since the previous commit.
+ *
+ * Runs `git diff --name-only <ref>` in the given workdir and filters
+ * results to only `.ts` files under the relevant source prefix.
+ *
+ * In a monorepo, git returns paths relative to the git root (e.g.
+ * `packages/api/src/foo.ts`). When `packagePrefix` is set to the
+ * story's workdir (e.g. `"packages/api"`), the filter is scoped to
+ * `<packagePrefix>/src/` instead of just `src/`.
+ *
+ * @param workdir       - Working directory to run git command in
+ * @param baseRef       - Git ref for diff base (default: HEAD~1)
+ * @param packagePrefix - Story workdir relative to repo root (e.g. "packages/api")
+ * @returns Array of changed .ts file paths relative to the git root
+ */
+export async function getChangedSourceFiles(
+  workdir: string,
+  baseRef?: string,
+  packagePrefix?: string,
+): Promise<string[]> {
   try {
     // FEAT-010: Use per-attempt baseRef for precise diff; fall back to HEAD~1 if not provided
     const ref = baseRef ?? "HEAD~1";
@@ -204,7 +224,9 @@ export async function getChangedSourceFiles(workdir: string, baseRef?: string):
 
     const lines = stdout.trim().split("\n").filter(Boolean);
 
-    return lines.filter((f) => f.startsWith("src/") && f.endsWith(".ts"));
+    // MW-006: scope filter to package prefix in monorepo
+    const srcPrefix = packagePrefix ? `${packagePrefix}/src/` : "src/";
+    return lines.filter((f) => f.startsWith(srcPrefix) && f.endsWith(".ts"));
   } catch {
     return [];
   }