opentakeoff-mcp 0.9.38 → 0.9.39

package/README.md

@@ -107,7 +107,7 @@ includes document text, shape vertices, or result payload content.
 |---|---|
 | `load_plan` | Open a plan PDF from disk. Default replaces the whole session; **`merge: true` ADDS the document to the working set** (#152) — plans + schedule + addenda as one takeoff, sheet graph spanning the whole set, marked set covering every worked sheet. Returns per-sheet dims, title-block `sheet_number`, and the detected drawn scale where present. |
 | `sheet_info` | One sheet's dims, vector segment count, scale status, detected suggestion, committed shape count. |
-| `set_scale` | Set a sheet's scale — exactly one of `label`, `upp`, `calibrate {p1, p2, feet}`, `use_detected`. |
+| `set_scale` | Set a sheet's scale — exactly one of `label`, `upp`, `calibrate {p1, p2, feet}`, `use_detected`. **Lands unconfirmed** (`confirmed: false`) until a human confirms in the canvas — see Scale rules. |
 | `one_click` | One-Click Area at (x, y): the sealed flood engine bounded by the plan linework, traced, vertices snapped — the SAME feet-true arguments the canvas passes at a click (gap sealing up to a door width, door-swing wedge inclusion, the half-foot minimum-passage rule), so an MCP trace and a canvas click at one seed measure the same square footage (pinned against the bench corpus goldens in `test/parity.test.ts`). Every trace carries the engine's account of itself: `confidence` 0–1 with `confidence_factors` naming each deduction (`gap_sealed_px`, `door_wedges`, `min_pass_delta`, …) — a review prioritizer, never a verification; a low score is a `view_sheet {overlay: true}` audit prompt, not a fact. On a SCANNED sheet (no usable linework) the flood falls back automatically to the rendered pixels — same engine as the canvas — with `raster_traced` disclosed on the reply and on the shape's origin (#154). Pass `condition` to commit (the full account stamps `origin` centrally at the commit); `role: "deduct"` subtracts. |
 | `detect_rooms` | Batch One-Click: reads every room-number label off the sheet's text layer and floods each — one call instead of `read_sheet_text` + reasoning + N `one_click` calls, through the SAME sealed engine per room (confidence + the engine account ride each room and its committed origin). Only cleanly-traced rooms come back; everything skipped is counted and reasoned in `withheld` (degenerate / duplicate / implausible / unresolved), never dropped silently. To commit: `assign_from_schedule: true` routes each room through its OWN room-finish schedule row and commits under the FLOOR finish that row states (rooms the schedule can't answer for return in `unresolved[]` with reasons and re-seedable coordinates); or pass `condition` to commit every room under one stated tag. |
 | `measure_polygon` | Area + perimeter of a polygon you supply (min 3 verts). Requires scale. |
@@ -216,6 +216,14 @@ space, which makes them usable directly as click targets.
 
 - A detected scale is a **suggestion** — it is never applied automatically.
   Adopting it is always an explicit `set_scale { use_detected: true }`.
+- **Agent proposes, human confirms.** `set_scale` is the agent surface, so a
+  scale set here lands **unconfirmed** (`confirmed: false` in the reply).
+  Quantities still flow — the gate is a flag, never a refusal — but
+  `takeoff_summary` names the affected sheets in
+  `scale_unconfirmed`, and the export/report carry `scale_confirmed` so the
+  canvas can ask the estimator to confirm (its scale menu grows a
+  **Confirm agent-set scale** row on import). Only a human act in the canvas
+  clears the flag.
 - `measure_polygon` and `measure_line` refuse without a scale:
   `Set the scale for <sheet> first — use set_scale (detected: <label>).`
 - `one_click` without a scale returns a **px-only preview**

package/dist/server-core.js

@@ -4199,7 +4199,11 @@ function reportJson({ projectName = "", rows = [], bySheet = [], scaleInfo = [],
     schema: "opentakeoff.report.v1",
     project_name: projectName || null,
     generated_with: "OpenTakeoff",
-    sheets: scaleInfo.map((si) => ({ sheet_id: si.sheet_id, sheet: label(si.sheet_id), scale_source: si.scale_source ?? si.source ?? "unknown" })),
+    // scale_confirmed (scale gate): false = an agent set this sheet's scale and
+    // no human confirmed it — the report's consumer should treat those sheets'
+    // quantities as standing on an unverified number. Absent input = true
+    // (human-era payloads predate the flag).
+    sheets: scaleInfo.map((si) => ({ sheet_id: si.sheet_id, sheet: label(si.sheet_id), scale_source: si.scale_source ?? si.source ?? "unknown", scale_confirmed: si.scale_confirmed !== false })),
     // custom-column values APPEND after materials (row key order otherwise
     // untouched). Iterating the DEFINED columns — never raw attrs — naturally
     // drops orphaned colIds; attrValue (the shared assigned-value rule) keeps
@@ -5399,11 +5403,13 @@ var Session = class _Session {
     }
     s.upp = upp;
     s.scaleSource = source === "label" ? "standard" : source === "calibrate" ? "calibrated" : source;
+    s.scaleConfirmed = false;
     return {
       sheet: s.key,
       upp,
       ...label ? { label } : {},
       source,
+      confirmed: false,
       // #153 — several DISTINCT scale notes on one sheet means enlarged plans
       // or details are likely; region measurements will warn when a
       // disagreeing note sits inside them, but say it up front too
@@ -6679,7 +6685,8 @@ var Session = class _Session {
   summary() {
     const rows = conditionTotals(this.conditions, this.shapes, this.seamCtx());
     const lean = rows.map(({ color, fill, hatch, materials, ...rest }) => rest);
-    return { conditions: lean, totals: grandTotals(rows) };
+    const unconfirmed = [...this.sheets.values()].filter((s) => s.upp != null && s.scaleConfirmed === false).map((s) => s.key);
+    return { conditions: lean, totals: grandTotals(rows), ...unconfirmed.length ? { scale_unconfirmed: unconfirmed } : {} };
   }
   deleteShape(id) {
     const i = this.shapes.findIndex((x) => x.id === id);
@@ -7458,7 +7465,15 @@ var Session = class _Session {
       schema: ANN_SCHEMA,
       project_name: "",
       units: "imperial",
-      sheets: [...this.sheets.values()].filter((s) => s.upp != null).map((s) => ({ sheet_id: s.key, units_per_px: s.upp })),
+      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 } : {}
+      })),
       conditions: this.conditions,
       shapes: this.shapes,
       markups: this.markups,
@@ -7488,7 +7503,7 @@ var Session = class _Session {
       projectName,
       rows,
       bySheet: sheetTotals(this.conditions, this.shapes),
-      scaleInfo: [...this.sheets.values()].filter((s) => s.upp != null).map((s) => ({ sheet_id: s.key, scale_source: s.scaleSource ?? "unknown" })),
+      scaleInfo: [...this.sheets.values()].filter((s) => s.upp != null).map((s) => ({ sheet_id: s.key, scale_source: s.scaleSource ?? "unknown", scale_confirmed: s.scaleConfirmed !== false })),
       markups: this.markups,
       rfis: [],
       rollGoods: rollReportRows(byCond, rows)
@@ -7722,6 +7737,7 @@ var setScaleOutput = {
   upp: z.number().describe("Real feet per image px at render scale 2.0"),
   label: z.string().optional().describe("The standard scale label, when set by label or detected note"),
   source: z.enum(["label", "upp", "calibrate", "detected"]),
+  confirmed: z.boolean().describe("Always false here: set_scale is the agent surface, and an agent-set scale stays UNCONFIRMED until a human confirms it in the canvas \u2014 quantities still flow, wearing the caveat"),
   warning: z.string().optional().describe("Present when the sheet carries MULTIPLE distinct scale notes (#153) \u2014 enlarged plans/details likely; region measurements under a disagreeing note will warn")
 };
 var oneClickOutput = {
@@ -7888,7 +7904,8 @@ var takeoffSummaryOutput = {
     lf_net: z.number(),
     ea: z.number(),
     sy_net: z.number()
-  }).passthrough()
+  }).passthrough(),
+  scale_unconfirmed: z.array(z.string()).optional().describe("Sheets whose scale is agent-set and no human has confirmed \u2014 these totals stand on an unverified scale; verify against a stated dimension or confirm in the canvas")
 };
 var exportTakeoffOutput = {
   schema: z.string(),
@@ -9792,6 +9809,7 @@ async function importTakeoff(session, filePath) {
     if (s && s.upp == null && row.units_per_px > 0) {
       s.upp = row.units_per_px;
       s.scaleSource = row.scale_source ?? "upp";
+      if (row.scale_confirmed === false) s.scaleConfirmed = false;
     }
   }
   const tagKey2 = (t) => String(t || "").trim().toUpperCase();
@@ -10375,7 +10393,7 @@ function registerResources(server, session) {
 // package.json
 var package_default = {
   name: "opentakeoff-mcp",
-  version: "0.9.38",
+  version: "0.9.39",
   mcpName: "io.github.Kentucky-ai/opentakeoff",
   type: "module",
   description: "OpenTakeoff MCP server \u2014 drive the takeoff engine from your MCP client over stdio.",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opentakeoff-mcp",
-  "version": "0.9.38",
+  "version": "0.9.39",
   "mcpName": "io.github.Kentucky-ai/opentakeoff",
   "type": "module",
   "description": "OpenTakeoff MCP server — drive the takeoff engine from your MCP client over stdio.",