pi-harness-runtime 1.1.407 → 1.1.410

package/packages/two-phase-review/dist/classify.js

@@ -0,0 +1,108 @@
+/**
+ * Todo classification for the two-phase workflow.
+ *
+ * Implementation work and queued review work must never be confused. The
+ * workflow therefore classifies every todo item before it decides whether
+ * the implementation phase is over.
+ */
+/** Prefix of a todo item that is implementation work. */
+export const IMPLEMENT_PREFIX = "[IMPLEMENT]";
+/** Prefix of a todo item that is the queued review. */
+export const REVIEW_PREFIX = "[REVIEW]";
+/** Prefix of a todo item that fixes a confirmed review finding. */
+export const REVIEW_FIX_PREFIX = "[REVIEW-FIX]";
+const KIND_BY_NAME = {
+    implement: "implement",
+    implementation: "implement",
+    impl: "implement",
+    review: "review",
+    review_pending: "review",
+    "review-pending": "review",
+    review_fix: "review_fix",
+    "review-fix": "review_fix",
+    reviewfix: "review_fix",
+    other: "other",
+};
+/**
+ * Classify a todo item.
+ *
+ * The metadata wins over the title: an explicit kind is more reliable than a
+ * prefix. A `[REVIEW-FIX]` title is a fix item, not a review item, so the fix
+ * prefix is checked first.
+ */
+export function classifyTodo(todo) {
+    if (!todo)
+        return "other";
+    const metadata = todo.metadata ?? {};
+    const declared = metadata.kind ?? metadata.todoKind;
+    if (typeof declared === "string") {
+        const key = declared.toLowerCase().trim().replace(/[\s-]+/g, "_");
+        const mapped = KIND_BY_NAME[key];
+        if (mapped)
+            return mapped;
+    }
+    const subject = (todo.subject ?? "").trim().toUpperCase();
+    if (subject.startsWith(REVIEW_FIX_PREFIX))
+        return "review_fix";
+    if (subject.startsWith(IMPLEMENT_PREFIX))
+        return "implement";
+    if (subject.startsWith(REVIEW_PREFIX))
+        return "review";
+    return "other";
+}
+/** Is the todo item still open? */
+export function isOpenTodo(todo) {
+    return todo.status === "pending" || todo.status === "in_progress";
+}
+/** Every open todo item. */
+export function openTodos(todos) {
+    return todos.filter((todo) => isOpenTodo(todo));
+}
+/**
+ * The open todo items that are still implementation work.
+ *
+ * A queued review item is not implementation work: it must never block the
+ * implementation. An unclassified item counts as implementation work, because
+ * a task that is tracked without a prefix is still work that has to finish.
+ */
+export function implementationTodos(todos) {
+    return openTodos(todos).filter((todo) => {
+        const kind = classifyTodo(todo);
+        return kind === "implement" || kind === "other";
+    });
+}
+/** The open todo items that represent the queued review. */
+export function reviewTodos(todos) {
+    return openTodos(todos).filter((todo) => classifyTodo(todo) === "review");
+}
+/** The open todo items that fix a confirmed review finding. */
+export function reviewFixTodos(todos) {
+    return openTodos(todos).filter((todo) => classifyTodo(todo) === "review_fix");
+}
+/** Does the task still have implementation work? */
+export function hasOpenImplementationWork(todos) {
+    return implementationTodos(todos).length > 0;
+}
+/** How many todo items of each kind are open. */
+export function summarizeTodos(todos) {
+    return {
+        implementation: implementationTodos(todos).length,
+        review: reviewTodos(todos).length,
+        reviewFix: reviewFixTodos(todos).length,
+    };
+}
+const PREFIX_BY_KIND = {
+    implement: IMPLEMENT_PREFIX,
+    review: REVIEW_PREFIX,
+    review_fix: REVIEW_FIX_PREFIX,
+    other: "",
+};
+/** Prefix a title with the kind of its todo item. */
+export function buildTodoSubject(kind, text) {
+    const prefix = PREFIX_BY_KIND[kind];
+    const clean = text.trim();
+    if (!prefix)
+        return clean;
+    return clean.toUpperCase().startsWith(prefix) ? clean : `${prefix} ${clean}`;
+}
+//# sourceMappingURL=classify.js.map

