@xopcai/xopc 0.0.123 → 0.0.124

package/dist/src/chat-commands/builtins/review.js

@@ -0,0 +1,167 @@
+import { effectiveWorkspacePathForSession } from "../../session/session-workspace.js";
+import { getProjectForSession } from "../../projects/workspace.js";
+import { commandRegistry } from "../registry.js";
+import { buildReviewDiffBundle, parseReviewTargetArgs, resolveGitRoot } from "../../review/review-git.js";
+import { formatReviewMarkdown } from "../../review/review-types.js";
+//#region src/chat-commands/builtins/review.ts
+const MAX_DIFF_CHARS = 6e4;
+function truncateForPrompt(text) {
+	if (text.length <= MAX_DIFF_CHARS) return {
+		text,
+		truncated: false
+	};
+	return {
+		text: text.slice(0, MAX_DIFF_CHARS) + "\n\n[diff truncated]",
+		truncated: true
+	};
+}
+function extractJsonObject(raw) {
+	const text = (raw.match(/```(?:json)?\s*([\s\S]*?)```/i)?.[1] ?? raw).trim();
+	const start = text.indexOf("{");
+	const end = text.lastIndexOf("}");
+	if (start < 0 || end <= start) throw new Error("review response did not contain JSON");
+	return JSON.parse(text.slice(start, end + 1));
+}
+function priorityFrom(raw) {
+	const n = typeof raw === "number" ? raw : Number.parseInt(String(raw ?? ""), 10);
+	if (n === 0 || n === 1 || n === 2 || n === 3) return n;
+	return 2;
+}
+function numberFrom(raw) {
+	return typeof raw === "number" && Number.isFinite(raw) ? raw : void 0;
+}
+function stringFrom(raw) {
+	return typeof raw === "string" ? raw.trim() : "";
+}
+function parseReviewJson(raw, target) {
+	const parsed = extractJsonObject(raw);
+	const rec = parsed && typeof parsed === "object" && !Array.isArray(parsed) ? parsed : {};
+	const findings = (Array.isArray(rec.findings) ? rec.findings : []).map((item) => {
+		const f = item && typeof item === "object" && !Array.isArray(item) ? item : {};
+		const loc = f.code_location && typeof f.code_location === "object" ? f.code_location : {};
+		const range = loc.line_range && typeof loc.line_range === "object" ? loc.line_range : {};
+		const title = stringFrom(f.title);
+		const body = stringFrom(f.body);
+		if (!title && !body) return void 0;
+		const finding = {
+			title: title || body.slice(0, 80),
+			body,
+			priority: priorityFrom(f.priority)
+		};
+		const confidenceScore = numberFrom(f.confidence_score ?? f.confidenceScore);
+		const filePath = stringFrom(loc.file_path ?? f.filePath);
+		const lineStart = numberFrom(range.start ?? f.lineStart);
+		const lineEnd = numberFrom(range.end ?? f.lineEnd);
+		if (confidenceScore !== void 0) finding.confidenceScore = confidenceScore;
+		if (filePath) finding.filePath = filePath;
+		if (lineStart !== void 0) finding.lineStart = lineStart;
+		if (lineEnd !== void 0) finding.lineEnd = lineEnd;
+		return finding;
+	}).filter((item) => Boolean(item));
+	const correctness = stringFrom(rec.overall_correctness ?? rec.overallCorrectness);
+	const overallCorrectness = correctness === "patch is correct" || correctness === "patch is incorrect" ? correctness : "unknown";
+	return {
+		type: "review",
+		target,
+		summary: stringFrom(rec.summary) || `${findings.length} finding${findings.length === 1 ? "" : "s"}`,
+		findings,
+		overallCorrectness,
+		overallExplanation: stringFrom(rec.overall_explanation ?? rec.overallExplanation),
+		overallConfidenceScore: numberFrom(rec.overall_confidence_score ?? rec.overallConfidenceScore),
+		generatedAt: Date.now(),
+		source: "model"
+	};
+}
+function buildReviewPrompt(params) {
+	return [
+		"You are a senior code reviewer. Review the provided git diff for correctness bugs only.",
+		"Prioritize concrete regressions, broken behavior, missing required tests for changed behavior, data loss, security issues, and runtime/type errors.",
+		"Do not mention style, broad refactors, praise, or speculative improvements.",
+		"Return strict JSON only with this shape:",
+		"{\"findings\":[{\"title\":\"...\",\"body\":\"...\",\"priority\":1,\"confidence_score\":0.9,\"code_location\":{\"file_path\":\"path\",\"line_range\":{\"start\":12,\"end\":12}}}],\"overall_correctness\":\"patch is correct\",\"overall_explanation\":\"...\",\"overall_confidence_score\":0.8,\"summary\":\"...\"}",
+		"Use priority 0 for release-blocking, 1 for high, 2 for medium, 3 for low.",
+		params.instructions ? `Extra reviewer instructions: ${params.instructions}` : "",
+		`Target: ${params.target}`,
+		params.truncated ? "The diff was truncated; only report findings supported by visible lines." : "",
+		"Git status:",
+		params.status || "(clean)",
+		"Diff stat:",
+		params.stat || "(none)",
+		"Diff:",
+		params.diff || "(none)"
+	].filter(Boolean).join("\n\n");
+}
+async function resolveWorkspace(ctx) {
+	const sessionConfigStore = ctx.getSessionConfigStore?.();
+	const sessionConfig = sessionConfigStore ? await sessionConfigStore.get(ctx.sessionKey).catch(() => null) : null;
+	return effectiveWorkspacePathForSession(ctx.config, ctx.sessionKey, sessionConfig, getProjectForSession(ctx.sessionKey));
+}
+async function buildReview(ctx, args) {
+	const cwd = await resolveGitRoot(await resolveWorkspace(ctx));
+	const reviewTarget = parseReviewTargetArgs(args);
+	const bundle = await buildReviewDiffBundle(cwd, reviewTarget);
+	const { text: diff, truncated } = truncateForPrompt(bundle.diff);
+	const target = bundle.targetLabel;
+	const instructions = reviewTarget.instructions ?? "";
+	if (!bundle.status.trim() && !bundle.diff.trim()) return {
+		type: "review",
+		target,
+		summary: "Working tree is clean.",
+		findings: [],
+		overallCorrectness: "patch is correct",
+		overallExplanation: "No uncommitted changes were found to review.",
+		generatedAt: Date.now(),
+		source: "local"
+	};
+	const prompt = buildReviewPrompt({
+		target,
+		status: bundle.status,
+		stat: bundle.stat,
+		diff,
+		truncated,
+		instructions
+	});
+	const answer = await ctx.btwQuery?.(prompt);
+	if (answer?.text && !answer.error) try {
+		return parseReviewJson(answer.text, target);
+	} catch {}
+	return {
+		type: "review",
+		target,
+		summary: "Review model was unavailable; returned a local diff summary.",
+		findings: [],
+		overallCorrectness: "unknown",
+		overallExplanation: [answer?.error ? `Reviewer model error: ${answer.error}` : "Reviewer model output could not be parsed.", bundle.stat.trim() ? `Changed files:\n${bundle.stat.trim()}` : "No textual diff stat was available."].join("\n\n"),
+		generatedAt: Date.now(),
+		source: "local"
+	};
+}
+const reviewCommand = {
+	id: "coding.review",
+	name: "review",
+	description: "Review current workspace changes for correctness issues",
+	category: "tool",
+	scope: [
+		"global",
+		"private",
+		"group"
+	],
+	acceptsArgs: true,
+	examples: ["/review", "/review focus on auth and persistence"],
+	handler: async (ctx, args) => {
+		await ctx.setTyping(true);
+		const review = await buildReview(ctx, args);
+		return {
+			content: formatReviewMarkdown(review),
+			success: true,
+			metadata: { review }
+		};
+	}
+};
+function registerReviewCommand() {
+	commandRegistry.register(reviewCommand);
+}
+//#endregion
+export { registerReviewCommand };
+
+//# sourceMappingURL=review.js.map

