@willcgage/module-schematic 0.67.0 → 0.69.0

package/dist/index.d.cts

@@ -673,7 +673,21 @@ declare function checkEndplateWidth(input: {
     config?: TrackConfig | "none" | null;
     trackOffsetInches?: number | null;
 }): EndplateWidthIssue[];
-/** Whether a doc is a single-endplate turnback (explicit flag or one endplate). */
+/**
+ * Whether a doc is a balloon loop — **the authored flag, and only that**.
+ *
+ * ⚠️ This used to ALSO infer a loop from `endplates.length === 1`, which was
+ * safe only while the sole way to have one endplate was to be a turnback. #184
+ * ended that: an *end of the line* or a *pocket* presents one conforming face
+ * too, so every single-ended module was silently classified as a loop — drawn
+ * with a bulb, its endplate A relabelled "Entry", and positions past the throat
+ * read as being inside a balloon that doesn't exist (#191).
+ *
+ * A loop is now only ever a loop because someone said so: the Loop checkbox and
+ * the return-loop generator both set `loop: true`. Checked before removing the
+ * inference — no stored doc relied on it (no module in the catalogue has one
+ * endplate without the flag, and none is `category:"loop"` or `dead_end`).
+ */
 declare function isLoopDoc(doc: ModuleSchematicDoc): boolean;
 declare const MAIN_TRACK_ID = "main";
 /** The second main on double-track modules — a real track entity so turnouts
@@ -1091,6 +1105,12 @@ interface ModuleFeatures {
         atFrac: number;
         doubleSide: "west" | "east";
     } | null;
+    /** Whether the module presents a far endplate at all. False for an *end of
+     * the line* or a *pocket*, which offer one conforming face and simply stop
+     * (#184) — a renderer must not label that end, or it announces a plate the
+     * module hasn't got and invites something to be coupled to it (#191). A loop
+     * is separate: see {@link loop} and {@link loopInterchange}. */
+    hasEndplateB: boolean;
     /** Lane extents across every feature (mains included; negative = outside
      * Main 1). Renderers size their vertical space from these. */
     laneMin: number;

package/dist/index.d.ts

@@ -673,7 +673,21 @@ declare function checkEndplateWidth(input: {
     config?: TrackConfig | "none" | null;
     trackOffsetInches?: number | null;
 }): EndplateWidthIssue[];
-/** Whether a doc is a single-endplate turnback (explicit flag or one endplate). */
+/**
+ * Whether a doc is a balloon loop — **the authored flag, and only that**.
+ *
+ * ⚠️ This used to ALSO infer a loop from `endplates.length === 1`, which was
+ * safe only while the sole way to have one endplate was to be a turnback. #184
+ * ended that: an *end of the line* or a *pocket* presents one conforming face
+ * too, so every single-ended module was silently classified as a loop — drawn
+ * with a bulb, its endplate A relabelled "Entry", and positions past the throat
+ * read as being inside a balloon that doesn't exist (#191).
+ *
+ * A loop is now only ever a loop because someone said so: the Loop checkbox and
+ * the return-loop generator both set `loop: true`. Checked before removing the
+ * inference — no stored doc relied on it (no module in the catalogue has one
+ * endplate without the flag, and none is `category:"loop"` or `dead_end`).
+ */
 declare function isLoopDoc(doc: ModuleSchematicDoc): boolean;
 declare const MAIN_TRACK_ID = "main";
 /** The second main on double-track modules — a real track entity so turnouts
@@ -1091,6 +1105,12 @@ interface ModuleFeatures {
         atFrac: number;
         doubleSide: "west" | "east";
     } | null;
+    /** Whether the module presents a far endplate at all. False for an *end of
+     * the line* or a *pocket*, which offer one conforming face and simply stop
+     * (#184) — a renderer must not label that end, or it announces a plate the
+     * module hasn't got and invites something to be coupled to it (#191). A loop
+     * is separate: see {@link loop} and {@link loopInterchange}. */
+    hasEndplateB: boolean;
     /** Lane extents across every feature (mains included; negative = outside
      * Main 1). Renderers size their vertical space from these. */
     laneMin: number;

package/dist/index.js

@@ -532,7 +532,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";
@@ -929,8 +929,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),
@@ -1862,6 +1868,7 @@ function moduleFeatures(doc) {
     crossings,
     crossovers,
     branchConnectors,
+    hasEndplateB: doc.endplates.some((e) => e.id === "B"),
     industries,
     laneMin: Math.min(...allLanes),
     laneMax: Math.max(...allLanes)