@willcgage/module-schematic 0.67.0 → 0.68.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

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

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),