@gethmy/agent 1.0.0 → 1.0.2

package/dist/review-completion.d.ts

@@ -0,0 +1,31 @@
+import type { HarmonyApiClient } from "@gethmy/mcp/src/api-client.js";
+import type { Card } from "@harmony/shared";
+import type { AgentConfig } from "./types.js";
+export interface ReviewFinding {
+    severity: "critical" | "major" | "minor";
+    title: string;
+    description: string;
+    category?: string;
+    location?: string;
+}
+export interface ScopeCheck {
+    status: "clean" | "drift" | "missing";
+    notes?: string;
+}
+export interface ReviewResult {
+    verdict: "approved" | "rejected";
+    summary: string;
+    scopeCheck?: ScopeCheck;
+    findings: ReviewFinding[];
+}
+/**
+ * Parse Claude's review output into a structured ReviewResult.
+ * Looks for a JSON block in the output.
+ */
+export declare function parseReviewOutput(stdout: string): ReviewResult;
+/**
+ * Post-review completion pipeline.
+ * Handles approved/rejected verdicts, creates subtasks for findings,
+ * and moves the card to the appropriate column.
+ */
+export declare function runReviewCompletion(client: HarmonyApiClient, card: Card, result: ReviewResult, config: AgentConfig, worktreePath: string, branchName: string | null): Promise<void>;

package/dist/review-completion.js

