prreviewbuddy 0.46.0 → 0.48.0

package/CHANGELOG.md

@@ -1,5 +1,55 @@
 # Changelog
 
+## 0.48.0
+
+**Active, Waiting and Done.** A review can now be marked **Waiting**, for when you have done your
+pass and expect another round, or **Done**, for when you are finished with it. The reviews list keeps
+your active reviews at the top, with **Waiting** below them and **Done** below that, folded away. Use
+the **⋯** menu on a card, or select several and use the bar at the bottom. **Move to active** puts a
+review back. This is separate from **Reviewed**, which still records that you have been through it.
+
+**Waiting reviews come back when something changes.** If a Waiting review gets new commits, or its
+pull request is merged, closed or reopened, it shows up in the main list again marked **Changed while
+waiting**. Marking it Waiting again puts it back. A Done review never comes back by itself; if it
+changes, it is marked **Changed since done** where it is. Marking a review Waiting starts watching
+it straight away, so a push made soon after still brings it back. Adding a Waiting or Done review to
+your review queue moves it back to Active.
+
+**The list no longer reorders when you open a review.** Reviews are ordered by what last happened to
+the change: when it was analysed, when new commits arrived, or when the pull request was merged,
+closed or reopened. The time on each card now says which of those it was, for example "Updated
+2 hours ago" or "Merged 1 day ago". Saved reviews still come first.
+
+**Load more instead of pages.** Each section shows fifteen reviews and a **Load more** button for the
+rest. Search and the filters still cover every review, in all three sections.
+
+**Done reviews are kept longer.** Like saved reviews, Done reviews are kept for ninety days rather
+than thirty, counting from the last time you opened or marked them.
+
+**Light mode.** The workspace now has a light theme as well as the dark one. The rail, the top bars,
+Buddy AI, the reviews list, Notifications and Settings are drawn in pale greys around a white
+review, so the review stays the brightest thing on the screen. Purple is kept for what you have
+chosen and for the main action, as a pale tint rather than a solid block.
+
+**Follows your device, or stays put.** Settings has a new **Appearance** section with **System**,
+**Light** and **Dark**. System is the default, and it changes with your device at once, without
+reloading the page, when your Mac switches between light and dark. Choosing a theme shows it on the
+Settings page straight away; **Save changes** keeps it. If you have not chosen, the workspace follows
+your device, so on a Mac set to light it now opens in light mode.
+
+**Readable in both.** Every piece of text in the light theme, including timestamps, counts and other
+small details, meets the WCAG 4.5:1 contrast level against whatever it sits on.
+
+**Fixed: buttons in a review that went blank on hover.** In the dark theme, pointing at **Open in**
+your editor, **Show more of this file**, **View files**, **Regenerate** or a diagram's zoom buttons
+turned the label white on the white review, so it disappeared. They now keep their dark label.
+
+**A clearer message when OpenAI refuses Codex while it is signed in.** If Codex says it is signed in
+but OpenAI answers "Incorrect API key", the review no longer suggests signing in again, and no longer
+suggests looking for an API key of your own: the key in that error is Codex's own. It now suggests
+`codex doctor` to see what Codex is using, and `codex features list` to find an experimental Codex
+feature to turn off, which is what fixed it when we saw it.
+
 ## 0.46.0
 
 **The change map: see what a change is about, and how its parts affect each other.** The Diagrams

package/dist/{delete_review-lmVdoybL.js → delete_review-D52wkt66.js}

