@willcgage/module-schematic 0.67.0 → 0.69.0

package/dist/index.cjs

@@ -534,7 +534,7 @@ function endplateWidthFor(widths, id) {
   return typeof w === "number" && w > 0 ? w : FREEMO_ENDPLATE_WIDTH_RECOMMENDED_INCHES;
 }
 function isLoopDoc(doc) {
-  return doc.loop === true || doc.endplates.length === 1;
+  return doc.loop === true;
 }
 var MAIN_TRACK_ID = "main";
 var MAIN2_TRACK_ID = "main2";
@@ -931,8 +931,14 @@ function docToState(doc, fallbackLength, moduleTracks = []) {
     loopReturn: loop && d.loopReturn === "main2" ? "main2" : "same",
     mainsSwapped: d.mainsSwapped === true,
     configA: configOf("A"),
-    // On a loop, a missing B means pure turnback; present = interchange loop.
-    configB: loop && !hasB ? "none" : configOf("B"),
+    // NO ENDPLATE B ⇒ "none", whatever kind of module this is. On a loop that
+    // reads as a pure turnback (a present B makes it an interchange); on an end
+    // of the line or a pocket it simply means the module has one face (#184).
+    // ⚠️ This used to be `loop && !hasB`, so "none" only survived a round trip on
+    // a loop — which passed its test yesterday ONLY because a single-endplate
+    // module was still being misclassified as one (#191). The absence of the
+    // plate is the fact; why it's absent doesn't change what to read.
+    configB: hasB ? configOf("B") : "none",
     branches: branchEps.map((ep) => ({
       label: ep.label ?? "Branch",
       pos: sc(ep.at.pos),
@@ -1864,6 +1870,7 @@ function moduleFeatures(doc) {
     crossings,
     crossovers,
     branchConnectors,
+    hasEndplateB: doc.endplates.some((e) => e.id === "B"),
     industries,
     laneMin: Math.min(...allLanes),
     laneMax: Math.max(...allLanes)