package/packages/two-phase-review/dist/classify.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"classify.js","sourceRoot":"","sources":["../src/classify.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAIH,yDAAyD;AACzD,MAAM,CAAC,MAAM,gBAAgB,GAAG,aAAa,CAAC;AAC9C,uDAAuD;AACvD,MAAM,CAAC,MAAM,aAAa,GAAG,UAAU,CAAC;AACxC,mEAAmE;AACnE,MAAM,CAAC,MAAM,iBAAiB,GAAG,cAAc,CAAC;AAUhD,MAAM,YAAY,GAA6B;IAC9C,SAAS,EAAE,WAAW;IACtB,cAAc,EAAE,WAAW;IAC3B,IAAI,EAAE,WAAW;IACjB,MAAM,EAAE,QAAQ;IAChB,cAAc,EAAE,QAAQ;IACxB,gBAAgB,EAAE,QAAQ;IAC1B,UAAU,EAAE,YAAY;IACxB,YAAY,EAAE,YAAY;IAC1B,SAAS,EAAE,YAAY;IACvB,KAAK,EAAE,OAAO;CACd,CAAC;AAEF;;;;;;GAMG;AACH,MAAM,UAAU,YAAY,CAAC,IAAiC;IAC7D,IAAI,CAAC,IAAI;QAAE,OAAO,OAAO,CAAC;IAE1B,MAAM,QAAQ,GAAG,IAAI,CAAC,QAAQ,IAAI,EAAE,CAAC;IACrC,MAAM,QAAQ,GAAG,QAAQ,CAAC,IAAI,IAAI,QAAQ,CAAC,QAAQ,CAAC;IACpD,IAAI,OAAO,QAAQ,KAAK,QAAQ,EAAE,CAAC;QAClC,MAAM,GAAG,GAAG,QAAQ,CAAC,WAAW,EAAE,CAAC,IAAI,EAAE,CAAC,OAAO,CAAC,SAAS,EAAE,GAAG,CAAC,CAAC;QAClE,MAAM,MAAM,GAAG,YAAY,CAAC,GAAG,CAAC,CAAC;QACjC,IAAI,MAAM;YAAE,OAAO,MAAM,CAAC;IAC3B,CAAC;IAED,MAAM,OAAO,GAAG,CAAC,IAAI,CAAC,OAAO,IAAI,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC,WAAW,EAAE,CAAC;IAC1D,IAAI,OAAO,CAAC,UAAU,CAAC,iBAAiB,CAAC;QAAE,OAAO,YAAY,CAAC;IAC/D,IAAI,OAAO,CAAC,UAAU,CAAC,gBAAgB,CAAC;QAAE,OAAO,WAAW,CAAC;IAC7D,IAAI,OAAO,CAAC,UAAU,CAAC,aAAa,CAAC;QAAE,OAAO,QAAQ,CAAC;IACvD,OAAO,OAAO,CAAC;AAChB,CAAC;AAED,mCAAmC;AACnC,MAAM,UAAU,UAAU,CAAC,IAAc;IACxC,OAAO,IAAI,CAAC,MAAM,KAAK,SAAS,IAAI,IAAI,CAAC,MAAM,KAAK,aAAa,CAAC;AACnE,CAAC;AAED,4BAA4B;AAC5B,MAAM,UAAU,SAAS,CAAC,KAAiB;IAC1C,OAAO,KAAK,CAAC,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,UAAU,CAAC,IAAI,CAAC,CAAC,CAAC;AACjD,CAAC;AAED;;;;;;GAMG;AACH,MAAM,UAAU,mBAAmB,CAAC,KAAiB;IACpD,OAAO,SAAS,CAAC,KAAK,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE;QACvC,MAAM,IAAI,GAAG,YAAY,CAAC,IAAI,CAAC,CAAC;QAChC,OAAO,IAAI,KAAK,WAAW,IAAI,IAAI,KAAK,OAAO,CAAC;IACjD,CAAC,CAAC,CAAC;AACJ,CAAC;AAED,4DAA4D;AAC5D,MAAM,UAAU,WAAW,CAAC,KAAiB;IAC5C,OAAO,SAAS,CAAC,KAAK,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,YAAY,CAAC,IAAI,CAAC,KAAK,QAAQ,CAAC,CAAC;AAC3E,CAAC;AAED,+DAA+D;AAC/D,MAAM,UAAU,cAAc,CAAC,KAAiB;IAC/C,OAAO,SAAS,CAAC,KAAK,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,YAAY,CAAC,IAAI,CAAC,KAAK,YAAY,CAAC,CAAC;AAC/E,CAAC;AAED,oDAAoD;AACpD,MAAM,UAAU,yBAAyB,CAAC,KAAiB;IAC1D,OAAO,mBAAmB,CAAC,KAAK,CAAC,CAAC,MAAM,GAAG,CAAC,CAAC;AAC9C,CAAC;AAED,iDAAiD;AACjD,MAAM,UAAU,cAAc,CAAC,KAAiB;IAK/C,OAAO;QACN,cAAc,EAAE,mBAAmB,CAAC,KAAK,CAAC,CAAC,MAAM;QACjD,MAAM,EAAE,WAAW,CAAC,KAAK,CAAC,CAAC,MAAM;QACjC,SAAS,EAAE,cAAc,CAAC,KAAK,CAAC,CAAC,MAAM;KACvC,CAAC;AACH,CAAC;AAED,MAAM,cAAc,GAA6B;IAChD,SAAS,EAAE,gBAAgB;IAC3B,MAAM,EAAE,aAAa;IACrB,UAAU,EAAE,iBAAiB;IAC7B,KAAK,EAAE,EAAE;CACT,CAAC;AAEF,qDAAqD;AACrD,MAAM,UAAU,gBAAgB,CAAC,IAAc,EAAE,IAAY;IAC5D,MAAM,MAAM,GAAG,cAAc,CAAC,IAAI,CAAC,CAAC;IACpC,MAAM,KAAK,GAAG,IAAI,CAAC,IAAI,EAAE,CAAC;IAC1B,IAAI,CAAC,MAAM;QAAE,OAAO,KAAK,CAAC;IAC1B,OAAO,KAAK,CAAC,WAAW,EAAE,CAAC,UAAU,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,GAAG,MAAM,IAAI,KAAK,EAAE,CAAC;AAC9E,CAAC"}

package/packages/two-phase-review/dist/index.d.ts

@@ -0,0 +1,20 @@
+/**
+ * Two-phase implementation and review workflow.
+ *
+ * Phase 1 finishes the implementation todos of a request. Phase 2 reviews the
+ * diff of that request. A review signal that arrives during Phase 1 is
+ * recorded and deferred, never executed between two implementation todos.
+ */
+export type { TwoPhasePhase, TodoKind, ReviewSignal, ReviewChunk, ReviewFinding, TwoPhaseState, } from "./types.js";
+export { IMPLEMENT_PREFIX, REVIEW_PREFIX, REVIEW_FIX_PREFIX, classifyTodo, isOpenTodo, openTodos, implementationTodos, reviewTodos, reviewFixTodos, hasOpenImplementationWork, summarizeTodos, buildTodoSubject, } from "./classify.js";
+export type { TodoLike } from "./classify.js";
+export { DEFAULT_REVIEW_TOOL_PATTERNS, DEFAULT_REVIEW_COMMAND_PATTERNS, parseExtraPatterns, isComprehensiveReviewTool, isReviewCommand, dispatchAgentName, detectReviewInvocation, explicitReviewRequest, extractFiles, signalKey, mergeSignals, summarizeSignals, } from "./signals.js";
+export type { ReviewInvocation, SignalSummary } from "./signals.js";
+export { EXCLUDED_SCOPE_PREFIXES, DEFAULT_CHUNK_SIZE, isReviewablePath, isGitRepository, currentRevision, revisionExists, moduleKey, countModules, parseNumstat, collectScope, chunkScope, shouldChunkScope, } from "./scope.js";
+export type { ScopeMetrics, ScopeResult } from "./scope.js";
+export { TWO_PHASE_STATE_DIR, createState, safeFileName, normalizeState, TwoPhaseStateStore, moveToPhase, recordSignal, transitionToReviewPending, startReview, markChunkDone, recordFinding, transitionToReviewFixes, allChunksDone, markComplete, hasQueuedReview, } from "./state.js";
+export { TWO_PHASE_CONTRACT, buildPhaseView, buildPhaseInstruction, buildReviewTodoSubject, buildReviewTodoDescription, buildChunkTodoSubject, buildChunkTodoDescription, isChunkTodo, syncChunkStatus, formatPhaseStatus, buildDeferralReason, } from "./rules.js";
+export type { PhaseView } from "./rules.js";
+export { HARNESS_MESSAGE_MARKER, initTwoPhaseReview, } from "./runtime.js";
+export type { TwoPhaseExtensionApi, TwoPhaseExtensionContext, TwoPhaseRuntime, TwoPhaseRuntimeOptions, TodoReadResult, TodoReader, } from "./runtime.js";
+//# sourceMappingURL=index.d.ts.map

