opentakeoff-mcp 0.9.84 → 0.9.86

package/dist/server-core.js

@@ -118,8 +118,65 @@ function memberEmbed(vpTransform, member, pageH, renderScale) {
   };
 }
 
-// ../web/src/lib/sheets.ts
+// ../web/src/lib/takeoffConstants.ts
+var TAKEOFF_SCHEMA = "opentakeoff.takeoff_canvas.v1";
+var REPORT_SCHEMA = "opentakeoff.report.v1";
 var RENDER_SCALE = 2;
+var PALETTE = Object.freeze([
+  "#c96442",
+  "#2f7d54",
+  "#2563eb",
+  "#9333ea",
+  "#b8860b",
+  "#0d9488",
+  "#be185d",
+  "#1f2937",
+  "#dc2626",
+  "#0891b2"
+]);
+var HATCH_IDS = Object.freeze([
+  "solid",
+  "diag",
+  "diag2",
+  "cross",
+  "diagdense",
+  "horiz",
+  "vert",
+  "grid",
+  "brick",
+  "plank",
+  "herring",
+  "basket",
+  "checker",
+  "wave",
+  "dots",
+  "speckle",
+  "iso",
+  "honeycomb",
+  "scan",
+  "plus",
+  "circuit",
+  "topo",
+  "woodgrain",
+  "chevron",
+  "pinwheel",
+  "harlequin",
+  "hexagon",
+  "penny",
+  "octagondot",
+  "fleur",
+  "concrete"
+]);
+var SNAP_CELL = 24;
+var SNAP_TOL = 7;
+function nextHatchId(conditionCount) {
+  return HATCH_IDS[1 + conditionCount % (HATCH_IDS.length - 1)];
+}
+function nextPaletteColor(conditionCount) {
+  return PALETTE[conditionCount % PALETTE.length];
+}
+
+// ../web/src/lib/sheets.ts
 function sheetBaseLabelFromKey(key) {
   if (typeof key !== "string" || !key || isStitchKey(key)) return "";
   const t = parseSheetKey(key);
@@ -1404,8 +1461,8 @@ function sweepHatchRuns(segs, meta, ws) {
       clusters.pop();
     }
   }
