clearotron 0.3.2 → 0.3.3-beta.0

package/driver/queue-markers.mjs

@@ -78,6 +78,50 @@ export const PROSE_PARTS = {
   campaignShape: ".campaignShape.txt",   // P2-C (Round-2 §8a): campaign-shape facts — stated launch shape
 };
 
+// ── ONE GOODS DESCRIPTION, TWO SPELLINGS, AND EVERY READER GETS THE SAME ANSWER ────────────────────
+//
+// Intake accepts a goods description under the current field or under the older spelling a forwarding
+// agent may still send. That acceptance was the gate's private knowledge: the scope stamp read the
+// current field alone, so a job written the older way passed intake and recorded that the matter named
+// no goods. Both sites were individually right and nothing compared them.
+//
+// Fixing the scope stamp alone moved the seam rather than closing it. Ten other readers — the clearance
+// and knockout prompts, the pharmaceutical test, the product-context derivation, the portal's own row
+// and the plan preview — still read the current field, so the scope file would carry a value the
+// prompts never saw, and the validator comparing the two would report the disagreement as a fault in
+// the frame.
+//
+// SO THE FOLD HAPPENS ONCE, WHERE THE JOB IS ASSEMBLED FOR A RUN, and every reader downstream sees one
+// field. Nothing is written back to the queue file: the manifest stays exactly as it was filed, which is
+// the same rule the intake gate follows about stamping derived values onto a job.
+//
+// Here, in the module that already owns the queue's own vocabulary, because the gate and the assembly
+// must never hold two opinions about what counts. A hand-copied predicate is how they drifted the first
+// time.
+
+/** The field names a goods description may arrive under, newest first. */
+export const GOODS_FIELDS = ["goods", "use"];
+
+/** The goods description a job carries under any accepted spelling, or null. Pure. */
+export function goodsOf(job) {
+  for (const f of GOODS_FIELDS) {
+    const v = job?.[f];
+    if (v != null && String(v).trim()) return v;
+  }
+  return null;
+}
+
+/**
+ * The same job with its goods description under the current field, whichever spelling it arrived in.
+ * Returns the job unchanged when there is nothing to fold, so a job already on the current field is
+ * byte-identical and no caller has to ask which it got.
+ */
+export function withFoldedGoods(job) {
+  const goods = goodsOf(job);
+  if (goods == null || job?.goods === goods) return job;
+  return { ...job, goods };
+}
+
 /** A consumed claim's sidecars, swept together by runner.mjs cleanupClaimSidecars. */
 export const CLAIM_SIDECAR_SUFFIXES = [".pid", ".meta", ".skips"];
 

package/driver/queue-watch-verdict.mjs

@@ -91,8 +91,8 @@ export function queueWatchVerdict({ queueDirs, watched, unitPath, unitError = nu
   // anywhere else; a privilege-limited read that answers "fine" is the exact failure this family of
   // checks exists to refuse, and it is the one that would make this guard decoration on the box that
   // matters most.
-  if (unitError) return { state: "skip", message: `the .path unit could not be read — ${unitPath}: ${unitError}` };
-  if (resolveError) return { state: "skip", message: `the queue dirs could not be resolved — ${resolveError}` };
+  if (unitError) return { state: "skip", blocked: true, message: `the .path unit could not be read — ${unitPath}: ${unitError}` };
+  if (resolveError) return { state: "skip", blocked: true, message: `the queue dirs could not be resolved — ${resolveError}` };
 
   const q = Array.isArray(queueDirs) ? queueDirs : null;
   const w = Array.isArray(watched) ? watched : [];

package/driver/register-availability.mjs

@@ -41,9 +41,9 @@
 // CORRECTED 2026-08-11 — THIS PARAGRAPH USED TO SAY "one qid per office … which is the shape
 // joinPlanToBands already has", AND THAT WAS FALSE ABOUT THE COMPILER IT DOCUMENTS. There is no
 // per-office qid. compileRegisterPlan narrows ONE shared `regions` array and hands it to every entry
-// (register-plan.mjs:846 compileRegisterPlan); the unreachable office produces no entry, so no qid, so no band block, so
+// (register-plan.mjs compileRegisterPlan); the unreachable office produces no entry, so no qid, so no band block, so
 // nothing ever reaches joinPlanToBands' deferred bucket — whose only source is a block stamped
-// `error:true && deferred:true` (register-plan.mjs:1489 extendRegisterPlan).
+// `error:true && deferred:true` (register-plan.mjs extendRegisterPlan).
 //
 // The consequence was not academic. `deferred_coverage` rode the plan and nothing that a reader sees
 // read it: coverage-form.mjs seeded its deferred rows from skeleton qids alone, so an EU+US matter on a