package/packages/two-phase-review/dist/index.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,YAAY,EACX,aAAa,EACb,QAAQ,EACR,YAAY,EACZ,WAAW,EACX,aAAa,EACb,aAAa,GACb,MAAM,YAAY,CAAC;AAEpB,OAAO,EACN,gBAAgB,EAChB,aAAa,EACb,iBAAiB,EACjB,YAAY,EACZ,UAAU,EACV,SAAS,EACT,mBAAmB,EACnB,WAAW,EACX,cAAc,EACd,yBAAyB,EACzB,cAAc,EACd,gBAAgB,GAChB,MAAM,eAAe,CAAC;AACvB,YAAY,EAAE,QAAQ,EAAE,MAAM,eAAe,CAAC;AAE9C,OAAO,EACN,4BAA4B,EAC5B,+BAA+B,EAC/B,kBAAkB,EAClB,yBAAyB,EACzB,eAAe,EACf,iBAAiB,EACjB,sBAAsB,EACtB,qBAAqB,EACrB,YAAY,EACZ,SAAS,EACT,YAAY,EACZ,gBAAgB,GAChB,MAAM,cAAc,CAAC;AACtB,YAAY,EAAE,gBAAgB,EAAE,aAAa,EAAE,MAAM,cAAc,CAAC;AAEpE,OAAO,EACN,uBAAuB,EACvB,kBAAkB,EAClB,gBAAgB,EAChB,eAAe,EACf,eAAe,EACf,cAAc,EACd,SAAS,EACT,YAAY,EACZ,YAAY,EACZ,YAAY,EACZ,UAAU,EACV,gBAAgB,GAChB,MAAM,YAAY,CAAC;AACpB,YAAY,EAAE,YAAY,EAAE,WAAW,EAAE,MAAM,YAAY,CAAC;AAE5D,OAAO,EACN,mBAAmB,EACnB,WAAW,EACX,YAAY,EACZ,cAAc,EACd,kBAAkB,EAClB,WAAW,EACX,YAAY,EACZ,yBAAyB,EACzB,WAAW,EACX,aAAa,EACb,aAAa,EACb,uBAAuB,EACvB,aAAa,EACb,YAAY,EACZ,eAAe,GACf,MAAM,YAAY,CAAC;AAEpB,OAAO,EACN,kBAAkB,EAClB,cAAc,EACd,qBAAqB,EACrB,sBAAsB,EACtB,0BAA0B,EAC1B,qBAAqB,EACrB,yBAAyB,EACzB,WAAW,EACX,eAAe,EACf,iBAAiB,EACjB,mBAAmB,GACnB,MAAM,YAAY,CAAC;AACpB,YAAY,EAAE,SAAS,EAAE,MAAM,YAAY,CAAC;AAE5C,OAAO,EACN,sBAAsB,EACtB,kBAAkB,GAClB,MAAM,cAAc,CAAC;AACtB,YAAY,EACX,oBAAoB,EACpB,wBAAwB,EACxB,eAAe,EACf,sBAAsB,EACtB,cAAc,EACd,UAAU,GACV,MAAM,cAAc,CAAC"}

package/packages/two-phase-review/dist/index.js

@@ -0,0 +1,14 @@
+/**
+ * Two-phase implementation and review workflow.
+ *
+ * Phase 1 finishes the implementation todos of a request. Phase 2 reviews the
+ * diff of that request. A review signal that arrives during Phase 1 is
+ * recorded and deferred, never executed between two implementation todos.
+ */
+export { IMPLEMENT_PREFIX, REVIEW_PREFIX, REVIEW_FIX_PREFIX, classifyTodo, isOpenTodo, openTodos, implementationTodos, reviewTodos, reviewFixTodos, hasOpenImplementationWork, summarizeTodos, buildTodoSubject, } from "./classify.js";
+export { DEFAULT_REVIEW_TOOL_PATTERNS, DEFAULT_REVIEW_COMMAND_PATTERNS, parseExtraPatterns, isComprehensiveReviewTool, isReviewCommand, dispatchAgentName, detectReviewInvocation, explicitReviewRequest, extractFiles, signalKey, mergeSignals, summarizeSignals, } from "./signals.js";
+export { EXCLUDED_SCOPE_PREFIXES, DEFAULT_CHUNK_SIZE, isReviewablePath, isGitRepository, currentRevision, revisionExists, moduleKey, countModules, parseNumstat, collectScope, chunkScope, shouldChunkScope, } from "./scope.js";
+export { TWO_PHASE_STATE_DIR, createState, safeFileName, normalizeState, TwoPhaseStateStore, moveToPhase, recordSignal, transitionToReviewPending, startReview, markChunkDone, recordFinding, transitionToReviewFixes, allChunksDone, markComplete, hasQueuedReview, } from "./state.js";
+export { TWO_PHASE_CONTRACT, buildPhaseView, buildPhaseInstruction, buildReviewTodoSubject, buildReviewTodoDescription, buildChunkTodoSubject, buildChunkTodoDescription, isChunkTodo, syncChunkStatus, formatPhaseStatus, buildDeferralReason, } from "./rules.js";
+export { HARNESS_MESSAGE_MARKER, initTwoPhaseReview, } from "./runtime.js";
+//# sourceMappingURL=index.js.map

package/packages/two-phase-review/dist/index.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAWH,OAAO,EACN,gBAAgB,EAChB,aAAa,EACb,iBAAiB,EACjB,YAAY,EACZ,UAAU,EACV,SAAS,EACT,mBAAmB,EACnB,WAAW,EACX,cAAc,EACd,yBAAyB,EACzB,cAAc,EACd,gBAAgB,GAChB,MAAM,eAAe,CAAC;AAGvB,OAAO,EACN,4BAA4B,EAC5B,+BAA+B,EAC/B,kBAAkB,EAClB,yBAAyB,EACzB,eAAe,EACf,iBAAiB,EACjB,sBAAsB,EACtB,qBAAqB,EACrB,YAAY,EACZ,SAAS,EACT,YAAY,EACZ,gBAAgB,GAChB,MAAM,cAAc,CAAC;AAGtB,OAAO,EACN,uBAAuB,EACvB,kBAAkB,EAClB,gBAAgB,EAChB,eAAe,EACf,eAAe,EACf,cAAc,EACd,SAAS,EACT,YAAY,EACZ,YAAY,EACZ,YAAY,EACZ,UAAU,EACV,gBAAgB,GAChB,MAAM,YAAY,CAAC;AAGpB,OAAO,EACN,mBAAmB,EACnB,WAAW,EACX,YAAY,EACZ,cAAc,EACd,kBAAkB,EAClB,WAAW,EACX,YAAY,EACZ,yBAAyB,EACzB,WAAW,EACX,aAAa,EACb,aAAa,EACb,uBAAuB,EACvB,aAAa,EACb,YAAY,EACZ,eAAe,GACf,MAAM,YAAY,CAAC;AAEpB,OAAO,EACN,kBAAkB,EAClB,cAAc,EACd,qBAAqB,EACrB,sBAAsB,EACtB,0BAA0B,EAC1B,qBAAqB,EACrB,yBAAyB,EACzB,WAAW,EACX,eAAe,EACf,iBAAiB,EACjB,mBAAmB,GACnB,MAAM,YAAY,CAAC;AAGpB,OAAO,EACN,sBAAsB,EACtB,kBAAkB,GAClB,MAAM,cAAc,CAAC"}

package/packages/two-phase-review/dist/rules.d.ts