-  const median2 = (arr2) => {
-    const a = arr2.slice().sort((x, y) => x - y);
+  const median2 = (arr3) => {
+    const a = arr3.slice().sort((x, y) => x - y);
     return a[a.length >> 1];
   };
   for (const members of clusters) {
@@ -1533,8 +1590,8 @@ function classifyOffsetAnnotationSegs(segs, meta, ws, maxOffsetPx, minOffsetPx,
         t1: Math.max(c.x1 * dxu + c.y1 * dyu, c.x2 * dxu + c.y2 * dyu),
         w: c.w
       };
-      const arr2 = bins.get(b);
-      if (arr2) arr2.push(r);
+      const arr3 = bins.get(b);
+      if (arr3) arr3.push(r);
       else bins.set(b, [r]);
     }
     for (const runs of bins.values()) {
@@ -1711,8 +1768,8 @@ function sweepDimensionStrings(segs, meta, ws, ftPx, rejects, dimTexts) {
         t0: Math.min(c.x1 * dx + c.y1 * dy, c.x2 * dx + c.y2 * dy),
         t1: Math.max(c.x1 * dx + c.y1 * dy, c.x2 * dx + c.y2 * dy)
       };
-      const arr2 = bins.get(b);
-      if (arr2) arr2.push(r);
+      const arr3 = bins.get(b);
+      if (arr3) arr3.push(r);
       else bins.set(b, [r]);
     }
     for (const [b, rows] of bins) {
@@ -6601,12 +6658,12 @@ function applyRuleToProject(rule, shapes, sheetData) {
     const d = sheetData.get(s.sheet_id);
     if (!d) continue;
     const ring = s.verts_norm.map(([nx, ny]) => [nx * d.imgW, ny * d.imgH]);
-    let arr2 = deductsBySheet.get(s.sheet_id);
-    if (!arr2) {
-      arr2 = [];
-      deductsBySheet.set(s.sheet_id, arr2);
+    let arr3 = deductsBySheet.get(s.sheet_id);
+    if (!arr3) {
+      arr3 = [];
+      deductsBySheet.set(s.sheet_id, arr3);
     }
-    arr2.push(ring);
+    arr3.push(ring);
   }
   const out = [];
   const acceptedBySheet = /* @__PURE__ */ new Map();
@@ -6631,12 +6688,12 @@ function applyRuleToProject(rule, shapes, sheetData) {
         verts_norm: ring.map(([x, y]) => [x / d.imgW, y / d.imgH]),
         area_sf: +areaSf.toFixed(2)
       });
-      let arr2 = acceptedBySheet.get(room.sheet_id);
-      if (!arr2) {
-        arr2 = [];
-        acceptedBySheet.set(room.sheet_id, arr2);
+      let arr3 = acceptedBySheet.get(room.sheet_id);
+      if (!arr3) {
+        arr3 = [];
+        acceptedBySheet.set(room.sheet_id, arr3);
       }
-      arr2.push(ring);
+      arr3.push(ring);
     }
   }
   return out;
@@ -6937,7 +6994,7 @@ function reportJson({ projectName = "", rows = [], bySheet = [], scaleInfo = [],
   const colDefs = (Array.isArray(conditionColumns) ? conditionColumns : []).filter((cc) => cc && typeof cc === "object" && typeof cc.id === "string");
   const attrs = attrsByCond instanceof Map ? attrsByCond : /* @__PURE__ */ new Map();
   return {
-    schema: "opentakeoff.report.v1",
+    schema: REPORT_SCHEMA,
     project_name: projectName || null,
     generated_with: "OpenTakeoff",
     // scale_confirmed (scale gate): false = an agent set this sheet's scale and
@@ -7655,15 +7712,82 @@ function drawMarks(ctx, toCanvas, marks, longEdge) {
   return drawn;
 }
 
+// ../web/src/lib/takeoffDocument.js
+var arr = (v) => Array.isArray(v) ? v : [];
+var obj = (v) => v && typeof v === "object" && !Array.isArray(v) ? v : {};
+var TAKEOFF_DOCUMENT_KEYS = Object.freeze([
+  "schema",
+  "project_name",
+  "units",
+  "client_info",
+  "sheets",
+  "conditions",
+  "condition_columns",
+  "shape_labels",
+  "palette",
+  "shapes",
+  "markups",
+  "rfis",
+  "approvals",
+  "proposals",
+  "condition_edit_proposals",
+  "rules",
+  "sheet_group",
+  "last_group",
+  "sheet_tabs",
+  "stitches",
+  "sheet_levels",
+  "layer_overrides",
+  "provenance_counters"
+]);
+function sheetEntry({ sheet_id, units_per_px, scale_source = void 0, scale_confirmed = void 0 }) {
+  return {
+    sheet_id,
+    units_per_px,
+    ...scale_source ? { scale_source } : {},
+    ...scale_confirmed === false ? { scale_confirmed: false } : {}
+  };
+}
+function buildTakeoffDocument(f = {}) {
+  const conditions = arr(f.conditions);
+  const pinned = arr(f.palette).filter((id) => conditions.some((c) => c && c.id === id));
+  const clientInfo = obj(f.client_info);
+  const hasClient = Object.values(clientInfo).some((v) => v && String(v).trim());
+  const sheetLevels = obj(f.sheet_levels);
+  const layerOverrides = obj(f.layer_overrides);
+  const prov = f.provenance_counters;
+  const hasProv = !!(prov && prov.shapes_deleted && Object.keys(prov.shapes_deleted).length);
+  return {
+    schema: TAKEOFF_SCHEMA,
+    project_name: typeof f.project_name === "string" ? f.project_name : "",
+    ...f.units === "metric" ? { units: "metric" } : {},
+    ...hasClient ? { client_info: clientInfo } : {},
+    sheets: arr(f.sheets),
+    conditions,
+    ...arr(f.condition_columns).length ? { condition_columns: f.condition_columns } : {},
+    ...arr(f.shape_labels).length ? { shape_labels: f.shape_labels } : {},
+    ...pinned.length ? { palette: pinned } : {},
+    shapes: arr(f.shapes),
+    markups: arr(f.markups),
+    rfis: arr(f.rfis),
+    ...arr(f.approvals).length ? { approvals: f.approvals } : {},
+    ...arr(f.proposals).length ? { proposals: f.proposals } : {},
+    ...arr(f.condition_edit_proposals).length ? { condition_edit_proposals: f.condition_edit_proposals } : {},
+    ...arr(f.rules).length ? { rules: f.rules } : {},
+    sheet_group: arr(f.sheet_group),
+    last_group: arr(f.last_group),
+    sheet_tabs: arr(f.sheet_tabs),
+    ...arr(f.stitches).length ? { stitches: f.stitches } : {},
+    ...Object.keys(sheetLevels).length ? { sheet_levels: sheetLevels } : {},
+    ...Object.keys(layerOverrides).length ? { layer_overrides: layerOverrides } : {},
+    ...hasProv ? { provenance_counters: prov } : {}
+  };
+}
+
 // src/session.ts
-var SNAP_CELL = 24;
-var SNAP_TOL = 7;
-var PALETTE = ["#c96442", "#2f7d54", "#2563eb", "#9333ea", "#b8860b", "#0d9488", "#be185d", "#1f2937", "#dc2626", "#0891b2"];
-var HATCH_IDS = ["solid", "diag", "diag2", "cross", "diagdense", "horiz", "vert", "grid", "brick", "plank", "herring", "basket", "checker", "wave", "dots", "speckle", "iso", "honeycomb", "scan", "plus", "circuit", "topo", "woodgrain", "chevron", "pinwheel", "harlequin", "hexagon", "penny", "octagondot", "fleur", "concrete"];
 var mintUuid2 = () => globalThis.crypto && typeof globalThis.crypto.randomUUID === "function" ? globalThis.crypto.randomUUID() : `${Date.now().toString(36)}-${Math.random().toString(36).slice(2, 10)}`;
 var uid = (p) => `${p}-${mintUuid2()}`;
 var nowIso2 = () => (/* @__PURE__ */ new Date()).toISOString();
-var ANN_SCHEMA = "opentakeoff.takeoff_canvas.v1";
 var sanitizeApprovals2 = sanitizeApprovals;
 var applyApprovalCommand2 = applyApprovalCommand;
 var CONTEXT_MIN_LEN_PX = 2;
@@ -8374,13 +8498,14 @@ var Session = class _Session {
   conditionFor(tag) {
     let c = this.conditions.find((x) => x.finish_tag === tag);
     if (!c) {
-      const lc = PALETTE[this.conditions.length % PALETTE.length];
+      const lc = nextPaletteColor(this.conditions.length);
       c = {
         id: uid("cnd"),
+        created_at: nowIso2(),
         finish_tag: tag,
         color: lc,
         fill: lc,
-        hatch: HATCH_IDS[1 + this.conditions.length % (HATCH_IDS.length - 1)],
+        hatch: nextHatchId(this.conditions.length),
         multiplier: 1,
         waste_pct: 0,
         materials: []
@@ -9584,9 +9709,9 @@ var Session = class _Session {
     }
     const tableRegions = /* @__PURE__ */ new Map();
     for (const tb of graph.tables) {
-      const arr2 = tableRegions.get(tb.sheet) ?? [];
-      arr2.push(tb.region);
-      tableRegions.set(tb.sheet, arr2);
+      const arr3 = tableRegions.get(tb.sheet) ?? [];
+      arr3.push(tb.region);
+      tableRegions.set(tb.sheet, arr3);
     }
     const VAL_RE = /^[0-9][0-9,]{0,6}$/;
     const perMark = /* @__PURE__ */ new Map();
@@ -10793,7 +10918,7 @@ var Session = class _Session {
       tag: newTag,
       mintId: (p) => uid(p),
       nowIso: nowIso2,
-      nextHatch: HATCH_IDS[1 + (this.conditions.length + 1) % (HATCH_IDS.length - 1)]
+      nextHatch: nextHatchId(this.conditions.length + 1)
     });
     if (parentPatch) Object.assign(src, parentPatch);
     this.conditions.push(twin);
@@ -11452,41 +11577,18 @@ var Session = class _Session {
   }
   exportPayload() {
     if (!this.docs.size) throw new UserError("No plan loaded \u2014 call load_plan first.");
-    return {
-      schema: ANN_SCHEMA,
+    return buildTakeoffDocument({
       project_name: "",
       units: "imperial",
-      sheets: [...this.sheets.values()].filter((s) => s.upp != null).map((s) => ({
-        sheet_id: s.key,
-        units_per_px: s.upp,
-        // provenance rides the payload (it used to be dropped here): the canvas
-        // hydrates scale_source for its report and scale_confirmed for the
-        // scale gate's confirm affordance — absent = confirmed (pre-flag docs)
-        ...s.scaleSource ? { scale_source: s.scaleSource } : {},
-        ...s.scaleConfirmed === false ? { scale_confirmed: false } : {}
-      })),
+      sheets: [...this.sheets.values()].filter((s) => s.upp != null).map((s) => sheetEntry({ sheet_id: s.key, units_per_px: s.upp, scale_source: s.scaleSource, scale_confirmed: s.scaleConfirmed })),
       conditions: this.conditions,
       shapes: this.shapes,
       markups: this.markups,
-      // approvals ride the payload additively (#176) — present only when any
-      // exist, exactly the canvas buildPayload's convention, so a verdict-free
-      // export stays byte-identical to a pre-#176 one
-      ...this.approvals.length ? { approvals: this.approvals } : {},
-      // RFIs (#364): the panel's own records, tombstones stripped — the app
-      // has no tombstone notion (its delete is a removal), and a withdrawn
-      // question must not resurface there as a stray Void row. Same
-      // present-only-when-any convention as approvals.
-      ...this.liveRfis().length ? { rfis: this.liveRfis() } : {},
-      // proposals (#365): the batches and the pending condition diffs ride
-      // the payload as transport — present only when any exist, so a
-      // proposal-free export stays byte-identical to a pre-#365 one.
-      ...this.proposals.length ? { proposals: this.proposals } : {},
-      ...this.conditionEditProposals.length ? { condition_edit_proposals: this.conditionEditProposals } : {},
-      sheet_group: [],
-      last_group: [],
-      sheet_tabs: [],
-      sheet_levels: {}
-    };
+      rfis: this.liveRfis(),
+      approvals: this.approvals,
+      proposals: this.proposals,
+      condition_edit_proposals: this.conditionEditProposals
+    });
   }
   /** The computed Report document — "opentakeoff.report.v1", the SAME schema
    * and math as the canvas Report's JSON export (web reportJson, totals.js):
@@ -12116,7 +12218,7 @@ var exportDxfOutput = {
 var exportTakeoffOutput = {
   schema: z.string(),
   project_name: z.string(),
-  units: z.string(),
+  units: z.string().optional().describe("Present only for a metric project; absent means imperial \u2014 the app's own diff-only convention"),
   sheets: z.array(z.object({
     sheet_id: z.string(),
     units_per_px: z.number(),
@@ -12148,7 +12250,7 @@ var exportTakeoffOutput = {
   sheet_group: z.array(z.unknown()),
   last_group: z.array(z.unknown()),
   sheet_tabs: z.array(z.unknown()),
-  sheet_levels: z.object({}).passthrough(),
+  sheet_levels: z.object({}).passthrough().optional().describe("Present only when a sheet carries a level label (the app omits it when empty)"),
   proposals: z.array(z.object({ id: z.string(), label: z.string(), rationale: z.string(), created_at: z.string(), withdrawn_at: z.string().optional() }).passthrough()).optional().describe("Proposal batches (#365) \u2014 present only when any exist. Shapes reference them by origin.proposal_id; the canvas shows one Accept per batch"),
   condition_edit_proposals: z.array(z.object({ id: z.string(), condition_id: z.string(), proposed: z.object({}).passthrough(), rationale: z.string(), proposed_at: z.string() }).passthrough()).optional().describe("Pending condition-edit diffs (#365) \u2014 present only when any exist. Nothing on the condition changes until the estimator accepts in the canvas")
 };
@@ -12358,7 +12460,7 @@ var reportMaterialLine = z.object({
   qty: z.number().describe("Computed order quantity")
 }).passthrough();
 var exportReportOutput = {
-  schema: z.literal("opentakeoff.report.v1"),
+  schema: z.literal(REPORT_SCHEMA),
   project_name: z.string().nullable(),
   generated_with: z.string(),
   sheets: z.array(z.object({ sheet_id: z.string(), sheet: z.string(), scale_source: z.string() }).passthrough()).describe("Scale provenance per sheet \u2014 how each scale was set"),
@@ -13467,9 +13569,9 @@ async function buildMarkedSetPdf({ projectName, dark, sheets, shapes, markups, a
   const { PDFDocument, StandardFonts, rgb, degrees, LineCapStyle } = await import("pdf-lib");
   const condById = Object.fromEntries(conditions.map((c) => [c.id, c]));
   const rfiNum = new Map((rfis || []).map((r) => [r.id, r.number]));
-  const byKey = (arr2) => {
+  const byKey = (arr3) => {
     const m = /* @__PURE__ */ new Map();
-    for (const s of arr2) {
+    for (const s of arr3) {
       const a = m.get(s.sheet_id) || [];
       a.push(s);
       m.set(s.sheet_id, a);
@@ -14182,25 +14284,6 @@ async function exportMarkedPdf(session, opts) {
 import path4 from "node:path";
 import { readFile as readFile3 } from "node:fs/promises";
 
-// ../web/src/lib/stamps.js
-var DEFAULT_STAMPS = [
-  { id: "stmp-direction", name: "Plank / tile direction", elements: [
-    { type: "arrow", from: [-0.05, 0], to: [0.05, 0], color: "#1f3fc7", weight: 1.5 }
-  ] },
-  { id: "stmp-seam", name: "Seam direction", elements: [
-    { type: "arrow", from: [-0.05, 0], to: [0.05, 0], color: "#b03a26", line_style: "dashed" }
-  ] },
-  { id: "stmp-origin", name: "Pattern origin", elements: [
-    { type: "bubble", at: [0, 0], r: 0.018, text: "PO", color: "#0d9488" }
-  ] }
-];
-var DEFAULT_STAMP_SETS = [
-  { id: "set-flooring", name: "Flooring shop drawings", stampIds: DEFAULT_STAMPS.map((s) => s.id) }
-];
-
-// ../web/src/lib/store.js
-var ANN_SCHEMA2 = "opentakeoff.takeoff_canvas.v1";
-
 // ../web/src/lib/reviewState.js
 function normalizeAgentReview(shape) {
   return shape?.origin?.actor === "agent" && shape.origin.reviewed == null ? { ...shape, origin: { ...shape.origin, reviewed: false } } : shape;
@@ -14214,12 +14297,12 @@ function parseTakeoffImport(text) {
   } catch {
     throw new Error("Couldn't import takeoff: that file is not valid JSON.");
   }
-  if (!doc || typeof doc !== "object" || Array.isArray(doc) || doc.schema !== ANN_SCHEMA2) {
-    throw new Error(`Couldn't import takeoff: not a takeoff export (expected schema "${ANN_SCHEMA2}" \u2014 the file export_takeoff or the app writes).`);
+  if (!doc || typeof doc !== "object" || Array.isArray(doc) || doc.schema !== TAKEOFF_SCHEMA) {
+    throw new Error(`Couldn't import takeoff: not a takeoff export (expected schema "${TAKEOFF_SCHEMA}" \u2014 the file export_takeoff or the app writes).`);
   }
   return doc;
 }
-var arr = (v) => Array.isArray(v) ? v : [];
+var arr2 = (v) => Array.isArray(v) ? v : [];
 var tagKey = (t) => String(t || "").trim().toUpperCase();
 var freeId = (id, taken) => {
   let n = 2, next = id;
@@ -14228,11 +14311,11 @@ var freeId = (id, taken) => {
 };
 function mergeTakeoffImport(current, imported, knownFiles = null) {
   const cur = current && typeof current === "object" ? current : {};
-  const impShapes = arr(imported.shapes).filter((s) => s && typeof s === "object" && typeof s.sheet_id === "string" && typeof s.id === "string").map(normalizeAgentReview);
-  const impConds = arr(imported.conditions).filter((c) => c && typeof c === "object" && typeof c.id === "string");
-  const localScales = new Map(arr(cur.sheets).filter((s) => typeof s?.sheet_id === "string").map((s) => [s.sheet_id, s.units_per_px]));
-  const incomingScales = new Map(arr(imported.sheets).filter((s) => typeof s?.sheet_id === "string").map((s) => [s.sheet_id, s.units_per_px]));
-  const existingIds = new Set(arr(cur.shapes).map((s) => s.id));
+  const impShapes = arr2(imported.shapes).filter((s) => s && typeof s === "object" && typeof s.sheet_id === "string" && typeof s.id === "string").map(normalizeAgentReview);
+  const impConds = arr2(imported.conditions).filter((c) => c && typeof c === "object" && typeof c.id === "string");
+  const localScales = new Map(arr2(cur.sheets).filter((s) => typeof s?.sheet_id === "string").map((s) => [s.sheet_id, s.units_per_px]));
+  const incomingScales = new Map(arr2(imported.sheets).filter((s) => typeof s?.sheet_id === "string").map((s) => [s.sheet_id, s.units_per_px]));
+  const existingIds = new Set(arr2(cur.shapes).map((s) => s.id));
   for (const s of impShapes) {
     if (existingIds.has(s.id) || s.measure_role === "count") continue;
     const local = localScales.get(s.sheet_id), incoming = incomingScales.get(s.sheet_id);
@@ -14246,20 +14329,20 @@ function mergeTakeoffImport(current, imported, knownFiles = null) {
     return [...new Set(added.map((s) => String(s.sheet_id).split("#")[0]).filter((f) => !known.has(f)))];
   };
   const pendingCount = (shapes) => shapes.filter((s) => s.origin?.reviewed === false).length;
-  if (!arr(cur.shapes).length && !arr(cur.markups).length && !arr(cur.approvals).length && !arr(cur.sheets).some((s) => s?.units_per_px > 0)) {
+  if (!arr2(cur.shapes).length && !arr2(cur.markups).length && !arr2(cur.approvals).length && !arr2(cur.sheets).some((s) => s?.units_per_px > 0)) {
     const payload2 = {
       ...imported,
       shapes: impShapes,
-      ...arr(imported.sheet_tabs).length ? {} : { sheet_tabs: arr(cur.sheet_tabs) },
-      ...arr(imported.sheet_group).length ? {} : { sheet_group: arr(cur.sheet_group) },
-      ...arr(imported.last_group).length ? {} : { last_group: arr(cur.last_group) }
+      ...arr2(imported.sheet_tabs).length ? {} : { sheet_tabs: arr2(cur.sheet_tabs) },
+      ...arr2(imported.sheet_group).length ? {} : { sheet_group: arr2(cur.sheet_group) },
+      ...arr2(imported.last_group).length ? {} : { last_group: arr2(cur.last_group) }
     };
     return {
       payload: payload2,
-      note: { replaced: true, shapes_added: impShapes.length, shapes_pending: pendingCount(impShapes), conditions_merged: 0, conditions_added: impConds.length, scales_adopted: arr(imported.sheets).length, unknown_files: unknownFiles(impShapes) }
+      note: { replaced: true, shapes_added: impShapes.length, shapes_pending: pendingCount(impShapes), conditions_merged: 0, conditions_added: impConds.length, scales_adopted: arr2(imported.sheets).length, unknown_files: unknownFiles(impShapes) }
     };
   }
-  const conditions = [...arr(cur.conditions)];
+  const conditions = [...arr2(cur.conditions)];
   const byTag = new Map(conditions.map((c) => [tagKey(c.finish_tag), c.id]));
   const condIds = new Set(conditions.map((c) => c.id));
   const condMap = /* @__PURE__ */ new Map();
@@ -14295,23 +14378,23 @@ function mergeTakeoffImport(current, imported, knownFiles = null) {
       conditions[i] = next;
     }
   }
-  const shapeIds = new Set(arr(cur.shapes).map((s) => s.id));
+  const shapeIds = new Set(arr2(cur.shapes).map((s) => s.id));
   const addedShapes = impShapes.filter((s) => !shapeIds.has(s.id)).map((s) => condMap.has(s.condition_id) && condMap.get(s.condition_id) !== s.condition_id ? { ...s, condition_id: condMap.get(s.condition_id) } : s);
-  const markupIds = new Set(arr(cur.markups).map((m) => m?.id).filter(Boolean));
-  const addedMarkups = arr(imported.markups).filter((m) => m && typeof m === "object" && (!m.id || !markupIds.has(m.id)));
-  const rfiIds = new Set(arr(cur.rfis).map((r) => r?.id).filter(Boolean));
-  const addedRfis = arr(imported.rfis).filter((r) => r && typeof r === "object" && r.id && !rfiIds.has(r.id));
-  const proposalIds = new Set(arr(cur.proposals).map((p) => p?.id).filter(Boolean));
-  const addedProposals = arr(imported.proposals).filter((p) => p && typeof p === "object" && typeof p.id === "string" && !proposalIds.has(p.id));
-  const editIds = new Set(arr(cur.condition_edit_proposals).map((p) => p?.id).filter(Boolean));
+  const markupIds = new Set(arr2(cur.markups).map((m) => m?.id).filter(Boolean));
+  const addedMarkups = arr2(imported.markups).filter((m) => m && typeof m === "object" && (!m.id || !markupIds.has(m.id)));
+  const rfiIds = new Set(arr2(cur.rfis).map((r) => r?.id).filter(Boolean));
+  const addedRfis = arr2(imported.rfis).filter((r) => r && typeof r === "object" && r.id && !rfiIds.has(r.id));
+  const proposalIds = new Set(arr2(cur.proposals).map((p) => p?.id).filter(Boolean));
+  const addedProposals = arr2(imported.proposals).filter((p) => p && typeof p === "object" && typeof p.id === "string" && !proposalIds.has(p.id));
+  const editIds = new Set(arr2(cur.condition_edit_proposals).map((p) => p?.id).filter(Boolean));
   const takenTags = new Set(conditions.map((c) => tagKey(c.finish_tag)));
-  const addedEdits = arr(imported.condition_edit_proposals).filter((p) => p && typeof p === "object" && typeof p.id === "string" && !editIds.has(p.id) && p.proposed && typeof p.proposed === "object").map((p) => condMap.has(p.condition_id) ? { ...p, condition_id: condMap.get(p.condition_id) } : p).filter((p) => condIds.has(p.condition_id)).filter((p) => p.proposed.finish_tag === void 0 || !takenTags.has(tagKey(p.proposed.finish_tag)) || tagKey(p.proposed.finish_tag) === tagKey(conditions.find((c) => c.id === p.condition_id)?.finish_tag)).filter((p) => !arr(cur.condition_edit_proposals).some((q) => q?.condition_id === p.condition_id));
-  const approvalIds = new Set(arr(cur.approvals).map((a) => a?.id).filter(Boolean));
+  const addedEdits = arr2(imported.condition_edit_proposals).filter((p) => p && typeof p === "object" && typeof p.id === "string" && !editIds.has(p.id) && p.proposed && typeof p.proposed === "object").map((p) => condMap.has(p.condition_id) ? { ...p, condition_id: condMap.get(p.condition_id) } : p).filter((p) => condIds.has(p.condition_id)).filter((p) => p.proposed.finish_tag === void 0 || !takenTags.has(tagKey(p.proposed.finish_tag)) || tagKey(p.proposed.finish_tag) === tagKey(conditions.find((c) => c.id === p.condition_id)?.finish_tag)).filter((p) => !arr2(cur.condition_edit_proposals).some((q) => q?.condition_id === p.condition_id));
+  const approvalIds = new Set(arr2(cur.approvals).map((a) => a?.id).filter(Boolean));
   const addedApprovals = sanitizeApprovals(imported.approvals).filter((a) => !approvalIds.has(a.id));
-  const sheets = [...arr(cur.sheets)];
+  const sheets = [...arr2(cur.sheets)];
   const scaled = new Set(sheets.map((s) => s?.sheet_id));
   let scalesAdopted = 0;
-  for (const s of arr(imported.sheets)) {
+  for (const s of arr2(imported.sheets)) {
     if (s && typeof s === "object" && s.sheet_id && s.units_per_px && !scaled.has(s.sheet_id)) {
       sheets.push(s);
       scaled.add(s.sheet_id);
@@ -14321,12 +14404,12 @@ function mergeTakeoffImport(current, imported, knownFiles = null) {
   const payload = {
     ...cur,
     conditions,
-    shapes: [...arr(cur.shapes), ...addedShapes],
-    markups: [...arr(cur.markups), ...addedMarkups],
-    ...addedRfis.length ? { rfis: [...arr(cur.rfis), ...addedRfis] } : {},
-    ...addedProposals.length ? { proposals: [...arr(cur.proposals), ...addedProposals] } : {},
-    ...addedEdits.length ? { condition_edit_proposals: [...arr(cur.condition_edit_proposals), ...addedEdits] } : {},
-    ...addedApprovals.length ? { approvals: [...arr(cur.approvals), ...addedApprovals] } : {},
+    shapes: [...arr2(cur.shapes), ...addedShapes],
+    markups: [...arr2(cur.markups), ...addedMarkups],
+    ...addedRfis.length ? { rfis: [...arr2(cur.rfis), ...addedRfis] } : {},
+    ...addedProposals.length ? { proposals: [...arr2(cur.proposals), ...addedProposals] } : {},
+    ...addedEdits.length ? { condition_edit_proposals: [...arr2(cur.condition_edit_proposals), ...addedEdits] } : {},
+    ...addedApprovals.length ? { approvals: [...arr2(cur.approvals), ...addedApprovals] } : {},
     sheets
   };
   return {
@@ -15055,7 +15138,7 @@ No review gate: the pencil-not-ink rule exists to stop an agent inventing geomet
 import { ResourceTemplate } from "@modelcontextprotocol/sdk/server/mcp.js";
 
 // src/wiki.generated.ts
-var WIKI_VERSION = "0.9.84";
+var WIKI_VERSION = "0.9.86";
 var WIKI_PAGES = [
   {
     "key": "index",
@@ -15118,8 +15201,8 @@ var WIKI_PAGES = [
     "uri": "takeoff://wiki/repo-guide",
     "title": "Working on the repository",
     "source": "docs/wiki/repo-guide.md",
-    "source_sha256": "f7ed92fc2e49cbb1e00b7e90335120dccd226bd837d1d7064c38bc9e74500cf7",
-    "text": "# Working on the repository\n\nOpenTakeoff is a **client-only React app**: a PDF construction-takeoff canvas for flooring (useful for any trade). No backend, no database, no auth\u2014everything runs and persists in the browser. Apache-2.0. (For the one-page project pitch and vision, see [`AGENT_BRIEF.md`](https://github.com/Kentucky-ai/opentakeoff/blob/main/AGENT_BRIEF.md); for capability \u2192 code mapping, see [`FEATURES.md`](https://github.com/Kentucky-ai/opentakeoff/blob/main/FEATURES.md).)\n\n## Run / build / check\n\n```bash\ncd web\nnvm use          # web/.nvmrc matches the root .nvmrc used by CI\nnpm install\nnpm run dev      # http://localhost:5173 \u2014 hot reload\nnpm test         # node:test over the pure geometry + totals math (test/*.test.ts)\nnpm run build    # \u2192 web/dist/ (static output; this is what Netlify deploys)\nnpm run check    # web typecheck + lint + test + benchmark + build\n```\n\n## Shipping \u2014 the required steps, every change\n\n`main` is protected on GitHub by a ruleset (PR-only, one approving review,\ngreen `web` check\u2014the repo owner has a standing bypass\nas the solo maintainer). **Merging to `main` deploys to production**\n(<https://opentakeoff.kentucky-ai.com>)\u2014Netlify's own git integration builds\nthe merge commit and publishes it. `netlify.toml` holds the whole recipe\n(`base = \"web\"`, `command = npm run build`, `publish = \"dist\"`), so the deploy\nruns a fresh build from the merged source; nothing is uploaded from CI.\n\n`.github/workflows/deploy.yml` used to do this by publishing `web/dist` with\n`--no-build`, and it was **deleted on 2026-07-13 in `e701f1a`** (\"deploys here\nare manual CLI; it fails on every push without the fork's secrets\"). Current checks and release automation live in\n[the workflow directory](https://github.com/Kentucky-ai/opentakeoff/blob/main/.github/workflows); `publish-mcp.yml` fires on\nan `mcp-v*` tag. The old line here said \"Netlify never builds anything\nitself\", which is now exactly backwards\u2014Netlify is the only thing that\nbuilds production. **Merge = deploy either way: that part has never changed.**\n\n> **This is the canonical `Kentucky-ai/opentakeoff` repo\u2014production is\n> <https://opentakeoff.kentucky-ai.com>, nothing else.** A downstream fork\n> (`knmurphy/opentakeoff`) tracks this repo as its own upstream and deploys\n> separately to `takeoff.345flooring.com`\u2014that URL belongs to *that* fork,\n> not this repo. If you see `takeoff.345flooring.com` referenced elsewhere in\n> this repo's docs, it's either an example value in the optional cloud-mode\n> guides or leftover content from that fork's docs that rode along in a\n> wholesale history merge (2026-07-13)\u2014treat it as describing the\n> *downstream* fork's deployment, not this one's.\n\nSo:\n\n1. **Branch first**\u2014never commit on `main`: `git checkout -b <topic>`.\n2. **`npm run check` before pushing** (in `web/`). It covers the independent web\n   job on Node 24; a green web check does not replace the MCP, protocol, docs,\n   capture or optional-server jobs.\n3. **Include review evidence in every PR**: screenshots or a short video for\n   visible changes; measured expected-versus-observed stats and reproducible\n   commands for engine/tool changes. Link the actual checks or evidence. This\n   repository is also the maintainer's public portfolio: keep claims verifiable\n   and private project data out of public artifacts.\n4. **Open a PR** and wait for the `web` check to pass. Don't merge red or\n   pending.\n5. **Squash-merge with branch delete**\n   (`gh pr merge <n> --squash --delete-branch`), then\n   `git checkout main && git pull --ff-only` and delete the local branch\n   (`git branch -D <topic>`\u2014squash merges need `-D`).\n6. **Remember a merge is a deploy.** Don't merge work you haven't verified in\n   the running app.\n\nThe tests cover the pure math (`web/test/geometry.test.ts`, `web/test/totals.test.ts`); the canvas itself is verified by hand\u2014**Vite does not flag undefined identifiers in JSX**, so grep for your new identifiers after editing and load the app once before you call it done. The bundled sample plan (`web/public/demo/`, wired to the \"Load sample plan\" button) is the fastest end-to-end check: load it, press `A`, trace a room, open Report.\n\n## Where things live\n\n| Concern | Path |\n|---|---|\n| **The canvas\u201490% of the app** | `web/src/pages/TakeoffCanvas.jsx` (one large, deliberately monolithic component) |\n| Geometry: vector extraction, One-Click flood fill, vertex snap | `web/src/lib/oneclick.ts` |\n| Sheet/page helpers, scale detection | `web/src/lib/sheets.ts` |\n| Totals and materials math (waste, SY, coverage \u2192 order qty) | `web/src/lib/totals.js` |\n| Persistence (IndexedDB + localStorage) | `web/src/lib/store.js` |\n| PDF/image/zip ingest | `web/src/lib/ingest.js` |\n| Icon set | `web/src/brand/icons.jsx` |\n| Design tokens (colors, spacing\u2014the source of truth) | `web/src/styles/tokens.css` |\n| Sheet gallery / report UI | `web/src/components/` |\n| Pure-math tests (node:test) | `web/test/` |\n| **Optional AI backend** (pluggable adapter: scale/room/finish suggestions) | `server/`\u2014`app.py` + `adapters/base.py` (interface) + `adapters/heuristic.py` (default, no model) |\n\n## How the canvas works (the mental model)\n\n- Each open sheet renders into a `<canvas>` bitmap; **all takeoff geometry is an SVG overlay** on top; pan/zoom is a single CSS transform on the stage div, written imperatively (`tfRef` \u2192 `style.transform`) to avoid React re-renders per frame.\n- Coordinates: pointer events (client px) \u2192 `toImage()` \u2192 **stage px**; committed shapes store **normalized [0..1] vertices per sheet** (`verts_norm`), so quantities survive re-renders and zoom.\n- Cursor-following UI (crosshair hairlines, readout chip, rubber band) updates through **direct DOM writes in `moveCrosshair`**\u2014never React state per mousemove. Keep it that way.\n- Angle snapping: `angleSnap()` locks in-progress segments to the 45\xB0 family; endpoint snap (`nearestSnap` over a spatial hash of PDF vector endpoints) takes priority. The committed click reuses the same locked point (`angleRef`).\n- Past ~1.15\xD7 zoom, a **detail-view canvas** re-renders the visible region from PDF vectors at the current zoom (crispness); the base bitmap stays as first paint.\n- pdf.js rendering schedules work on `requestAnimationFrame`\u2014a fully hidden/occluded window will pause mid-render by design; it resumes when visible.\n\n## Conventions\n\n- **SVG presentation attributes take literal colors** (CSS vars don't resolve there): cobalt `#1f3fc7`, danger `#b03a26`, positive `#1f6b4a`\u2014centralized in `web/src/lib/ui.js` (`SVG`, with HUD-dark counterparts through `svgAccent(isDark)`). DOM/HTML chrome may use `var(--\u2026)` from `tokens.css`.\n- Condition palettes (`PALETTE` in `web/src/components/hatches.jsx`, the seeded condition colors in `FLOORING_DEFAULTS` in `web/src/lib/canvasConstants.js`, and the mirrored copies in `mcp/src/session.ts`) are **user data**\u2014don't re-theme them.\n- Waste applies only in the report (order quantities), never to live measured numbers.\n- Keyboard shortcuts are single letters registered on `window` (see `docs/USER_GUIDE.md` \xA715); toolbar menus pause them through `menuDepthRef`.\n- Brand voice: **precision instrument** (2026-08 overhaul). Light theme = \"ice\": bright white surfaces on a cool field, cool-slate neutrals, cobalt the one saturated thing. Dark theme = \"HUD\": true-black cockpit, electric blue `#3f8cff`, phosphor `--glow` on exactly five elements (active tool face, status verb, hero quantity, primary CTA, calibration dot). Square corners; the single sanctioned radius is `--r-1` on floating chrome. Mono tabular numerals on every readout. Drafting-table language stays. No vendor mimicry.\n- Layout/spacing/type come from the token scales in `tokens.css` (`--sp-*`, `--fs-*`, `--ctl-*`); zIndex comes from the `Z` ladder in `web/src/lib/ui.js`. No new magic numbers.\n\n## Docs to keep in sync when you change behavior\n\nThe draft Takeoff Protocol lives in [`protocol/`](https://github.com/Kentucky-ai/opentakeoff/blob/main/protocol/README.md). When\nchanging persisted fields, consult its [inventory](https://github.com/Kentucky-ai/opentakeoff/blob/main/protocol/INVENTORY.md) and\n[compatibility report](https://github.com/Kentucky-ai/opentakeoff/blob/main/protocol/ACADEMY_COMPATIBILITY.md). Run\n`npm run check --prefix protocol` after installing web, MCP, and protocol\ndependencies. Refresh schema references with\n`node protocol/scripts/check-docs.mjs --write`. The draft does not authorize\nchanging writer formats or adopting a migration.\n\n1. `README.md` (Features + \"What's in the box\")\n2. `docs/USER_GUIDE.md` (shortcuts + the relevant section)\n3. `CHANGELOG.md`\n\nTouching the **MCP server** also requires checking its runtime guidance and\ngenerated references:\n\n4. `mcp/src/tools.ts`\u2014the tool's own `description` is part of its integration.\n   An MCP client reads it at runtime. Initialize instructions and packaged wiki\n   resources are also agent-facing surfaces, so keep them consistent with the\n   runtime behavior.\n5. `mcp/server.ts`\u2014the `instructions` block sent at `initialize`. This is the\n   decision tree every client receives before its first call. A new *verb* does\n   not belong here; a new *step in the standard finish* does. Packaged wiki\n   resources are another deliberate agent-facing surface; update their source\n   pages and run the wiki/resource checks when their guidance changes.\n6. `mcp/README.md` (the tool table) and `docs/MCP.md` (the reach-for-it ordering,\n   the example session, and the tool count in its opening line). A new tool also\n   needs a row in `mcp/src/staging.ts`'s `TOOL_STAGES`\u2014the four lists must\n   partition the tool set exactly, and a test fails CI if one doesn't.\n   `npm run check:tool-count --prefix mcp` checks default, gated and setup counts across the current entry points,\n   reference-table coverage, and [`docs/MCP_TOOL_INDEX.md`](takeoff://wiki/tool-index)\n   against runtime schemas. Add `-- --write` to regenerate counts and the index. If the\n   change alters *doctrine* rather than adding a verb\u2014what withholds, what\n   refuses, what has no agent verb\u2014it belongs in `docs/AGENT_GUIDE.md` too,\n   and the tool count appears there and in `docs/USER_GUIDE.md` \xA714.\n7. **Version fields must agree.** The MCP source version is\n   in `mcp/package.json`; its lockfile has two matching fields, while\n   `mcp/server.json` and `web/public/.well-known/mcp.json` each have two. The\n   version checker checks all seven MCP fields. Separately, it checks the web\n   package version against the two root-version fields in its lockfile.\n   Check current `origin/main` and existing tags before\n   reserving a number. The web package has its own version; it is not the MCP\n   release number.\n8. `npm run check:tool-count --prefix mcp` runs version agreement before checking\n   counts and the schema-backed inventory. With `-- --write`, it refuses version\n   mismatches before changing generated documents; update release metadata\n   explicitly, then regenerate. Run `npm run check:versions --prefix mcp` for the\n   version check alone. Wiki content has its own generator:\n   `npm run check:wiki --prefix mcp -- --write`. Review generated diffs and run\n   both checks without `--write` afterward.\n\nArchitecture rather than behavior\u2014what MCP is versus what the `/ai` sandbox\nis, and why the server imports the web engine in-process\u2014lives at the end of\n[`docs/MCP.md`](https://github.com/Kentucky-ai/opentakeoff/blob/main/docs/MCP.md) (\"Where this sits\") and in\n[`server/README.md`](https://github.com/Kentucky-ai/opentakeoff/blob/main/server/README.md).\n\n## The doc set, and who each one is for\n\nFour documents carry the product, and they're deliberately split by audience\u2014don't\nanswer an estimator's question in the agent manual or vice versa:\n\n| Document | Audience | What belongs in it |\n|---|---|---|\n| [`README.md`](https://github.com/Kentucky-ai/opentakeoff/blob/main/README.md) | everyone, ~60 seconds | what this is, the three doors, what's in the box |\n| [`docs/USER_GUIDE.md`](https://github.com/Kentucky-ai/opentakeoff/blob/main/docs/USER_GUIDE.md) | the estimator at the canvas | every shipped UI behavior, the working order on a real bid, the glossary |\n| [`docs/AGENT_GUIDE.md`](https://github.com/Kentucky-ai/opentakeoff/blob/main/docs/AGENT_GUIDE.md) | an agent driving the engine | the operating model, the standard finish, withheld doctrine, staging, refusal\u2192next-move |\n| [`mcp/README.md`](https://github.com/Kentucky-ai/opentakeoff/blob/main/mcp/README.md) | an agent's integrator | tool-by-tool reference, resources, coordinate contract, limits |\n\nAll of them follow one house style\u2014the Apple Style Guide, with the rules that\nactually come up written out in [`CONTRIBUTING.md`](https://github.com/Kentucky-ai/opentakeoff/blob/main/CONTRIBUTING.md#docs-house-style).\nInterface text quoted in a doc is copied from the code verbatim, so changing a\nmessage means changing it in both places.\n\n[`AGENT_BRIEF.md`](https://github.com/Kentucky-ai/opentakeoff/blob/main/AGENT_BRIEF.md) is the one-page orientation that routes to\nall four. A behavior change usually touches two of them; a new MCP tool touches\nthree plus this file's sync list above.\n"
+    "source_sha256": "9ef949a3dca1f992a005764a825ca86b5f4da1fd631e9eed1011b8d6fe967ab3",
+    "text": "# Working on the repository\n\nOpenTakeoff is a **client-only React app**: a PDF construction-takeoff canvas for flooring (useful for any trade). No backend, no database, no auth\u2014everything runs and persists in the browser. Apache-2.0. (For the one-page project pitch and vision, see [`AGENT_BRIEF.md`](https://github.com/Kentucky-ai/opentakeoff/blob/main/AGENT_BRIEF.md); for capability \u2192 code mapping, see [`FEATURES.md`](https://github.com/Kentucky-ai/opentakeoff/blob/main/FEATURES.md).)\n\n## Run / build / check\n\n```bash\ncd web\nnvm use          # web/.nvmrc matches the root .nvmrc used by CI\nnpm install\nnpm run dev      # http://localhost:5173 \u2014 hot reload\nnpm test         # node:test over the pure geometry + totals math (test/*.test.ts)\nnpm run build    # \u2192 web/dist/ (static output; this is what Netlify deploys)\nnpm run check    # web typecheck + lint + test + benchmark + build\n```\n\n## Shipping \u2014 the required steps, every change\n\n`main` is protected on GitHub by a ruleset (PR-only, one approving review,\ngreen `web` check\u2014the repo owner has a standing bypass\nas the solo maintainer). **Merging to `main` deploys to production**\n(<https://opentakeoff.kentucky-ai.com>)\u2014Netlify's own git integration builds\nthe merge commit and publishes it. `netlify.toml` holds the whole recipe\n(`base = \"web\"`, `command = npm run build`, `publish = \"dist\"`), so the deploy\nruns a fresh build from the merged source; nothing is uploaded from CI.\n\n`.github/workflows/deploy.yml` used to do this by publishing `web/dist` with\n`--no-build`, and it was **deleted on 2026-07-13 in `e701f1a`** (\"deploys here\nare manual CLI; it fails on every push without the fork's secrets\"). Current checks and release automation live in\n[the workflow directory](https://github.com/Kentucky-ai/opentakeoff/blob/main/.github/workflows); `publish-mcp.yml` fires on\nan `mcp-v*` tag. The old line here said \"Netlify never builds anything\nitself\", which is now exactly backwards\u2014Netlify is the only thing that\nbuilds production. **Merge = deploy either way: that part has never changed.**\n\n> **This is the canonical `Kentucky-ai/opentakeoff` repo\u2014production is\n> <https://opentakeoff.kentucky-ai.com>, nothing else.** A downstream fork\n> (`knmurphy/opentakeoff`) tracks this repo as its own upstream and deploys\n> separately to `takeoff.345flooring.com`\u2014that URL belongs to *that* fork,\n> not this repo. If you see `takeoff.345flooring.com` referenced elsewhere in\n> this repo's docs, it's either an example value in the optional cloud-mode\n> guides or leftover content from that fork's docs that rode along in a\n> wholesale history merge (2026-07-13)\u2014treat it as describing the\n> *downstream* fork's deployment, not this one's.\n\nSo:\n\n1. **Branch first**\u2014never commit on `main`: `git checkout -b <topic>`.\n2. **`npm run check` before pushing** (in `web/`). It covers the independent web\n   job on Node 24; a green web check does not replace the MCP, protocol, docs,\n   capture or optional-server jobs.\n3. **Include review evidence in every PR**: screenshots or a short video for\n   visible changes; measured expected-versus-observed stats and reproducible\n   commands for engine/tool changes. Link the actual checks or evidence. This\n   repository is also the maintainer's public portfolio: keep claims verifiable\n   and private project data out of public artifacts.\n4. **Open a PR** and wait for the `web` check to pass. Don't merge red or\n   pending.\n5. **Squash-merge with branch delete**\n   (`gh pr merge <n> --squash --delete-branch`), then\n   `git checkout main && git pull --ff-only` and delete the local branch\n   (`git branch -D <topic>`\u2014squash merges need `-D`).\n6. **Remember a merge is a deploy.** Don't merge work you haven't verified in\n   the running app.\n\nThe tests cover the pure math (`web/test/geometry.test.ts`, `web/test/totals.test.ts`); the canvas itself is verified by hand\u2014**Vite does not flag undefined identifiers in JSX**, so grep for your new identifiers after editing and load the app once before you call it done. The bundled sample plan (`web/public/demo/`, wired to the \"Load sample plan\" button) is the fastest end-to-end check: load it, press `A`, trace a room, open Report.\n\n## Where things live\n\n| Concern | Path |\n|---|---|\n| **The canvas\u201490% of the app** | `web/src/pages/TakeoffCanvas.jsx` (one large, deliberately monolithic component) |\n| Geometry: vector extraction, One-Click flood fill, vertex snap | `web/src/lib/oneclick.ts` |\n| Sheet/page helpers, scale detection | `web/src/lib/sheets.ts` |\n| Totals and materials math (waste, SY, coverage \u2192 order qty) | `web/src/lib/totals.js` |\n| Persistence (IndexedDB + localStorage) | `web/src/lib/store.js` |\n| PDF/image/zip ingest | `web/src/lib/ingest.js` |\n| Icon set | `web/src/brand/icons.jsx` |\n| Design tokens (colors, spacing\u2014the source of truth) | `web/src/styles/tokens.css` |\n| Sheet gallery / report UI | `web/src/components/` |\n| Pure-math tests (node:test) | `web/test/` |\n| **Optional AI backend** (pluggable adapter: scale/room/finish suggestions) | `server/`\u2014`app.py` + `adapters/base.py` (interface) + `adapters/heuristic.py` (default, no model) |\n\n## How the canvas works (the mental model)\n\n- Each open sheet renders into a `<canvas>` bitmap; **all takeoff geometry is an SVG overlay** on top; pan/zoom is a single CSS transform on the stage div, written imperatively (`tfRef` \u2192 `style.transform`) to avoid React re-renders per frame.\n- Coordinates: pointer events (client px) \u2192 `toImage()` \u2192 **stage px**; committed shapes store **normalized [0..1] vertices per sheet** (`verts_norm`), so quantities survive re-renders and zoom.\n- Cursor-following UI (crosshair hairlines, readout chip, rubber band) updates through **direct DOM writes in `moveCrosshair`**\u2014never React state per mousemove. Keep it that way.\n- Angle snapping: `angleSnap()` locks in-progress segments to the 45\xB0 family; endpoint snap (`nearestSnap` over a spatial hash of PDF vector endpoints) takes priority. The committed click reuses the same locked point (`angleRef`).\n- Past ~1.15\xD7 zoom, a **detail-view canvas** re-renders the visible region from PDF vectors at the current zoom (crispness); the base bitmap stays as first paint.\n- pdf.js rendering schedules work on `requestAnimationFrame`\u2014a fully hidden/occluded window will pause mid-render by design; it resumes when visible.\n\n## Conventions\n\n- **SVG presentation attributes take literal colors** (CSS vars don't resolve there): cobalt `#1f3fc7`, danger `#b03a26`, positive `#1f6b4a`\u2014centralized in `web/src/lib/ui.js` (`SVG`, with HUD-dark counterparts through `svgAccent(isDark)`). DOM/HTML chrome may use `var(--\u2026)` from `tokens.css`.\n- Condition palettes (`PALETTE` and `HATCH_IDS` in `web/src/lib/takeoffConstants.ts`, which both the canvas and `mcp/src/session.ts` import, and the seeded condition colors in `FLOORING_DEFAULTS` in `web/src/lib/canvasConstants.js`) are **user data**\u2014don't re-theme them. The same module owns the takeoff and report schema ids, `RENDER_SCALE` and the snap constants; parity tests in `web/test`, `mcp/test` and `protocol/test` fail if a copy reappears.\n- Waste applies only in the report (order quantities), never to live measured numbers.\n- Keyboard shortcuts are single letters registered on `window` (see `docs/USER_GUIDE.md` \xA715); toolbar menus pause them through `menuDepthRef`.\n- Brand voice: **precision instrument** (2026-08 overhaul). Light theme = \"ice\": bright white surfaces on a cool field, cool-slate neutrals, cobalt the one saturated thing. Dark theme = \"HUD\": true-black cockpit, electric blue `#3f8cff`, phosphor `--glow` on exactly five elements (active tool face, status verb, hero quantity, primary CTA, calibration dot). Square corners; the single sanctioned radius is `--r-1` on floating chrome. Mono tabular numerals on every readout. Drafting-table language stays. No vendor mimicry.\n- Layout/spacing/type come from the token scales in `tokens.css` (`--sp-*`, `--fs-*`, `--ctl-*`); zIndex comes from the `Z` ladder in `web/src/lib/ui.js`. No new magic numbers.\n\n## Docs to keep in sync when you change behavior\n\nThe draft Takeoff Protocol lives in [`protocol/`](https://github.com/Kentucky-ai/opentakeoff/blob/main/protocol/README.md). When\nchanging persisted fields, consult its [inventory](https://github.com/Kentucky-ai/opentakeoff/blob/main/protocol/INVENTORY.md) and\n[compatibility report](https://github.com/Kentucky-ai/opentakeoff/blob/main/protocol/ACADEMY_COMPATIBILITY.md). Run\n`npm run check --prefix protocol` after installing web, MCP, and protocol\ndependencies. Refresh schema references with\n`node protocol/scripts/check-docs.mjs --write`. The draft does not authorize\nchanging writer formats or adopting a migration.\n\n1. `README.md` (Features + \"What's in the box\")\n2. `docs/USER_GUIDE.md` (shortcuts + the relevant section)\n3. `CHANGELOG.md`\n\nTouching the **MCP server** also requires checking its runtime guidance and\ngenerated references:\n\n4. `mcp/src/tools.ts`\u2014the tool's own `description` is part of its integration.\n   An MCP client reads it at runtime. Initialize instructions and packaged wiki\n   resources are also agent-facing surfaces, so keep them consistent with the\n   runtime behavior.\n5. `mcp/server.ts`\u2014the `instructions` block sent at `initialize`. This is the\n   decision tree every client receives before its first call. A new *verb* does\n   not belong here; a new *step in the standard finish* does. Packaged wiki\n   resources are another deliberate agent-facing surface; update their source\n   pages and run the wiki/resource checks when their guidance changes.\n6. `mcp/README.md` (the tool table) and `docs/MCP.md` (the reach-for-it ordering,\n   the example session, and the tool count in its opening line). A new tool also\n   needs a row in `mcp/src/staging.ts`'s `TOOL_STAGES`\u2014the four lists must\n   partition the tool set exactly, and a test fails CI if one doesn't.\n   `npm run check:tool-count --prefix mcp` checks default, gated and setup counts across the current entry points,\n   reference-table coverage, and [`docs/MCP_TOOL_INDEX.md`](takeoff://wiki/tool-index)\n   against runtime schemas. Add `-- --write` to regenerate counts and the index. If the\n   change alters *doctrine* rather than adding a verb\u2014what withholds, what\n   refuses, what has no agent verb\u2014it belongs in `docs/AGENT_GUIDE.md` too,\n   and the tool count appears there and in `docs/USER_GUIDE.md` \xA714.\n7. **Version fields must agree.** The MCP source version is\n   in `mcp/package.json`; its lockfile has two matching fields, while\n   `mcp/server.json` and `web/public/.well-known/mcp.json` each have two. The\n   version checker checks all seven MCP fields. Separately, it checks the web\n   package version against the two root-version fields in its lockfile.\n   Check current `origin/main` and existing tags before\n   reserving a number. The web package has its own version; it is not the MCP\n   release number.\n8. `npm run check:tool-count --prefix mcp` runs version agreement before checking\n   counts and the schema-backed inventory. With `-- --write`, it refuses version\n   mismatches before changing generated documents; update release metadata\n   explicitly, then regenerate. Run `npm run check:versions --prefix mcp` for the\n   version check alone. Wiki content has its own generator:\n   `npm run check:wiki --prefix mcp -- --write`. Review generated diffs and run\n   both checks without `--write` afterward.\n\nArchitecture rather than behavior\u2014what MCP is versus what the `/ai` sandbox\nis, and why the server imports the web engine in-process\u2014lives at the end of\n[`docs/MCP.md`](https://github.com/Kentucky-ai/opentakeoff/blob/main/docs/MCP.md) (\"Where this sits\") and in\n[`server/README.md`](https://github.com/Kentucky-ai/opentakeoff/blob/main/server/README.md).\n\n## The doc set, and who each one is for\n\nFour documents carry the product, and they're deliberately split by audience\u2014don't\nanswer an estimator's question in the agent manual or vice versa:\n\n| Document | Audience | What belongs in it |\n|---|---|---|\n| [`README.md`](https://github.com/Kentucky-ai/opentakeoff/blob/main/README.md) | everyone, ~60 seconds | what this is, the three doors, what's in the box |\n| [`docs/USER_GUIDE.md`](https://github.com/Kentucky-ai/opentakeoff/blob/main/docs/USER_GUIDE.md) | the estimator at the canvas | every shipped UI behavior, the working order on a real bid, the glossary |\n| [`docs/AGENT_GUIDE.md`](https://github.com/Kentucky-ai/opentakeoff/blob/main/docs/AGENT_GUIDE.md) | an agent driving the engine | the operating model, the standard finish, withheld doctrine, staging, refusal\u2192next-move |\n| [`mcp/README.md`](https://github.com/Kentucky-ai/opentakeoff/blob/main/mcp/README.md) | an agent's integrator | tool-by-tool reference, resources, coordinate contract, limits |\n\nAll of them follow one house style\u2014the Apple Style Guide, with the rules that\nactually come up written out in [`CONTRIBUTING.md`](https://github.com/Kentucky-ai/opentakeoff/blob/main/CONTRIBUTING.md#docs-house-style).\nInterface text quoted in a doc is copied from the code verbatim, so changing a\nmessage means changing it in both places.\n\n[`AGENT_BRIEF.md`](https://github.com/Kentucky-ai/opentakeoff/blob/main/AGENT_BRIEF.md) is the one-page orientation that routes to\nall four. A behavior change usually touches two of them; a new MCP tool touches\nthree plus this file's sync list above.\n"
   },
   {
     "key": "tool-index",
@@ -15493,7 +15576,7 @@ function nameTheStageInRefusals(server) {
 // package.json
 var package_default = {
   name: "opentakeoff-mcp",
-  version: "0.9.84",
+  version: "0.9.86",
   mcpName: "io.github.Kentucky-ai/opentakeoff",
   type: "module",
   description: "OpenTakeoff MCP server \u2014 drive the takeoff engine from your MCP client over stdio.",
@@ -15513,7 +15596,7 @@ var package_default = {
     "check:wiki": "node scripts/check-wiki.mjs",
     "check:versions": "node ../scripts/check-version-consistency.mjs",
     "check:tool-count": "node --import tsx scripts/check-tool-count.mjs",
-    test: "node --import tsx --test test/checkers.test.ts test/conformance.test.ts test/context.test.ts test/dxf.test.ts test/e2e.test.ts test/gate.test.ts test/graphEvalRowsym.test.ts test/labels.test.ts test/mep.test.ts test/overlap.test.ts test/parity.test.ts test/proposals.test.ts test/raster.test.ts test/resources.test.ts test/safewrite.test.ts test/scalewarn.test.ts test/scope.test.ts test/session.test.ts test/staging.test.ts test/sweepguard.test.ts test/tools.test.ts test/transitions.test.ts test/twins.test.ts test/view.test.ts test/wiki.test.ts"
+    test: "node --import tsx --test test/checkers.test.ts test/conformance.test.ts test/constants.test.ts test/context.test.ts test/document.test.ts test/dxf.test.ts test/e2e.test.ts test/gate.test.ts test/graphEvalRowsym.test.ts test/labels.test.ts test/mep.test.ts test/overlap.test.ts test/parity.test.ts test/proposals.test.ts test/raster.test.ts test/resources.test.ts test/safewrite.test.ts test/scalewarn.test.ts test/scope.test.ts test/session.test.ts test/staging.test.ts test/sweepguard.test.ts test/tools.test.ts test/transitions.test.ts test/twins.test.ts test/view.test.ts test/wiki.test.ts"
   },
   dependencies: {
     "@modelcontextprotocol/sdk": "^1.12.0",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opentakeoff-mcp",
-  "version": "0.9.84",
+  "version": "0.9.86",
   "mcpName": "io.github.Kentucky-ai/opentakeoff",
   "type": "module",
   "description": "OpenTakeoff MCP server — drive the takeoff engine from your MCP client over stdio.",
@@ -20,7 +20,7 @@
     "check:wiki": "node scripts/check-wiki.mjs",
     "check:versions": "node ../scripts/check-version-consistency.mjs",
     "check:tool-count": "node --import tsx scripts/check-tool-count.mjs",
-    "test": "node --import tsx --test test/checkers.test.ts test/conformance.test.ts test/context.test.ts test/dxf.test.ts test/e2e.test.ts test/gate.test.ts test/graphEvalRowsym.test.ts test/labels.test.ts test/mep.test.ts test/overlap.test.ts test/parity.test.ts test/proposals.test.ts test/raster.test.ts test/resources.test.ts test/safewrite.test.ts test/scalewarn.test.ts test/scope.test.ts test/session.test.ts test/staging.test.ts test/sweepguard.test.ts test/tools.test.ts test/transitions.test.ts test/twins.test.ts test/view.test.ts test/wiki.test.ts"
+    "test": "node --import tsx --test test/checkers.test.ts test/conformance.test.ts test/constants.test.ts test/context.test.ts test/document.test.ts test/dxf.test.ts test/e2e.test.ts test/gate.test.ts test/graphEvalRowsym.test.ts test/labels.test.ts test/mep.test.ts test/overlap.test.ts test/parity.test.ts test/proposals.test.ts test/raster.test.ts test/resources.test.ts test/safewrite.test.ts test/scalewarn.test.ts test/scope.test.ts test/session.test.ts test/staging.test.ts test/sweepguard.test.ts test/tools.test.ts test/transitions.test.ts test/twins.test.ts test/view.test.ts test/wiki.test.ts"
   },
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.12.0",