@willcgage/module-schematic 0.66.0 → 0.68.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";
@@ -703,18 +703,20 @@ function stateToDoc(state, recordNumber) {
             ]
           ) : [
             { id: "A", label: "West", tracks: [{ trackId: MAIN_TRACK_ID, lane: 0, config: state.configA }] },
-            // Non-loop modules always have a real B ("none" never applies).
-            {
-              id: "B",
-              label: "East",
-              tracks: [
-                {
-                  trackId: MAIN_TRACK_ID,
-                  lane: 0,
-                  config: state.configB === "none" ? "single" : state.configB
-                }
-              ]
-            }
+            // ⚠️ `configB: "none"` means the module HAS NO FAR ENDPLATE, and
+            // that is not loop-only (#184). An *end of the line* or a *pocket*
+            // presents one conforming face and the track simply stops. The
+            // standard governs the ends a module offers for joining; it never
+            // required a module to offer two. This used to coerce "none" to
+            // "single" and emit a B regardless, so a single-ended module was
+            // impossible to author.
+            ...state.configB !== "none" ? [
+              {
+                id: "B",
+                label: "East",
+                tracks: [{ trackId: MAIN_TRACK_ID, lane: 0, config: state.configB }]
+              }
+            ] : []
           ],
           // Branch endplates C, D, … — junction connections at pos, off one side
           // (#170). A set can carry several (e.g. a second railroad through).
@@ -929,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),
@@ -1984,7 +1992,7 @@ function deriveEndplatePoses(geo) {
       trackOffsets: offsetsFor(cfg(0), half)
     })
   );
-  const noB = geo.geometryType === "dead_end" || geo.loop === true;
+  const noB = geo.geometryType === "dead_end" || geo.loop === true || geo.endplateConfigs?.[1] === "none";
   const end = sectionedEndPose({ sections: geo.sections });
   if (!noB && end) {
     poses.push(