@immense/vue-pom-generator 1.0.42 → 1.0.43

package/RELEASE_NOTES.md

@@ -1,25 +1,23 @@
-● 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.
+● ## Highlights
 
-  ```markdown
-  ## Highlights
-
-  - 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
+  - Fixed dev-mode POM generation to achieve parity with build mode behavior
+  - Added comprehensive regression test suite (284 lines) covering build–serve parity scenarios
+  - Enhanced plugin system to ensure consistent POM generation across development and production
+  environments
 
   ## Changes
 
-  **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
+  **Bug Fixes**
+  - Resolved discrepancies between dev-mode and build-mode POM generation (#5)
+
+  **Testing & Quality**
+  - Added build–serve parity regression tests to prevent future inconsistencies (#7)
+  - New test file: `tests/build-serve-parity.test.ts` with comprehensive coverage
 
-  **Testing**
-  - Added regression coverage for strict collision failures during dev startup
+  **Plugin System**
+  - Extended build plugin functionality with improved POM generation logic
+  - Enhanced dev plugin to match build-mode behavior
+  - Updated support plugins for better parity handling
 
   ## Breaking Changes
 
@@ -27,12 +25,13 @@
 
   ## Pull Requests Included
 
-  - #6 fix: fail fast on dev snapshot generation errors
-  (https://github.com/immense/vue-pom-generator/pull/6)
+  - [#7](https://github.com/immense/vue-pom-generator/pull/7) test: add build–serve parity
+  regression tests (by @mayfieldiv)
+  - [#5](https://github.com/immense/vue-pom-generator/pull/5) fix: dev-mode POM generation parity
+  with build mode (by @mayfieldiv)
 
   ## Testing
 
-  Added regression test coverage for strict collision failures during dev startup. All tests
-  passing with validation via eslint, typecheck, build, and test suite.
-  ```
+  Added 284 lines of regression tests ensuring build–serve parity. Tests validate consistent POM
+  generation behavior across development and production modes.
 

package/dist/index.cjs

@@ -26,13 +26,13 @@ const path = require("node:path");
 const process = require("node:process");
 const node_url = require("node:url");
 const fs = require("node:fs");
+const compilerDom = require("@vue/compiler-dom");
+const compilerSfc = require("@vue/compiler-sfc");
 const parser = require("@babel/parser");
 const jsdom = require("jsdom");
 const compilerCore = require("@vue/compiler-core");
 const types = require("@babel/types");
 const node_perf_hooks = require("node:perf_hooks");
-const compilerDom = require("@vue/compiler-dom");
-const compilerSfc = require("@vue/compiler-sfc");
 const virtualImport = require("vite-plugin-virtual");
 const vue = require("@vitejs/plugin-vue");
 var _documentCurrentScript = typeof document !== "undefined" ? document.currentScript : null;
@@ -4768,148 +4768,6 @@ function getConstructor(childrenComponent, componentHierarchyMap, attachmentsFor
   return `${content}
 `;
 }
