@nathapp/nax 0.50.3 → 0.51.2

package/src/prompts/types.ts

@@ -1,21 +0,0 @@
-/**
- * PromptBuilder Types
- *
- * Shared types for the unified prompt building system.
- */
-
-/** Role determining which default template body to use */
-export type PromptRole = "test-writer" | "implementer" | "verifier" | "single-session" | "tdd-simple" | "batch";
-
-/** A single section of a composed prompt */
-export interface PromptSection {
-  /** Unique section identifier */
-  id: string;
-  /** Section content */
-  content: string;
-  /** Whether this section can be removed by user override */
-  overridable: boolean;
-}
-
-/** Options passed to PromptBuilder.for() */
-export type PromptOptions = Record<string, unknown>;

package/src/queue/index.ts

@@ -1,2 +0,0 @@
-export type { QueueItem, QueueItemStatus, QueueStats, QueueCommand, QueueFileResult } from "./types";
-export { QueueManager, parseQueueFile } from "./manager";

package/src/queue/manager.ts

@@ -1,254 +0,0 @@
-/**
- * Queue Manager
- *
- * Manages priority queue for user stories.
- * Supports dependency-based ordering and multi-agent execution.
- */
-
-import type { QueueCommand, QueueFileResult, QueueItem, QueueItemStatus, QueueStats } from "./types";
-
-export class QueueManager {
-  private items: QueueItem[] = [];
-
-  /**
-   * Add a story to the queue.
-   * Priority is used for ordering (higher = more urgent).
-   */
-  enqueue(storyId: string, title: string, priority: number): void {
-    const existing = this.items.find((item) => item.storyId === storyId);
-    if (existing) {
-      throw new Error(`Story ${storyId} already in queue`);
-    }
-
-    const item: QueueItem = {
-      storyId,
-      title,
-      priority,
-      status: "pending",
-      attempts: 0,
-      addedAt: new Date(),
-    };
-
-    this.items.push(item);
-    // Sort by priority (descending) — higher priority first
-    this.items.sort((a, b) => b.priority - a.priority);
-  }
-
-  /**
-   * Get the next pending item (highest priority).
-   * Returns null if no pending items.
-   */
-  dequeue(): QueueItem | null {
-    const pending = this.items.find((item) => item.status === "pending");
-    return pending ?? null;
-  }
-
-  /**
-   * Peek at the next pending item without removing it.
-   */
-  peek(): QueueItem | null {
-    return this.dequeue();
-  }
-
-  /**
-   * Mark a story as in-progress and assign it to an agent.
-   */
-  markInProgress(storyId: string, assignedAgent: string): void {
-    const item = this.items.find((i) => i.storyId === storyId);
-    if (!item) {
-      throw new Error(`Story ${storyId} not found in queue`);
-    }
-
-    item.status = "in-progress";
-    item.assignedAgent = assignedAgent;
-    item.startedAt = new Date();
-    item.attempts += 1;
-  }
-
-  /**
-   * Mark a story as completed.
-   */
-  markComplete(storyId: string): void {
-    const item = this.items.find((i) => i.storyId === storyId);
-    if (!item) {
-      throw new Error(`Story ${storyId} not found in queue`);
-    }
-
-    item.status = "completed";
-    item.completedAt = new Date();
-  }
-
-  /**
-   * Mark a story as failed with an error message.
-   */
-  markFailed(storyId: string, error: string): void {
-    const item = this.items.find((i) => i.storyId === storyId);
-    if (!item) {
-      throw new Error(`Story ${storyId} not found in queue`);
-    }
-
-    item.status = "failed";
-    item.error = error;
-    item.completedAt = new Date();
-  }
-
-  /**
-   * Reset a story back to pending (e.g., for retry).
-   */
-  resetToPending(storyId: string): void {
-    const item = this.items.find((i) => i.storyId === storyId);
-    if (!item) {
-      throw new Error(`Story ${storyId} not found in queue`);
-    }
-
-    item.status = "pending";
-    item.assignedAgent = undefined;
-    item.startedAt = undefined;
-    item.completedAt = undefined;
-    item.error = undefined;
-    // Re-sort after status change
-    this.items.sort((a, b) => b.priority - a.priority);
-  }
-
-  /**
-   * Get all items with a specific status.
-   */
-  getByStatus(status: QueueItemStatus): QueueItem[] {
-    return this.items.filter((item) => item.status === status);
-  }
-
-  /**
-   * Get a specific item by story ID.
-   */
-  getItem(storyId: string): QueueItem | null {
-    return this.items.find((i) => i.storyId === storyId) ?? null;
-  }
-
-  /**
-   * Get all items in the queue.
-   */
-  getAllItems(): QueueItem[] {
-    return [...this.items];
-  }
-
-  /**
-   * Get queue statistics.
-   */
-  getStats(): QueueStats {
-    return {
-      total: this.items.length,
-      pending: this.items.filter((i) => i.status === "pending").length,
-      inProgress: this.items.filter((i) => i.status === "in-progress").length,
-      completed: this.items.filter((i) => i.status === "completed").length,
-      failed: this.items.filter((i) => i.status === "failed").length,
-      skipped: this.items.filter((i) => i.status === "skipped").length,
-    };
-  }
-
-  /**
-   * Remove a story from the queue entirely.
-   */
-  remove(storyId: string): void {
-    const index = this.items.findIndex((i) => i.storyId === storyId);
-    if (index === -1) {
-      throw new Error(`Story ${storyId} not found in queue`);
-    }
-    this.items.splice(index, 1);
-  }
-
-  /**
-   * Clear all items from the queue.
-   */
-  clear(): void {
-    this.items = [];
-  }
-
-  /**
-   * Check if the queue is empty.
-   */
-  isEmpty(): boolean {
-    return this.items.length === 0;
-  }
-
-  /**
-   * Check if there are any pending items.
-   */
-  hasPending(): boolean {
-    return this.items.some((i) => i.status === "pending");
-  }
-
-  /**
-   * Mark a story as skipped.
-   */
-  markSkipped(storyId: string): void {
-    const item = this.items.find((i) => i.storyId === storyId);
-    if (!item) {
-      throw new Error(`Story ${storyId} not found in queue`);
-    }
-
-    item.status = "skipped";
-    item.error = "Skipped by user command";
-    item.completedAt = new Date();
-  }
-}
-
-/**
- * Parse queue file content into commands and guidance.
- *
- * Commands:
- * - PAUSE: Pause execution after current story
- * - ABORT: Mark all remaining stories as skipped and stop
- * - SKIP US-XXX: Skip a specific story
- *
- * Everything else after "--- PENDING ---" is treated as guidance text.
- */
-export function parseQueueFile(content: string): QueueFileResult {
-  const commands: QueueCommand[] = [];
-  const guidance: string[] = [];
-
-  const lines = content.split("\n");
-  let inPendingSection = false;
-
-  for (const line of lines) {
-    const trimmed = line.trim();
-
-    // Skip empty lines
-    if (!trimmed) {
-      continue;
-    }
-
-    // Check for pending section marker
-    if (trimmed === "--- PENDING ---") {
-      inPendingSection = true;
-      continue;
-    }
-
-    // Parse commands (case-insensitive)
-    const upper = trimmed.toUpperCase();
-
-    if (upper === "PAUSE") {
-      commands.push({ type: "PAUSE" });
-    } else if (upper === "ABORT") {
-      commands.push({ type: "ABORT" });
-    } else if (upper.startsWith("SKIP ")) {
-      // Extract story ID after "SKIP"
-      const storyId = trimmed.substring(5).trim();
-      if (storyId) {
-        commands.push({ type: "SKIP", storyId });
-      } else {
-        // No story ID, treat as guidance
-        guidance.push(trimmed);
-      }
-    } else if (upper === "SKIP") {
-      // SKIP with no story ID, treat as guidance
-      guidance.push(trimmed);
-    } else {
-      // Not a command, treat as guidance if in pending section
-      if (inPendingSection) {
-        guidance.push(trimmed);
-      }
-    }
-  }
-
-  return { commands, guidance };
-}