package/dist/src/chat-commands/builtins/review.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"review.js","names":[],"sources":["../../../../src/chat-commands/builtins/review.ts"],"sourcesContent":["import type { CommandDefinition, CommandContext } from '../types.js';\nimport { commandRegistry } from '../registry.js';\nimport { effectiveWorkspacePathForSession } from '../../session/session-workspace.js';\nimport { getProjectForSession } from '../../projects/workspace.js';\nimport {\n  buildReviewDiffBundle,\n  parseReviewTargetArgs,\n  resolveGitRoot,\n} from '../../review/review-git.js';\nimport {\n  formatReviewMarkdown,\n  type ReviewFinding,\n  type ReviewOutput,\n  type ReviewPriority,\n} from '../../review/review-types.js';\n\nconst MAX_DIFF_CHARS = 60_000;\n\nfunction truncateForPrompt(text: string): { text: string; truncated: boolean } {\n  if (text.length <= MAX_DIFF_CHARS) return { text, truncated: false };\n  return {\n    text: text.slice(0, MAX_DIFF_CHARS) + '\\n\\n[diff truncated]',\n    truncated: true,\n  };\n}\n\nfunction extractJsonObject(raw: string): unknown {\n  const fenced = raw.match(/```(?:json)?\\s*([\\s\\S]*?)```/i);\n  const text = (fenced?.[1] ?? raw).trim();\n  const start = text.indexOf('{');\n  const end = text.lastIndexOf('}');\n  if (start < 0 || end <= start) throw new Error('review response did not contain JSON');\n  return JSON.parse(text.slice(start, end + 1));\n}\n\nfunction priorityFrom(raw: unknown): ReviewPriority {\n  const n = typeof raw === 'number' ? raw : Number.parseInt(String(raw ?? ''), 10);\n  if (n === 0 || n === 1 || n === 2 || n === 3) return n;\n  return 2;\n}\n\nfunction numberFrom(raw: unknown): number | undefined {\n  return typeof raw === 'number' && Number.isFinite(raw) ? raw : undefined;\n}\n\nfunction stringFrom(raw: unknown): string {\n  return typeof raw === 'string' ? raw.trim() : '';\n}\n\nfunction parseReviewJson(raw: string, target: string): ReviewOutput {\n  const parsed = extractJsonObject(raw);\n  const rec = parsed && typeof parsed === 'object' && !Array.isArray(parsed)\n    ? parsed as Record<string, unknown>\n    : {};\n  const rawFindings = Array.isArray(rec.findings) ? rec.findings : [];\n  const findings: ReviewFinding[] = rawFindings\n    .map((item) => {\n      const f = item && typeof item === 'object' && !Array.isArray(item)\n        ? item as Record<string, unknown>\n        : {};\n      const loc = f.code_location && typeof f.code_location === 'object'\n        ? f.code_location as Record<string, unknown>\n        : {};\n      const range = loc.line_range && typeof loc.line_range === 'object'\n        ? loc.line_range as Record<string, unknown>\n        : {};\n      const title = stringFrom(f.title);\n      const body = stringFrom(f.body);\n      if (!title && !body) return undefined;\n      const finding: ReviewFinding = {\n        title: title || body.slice(0, 80),\n        body,\n        priority: priorityFrom(f.priority),\n      };\n      const confidenceScore = numberFrom(f.confidence_score ?? f.confidenceScore);\n      const filePath = stringFrom(loc.file_path ?? f.filePath);\n      const lineStart = numberFrom(range.start ?? f.lineStart);\n      const lineEnd = numberFrom(range.end ?? f.lineEnd);\n      if (confidenceScore !== undefined) finding.confidenceScore = confidenceScore;\n      if (filePath) finding.filePath = filePath;\n      if (lineStart !== undefined) finding.lineStart = lineStart;\n      if (lineEnd !== undefined) finding.lineEnd = lineEnd;\n      return finding;\n    })\n    .filter((item): item is ReviewFinding => Boolean(item));\n\n  const correctness = stringFrom(rec.overall_correctness ?? rec.overallCorrectness);\n  const overallCorrectness =\n    correctness === 'patch is correct' || correctness === 'patch is incorrect'\n      ? correctness\n      : 'unknown';\n  return {\n    type: 'review',\n    target,\n    summary: stringFrom(rec.summary) || `${findings.length} finding${findings.length === 1 ? '' : 's'}`,\n    findings,\n    overallCorrectness,\n    overallExplanation: stringFrom(rec.overall_explanation ?? rec.overallExplanation),\n    overallConfidenceScore: numberFrom(rec.overall_confidence_score ?? rec.overallConfidenceScore),\n    generatedAt: Date.now(),\n    source: 'model',\n  };\n}\n\nfunction buildReviewPrompt(params: {\n  target: string;\n  status: string;\n  stat: string;\n  diff: string;\n  truncated: boolean;\n  instructions: string;\n}): string {\n  return [\n    'You are a senior code reviewer. Review the provided git diff for correctness bugs only.',\n    'Prioritize concrete regressions, broken behavior, missing required tests for changed behavior, data loss, security issues, and runtime/type errors.',\n    'Do not mention style, broad refactors, praise, or speculative improvements.',\n    'Return strict JSON only with this shape:',\n    '{\"findings\":[{\"title\":\"...\",\"body\":\"...\",\"priority\":1,\"confidence_score\":0.9,\"code_location\":{\"file_path\":\"path\",\"line_range\":{\"start\":12,\"end\":12}}}],\"overall_correctness\":\"patch is correct\",\"overall_explanation\":\"...\",\"overall_confidence_score\":0.8,\"summary\":\"...\"}',\n    'Use priority 0 for release-blocking, 1 for high, 2 for medium, 3 for low.',\n    params.instructions ? `Extra reviewer instructions: ${params.instructions}` : '',\n    `Target: ${params.target}`,\n    params.truncated ? 'The diff was truncated; only report findings supported by visible lines.' : '',\n    'Git status:',\n    params.status || '(clean)',\n    'Diff stat:',\n    params.stat || '(none)',\n    'Diff:',\n    params.diff || '(none)',\n  ].filter(Boolean).join('\\n\\n');\n}\n\nasync function resolveWorkspace(ctx: CommandContext): Promise<string> {\n  const sessionConfigStore = ctx.getSessionConfigStore?.();\n  const sessionConfig = sessionConfigStore\n    ? await sessionConfigStore.get(ctx.sessionKey).catch(() => null)\n    : null;\n  return effectiveWorkspacePathForSession(\n    ctx.config,\n    ctx.sessionKey,\n    sessionConfig,\n    getProjectForSession(ctx.sessionKey),\n  );\n}\n\nasync function buildReview(ctx: CommandContext, args: string): Promise<ReviewOutput> {\n  const workspace = await resolveWorkspace(ctx);\n  const cwd = await resolveGitRoot(workspace);\n  const reviewTarget = parseReviewTargetArgs(args);\n  const bundle = await buildReviewDiffBundle(cwd, reviewTarget);\n  const { text: diff, truncated } = truncateForPrompt(bundle.diff);\n  const target = bundle.targetLabel;\n  const instructions = reviewTarget.instructions ?? '';\n\n  if (!bundle.status.trim() && !bundle.diff.trim()) {\n    return {\n      type: 'review',\n      target,\n      summary: 'Working tree is clean.',\n      findings: [],\n      overallCorrectness: 'patch is correct',\n      overallExplanation: 'No uncommitted changes were found to review.',\n      generatedAt: Date.now(),\n      source: 'local',\n    };\n  }\n\n  const prompt = buildReviewPrompt({\n    target,\n    status: bundle.status,\n    stat: bundle.stat,\n    diff,\n    truncated,\n    instructions,\n  });\n  const answer = await ctx.btwQuery?.(prompt);\n  if (answer?.text && !answer.error) {\n    try {\n      return parseReviewJson(answer.text, target);\n    } catch {\n      // Fall through to a deterministic local summary when model output is not parseable.\n    }\n  }\n\n  return {\n    type: 'review',\n    target,\n    summary: 'Review model was unavailable; returned a local diff summary.',\n    findings: [],\n    overallCorrectness: 'unknown',\n    overallExplanation: [\n      answer?.error ? `Reviewer model error: ${answer.error}` : 'Reviewer model output could not be parsed.',\n      bundle.stat.trim() ? `Changed files:\\n${bundle.stat.trim()}` : 'No textual diff stat was available.',\n    ].join('\\n\\n'),\n    generatedAt: Date.now(),\n    source: 'local',\n  };\n}\n\nconst reviewCommand: CommandDefinition = {\n  id: 'coding.review',\n  name: 'review',\n  description: 'Review current workspace changes for correctness issues',\n  category: 'tool',\n  scope: ['global', 'private', 'group'],\n  acceptsArgs: true,\n  examples: ['/review', '/review focus on auth and persistence'],\n  handler: async (ctx: CommandContext, args: string) => {\n    await ctx.setTyping(true);\n    const review = await buildReview(ctx, args);\n    return {\n      content: formatReviewMarkdown(review),\n      success: true,\n      metadata: { review },\n    };\n  },\n};\n\nexport function registerReviewCommand(): void {\n  commandRegistry.register(reviewCommand);\n}\n"],"mappings":";;;;;;AAgBA,MAAM,iBAAiB;AAEvB,SAAS,kBAAkB,MAAoD;AAC7E,KAAI,KAAK,UAAU,eAAgB,QAAO;EAAE;EAAM,WAAW;EAAO;AACpE,QAAO;EACL,MAAM,KAAK,MAAM,GAAG,eAAe,GAAG;EACtC,WAAW;EACZ;;AAGH,SAAS,kBAAkB,KAAsB;CAE/C,MAAM,QADS,IAAI,MAAM,gCACL,GAAG,MAAM,KAAK,MAAM;CACxC,MAAM,QAAQ,KAAK,QAAQ,IAAI;CAC/B,MAAM,MAAM,KAAK,YAAY,IAAI;AACjC,KAAI,QAAQ,KAAK,OAAO,MAAO,OAAM,IAAI,MAAM,uCAAuC;AACtF,QAAO,KAAK,MAAM,KAAK,MAAM,OAAO,MAAM,EAAE,CAAC;;AAG/C,SAAS,aAAa,KAA8B;CAClD,MAAM,IAAI,OAAO,QAAQ,WAAW,MAAM,OAAO,SAAS,OAAO,OAAO,GAAG,EAAE,GAAG;AAChF,KAAI,MAAM,KAAK,MAAM,KAAK,MAAM,KAAK,MAAM,EAAG,QAAO;AACrD,QAAO;;AAGT,SAAS,WAAW,KAAkC;AACpD,QAAO,OAAO,QAAQ,YAAY,OAAO,SAAS,IAAI,GAAG,MAAM,KAAA;;AAGjE,SAAS,WAAW,KAAsB;AACxC,QAAO,OAAO,QAAQ,WAAW,IAAI,MAAM,GAAG;;AAGhD,SAAS,gBAAgB,KAAa,QAA8B;CAClE,MAAM,SAAS,kBAAkB,IAAI;CACrC,MAAM,MAAM,UAAU,OAAO,WAAW,YAAY,CAAC,MAAM,QAAQ,OAAO,GACtE,SACA,EAAE;CAEN,MAAM,YADc,MAAM,QAAQ,IAAI,SAAS,GAAG,IAAI,WAAW,EAAE,EAEhE,KAAK,SAAS;EACb,MAAM,IAAI,QAAQ,OAAO,SAAS,YAAY,CAAC,MAAM,QAAQ,KAAK,GAC9D,OACA,EAAE;EACN,MAAM,MAAM,EAAE,iBAAiB,OAAO,EAAE,kBAAkB,WACtD,EAAE,gBACF,EAAE;EACN,MAAM,QAAQ,IAAI,cAAc,OAAO,IAAI,eAAe,WACtD,IAAI,aACJ,EAAE;EACN,MAAM,QAAQ,WAAW,EAAE,MAAM;EACjC,MAAM,OAAO,WAAW,EAAE,KAAK;AAC/B,MAAI,CAAC,SAAS,CAAC,KAAM,QAAO,KAAA;EAC5B,MAAM,UAAyB;GAC7B,OAAO,SAAS,KAAK,MAAM,GAAG,GAAG;GACjC;GACA,UAAU,aAAa,EAAE,SAAS;GACnC;EACD,MAAM,kBAAkB,WAAW,EAAE,oBAAoB,EAAE,gBAAgB;EAC3E,MAAM,WAAW,WAAW,IAAI,aAAa,EAAE,SAAS;EACxD,MAAM,YAAY,WAAW,MAAM,SAAS,EAAE,UAAU;EACxD,MAAM,UAAU,WAAW,MAAM,OAAO,EAAE,QAAQ;AAClD,MAAI,oBAAoB,KAAA,EAAW,SAAQ,kBAAkB;AAC7D,MAAI,SAAU,SAAQ,WAAW;AACjC,MAAI,cAAc,KAAA,EAAW,SAAQ,YAAY;AACjD,MAAI,YAAY,KAAA,EAAW,SAAQ,UAAU;AAC7C,SAAO;GACP,CACD,QAAQ,SAAgC,QAAQ,KAAK,CAAC;CAEzD,MAAM,cAAc,WAAW,IAAI,uBAAuB,IAAI,mBAAmB;CACjF,MAAM,qBACJ,gBAAgB,sBAAsB,gBAAgB,uBAClD,cACA;AACN,QAAO;EACL,MAAM;EACN;EACA,SAAS,WAAW,IAAI,QAAQ,IAAI,GAAG,SAAS,OAAO,UAAU,SAAS,WAAW,IAAI,KAAK;EAC9F;EACA;EACA,oBAAoB,WAAW,IAAI,uBAAuB,IAAI,mBAAmB;EACjF,wBAAwB,WAAW,IAAI,4BAA4B,IAAI,uBAAuB;EAC9F,aAAa,KAAK,KAAK;EACvB,QAAQ;EACT;;AAGH,SAAS,kBAAkB,QAOhB;AACT,QAAO;EACL;EACA;EACA;EACA;EACA;EACA;EACA,OAAO,eAAe,gCAAgC,OAAO,iBAAiB;EAC9E,WAAW,OAAO;EAClB,OAAO,YAAY,6EAA6E;EAChG;EACA,OAAO,UAAU;EACjB;EACA,OAAO,QAAQ;EACf;EACA,OAAO,QAAQ;EAChB,CAAC,OAAO,QAAQ,CAAC,KAAK,OAAO;;AAGhC,eAAe,iBAAiB,KAAsC;CACpE,MAAM,qBAAqB,IAAI,yBAAyB;CACxD,MAAM,gBAAgB,qBAClB,MAAM,mBAAmB,IAAI,IAAI,WAAW,CAAC,YAAY,KAAK,GAC9D;AACJ,QAAO,iCACL,IAAI,QACJ,IAAI,YACJ,eACA,qBAAqB,IAAI,WAAW,CACrC;;AAGH,eAAe,YAAY,KAAqB,MAAqC;CAEnF,MAAM,MAAM,MAAM,eAAe,MADT,iBAAiB,IAAI,CACF;CAC3C,MAAM,eAAe,sBAAsB,KAAK;CAChD,MAAM,SAAS,MAAM,sBAAsB,KAAK,aAAa;CAC7D,MAAM,EAAE,MAAM,MAAM,cAAc,kBAAkB,OAAO,KAAK;CAChE,MAAM,SAAS,OAAO;CACtB,MAAM,eAAe,aAAa,gBAAgB;AAElD,KAAI,CAAC,OAAO,OAAO,MAAM,IAAI,CAAC,OAAO,KAAK,MAAM,CAC9C,QAAO;EACL,MAAM;EACN;EACA,SAAS;EACT,UAAU,EAAE;EACZ,oBAAoB;EACpB,oBAAoB;EACpB,aAAa,KAAK,KAAK;EACvB,QAAQ;EACT;CAGH,MAAM,SAAS,kBAAkB;EAC/B;EACA,QAAQ,OAAO;EACf,MAAM,OAAO;EACb;EACA;EACA;EACD,CAAC;CACF,MAAM,SAAS,MAAM,IAAI,WAAW,OAAO;AAC3C,KAAI,QAAQ,QAAQ,CAAC,OAAO,MAC1B,KAAI;AACF,SAAO,gBAAgB,OAAO,MAAM,OAAO;SACrC;AAKV,QAAO;EACL,MAAM;EACN;EACA,SAAS;EACT,UAAU,EAAE;EACZ,oBAAoB;EACpB,oBAAoB,CAClB,QAAQ,QAAQ,yBAAyB,OAAO,UAAU,8CAC1D,OAAO,KAAK,MAAM,GAAG,mBAAmB,OAAO,KAAK,MAAM,KAAK,sCAChE,CAAC,KAAK,OAAO;EACd,aAAa,KAAK,KAAK;EACvB,QAAQ;EACT;;AAGH,MAAM,gBAAmC;CACvC,IAAI;CACJ,MAAM;CACN,aAAa;CACb,UAAU;CACV,OAAO;EAAC;EAAU;EAAW;EAAQ;CACrC,aAAa;CACb,UAAU,CAAC,WAAW,wCAAwC;CAC9D,SAAS,OAAO,KAAqB,SAAiB;AACpD,QAAM,IAAI,UAAU,KAAK;EACzB,MAAM,SAAS,MAAM,YAAY,KAAK,KAAK;AAC3C,SAAO;GACL,SAAS,qBAAqB,OAAO;GACrC,SAAS;GACT,UAAU,EAAE,QAAQ;GACrB;;CAEJ;AAED,SAAgB,wBAA8B;AAC5C,iBAAgB,SAAS,cAAc"}

