@siuver/omp-debug-mode 0.1.3 → 0.1.4

package/src/ui.ts

@@ -1,10 +1,9 @@
 import type { ExtensionContext } from "@oh-my-pi/pi-coding-agent";
 import { PROCEED_REMINDER } from "./methodology";
-import type { DebugState } from "./state";
+import type { DebugState, EvidenceRequest } from "./state";
+import { pendingEvidenceRequests } from "./state";
 
-/** `setWidget` caps string-array widgets at 10 lines. */
 export const WIDGET_MAX_LINES = 10;
-const WIDGET_MAX_STEPS = 6;
 const WIDGET_MAX_WIDTH = 90;
 
 export type WidgetTone = "accent" | "dim";
@@ -24,6 +23,13 @@ function clip(text: string): string {
 	return text.length > WIDGET_MAX_WIDTH ? `${text.slice(0, WIDGET_MAX_WIDTH - 1)}…` : text;
 }
 
+/** The command surface is the only interaction path at the gate. */
+const COMMAND_LINES = [
+	"/debug-proceed [details]",
+	"/debug-evidence <request-id> <path>",
+	"/debug-done · /debug-abort · /debug-status",
+] as const;
+
 function plural(count: number, singular: string): string {
 	return `${count} ${singular}${count === 1 ? "" : "s"}`;
 }
@@ -33,27 +39,46 @@ function logEntries(count: number): string {
 }
 
 /**
- * Reproduction-gate widget: the call to action, an excerpt of the agent's
- * reproduction steps, and the live evidence counter. Always within the host's
- * line budget so nothing important is silently dropped.
+ * Reproduction-gate widget: the call to action, pending user evidence,
+ * reproduction-step context, the command surface, and the live evidence
+ * counter. Commands are the only interaction path — there is no menu. The
+ * live log counter is always the final line within the host's line budget.
  */
-export function waitingWidgetLines(state: DebugState, logCount: number): WidgetLine[] {
+export function waitingWidgetLines(
+	state: DebugState,
+	logCount: number,
+	pendingEvidence: EvidenceRequest[] = pendingEvidenceRequests(state, state.round),
+): WidgetLine[] {
 	const lines: WidgetLine[] = [{ text: PROCEED_REMINDER, tone: "accent" }];
-	const shown = state.reproductionSteps.slice(0, WIDGET_MAX_STEPS);
-	for (const step of shown) lines.push({ text: clip(step), tone: "dim" });
-	const hidden = state.reproductionSteps.length - shown.length;
-	if (hidden > 0) lines.push({ text: `… +${plural(hidden, "more step")} in the transcript`, tone: "dim" });
+	for (const request of pendingEvidence.slice(0, 2)) {
+		lines.push({ text: clip(`↪ ${request.id} ${request.title}: ${request.instructions[0] ?? ""}`), tone: "accent" });
+	}
+	for (const command of COMMAND_LINES) lines.push({ text: command, tone: "accent" });
+
+	// Reproduction details are useful context, but command discoverability and
+	// the live log counter must survive the host's ten-line widget limit.
+	const reserved = lines.length + 2;
+	const stepBudget = Math.max(0, WIDGET_MAX_LINES - reserved);
+	if (stepBudget > 0 && state.reproductionSteps.length > 0) {
+		const showMoreLine = state.reproductionSteps.length > stepBudget;
+		const shownCount = showMoreLine ? Math.max(0, stepBudget - 1) : stepBudget;
+		for (const step of state.reproductionSteps.slice(0, shownCount)) {
+			lines.push({ text: clip(step), tone: "dim" });
+		}
+		const hidden = state.reproductionSteps.length - shownCount;
+		if (hidden > 0) lines.push({ text: `… +${plural(hidden, "more step")} in the transcript`, tone: "dim" });
+	}
 	lines.push({
-		text: `/debug-menu · run ${state.runId ?? "none"} — ${logEntries(logCount)}`,
+		text: `evidence: ${plural(pendingEvidence.length, "pending request")}, ${plural(state.evidenceArtifacts.length, "attached artifact")}`,
+		tone: pendingEvidence.length > 0 || state.evidenceArtifacts.length > 0 ? "accent" : "dim",
+	});
+	lines.push({
+		text: `run ${state.runId ?? "none"} — ${logEntries(logCount)}`,
 		tone: logCount > 0 ? "accent" : "dim",
 	});
 	return lines.slice(0, WIDGET_MAX_LINES);
 }
 
-export function reviewMenuTitle(round: number, logCount: number, probeCount: number): string {
-	return `Review debug round ${round} · ${logEntries(logCount)} · ${plural(probeCount, "probe")}`;
-}
-
 /**
  * Render the status entry and the reproduction widget. `getLogCount` is only
  * consulted at the reproduction gate so idle phases do not touch the log files.

package/src/review-actions.ts

@@ -1,22 +0,0 @@
-export const REVIEW_MARK_FIXED = "Mark as fixed";
-export const REVIEW_PROCEED = "Proceed";
-export const REVIEW_ADD_DETAILS = "Add reproduction details";
-export const REVIEW_ABORT = "Abort debug mode";
-
-export const REVIEW_OPTIONS = [
-	REVIEW_MARK_FIXED,
-	REVIEW_PROCEED,
-	REVIEW_ADD_DETAILS,
-	REVIEW_ABORT,
-] as const;
-
-export const REVIEW_DESCRIPTIONS: Record<string, string> = {
-	[REVIEW_MARK_FIXED]: "Remove every probe and summarize the root cause",
-	[REVIEW_PROCEED]: "Read the captured logs, judge each hypothesis, continue",
-	[REVIEW_ADD_DETAILS]: "Describe what you observed before continuing",
-	[REVIEW_ABORT]: "Stop debugging and delete the logs; code changes stay",
-};
-
-export function reviewMenuOptions(): { label: string; description: string }[] {
-	return REVIEW_OPTIONS.map(label => ({ label, description: REVIEW_DESCRIPTIONS[label] ?? "" }));
-}