@@ -0,0 +1,81 @@
+/**
+ * The workflow rules.
+ *
+ * The rules are injected into the system prompt: they tell the agent how the
+ * two phases are separated and what it has to do in the current phase.
+ */
+import type { ReviewChunk, ReviewSignal, TwoPhasePhase, TwoPhaseState } from "./types.js";
+import { type TodoLike } from "./classify.js";
+/** The contract of the workflow. */
+export declare const TWO_PHASE_CONTRACT = "## Two-phase implementation and review\n\nA task with several steps runs in two phases.\n\n### Phase 1 - implementation\n- Work through every open implementation todo of the current request.\n- Do not start a comprehensive review between implementation todos. A review that is requested while the implementation runs is queued, not executed.\n- Keep the task-level checks active: syntax errors, broken imports, type errors of the code you write right now and failing required tests are part of the implementation and are fixed now.\n- Broader feedback (style, maintainability, a full diff review) is queued as one [REVIEW] todo. Never discard it.\n- Queued review work is not an implementation blocker. The request is not complete while implementation todos remain.\n\n### Phase 2 - review\n- The review covers the diff of the task, not the whole repository.\n- Review the scope in chunks, per module. Use independent reviewers for independent chunks.\n- Record every confirmed finding as a [REVIEW-FIX] todo, grouped by root cause and severity. A minor suggestion does not interrupt a running chunk.\n- Fix the confirmed findings after the review pass, check the changed area, and run one final integration review over the task diff.\n- The request is complete only when the implementation todos, the review chunks and the required fix todos are done or explicitly accepted.\n\n### Todo prefixes\n- [IMPLEMENT] implementation work of the current request\n- [REVIEW] the queued review of the task diff\n- [REVIEW-FIX] a confirmed finding that has to be fixed\n\nA user pause, an approval requirement, a security issue or any other safety rule stops the work immediately. The phase order never overrides the intent of the user.";
+/** The current state of the workflow as the agent sees it. */
+export interface PhaseView {
+    phase: TwoPhasePhase;
+    /** How many implementation todos are still open. */
+    implementationOpen: number;
+    /** How many review todos are open. */
+    reviewOpen: number;
+    /** How many review-fix todos are open. */
+    reviewFixOpen: number;
+    /** How many files the review covers. */
+    scopeFiles: number;
+    /** The review chunks and their state. */
+    chunks: {
+        id: string;
+        label: string;
+        status: string;
+    }[];
+    /** Is a review queued? */
+    queued: boolean;
+    /** The signals that queued the review. */
+    signals: ReviewSignal[];
+}
+/** Build the phase view from the state and the todo list. */
+export declare function buildPhaseView(state: TwoPhaseState, todos: TodoLike[]): PhaseView;
+/** The subject of the one queued review todo. */
+export declare function buildReviewTodoSubject(state: TwoPhaseState): string;
+/** The description of the one queued review todo. */
+export declare function buildReviewTodoDescription(state: TwoPhaseState): string;
+/**
+ * The status line of the workflow.
+ *
+ * The line is plain ASCII: it is shown in the footer of the session.
+ */
+export declare function formatPhaseStatus(state: TwoPhaseState): string;
+/**
+ * The subject of the todo that covers one review chunk.
+ *
+ * The chunk id is part of the subject, so the runtime can tell which chunk a
+ * todo belongs to.
+ */
+export declare function buildChunkTodoSubject(chunk: ReviewChunk): string;
+/** The description of the todo that covers one review chunk. */
+export declare function buildChunkTodoDescription(chunk: ReviewChunk): string;
+/** Does the todo item belong to the review chunk? */
+export declare function isChunkTodo(todo: TodoLike, chunk: ReviewChunk): boolean;
+/**
+ * Derive the status of the review chunks from the todo list.
+ *
+ * The status only moves forward: a chunk that is done stays done, so a todo
+ * list that is briefly empty (a restart, a compaction) can never erase the
+ * progress of the review.
+ */
+export declare function syncChunkStatus(state: TwoPhaseState, todos: TodoLike[]): TwoPhaseState;
+/**
+ * The reason a comprehensive review was deferred.
+ *
+ * The reason has to be actionable: the agent continues with the
+ * implementation and keeps the task-level checks alive.
+ */
+export declare function buildDeferralReason(state: TwoPhaseState, context: {
+    source: string;
+    implementationOpen: number;
+}): string;
+/**
+ * The instruction for the current phase.
+ *
+ * The instruction is short: it names the phase, the open work and what the
+ * agent has to do next.
+ */
+export declare function buildPhaseInstruction(state: TwoPhaseState, view: PhaseView): string;
+//# sourceMappingURL=rules.d.ts.map

package/packages/two-phase-review/dist/rules.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"rules.d.ts","sourceRoot":"","sources":["../src/rules.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,EAAE,WAAW,EAAE,YAAY,EAAE,aAAa,EAAE,aAAa,EAAE,MAAM,YAAY,CAAC;AAC1F,OAAO,EAKN,KAAK,QAAQ,EACb,MAAM,eAAe,CAAC;AAGvB,oCAAoC;AACpC,eAAO,MAAM,kBAAkB,msDAuBsI,CAAC;AAEtK,8DAA8D;AAC9D,MAAM,WAAW,SAAS;IACzB,KAAK,EAAE,aAAa,CAAC;IACrB,oDAAoD;IACpD,kBAAkB,EAAE,MAAM,CAAC;IAC3B,sCAAsC;IACtC,UAAU,EAAE,MAAM,CAAC;IACnB,0CAA0C;IAC1C,aAAa,EAAE,MAAM,CAAC;IACtB,wCAAwC;IACxC,UAAU,EAAE,MAAM,CAAC;IACnB,yCAAyC;IACzC,MAAM,EAAE;QAAE,EAAE,EAAE,MAAM,CAAC;QAAC,KAAK,EAAE,MAAM,CAAC;QAAC,MAAM,EAAE,MAAM,CAAA;KAAE,EAAE,CAAC;IACxD,0BAA0B;IAC1B,MAAM,EAAE,OAAO,CAAC;IAChB,0CAA0C;IAC1C,OAAO,EAAE,YAAY,EAAE,CAAC;CACxB;AAED,6DAA6D;AAC7D,wBAAgB,cAAc,CAAC,KAAK,EAAE,aAAa,EAAE,KAAK,EAAE,QAAQ,EAAE,GAAG,SAAS,CAgBjF;AAED,iDAAiD;AACjD,wBAAgB,sBAAsB,CAAC,KAAK,EAAE,aAAa,GAAG,MAAM,CAKnE;AAED,qDAAqD;AACrD,wBAAgB,0BAA0B,CAAC,KAAK,EAAE,aAAa,GAAG,MAAM,CAYvE;AAED;;;;GAIG;AACH,wBAAgB,iBAAiB,CAAC,KAAK,EAAE,aAAa,GAAG,MAAM,CAiB9D;AAED;;;;;GAKG;AACH,wBAAgB,qBAAqB,CAAC,KAAK,EAAE,WAAW,GAAG,MAAM,CAEhE;AAED,gEAAgE;AAChE,wBAAgB,yBAAyB,CAAC,KAAK,EAAE,WAAW,GAAG,MAAM,CAQpE;AAED,qDAAqD;AACrD,wBAAgB,WAAW,CAAC,IAAI,EAAE,QAAQ,EAAE,KAAK,EAAE,WAAW,GAAG,OAAO,CAQvE;AAED;;;;;;GAMG;AACH,wBAAgB,eAAe,CAAC,KAAK,EAAE,aAAa,EAAE,KAAK,EAAE,QAAQ,EAAE,GAAG,aAAa,CAatF;AAED;;;;;GAKG;AACH,wBAAgB,mBAAmB,CAClC,KAAK,EAAE,aAAa,EACpB,OAAO,EAAE;IAAE,MAAM,EAAE,MAAM,CAAC;IAAC,kBAAkB,EAAE,MAAM,CAAA;CAAE,GACrD,MAAM,CAOR;AAED;;;;;GAKG;AACH,wBAAgB,qBAAqB,CAAC,KAAK,EAAE,aAAa,EAAE,IAAI,EAAE,SAAS,GAAG,MAAM,CA6BnF"}