package/src/queue/types.ts

@@ -1,54 +0,0 @@
-/**
- * Queue Types
- *
- * Priority queue for scheduling user stories across agents.
- * Enables future parallel agent execution.
- */
-
-/** Queue item status */
-export type QueueItemStatus = "pending" | "in-progress" | "completed" | "failed" | "skipped";
-
-/** Queue item representing a user story to be executed */
-export interface QueueItem {
-  /** Unique story ID */
-  storyId: string;
-  /** Display title */
-  title: string;
-  /** Priority score (higher = more urgent) */
-  priority: number;
-  /** Current status */
-  status: QueueItemStatus;
-  /** Assigned agent (if any) */
-  assignedAgent?: string;
-  /** Number of retry attempts */
-  attempts: number;
-  /** Timestamp when added to queue */
-  addedAt: Date;
-  /** Timestamp when started (if in-progress) */
-  startedAt?: Date;
-  /** Timestamp when completed/failed */
-  completedAt?: Date;
-  /** Error message (if failed) */
-  error?: string;
-}
-
-/** Queue statistics */
-export interface QueueStats {
-  total: number;
-  pending: number;
-  inProgress: number;
-  completed: number;
-  failed: number;
-  skipped: number;
-}
-
-/** Queue command for mid-run control */
-export type QueueCommand = { type: "PAUSE" } | { type: "ABORT" } | { type: "SKIP"; storyId: string };
-
-/** Result of parsing a queue file */
-export interface QueueFileResult {
-  /** Parsed commands from the queue file */
-  commands: QueueCommand[];
-  /** Non-command lines (existing guidance behavior) */
-  guidance: string[];
-}

package/src/review/index.ts

@@ -1,8 +0,0 @@
-/**
- * Review Module
- *
- * Post-implementation quality verification
- */
-
-export * from "./types";
-export * from "./runner";

package/src/review/orchestrator.ts