@@ -0,0 +1,253 @@
+import { addLabelByName, moveCardToColumn } from "./board-helpers.js";
+import { createPullRequest, detectGitProvider, pushBranch } from "./git-pr.js";
+import { log } from "./log.js";
+import { cleanupWorktree } from "./worktree.js";
+const TAG = "review-completion";
+const MAX_FINDINGS = 10;
+const REVIEW_MARKER = "---\n**Review:";
+/**
+ * Parse Claude's review output into a structured ReviewResult.
+ * Looks for a JSON block in the output.
+ */
+export function parseReviewOutput(stdout) {
+    // Try to find a JSON block in the output
+    const jsonMatch = stdout.match(/```json\s*([\s\S]*?)```/);
+    const rawJson = jsonMatch?.[1]?.trim() ?? stdout.trim();
+    try {
+        const parsed = JSON.parse(rawJson);
+        const verdict = parsed.verdict === "approved" || parsed.verdict === "rejected"
+            ? parsed.verdict
+            : "rejected";
+        const findings = Array.isArray(parsed.findings)
+            ? parsed.findings
+                .filter((f) => typeof f === "object" && f !== null && "title" in f)
+                .map((f) => ({
+                severity: f.severity === "critical"
+                    ? "critical"
+                    : f.severity === "minor"
+                        ? "minor"
+                        : "major",
+                title: String(f.title ?? "Untitled finding"),
+                description: String(f.description ?? ""),
+                category: f.category ? String(f.category) : undefined,
+                location: f.location ? String(f.location) : undefined,
+            }))
+            : [];
+        const scopeCheck = parsed.scopeCheck &&
+            typeof parsed.scopeCheck === "object" &&
+            "status" in parsed.scopeCheck
+            ? {
+                status: ["clean", "drift", "missing"].includes(parsed.scopeCheck.status)
+                    ? parsed.scopeCheck.status
+                    : "clean",
+                notes: parsed.scopeCheck.notes
+                    ? String(parsed.scopeCheck.notes)
+                    : undefined,
+            }
+            : undefined;
+        return {
+            verdict,
+            summary: String(parsed.summary ?? "").slice(0, 2000),
+            scopeCheck,
+            findings,
+        };
+    }
+    catch {
+        // If we can't parse JSON, treat as rejection with the raw output as summary
+        log.warn(TAG, "Failed to parse review JSON output, treating as rejection");
+        return {
+            verdict: "rejected",
+            summary: stdout.slice(0, 500),
+            findings: [],
+        };
+    }
+}
+/**
+ * Get the current review cycle count from card description.
+ * Looks for: Review cycle: N/M
+ */
+function getReviewCycle(description) {
+    if (!description)
+        return 0;
+    const match = description.match(/Review cycle:\s*(\d+)/);
+    return match ? parseInt(match[1], 10) : 0;
+}
+/**
+ * Update the review cycle marker in the card description.
+ */
+function updateReviewCycleMarker(description, cycle, maxCycles) {
+    const marker = `Review cycle: ${cycle}/${maxCycles}`;
+    const existing = description.match(/Review cycle:\s*\d+\/\d+/);
+    if (existing) {
+        return description.replace(/Review cycle:\s*\d+\/\d+/, marker);
+    }
+    return `${description}\n\n${marker}`;
+}
+/**
+ * Strip any previous review summary block from the description.
+ */
+function stripReviewSummary(description) {
+    const idx = description.indexOf(REVIEW_MARKER);
+    if (idx === -1)
+        return description;
+    return description.slice(0, idx).trimEnd();
+}
+/**
+ * Post-review completion pipeline.
+ * Handles approved/rejected verdicts, creates subtasks for findings,
+ * and moves the card to the appropriate column.
+ */
+export async function runReviewCompletion(client, card, result, config, worktreePath, branchName) {
+    // Re-fetch card for fresh description (avoids stale data from enqueue time)
+    let freshDesc;
+    try {
+        const { card: fresh } = (await client.getCard(card.id));
+        freshDesc = fresh.description || "";
+    }
+    catch {
+        freshDesc = card.description || "";
+    }
+    const currentCycle = getReviewCycle(freshDesc) + 1;
+    const maxCycles = config.review.maxReviewCycles;
+    if (result.verdict === "approved") {
+        // Ensure branch is pushed (skip in local mode — no branch to push)
+        let prUrl = null;
+        if (branchName) {
+            pushBranch(branchName, worktreePath);
+            // Create PR if configured
+            if (config.review.createPR) {
+                const provider = detectGitProvider(worktreePath);
+                prUrl = createPullRequest(card, branchName, worktreePath, config, provider);
+            }
+        }
+        // Add "Ready to Merge" label
+        await addLabelByName(client, card, config.review.approvedLabel, config.review.approvedLabelColor);
+        // Post approval summary (card stays in Review, strip stale cycle marker)
+        if (config.review.postFindings) {
+            const baseDesc = stripReviewSummary(freshDesc).replace(/\n\nReview cycle:\s*\d+\/\d+/, "");
+            const scopeLine = result.scopeCheck
+                ? `\nScope: ${result.scopeCheck.status}${result.scopeCheck.notes ? ` — ${result.scopeCheck.notes}` : ""}`
+                : "";
+            const summaryParts = [
+                `\n\n${REVIEW_MARKER} Approved**`,
+                result.summary ? `\n${result.summary}` : "",
+                scopeLine,
+                result.findings.length > 0
+                    ? `\n${result.findings.length} minor finding(s) noted.`
+                    : "",
+            ];
+            if (prUrl) {
+                summaryParts.push(`\nPR: ${prUrl}`);
+            }
+            const summary = summaryParts.join("");
+            try {
+                await client.updateCard(card.id, { description: baseDesc + summary });
+            }
+            catch (err) {
+                log.error(TAG, `Failed to update description: ${err instanceof Error ? err.message : err}`);
+            }
+        }
+        await client.endAgentSession(card.id, {
+            status: "completed",
+            progressPercent: 100,
+        });
+        log.info(TAG, `#${card.short_id} approved${prUrl ? ` — PR: ${prUrl}` : ""} — labeled "${config.review.approvedLabel}"`);
+    }
+    else {
+        // Rejected
+        const criticalFindings = result.findings
+            .filter((f) => f.severity === "critical")
+            .slice(0, MAX_FINDINGS);
+        const majorFindings = result.findings
+            .filter((f) => f.severity === "major")
+            .slice(0, MAX_FINDINGS);
+        const linkedFindings = [...criticalFindings, ...majorFindings];
+        const minorFindings = result.findings
+            .filter((f) => f.severity === "minor")
+            .slice(0, MAX_FINDINGS);
+        // Check if we've exceeded max review cycles
+        if (currentCycle >= maxCycles) {
+            log.warn(TAG, `#${card.short_id} reached max review cycles (${maxCycles}), moving to Done with note`);
+            await moveCardToColumn(client, card, config.review.moveToColumn);
+            const baseDesc = stripReviewSummary(freshDesc);
+            const summary = [
+                `\n\n${REVIEW_MARKER} Needs human review**`,
+                `\nReached max review cycles (${maxCycles}). Please review manually.`,
+                result.summary ? `\n${result.summary}` : "",
+            ].join("");
+            try {
+                await client.updateCard(card.id, { description: baseDesc + summary });
+            }
+            catch (err) {
+                log.error(TAG, `Failed to update description: ${err instanceof Error ? err.message : err}`);
+            }
+            await client.endAgentSession(card.id, { status: "completed" });
+            if (branchName) {
+                cleanupWorktree(worktreePath, branchName);
+            }
+            return;
+        }
+        // Post findings
+        if (config.review.postFindings) {
+            // Add critical + major findings as new linked cards (parallel)
+            await Promise.all(linkedFindings.map(async (finding) => {
+                try {
+                    const locationLine = finding.location
+                        ? `\n**Location:** ${finding.location}`
+                        : "";
+                    const newCard = await client.createCard(card.project_id, {
+                        title: `[Review: ${finding.severity}] ${finding.title}`,
+                        description: `Found during review of #${card.short_id} (${finding.severity}):\n\n${finding.description}${locationLine}`,
+                    });
+                    const newCardId = newCard?.card?.id;
+                    if (newCardId) {
+                        await client.addLinkToCard(card.id, newCardId, "relates_to");
+                    }
+                }
+                catch (err) {
+                    log.error(TAG, `Failed to create finding card: ${err instanceof Error ? err.message : err}`);
+                }
+            }));
+            // Add minor findings as subtasks (parallel)
+            await Promise.all(minorFindings.map(async (finding) => {
+                try {
+                    const title = finding.title.length > 120
+                        ? `${finding.title.slice(0, 117)}...`
+                        : finding.title;
+                    await client.createSubtask(card.id, title);
+                }
+                catch (err) {
+                    log.error(TAG, `Failed to create subtask: ${err instanceof Error ? err.message : err}`);
+                }
+            }));
+            // Update description with review summary and cycle counter
+            const baseDesc = stripReviewSummary(freshDesc);
+            const updatedDesc = updateReviewCycleMarker(baseDesc, currentCycle, maxCycles);
+            const scopeLine = result.scopeCheck
+                ? `\nScope: ${result.scopeCheck.status}${result.scopeCheck.notes ? ` — ${result.scopeCheck.notes}` : ""}`
+                : "";
+            const summary = [
+                `\n\n${REVIEW_MARKER} Rejected**`,
+                result.summary ? `\n${result.summary}` : "",
+                scopeLine,
+                `\n${criticalFindings.length} critical, ${majorFindings.length} major, ${minorFindings.length} minor finding(s).`,
+            ].join("");
+            try {
+                await client.updateCard(card.id, {
+                    description: updatedDesc + summary,
+                });
+            }
+            catch (err) {
+                log.error(TAG, `Failed to update description: ${err instanceof Error ? err.message : err}`);
+            }
+        }
+        // Move back to failColumn (To Do) for re-implementation
+        await moveCardToColumn(client, card, config.review.failColumn);
+        await client.endAgentSession(card.id, { status: "paused" });
+        log.info(TAG, `#${card.short_id} rejected (cycle ${currentCycle}/${maxCycles}) — moved to "${config.review.failColumn}"`);
+    }
+    // Cleanup worktree (skip in local mode — no worktree to clean)
+    if (branchName) {
+        cleanupWorktree(worktreePath, branchName);
+    }
+}