package/packages/two-phase-review/dist/rules.js

@@ -0,0 +1,195 @@
+/**
+ * The workflow rules.
+ *
+ * The rules are injected into the system prompt: they tell the agent how the
+ * two phases are separated and what it has to do in the current phase.
+ */
+import { REVIEW_FIX_PREFIX, REVIEW_PREFIX, classifyTodo, summarizeTodos, } from "./classify.js";
+import { summarizeSignals } from "./signals.js";
+/** The contract of the workflow. */
+export const TWO_PHASE_CONTRACT = `## Two-phase implementation and review
+
+A task with several steps runs in two phases.
+
+### Phase 1 - implementation
+- Work through every open implementation todo of the current request.
+- Do not start a comprehensive review between implementation todos. A review that is requested while the implementation runs is queued, not executed.
+- Keep the task-level checks active: syntax errors, broken imports, type errors of the code you write right now and failing required tests are part of the implementation and are fixed now.
+- Broader feedback (style, maintainability, a full diff review) is queued as one [REVIEW] todo. Never discard it.
+- Queued review work is not an implementation blocker. The request is not complete while implementation todos remain.
+
+### Phase 2 - review
+- The review covers the diff of the task, not the whole repository.
+- Review the scope in chunks, per module. Use independent reviewers for independent chunks.
+- Record every confirmed finding as a [REVIEW-FIX] todo, grouped by root cause and severity. A minor suggestion does not interrupt a running chunk.
+- Fix the confirmed findings after the review pass, check the changed area, and run one final integration review over the task diff.
+- The request is complete only when the implementation todos, the review chunks and the required fix todos are done or explicitly accepted.
+
+### Todo prefixes
+- [IMPLEMENT] implementation work of the current request
+- ${REVIEW_PREFIX} the queued review of the task diff
+- ${REVIEW_FIX_PREFIX} a confirmed finding that has to be fixed
+
+A user pause, an approval requirement, a security issue or any other safety rule stops the work immediately. The phase order never overrides the intent of the user.`;
+/** Build the phase view from the state and the todo list. */
+export function buildPhaseView(state, todos) {
+    const summary = summarizeTodos(todos);
+    return {
+        phase: state.phase,
+        implementationOpen: summary.implementation,
+        reviewOpen: summary.review,
+        reviewFixOpen: summary.reviewFix,
+        scopeFiles: state.scopeFiles.length,
+        chunks: state.chunks.map((chunk) => ({
+            id: chunk.id,
+            label: chunk.label,
+            status: chunk.status,
+        })),
+        queued: state.signals.length > 0,
+        signals: state.signals,
+    };
+}
+/** The subject of the one queued review todo. */
+export function buildReviewTodoSubject(state) {
+    const summary = summarizeSignals(state.signals);
+    const first = summary.sources[0] ?? "review signal";
+    const more = summary.sources.length > 1 ? ` +${summary.sources.length - 1} more` : "";
+    return `${REVIEW_PREFIX} Review the task diff (queued by ${first}${more})`;
+}
+/** The description of the one queued review todo. */
+export function buildReviewTodoDescription(state) {
+    const summary = summarizeSignals(state.signals);
+    const lines = [];
+    if (summary.sources.length > 0)
+        lines.push(`Source: ${summary.sources.join(", ")}`);
+    if (summary.reason)
+        lines.push(`Reason: ${summary.reason}`);
+    if (state.scopeFiles.length > 0) {
+        const shown = state.scopeFiles.slice(0, 20);
+        const rest = state.scopeFiles.length - shown.length;
+        lines.push(`Scope: ${shown.join(", ")}${rest > 0 ? ` (+${rest} more)` : ""}`);
+    }
+    lines.push("The review starts after the implementation todos are done.");
+    return lines.join("\n");
+}
+/**
+ * The status line of the workflow.
+ *
+ * The line is plain ASCII: it is shown in the footer of the session.
+ */
+export function formatPhaseStatus(state) {
+    switch (state.phase) {
+        case "implementation":
+            return "[PHASE] implementation";
+        case "review_pending":
+            return "[PHASE] review_pending";
+        case "review": {
+            const done = state.chunks.filter((chunk) => chunk.status === "done").length;
+            return `[PHASE] review (${done}/${state.chunks.length} chunk(s), ${state.scopeFiles.length} file(s))`;
+        }
+        case "review_fixes":
+            return `[PHASE] review_fixes (${state.findings.length} finding(s))`;
+        case "complete":
+            return "[PHASE] complete";
+        default:
+            return "[PHASE] implementation";
+    }
+}
+/**
+ * The subject of the todo that covers one review chunk.
+ *
+ * The chunk id is part of the subject, so the runtime can tell which chunk a
+ * todo belongs to.
+ */
+export function buildChunkTodoSubject(chunk) {
+    return `${REVIEW_PREFIX} ${chunk.id}: ${chunk.label}`;
+}
+/** The description of the todo that covers one review chunk. */
+export function buildChunkTodoDescription(chunk) {
+    const files = chunk.files.map((file) => `- ${file}`).join("\n");
+    return [
+        `Review the files of ${chunk.id} (${chunk.label}):`,
+        files,
+        "",
+        `Review only this chunk. Record confirmed findings as ${REVIEW_FIX_PREFIX} todos grouped by root cause and severity.`,
+    ].join("\n");
+}
+/** Does the todo item belong to the review chunk? */
+export function isChunkTodo(todo, chunk) {
+    if (classifyTodo(todo) !== "review")
+        return false;
+    const subject = todo.subject ?? "";
+    const at = subject.indexOf(chunk.id);
+    if (at < 0)
+        return false;
+    // "chunk-1" must not match "chunk-10".
+    const after = subject.slice(at + chunk.id.length, at + chunk.id.length + 1);
+    return after === "" || !/[0-9a-z]/i.test(after);
+}
+/**
+ * Derive the status of the review chunks from the todo list.
+ *
+ * The status only moves forward: a chunk that is done stays done, so a todo
+ * list that is briefly empty (a restart, a compaction) can never erase the
+ * progress of the review.
+ */
+export function syncChunkStatus(state, todos) {
+    if (state.chunks.length === 0)
+        return state;
+    let changed = false;
+    const chunks = state.chunks.map((chunk) => {
+        if (chunk.status === "done")
+            return chunk;
+        const done = todos.some((todo) => isChunkTodo(todo, chunk) && todo.status === "completed");
+        if (!done)
+            return chunk;
+        changed = true;
+        return { ...chunk, status: "done" };
+    });
+    return changed ? { ...state, chunks } : state;
+}
+/**
+ * The reason a comprehensive review was deferred.
+ *
+ * The reason has to be actionable: the agent continues with the
+ * implementation and keeps the task-level checks alive.
+ */
+export function buildDeferralReason(state, context) {
+    const sources = [...new Set(state.signals.map((signal) => signal.source))];
+    return [
+        `Phase 1 (implementation) is still running: ${context.implementationOpen} implementation todo(s) are open.`,
+        `The comprehensive review (${sources.join(", ") || context.source}) is queued and starts after the implementation todos are done.`,
+        "Continue with the implementation. Keep task-level checks (syntax errors, broken imports, type errors of the code you write now, failing required tests) active and fix them now.",
+    ].join(" ");
+}
+/**
+ * The instruction for the current phase.
+ *
+ * The instruction is short: it names the phase, the open work and what the
+ * agent has to do next.
+ */
+export function buildPhaseInstruction(state, view) {
+    if (state.phase === "implementation") {
+        if (!view.queued) {
+            return `[two-phase] Phase 1 (implementation): ${view.implementationOpen} implementation todo(s) open. Finish the implementation first; keep task-level checks (type errors, failing tests, build) active. Do not start a comprehensive review.`;
+        }
+        const summary = summarizeSignals(state.signals);
+        return `[two-phase] Phase 1 (implementation): ${view.implementationOpen} implementation todo(s) open. A comprehensive review is queued (${summary.sources.join(", ") || "review signal"}) and must not start yet. Fix correctness problems of the current code now; queue broader feedback as one ${REVIEW_PREFIX} todo. The request is not complete while implementation todos remain.`;
+    }
+    if (state.phase === "review_pending") {
+        return `[two-phase] Phase 2 (review): the implementation is done, the review scope is being derived from the task diff (${view.scopeFiles} file(s)).`;
+    }
+    if (state.phase === "review") {
+        const chunkLines = view.chunks.map((chunk) => `- ${chunk.id}: ${chunk.label} [${chunk.status}]`);
+        return [
+            `[two-phase] Phase 2 (review): the implementation is done. Review scope: ${view.scopeFiles} file(s) in ${view.chunks.length} chunk(s).`,
+            ...chunkLines,
+            `Review each chunk in scope (never the whole repository). Record confirmed findings as ${REVIEW_FIX_PREFIX} todos grouped by root cause and severity, fix them after the pass, then run one final integration review over the task diff. The request is not complete while review or fix items remain.`,
+        ].join("\n");
+    }
+    if (state.phase === "review_fixes") {
+        return `[two-phase] Phase 2 (review fixes): ${view.reviewFixOpen} review-fix todo(s) open. Fix the confirmed findings, check the changed area, then run one final integration review over the task diff.`;
+    }
+    return "[two-phase] The task is complete: implementation, review and the required fixes are done.";
+}
+//# sourceMappingURL=rules.js.map

