@immense/vue-pom-generator 1.0.40 → 1.0.42

package/RELEASE_NOTES.md

@@ -1,24 +1,25 @@
-● # Release Notes - v1.0.40
+● Based on the PR information and the specified release window (2026-04-06T12:50:48-05:00 to
+  2026-04-06T17:25:31-05:00), only PR #6 falls within this window. PRs #1 and #4 were merged
+  earlier (February and April 1st respectively) and should not be included in v1.0.42.
 
+  ```markdown
   ## Highlights
 
-  - Fixed keyed POM deduplication logic
-  - Fixed C# navigation return handling
-  - Added PR release-notes preview comment automation
-  - Improved test coverage with new test cases for class generation and utilities
+  - Dev mode now fails fast on snapshot generation errors instead of silently continuing
+  - Snapshot maps are rebuilt atomically to prevent partial state after compilation failures
+  - Dev server behavior now matches build-mode semantics for strict snapshot errors
+  - Added regression test coverage for strict collision failures during dev startup
 
   ## Changes
 
-  **Bug Fixes**
-  - Fixed keyed POM dedupe logic in utils
-  - Corrected C# navigation return handling in class generation
-
-  **Tooling & Automation**
-  - Added automated PR release-notes preview comments
+  **Error Handling**
+  - Dev startup now awaits snapshot generation so strict transform failures reject instead of
+  silently continuing
+  - Snapshot maps are rebuilt atomically during startup and batched regeneration to avoid partial
+  state on failed recompiles
 
   **Testing**
-  - Added 60+ new test cases for class generation coverage
-  - Added 50+ new test cases for utilities coverage
+  - Added regression coverage for strict collision failures during dev startup
 
   ## Breaking Changes
 
@@ -26,13 +27,12 @@
 
   ## Pull Requests Included
 
-  - #4 Fix keyed POM dedupe and C# navigation returns
-  (https://github.com/immense/vue-pom-generator/pull/4) by @dkattan
-  - #1 Add PR release-notes preview comments (https://github.com/immense/vue-pom-generator/pull/1)
-   by @dkattan
+  - #6 fix: fail fast on dev snapshot generation errors
+  (https://github.com/immense/vue-pom-generator/pull/6)
 
   ## Testing
 
-  Added comprehensive test coverage with 110+ new test cases across class generation and utility
-  functions.
+  Added regression test coverage for strict collision failures during dev startup. All tests
+  passing with validation via eslint, typecheck, build, and test suite.
+  ```
 

package/class-generation/index.ts

@@ -75,7 +75,7 @@ interface CustomPomAttachment {
   className: string;
   propertyName: string;
   attachWhenUsesComponents: string[];
-  attachTo?: "views" | "components" | "both";
+  attachTo?: "views" | "components" | "both" | "pagesAndComponents";
   flatten?: boolean;
 }
 

package/dist/class-generation/index.d.ts