package/dist/review-knowledge.d.ts

@@ -0,0 +1,14 @@
+/**
+ * Embedded review and QA knowledge for the review worker.
+ * Condensed from the /review checklist and /qa skill.
+ */
+/**
+ * Static system prompt with review methodology.
+ * Covers two-pass review categories, suppressions, and severity classification.
+ */
+export declare const REVIEW_SYSTEM_PROMPT = "You are a senior code reviewer. Follow this two-pass methodology strictly.\nReport findings; do NOT fix them. This is a read-only review.\n\n## Two-Pass Review\n\n### Pass 1 \u2014 CRITICAL (highest severity)\n\n**SQL & Data Safety**\n- String interpolation in SQL \u2014 use parameterized queries / prepared statements\n- TOCTOU races: check-then-set patterns that should be atomic WHERE + UPDATE\n\n**Race Conditions & Concurrency**\n- Read-check-write without uniqueness constraint or duplicate key handling\n- Status transitions without atomic WHERE old_status UPDATE SET new_status\n- Unsafe HTML rendering (dangerouslySetInnerHTML, v-html) on user-controlled data (XSS)\n\n**LLM Output Trust Boundary**\n- LLM-generated values written to DB without format validation (EMAIL_REGEXP, URI.parse, .trim())\n- Structured tool output accepted without type/shape checks before database writes\n\n**Enum & Value Completeness**\n- When the diff introduces a new enum/status/type value, trace it through every consumer\n- Check allowlists, filter arrays, and case/if-elsif chains for the new value\n- Use Grep to find all references to sibling values and Read each match \u2014 look OUTSIDE the diff\n\n### Pass 2 \u2014 INFORMATIONAL (lower severity)\n\n**Conditional Side Effects**\n- Code paths that branch but forget a side effect on one branch (e.g., promoting without attaching URL)\n\n**Dead Code & Consistency**\n- Variables assigned but never read\n- Comments/docstrings describing old behavior after code changed\n\n**Test Gaps**\n- Missing negative-path tests for new error handling\n- Security enforcement features without integration tests\n\n**Completeness Gaps**\n- Partial enum handling, incomplete error paths, missing edge cases that are straightforward to add\n\n**View/Frontend**\n- O(n*m) lookups in views (Array.find in a loop instead of Map/index)\n- Inline styles re-parsed every render\n\n## Severity Classification\n\n- **critical**: SQL safety, race conditions, XSS, LLM trust boundary violations, enum completeness gaps causing runtime errors\n- **major**: Missing requirements, broken functionality, significant completeness gaps, conditional side effects\n- **minor**: Dead code, stale comments, test gaps, minor view issues, cosmetic completeness gaps\n\n## Suppressions \u2014 DO NOT flag these\n\n- Redundancy that aids readability (e.g., present? redundant with length > 20)\n- \"Add a comment explaining why this threshold was chosen\" \u2014 thresholds change, comments rot\n- Consistency-only changes (wrapping a value to match how another constant is guarded)\n- Regex edge cases when input is constrained and the edge case never occurs in practice\n- Eval threshold changes \u2014 these are tuned empirically\n- Harmless no-ops (e.g., .reject on an element never in the array)\n- ANYTHING already addressed in the diff you are reviewing \u2014 read the FULL diff before flagging";
+/**
+ * Visual QA checklist for browser-based verification.
+ * Condensed from the /qa skill's per-page exploration checklist.
+ */
+export declare const QA_VISUAL_CHECKLIST = "## Visual QA Checklist\n\nFor each page affected by the changes:\n\n1. **Visual scan** \u2014 Screenshot the page. Check for layout breaks, broken images, alignment issues, z-index problems.\n2. **Interactive elements** \u2014 Click every button, link, and control. Does each do what it says?\n3. **Forms** \u2014 Fill and submit. Test empty submission, invalid data, edge cases.\n4. **Navigation** \u2014 Check all paths in/out. Breadcrumbs, back button, deep links.\n5. **States** \u2014 Check empty state, loading state, error state, overflow state.\n6. **Console** \u2014 Check for JS exceptions, failed network requests (4xx/5xx), CORS errors after interactions.\n7. **Responsiveness** \u2014 If the change is visual, check mobile viewport (375px).\n\n### SPA-Specific (React/Vite)\n- Use snapshot for navigation \u2014 client-side routes may not appear in link lists.\n- Check for stale state: navigate away and back \u2014 does data refresh correctly?\n- Test browser back/forward \u2014 does the app handle history correctly?\n- Watch for hydration errors or layout shifts after dynamic content loads.";

