@gethmy/agent 1.7.1 → 1.7.3

package/dist/review-prompt.js

@@ -1,103 +0,0 @@
-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

@@ -1,56 +0,0 @@
-import type { HarmonyApiClient } from "@gethmy/mcp/src/api-client.js";
-import type { Card, Column, Label, Subtask } from "@harmony/shared";
-import { type StateStore } from "./state-store.js";
-import { type AgentConfig, type WorkerState } from "./types.js";
-export declare class ReviewWorker {
-    private config;
-    private client;
-    private onDone;
-    private stateStore;
-    private workspaceId?;
-    id: number;
-    state: WorkerState;
-    cardId: string | null;
-    branchName: string | null;
-    worktreePath: string | null;
-    startedAt: number | null;
-    private process;
-    private devServerProcess;
-    private timeoutTimer;
-    private heartbeatTimer;
-    private progressTracker;
-    private lastSessionStats;
-    private aborted;
-    private runId;
-    private lastRunLogPath;
-    private sessionId;
-    constructor(id: number, config: AgentConfig, client: HarmonyApiClient, _userEmail: string, onDone: (worker: ReviewWorker) => void, stateStore: StateStore, workspaceId?: string | undefined, _projectId?: string);
-    private startHeartbeat;
-    private stopHeartbeat;
-    private recordPhase;
-    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 spawnClaude;
-    private killDevServer;
-    private resolveLocalChanges;
-    private cleanup;
-}