@@ -1,154 +0,0 @@
-/**
- * Review Orchestrator (ADR-005, Phase 2)
- *
- * Single entry point for all post-implementation review. Delegates to the
- * review runner and plugin reviewers. Provides a unified result.
- *
- * Usage:
- *   const result = await reviewOrchestrator.review(config, workdir, executionConfig, plugins);
- */
-
-import { spawn } from "bun";
-import type { NaxConfig } from "../config";
-import { getSafeLogger } from "../logger";
-import type { PluginRegistry } from "../plugins";
-import { errorMessage } from "../utils/errors";
-import { runReview } from "./runner";
-import type { ReviewConfig, ReviewResult } from "./types";
-
-/**
- * Injectable dependencies for getChangedFiles() — allows tests to intercept
- * spawn calls without requiring the git binary.
- *
- * @internal
- */
-export const _orchestratorDeps = { spawn };
-
-async function getChangedFiles(workdir: string, baseRef?: string): Promise<string[]> {
-  try {
-    const diffArgs = ["diff", "--name-only"];
-    const [stagedProc, unstagedProc, baseProc] = [
-      _orchestratorDeps.spawn({ cmd: ["git", ...diffArgs, "--cached"], cwd: workdir, stdout: "pipe", stderr: "pipe" }),
-      _orchestratorDeps.spawn({ cmd: ["git", ...diffArgs], cwd: workdir, stdout: "pipe", stderr: "pipe" }),
-      baseRef
-        ? _orchestratorDeps.spawn({
-            cmd: ["git", ...diffArgs, `${baseRef}...HEAD`],
-            cwd: workdir,
-            stdout: "pipe",
-            stderr: "pipe",
-          })
-        : null,
-    ];
-
-    await Promise.all([stagedProc.exited, unstagedProc.exited, baseProc?.exited]);
-
-    const [staged, unstaged, based] = await Promise.all([
-      new Response(stagedProc.stdout).text().then((t) => t.trim().split("\n").filter(Boolean)),
-      new Response(unstagedProc.stdout).text().then((t) => t.trim().split("\n").filter(Boolean)),
-      baseProc
-        ? new Response(baseProc.stdout).text().then((t) => t.trim().split("\n").filter(Boolean))
-        : Promise.resolve([]),
-    ]);
-
-    return Array.from(new Set([...staged, ...unstaged, ...based]));
-  } catch {
-    return [];
-  }
-}
-
-export interface OrchestratorReviewResult {
-  /** Built-in review result (typecheck, lint, format) */
-  builtIn: ReviewResult;
-  /** Whether ALL checks passed (built-in + plugin reviewers) */
-  success: boolean;
-  /** Failure reason if success === false */
-  failureReason?: string;
-  /** Plugin reviewer hard-failure flag (determines escalate vs fail) */
-  pluginFailed: boolean;
-}
-
-export class ReviewOrchestrator {
-  /** Run built-in checks + plugin reviewers. Returns unified result. */
-  async review(
-    reviewConfig: ReviewConfig,
-    workdir: string,
-    executionConfig: NaxConfig["execution"],
-    plugins?: PluginRegistry,
-    storyGitRef?: string,
-    scopePrefix?: string,
-  ): Promise<OrchestratorReviewResult> {
-    const logger = getSafeLogger();
-
-    const builtIn = await runReview(reviewConfig, workdir, executionConfig);
-
-    if (!builtIn.success) {
-      return { builtIn, success: false, failureReason: builtIn.failureReason, pluginFailed: false };
-    }
-
-    if (reviewConfig.pluginMode === "deferred") {
-      logger?.debug("review", "Plugin reviewers deferred — skipping per-story execution");
-      return { builtIn, success: true, pluginFailed: false };
-    }
-
-    if (plugins) {
-      const reviewers = plugins.getReviewers();
-      if (reviewers.length > 0) {
-        // 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: scopedFiles.length,
-          });
-          try {
-            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,
-              exitCode: result.exitCode,
-              output: result.output?.slice(0, 500),
-              findings: result.findings?.length ?? 0,
-            });
-            pluginResults.push({
-              name: reviewer.name,
-              passed: result.passed,
-              output: result.output,
-              exitCode: result.exitCode,
-              findings: result.findings,
-            });
-            if (!result.passed) {
-              builtIn.pluginReviewers = pluginResults;
-              return {
-                builtIn,
-                success: false,
-                failureReason: `plugin reviewer '${reviewer.name}' failed`,
-                pluginFailed: true,
-              };
-            }
-          } catch (error) {
-            const errorMsg = errorMessage(error);
-            logger?.warn("review", `Plugin reviewer threw error: ${reviewer.name}`, { error: errorMsg });
-            pluginResults.push({ name: reviewer.name, passed: false, output: "", error: errorMsg });
-            builtIn.pluginReviewers = pluginResults;
-            return {
-              builtIn,
-              success: false,
-              failureReason: `plugin reviewer '${reviewer.name}' threw error`,
-              pluginFailed: true,
-            };
-          }
-        }
-        builtIn.pluginReviewers = pluginResults;
-      }
-    }
-
-    return { builtIn, success: true, pluginFailed: false };
-  }
-}
-
-export const reviewOrchestrator = new ReviewOrchestrator();