@semiont/jobs 0.6.8 → 0.6.10

package/dist/worker-main.js

@@ -1,4 +1,4 @@
-import { JOB_TYPES, replyChannelsFor, createTomlConfigLoader, didToAgent, baseUrl, STARTUP_FETCH_RETRY, retryWithBackoff, serviceAccountToken, isTransientFetchError, isObject, isString, isNumber, RETRY_RULES, resourceId, getPrimaryMediaType, isGenerationJobParams, capabilitiesOf, assembleAnnotation, findClaimSpan, busRequest, BusRequestError, isArray, textSourceOf, yieldsGeometryOf, decodeRepresentation, GENERATABLE_MEDIA_TYPES, locate, createFragmentSelector, deriveViews, annotationId, estimateTokens, reconcileSelector, getLocaleEnglishName, cutChunk, chunkText } from '@semiont/core';
+import { replyChannelsFor, createTomlConfigLoader, JOB_TYPES, didToAgent, baseUrl, STARTUP_FETCH_RETRY, retryWithBackoff, serviceAccountToken, isTransientFetchError, isObject, isString, isNumber, RETRY_RULES, getPrimaryMediaType, isGenerationJobParams, capabilitiesOf, assembleAnnotation, findClaimSpan, busRequest, BusRequestError, textSourceOf, yieldsGeometryOf, decodeRepresentation, GENERATABLE_MEDIA_TYPES, locate, createFragmentSelector, deriveViews, isResourceId, estimateTokens, reconcileSelector, getLocaleEnglishName, cutChunk, chunkText } from '@semiont/core';
 import { archivistContentReads, extractPdfTextLayer, withinByteBudget, MAX_PDF_BYTES } from '@semiont/content';
 import { withSpan, SpanKind, recordJobOutcome, recordAnchorOutcome, recordDetectionCall } from '@semiont/observability';
 import { createInferenceClient, answerLimitsRequests, StructuredReadError } from '@semiont/inference';
