@immense/vue-pom-generator 1.0.41 → 1.0.42

package/RELEASE_NOTES.md

@@ -1,41 +1,38 @@
-● # Release Notes - v1.0.41
+● 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 click test ID scoping to prevent conflicts
-  - Aligned attachment scopes for consistent behavior
-  - Aligned dev snapshot options across plugin configurations
-  - Added PR release notes preview comments via GitHub Actions
-  - Expanded test coverage with new dev plugin options tests
+  - 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 click test ID scoping issues
-  - Aligned attachment scopes for proper C# navigation returns
-  - Aligned dev snapshot options across build and dev plugins
-
-  **Tooling & CI**
-  - Updated release workflow to use npm 11.5.1 via npx
-  - 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 comprehensive dev plugin options test suite (`tests/dev-plugin-options.test.ts`)
-  - Expanded transform tests with additional coverage
+  - Added regression coverage for strict collision failures during dev startup
+
+  ## Breaking Changes
 
-  **Internal**
-  - Updated utility functions for improved scoping logic
-  - Refined plugin configuration handling
+  None
 
   ## Pull Requests Included
 
-  - [#4 Fix keyed POM dedupe and C# navigation
-  returns](https://github.com/immense/vue-pom-generator/pull/4) (@dkattan)
-  - [#1 Add PR release-notes preview
-  comments](https://github.com/immense/vue-pom-generator/pull/1) (@dkattan)
+  - #6 fix: fail fast on dev snapshot generation errors
+  (https://github.com/immense/vue-pom-generator/pull/6)
 
   ## Testing
 
-  Added 160+ lines of new tests for dev plugin options. Existing transform tests expanded with
-  additional assertions.
+  Added regression test coverage for strict collision failures during dev startup. All tests
+  passing with validation via eslint, typecheck, build, and test suite.
+  ```
 

package/dist/index.cjs

@@ -6037,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) {
@@ -6114,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);
@@ -6132,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");
@@ -6147,36 +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",
-                  nameCollisionBehavior,
-                  testIdAttribute,
-                  warn: (message) => loggerRef.current.warn(message),
-                  vueFilesPathMap: snapshotVuePathMap,
-                  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;
@@ -6187,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})`);
@@ -6218,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;
@@ -6226,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}`);
@@ -6249,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) {
@@ -6287,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) => {
@@ -6363,6 +6368,7 @@ function createDevProcessorPlugin(options) {
       });
       setTimeout(() => {
       }, 250);
+      await initialBuildPromise;
     }
   };
 }