package/dist/review-knowledge.js

@@ -0,0 +1,89 @@
+/**
+ * Embedded review and QA knowledge for the review worker.
+ * Condensed from the /review checklist and /qa skill.
+ */
+/**
+ * Static system prompt with review methodology.
+ * Covers two-pass review categories, suppressions, and severity classification.
+ */
+export const REVIEW_SYSTEM_PROMPT = `You are a senior code reviewer. Follow this two-pass methodology strictly.
+Report findings; do NOT fix them. This is a read-only review.
+
+## Two-Pass Review
+
+### Pass 1 — CRITICAL (highest severity)
+
+**SQL & Data Safety**
+- String interpolation in SQL — use parameterized queries / prepared statements
+- TOCTOU races: check-then-set patterns that should be atomic WHERE + UPDATE
+
+**Race Conditions & Concurrency**
+- Read-check-write without uniqueness constraint or duplicate key handling
+- Status transitions without atomic WHERE old_status UPDATE SET new_status
+- Unsafe HTML rendering (dangerouslySetInnerHTML, v-html) on user-controlled data (XSS)
+
+**LLM Output Trust Boundary**
+- LLM-generated values written to DB without format validation (EMAIL_REGEXP, URI.parse, .trim())
+- Structured tool output accepted without type/shape checks before database writes
+
+**Enum & Value Completeness**
+- When the diff introduces a new enum/status/type value, trace it through every consumer
+- Check allowlists, filter arrays, and case/if-elsif chains for the new value
+- Use Grep to find all references to sibling values and Read each match — look OUTSIDE the diff
+
+### Pass 2 — INFORMATIONAL (lower severity)
+
+**Conditional Side Effects**
+- Code paths that branch but forget a side effect on one branch (e.g., promoting without attaching URL)
+
+**Dead Code & Consistency**
+- Variables assigned but never read
+- Comments/docstrings describing old behavior after code changed
+
+**Test Gaps**
+- Missing negative-path tests for new error handling
+- Security enforcement features without integration tests
+
+**Completeness Gaps**
+- Partial enum handling, incomplete error paths, missing edge cases that are straightforward to add
+
+**View/Frontend**
+- O(n*m) lookups in views (Array.find in a loop instead of Map/index)
+- Inline styles re-parsed every render
+
+## Severity Classification
+
+- **critical**: SQL safety, race conditions, XSS, LLM trust boundary violations, enum completeness gaps causing runtime errors
+- **major**: Missing requirements, broken functionality, significant completeness gaps, conditional side effects
+- **minor**: Dead code, stale comments, test gaps, minor view issues, cosmetic completeness gaps
+
+## Suppressions — DO NOT flag these
+
+- Redundancy that aids readability (e.g., present? redundant with length > 20)
+- "Add a comment explaining why this threshold was chosen" — thresholds change, comments rot
+- Consistency-only changes (wrapping a value to match how another constant is guarded)
+- Regex edge cases when input is constrained and the edge case never occurs in practice
+- Eval threshold changes — these are tuned empirically
+- Harmless no-ops (e.g., .reject on an element never in the array)
+- ANYTHING already addressed in the diff you are reviewing — read the FULL diff before flagging`;
+/**
+ * Visual QA checklist for browser-based verification.
+ * Condensed from the /qa skill's per-page exploration checklist.
+ */
+export const QA_VISUAL_CHECKLIST = `## Visual QA Checklist
+
+For each page affected by the changes:
+
+1. **Visual scan** — Screenshot the page. Check for layout breaks, broken images, alignment issues, z-index problems.
+2. **Interactive elements** — Click every button, link, and control. Does each do what it says?
+3. **Forms** — Fill and submit. Test empty submission, invalid data, edge cases.
+4. **Navigation** — Check all paths in/out. Breadcrumbs, back button, deep links.
+5. **States** — Check empty state, loading state, error state, overflow state.
+6. **Console** — Check for JS exceptions, failed network requests (4xx/5xx), CORS errors after interactions.
+7. **Responsiveness** — If the change is visual, check mobile viewport (375px).
+
+### SPA-Specific (React/Vite)
+- Use snapshot for navigation — client-side routes may not appear in link lists.
+- Check for stale state: navigate away and back — does data refresh correctly?
+- Test browser back/forward — does the app handle history correctly?
+- Watch for hydration errors or layout shifts after dynamic content loads.`;