package/dist/src/chat-commands/index.d.ts

@@ -20,6 +20,7 @@ export { registerContextCommands } from './builtins/context.js';
 export { registerGoalCommand } from './builtins/goal.js';
 export { registerSubgoalCommand } from './builtins/subgoal.js';
 export { registerWorkflowCommands } from './builtins/workflow.js';
+export { registerReviewCommand } from './builtins/review.js';
 export { registerAgentEditCommand } from './agent-edit.js';
 /**
  * Initialize the command system with all built-in commands

package/dist/src/chat-commands/index.js

@@ -11,6 +11,7 @@ import { registerContextCommands } from "./builtins/context.js";
 import { registerGoalCommand } from "./builtins/goal.js";
 import { registerSubgoalCommand } from "./builtins/subgoal.js";
 import { registerWorkflowCommands } from "./builtins/workflow.js";
+import { registerReviewCommand } from "./builtins/review.js";
 import { registerAgentEditCommand } from "./agent-edit.js";
 import { CommandContextImpl, createCommandContext } from "./context.js";
 //#region src/chat-commands/index.ts
@@ -34,9 +35,10 @@ function initializeCommands() {
 	registerGoalCommand();
 	registerSubgoalCommand();
 	registerWorkflowCommands();
+	registerReviewCommand();
 	registerAgentEditCommand();
 }
 //#endregion
-export { CommandContextImpl, CommandRegistry, commandRegistry, createCommandContext, generateSessionKey, getSessionDisplayName, initializeCommands, normalizeTelegramCommandName, parseSlashCommand, registerAgentEditCommand, registerConfigCommand, registerContextCommands, registerGoalCommand, registerModelCommands, registerSessionCommands, registerSubgoalCommand, registerSystemCommands, registerTTSCommands, registerThinkingCommands, registerWorkflowCommands };
+export { CommandContextImpl, CommandRegistry, commandRegistry, createCommandContext, generateSessionKey, getSessionDisplayName, initializeCommands, normalizeTelegramCommandName, parseSlashCommand, registerAgentEditCommand, registerConfigCommand, registerContextCommands, registerGoalCommand, registerModelCommands, registerReviewCommand, registerSessionCommands, registerSubgoalCommand, registerSystemCommands, registerTTSCommands, registerThinkingCommands, registerWorkflowCommands };
 
 //# sourceMappingURL=index.js.map

package/dist/src/chat-commands/index.js.map

@@ -1 +1 @@
-{"version":3,"file":"index.js","names":[],"sources":["../../../src/chat-commands/index.ts"],"sourcesContent":["/**\n * Unified Command System\n *\n * Provides a platform-agnostic command system that works across\n * Telegram, Feishu, Web UI, CLI, and other channel surfaces.\n */\n\n// Built-in Commands (import first to avoid circular deps)\nimport { registerSessionCommands } from './builtins/session.js';\nimport { registerModelCommands } from './builtins/model.js';\nimport { registerSystemCommands } from './builtins/system.js';\nimport { registerTTSCommands } from './builtins/tts.js';\nimport { registerThinkingCommands } from './builtins/thinking.js';\nimport { registerConfigCommand } from './builtins/config.js';\nimport { registerContextCommands } from './builtins/context.js';\nimport { registerGoalCommand } from './builtins/goal.js';\nimport { registerSubgoalCommand } from './builtins/subgoal.js';\nimport { registerWorkflowCommands } from './builtins/workflow.js';\nimport { registerAgentEditCommand } from './agent-edit.js';\n\n// Types\nexport type {\n  MessageSource,\n  UnifiedMessage,\n  PlatformMetadata,\n  MessageAttachment,\n  CommandDefinition,\n  CommandCategory,\n  CommandScope,\n  CommandHandler,\n  CommandResult,\n  CommandContext,\n  ReplyOptions,\n  UIComponent,\n  ButtonGroup,\n  SelectMenu,\n  ModelPicker,\n  UsageDisplay,\n  SessionList,\n  TextInput,\n  ProviderInfo,\n  ModelInfo,\n  UsageStats,\n  SessionInfo,\n  PlatformFeature,\n  ChannelAdapter,\n  ReplyPayload,\n} from './types.js';\n\n// Session display helpers\nexport { generateSessionKey, getSessionDisplayName, type SessionKeyContext } from './session-key.js';\n\n// Command parsing helpers\nexport { normalizeTelegramCommandName, parseSlashCommand } from './command-parse.js';\n\n// Registry\nexport { CommandRegistry, commandRegistry } from './registry.js';\nexport type { CommandRegistry as CommandRegistryType } from './types.js';\n\n// Context\nexport { CommandContextImpl, createCommandContext } from './context.js';\n\n// Built-in Commands\nexport { registerSessionCommands } from './builtins/session.js';\nexport { registerModelCommands } from './builtins/model.js';\nexport { registerSystemCommands } from './builtins/system.js';\nexport { registerTTSCommands } from './builtins/tts.js';\nexport { registerThinkingCommands } from './builtins/thinking.js';\nexport { registerConfigCommand } from './builtins/config.js';\nexport { registerContextCommands } from './builtins/context.js';\nexport { registerGoalCommand } from './builtins/goal.js';\nexport { registerSubgoalCommand } from './builtins/subgoal.js';\nexport { registerWorkflowCommands } from './builtins/workflow.js';\nexport { registerAgentEditCommand } from './agent-edit.js';\n\n/**\n * Initialize the command system with all built-in commands\n */\nexport function initializeCommands(): void {\n  registerSessionCommands();\n  registerModelCommands();\n  registerSystemCommands();\n  registerConfigCommand();\n  registerContextCommands();\n  registerTTSCommands();\n  registerThinkingCommands();\n  registerGoalCommand();\n  registerSubgoalCommand();\n  registerWorkflowCommands();\n  registerAgentEditCommand();\n}\n"],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;AA8EA,SAAgB,qBAA2B;AACzC,0BAAyB;AACzB,wBAAuB;AACvB,yBAAwB;AACxB,wBAAuB;AACvB,0BAAyB;AACzB,sBAAqB;AACrB,2BAA0B;AAC1B,sBAAqB;AACrB,yBAAwB;AACxB,2BAA0B;AAC1B,2BAA0B"}
+{"version":3,"file":"index.js","names":[],"sources":["../../../src/chat-commands/index.ts"],"sourcesContent":["/**\n * Unified Command System\n *\n * Provides a platform-agnostic command system that works across\n * Telegram, Feishu, Web UI, CLI, and other channel surfaces.\n */\n\n// Built-in Commands (import first to avoid circular deps)\nimport { registerSessionCommands } from './builtins/session.js';\nimport { registerModelCommands } from './builtins/model.js';\nimport { registerSystemCommands } from './builtins/system.js';\nimport { registerTTSCommands } from './builtins/tts.js';\nimport { registerThinkingCommands } from './builtins/thinking.js';\nimport { registerConfigCommand } from './builtins/config.js';\nimport { registerContextCommands } from './builtins/context.js';\nimport { registerGoalCommand } from './builtins/goal.js';\nimport { registerSubgoalCommand } from './builtins/subgoal.js';\nimport { registerWorkflowCommands } from './builtins/workflow.js';\nimport { registerReviewCommand } from './builtins/review.js';\nimport { registerAgentEditCommand } from './agent-edit.js';\n\n// Types\nexport type {\n  MessageSource,\n  UnifiedMessage,\n  PlatformMetadata,\n  MessageAttachment,\n  CommandDefinition,\n  CommandCategory,\n  CommandScope,\n  CommandHandler,\n  CommandResult,\n  CommandContext,\n  ReplyOptions,\n  UIComponent,\n  ButtonGroup,\n  SelectMenu,\n  ModelPicker,\n  UsageDisplay,\n  SessionList,\n  TextInput,\n  ProviderInfo,\n  ModelInfo,\n  UsageStats,\n  SessionInfo,\n  PlatformFeature,\n  ChannelAdapter,\n  ReplyPayload,\n} from './types.js';\n\n// Session display helpers\nexport { generateSessionKey, getSessionDisplayName, type SessionKeyContext } from './session-key.js';\n\n// Command parsing helpers\nexport { normalizeTelegramCommandName, parseSlashCommand } from './command-parse.js';\n\n// Registry\nexport { CommandRegistry, commandRegistry } from './registry.js';\nexport type { CommandRegistry as CommandRegistryType } from './types.js';\n\n// Context\nexport { CommandContextImpl, createCommandContext } from './context.js';\n\n// Built-in Commands\nexport { registerSessionCommands } from './builtins/session.js';\nexport { registerModelCommands } from './builtins/model.js';\nexport { registerSystemCommands } from './builtins/system.js';\nexport { registerTTSCommands } from './builtins/tts.js';\nexport { registerThinkingCommands } from './builtins/thinking.js';\nexport { registerConfigCommand } from './builtins/config.js';\nexport { registerContextCommands } from './builtins/context.js';\nexport { registerGoalCommand } from './builtins/goal.js';\nexport { registerSubgoalCommand } from './builtins/subgoal.js';\nexport { registerWorkflowCommands } from './builtins/workflow.js';\nexport { registerReviewCommand } from './builtins/review.js';\nexport { registerAgentEditCommand } from './agent-edit.js';\n\n/**\n * Initialize the command system with all built-in commands\n */\nexport function initializeCommands(): void {\n  registerSessionCommands();\n  registerModelCommands();\n  registerSystemCommands();\n  registerConfigCommand();\n  registerContextCommands();\n  registerTTSCommands();\n  registerThinkingCommands();\n  registerGoalCommand();\n  registerSubgoalCommand();\n  registerWorkflowCommands();\n  registerReviewCommand();\n  registerAgentEditCommand();\n}\n"],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;AAgFA,SAAgB,qBAA2B;AACzC,0BAAyB;AACzB,wBAAuB;AACvB,yBAAwB;AACxB,wBAAuB;AACvB,0BAAyB;AACzB,sBAAqB;AACrB,2BAA0B;AAC1B,sBAAqB;AACrB,yBAAwB;AACxB,2BAA0B;AAC1B,wBAAuB;AACvB,2BAA0B"}

package/dist/src/cli/commands/doctor/checks/database-schema.js

@@ -52,7 +52,7 @@ async function checkDatabaseSchema(ctx) {
 		id: "database-schema",
 		label: "Database schema",
 		status: "pass",
-		message: `SQLite schema v${status.dbVersion} (current release v20).`,
+		message: `SQLite schema v${status.dbVersion} (current release v21).`,
 		hints: []
 	};
 }