package/packages/two-phase-review/dist/rules.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"rules.js","sourceRoot":"","sources":["../src/rules.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAGH,OAAO,EACN,iBAAiB,EACjB,aAAa,EACb,YAAY,EACZ,cAAc,GAEd,MAAM,eAAe,CAAC;AACvB,OAAO,EAAE,gBAAgB,EAAE,MAAM,cAAc,CAAC;AAEhD,oCAAoC;AACpC,MAAM,CAAC,MAAM,kBAAkB,GAAG;;;;;;;;;;;;;;;;;;;;IAoB9B,aAAa;IACb,iBAAiB;;qKAEgJ,CAAC;AAqBtK,6DAA6D;AAC7D,MAAM,UAAU,cAAc,CAAC,KAAoB,EAAE,KAAiB;IACrE,MAAM,OAAO,GAAG,cAAc,CAAC,KAAK,CAAC,CAAC;IACtC,OAAO;QACN,KAAK,EAAE,KAAK,CAAC,KAAK;QAClB,kBAAkB,EAAE,OAAO,CAAC,cAAc;QAC1C,UAAU,EAAE,OAAO,CAAC,MAAM;QAC1B,aAAa,EAAE,OAAO,CAAC,SAAS;QAChC,UAAU,EAAE,KAAK,CAAC,UAAU,CAAC,MAAM;QACnC,MAAM,EAAE,KAAK,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,CAAC;YACpC,EAAE,EAAE,KAAK,CAAC,EAAE;YACZ,KAAK,EAAE,KAAK,CAAC,KAAK;YAClB,MAAM,EAAE,KAAK,CAAC,MAAM;SACpB,CAAC,CAAC;QACH,MAAM,EAAE,KAAK,CAAC,OAAO,CAAC,MAAM,GAAG,CAAC;QAChC,OAAO,EAAE,KAAK,CAAC,OAAO;KACtB,CAAC;AACH,CAAC;AAED,iDAAiD;AACjD,MAAM,UAAU,sBAAsB,CAAC,KAAoB;IAC1D,MAAM,OAAO,GAAG,gBAAgB,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;IAChD,MAAM,KAAK,GAAG,OAAO,CAAC,OAAO,CAAC,CAAC,CAAC,IAAI,eAAe,CAAC;IACpD,MAAM,IAAI,GAAG,OAAO,CAAC,OAAO,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,KAAK,OAAO,CAAC,OAAO,CAAC,MAAM,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,CAAC;IACtF,OAAO,GAAG,aAAa,oCAAoC,KAAK,GAAG,IAAI,GAAG,CAAC;AAC5E,CAAC;AAED,qDAAqD;AACrD,MAAM,UAAU,0BAA0B,CAAC,KAAoB;IAC9D,MAAM,OAAO,GAAG,gBAAgB,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;IAChD,MAAM,KAAK,GAAa,EAAE,CAAC;IAC3B,IAAI,OAAO,CAAC,OAAO,CAAC,MAAM,GAAG,CAAC;QAAE,KAAK,CAAC,IAAI,CAAC,WAAW,OAAO,CAAC,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACpF,IAAI,OAAO,CAAC,MAAM;QAAE,KAAK,CAAC,IAAI,CAAC,WAAW,OAAO,CAAC,MAAM,EAAE,CAAC,CAAC;IAC5D,IAAI,KAAK,CAAC,UAAU,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACjC,MAAM,KAAK,GAAG,KAAK,CAAC,UAAU,CAAC,KAAK,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;QAC5C,MAAM,IAAI,GAAG,KAAK,CAAC,UAAU,CAAC,MAAM,GAAG,KAAK,CAAC,MAAM,CAAC;QACpD,KAAK,CAAC,IAAI,CAAC,UAAU,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG,IAAI,GAAG,CAAC,CAAC,CAAC,CAAC,MAAM,IAAI,QAAQ,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;IAC/E,CAAC;IACD,KAAK,CAAC,IAAI,CAAC,4DAA4D,CAAC,CAAC;IACzE,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AACzB,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,iBAAiB,CAAC,KAAoB;IACrD,QAAQ,KAAK,CAAC,KAAK,EAAE,CAAC;QACrB,KAAK,gBAAgB;YACpB,OAAO,wBAAwB,CAAC;QACjC,KAAK,gBAAgB;YACpB,OAAO,wBAAwB,CAAC;QACjC,KAAK,QAAQ,EAAE,CAAC;YACf,MAAM,IAAI,GAAG,KAAK,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,CAAC,MAAM,KAAK,MAAM,CAAC,CAAC,MAAM,CAAC;YAC5E,OAAO,mBAAmB,IAAI,IAAI,KAAK,CAAC,MAAM,CAAC,MAAM,cAAc,KAAK,CAAC,UAAU,CAAC,MAAM,WAAW,CAAC;QACvG,CAAC;QACD,KAAK,cAAc;YAClB,OAAO,yBAAyB,KAAK,CAAC,QAAQ,CAAC,MAAM,cAAc,CAAC;QACrE,KAAK,UAAU;YACd,OAAO,kBAAkB,CAAC;QAC3B;YACC,OAAO,wBAAwB,CAAC;IAClC,CAAC;AACF,CAAC;AAED;;;;;GAKG;AACH,MAAM,UAAU,qBAAqB,CAAC,KAAkB;IACvD,OAAO,GAAG,aAAa,IAAI,KAAK,CAAC,EAAE,KAAK,KAAK,CAAC,KAAK,EAAE,CAAC;AACvD,CAAC;AAED,gEAAgE;AAChE,MAAM,UAAU,yBAAyB,CAAC,KAAkB;IAC3D,MAAM,KAAK,GAAG,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,KAAK,IAAI,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAChE,OAAO;QACN,uBAAuB,KAAK,CAAC,EAAE,KAAK,KAAK,CAAC,KAAK,IAAI;QACnD,KAAK;QACL,EAAE;QACF,wDAAwD,iBAAiB,4CAA4C;KACrH,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AACd,CAAC;AAED,qDAAqD;AACrD,MAAM,UAAU,WAAW,CAAC,IAAc,EAAE,KAAkB;IAC7D,IAAI,YAAY,CAAC,IAAI,CAAC,KAAK,QAAQ;QAAE,OAAO,KAAK,CAAC;IAClD,MAAM,OAAO,GAAG,IAAI,CAAC,OAAO,IAAI,EAAE,CAAC;IACnC,MAAM,EAAE,GAAG,OAAO,CAAC,OAAO,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC;IACrC,IAAI,EAAE,GAAG,CAAC;QAAE,OAAO,KAAK,CAAC;IACzB,uCAAuC;IACvC,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,EAAE,GAAG,KAAK,CAAC,EAAE,CAAC,MAAM,EAAE,EAAE,GAAG,KAAK,CAAC,EAAE,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC;IAC5E,OAAO,KAAK,KAAK,EAAE,IAAI,CAAC,WAAW,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;AACjD,CAAC;AAED;;;;;;GAMG;AACH,MAAM,UAAU,eAAe,CAAC,KAAoB,EAAE,KAAiB;IACtE,IAAI,KAAK,CAAC,MAAM,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,KAAK,CAAC;IAC5C,IAAI,OAAO,GAAG,KAAK,CAAC;IACpB,MAAM,MAAM,GAAG,KAAK,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,KAAK,EAAE,EAAE;QACzC,IAAI,KAAK,CAAC,MAAM,KAAK,MAAM;YAAE,OAAO,KAAK,CAAC;QAC1C,MAAM,IAAI,GAAG,KAAK,CAAC,IAAI,CACtB,CAAC,IAAI,EAAE,EAAE,CAAC,WAAW,CAAC,IAAI,EAAE,KAAK,CAAC,IAAI,IAAI,CAAC,MAAM,KAAK,WAAW,CACjE,CAAC;QACF,IAAI,CAAC,IAAI;YAAE,OAAO,KAAK,CAAC;QACxB,OAAO,GAAG,IAAI,CAAC;QACf,OAAO,EAAE,GAAG,KAAK,EAAE,MAAM,EAAE,MAAe,EAAE,CAAC;IAC9C,CAAC,CAAC,CAAC;IACH,OAAO,OAAO,CAAC,CAAC,CAAC,EAAE,GAAG,KAAK,EAAE,MAAM,EAAE,CAAC,CAAC,CAAC,KAAK,CAAC;AAC/C,CAAC;AAED;;;;;GAKG;AACH,MAAM,UAAU,mBAAmB,CAClC,KAAoB,EACpB,OAAuD;IAEvD,MAAM,OAAO,GAAG,CAAC,GAAG,IAAI,GAAG,CAAC,KAAK,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC,MAAM,EAAE,EAAE,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC;IAC3E,OAAO;QACN,8CAA8C,OAAO,CAAC,kBAAkB,mCAAmC;QAC3G,6BAA6B,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,OAAO,CAAC,MAAM,iEAAiE;QAClI,kLAAkL;KAClL,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;AACb,CAAC;AAED;;;;;GAKG;AACH,MAAM,UAAU,qBAAqB,CAAC,KAAoB,EAAE,IAAe;IAC1E,IAAI,KAAK,CAAC,KAAK,KAAK,gBAAgB,EAAE,CAAC;QACtC,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,CAAC;YAClB,OAAO,yCAAyC,IAAI,CAAC,kBAAkB,wKAAwK,CAAC;QACjP,CAAC;QACD,MAAM,OAAO,GAAG,gBAAgB,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;QAChD,OAAO,yCAAyC,IAAI,CAAC,kBAAkB,mEAAmE,OAAO,CAAC,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,eAAe,6GAA6G,aAAa,uEAAuE,CAAC;IAC1X,CAAC;IAED,IAAI,KAAK,CAAC,KAAK,KAAK,gBAAgB,EAAE,CAAC;QACtC,OAAO,mHAAmH,IAAI,CAAC,UAAU,YAAY,CAAC;IACvJ,CAAC;IAED,IAAI,KAAK,CAAC,KAAK,KAAK,QAAQ,EAAE,CAAC;QAC9B,MAAM,UAAU,GAAG,IAAI,CAAC,MAAM,CAAC,GAAG,CACjC,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,KAAK,CAAC,EAAE,KAAK,KAAK,CAAC,KAAK,KAAK,KAAK,CAAC,MAAM,GAAG,CAC5D,CAAC;QACF,OAAO;YACN,2EAA2E,IAAI,CAAC,UAAU,eAAe,IAAI,CAAC,MAAM,CAAC,MAAM,YAAY;YACvI,GAAG,UAAU;YACb,yFAAyF,iBAAiB,6LAA6L;SACvS,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACd,CAAC;IAED,IAAI,KAAK,CAAC,KAAK,KAAK,cAAc,EAAE,CAAC;QACpC,OAAO,uCAAuC,IAAI,CAAC,aAAa,yIAAyI,CAAC;IAC3M,CAAC;IAED,OAAO,2FAA2F,CAAC;AACpG,CAAC"}