@@ -3766,9 +3766,9 @@ var require_mapOneOrManyArgs = __commonJS({
     Object.defineProperty(exports, "__esModule", { value: true });
     exports.mapOneOrManyArgs = void 0;
     var map_1 = require_map();
-    var isArray2 = Array.isArray;
+    var isArray = Array.isArray;
     function callOrApply(fn, args) {
-      return isArray2(args) ? fn.apply(void 0, __spreadArray([], __read(args))) : fn(args);
+      return isArray(args) ? fn.apply(void 0, __spreadArray([], __read(args))) : fn(args);
     }
     function mapOneOrManyArgs(fn) {
       return map_1.map(function(args) {
@@ -3913,14 +3913,14 @@ var require_argsArgArrayOrObject = __commonJS({
   "../../node_modules/rxjs/dist/cjs/internal/util/argsArgArrayOrObject.js"(exports) {
     Object.defineProperty(exports, "__esModule", { value: true });
     exports.argsArgArrayOrObject = void 0;
-    var isArray2 = Array.isArray;
+    var isArray = Array.isArray;
     var getPrototypeOf = Object.getPrototypeOf;
     var objectProto = Object.prototype;
     var getKeys = Object.keys;
     function argsArgArrayOrObject(args) {
       if (args.length === 1) {
         var first_1 = args[0];
-        if (isArray2(first_1)) {
+        if (isArray(first_1)) {
           return { args: first_1, keys: null };
         }
         if (isPOJO(first_1)) {
@@ -4668,9 +4668,9 @@ var require_argsOrArgArray = __commonJS({
   "../../node_modules/rxjs/dist/cjs/internal/util/argsOrArgArray.js"(exports) {
     Object.defineProperty(exports, "__esModule", { value: true });
     exports.argsOrArgArray = void 0;
-    var isArray2 = Array.isArray;
+    var isArray = Array.isArray;
     function argsOrArgArray(args) {
-      return args.length === 1 && isArray2(args[0]) ? args[0] : args;
+      return args.length === 1 && isArray(args[0]) ? args[0] : args;
     }
     exports.argsOrArgArray = argsOrArgArray;
   }
@@ -9208,19 +9208,6 @@ var require_cjs = __commonJS({
 
 // src/job-claim-adapter.ts
 var import_rxjs = __toESM(require_cjs());
-function readUnitCursors(raw, completedUnits) {
-  if (!isObject(raw)) return {};
-  const done = new Set(completedUnits);
-  const cursors = {};
-  for (const [unit, value] of Object.entries(raw)) {
-    if (done.has(unit) || !isObject(value)) continue;
-    const { next, size, found, emitted } = value;
-    if (!isNumber(next) || !isNumber(size) || next < 0 || size < 1) continue;
-    if (!isNumber(found) || !isNumber(emitted) || found < 0 || emitted < 0) continue;
-    cursors[unit] = { next, size, found, emitted };
-  }
-  return cursors;
-}
 function createJobClaimAdapter(options) {
   const { bus, jobTypes } = options;
   const activeJob$ = new import_rxjs.BehaviorSubject(null);
@@ -9239,9 +9226,9 @@ function createJobClaimAdapter(options) {
   let activeSince = null;
   const iso = (t) => t === null ? null : new Date(t).toISOString();
   const claimNext = async () => {
-    let record;
+    let claimed;
     try {
-      record = await busRequest(bus, "job:claim", { types: jobTypes }, 1e4);
+      claimed = await busRequest(bus, "job:claim", { types: jobTypes }, 1e4);
     } catch (error) {
       if (error instanceof BusRequestError) {
         if (error.code === "bus.none-pending") return { declined: true };
@@ -9249,25 +9236,19 @@ function createJobClaimAdapter(options) {
       }
       return { refused: { code: null, message: error instanceof Error ? error.message : String(error) } };
     }
-    if (!isString(record.metadata?.id) || !isString(record.metadata?.type)) {
-      return { refused: { code: null, message: "claimed record carries no job id or type" } };
-    }
-    const completedUnits = isArray(record.metadata?.completedUnits) ? record.metadata.completedUnits.filter(isString) : [];
-    const unitCursors = readUnitCursors(record.metadata?.unitCursors, completedUnits);
-    const params = record.params ?? {};
+    const { metadata, params } = claimed;
     return {
       job: {
-        jobId: record.metadata.id,
-        type: record.metadata.type,
-        resourceId: isString(params.resourceId) ? params.resourceId : "",
-        userId: record.metadata?.userId ?? "",
+        jobId: metadata.id,
+        type: metadata.type,
+        resourceId: params.resourceId,
         params,
-        completedUnits,
-        unitCursors,
-        // Absent or malformed metadata reads as "no budget left" — a worker
-        // that cannot see the budget must not claim a retry is coming.
-        retryCount: isNumber(record.metadata?.retryCount) ? record.metadata.retryCount : 0,
-        maxRetries: isNumber(record.metadata?.maxRetries) ? record.metadata.maxRetries : 0
+        // Both appear on the record once an attempt has checkpointed, and a
+        // finished unit has no cursor: the dispatcher drops it at the merge.
+        completedUnits: metadata.completedUnits ?? [],
+        unitCursors: metadata.unitCursors ?? {},
+        retryCount: metadata.retryCount,
+        maxRetries: metadata.maxRetries
       }
     };
   };
@@ -9384,10 +9365,8 @@ function createJobClaimAdapter(options) {
 function willRetryAfter(metadata, failureClass) {
   return failureClass !== "deterministic" && metadata.retryCount < metadata.maxRetries;
 }
-var KNOWN_JOB_TYPES = new Set(JOB_TYPES);
-function isJobType(value) {
-  return KNOWN_JOB_TYPES.has(value);
-}
+
+// src/types.ts
 function asJobParams(params) {
   if (typeof params.resourceId !== "string") {
     throw new Error("Job params are missing a resourceId");
@@ -9949,7 +9928,8 @@ Example format:
   /**
    * Build a prompt for detecting structural tags
    *
-   * @param content - The full text content to analyze (NOT truncated for structural analysis)
+   * @param content - The text content to analyze — a chunk sized by the
+   *   caller from derived provider limits; NEVER re-truncated here
    * @param category - The specific category to detect
    * @param schemaName - Human-readable schema name
    * @param schemaDescription - Schema description
@@ -10377,7 +10357,7 @@ ${piece}
   }
   if (items.length * YIELD_COLLAPSE_BAND < counted) {
     throw new YieldCollapseError(
-      `Extraction found ${items.length} entities where a count call reports ~${counted} mentions (band \xD7${YIELD_COLLAPSE_BAND}) on a ${piece.length}-char chunk \u2014 silent yield collapse (F7): deterministic \u2014 a same-size retry returns the identical under-report.`,
+      `Extraction found ${items.length} entities where a count call reports ~${counted} mentions (band \xD7${YIELD_COLLAPSE_BAND}) on a ${piece.length}-char chunk \u2014 silent yield collapse: deterministic \u2014 a same-size retry returns the identical under-report.`,
       [...items],
       { found: items.length, counted, pieceChars: piece.length }
     );
@@ -10752,8 +10732,6 @@ function compileTypst(source) {
     rmSync(dir, { recursive: true, force: true });
   }
 }
-
-// src/workers/generation/citation-resolver.ts
 var CITATION_TOKEN = /\[\[([^\s[\]/]+)(?:\/([^\s[\]/]+))?\]\]/g;
 function collectContextResourceIds(context) {
   const ids = /* @__PURE__ */ new Set();
@@ -10779,7 +10757,7 @@ function resolveCitationTokens(content, validResourceIds, logger2) {
     const annotationId = match[2];
     clean += content.slice(last, match.index).replace(/[ \t]+$/, "");
     last = match.index + token.length;
-    if (!validResourceIds.has(citedResourceId)) {
+    if (!isResourceId(citedResourceId) || !validResourceIds.has(citedResourceId)) {
       logger2.warn("Citation token references an id absent from the provided context \u2014 dropped", {
         resourceId: citedResourceId
       });
@@ -11020,9 +10998,8 @@ async function processCommentJob(content, inferenceClient, params, buildAnnotati
       found += comments.length;
       const fresh = dedupe(comments.map(
         (c) => (
-          // Match the pre-#651 CommentAnnotationWorker: include format and
-          // language on the body TextualBody. Optional in the schema, but
-          // consumers that do language-aware rendering rely on them.
+          // Format and language go on the body TextualBody: optional in the
+          // schema, but consumers that do language-aware rendering rely on them.
           buildAnnotation("commenting", c, [
             { type: "TextualBody", value: c.comment, purpose: "commenting", format: "text/plain", language: bodyLanguage }
           ])
@@ -11063,8 +11040,7 @@ async function processAssessmentJob(content, inferenceClient, params, buildAnnot
       const fresh = dedupe(assessments.map(
         (a) => (
           // Single-object body with purpose aligned to motivation, matching the
-          // pre-#651 AssessmentAnnotationWorker's shape and the majority of
-          // persisted assessments. Do not switch to an array or to
+          // majority of persisted assessments. Do not switch to an array or to
           // purpose='describing' — that loses the "this is an assessment, not
           // a description" signal and breaks existing readers that access
           // `body.value` directly on the object.
@@ -11132,11 +11108,11 @@ async function processReferenceJob(content, inferenceClient, params, buildAnnota
       params.includeDescriptiveReferences ?? false,
       logger2,
       params.sourceLanguage,
-      // Liveness heartbeat (DETECTION-HEARTBEAT): fires at chunk boundaries and
-      // every ~15 s while a call is in flight, so a long single-chunk call is
-      // not silent. It repeats the current position rather than inventing an
-      // advance — the stall watchdog, janitor and client timeout need a signal,
-      // not a monotone.
+      // Liveness heartbeat: fires at chunk boundaries and every ~15 s while a
+      // call is in flight, so a long single-chunk call is not silent. It
+      // repeats the current position rather than inventing an advance — the
+      // stall watchdog, janitor and client timeout need a signal, not a
+      // monotone.
       () => emitTypeProgress(entityTypeName),
       (verdict) => {
         underReported = {
@@ -11293,7 +11269,7 @@ async function processGenerationJob(inferenceClient, params, onProgress, logger2
       `Unsupported outputMediaType for generation: ${outputMediaType}. Generation produces ${GENERATABLE_MEDIA_TYPES.join(" or ")}.`
     );
   }
-  const title = params.title ?? "Untitled";
+  const title = params.title;
   const entityTypes = (params.entityTypes ?? []).map(String);
   if (outputMediaType === "application/pdf") {
     onProgress(5, { code: "generating-resource" });
@@ -11373,12 +11349,7 @@ async function processGenerationJob(inferenceClient, params, onProgress, logger2
       title,
       format: outputMediaType,
       citations: citations2,
-      result: {
-        kind: "generation",
-        resourceId: "",
-        resourceName: title,
-        truncated: generated2.truncated
-      }
+      truncated: generated2.truncated
     };
   }
   onProgress(5, { code: "generating-resource" });
@@ -11414,12 +11385,7 @@ async function processGenerationJob(inferenceClient, params, onProgress, logger2
     title,
     format: outputMediaType,
     citations,
-    result: {
-      kind: "generation",
-      resourceId: "",
-      resourceName: title,
-      truncated: generated.truncated
-    }
+    truncated: generated.truncated
   };
 }
 
@@ -11503,11 +11469,11 @@ var CommitDurabilityError = class extends Error {
   durability;
   name = "CommitDurabilityError";
 };
-async function probeDurability(session, resourceId$1, annotations) {
+async function probeDurability(session, resourceId, annotations) {
   const last = annotations[annotations.length - 1];
   if (!last) return "probe-unreachable";
   try {
-    await session.client.browse.annotation(resourceId(resourceId$1), annotationId(String(last.id))).fresh();
+    await session.client.browse.annotation(resourceId, last.id).fresh();
     return "probe-confirmed";
   } catch (error) {
     const code = isObject(error) && isString(error.code) ? error.code : void 0;
@@ -11564,32 +11530,29 @@ function startWorkerProcess(config) {
       completedUnitsByJob.delete(job.jobId);
       unitCursorsByJob.delete(job.jobId);
       const failAnnotationId = referenceIdOf(job);
-      if (isJobType(job.type)) {
-        emitEvent(session, "job:fail", {
-          resourceId: job.resourceId,
-          jobId: job.jobId,
-          jobType: job.type,
-          ...failAnnotationId ? { annotationId: failAnnotationId } : {},
-          error: message,
-          ...completedUnits && completedUnits.length > 0 ? { completedUnits } : {},
-          // Where each unfinished unit got to. Absent rather than `{}` when
-          // nothing was reached: an empty object would claim units were
-          // tracked and none progressed.
-          ...unitCursors && Object.keys(unitCursors).length > 0 ? { unitCursors } : {},
-          ...failureClass !== void 0 ? { failureClass } : {},
-          // What the commit path OBSERVED about durability, when the failure
-          // came from a commit at all. Present only on that path: absent means
-          // the question never arose, never that durability was ruled out.
-          ...error instanceof CommitDurabilityError ? { durability: error.durability } : {},
-          // Whether this failure is the END, answered by the same predicate
-          // the queue applies at failJob (JOB-RESTART-SAFETY P5). Without
-          // it a client cannot tell a recovering run from a dead one: it
-          // sees job:fail either way and would end its stream on a job the
-          // queue is about to re-run.
-          willRetry: willRetryAfter(job, failureClass)
-        }).catch(() => {
-        });
-      }
+      emitEvent(session, "job:fail", {
+        resourceId: job.resourceId,
+        jobId: job.jobId,
+        jobType: job.type,
+        ...failAnnotationId ? { annotationId: failAnnotationId } : {},
+        error: message,
+        ...completedUnits && completedUnits.length > 0 ? { completedUnits } : {},
+        // Where each unfinished unit got to. Absent rather than `{}` when
+        // nothing was reached: an empty object would claim units were
+        // tracked and none progressed.
+        ...unitCursors && Object.keys(unitCursors).length > 0 ? { unitCursors } : {},
+        ...failureClass !== void 0 ? { failureClass } : {},
+        // What the commit path OBSERVED about durability, when the failure
+        // came from a commit at all. Present only on that path: absent means
+        // the question never arose, never that durability was ruled out.
+        ...error instanceof CommitDurabilityError ? { durability: error.durability } : {},
+        // Whether this failure is the END, answered by the same predicate
+        // the queue applies at failJob. Without it a client cannot tell a
+        // recovering run from a dead one: it sees job:fail either way and
+        // would end its stream on a job the queue is about to re-run.
+        willRetry: willRetryAfter(job, failureClass)
+      }).catch(() => {
+      });
       adapter.failJob(job.jobId, message);
     }).finally(() => {
       if (activeCancel?.jobId === job.jobId) activeCancel = null;
@@ -11623,17 +11586,11 @@ async function handleJob(adapter, config, job, completedUnitsByJob = /* @__PURE_
 }
 async function handleJobInner(adapter, config, job, completedUnitsByJob, signal, unitCursorsByJob = /* @__PURE__ */ new Map()) {
   const { session, inferenceClient, generator } = config;
-  const { jobId } = job;
-  if (!isJobType(job.type)) {
-    adapter.failJob(jobId, `Unrecognized job type: ${job.type}`);
-    return;
-  }
-  const jobType = job.type;
-  const resourceId$1 = resourceId(job.resourceId);
+  const { jobId, type: jobType, resourceId } = job;
   const annotationId = referenceIdOf(job);
   const attempt = job.retryCount + 1;
   const lifecycleBase = {
-    resourceId: resourceId$1,
+    resourceId,
     jobId,
     jobType,
     attempt,
@@ -11652,22 +11609,22 @@ async function handleJobInner(adapter, config, job, completedUnitsByJob, signal,
   }
   let ready = null;
   if (jobType !== "generation") {
-    const descriptor = await session.client.browse.resource(resourceId$1).fresh();
+    const descriptor = await session.client.browse.resource(resourceId).fresh();
     const mediaType = getPrimaryMediaType(descriptor);
     const source = await withSpan(
       "detection:prepare",
-      () => prepareDetection(mediaType ?? "", config.contentReads, resourceId$1, generator, (rid) => session.client.browse.resourceAnchoredText(rid)),
-      { attrs: { "resource.id": resourceId$1, "media.type": mediaType ?? "unknown" } }
+      () => prepareDetection(mediaType ?? "", config.contentReads, resourceId, generator, (rid) => session.client.browse.resourceAnchoredText(rid)),
+      { attrs: { "resource.id": resourceId, "media.type": mediaType ?? "unknown" } }
     );
     if ("declined" in source) {
       if (source.declined === "not-yet") {
-        throw new Error(`Anchored text not yet derived for resource ${resourceId$1} \u2014 Smelter has not settled; retrying`);
+        throw new Error(`Anchored text not yet derived for resource ${resourceId} \u2014 Smelter has not settled; retrying`);
       }
       if (source.declined === "no-extractor") {
-        throw new DeterministicJobError(`Cannot run ${jobType} on resource ${resourceId$1}: media type '${mediaType ?? "unknown"}' has no extractable text to analyze`);
+        throw new DeterministicJobError(`Cannot run ${jobType} on resource ${resourceId}: media type '${mediaType ?? "unknown"}' has no extractable text to analyze`);
       }
       if (source.declined === "no-map" || source.declined === "unknown") {
-        throw new DeterministicJobError(`Cannot run ${jobType} on resource ${resourceId$1}: anchored-text consult returned '${source.declined}'`);
+        throw new DeterministicJobError(`Cannot run ${jobType} on resource ${resourceId}: anchored-text consult returned '${source.declined}'`);
       }
       await emitEvent(session, "job:complete", {
         ...terminalBase(),
@@ -11698,7 +11655,7 @@ async function handleJobInner(adapter, config, job, completedUnitsByJob, signal,
   };
   const unitCursors = /* @__PURE__ */ new Map();
   const commitChunk = async (annotations, checkpoint) => {
-    record(await commitAnnotations(session, String(resourceId$1), annotations, jobId));
+    record(await commitAnnotations(session, resourceId, annotations, jobId));
     unitCursors.set(checkpoint.unit, checkpoint.cursor);
     unitCursorsByJob.set(job.jobId, Object.fromEntries(unitCursors));
     await emitEvent(session, "job:checkpoint", {
@@ -11717,8 +11674,9 @@ async function handleJobInner(adapter, config, job, completedUnitsByJob, signal,
       // The durability write, per chunk, awaited; folds into the terminal
       // durability evidence like every commit, and carries the unit's cursor.
       commitChunk,
-      // …and the other direction: where an earlier attempt left each unit
-      // (CHUNK-GRAIN-RESUME P3). Empty on a first attempt.
+      // …and the other direction: where an earlier attempt left each unit, so
+      // a partway unit resumes at its offset instead of the top. Empty on a
+      // first attempt.
       job.unitCursors
     );
     await emitEvent(session, "job:complete", {
@@ -11736,8 +11694,8 @@ async function handleJobInner(adapter, config, job, completedUnitsByJob, signal,
       // The durability write, per chunk, awaited; folds into the terminal
       // durability evidence like every commit, and carries the unit's cursor.
       commitChunk,
-      // …and the other direction: where an earlier attempt left each unit
-      // (CHUNK-GRAIN-RESUME P3). Empty on a first attempt.
+      // …and the other direction: where an earlier attempt left each unit.
+      // Empty on a first attempt.
       job.unitCursors
     );
     await emitEvent(session, "job:complete", {
@@ -11755,8 +11713,8 @@ async function handleJobInner(adapter, config, job, completedUnitsByJob, signal,
       // The durability write, per chunk, awaited; folds into the terminal
       // durability evidence like every commit, and carries the unit's cursor.
       commitChunk,
-      // …and the other direction: where an earlier attempt left each unit
-      // (CHUNK-GRAIN-RESUME P3). Empty on a first attempt.
+      // …and the other direction: where an earlier attempt left each unit.
+      // Empty on a first attempt.
       job.unitCursors
     );
     await emitEvent(session, "job:complete", {
@@ -11794,8 +11752,8 @@ async function handleJobInner(adapter, config, job, completedUnitsByJob, signal,
       // The durability write, per chunk, awaited; folds into the terminal
       // durability evidence like every commit, and carries the unit's cursor.
       commitChunk,
-      // …and the other direction: where an earlier attempt left each unit
-      // (CHUNK-GRAIN-RESUME P3). Empty on a first attempt.
+      // …and the other direction: where an earlier attempt left each unit.
+      // Empty on a first attempt.
       job.unitCursors
     );
     if (signal?.aborted) {
@@ -11821,8 +11779,8 @@ async function handleJobInner(adapter, config, job, completedUnitsByJob, signal,
       // The durability write, per chunk, awaited; folds into the terminal
       // durability evidence like every commit, and carries the unit's cursor.
       commitChunk,
-      // …and the other direction: where an earlier attempt left each unit
-      // (CHUNK-GRAIN-RESUME P3). Empty on a first attempt.
+      // …and the other direction: where an earlier attempt left each unit.
+      // Empty on a first attempt.
       job.unitCursors
     );
     await emitEvent(session, "job:complete", {
@@ -11842,7 +11800,6 @@ async function handleJobInner(adapter, config, job, completedUnitsByJob, signal,
       onProgress,
       config.logger
     );
-    const genParams = job.params;
     const genReferenceId = referenceIdOf(job);
     const storageUri = job.params.storageUri;
     const expectedExtension = capabilitiesOf(genResult.format)?.extension;
@@ -11859,11 +11816,11 @@ async function handleJobInner(adapter, config, job, completedUnitsByJob, signal,
       file: Buffer.from(genResult.content),
       format: genResult.format,
       storageUri,
-      sourceResourceId: resourceId$1,
+      sourceResourceId: resourceId,
       ...genReferenceId ? { sourceAnnotationId: genReferenceId } : {},
-      ...genParams.prompt ? { generationPrompt: genParams.prompt } : {},
-      ...genParams.language ? { language: genParams.language } : {},
-      ...genParams.entityTypes && genParams.entityTypes.length > 0 ? { entityTypes: genParams.entityTypes } : {},
+      ...job.params.prompt ? { generationPrompt: job.params.prompt } : {},
+      ...job.params.language ? { language: job.params.language } : {},
+      ...job.params.entityTypes && job.params.entityTypes.length > 0 ? { entityTypes: job.params.entityTypes } : {},
       generator,
       // The resource cites the job it fulfils; who asked for it is derived
       // downstream from that job's events, never stated here.
@@ -11873,12 +11830,12 @@ async function handleJobInner(adapter, config, job, completedUnitsByJob, signal,
       const { annotation: provenanceRef } = assembleAnnotation(
         {
           motivation: "linking",
-          target: { source: String(resourceId$1) },
-          body: { type: "SpecificResource", source: String(newResourceId), purpose: "linking" }
+          target: { source: resourceId },
+          body: { type: "SpecificResource", source: newResourceId, purpose: "linking" }
         },
         generator
       );
-      record(await commitAnnotations(session, String(resourceId$1), [provenanceRef], jobId));
+      record(await commitAnnotations(session, resourceId, [provenanceRef], jobId));
     }
     const citationRefs = [];
     if (genResult.format === "application/pdf" && genResult.citations.length > 0) {
@@ -11903,7 +11860,7 @@ async function handleJobInner(adapter, config, job, completedUnitsByJob, signal,
           }
           const citationRef = buildPdfAnnotation(
             layer,
-            resourceId(String(newResourceId)),
+            newResourceId,
             generator,
             "linking",
             { exact: layer.text.slice(span.start, span.end), start: span.start, end: span.end },
@@ -11918,7 +11875,7 @@ async function handleJobInner(adapter, config, job, completedUnitsByJob, signal,
           {
             motivation: "linking",
             target: {
-              source: String(newResourceId),
+              source: newResourceId,
               selector: [
                 { type: "TextPositionSelector", start: citation.start, end: citation.end },
                 { type: "TextQuoteSelector", exact: citation.exact }
@@ -11931,10 +11888,10 @@ async function handleJobInner(adapter, config, job, completedUnitsByJob, signal,
         citationRefs.push(citationRef);
       }
     }
-    record(await commitAnnotations(session, String(newResourceId), citationRefs, jobId));
+    record(await commitAnnotations(session, newResourceId, citationRefs, jobId));
     await emitEvent(session, "job:complete", {
       ...terminalBase(),
-      result: { kind: "generation", resourceId: newResourceId, resourceName: genResult.title, truncated: genResult.result.truncated }
+      result: { kind: "generation", resourceId: newResourceId, resourceName: genResult.title, truncated: genResult.truncated }
     });
     adapter.completeJob();
   } else {
@@ -11985,26 +11942,25 @@ var WORKER_AWAITED_OPERATIONS = [
   "job:claim",
   "browse:resource-requested",
   // Canonical geometry for a geometry-bearing detection: the consult behind
-  // `ConsultAnchoredText` (SMELTER-OWNS-OCR P2). Its omission broke every
-  // PDF detection job at the transport probe
-  // (.plans/WORKER-ANCHORED-TEXT-CHANNEL.md); the census below now fails the
+  // `ConsultAnchoredText`, answered by the Smelter. Without it every PDF
+  // detection job fails at the transport probe; the census below fails the
   // BUILD when this list and the declared awaits drift.
   "browse:anchored-text-requested",
-  // Durability acknowledgement for a unit's annotations (JOB-RESTART-SAFETY
-  // P6). The worker AWAITS this one — a unit may not advance until its
-  // annotations are in the event log — so its replies must be in the narrow
-  // channel set or every commit fails fast with `bus.unsubscribed`.
+  // Durability acknowledgement for a unit's annotations. The worker AWAITS
+  // this one — a unit may not advance until its annotations are in the event
+  // log — so its replies must be in the narrow channel set or every commit
+  // fails fast with `bus.unsubscribed`.
   "mark:commit",
-  // The durability probe for a commit whose acknowledgement never routed
-  // (COMMIT-ACK-FALSE-FAILURE F1). SINGULAR by design: the annotation LIST
-  // channel is the multi-MB fan-out this narrowing exists to keep out, and a
-  // rare error path is no reason to let it back in.
+  // The durability probe for a commit whose acknowledgement never routed.
+  // SINGULAR by design: the annotation LIST channel is the multi-MB fan-out
+  // this narrowing exists to keep out, and a rare error path is no reason to
+  // let it in.
   "browse:annotation-requested"
 ];
 var WORKER_CONSUMED_BROADCASTS = [
   // The queue announcement the claim adapter races for.
   "job:queued",
-  // Cooperative cancellation of the ACTIVE job (JOB-RESTART-SAFETY P4).
+  // Cooperative cancellation of the ACTIVE job: it stops at a unit boundary.
   "job:cancel-requested"
 ];
 var WORKER_ANSWERED_OPERATIONS = [
@@ -12126,8 +12082,9 @@ async function startAgentWorker(opts) {
     generator,
     // Byte reads for decode-path media only. A geometry-bearing type's text
     // never comes from bytes here — it is CONSULTED from the Smelter's
-    // canonical anchored text over the bus (SMELTER-OWNS-OCR P2), and this
-    // worker cannot derive even by mistake (READ-VS-EXTRACT P2).
+    // canonical anchored text over the bus, and this worker cannot derive
+    // even by mistake: deriving needs the anchored-text store, which only
+    // the Smelter holds.
     contentReads: contentReads2,
     logger: logger2
   });