-function summarizeHierarchyMap(componentHierarchyMap) {
-  let interactiveComponentCount = 0;
-  let dataTestIdCount = 0;
-  for (const dependencies of componentHierarchyMap.values()) {
-    const selectorCount = dependencies.dataTestIdSet?.size ?? 0;
-    if (selectorCount > 0) {
-      interactiveComponentCount += 1;
-      dataTestIdCount += selectorCount;
-    }
-  }
-  return {
-    entryCount: componentHierarchyMap.size,
-    interactiveComponentCount,
-    dataTestIdCount
-  };
-}
-function isLessRich(candidate, previous) {
-  if (candidate.dataTestIdCount !== previous.dataTestIdCount) {
-    return candidate.dataTestIdCount < previous.dataTestIdCount;
-  }
-  if (candidate.interactiveComponentCount !== previous.interactiveComponentCount) {
-    return candidate.interactiveComponentCount < previous.interactiveComponentCount;
-  }
-  return candidate.entryCount < previous.entryCount;
-}
-function createBuildProcessorPlugin(options) {
-  const {
-    componentHierarchyMap,
-    vueFilesPathMap,
-    viewsDir,
-    scanDirs,
-    basePageClassPath,
-    normalizedBasePagePath,
-    outDir,
-    emitLanguages,
-    csharp,
-    generateFixtures,
-    customPomAttachments,
-    projectRootRef,
-    customPomDir,
-    customPomImportAliases,
-    customPomImportNameCollisionBehavior,
-    testIdAttribute,
-    routerAwarePoms,
-    resolvedRouterEntry,
-    routerType,
-    routerModuleShims,
-    loggerRef
-  } = options;
-  let lastGeneratedMetrics = {
-    entryCount: 0,
-    interactiveComponentCount: 0,
-    dataTestIdCount: 0
-  };
-  return {
-    name: "vue-pom-generator-build",
-    // This plugin exists to generate code on build output; it is not needed during dev-server HMR.
-    apply: "build",
-    enforce: "pre",
-    async buildStart() {
-      if (!routerAwarePoms) {
-        setRouteNameToComponentNameMap(/* @__PURE__ */ new Map());
-        setResolveToComponentNameFn(() => null);
-        return;
-      }
-      let result;
-      if (routerType === "nuxt") {
-        result = await introspectNuxtPages(projectRootRef.current);
-      } else {
-        if (!resolvedRouterEntry)
-          throw new Error("[vue-pom-generator] router.entry is required when router introspection is enabled.");
-        result = await parseRouterFileFromCwd(resolvedRouterEntry, {
-          moduleShims: routerModuleShims,
-          componentNaming: {
-            projectRoot: projectRootRef.current,
-            viewsDirAbs: path.isAbsolute(viewsDir) ? viewsDir : path.resolve(projectRootRef.current, viewsDir),
-            scanDirs
-          }
-        });
-      }
-      const { routeNameMap, routePathMap } = result;
-      setRouteNameToComponentNameMap(routeNameMap);
-      setResolveToComponentNameFn((to) => {
-        if (typeof to === "string") {
-          return routePathMap.get(to) ?? null;
-        }
-        const maybe = to;
-        if (typeof maybe.name === "string" && maybe.name.length) {
-          const key = toPascalCase(maybe.name);
-          return routeNameMap.get(key) ?? null;
-        }
-        if (typeof maybe.path === "string" && maybe.path.length) {
-          return routePathMap.get(maybe.path) ?? null;
-        }
-        return null;
-      });
-      if (!fs.existsSync(basePageClassPath)) {
-        this.error(`BasePage.ts not found at ${basePageClassPath}. Ensure it is included in the build.`);
-      }
-      this.addWatchFile(basePageClassPath);
-      const pointerPath = path.resolve(path.dirname(basePageClassPath), "Pointer.ts");
-      if (!fs.existsSync(pointerPath)) {
-        this.error(`Pointer.ts not found at ${pointerPath}. Ensure it is included in the build.`);
-      }
-      this.addWatchFile(pointerPath);
-    },
-    async buildEnd(error) {
-      if (error) {
-        return;
-      }
-      const metrics = summarizeHierarchyMap(componentHierarchyMap);
-      if (metrics.dataTestIdCount <= 0) {
-        return;
-      }
-      if (isLessRich(metrics, lastGeneratedMetrics)) {
-        return;
-      }
-      await generateFiles(componentHierarchyMap, vueFilesPathMap, normalizedBasePagePath, {
-        outDir,
-        emitLanguages,
-        csharp,
-        generateFixtures,
-        customPomAttachments,
-        projectRoot: projectRootRef.current,
-        customPomDir,
-        customPomImportAliases,
-        customPomImportNameCollisionBehavior,
-        testIdAttribute,
-        vueRouterFluentChaining: routerAwarePoms,
-        routerEntry: resolvedRouterEntry,
-        routerType,
-        viewsDir,
-        scanDirs
-      });
-      lastGeneratedMetrics = metrics;
-      loggerRef.current.info(`generated POMs (${metrics.entryCount} entries, ${metrics.interactiveComponentCount} interactive components, ${metrics.dataTestIdCount} selectors)`);
-    },
-    closeBundle() {
-      loggerRef.current.info("build complete");
-    }
-  };
-}
 const TESTID_CLICK_EVENT_NAME = "__testid_event__";
 const TESTID_CLICK_EVENT_STRICT_FLAG = "__testid_click_event_strict__";
 const CLICK_EVENT_NAME = TESTID_CLICK_EVENT_NAME;
@@ -5992,6 +5850,265 @@ Fix: remove the explicit ${attrLabel}, or change existingIdBehavior to "overwrit
     }
   };
 }
