gyojeong 0.1.0 → 0.2.0

package/scripts/review-cli.mjs

@@ -985,7 +985,6 @@ var string = (params) => {
 };
 var integer = /^-?\d+$/;
 var number = /^-?\d+(?:\.\d+)?$/;
-var boolean = /^(?:true|false)$/i;
 var lowercase = /^[^A-Z]*$/;
 var uppercase = /^[^a-z]*$/;
 
@@ -1868,27 +1867,6 @@ var $ZodNumberFormat = /* @__PURE__ */ $constructor("$ZodNumberFormat", (inst, d
   $ZodCheckNumberFormat.init(inst, def);
   $ZodNumber.init(inst, def);
 });
-var $ZodBoolean = /* @__PURE__ */ $constructor("$ZodBoolean", (inst, def) => {
-  $ZodType.init(inst, def);
-  inst._zod.pattern = boolean;
-  inst._zod.parse = (payload, _ctx) => {
-    if (def.coerce)
-      try {
-        payload.value = Boolean(payload.value);
-      } catch (_) {
-      }
-    const input = payload.value;
-    if (typeof input === "boolean")
-      return payload;
-    payload.issues.push({
-      expected: "boolean",
-      code: "invalid_type",
-      input,
-      inst
-    });
-    return payload;
-  };
-});
 var $ZodUnknown = /* @__PURE__ */ $constructor("$ZodUnknown", (inst, def) => {
   $ZodType.init(inst, def);
   inst._zod.parse = (payload) => payload;
@@ -2504,28 +2482,6 @@ var $ZodEnum = /* @__PURE__ */ $constructor("$ZodEnum", (inst, def) => {
     return payload;
   };
 });
-var $ZodLiteral = /* @__PURE__ */ $constructor("$ZodLiteral", (inst, def) => {
-  $ZodType.init(inst, def);
-  if (def.values.length === 0) {
-    throw new Error("Cannot create literal schema with no valid values");
-  }
-  const values = new Set(def.values);
-  inst._zod.values = values;
-  inst._zod.pattern = new RegExp(`^(${def.values.map((o) => typeof o === "string" ? escapeRegex(o) : o ? escapeRegex(o.toString()) : String(o)).join("|")})$`);
-  inst._zod.parse = (payload, _ctx) => {
-    const input = payload.value;
-    if (values.has(input)) {
-      return payload;
-    }
-    payload.issues.push({
-      code: "invalid_value",
-      values: def.values,
-      input,
-      inst
-    });
-    return payload;
-  };
-});
 var $ZodTransform = /* @__PURE__ */ $constructor("$ZodTransform", (inst, def) => {
   $ZodType.init(inst, def);
   inst._zod.optin = "optional";
@@ -3246,13 +3202,6 @@ function _int(Class2, params) {
   });
 }
 // @__NO_SIDE_EFFECTS__
-function _boolean(Class2, params) {
-  return new Class2({
-    type: "boolean",
-    ...normalizeParams(params)
-  });
-}
-// @__NO_SIDE_EFFECTS__
 function _unknown(Class2) {
   return new Class2({
     type: "unknown"
@@ -3898,9 +3847,6 @@ var numberProcessor = (schema, ctx, _json, _params) => {
   if (typeof multipleOf === "number")
     json.multipleOf = multipleOf;
 };
-var booleanProcessor = (_schema, _ctx, json, _params) => {
-  json.type = "boolean";
-};
 var neverProcessor = (_schema, _ctx, json, _params) => {
   json.not = {};
 };
@@ -3915,46 +3861,6 @@ var enumProcessor = (schema, _ctx, json, _params) => {
     json.type = "string";
   json.enum = values;
 };
-var literalProcessor = (schema, ctx, json, _params) => {
-  const def = schema._zod.def;
-  const vals = [];
-  for (const val of def.values) {
-    if (val === void 0) {
-      if (ctx.unrepresentable === "throw") {
-        throw new Error("Literal `undefined` cannot be represented in JSON Schema");
-      } else {
-      }
-    } else if (typeof val === "bigint") {
-      if (ctx.unrepresentable === "throw") {
-        throw new Error("BigInt literals cannot be represented in JSON Schema");
-      } else {
-        vals.push(Number(val));
-      }
-    } else {
-      vals.push(val);
-    }
-  }
-  if (vals.length === 0) {
-  } else if (vals.length === 1) {
-    const val = vals[0];
-    json.type = val === null ? "null" : typeof val;
-    if (ctx.target === "draft-04" || ctx.target === "openapi-3.0") {
-      json.enum = [val];
-    } else {
-      json.const = val;
-    }
-  } else {
-    if (vals.every((v) => typeof v === "number"))
-      json.type = "number";
-    if (vals.every((v) => typeof v === "string"))
-      json.type = "string";
-    if (vals.every((v) => typeof v === "boolean"))
-      json.type = "boolean";
-    if (vals.every((v) => v === null))
-      json.type = "null";
-    json.enum = vals;
-  }
-};
 var customProcessor = (_schema, ctx, _json, _params) => {
   if (ctx.unrepresentable === "throw") {
     throw new Error("Custom types cannot be represented in JSON Schema");
@@ -4659,14 +4565,6 @@ var ZodNumberFormat = /* @__PURE__ */ $constructor("ZodNumberFormat", (inst, def
 function int(params) {
   return _int(ZodNumberFormat, params);
 }
-var ZodBoolean = /* @__PURE__ */ $constructor("ZodBoolean", (inst, def) => {
-  $ZodBoolean.init(inst, def);
-  ZodType.init(inst, def);
-  inst._zod.processJSONSchema = (ctx, json, params) => booleanProcessor(inst, ctx, json, params);
-});
-function boolean2(params) {
-  return _boolean(ZodBoolean, params);
-}
 var ZodUnknown = /* @__PURE__ */ $constructor("ZodUnknown", (inst, def) => {
   $ZodUnknown.init(inst, def);
   ZodType.init(inst, def);
@@ -4857,27 +4755,6 @@ function _enum(values, params) {
     ...util_exports.normalizeParams(params)
   });
 }
-var ZodLiteral = /* @__PURE__ */ $constructor("ZodLiteral", (inst, def) => {
-  $ZodLiteral.init(inst, def);
-  ZodType.init(inst, def);
-  inst._zod.processJSONSchema = (ctx, json, params) => literalProcessor(inst, ctx, json, params);
-  inst.values = new Set(def.values);
-  Object.defineProperty(inst, "value", {
-    get() {
-      if (def.values.length > 1) {
-        throw new Error("This schema contains multiple valid literal values. Use `.values` instead.");
-      }
-      return def.values[0];
-    }
-  });
-});
-function literal(value, params) {
-  return new ZodLiteral({
-    type: "literal",
-    values: Array.isArray(value) ? value : [value],
-    ...util_exports.normalizeParams(params)
-  });
-}
 var ZodTransform = /* @__PURE__ */ $constructor("ZodTransform", (inst, def) => {
   $ZodTransform.init(inst, def);
   ZodType.init(inst, def);
@@ -5074,7 +4951,6 @@ var BoundedCorrectionSchema = trimmedNfcBoundedText(4e3, "correction").refine((v
 });
 var IdempotencyKeySchema = string2().trim().min(1).max(128).regex(/^[A-Za-z0-9._:-]+$/);
 var IdentifierSchema = uuid2();
-var DomainSchema = string2().trim().min(1).max(64).regex(/^[a-z][a-z0-9_-]*$/);
 var IsoDateTimeSchema = iso_exports.datetime();
 var ReviewAttachmentFilenameSchema = nfcBoundedText(255, "filename").refine((value) => value.length > 0, { message: "filename must not be empty" }).refine((value) => value !== "." && value !== "..", { message: "filename must be a basename" }).refine((value) => !/[\/\\\0]/u.test(value), { message: "filename must be a basename" });
 var ReviewAttachmentMediaTypeSchema = _enum(["image/png", "image/jpeg", "image/webp"]);
@@ -5174,21 +5050,6 @@ var LanguageTagSchema = string2().transform((value, context) => {
     return NEVER;
   }
 });
-var OperationSchema = _enum(["translate", "rewrite", "post_edit", "review"]);
-var SupportLevelSchema = _enum(["verified", "experimental", "unsupported"]);
-var RouteSchema = object({
-  operation: OperationSchema,
-  source_language: LanguageTagSchema,
-  target_language: LanguageTagSchema,
-  domain: DomainSchema,
-  support_level: SupportLevelSchema
-}).strict();
-var StructuredErrorSchema = object({
-  code: string2().min(1).max(80),
-  retryable: boolean2(),
-  message: string2().min(1).max(1e3),
-  next_action: string2().min(1).max(1e3)
-}).strict();
 var MAX_SURFACE_SELECTOR_CODE_POINTS = 500;
 var MAX_SURFACE_TILES = 32;
 var MAX_SURFACE_TILE_BYTES = 2 * 1024 * 1024;
@@ -5251,20 +5112,6 @@ var SurfaceSnapshotInputSchema = object({
   tiles: array(SurfaceTileInputSchema).min(1).max(MAX_SURFACE_TILES),
   units: array(SurfaceUnitSchema).min(1).max(MAX_SURFACE_UNITS)
 }).strict();
-var SurfaceTileManifestSchema = object({
-  index: int().nonnegative(),
-  top: int().nonnegative(),
-  height: int().positive(),
-  media_type: _enum(["image/png", "image/jpeg", "image/webp"]),
-  size_bytes: int().positive()
-}).strict();
-var SurfaceSnapshotManifestSchema = object({
-  page_width: int().positive(),
-  page_height: int().positive(),
-  captured_at: IsoDateTimeSchema,
-  tiles: array(SurfaceTileManifestSchema).min(1).max(MAX_SURFACE_TILES),
-  units: array(SurfaceUnitSchema).min(1).max(MAX_SURFACE_UNITS)
-}).strict();
 var SourceSurfaceSchema = object({
   target_url: url().refine((value) => ["http:", "https:"].includes(new URL(value).protocol), {
     message: "target_url must be an http(s) URL"
@@ -5281,9 +5128,7 @@ var TranslationPublishInputSchema = object({
   run_id: IdentifierSchema,
   source_text: NfcBoundedTextSchema,
   jobs: array(TranslationPublishJobSchema).length(1),
-  attachments: array(unknown()).max(MAX_REVIEW_ATTACHMENTS).pipe(array(ReviewAttachmentInputSchema)).optional(),
-  upload_consent: literal(true),
-  idempotency_key: IdempotencyKeySchema
+  attachments: array(unknown()).max(MAX_REVIEW_ATTACHMENTS).pipe(array(ReviewAttachmentInputSchema)).optional()
 }).strict().superRefine((value, context) => {
   const filenames = /* @__PURE__ */ new Set();
   let totalSize = 0;
@@ -5919,7 +5764,7 @@ function check({ source, draft, mode = detectMode(source), standards = { terms:
 }
 
 // ../../skills/gyojeong/scripts/feedback-resolution-core.mjs
-var MAX_TEXT_CODE_POINTS2 = 2e6;
+var MAX_TEXT_CODE_POINTS2 = 2 * 1024 * 1024;
 var MAX_FEEDBACK = 1e3;
 var MAX_SELECTION_CODE_POINTS = 4e3;
 var IDENTIFIER = /^[0-9a-f]{8}-[0-9a-f]{4}-[1-8][0-9a-f]{3}-[89ab][0-9a-f]{3}-[0-9a-f]{12}$/iu;
@@ -6237,6 +6082,24 @@ import {
 } from "node:fs";
 import { hostname, homedir as homedir2 } from "node:os";
 import path2 from "node:path";
+
+// ../feedback-core/dist/index.js
+var createSelectionValidator = (text) => {
+  const normalizedText = text.normalize("NFC");
+  const textCodePoints = Array.from(normalizedText);
+  return (selection) => {
+    if (!Number.isFinite(selection.start) || !Number.isFinite(selection.end) || !Number.isInteger(selection.start) || !Number.isInteger(selection.end) || selection.start < 0 || selection.start >= selection.end || selection.end > textCodePoints.length) {
+      return { ok: false, reason: "stale_selection" };
+    }
+    const selectedSlice = textCodePoints.slice(selection.start, selection.end).join("");
+    if (selectedSlice !== selection.selected_text.normalize("NFC")) {
+      return { ok: false, reason: "stale_selection" };
+    }
+    return { ok: true, normalized_text: normalizedText };
+  };
+};
+
+// ../../scripts/review-store.mjs
 var REVIEW_ID = /^[0-9a-f]{8}-[0-9a-f]{4}-4[0-9a-f]{3}-[89ab][0-9a-f]{3}-[0-9a-f]{12}$/u;
 var LOCK_STALE_MS = 3e4;
 var LOCK_WAIT_MS = 1e4;
@@ -6366,10 +6229,9 @@ var createRun = (home, input, { mode, check: check2 } = {}) => {
       id,
       operation: mode === "polish" ? "rewrite" : "translate",
       source_language: mode === "polish" ? "ko" : "en",
-      target_languages: [job.target_language],
-      domain: "general"
+      target_languages: [job.target_language]
     },
-    job: { id: randomUUID(), target_language: job.target_language, support_level: "experimental" },
+    job: { id: randomUUID(), target_language: job.target_language },
     source_text: input.source_text,
     draft_text: job.draft_text,
     attachments,
@@ -6379,13 +6241,19 @@ var createRun = (home, input, { mode, check: check2 } = {}) => {
   };
   const staging = `${dir}.${process.pid}.tmp`;
   ensurePrivateDir(staging);
-  (input.attachments ?? []).forEach((attachment, index) => {
-    writeFileSync(path2.join(staging, `attachment-${index}.b64`), attachment.content_base64, { mode: 384 });
-  });
-  writeFileSync(path2.join(staging, "content.json"), serializeReviewContent(content), { mode: 384 });
-  writeFileSync(path2.join(staging, "feedback.json"), "[]", { mode: 384 });
-  writeFileSync(path2.join(staging, "state.json"), JSON.stringify({ processed_feedback_ids: [] }), { mode: 384 });
-  renameSync(staging, dir);
+  try {
+    (input.attachments ?? []).forEach((attachment, index) => {
+      writeFileSync(path2.join(staging, `attachment-${index}.b64`), attachment.content_base64, { mode: 384 });
+    });
+    writeFileSync(path2.join(staging, "content.json"), serializeReviewContent(content), { mode: 384 });
+    writeFileSync(path2.join(staging, "feedback.json"), "[]", { mode: 384 });
+    writeFileSync(path2.join(staging, "state.json"), JSON.stringify({ processed_feedback_ids: [] }), { mode: 384 });
+    writeFileSync(path2.join(staging, "meta.json"), JSON.stringify({ created_at: createdAt }), { mode: 384 });
+    renameSync(staging, dir);
+  } catch (error2) {
+    rmSync(staging, { recursive: true, force: true });
+    throw error2;
+  }
   return { id, job_id: content.job.id, created_at: createdAt };
 };
 var readContent = (home, id) => {
@@ -6407,6 +6275,55 @@ var readRun = (home, id) => {
     pending_feedback_ids: feedback2.map((item) => item.id).filter((fid) => !processed2.has(fid))
   };
 };
+var selectionChecker = (text) => {
+  let validate;
+  return (selection) => {
+    validate ??= createSelectionValidator(text);
+    try {
+      return validate(selection).ok;
+    } catch {
+      return false;
+    }
+  };
+};
+var feedbackDraftVersion = (item, content, fitsCurrent = selectionChecker(content.draft_text)) => {
+  if (item.draft_version !== void 0) return item.draft_version;
+  const current = content.draft_version ?? 0;
+  const legacyRevised = content.draft_version === void 0 && content.revised_at !== void 0;
+  return !legacyRevised || fitsCurrent(item.selection) ? current : -1;
+};
+var pendingFeedbackByDraft = (home, id, run) => {
+  const pending = new Set(run.pending_feedback_ids);
+  const fitsCurrent = selectionChecker(run.content.draft_text);
+  const groups = /* @__PURE__ */ new Map();
+  const versionById = /* @__PURE__ */ new Map();
+  for (const item of run.feedback) {
+    if (!pending.has(item.id)) continue;
+    const version2 = feedbackDraftVersion(item, run.content, fitsCurrent);
+    versionById.set(item.id, version2);
+    if (!groups.has(version2)) groups.set(version2, []);
+    groups.get(version2).push(item);
+  }
+  const drafts = /* @__PURE__ */ new Map();
+  const unavailable = [];
+  for (const version2 of groups.keys()) {
+    try {
+      drafts.set(version2, readDraftVersion(home, id, version2, run.content));
+    } catch (error2) {
+      if (!(error2 instanceof StoreProblem)) throw error2;
+      unavailable.push(version2);
+    }
+  }
+  return { groups, drafts, unavailable, versionById };
+};
+var readDraftVersion = (home, id, version2, content = readContent(home, id)) => {
+  if (version2 === (content.draft_version ?? 0)) return content.draft_text;
+  const file = path2.join(runDir(home, id), `draft-${version2}.txt`);
+  if (!Number.isInteger(version2) || version2 < 0 || !existsSync2(file)) {
+    throw problem(409, "storage_integrity", "\uD53C\uB4DC\uBC31\uC744 \uB0A8\uAE34 \uCD08\uC548\uC744 \uCC3E\uC744 \uC218 \uC5C6\uC2B5\uB2C8\uB2E4.", "\uAC80\uC218 \uAE30\uB85D\uC744 \uD655\uC778\uD574 \uC8FC\uC138\uC694.");
+  }
+  return readFileSync2(file, "utf8");
+};
 var markProcessed = (home, id, feedbackIds) => {
   const dir = runDir(home, id);
   assertRunExists(dir);
@@ -6420,7 +6337,13 @@ var markProcessed = (home, id, feedbackIds) => {
     const state = readJson(path2.join(dir, "state.json"), { processed_feedback_ids: [] });
     const processed2 = [.../* @__PURE__ */ new Set([...state.processed_feedback_ids ?? [], ...feedbackIds])];
     writeAtomic(path2.join(dir, "state.json"), JSON.stringify({ ...state, processed_feedback_ids: processed2 }));
-    return { processed: processed2.length, pending: feedback2.filter((item) => !processed2.includes(item.id)).length };
+    const done = new Set(processed2);
+    const needed = new Set(feedback2.filter((item) => !done.has(item.id)).map((item) => item.draft_version));
+    for (const file of readdirSync(dir)) {
+      const match = /^draft-(\d+)\.txt$/u.exec(file);
+      if (match && !needed.has(Number(match[1]))) rmSync(path2.join(dir, file), { force: true });
+    }
+    return { processed: processed2.length, pending: feedback2.filter((item) => !done.has(item.id)).length };
   });
 };
 var reviseDraft = (home, id, draftText, { check: check2 } = {}) => {
@@ -6431,9 +6354,31 @@ var reviseDraft = (home, id, draftText, { check: check2 } = {}) => {
     if (content.draft_text === draftText) {
       throw problem(409, "unchanged_draft", "\uC218\uC815\uBCF8\uC774 \uAE30\uC874 \uCD08\uC548\uACFC \uAC19\uC2B5\uB2C8\uB2E4.", "\uC218\uC815\uD55C \uD30C\uC77C\uC744 \uC9C0\uC815\uD588\uB294\uC9C0 \uD655\uC778\uD574 \uC8FC\uC138\uC694.");
     }
+    if (content.source_surface?.snapshot) {
+      throw problem(409, "surface_review", "\uCEA1\uCC98\uD55C \uD398\uC774\uC9C0 \uAC80\uC218\uB294 \uC218\uC815\uBCF8\uC73C\uB85C \uBC14\uAFC0 \uC218 \uC5C6\uC2B5\uB2C8\uB2E4.", "\uC218\uC815\uD55C \uD30C\uC77C\uB85C \uC0C8 \uAC80\uC218\uB97C \uAC8C\uC2DC\uD574 \uC8FC\uC138\uC694.");
+    }
+    const version2 = content.draft_version ?? 0;
     const { source_surface: _surface, check: _check2, ...rest } = content;
-    const next = { ...rest, draft_text: draftText, ...check2 ? { check: check2 } : {}, revised_at: (/* @__PURE__ */ new Date()).toISOString() };
-    writeAtomic(path2.join(dir, "content.json"), serializeReviewContent(next));
+    const next = {
+      ...rest,
+      draft_text: draftText,
+      draft_version: version2 + 1,
+      ...check2 ? { check: check2 } : {},
+      revised_at: (/* @__PURE__ */ new Date()).toISOString()
+    };
+    const serialized = serializeReviewContent(next);
+    const feedback2 = readJson(path2.join(dir, "feedback.json"), []);
+    const unversioned = feedback2.filter((item) => item.draft_version === void 0);
+    if (unversioned.length) {
+      const fitsCurrent = selectionChecker(content.draft_text);
+      for (const item of unversioned) item.draft_version = feedbackDraftVersion(item, content, fitsCurrent);
+      writeAtomic(path2.join(dir, "feedback.json"), JSON.stringify(feedback2, null, 2));
+    }
+    const processed2 = new Set(readJson(path2.join(dir, "state.json"), { processed_feedback_ids: [] }).processed_feedback_ids ?? []);
+    if (feedback2.some((item) => !processed2.has(item.id) && item.draft_version === version2)) {
+      writeAtomic(path2.join(dir, `draft-${version2}.txt`), content.draft_text);
+    }
+    writeAtomic(path2.join(dir, "content.json"), serialized);
     return { id, revised_at: next.revised_at };
   });
 };
@@ -6457,15 +6402,23 @@ var listRuns = (home, now = Date.now()) => {
   if (!existsSync2(runsDir(home))) return [];
   return readdirSync(runsDir(home)).filter((name) => REVIEW_ID.test(name)).flatMap((id) => {
     try {
-      const run = readRun(home, id);
-      const createdAt = run.content.created_at;
+      const dir = runDir(home, id);
+      let meta2 = null;
+      try {
+        meta2 = readJson(path2.join(dir, "meta.json"), null);
+      } catch {
+        meta2 = null;
+      }
+      const createdAt = (meta2?.created_at ? meta2 : readJson(path2.join(dir, "content.json"), null)).created_at;
+      const feedback2 = readJson(path2.join(dir, "feedback.json"), []);
+      const processed2 = new Set(readJson(path2.join(dir, "state.json"), { processed_feedback_ids: [] }).processed_feedback_ids ?? []);
       return [
         {
           review_id: id,
           created_at: createdAt,
           age_days: Math.floor((now - Date.parse(createdAt)) / 864e5),
-          feedback: run.feedback.length,
-          pending: run.pending_feedback_ids.length
+          feedback: feedback2.length,
+          pending: feedback2.filter((item) => !processed2.has(item.id)).length
         }
       ];
     } catch {
@@ -6495,12 +6448,13 @@ var parseArgs = (argv) => {
   }
   return options;
 };
-var readText = (file) => {
+var readRaw = (file, maxBytes = MAX_TEXT_BYTES) => {
   const stat = statSync2(file, { throwIfNoEntry: false });
   if (!stat?.isFile()) throw new UsageError(`not a file: ${file}`);
-  if (stat.size > MAX_TEXT_BYTES) throw new UsageError(`file larger than 2 MiB: ${file}`);
-  return readFileSync3(file, "utf8").normalize("NFC");
+  if (stat.size > maxBytes) throw new UsageError(`file larger than ${maxBytes / 1024 / 1024} MiB: ${file}`);
+  return readFileSync3(file, "utf8");
 };
+var readText = (file) => readRaw(file).normalize("NFC");
 var requireOptions = (options, ...keys) => {
   for (const key of keys) if (!options[key]) throw new UsageError(`--${key} is required`);
 };
@@ -6567,10 +6521,15 @@ var ensureServer = async (home) => {
     release();
   }
 };
-var reviewUrls = (port, id) => {
-  const local = `http://127.0.0.1:${port}/#review=${id}`;
-  const { publicOrigin } = loadIdleConfig();
-  return publicOrigin ? { url: local, remote_url: `${publicOrigin}/#review=${id}` } : { url: local };
+var reviewUrls = (port, id, content, { publicOrigin } = loadIdleConfig()) => {
+  const url3 = `http://127.0.0.1:${port}/#review=${id}`;
+  const surfaceUrl = content.source_surface?.snapshot ? `http://127.0.0.1:${port}/feedback-wrapper#review=${id}` : void 0;
+  return {
+    url: url3,
+    ...surfaceUrl ? { surface_url: surfaceUrl } : {},
+    ...publicOrigin ? { remote_url: `${publicOrigin}/#review=${id}` } : {},
+    ...publicOrigin && surfaceUrl ? { remote_surface_url: `${publicOrigin}/feedback-wrapper#review=${id}` } : {}
+  };
 };
 var attachmentInput = (file) => {
   const mediaType = MEDIA_TYPES[path3.extname(file).toLowerCase()];
@@ -6583,21 +6542,19 @@ var publish = async (home, options) => {
   const draftText = readText(options.draft);
   const mode = options.mode ?? detectMode(sourceText);
   if (mode !== "translate" && mode !== "polish") throw new UsageError("--mode must be translate or polish");
-  const surface = options.surface ? SourceSurfaceSchema.parse(JSON.parse(readText(options.surface))) : void 0;
+  const surface = options.surface ? JSON.parse(readRaw(options.surface, MAX_REVIEW_CONTENT_BYTES)) : void 0;
   const input = TranslationPublishInputSchema.parse({
     run_id: randomUUID2(),
     source_text: sourceText,
     jobs: [{ target_language: "ko", draft_text: draftText, ...surface ? { source_surface: surface } : {} }],
-    ...options.attach.length ? { attachments: options.attach.map(attachmentInput) } : {},
-    upload_consent: true,
-    idempotency_key: randomUUID2()
+    ...options.attach.length ? { attachments: options.attach.map(attachmentInput) } : {}
   });
   const report = check({ source: sourceText, draft: draftText, mode, standards: loadStandards() });
   const run = createRun(home, input, { mode, check: report });
   const checkSummary = { result: report.result, exit_code: report.exit_code };
   try {
     const { port } = await ensureServer(home);
-    return { review_id: run.id, ...reviewUrls(port, run.id), check: checkSummary };
+    return { review_id: run.id, ...reviewUrls(port, run.id, { source_surface: surface }), check: checkSummary };
   } catch (error2) {
     error2.body = {
       code: "server_unavailable",
@@ -6621,11 +6578,23 @@ var revise = (home, options) => {
 var feedback = (home, options) => {
   requireOptions(options, "review");
   const run = readRun(home, options.review);
+  const current = run.content.draft_version ?? 0;
+  const { drafts, unavailable, versionById } = pendingFeedbackByDraft(home, options.review, run);
+  const earlierDrafts = Object.fromEntries([...drafts].filter(([version2]) => version2 !== current));
+  const fitsCurrent = selectionChecker(run.content.draft_text);
   return {
     review_id: options.review,
     source_text: run.content.source_text,
     draft_text: run.content.draft_text,
-    feedback: run.feedback,
+    draft_version: current,
+    ...Object.keys(earlierDrafts).length ? { earlier_drafts: earlierDrafts } : {},
+    // Items on these versions cannot be located; ask the user to leave them again.
+    ...unavailable.length ? { unavailable_draft_versions: unavailable } : {},
+    // Items written without a version carry the one they resolve to, so every item can be matched
+    // against draft_version, earlier_drafts, and unavailable_draft_versions.
+    feedback: run.feedback.map(
+      (item) => item.draft_version === void 0 ? { ...item, draft_version: versionById.get(item.id) ?? feedbackDraftVersion(item, run.content, fitsCurrent) } : item
+    ),
     processed_feedback_ids: run.processed_feedback_ids,
     pending_feedback_ids: run.pending_feedback_ids
   };
@@ -6635,33 +6604,76 @@ var verifyFeedback = (home, options) => {
   const run = readRun(home, options.review);
   const pending = new Set(run.pending_feedback_ids);
   const items = run.feedback.filter((item) => pending.has(item.id));
-  if (items.length === 0) return { review_id: options.review, summary: { resolved: 0, unresolved: 0, not_verifiable: 0 }, resolutions: [] };
-  const payload = validatePayload({
-    schema_version: 1,
-    before_text: run.content.draft_text,
-    after_text: readText(options.draft),
-    resume: {
-      run_id: options.review,
-      state: "reviewed",
-      feedback: items.map((item) => ({
-        feedback_id: item.id,
-        job_id: item.job_id,
-        category: item.category,
-        scope: item.scope,
-        selection: item.selection,
-        has_correction: typeof item.correction === "string",
-        ...item.term ? { term: item.term } : {}
-      })),
-      corrections: items.filter((item) => typeof item.correction === "string").map((item) => ({
-        correction_id: item.id,
-        feedback_id: item.id,
-        category: item.category,
-        scope: item.scope,
-        correction: item.correction
-      }))
+  const summary = { resolved: 0, unresolved: 0, not_verifiable: 0 };
+  if (items.length === 0) return { review_id: options.review, schema_version: 1, summary, resolutions: [] };
+  const afterText = readText(options.draft);
+  const { groups, drafts } = pendingFeedbackByDraft(home, options.review, run);
+  const resolutionById = /* @__PURE__ */ new Map();
+  const notVerifiable = (item, check2) => resolutionById.set(item.id, { feedback_id: item.id, category: item.category, status: "not_verifiable", checks: [check2] });
+  for (const [version2, group] of groups) {
+    const beforeText = drafts.get(version2);
+    if (beforeText === void 0) {
+      for (const item of group) notVerifiable(item, "draft_unavailable");
+      continue;
     }
-  });
-  return { review_id: options.review, ...verify(payload) };
+    const fits = selectionChecker(beforeText);
+    const fitting = [];
+    for (const item of group) {
+      if (fits(item.selection)) {
+        fitting.push(item);
+      } else {
+        notVerifiable(item, "selection_mismatch");
+      }
+    }
+    if (fitting.length === 0) continue;
+    const payloadFor = (entries) => validatePayload({
+      schema_version: 1,
+      before_text: beforeText,
+      after_text: afterText,
+      resume: {
+        run_id: options.review,
+        state: "reviewed",
+        feedback: entries.map((item) => ({
+          feedback_id: item.id,
+          job_id: item.job_id,
+          category: item.category,
+          scope: item.scope,
+          selection: item.selection,
+          has_correction: typeof item.correction === "string",
+          ...item.term ? { term: item.term } : {}
+        })),
+        corrections: entries.filter((item) => typeof item.correction === "string").map((item) => ({
+          correction_id: item.id,
+          feedback_id: item.id,
+          category: item.category,
+          scope: item.scope,
+          correction: item.correction
+        }))
+      }
+    });
+    const batches = [];
+    const collect = (entries) => {
+      try {
+        batches.push(payloadFor(entries));
+      } catch (error2) {
+        if (!(error2 instanceof ResolutionInputError)) throw error2;
+        if (entries.length === 1) {
+          notVerifiable(entries[0], "invalid_feedback");
+          return;
+        }
+        const middle = Math.ceil(entries.length / 2);
+        collect(entries.slice(0, middle));
+        collect(entries.slice(middle));
+      }
+    };
+    collect(fitting);
+    for (const payload of batches) {
+      for (const resolution of verify(payload).resolutions) resolutionById.set(resolution.feedback_id, resolution);
+    }
+  }
+  const resolutions = items.map((item) => resolutionById.get(item.id));
+  for (const resolution of resolutions) summary[resolution.status] += 1;
+  return { review_id: options.review, schema_version: 1, summary, resolutions };
 };
 var processed = (home, options) => {
   requireOptions(options, "review", "ids");
@@ -6670,9 +6682,9 @@ var processed = (home, options) => {
 };
 var url2 = async (home, options) => {
   requireOptions(options, "review");
-  readContent(home, options.review);
+  const content = readContent(home, options.review);
   const { port } = await ensureServer(home);
-  return { review_id: options.review, ...reviewUrls(port, options.review) };
+  return { review_id: options.review, ...reviewUrls(port, options.review, content ?? {}) };
 };
 var list = (home) => ({
   stale_after_days: STALE_DAYS,
@@ -6714,5 +6726,6 @@ if (process.argv[1] && path3.resolve(process.argv[1]) === fileURLToPath(import.m
 }
 export {
   ensureServer,
-  main
+  main,
+  reviewUrls
 };