bambu-printer-mcp 1.1.20 → 1.1.21

package/CONTRIBUTORS.md

@@ -6,6 +6,7 @@ Thank you to everyone who builds, tests, reports problems, and shares real print
 
 | Contributor | Contribution |
 | --- | --- |
+| [JaviOFC](https://github.com/JaviOFC) | Reports the X2D identity rejection in [#36](https://github.com/DMontgomery40/bambu-printer-mcp/issues/36) and supplies current firmware, connection-mode, status-field, serial-prefix, and cloud-job evidence showing that `model_id` can contain a print-job identifier while `model` correctly identifies the printer. |
 | [Izzy Mansurov (sapoepsilon)](https://github.com/sapoepsilon) | Fixes Bambu Studio 02.08 P2S extruder-variant metadata and airduct-command inspection in [#33](https://github.com/DMontgomery40/bambu-printer-mcp/pull/33). Supplies regression tests and offline inspection evidence from four real P2S 0.6 mm PLA/PETG jobs and unsafe mutated copies; this contribution does not claim a physical print. |
 | [Boardy (@boardyai)](https://x.com/boardyai) | Raises the nozzle-verification question that informs the added printer safety checks. |
 | Fable red-team review, shared by David Montgomery | Supplies archive-ambiguity reproductions and hardware-safety review findings used for plate binding, upload checks, temperature policy, and human preflight. Network/security suggestions remain outside this hardware-safety release. |

package/dist/safety/printer-state.js

@@ -13,11 +13,17 @@ function object(value) {
     return !!value && typeof value === "object" && !Array.isArray(value);
 }
 function observedIdentity(raw, configuredSerial) {
-    const declarations = [raw.model_id, raw.model, raw.printer_type, raw.device?.devModel, raw.device?.dev_model]
+    const recognizedModelFromModelId = typeof raw.model_id === "string" && raw.model_id.trim()
+        ? normalizeModel(MODEL_IDS[raw.model_id.trim().toUpperCase()] ?? raw.model_id)
+        : undefined;
+    const declarations = [raw.model, raw.printer_type, raw.device?.devModel, raw.device?.dev_model]
         .filter(value => typeof value === "string" && value.trim());
     const models = declarations.map(value => normalizeModel(MODEL_IDS[value.trim().toUpperCase()] ?? value));
     if (models.some(model => !model))
         throw new Error("Printer reported an unknown model identity; cannot verify this printer.");
+    if (recognizedModelFromModelId)
+        models.unshift(recognizedModelFromModelId);
+    const hasReportedModel = models.length > 0;
     const serials = [raw.serial, raw.serial_number, raw.sn, raw.device?.sn,
         ...(Array.isArray(raw.modules) ? raw.modules.filter((module) => module?.name === "ota").map((module) => module.sn) : [])]
         .filter(value => typeof value === "string" && value.trim()).map(value => value.trim());
@@ -32,7 +38,7 @@ function observedIdentity(raw, configuredSerial) {
     if (new Set(models).size > 1)
         throw new Error("Printer returned contradictory model identity information.");
     const model = models[0];
-    return { model, observedSerial: serials[0], identitySource: declarations.length ? "report" : "module-serial" };
+    return { model, observedSerial: serials[0], identitySource: hasReportedModel ? "report" : "module-serial" };
 }
 /** Only raw reports received after this request are evidence. Never read printer.data here. */
 export function readFreshPrinterStatus(printer, serial, timeoutMs = 5000) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bambu-printer-mcp",
-  "version": "1.1.20",
+  "version": "1.1.21",
   "description": "MCP server for Bambu Lab 3D printers with STL manipulation, slicing, and print management",
   "main": "dist/index.js",
   "repository": {

package/src/safety/printer-state.ts

@@ -15,10 +15,15 @@ function object(value: unknown): value is Record<string, any> {
   return !!value && typeof value === "object" && !Array.isArray(value);
 }
 function observedIdentity(raw: Record<string, any>, configuredSerial: string) {
-  const declarations = [raw.model_id, raw.model, raw.printer_type, raw.device?.devModel, raw.device?.dev_model]
+  const recognizedModelFromModelId = typeof raw.model_id === "string" && raw.model_id.trim()
+    ? normalizeModel(MODEL_IDS[raw.model_id.trim().toUpperCase()] ?? raw.model_id)
+    : undefined;
+  const declarations = [raw.model, raw.printer_type, raw.device?.devModel, raw.device?.dev_model]
     .filter(value => typeof value === "string" && value.trim());
   const models = declarations.map(value => normalizeModel(MODEL_IDS[value.trim().toUpperCase()] ?? value));
   if (models.some(model => !model)) throw new Error("Printer reported an unknown model identity; cannot verify this printer.");
+  if (recognizedModelFromModelId) models.unshift(recognizedModelFromModelId);
+  const hasReportedModel = models.length > 0;
   const serials = [raw.serial, raw.serial_number, raw.sn, raw.device?.sn,
     ...(Array.isArray(raw.modules) ? raw.modules.filter((module: any) => module?.name === "ota").map((module: any) => module.sn) : [])]
     .filter(value => typeof value === "string" && value.trim()).map(value => value.trim());
@@ -31,7 +36,7 @@ function observedIdentity(raw: Record<string, any>, configuredSerial: string) {
   }
   if (new Set(models).size > 1) throw new Error("Printer returned contradictory model identity information.");
   const model = models[0];
-  return { model, observedSerial: serials[0], identitySource: declarations.length ? "report" : "module-serial" };
+  return { model, observedSerial: serials[0], identitySource: hasReportedModel ? "report" : "module-serial" };
 }
 
 /** Only raw reports received after this request are evidence. Never read printer.data here. */