@@ -1129,6 +1129,19 @@ var CodexUnavailableError = class extends AgentUnavailableError {};
 */
 var AUTHENTICATION = /not (?:logged|signed) in|unauthori[sz]ed|401|authenticat|credential|sign[- ]?in|log ?in/i;
 /**
+* OpenAI refusing a request Codex believes is signed in, which is not nobody being signed in.
+*
+* Seen with Codex reporting "Logged in using ChatGPT" and no API key anywhere in the reviewer's
+* environment: the key named in the error is Codex's own, not one of theirs. Signing in again did
+* not clear it; turning off an experimental Codex feature did. So the advice is Codex's own
+* diagnosis and its feature switches, never a login loop and never a hunt for an API key. Still an
+* authentication failure, so no Retry is offered.
+*/
+var REJECTED_KEY = /incorrect api key|invalid[_ ]api[_ ]key/i;
+function isAuthentication(detail) {
+	return AUTHENTICATION.test(detail) || REJECTED_KEY.test(detail);
+}
+/**
 * Which kind of failure the agent's own sentence describes.
 *
 * Only the one distinction that changes what a person is offered: a run that failed because nobody
@@ -1137,7 +1150,7 @@ var AUTHENTICATION = /not (?:logged|signed) in|unauthori[sz]ed|401|authenticat|c
 * label for them.
 */
 function kindOf(detail) {
-	return AUTHENTICATION.test(detail) ? "agent_authentication_failed" : "agent_execution_failed";
+	return isAuthentication(detail) ? "agent_authentication_failed" : "agent_execution_failed";
 }
 var codexAgent = {
 	id: CODEX_AGENT_ID,
@@ -1147,8 +1160,10 @@ var codexAgent = {
 	captureEnv: captureEnv$1,
 	/** Which tool, which profile, and the one command that changes it. See `agent.ts` for the budget. */
 	explainFailure(detail, env = {}) {
-		if (!AUTHENTICATION.test(detail)) return null;
-		return `Codex could not sign in. This review was started using ${env.CODEX_HOME ? `the Codex profile at \`${env.CODEX_HOME}\`` : "your default Codex profile"}, so run \`codex login\` against that one.`;
+		if (!isAuthentication(detail)) return null;
+		const profile = env.CODEX_HOME ? `the Codex profile at \`${env.CODEX_HOME}\`` : "your default Codex profile";
+		if (REJECTED_KEY.test(detail)) return `OpenAI refused a request from ${profile} even though Codex reports being signed in, which usually points to Codex's own setup rather than your sign-in. Run \`codex doctor\` to see what it is using, and check \`codex features list\` for an experimental feature to turn off with \`codex features disable <name>\`.`;
+		return `Codex could not sign in. This review was started using ${profile}, so run \`codex login\` against that one.`;
 	},
 	/**
 	* Whether the command is here and runs. Deliberately not whether it is signed in.
@@ -4112,6 +4127,95 @@ function normalise(ids) {
 	return seen;
 }
 //#endregion
+//#region ../../packages/review-harness/src/workspace/lifecycle.ts
+/**
+* What happens next with a change, as the reviewer says it: still in play, parked until something
+* moves, or finished with.
+*
+* A separate axis from New / In progress / Reviewed, which says what the reviewer has *done*. A
+* change can be reviewed and waiting for another push, or reviewed and done, and one field cannot
+* hold both answers.
+*
+* One registry that the stored type, the card, the menus, the bulk bar and the route all derive
+* from. Pure on purpose: the reviews page's script imports it to move a card between sections after
+* a mark, without asking the server for the page again.
+*/
+var LIFECYCLES = [
+	{
+		id: "active",
+		action: "Move to active"
+	},
+	{
+		id: "waiting",
+		action: "Mark waiting"
+	},
+	{
+		id: "done",
+		action: "Mark done"
+	}
+];
+function isLifecycle(value) {
+	return LIFECYCLES.some((lifecycle) => lifecycle.id === value);
+}
+/** A stored value read defensively: anything this code did not write reads as Active. */
+function parseLifecycle(raw) {
+	if (typeof raw !== "object" || raw === null) return void 0;
+	const { state, at } = raw;
+	if (state !== "waiting" && state !== "done" || typeof at !== "number" || !Number.isFinite(at)) return void 0;
+	return {
+		state,
+		at
+	};
+}
+function lifecycleAction(lifecycle) {
+	return LIFECYCLES.find((entry) => entry.id === lifecycle).action;
+}
+var ACTIVITY_LABELS = {
+	created: "Created",
+	updated: "Updated",
+	merged: "Merged",
+	closed: "Closed",
+	reopened: "Reopened"
+};
+function activityLabel(kind) {
+	return ACTIVITY_LABELS[kind];
+}
+/**
+* Which section a card is drawn in.
+*
+* Waiting means "bring this back when something changes", so a Waiting review whose change moved
+* after it was parked is drawn in Active. Derived, and nothing is written: the stored state stays
+* Waiting until the reviewer says otherwise, and marking it Waiting again moves the baseline.
+*
+* Done means "leave me alone", so it never comes back by itself. `created` can never resurface
+* anything either, because a new analysis is a new record, and a new record is Active.
+*/
+function sectionOf(lifecycle, activity) {
+	if (!lifecycle) return "active";
+	if (lifecycle.state === "waiting" && changedSince(lifecycle, activity)) return "active";
+	return lifecycle.state;
+}
+/** The chip that says why a card is where it is, or absent when its section already says it. */
+function lifecycleChip(lifecycle, activity) {
+	if (!lifecycle || !changedSince(lifecycle, activity)) return void 0;
+	return lifecycle.state === "waiting" ? "Changed while waiting" : "Changed since done";
+}
+function changedSince(lifecycle, activity) {
+	return activity !== void 0 && activity.kind !== "created" && activity.at > lifecycle.at;
+}
+/**
+* Which lifecycle items a card's menu offers.
+*
+* Every state but the one the card is already in, which would be an item that does nothing. A card
+* carrying a chip is the exception and offers all three: re-marking it is how the reviewer says
+* they have seen the change, so "Mark waiting" on a Waiting review that changed is the settle
+* action, not a no-op.
+*/
+function offersLifecycle(target, lifecycle, activity) {
+	if (lifecycleChip(lifecycle, activity) !== void 0) return true;
+	return target !== (lifecycle?.state ?? "active");
+}
+//#endregion
 //#region ../../packages/review-harness/src/workspace/store.ts
 var WORKSPACES_DIR = join(STORE_ROOT, "workspaces");
 /**
@@ -4481,8 +4585,20 @@ function followedRefName(branch, followedRef) {
 * can return is the two added together. Neither constant says that on its own, which is why it is
 * said here.
 */
-function indexReviewGroups(limit, keptLimit, keptIds) {
-	return selectIndexGroups(groupByLineage(recentWorkspaces(Number.MAX_SAFE_INTEGER)), limit, keptLimit, keptIds);
+function indexReviewGroups(limit, keptLimit, keptIds, rank, groups = groupByLineage(recentWorkspaces(Number.MAX_SAFE_INTEGER))) {
+	return selectIndexGroups(groups, limit, keptLimit, keptIds, rank);
+}
+/**
+* What the reviewer explicitly kept, which is what stops the recency window hiding it: the queue,
+* Saved, and anything marked Waiting or Done.
+*
+* Done is the reviewer's archive, and an archive the recency window can empty is not one. Waiting
+* is work they expect back, and a push to it must have a card to come back on.
+*/
+function indexKeptIds(reviews, queue) {
+	const kept = new Set(queue);
+	for (const review of reviews) if (review.pinned || review.lifecycle !== void 0) kept.add(review.id);
+	return kept;
 }
 /**
 * The choosing, with the disk left out, in the same spirit as `groupByLineage` next door.
@@ -4494,12 +4610,18 @@ function indexReviewGroups(limit, keptLimit, keptIds) {
 *
 * The kept side is capped for the same reason the recent side is: `queue.json` is a file on a disk
 * somebody can edit, and an uncapped read of it is an index with five thousand cards on it.
+*
+* **Windowed by `rank`, which is what the list is ordered by.** The index passes what last happened
+* to each change, so a branch analysed two months ago and pushed to this morning is inside the
+* window rather than cut before its activity is ever looked at. Absent, it is when the newest
+* review was made, which is what every other caller has always meant by recent.
 */
-function selectIndexGroups(groups, limit, keptLimit, keptIds) {
+function selectIndexGroups(groups, limit, keptLimit, keptIds, rank = (group) => group.latest.createdAt) {
 	const kept = [];
 	const rest = [];
 	for (const group of groups) (group.history.some((review) => keptIds.has(review.id)) ? kept : rest).push(group);
-	return [...rest.slice(0, limit), ...kept.slice(0, keptLimit)].sort((a, b) => b.latest.createdAt - a.latest.createdAt);
+	const byRank = (a, b) => rank(b) - rank(a);
+	return [...rest.sort(byRank).slice(0, limit), ...kept.sort(byRank).slice(0, keptLimit)].sort(byRank);
 }
 /**
 * The grouping itself, with the disk left out.
@@ -4626,6 +4748,7 @@ function summarise(workspace) {
 		lastViewedAt: workspace.lastViewedAt,
 		completedAt: workspace.completedAt,
 		pinned: workspace.pinned === true,
+		lifecycle: parseLifecycle(workspace.lifecycle),
 		paths: files.slice(0, MAX_INDEXED_PATHS).map((file) => file.name),
 		prNumber: workspace.changeSet.data.pullRequest?.number || workspace.prContext?.pullRequest.number,
 		commitSha: workspace.session.metadata?.commitSha || (workspace.changeSet.targetType === "commit" ? workspace.changeSet.data.pullRequest.head.sha : "") || void 0,
@@ -4803,7 +4926,8 @@ function isKept(entry, queued) {
 	const id = workspaceIdFrom(entry);
 	if (id === null) return false;
 	if (queued.has(id)) return true;
-	return loadWorkspace(id)?.pinned === true;
+	const workspace = loadWorkspace(id);
+	return workspace?.pinned === true || parseLifecycle(workspace?.lifecycle)?.state === "done";
 }
 /** The review an entry belongs to, whether it is the record or the usage sidecar beside it. */
 function workspaceIdFrom(entry) {
@@ -12651,4 +12775,4 @@ function lineageIds(id) {
 	return [.../* @__PURE__ */ new Set([id, ...lineage.map((review) => review.id)])];
 }
 //#endregion
-export { queueForReview as $, positionInLineage as $t, createPrState as A, writeAgentPreference as An, allJobs as At, toneOf as B, purposeOf as Bt, readEvents as C, displayRef as Cn, forgeFor as Ct, announceJob as D, resolveModel as Dn, checkReview as Dt, telemetryUploadConsent as E, resolveAgent as En, checkCodeFreshness as Et, markAllRead as F, agentById as Fn, saveJob as Ft, extractJsonText as G, groupByLineage as Gt, withUsageRecorded as H, driverGone as Ht, markRead as I, agentFor as In, PHASES as It, readMarker as J, lineageKeyFor as Jt, ReviewBeingDeletedError as K, indexReviewGroups as Kt, pruneNotifications as L, detectAgents as Ln, isLiveJob as Lt, eventName as M, STORE_ROOT as Mn, fail as Mt, headline as N, AGENT_IDS as Nn, isTerminal as Nt, branchOf as O, clearAgentPreference as On, followedBranch as Ot, listNotifications as P, DEFAULT_AGENT_ID as Pn, loadJob as Pt, compactQueue as Q, mutateWorkspace as Qt, readNotification as R, AgentCancelledError as Rn, phasesFor as Rt, startJob as S, resolveTarget as Sn, forgeResolver as St, recordTelemetryUploadConsent as T, git as Tn, fillFileUrlTemplate as Tt, EXPLAIN_SIMPLY_PROMPT as U, isClaimed as Ut, unreadCount as V, clearClaim as Vt, asString as W, followedRefName as Wt, modelHelpLines as X, loadWorkspace as Xt, removeWorktree as Y, lineagePosition as Yt, relativeTime as Z, matchingWorkspaceIds as Zt, agentEnvOf as _, issuesOutstanding as _n, parseWorkspacePort as _t, wasBlocked as a, touchWorkspace as an, reviewsUrl as at, reanalyseReview as b, processDiscussion as bn, writePortPreference as bt, RefreshUnavailableError as c, dequeueOnComplete as cn, stopServer as ct, refreshPrContext as d, queueRemove as dn, BUILD_VERSION as dt, previousKissResult as en, readIndexToken as et, isUnchanged as f, queueStamp as fn, PACKAGE_NAME as ft, runningJobs as g, isQuestionOutstanding as gn, MIN_WORKSPACE_PORT as gt, liveUpdateFor as h, isIssueOutstanding as hn, MAX_WORKSPACE_PORT as ht, liveJobsFor as i, summarise as in, readServerRecord as it, createPrUpdate as j, CONFIG_PATH as jn, endedAt as jt, subjectOf as k, readAgentPreference as kn, reviewedShaOf as kt, mergeUpdateInto as l, queueAdd as ln, workspaceUrl as lt, UpdateAlreadyRunningError as m, doneVerb as mn, DEFAULT_WORKSPACE_PORT as mt, deleteReview as n, reviewedCommit as nn, clearServerRecord as nt, discardJob as o, workspaceRevision as on, serverEntryExists as ot, askCheckout as p, readQueue as pn, feedbackUrl as pt, MANAGED_ROOT as q, latestKissRun as qt, lineageIds as r, reviewedRepositories as rn, ensureServer as rt, FETCH_TIMEOUT_MS as s, workspaceStamps as sn, spawnServer as st, deleteLineage as t, recentWorkspaces as tn, bootstrapUrl as tt, updateReview as u, queueAddMany as un, writeServerRecord as ut, recordKissRun as v, questionsOutstanding as vn, resolveWorkspacePort as vt, record as w, isCommitReview as wn, remoteHost as wt, runJob as x, describeAuthorship as xn, claimRequestUrl as xt, startKissJob as y, reviewerDispositions as yn, statedWorkspacePort as yt, removeNotification as z, AgentUnavailableError as zn, progressSteps as zt };
+export { queueForReview as $, mutateWorkspace as $t, createPrState as A, describeAuthorship as An, allJobs as At, toneOf as B, CONFIG_PATH as Bn, purposeOf as Bt, readEvents as C, doneVerb as Cn, forgeFor as Ct, announceJob as D, questionsOutstanding as Dn, checkReview as Dt, telemetryUploadConsent as E, issuesOutstanding as En, checkCodeFreshness as Et, markAllRead as F, resolveAgent as Fn, saveJob as Ft, extractJsonText as G, agentFor as Gn, groupByLineage as Gt, withUsageRecorded as H, AGENT_IDS as Hn, driverGone as Ht, markRead as I, resolveModel as In, PHASES as It, readMarker as J, AgentUnavailableError as Jn, latestKissRun as Jt, ReviewBeingDeletedError as K, detectAgents as Kn, indexKeptIds as Kt, pruneNotifications as L, clearAgentPreference as Ln, isLiveJob as Lt, eventName as M, displayRef as Mn, fail as Mt, headline as N, isCommitReview as Nn, isTerminal as Nt, branchOf as O, reviewerDispositions as On, followedBranch as Ot, listNotifications as P, git as Pn, loadJob as Pt, compactQueue as Q, matchingWorkspaceIds as Qt, readNotification as R, readAgentPreference as Rn, phasesFor as Rt, startJob as S, readQueue as Sn, forgeResolver as St, recordTelemetryUploadConsent as T, isQuestionOutstanding as Tn, fillFileUrlTemplate as Tt, EXPLAIN_SIMPLY_PROMPT as U, DEFAULT_AGENT_ID as Un, isClaimed as Ut, unreadCount as V, STORE_ROOT as Vn, clearClaim as Vt, asString as W, agentById as Wn, followedRefName as Wt, modelHelpLines as X, lineagePosition as Xt, removeWorktree as Y, lineageKeyFor as Yt, relativeTime as Z, loadWorkspace as Zt, agentEnvOf as _, dequeueOnComplete as _n, parseWorkspacePort as _t, wasBlocked as a, summarise as an, reviewsUrl as at, reanalyseReview as b, queueRemove as bn, writePortPreference as bt, RefreshUnavailableError as c, workspaceStamps as cn, stopServer as ct, refreshPrContext as d, isLifecycle as dn, BUILD_VERSION as dt, positionInLineage as en, readIndexToken as et, isUnchanged as f, lifecycleAction as fn, PACKAGE_NAME as ft, runningJobs as g, sectionOf as gn, MIN_WORKSPACE_PORT as gt, liveUpdateFor as h, parseLifecycle as hn, MAX_WORKSPACE_PORT as ht, liveJobsFor as i, reviewedRepositories as in, readServerRecord as it, createPrUpdate as j, resolveTarget as jn, endedAt as jt, subjectOf as k, processDiscussion as kn, reviewedShaOf as kt, mergeUpdateInto as l, LIFECYCLES as ln, workspaceUrl as lt, UpdateAlreadyRunningError as m, offersLifecycle as mn, DEFAULT_WORKSPACE_PORT as mt, deleteReview as n, recentWorkspaces as nn, clearServerRecord as nt, discardJob as o, touchWorkspace as on, serverEntryExists as ot, askCheckout as p, lifecycleChip as pn, feedbackUrl as pt, MANAGED_ROOT as q, AgentCancelledError as qn, indexReviewGroups as qt, lineageIds as r, reviewedCommit as rn, ensureServer as rt, FETCH_TIMEOUT_MS as s, workspaceRevision as sn, spawnServer as st, deleteLineage as t, previousKissResult as tn, bootstrapUrl as tt, updateReview as u, activityLabel as un, writeServerRecord as ut, recordKissRun as v, queueAdd as vn, resolveWorkspacePort as vt, record as w, isIssueOutstanding as wn, remoteHost as wt, runJob as x, queueStamp as xn, claimRequestUrl as xt, startKissJob as y, queueAddMany as yn, statedWorkspacePort as yt, removeNotification as z, writeAgentPreference as zn, progressSteps as zt };

package/dist/main.js

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-import { $ as queueForReview, $t as positionInLineage, An as writeAgentPreference, At as allJobs, Bt as purposeOf, Cn as displayRef, Dn as resolveModel, Dt as checkReview, E as telemetryUploadConsent, En as resolveAgent, Fn as agentById, Gt as groupByLineage, Ht as driverGone, It as PHASES, J as readMarker, Jt as lineageKeyFor, Ln as detectAgents, Mn as STORE_ROOT, Nn as AGENT_IDS, Nt as isTerminal, Pn as DEFAULT_AGENT_ID, Pt as loadJob, Qt as mutateWorkspace, S as startJob, Sn as resolveTarget, St as forgeResolver, T as recordTelemetryUploadConsent, Tn as git, Ut as isClaimed, Vt as clearClaim, Wt as followedRefName, X as modelHelpLines, Xt as loadWorkspace, Y as removeWorktree, Z as relativeTime, Zt as matchingWorkspaceIds, _t as parseWorkspacePort, a as wasBlocked, at as reviewsUrl, b as reanalyseReview, bt as writePortPreference, ct as stopServer, dt as BUILD_VERSION, et as readIndexToken, ft as PACKAGE_NAME, g as runningJobs, gt as MIN_WORKSPACE_PORT, h as liveUpdateFor, ht as MAX_WORKSPACE_PORT, i as liveJobsFor, in as summarise, jn as CONFIG_PATH, kn as readAgentPreference, l as mergeUpdateInto, lt as workspaceUrl, m as UpdateAlreadyRunningError, mt as DEFAULT_WORKSPACE_PORT, n as deleteReview$1, q as MANAGED_ROOT, rn as reviewedRepositories, rt as ensureServer, tn as recentWorkspaces, tt as bootstrapUrl, u as updateReview, v as recordKissRun, w as record$1, x as runJob, yt as statedWorkspacePort, zt as progressSteps } from "./delete_review-lmVdoybL.js";
+import { $ as queueForReview, $t as mutateWorkspace, At as allJobs, Bn as CONFIG_PATH, Bt as purposeOf, Dt as checkReview, E as telemetryUploadConsent, Fn as resolveAgent, Gt as groupByLineage, Hn as AGENT_IDS, Ht as driverGone, In as resolveModel, It as PHASES, J as readMarker, Kn as detectAgents, Mn as displayRef, Nt as isTerminal, Pn as git, Pt as loadJob, Qt as matchingWorkspaceIds, Rn as readAgentPreference, S as startJob, St as forgeResolver, T as recordTelemetryUploadConsent, Un as DEFAULT_AGENT_ID, Ut as isClaimed, Vn as STORE_ROOT, Vt as clearClaim, Wn as agentById, Wt as followedRefName, X as modelHelpLines, Y as removeWorktree, Yt as lineageKeyFor, Z as relativeTime, Zt as loadWorkspace, _t as parseWorkspacePort, a as wasBlocked, an as summarise, at as reviewsUrl, b as reanalyseReview, bt as writePortPreference, ct as stopServer, dt as BUILD_VERSION, en as positionInLineage, et as readIndexToken, ft as PACKAGE_NAME, g as runningJobs, gt as MIN_WORKSPACE_PORT, h as liveUpdateFor, ht as MAX_WORKSPACE_PORT, i as liveJobsFor, in as reviewedRepositories, jn as resolveTarget, l as mergeUpdateInto, lt as workspaceUrl, m as UpdateAlreadyRunningError, mt as DEFAULT_WORKSPACE_PORT, n as deleteReview$1, nn as recentWorkspaces, q as MANAGED_ROOT, rt as ensureServer, tt as bootstrapUrl, u as updateReview, v as recordKissRun, w as record$1, x as runJob, yt as statedWorkspacePort, zn as writeAgentPreference, zt as progressSteps } from "./delete_review-D52wkt66.js";
 import { basename, dirname, join, resolve } from "node:path";
 import { spawn } from "node:child_process";
 import { existsSync, mkdirSync, readdirSync, realpathSync, rmSync, statSync, writeFileSync } from "node:fs";