package/packages/two-phase-review/dist/runtime.d.ts

@@ -0,0 +1,70 @@
+/**
+ * The runtime wiring of the two-phase workflow.
+ *
+ * The wiring lives in the package so it can be tested without a Pi session:
+ * the extension API, the todo reader and the workspace are injected. Every
+ * decision that the runtime takes is conservative - when the state of the
+ * todo list is unknown, the workflow waits instead of guessing.
+ */
+import { type TodoLike } from "./classify.js";
+import { TwoPhaseStateStore } from "./state.js";
+import type { TwoPhasePhase, TwoPhaseState } from "./types.js";
+/** The marker every message of the harness itself carries. */
+export declare const HARNESS_MESSAGE_MARKER = "[harness:";
+/** The part of the extension API the workflow needs. */
+export interface TwoPhaseExtensionApi {
+    on(event: string, handler: (...args: any[]) => unknown): void;
+    appendEntry?(type: string, data: unknown): void;
+    sendUserMessage?(content: string, options?: {
+        deliverAs?: "steer" | "followUp";
+    }): Promise<void> | void;
+}
+/** The part of the extension context the workflow needs. */
+export interface TwoPhaseExtensionContext {
+    sessionManager?: {
+        getSessionId?: () => string | undefined;
+    };
+    ui?: {
+        setStatus?: (key: string, value: string) => void;
+    };
+}
+/** The result of reading the todo list of the session. */
+export interface TodoReadResult {
+    /** Is the todo list readable at all? */
+    available: boolean;
+    /** The todo items (empty when the list is not readable). */
+    todos: TodoLike[];
+    /** Why the list is not readable. */
+    reason?: string;
+}
+/** Reads the todo list of the session. */
+export type TodoReader = (ctx: unknown) => Promise<TodoReadResult>;
+/** The options of the runtime. */
+export interface TwoPhaseRuntimeOptions {
+    pi: TwoPhaseExtensionApi;
+    /** Reads the todo list of the session. */
+    readTodos: TodoReader;
+    /** The root of the workspace (the Git repository of the task). */
+    workspaceRoot?: () => string;
+    /** Where the state of the workflow is stored. */
+    store?: TwoPhaseStateStore;
+    /** Writes a line into the log of the harness. */
+    log?: (message: string) => void;
+    /** Extra review tool patterns of the deployment. */
+    extraPatterns?: string[];
+}
+/** The runtime of the workflow. */
+export interface TwoPhaseRuntime {
+    /** The persisted state of the workflow. */
+    getState(): TwoPhaseState;
+    /** The phase the workflow is in (derived from the state and the todo list). */
+    getPhase(todos?: TodoLike[]): TwoPhasePhase;
+}
+/**
+ * Register the two-phase workflow in the session.
+ *
+ * The handlers are registered before the function returns, so a session that
+ * starts right after the registration is already covered.
+ */
+export declare function initTwoPhaseReview(options: TwoPhaseRuntimeOptions): Promise<TwoPhaseRuntime>;
+//# sourceMappingURL=runtime.d.ts.map