+function summarizeHierarchyMap(componentHierarchyMap) {
+  let interactiveComponentCount = 0;
+  let dataTestIdCount = 0;
+  for (const dependencies of componentHierarchyMap.values()) {
+    const selectorCount = dependencies.dataTestIdSet?.size ?? 0;
+    if (selectorCount > 0) {
+      interactiveComponentCount += 1;
+      dataTestIdCount += selectorCount;
+    }
+  }
+  return {
+    entryCount: componentHierarchyMap.size,
+    interactiveComponentCount,
+    dataTestIdCount
+  };
+}
+function isLessRich(candidate, previous) {
+  if (candidate.dataTestIdCount !== previous.dataTestIdCount) {
+    return candidate.dataTestIdCount < previous.dataTestIdCount;
+  }
+  if (candidate.interactiveComponentCount !== previous.interactiveComponentCount) {
+    return candidate.interactiveComponentCount < previous.interactiveComponentCount;
+  }
+  return candidate.entryCount < previous.entryCount;
+}
+function createBuildProcessorPlugin(options) {
+  const {
+    componentHierarchyMap,
+    vueFilesPathMap,
+    viewsDir,
+    scanDirs,
+    basePageClassPath,
+    normalizedBasePagePath,
+    outDir,
+    emitLanguages,
+    csharp,
+    generateFixtures,
+    customPomAttachments,
+    projectRootRef,
+    customPomDir,
+    customPomImportAliases,
+    customPomImportNameCollisionBehavior,
+    testIdAttribute,
+    nameCollisionBehavior,
+    existingIdBehavior,
+    nativeWrappers,
+    excludedComponents,
+    getWrapperSearchRoots,
+    routerAwarePoms,
+    resolvedRouterEntry,
+    routerType,
+    routerModuleShims,
+    loggerRef
+  } = options;
+  let lastGeneratedMetrics = {
+    entryCount: 0,
+    interactiveComponentCount: 0,
+    dataTestIdCount: 0
+  };
+  const getViewsDirAbs = () => path.isAbsolute(viewsDir) ? viewsDir : path.resolve(projectRootRef.current, viewsDir);
+  const getScriptInfo = (source, filename) => {
+    try {
+      const { descriptor } = compilerSfc.parse(source, { filename });
+      if (!descriptor.script && !descriptor.scriptSetup)
+        return { bindings: void 0, isScriptSetup: false };
+      const scriptBlock = compilerSfc.compileScript(descriptor, { id: filename });
+      return { bindings: scriptBlock.bindings, isScriptSetup: !!descriptor.scriptSetup };
+    } catch {
+      return { bindings: void 0, isScriptSetup: false };
+    }
+  };
+  const supplementHierarchyFromFilesystem = () => {
+    const walkFilesRecursive = (rootDir) => {
+      const out = [];
+      const stack = [rootDir];
+      while (stack.length) {
+        const dir = stack.pop();
+        if (!dir) continue;
+        let entries = [];
+        try {
+          entries = fs.readdirSync(dir, { withFileTypes: true });
+        } catch {
+          continue;
+        }
+        for (const ent of entries) {
+          if (ent.isDirectory()) {
+            if (ent.name === "node_modules" || ent.name === ".git" || ent.name === "dist")
+              continue;
+            stack.push(path.join(dir, ent.name));
+            continue;
+          }
+          if (ent.isFile() && ent.name.endsWith(".vue")) {
+            out.push(path.join(dir, ent.name));
+          }
+        }
+      }
+      return out;
+    };
+    let supplemented = 0;
+    for (const dir of scanDirs) {
+      const absDir = path.resolve(projectRootRef.current, dir);
+      if (!fs.existsSync(absDir))
+        continue;
+      for (const filePath of walkFilesRecursive(absDir)) {
+        const absolutePath = path.resolve(filePath);
+        const componentName = resolveComponentNameFromPath({
+          filename: absolutePath,
+          projectRoot: projectRootRef.current,
+          viewsDirAbs: getViewsDirAbs(),
+          scanDirs,
+          extraRoots: [process.cwd()]
+        });
+        if (componentHierarchyMap.has(componentName))
+          continue;
+        let sfc = "";
+        try {
+          sfc = fs.readFileSync(absolutePath, "utf8");
+        } catch {
+          continue;
+        }
+        const { descriptor } = compilerSfc.parse(sfc, { filename: absolutePath });
+        const template = descriptor.template?.content ?? "";
+        if (!template.trim()) {
+          vueFilesPathMap.set(componentName, absolutePath);
+          componentHierarchyMap.set(componentName, {
+            filePath: absolutePath,
+            childrenComponentSet: /* @__PURE__ */ new Set(),
+            usedComponentSet: /* @__PURE__ */ new Set(),
+            dataTestIdSet: /* @__PURE__ */ new Set(),
+            isView: false,
+            methodsContent: ""
+          });
+          supplemented++;
+          continue;
+        }
+        const { bindings: bindingMetadata, isScriptSetup } = getScriptInfo(sfc, absolutePath);
+        vueFilesPathMap.set(componentName, absolutePath);
+        try {
+          compilerDom__namespace.compile(template, {
+            filename: absolutePath,
+            prefixIdentifiers: true,
+            inline: isScriptSetup,
+            bindingMetadata,
+            nodeTransforms: [
+              createTestIdTransform(
+                componentName,
+                componentHierarchyMap,
+                nativeWrappers,
+                excludedComponents,
+                getViewsDirAbs(),
+                {
+                  existingIdBehavior: existingIdBehavior ?? "preserve",
+                  testIdAttribute,
+                  nameCollisionBehavior,
+                  warn: (message) => loggerRef.current.warn(message),
+                  vueFilesPathMap,
+                  wrapperSearchRoots: getWrapperSearchRoots()
+                }
+              )
+            ]
+          });
+        } catch {
+        }
+        supplemented++;
+      }
+    }
+    if (supplemented > 0) {
+      loggerRef.current.info(`supplemented ${supplemented} components from filesystem walk (not in build graph)`);
+    }
+  };
+  return {
+    name: "vue-pom-generator-build",
+    // This plugin exists to generate code on build output; it is not needed during dev-server HMR.
+    apply: "build",
+    enforce: "pre",
+    async buildStart() {
+      if (!routerAwarePoms) {
+        setRouteNameToComponentNameMap(/* @__PURE__ */ new Map());
+        setResolveToComponentNameFn(() => null);
+        return;
+      }
+      let result;
+      if (routerType === "nuxt") {
+        result = await introspectNuxtPages(projectRootRef.current);
+      } else {
+        if (!resolvedRouterEntry)
+          throw new Error("[vue-pom-generator] router.entry is required when router introspection is enabled.");
+        result = await parseRouterFileFromCwd(resolvedRouterEntry, {
+          moduleShims: routerModuleShims,
+          componentNaming: {
+            projectRoot: projectRootRef.current,
+            viewsDirAbs: path.isAbsolute(viewsDir) ? viewsDir : path.resolve(projectRootRef.current, viewsDir),
+            scanDirs
+          }
+        });
+      }
+      const { routeNameMap, routePathMap } = result;
+      setRouteNameToComponentNameMap(routeNameMap);
+      setResolveToComponentNameFn((to) => {
+        if (typeof to === "string") {
+          return routePathMap.get(to) ?? null;
+        }
+        const maybe = to;
+        if (typeof maybe.name === "string" && maybe.name.length) {
+          const key = toPascalCase(maybe.name);
+          return routeNameMap.get(key) ?? null;
+        }
+        if (typeof maybe.path === "string" && maybe.path.length) {
+          return routePathMap.get(maybe.path) ?? null;
+        }
+        return null;
+      });
+      if (!fs.existsSync(basePageClassPath)) {
+        this.error(`BasePage.ts not found at ${basePageClassPath}. Ensure it is included in the build.`);
+      }
+      this.addWatchFile(basePageClassPath);
+      const pointerPath = path.resolve(path.dirname(basePageClassPath), "Pointer.ts");
+      if (!fs.existsSync(pointerPath)) {
+        this.error(`Pointer.ts not found at ${pointerPath}. Ensure it is included in the build.`);
+      }
+      this.addWatchFile(pointerPath);
+    },
+    async buildEnd(error) {
+      if (error) {
+        return;
+      }
+      supplementHierarchyFromFilesystem();
+      const metrics = summarizeHierarchyMap(componentHierarchyMap);
+      if (metrics.dataTestIdCount <= 0) {
+        return;
+      }
+      if (isLessRich(metrics, lastGeneratedMetrics)) {
+        return;
+      }
+      await generateFiles(componentHierarchyMap, vueFilesPathMap, normalizedBasePagePath, {
+        outDir,
+        emitLanguages,
+        csharp,
+        generateFixtures,
+        customPomAttachments,
+        projectRoot: projectRootRef.current,
+        customPomDir,
+        customPomImportAliases,
+        customPomImportNameCollisionBehavior,
+        testIdAttribute,
+        vueRouterFluentChaining: routerAwarePoms,
+        routerEntry: resolvedRouterEntry,
+        routerType,
+        viewsDir,
+        scanDirs
+      });
+      lastGeneratedMetrics = metrics;
+      loggerRef.current.info(`generated POMs (${metrics.entryCount} entries, ${metrics.interactiveComponentCount} interactive components, ${metrics.dataTestIdCount} selectors)`);
+    },
+    closeBundle() {
+      loggerRef.current.info("build complete");
+    }
+  };
+}
 function createDevProcessorPlugin(options) {
   const {
     nativeWrappers,
@@ -6011,6 +6128,7 @@ function createDevProcessorPlugin(options) {
     customPomImportAliases,
     customPomImportNameCollisionBehavior,
     nameCollisionBehavior = "suffix",
+    existingIdBehavior,
     testIdAttribute,
     routerAwarePoms,
     resolvedRouterEntry,
@@ -6088,6 +6206,17 @@ function createDevProcessorPlugin(options) {
         });
         return descriptor.template?.content ?? "";
       };
+      const getScriptInfo = (source, filename) => {
+        try {
+          const { descriptor } = compilerSfc.parse(source, { filename });
+          if (!descriptor.script && !descriptor.scriptSetup)
+            return { bindings: void 0, isScriptSetup: false };
+          const scriptBlock = compilerSfc.compileScript(descriptor, { id: filename });
+          return { bindings: scriptBlock.bindings, isScriptSetup: !!descriptor.scriptSetup };
+        } catch {
+          return { bindings: void 0, isScriptSetup: false };
+        }
+      };
       const walkFilesRecursive = (rootDir) => {
         const out = [];
         const stack = [rootDir];
@@ -6147,9 +6276,12 @@ function createDevProcessorPlugin(options) {
         const template = extractTemplateFromSfc(sfc, absolutePath);
         if (!template.trim())
           return { componentName, ms: node_perf_hooks.performance.now() - started, compiled: true };
+        const { bindings: bindingMetadata, isScriptSetup } = getScriptInfo(sfc, absolutePath);
         compilerDom__namespace.compile(template, {
           filename: absolutePath,
           prefixIdentifiers: true,
+          inline: isScriptSetup,
+          bindingMetadata,
           nodeTransforms: [
             createTestIdTransform(
               componentName,
@@ -6158,7 +6290,7 @@ function createDevProcessorPlugin(options) {
               excludedComponents,
               getViewsDirAbs(),
               {
-                existingIdBehavior: "preserve",
+                existingIdBehavior: existingIdBehavior ?? "preserve",
                 nameCollisionBehavior,
                 testIdAttribute,
                 warn: (message) => loggerRef.current.warn(message),
@@ -6400,6 +6532,7 @@ function createSupportPlugins(options) {
     scanDirs,
     getWrapperSearchRoots,
     nameCollisionBehavior = "suffix",
+    existingIdBehavior,
     outDir,
     emitLanguages,
     csharp,
@@ -6453,6 +6586,11 @@ function createSupportPlugins(options) {
     customPomImportAliases,
     customPomImportNameCollisionBehavior,
     testIdAttribute,
+    nameCollisionBehavior,
+    existingIdBehavior,
+    nativeWrappers,
+    excludedComponents,
+    getWrapperSearchRoots,
     routerAwarePoms,
     routerType,
     resolvedRouterEntry,
@@ -6477,6 +6615,7 @@ function createSupportPlugins(options) {
     customPomImportAliases,
     customPomImportNameCollisionBehavior,
     nameCollisionBehavior,
+    existingIdBehavior,
     testIdAttribute,
     routerAwarePoms,
     routerType,
@@ -7058,6 +7197,7 @@ function createVuePomGeneratorPlugins(options = {}) {
     scanDirs,
     getWrapperSearchRoots: getWrapperSearchRootsAbs,
     nameCollisionBehavior,
+    existingIdBehavior,
     outDir,
     emitLanguages,
     csharp,