package/dist/review-prompt.d.ts

@@ -0,0 +1,12 @@
+import type { EnrichedCard } from "./types.js";
+/**
+ * Build the static system prompt for the review agent.
+ * Contains review methodology, checklist, and QA guidance.
+ * Passed via --append-system-prompt to Claude CLI.
+ */
+export declare function buildReviewSystemPrompt(): string;
+/**
+ * Build the card-specific user prompt for the review agent.
+ * Contains the diff, requirements, and structured review steps.
+ */
+export declare function buildReviewUserPrompt(enriched: EnrichedCard, branchName: string | null, worktreePath: string, previewUrl: string, diff: string, baseBranch: string): string;

package/dist/review-prompt.js

@@ -0,0 +1,103 @@
+import { QA_VISUAL_CHECKLIST, REVIEW_SYSTEM_PROMPT, } from "./review-knowledge.js";
+/**
+ * Build the static system prompt for the review agent.
+ * Contains review methodology, checklist, and QA guidance.
+ * Passed via --append-system-prompt to Claude CLI.
+ */
+export function buildReviewSystemPrompt() {
+    return `You are a code review agent. You review changes made by an implementation agent.
+You are thorough, specific, and cite file:line locations for every finding.
+
+${REVIEW_SYSTEM_PROMPT}
+
+${QA_VISUAL_CHECKLIST}`;
+}
+/**
+ * Build the card-specific user prompt for the review agent.
+ * Contains the diff, requirements, and structured review steps.
+ */
+export function buildReviewUserPrompt(enriched, branchName, worktreePath, previewUrl, diff, baseBranch) {
+    const { card, labels, subtasks } = enriched;
+    const labelStr = labels.length > 0 ? labels.map((l) => l.name).join(", ") : "none";
+    const subtaskStr = subtasks.length > 0
+        ? subtasks
+            .map((s) => `- [${s.completed ? "x" : " "}] ${s.title}`)
+            .join("\n")
+        : "No subtasks defined.";
+    const description = card.description?.trim() || "No description provided.";
+    const truncatedDiff = diff.length > 80_000
+        ? `${diff.slice(0, 80_000)}\n\n... (diff truncated at 80K characters)`
+        : diff;
+    const branchLine = branchName
+        ? `**Branch**: ${branchName}`
+        : `**Mode**: Local review (no branch — reviewing working tree changes)`;
+    return `## Card: #${card.short_id} - ${card.title}
+**Labels**: ${labelStr}
+${branchLine}
+
+## Original Requirements
+${description}
+
+## Subtasks (Acceptance Criteria)
+${subtaskStr}
+
+## Diff ${branchName ? `(origin/${baseBranch}..HEAD)` : "(local changes)"}
+\`\`\`diff
+${truncatedDiff}
+\`\`\`
+
+## Review Steps
+
+Follow these steps in order:
+
+### Step 1: Scope Check
+Compare the diff against the card description and subtasks above.
+- Are all requirements from the description addressed?
+- Are all subtasks implemented?
+- Is there scope creep — changes unrelated to the card requirements?
+Flag any missing requirements or scope drift.
+
+### Step 2: Code Review (Two-Pass)
+Apply the two-pass review from your system instructions:
+- **Pass 1 (CRITICAL)**: SQL safety, race conditions, LLM trust boundary, enum completeness.
+- **Pass 2 (INFORMATIONAL)**: Conditional side effects, dead code, test gaps, completeness gaps, view issues.
+
+For enum completeness checks, use Grep and Read to trace new values through consumers OUTSIDE the diff.
+
+### Step 3: Visual QA
+Use the \`/browse\` skill to navigate to ${previewUrl} and apply the visual QA checklist:
+- Verify the UI changes match the requirements
+- Check interactive elements, forms, navigation
+- Check console for JS errors and failed network requests
+- Test on mobile viewport if the change is visual
+
+### Step 4: Output
+After completing all steps, output EXACTLY one JSON block (and nothing else after it):
+
+\`\`\`json
+{
+  "verdict": "approved" | "rejected",
+  "summary": "Brief overall assessment",
+  "scopeCheck": {
+    "status": "clean" | "drift" | "missing",
+    "notes": "Optional explanation of scope issues"
+  },
+  "findings": [
+    {
+      "severity": "critical" | "major" | "minor",
+      "category": "sql-safety | race-condition | llm-trust | enum-completeness | visual | functional | ux | console | scope | other",
+      "title": "Short title",
+      "description": "Detailed description of the issue",
+      "location": "file:line (if applicable)"
+    }
+  ]
+}
+\`\`\`
+
+**Decision rules:**
+- **rejected**: Any \`critical\` finding, unaddressed requirements, or 2+ \`major\` findings.
+- **approved**: No critical findings, at most 1 major finding with minor findings OK.
+
+**Do NOT modify any code.** This is a read-only review.
+${branchName ? `You are reviewing code in a git worktree at \`${worktreePath}\` on branch \`${branchName}\`.` : `You are reviewing local changes in the repository at \`${worktreePath}\`.`}`;
+}

package/dist/review-worker.d.ts

@@ -0,0 +1,46 @@
+import type { HarmonyApiClient } from "@gethmy/mcp/src/api-client.js";
+import type { Card, Column, Label, Subtask } from "@harmony/shared";
+import { type AgentConfig, type WorkerState } from "./types.js";
+export declare class ReviewWorker {
+    private config;
+    private client;
+    private onDone;
+    id: number;
+    state: WorkerState;
+    cardId: string | null;
+    branchName: string | null;
+    worktreePath: string | null;
+    startedAt: number | null;
+    private process;
+    private devServerProcess;
+    private timeoutTimer;
+    private progressTracker;
+    private aborted;
+    constructor(id: number, config: AgentConfig, client: HarmonyApiClient, _userEmail: string, onDone: (worker: ReviewWorker) => void);
+    get tag(): string;
+    get isIdle(): boolean;
+    private get reviewPort();
+    get isActive(): boolean;
+    /**
+     * Start reviewing a card. Runs the full lifecycle:
+     * PREPARING → REVIEWING → COMPLETING → IDLE
+     */
+    run(card: Card, column: Column, labels: Label[], subtasks: Subtask[]): Promise<void>;
+    /**
+     * Pause the current review by suspending the Claude process (SIGSTOP).
+     */
+    pause(): Promise<void>;
+    /**
+     * Resume the Claude process after a pause (SIGCONT).
+     */
+    resume(): Promise<void>;
+    /**
+     * Cancel the current review. Sends escalating signals to both processes.
+     */
+    cancel(): Promise<void>;
+    private checkDevServer;
+    private spawnClaude;
+    private waitForExit;
+    private killDevServer;
+    private cleanup;
+}