package/dist/src/cli/commands/gateway/lifecycle.js

@@ -1,8 +1,8 @@
 import { init_paths, resolveConfigPath } from "../../../config/paths.js";
 import { loadConfig } from "../../../config/loader.js";
 import "../../../config/index.js";
-import { isRestartEnabled } from "../../../config/commands.flags.js";
 import { resolveGatewayService } from "../../../daemon/service.js";
+import { isRestartEnabled } from "../../../config/commands.flags.js";
 import { authorizeGatewaySigusr1Restart, writeGatewayRestartIntentSync } from "../../../infra/restart.js";
 import { findVerifiedGatewayListenerPidsOnPortSync, formatGatewayPidList, signalVerifiedGatewayPidSync } from "../../../infra/gateway-processes.js";
 import { runServiceRestart, runServiceStop } from "./lifecycle-core.js";

package/dist/src/gateway/agent-builtin-tools.d.ts

@@ -2,5 +2,5 @@
  * Built-in agent tool `name` values for gateway admin UI (disable toggles).
  * Extension tools are not listed; users can still add arbitrary strings in config.
  */
-export declare const GATEWAY_BUILTIN_TOOL_IDS: readonly ["read_file", "write_file", "edit_file", "list_dir", "grep", "find", "shell", "session_status", "dreaming", "tool_manual", "clarify", "todo", "skills_list", "skill_view", "skill_manage", "web_search", "web_fetch", "web_extract", "send_message", "send_media", "read_media", "create_share", "text_to_speech", "memory_search", "memory_get", "curated_memory", "session_search", "automation", "workflow", "delegate_task", "execute_code", "image", "image_generate", "browser_use", "extensions", "bundle-mcp"];
+export declare const GATEWAY_BUILTIN_TOOL_IDS: readonly ["read_file", "write_file", "apply_patch", "list_dir", "grep", "find", "exec_command", "update_plan", "session_status", "dreaming", "tool_manual", "clarify", "todo", "skills_list", "skill_view", "skill_manage", "web_search", "web_fetch", "web_extract", "send_message", "send_media", "read_media", "create_share", "text_to_speech", "memory_search", "memory_get", "curated_memory", "session_search", "automation", "workflow", "delegate_task", "execute_code", "image", "image_generate", "browser_use", "extensions", "bundle-mcp"];
 export type GatewayBuiltinToolId = (typeof GATEWAY_BUILTIN_TOOL_IDS)[number];