package/packages/two-phase-review/dist/runtime.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"runtime.d.ts","sourceRoot":"","sources":["../src/runtime.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAEH,OAAO,EAIN,KAAK,QAAQ,EACb,MAAM,eAAe,CAAC;AAOvB,OAAO,EACN,kBAAkB,EASlB,MAAM,YAAY,CAAC;AAYpB,OAAO,KAAK,EAAE,aAAa,EAAE,aAAa,EAAE,MAAM,YAAY,CAAC;AAE/D,8DAA8D;AAC9D,eAAO,MAAM,sBAAsB,cAAc,CAAC;AAElD,wDAAwD;AACxD,MAAM,WAAW,oBAAoB;IACpC,EAAE,CAAC,KAAK,EAAE,MAAM,EAAE,OAAO,EAAE,CAAC,GAAG,IAAI,EAAE,GAAG,EAAE,KAAK,OAAO,GAAG,IAAI,CAAC;IAC9D,WAAW,CAAC,CAAC,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,OAAO,GAAG,IAAI,CAAC;IAChD,eAAe,CAAC,CAAC,OAAO,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE;QAAE,SAAS,CAAC,EAAE,OAAO,GAAG,UAAU,CAAA;KAAE,GAAG,OAAO,CAAC,IAAI,CAAC,GAAG,IAAI,CAAC;CACxG;AAED,4DAA4D;AAC5D,MAAM,WAAW,wBAAwB;IACxC,cAAc,CAAC,EAAE;QAAE,YAAY,CAAC,EAAE,MAAM,MAAM,GAAG,SAAS,CAAA;KAAE,CAAC;IAC7D,EAAE,CAAC,EAAE;QAAE,SAAS,CAAC,EAAE,CAAC,GAAG,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,KAAK,IAAI,CAAA;KAAE,CAAC;CAC1D;AAED,0DAA0D;AAC1D,MAAM,WAAW,cAAc;IAC9B,wCAAwC;IACxC,SAAS,EAAE,OAAO,CAAC;IACnB,4DAA4D;IAC5D,KAAK,EAAE,QAAQ,EAAE,CAAC;IAClB,oCAAoC;IACpC,MAAM,CAAC,EAAE,MAAM,CAAC;CAChB;AAED,0CAA0C;AAC1C,MAAM,MAAM,UAAU,GAAG,CAAC,GAAG,EAAE,OAAO,KAAK,OAAO,CAAC,cAAc,CAAC,CAAC;AAEnE,kCAAkC;AAClC,MAAM,WAAW,sBAAsB;IACtC,EAAE,EAAE,oBAAoB,CAAC;IACzB,0CAA0C;IAC1C,SAAS,EAAE,UAAU,CAAC;IACtB,kEAAkE;IAClE,aAAa,CAAC,EAAE,MAAM,MAAM,CAAC;IAC7B,iDAAiD;IACjD,KAAK,CAAC,EAAE,kBAAkB,CAAC;IAC3B,iDAAiD;IACjD,GAAG,CAAC,EAAE,CAAC,OAAO,EAAE,MAAM,KAAK,IAAI,CAAC;IAChC,oDAAoD;IACpD,aAAa,CAAC,EAAE,MAAM,EAAE,CAAC;CACzB;AAED,mCAAmC;AACnC,MAAM,WAAW,eAAe;IAC/B,2CAA2C;IAC3C,QAAQ,IAAI,aAAa,CAAC;IAC1B,+EAA+E;IAC/E,QAAQ,CAAC,KAAK,CAAC,EAAE,QAAQ,EAAE,GAAG,aAAa,CAAC;CAC5C;AAsBD;;;;;GAKG;AACH,wBAAsB,kBAAkB,CACvC,OAAO,EAAE,sBAAsB,GAC7B,OAAO,CAAC,eAAe,CAAC,CA0Q1B"}