@@ -8,7 +8,7 @@ interface CustomPomAttachment {
     className: string;
     propertyName: string;
     attachWhenUsesComponents: string[];
-    attachTo?: "views" | "components" | "both";
+    attachTo?: "views" | "components" | "both" | "pagesAndComponents";
     flatten?: boolean;
 }
 export interface GenerateFilesOptions {

package/dist/class-generation/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../class-generation/index.ts"],"names":[],"mappings":"AAMA,OAAO,EAAE,oCAAoC,EAAE,MAAM,sBAAsB,CAAC;AAE5E,OAAO,EAAE,sBAAsB,EAAoE,MAAM,UAAU,CAAC;AAQpH,OAAO,EAAE,oCAAoC,EAAE,CAAC;AA8ChD,UAAU,SAAS;IACjB,QAAQ,EAAE,MAAM,CAAC;CAClB;AASD,UAAU,mBAAmB;IAC3B,SAAS,EAAE,MAAM,CAAC;IAClB,YAAY,EAAE,MAAM,CAAC;IACrB,wBAAwB,EAAE,MAAM,EAAE,CAAC;IACnC,QAAQ,CAAC,EAAE,OAAO,GAAG,YAAY,GAAG,MAAM,CAAC;IAC3C,OAAO,CAAC,EAAE,OAAO,CAAC;CACnB;AA8PD,MAAM,WAAW,oBAAoB;IACnC;;;;OAIG;IACH,MAAM,CAAC,EAAE,MAAM,CAAC;IAEhB;;;;;;;;;;;;;;;OAeG;IACH,gBAAgB,CAAC,EAAE,OAAO,GAAG,MAAM,GAAG;QAAE,MAAM,CAAC,EAAE,MAAM,CAAA;KAAE,CAAC;IAE1D;;;OAGG;IACH,WAAW,CAAC,EAAE,MAAM,CAAC;IAErB;;;OAGG;IACH,YAAY,CAAC,EAAE,MAAM,CAAC;IAEtB;;;;;;;OAOG;IACH,sBAAsB,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAEhD;;;;;OAKG;IACH,oCAAoC,CAAC,EAAE,OAAO,GAAG,OAAO,CAAC;IAEzD;;;;OAIG;IACH,oBAAoB,CAAC,EAAE,mBAAmB,EAAE,CAAC;IAE7C,yEAAyE;IACzE,eAAe,CAAC,EAAE,MAAM,CAAC;IAEzB,uDAAuD;IACvD,aAAa,CAAC,EAAE,KAAK,CAAC,IAAI,GAAG,QAAQ,CAAC,CAAC;IAEvC,6BAA6B;IAC7B,MAAM,CAAC,EAAE;QACP,SAAS,CAAC,EAAE,MAAM,CAAC;KACpB,CAAC;IAEF,6EAA6E;IAC7E,uBAAuB,CAAC,EAAE,OAAO,CAAC;IAElC,2FAA2F;IAC3F,WAAW,CAAC,EAAE,MAAM,CAAC;IAErB,mDAAmD;IACnD,UAAU,CAAC,EAAE,YAAY,GAAG,MAAM,CAAC;IAEnC,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,QAAQ,CAAC,EAAE,MAAM,EAAE,CAAC;IAEpB,oBAAoB,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,SAAS,CAAC,CAAC;CAClD;AA+BD,wBAAsB,aAAa,CACjC,qBAAqB,EAAE,GAAG,CAAC,MAAM,EAAE,sBAAsB,CAAC,EAC1D,eAAe,EAAE,GAAG,CAAC,MAAM,EAAE,MAAM,CAAC,EACpC,iBAAiB,EAAE,MAAM,EACzB,OAAO,GAAE,oBAAyB,iBAmFnC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../class-generation/index.ts"],"names":[],"mappings":"AAMA,OAAO,EAAE,oCAAoC,EAAE,MAAM,sBAAsB,CAAC;AAE5E,OAAO,EAAE,sBAAsB,EAAoE,MAAM,UAAU,CAAC;AAQpH,OAAO,EAAE,oCAAoC,EAAE,CAAC;AA8ChD,UAAU,SAAS;IACjB,QAAQ,EAAE,MAAM,CAAC;CAClB;AASD,UAAU,mBAAmB;IAC3B,SAAS,EAAE,MAAM,CAAC;IAClB,YAAY,EAAE,MAAM,CAAC;IACrB,wBAAwB,EAAE,MAAM,EAAE,CAAC;IACnC,QAAQ,CAAC,EAAE,OAAO,GAAG,YAAY,GAAG,MAAM,GAAG,oBAAoB,CAAC;IAClE,OAAO,CAAC,EAAE,OAAO,CAAC;CACnB;AA8PD,MAAM,WAAW,oBAAoB;IACnC;;;;OAIG;IACH,MAAM,CAAC,EAAE,MAAM,CAAC;IAEhB;;;;;;;;;;;;;;;OAeG;IACH,gBAAgB,CAAC,EAAE,OAAO,GAAG,MAAM,GAAG;QAAE,MAAM,CAAC,EAAE,MAAM,CAAA;KAAE,CAAC;IAE1D;;;OAGG;IACH,WAAW,CAAC,EAAE,MAAM,CAAC;IAErB;;;OAGG;IACH,YAAY,CAAC,EAAE,MAAM,CAAC;IAEtB;;;;;;;OAOG;IACH,sBAAsB,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAEhD;;;;;OAKG;IACH,oCAAoC,CAAC,EAAE,OAAO,GAAG,OAAO,CAAC;IAEzD;;;;OAIG;IACH,oBAAoB,CAAC,EAAE,mBAAmB,EAAE,CAAC;IAE7C,yEAAyE;IACzE,eAAe,CAAC,EAAE,MAAM,CAAC;IAEzB,uDAAuD;IACvD,aAAa,CAAC,EAAE,KAAK,CAAC,IAAI,GAAG,QAAQ,CAAC,CAAC;IAEvC,6BAA6B;IAC7B,MAAM,CAAC,EAAE;QACP,SAAS,CAAC,EAAE,MAAM,CAAC;KACpB,CAAC;IAEF,6EAA6E;IAC7E,uBAAuB,CAAC,EAAE,OAAO,CAAC;IAElC,2FAA2F;IAC3F,WAAW,CAAC,EAAE,MAAM,CAAC;IAErB,mDAAmD;IACnD,UAAU,CAAC,EAAE,YAAY,GAAG,MAAM,CAAC;IAEnC,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,QAAQ,CAAC,EAAE,MAAM,EAAE,CAAC;IAEpB,oBAAoB,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,SAAS,CAAC,CAAC;CAClD;AA+BD,wBAAsB,aAAa,CACjC,qBAAqB,EAAE,GAAG,CAAC,MAAM,EAAE,sBAAsB,CAAC,EAC1D,eAAe,EAAE,GAAG,CAAC,MAAM,EAAE,MAAM,CAAC,EACpC,iBAAiB,EAAE,MAAM,EACzB,OAAO,GAAE,oBAAyB,iBAmFnC"}

package/dist/index.cjs

@@ -739,14 +739,16 @@ function nodeHasForDirective(node) {
 function getKeyDirective(node) {
   return findDirectiveByName(node, "bind", "key") ?? null;
 }
-function getKeyDirectiveValue(node, context = null) {
+function getKeyDirectiveValue(node, _context = null) {
   const keyDirective = getKeyDirective(node);
-  let value = keyDirective?.exp?.loc.source;
-  if (value) {
-    if (context) {
-      value = compilerCore.stringifyExpression(keyDirective.exp);
-    }
-    return `\${${value}}`;
+  const rawSource = keyDirective?.exp?.loc.source?.trim();
+  if (rawSource) {
+    return `\${${rawSource}}`;
+  }
+  if (keyDirective?.exp) {
+    const value = compilerCore.stringifyExpression(keyDirective.exp);
+    if (value)
+      return `\${${value}}`;
   }
   return null;
 }
@@ -6008,6 +6010,7 @@ function createDevProcessorPlugin(options) {
     customPomDir,
     customPomImportAliases,
     customPomImportNameCollisionBehavior,
+    nameCollisionBehavior = "suffix",
     testIdAttribute,
     routerAwarePoms,
     resolvedRouterEntry,
@@ -6034,7 +6037,7 @@ function createDevProcessorPlugin(options) {
         return;
       scheduleVueFileRegen(ctx.file, "hmr");
     },
-    configureServer(server) {
+    async configureServer(server) {
       const getViewsDirAbs = () => path.isAbsolute(viewsDir) ? viewsDir : path.resolve(projectRootRef.current, viewsDir);
       const routerInitPromise = (async () => {
         if (!routerAwarePoms) {
@@ -6111,8 +6114,8 @@ function createDevProcessorPlugin(options) {
         }
         return out;
       };
-      const snapshotHierarchy = /* @__PURE__ */ new Map();
-      const snapshotVuePathMap = /* @__PURE__ */ new Map();
+      let snapshotHierarchy = /* @__PURE__ */ new Map();
+      let snapshotVuePathMap = /* @__PURE__ */ new Map();
       const filePathToComponentName = /* @__PURE__ */ new Map();
       const getComponentNameForFile = (filePath) => {
         const normalized = path.resolve(filePath);
@@ -6129,12 +6132,12 @@ function createDevProcessorPlugin(options) {
         filePathToComponentName.set(normalized, name);
         return name;
       };
-      const compileVueFileIntoSnapshot = (filePath) => {
+      const compileVueFileIntoSnapshot = (filePath, targetHierarchy = snapshotHierarchy, targetVuePathMap = snapshotVuePathMap) => {
         const started = node_perf_hooks.performance.now();
         const absolutePath = path.resolve(filePath);
         const componentName = getComponentNameForFile(absolutePath);
-        snapshotVuePathMap.set(componentName, absolutePath);
-        snapshotHierarchy.delete(componentName);
+        targetVuePathMap.set(componentName, absolutePath);
+        targetHierarchy.delete(componentName);
         let sfc = "";
         try {
           sfc = fs.readFileSync(absolutePath, "utf8");
@@ -6144,29 +6147,33 @@ function createDevProcessorPlugin(options) {
         const template = extractTemplateFromSfc(sfc, absolutePath);
         if (!template.trim())
           return { componentName, ms: node_perf_hooks.performance.now() - started, compiled: true };
-        try {
-          compilerDom__namespace.compile(template, {
-            filename: absolutePath,
-            prefixIdentifiers: true,
-            nodeTransforms: [
-              createTestIdTransform(
-                componentName,
-                snapshotHierarchy,
-                nativeWrappers,
-                excludedComponents,
-                getViewsDirAbs(),
-                { existingIdBehavior: "preserve", testIdAttribute, wrapperSearchRoots: getWrapperSearchRoots() }
-              )
-            ]
-          });
-        } catch {
-        }
+        compilerDom__namespace.compile(template, {
+          filename: absolutePath,
+          prefixIdentifiers: true,
+          nodeTransforms: [
+            createTestIdTransform(
+              componentName,
+              targetHierarchy,
+              nativeWrappers,
+              excludedComponents,
+              getViewsDirAbs(),
+              {
+                existingIdBehavior: "preserve",
+                nameCollisionBehavior,
+                testIdAttribute,
+                warn: (message) => loggerRef.current.warn(message),
+                vueFilesPathMap: targetVuePathMap,
+                wrapperSearchRoots: getWrapperSearchRoots()
+              }
+            )
+          ]
+        });
         return { componentName, ms: node_perf_hooks.performance.now() - started, compiled: true };
       };
       const fullRebuildSnapshotFromFilesystem = () => {
         const t0 = node_perf_hooks.performance.now();
-        snapshotHierarchy.clear();
-        snapshotVuePathMap.clear();
+        const nextHierarchy = /* @__PURE__ */ new Map();
+        const nextVuePathMap = /* @__PURE__ */ new Map();
         filePathToComponentName.clear();
         let totalVueFiles = 0;
         let compiledCount = 0;
@@ -6177,11 +6184,13 @@ function createDevProcessorPlugin(options) {
           const vueFiles = walkFilesRecursive(absDir);
           totalVueFiles += vueFiles.length;
           for (const file of vueFiles) {
-            const res = compileVueFileIntoSnapshot(file);
+            const res = compileVueFileIntoSnapshot(file, nextHierarchy, nextVuePathMap);
             if (res.compiled)
               compiledCount++;
           }
         }
+        snapshotHierarchy = nextHierarchy;
+        snapshotVuePathMap = nextVuePathMap;
         const t1 = node_perf_hooks.performance.now();
         logInfo(`initial scan: found ${totalVueFiles} .vue files in ${scanDirs.join(", ")}`);
         logInfo(`initial compile: ${compiledCount}/${totalVueFiles} files in ${formatMs(t1 - t0)} (components=${snapshotHierarchy.size})`);
@@ -6198,6 +6207,8 @@ function createDevProcessorPlugin(options) {
           customPomDir,
           customPomImportAliases,
           customPomImportNameCollisionBehavior,
+          viewsDir,
+          scanDirs,
           testIdAttribute,
           vueRouterFluentChaining: routerAwarePoms,
           routerEntry: resolvedRouterEntry,
@@ -6206,6 +6217,10 @@ function createDevProcessorPlugin(options) {
         const t1 = node_perf_hooks.performance.now();
         logInfo(`generate(${reason}): components=${snapshotHierarchy.size} in ${formatMs(t1 - t0)}`);
       };
+      let timer = null;
+      let maxWaitTimer = null;
+      const pendingChangedVueFiles = /* @__PURE__ */ new Set();
+      const pendingDeletedComponents = /* @__PURE__ */ new Set();
       const initialBuildPromise = (async () => {
         const t0 = node_perf_hooks.performance.now();
         await routerInitPromise;
@@ -6214,13 +6229,44 @@ function createDevProcessorPlugin(options) {
         const t1 = node_perf_hooks.performance.now();
         logInfo(`startup total: ${formatMs(t1 - t0)}`);
       })();
+      const logGenerationError = (reason, message) => {
+        server.config.logger.error(`[vue-pom-generator] dev generation failed during ${reason}: ${message}`);
+      };
+      const regenerateFromPending = async (reason) => {
+        const t0 = node_perf_hooks.performance.now();
+        await initialBuildPromise;
+        const nextHierarchy = new Map(snapshotHierarchy);
+        const nextVuePathMap = new Map(snapshotVuePathMap);
+        for (const componentName of pendingDeletedComponents) {
+          nextHierarchy.delete(componentName);
+          nextVuePathMap.delete(componentName);
+        }
+        const files = Array.from(pendingChangedVueFiles);
+        const deletedCount = pendingDeletedComponents.size;
+        pendingChangedVueFiles.clear();
+        pendingDeletedComponents.clear();
+        let compileMs = 0;
+        for (const f of files) {
+          const res = compileVueFileIntoSnapshot(f, nextHierarchy, nextVuePathMap);
+          compileMs += res.ms;
+        }
+        snapshotHierarchy = nextHierarchy;
+        snapshotVuePathMap = nextVuePathMap;
+        const t1 = node_perf_hooks.performance.now();
+        generateAggregatedFromSnapshot(reason);
+        const t2 = node_perf_hooks.performance.now();
+        return {
+          files,
+          deletedCount,
+          compileMs,
+          preGenerateMs: t1 - t0,
+          generateMs: t2 - t1,
+          totalMs: t2 - t0
+        };
+      };
       const watchedVueGlobs = scanDirs.map((dir) => path.resolve(projectRootRef.current, dir, "**", "*.vue"));
       const watchedPluginGlob = path.resolve(projectRootRef.current, "vite-plugins", "vue-pom-generator", "**", "*.ts");
       server.watcher.add([...watchedVueGlobs, watchedPluginGlob, basePageClassPath]);
-      let timer = null;
-      let maxWaitTimer = null;
-      const pendingChangedVueFiles = /* @__PURE__ */ new Set();
-      const pendingDeletedComponents = /* @__PURE__ */ new Set();
       scheduleVueFileRegenLocal = (filePath, source) => {
         pendingChangedVueFiles.add(filePath);
         logDebug(`queued(${source}): files=${pendingChangedVueFiles.size} deleted=${pendingDeletedComponents.size}`);
@@ -6237,29 +6283,14 @@ function createDevProcessorPlugin(options) {
               timer = null;
             }
             maxWaitTimer = null;
-            void (async () => {
-              const t0 = node_perf_hooks.performance.now();
-              await initialBuildPromise;
-              for (const componentName of pendingDeletedComponents) {
-                snapshotHierarchy.delete(componentName);
-                snapshotVuePathMap.delete(componentName);
-              }
-              const files = Array.from(pendingChangedVueFiles);
-              const deletedCount = pendingDeletedComponents.size;
-              pendingChangedVueFiles.clear();
-              pendingDeletedComponents.clear();
-              let compileMs = 0;
-              for (const f of files) {
-                const res = compileVueFileIntoSnapshot(f);
-                compileMs += res.ms;
-              }
-              const t1 = node_perf_hooks.performance.now();
-              generateAggregatedFromSnapshot("max-wait");
-              const t2 = node_perf_hooks.performance.now();
+            void regenerateFromPending("max-wait").then(({ files, deletedCount, compileMs, preGenerateMs, generateMs, totalMs }) => {
               logInfo(
-                `max-wait: files=${files.length} deleted=${deletedCount} compile=${formatMs(compileMs)} wall=${formatMs(t1 - t0)} gen=${formatMs(t2 - t1)} total=${formatMs(t2 - t0)}`
+                `max-wait: files=${files.length} deleted=${deletedCount} compile=${formatMs(compileMs)} wall=${formatMs(preGenerateMs)} gen=${formatMs(generateMs)} total=${formatMs(totalMs)}`
               );
-            })();
+            }).catch((error) => {
+              const message = error instanceof Error ? error.message : String(error);
+              logGenerationError("max-wait", message);
+            });
           }, MAX_WAIT_MS);
         }
         if (wasEmpty) {
@@ -6275,31 +6306,17 @@ function createDevProcessorPlugin(options) {
             clearTimeout(maxWaitTimer);
             maxWaitTimer = null;
           }
-          void (async () => {
-            const t0 = node_perf_hooks.performance.now();
-            await initialBuildPromise;
-            for (const componentName of pendingDeletedComponents) {
-              snapshotHierarchy.delete(componentName);
-              snapshotVuePathMap.delete(componentName);
-            }
-            const files = Array.from(pendingChangedVueFiles);
-            const deletedCount = pendingDeletedComponents.size;
-            pendingChangedVueFiles.clear();
-            pendingDeletedComponents.clear();
-            let compileMs = 0;
-            for (const f of files) {
-              const res = compileVueFileIntoSnapshot(f);
-              compileMs += res.ms;
-            }
-            const t1 = node_perf_hooks.performance.now();
-            generateAggregatedFromSnapshot(files.length || deletedCount ? "batched" : "noop");
-            const t2 = node_perf_hooks.performance.now();
+          const reason = pendingChangedVueFiles.size || pendingDeletedComponents.size ? "batched" : "noop";
+          void regenerateFromPending(reason).then(({ files, deletedCount, compileMs, preGenerateMs, generateMs, totalMs }) => {
             if (files.length || deletedCount) {
               logInfo(
-                `batched: files=${files.length} deleted=${deletedCount} compile=${formatMs(compileMs)} wall=${formatMs(t1 - t0)} gen=${formatMs(t2 - t1)} total=${formatMs(t2 - t0)}`
+                `batched: files=${files.length} deleted=${deletedCount} compile=${formatMs(compileMs)} wall=${formatMs(preGenerateMs)} gen=${formatMs(generateMs)} total=${formatMs(totalMs)}`
               );
             }
-          })();
+          }).catch((error) => {
+            const message = error instanceof Error ? error.message : String(error);
+            logGenerationError(reason, message);
+          });
         }, 75);
       }
       server.watcher.on("change", async (changedPath) => {
@@ -6351,6 +6368,7 @@ function createDevProcessorPlugin(options) {
       });
       setTimeout(() => {
       }, 250);
+      await initialBuildPromise;
     }
   };
 }
@@ -6381,6 +6399,7 @@ function createSupportPlugins(options) {
     viewsDir,
     scanDirs,
     getWrapperSearchRoots,
+    nameCollisionBehavior = "suffix",
     outDir,
     emitLanguages,
     csharp,
@@ -6457,6 +6476,7 @@ function createSupportPlugins(options) {
     customPomDir,
     customPomImportAliases,
     customPomImportNameCollisionBehavior,
+    nameCollisionBehavior,
     testIdAttribute,
     routerAwarePoms,
     routerType,
@@ -7037,6 +7057,7 @@ function createVuePomGeneratorPlugins(options = {}) {
     viewsDir,
     scanDirs,
     getWrapperSearchRoots: getWrapperSearchRootsAbs,
+    nameCollisionBehavior,
     outDir,
     emitLanguages,
     csharp,