package/dist/src/gateway/agent-builtin-tools.js

@@ -6,11 +6,12 @@
 const GATEWAY_BUILTIN_TOOL_IDS = [
 	"read_file",
 	"write_file",
-	"edit_file",
+	"apply_patch",
 	"list_dir",
 	"grep",
 	"find",
-	"shell",
+	"exec_command",
+	"update_plan",
 	"session_status",
 	"dreaming",
 	"tool_manual",

package/dist/src/gateway/agent-builtin-tools.js.map

@@ -1 +1 @@
-{"version":3,"file":"agent-builtin-tools.js","names":[],"sources":["../../../src/gateway/agent-builtin-tools.ts"],"sourcesContent":["/**\n * Built-in agent tool `name` values for gateway admin UI (disable toggles).\n * Extension tools are not listed; users can still add arbitrary strings in config.\n */\nexport const GATEWAY_BUILTIN_TOOL_IDS = [\n  'read_file',\n  'write_file',\n  'edit_file',\n  'list_dir',\n  'grep',\n  'find',\n  'shell',\n  'session_status',\n  'dreaming',\n  'tool_manual',\n  'clarify',\n  'todo',\n  'skills_list',\n  'skill_view',\n  'skill_manage',\n  'web_search',\n  'web_fetch',\n  'web_extract',\n  'send_message',\n  'send_media',\n  'read_media',\n  'create_share',\n  'text_to_speech',\n  'memory_search',\n  'memory_get',\n  'curated_memory',\n  'session_search',\n  'automation',\n  'workflow',\n  'delegate_task',\n  'execute_code',\n  'image',\n  'image_generate',\n  'browser_use',\n  'extensions',\n  'bundle-mcp',\n] as const;\n\nexport type GatewayBuiltinToolId = (typeof GATEWAY_BUILTIN_TOOL_IDS)[number];\n"],"mappings":";;;;;AAIA,MAAa,2BAA2B;CACtC;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACD"}
+{"version":3,"file":"agent-builtin-tools.js","names":[],"sources":["../../../src/gateway/agent-builtin-tools.ts"],"sourcesContent":["/**\n * Built-in agent tool `name` values for gateway admin UI (disable toggles).\n * Extension tools are not listed; users can still add arbitrary strings in config.\n */\nexport const GATEWAY_BUILTIN_TOOL_IDS = [\n  'read_file',\n  'write_file',\n  'apply_patch',\n  'list_dir',\n  'grep',\n  'find',\n  'exec_command',\n  'update_plan',\n  'session_status',\n  'dreaming',\n  'tool_manual',\n  'clarify',\n  'todo',\n  'skills_list',\n  'skill_view',\n  'skill_manage',\n  'web_search',\n  'web_fetch',\n  'web_extract',\n  'send_message',\n  'send_media',\n  'read_media',\n  'create_share',\n  'text_to_speech',\n  'memory_search',\n  'memory_get',\n  'curated_memory',\n  'session_search',\n  'automation',\n  'workflow',\n  'delegate_task',\n  'execute_code',\n  'image',\n  'image_generate',\n  'browser_use',\n  'extensions',\n  'bundle-mcp',\n] as const;\n\nexport type GatewayBuiltinToolId = (typeof GATEWAY_BUILTIN_TOOL_IDS)[number];\n"],"mappings":";;;;;AAIA,MAAa,2BAA2B;CACtC;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACD"}

package/dist/src/gateway/chat-stream/mapper.d.ts

@@ -15,7 +15,12 @@ export declare class ChatStreamMapper {
     private currentAssistantMessageId;
     private lastAssistantMessageId;
     private currentAssistantText;
+    private currentAssistantReviewEmitted;
     private toolCallToMessageId;
+    private turnDiffs;
+    private turnDiffFiles;
+    private turnDiffAdded;
+    private turnDiffRemoved;
     private started;
     private ended;
     constructor(opts: ChatStreamMapperOptions);
@@ -32,4 +37,5 @@ export declare class ChatStreamMapper {
     private ensureAssistantMessageId;
     private nextAssistantMessageId;
     private make;
+    private recordPatchDiff;
 }

package/dist/src/gateway/chat-stream/mapper.js

@@ -4,7 +4,12 @@ var ChatStreamMapper = class {
 	currentAssistantMessageId;
 	lastAssistantMessageId;
 	currentAssistantText = "";
+	currentAssistantReviewEmitted = false;
 	toolCallToMessageId = /* @__PURE__ */ new Map();
+	turnDiffs = [];
+	turnDiffFiles = /* @__PURE__ */ new Set();
+	turnDiffAdded = 0;
+	turnDiffRemoved = 0;
 	started = false;
 	ended = false;
 	constructor(opts) {
@@ -80,12 +85,21 @@ var ChatStreamMapper = class {
 		const messageId = this.nextAssistantMessageId();
 		this.currentAssistantMessageId = messageId;
 		this.currentAssistantText = "";
+		this.currentAssistantReviewEmitted = false;
 		return [this.make("assistant_message_start", { messageId })];
 	}
 	mapMessageUpdate(event) {
 		const message = event.message;
 		if (message?.role !== "assistant") return [];
 		const messageId = this.ensureAssistantMessageId();
+		const review = extractReview(message);
+		if (review && !this.currentAssistantReviewEmitted) {
+			this.currentAssistantReviewEmitted = true;
+			return [this.make("review", {
+				messageId,
+				review
+			})];
+		}
 		const delta = event.assistantMessageEvent;
 		if (delta?.type === "text_delta" && typeof delta.delta === "string" && delta.delta) {
 			const text = extractTextFromMessage(message);
@@ -118,6 +132,14 @@ var ChatStreamMapper = class {
 		if (message?.role !== "assistant") return [];
 		const messageId = this.ensureAssistantMessageId();
 		const events = [];
+		const review = extractReview(message);
+		if (review && !this.currentAssistantReviewEmitted) {
+			this.currentAssistantReviewEmitted = true;
+			events.push(this.make("review", {
+				messageId,
+				review
+			}));
+		}
 		const text = extractTextFromMessage(message);
 		const suffix = appendSuffix(this.currentAssistantText, text);
 		if (suffix) events.push(this.make("assistant_delta", {
@@ -126,6 +148,19 @@ var ChatStreamMapper = class {
 		}));
 		this.currentAssistantText = text || this.currentAssistantText;
 		events.push(this.make("thinking_end", { messageId }));
+		if (this.turnDiffs.length > 0) {
+			events.push(this.make("turn_diff", {
+				messageId,
+				files: [...this.turnDiffFiles],
+				diff: this.turnDiffs.join("\n"),
+				added: this.turnDiffAdded,
+				removed: this.turnDiffRemoved
+			}));
+			this.turnDiffs = [];
+			this.turnDiffFiles.clear();
+			this.turnDiffAdded = 0;
+			this.turnDiffRemoved = 0;
+		}
 		events.push(this.make("assistant_message_end", {
 			messageId,
 			usage: extractUsage(message)
@@ -133,19 +168,31 @@ var ChatStreamMapper = class {
 		this.lastAssistantMessageId = messageId;
 		this.currentAssistantMessageId = void 0;
 		this.currentAssistantText = "";
+		this.currentAssistantReviewEmitted = false;
 		return events;
 	}
 	mapToolStart(event) {
 		const toolCallId = String(event.toolCallId ?? "");
 		if (!toolCallId) return [];
 		const messageId = this.ensureAssistantMessageId();
+		const toolName = String(event.toolName ?? "unknown");
 		this.toolCallToMessageId.set(toolCallId, messageId);
-		return [this.make("tool_start", {
+		const events = [this.make("tool_start", {
 			messageId,
 			toolCallId,
-			toolName: String(event.toolName ?? "unknown"),
+			toolName,
 			args: event.args
 		})];
+		if (toolName === "exec_command") {
+			const args = asRecord(event.args);
+			events.push(this.make("command_started", {
+				messageId,
+				toolCallId,
+				command: String(args?.cmd ?? args?.command ?? ""),
+				cwd: typeof args?.cwd === "string" ? args.cwd : void 0
+			}));
+		}
+		return events;
 	}
 	mapToolUpdate(event) {
 		const toolCallId = String(event.toolCallId ?? "");
@@ -154,27 +201,79 @@ var ChatStreamMapper = class {
 		const partial = event.partialResult;
 		const details = extractDetails(partial);
 		const textDelta = extractText(partial);
-		return [this.make("tool_update", {
+		const toolName = String(event.toolName ?? "unknown");
+		const events = [this.make("tool_update", {
 			messageId,
 			toolCallId,
-			toolName: String(event.toolName ?? "unknown"),
+			toolName,
 			details,
 			textDelta
 		})];
+		if (toolName === "exec_command") {
+			const rec = asRecord(details);
+			if (rec?.kind === "command_output_delta") {
+				const stream = rec.stream === "stderr" ? "stderr" : "stdout";
+				const delta = typeof rec.delta === "string" ? rec.delta : "";
+				if (delta) events.push(this.make("command_output_delta", {
+					messageId,
+					toolCallId,
+					stream,
+					delta
+				}));
+			}
+		}
+		return events;
 	}
 	mapToolEnd(event) {
 		const toolCallId = String(event.toolCallId ?? "");
 		if (!toolCallId) return [];
 		const messageId = this.toolCallToMessageId.get(toolCallId) ?? this.ensureAssistantMessageId();
 		const result = normalizeToolResult(event.result);
-		return [this.make("tool_end", {
+		const toolName = String(event.toolName ?? "unknown");
+		const events = [this.make("tool_end", {
 			messageId,
 			toolCallId,
-			toolName: String(event.toolName ?? "unknown"),
+			toolName,
 			status: event.isError ? "error" : "success",
 			result,
 			errorMessage: event.isError ? result?.text : void 0
 		})];
+		const details = asRecord(result?.details);
+		if (toolName === "exec_command" && details) events.push(this.make("command_completed", {
+			messageId,
+			toolCallId,
+			command: typeof details.command === "string" ? details.command : "",
+			cwd: typeof details.cwd === "string" ? details.cwd : void 0,
+			exitCode: typeof details.exitCode === "number" ? details.exitCode : null,
+			durationMs: typeof details.durationMs === "number" ? details.durationMs : void 0,
+			timedOut: details.timedOut === true,
+			truncated: details.truncated === true
+		}));
+		if (toolName === "apply_patch" && details) {
+			const changes = Array.isArray(details.changes) ? details.changes : [];
+			const diff = typeof details.diff === "string" ? details.diff : "";
+			const added = typeof details.added === "number" ? details.added : 0;
+			const removed = typeof details.removed === "number" ? details.removed : 0;
+			if (diff || changes.length > 0) {
+				this.recordPatchDiff(details);
+				events.push(this.make("patch_applied", {
+					messageId,
+					toolCallId,
+					changes,
+					diff,
+					added,
+					removed
+				}));
+			}
+		}
+		if (toolName === "update_plan" && details) {
+			const plan = extractTurnPlan(details);
+			if (plan) events.push(this.make("turn_plan", {
+				messageId,
+				...plan
+			}));
+		}
+		return events;
 	}
 	ensureAssistantMessageId() {
 		if (!this.currentAssistantMessageId) this.currentAssistantMessageId = this.nextAssistantMessageId();
@@ -193,6 +292,18 @@ var ChatStreamMapper = class {
 			payload
 		};
 	}
+	recordPatchDiff(details) {
+		const diff = typeof details.diff === "string" ? details.diff : "";
+		if (diff) this.turnDiffs.push(diff);
+		if (typeof details.added === "number") this.turnDiffAdded += details.added;
+		if (typeof details.removed === "number") this.turnDiffRemoved += details.removed;
+		const changes = Array.isArray(details.changes) ? details.changes : [];
+		for (const change of changes) {
+			const rec = asRecord(change);
+			const path = typeof rec?.moveTo === "string" ? rec.moveTo : typeof rec?.path === "string" ? rec.path : void 0;
+			if (path) this.turnDiffFiles.add(path);
+		}
+	}
 };
 function userMessageFromLegacyEvent(event) {
 	return {
@@ -205,6 +316,18 @@ function userMessageFromLegacyEvent(event) {
 function extractTextFromMessage(message) {
 	return extractText(message.content);
 }
+function extractReview(message) {
+	const metadata = message.metadata;
+	const metadataReview = asRecord(metadata)?.review;
+	if (isReviewBlock(metadataReview)) return metadataReview;
+	const content = message.content;
+	if (Array.isArray(content)) {
+		for (const block of content) if (isReviewBlock(block)) return block;
+	}
+}
+function isReviewBlock(value) {
+	return !!value && typeof value === "object" && !Array.isArray(value) && value.type === "review";
+}
 function extractText(value) {
 	if (typeof value === "string") return value;
 	if (!Array.isArray(value)) return void 0;
@@ -238,6 +361,9 @@ function extractDetails(value) {
 	if (!value || typeof value !== "object") return void 0;
 	return value.details;
 }
+function asRecord(value) {
+	return value && typeof value === "object" && !Array.isArray(value) ? value : null;
+}
 function normalizeToolResult(value) {
 	if (value == null) return void 0;
 	if (typeof value === "string") return { text: value };
@@ -249,6 +375,23 @@ function normalizeToolResult(value) {
 		text: extractText(rec.content) ?? (typeof rec.text === "string" ? rec.text : void 0)
 	};
 }
+function extractTurnPlan(details) {
+	const plan = (Array.isArray(details.plan) ? details.plan : []).map((item) => {
+		const rec = asRecord(item);
+		const step = typeof rec?.step === "string" ? rec.step.trim() : "";
+		const status = rec?.status;
+		if (!step || status !== "pending" && status !== "in_progress" && status !== "completed") return;
+		return {
+			step,
+			status
+		};
+	}).filter((item) => Boolean(item));
+	if (plan.length === 0) return void 0;
+	return {
+		explanation: typeof details.explanation === "string" && details.explanation.trim() ? details.explanation.trim() : void 0,
+		plan
+	};
+}
 //#endregion
 export { ChatStreamMapper };