@immense/vue-pom-generator 1.0.32 → 1.0.33

package/RELEASE_NOTES.md

@@ -1,38 +1,43 @@
-● # Release Notes: v1.0.32
+● # Release Notes: v1.0.33
 
   ## Highlights
 
-  - Fixed handling of missing helpers and keyed ID branches in class generation
-  - Added robust error tolerance for edge cases in Vue component transformation
-  - Expanded test coverage with 109 new test lines across generated TypeScript and utility
-  coverage tests
-  - Introduced PR release-notes preview comments via GitHub Actions
+  - Fixed singleton key test-id inference to correctly handle edge cases
+  - Added comprehensive generation metrics and diagnostics system for better visibility into
+  transform operations
+  - Introduced new `generation-metrics.ts` module for tracking and reporting code generation
+  statistics
+  - Expanded transform logic with 192 additional lines of processing logic
+  - Enhanced plugin architecture with improved metric collection in dev and build modes
 
   ## Changes
 
-  **Bug Fixes**
-  - Tolerate missing helpers and keyed ID branches in class generation logic
-  (`class-generation/index.ts`, `utils.ts`)
+  **Core Transformation**
+  - Fixed singleton key test-id inference logic
+  - Expanded `transform.ts` with enhanced processing capabilities
 
-  **Testing**
-  - Added comprehensive tests for generated TypeScript output (`tests/generated-tsc.test.ts`: +74
-  lines)
-  - Expanded utility function coverage tests (`tests/utils-coverage.test.ts`: +35 lines)
-
-  **CI/Automation**
-  - Added automated PR release-notes preview comments to pull requests
+  **Metrics & Diagnostics**
+  - Added `plugin/support/generation-metrics.ts` for tracking generation statistics
+  - Updated `plugin/vue-plugin.ts` with enhanced metrics collection (85 lines modified)
+  - Improved diagnostic output in `plugin/support/dev-plugin.ts` and
+  `plugin/support/build-plugin.ts`
 
-  ## Breaking Changes
+  **Testing**
+  - Added comprehensive test suite in `tests/generation-metrics.test.ts` (47 tests)
+  - Expanded `tests/transform.test.ts` with 131 additional test cases
+  - Enhanced `tests/options.test.ts` with 72 new test cases
 
-  None
+  **Infrastructure**
+  - Updated plugin creation in `plugin/create-vue-pom-generator-plugins.ts`
+  - Minor utility adjustments in `utils.ts`
 
   ## Pull Requests Included
 
-  - [#1](https://github.com/immense/vue-pom-generator/pull/1) Add PR release-notes preview
-  comments (@dkattan)
+  - #1 Add PR release-notes preview comments (https://github.com/immense/vue-pom-generator/pull/1)
+   by @dkattan
 
   ## Testing
 
-  Significant test additions: 74 lines in generated TypeScript tests and 35 lines in utility
-  coverage tests validate the fix for missing helpers and keyed ID branches.
+  Added 250+ new test cases across generation metrics, transform logic, and options handling. All
+  tests passing.
 

package/dist/index.cjs

@@ -4448,6 +4448,32 @@ function getConstructor(childrenComponent, componentHierarchyMap, attachmentsFor
   return `${content}
 `;
 }
+function getGenerationMetrics(componentHierarchyMap) {
+  let selectorCount = 0;
+  let generatedMethodCount = 0;
+  for (const deps of componentHierarchyMap.values()) {
+    selectorCount += deps.dataTestIdSet?.size ?? 0;
+    generatedMethodCount += deps.generatedMethods?.size ?? 0;
+  }
+  return {
+    entryCount: componentHierarchyMap.size,
+    selectorCount,
+    generatedMethodCount
+  };
+}
+function isLessRich(current, previous) {
+  if (current.entryCount !== previous.entryCount) {
+    return current.entryCount < previous.entryCount;
+  }
+  if (current.selectorCount !== previous.selectorCount) {
+    return current.selectorCount < previous.selectorCount;
+  }
+  return current.generatedMethodCount < previous.generatedMethodCount;
+}
+function getGenerationMetricsKey(projectRoot, outDir) {
+  return path.resolve(projectRoot, outDir ?? "./pom");
+}
+const buildGenerationMetricsByOutputKey = /* @__PURE__ */ new Map();
 function createBuildProcessorPlugin(options) {
   const {
     componentHierarchyMap,
@@ -4470,7 +4496,6 @@ function createBuildProcessorPlugin(options) {
     routerModuleShims,
     loggerRef
   } = options;
-  let lastGeneratedEntryCount = 0;
   return {
     name: "vue-pom-generator-build",
     // This plugin exists to generate code on build output; it is not needed during dev-server HMR.
@@ -4517,11 +4542,13 @@ function createBuildProcessorPlugin(options) {
       this.addWatchFile(pointerPath);
     },
     buildEnd() {
-      const entryCount = componentHierarchyMap.size;
-      if (entryCount <= 0) {
+      const metrics = getGenerationMetrics(componentHierarchyMap);
+      if (metrics.entryCount <= 0 || metrics.selectorCount <= 0) {
         return;
       }
-      if (entryCount < lastGeneratedEntryCount) {
+      const generationMetricsKey = getGenerationMetricsKey(projectRootRef.current, outDir);
+      const previousMetrics = buildGenerationMetricsByOutputKey.get(generationMetricsKey);
+      if (previousMetrics && isLessRich(metrics, previousMetrics)) {
         return;
       }
       generateFiles(componentHierarchyMap, vueFilesPathMap, normalizedBasePagePath, {
@@ -4539,8 +4566,8 @@ function createBuildProcessorPlugin(options) {
         routerEntry: resolvedRouterEntry,
         routerType
       });
-      lastGeneratedEntryCount = entryCount;
-      loggerRef.current.info(`generated POMs (${entryCount} entries)`);
+      buildGenerationMetricsByOutputKey.set(generationMetricsKey, metrics);
+      loggerRef.current.info(`generated POMs (${metrics.entryCount} entries, ${metrics.selectorCount} selectors)`);
     },
     closeBundle() {
       loggerRef.current.info("build complete");
@@ -4578,6 +4605,89 @@ function toKebabCaseTag(tag) {
   }
   return result;
 }
+function getStaticAttributeContent(element, name) {
+  const attr = element.props.find((prop) => {
+    return prop.type === compilerCore.NodeTypes.ATTRIBUTE && prop.name === name;
+  });
+  return attr?.value?.content?.trim() || null;
+}
+function getNativeHtmlControlRole(element) {
+  const tag = (element.tag || "").toLowerCase();
+  const type = (getStaticAttributeContent(element, "type") || "").toLowerCase();
+  if (tag === "textarea") {
+    return "input";
+  }
+  if (tag === "select") {
+    return "select";
+  }
+  if (tag !== "input") {
+    return null;
+  }
+  if (type === "radio") {
+    return "radio";
+  }
+  if (type === "checkbox") {
+    return "checkbox";
+  }
+  return "input";
+}
+function normalizeControlLabelText(value) {
+  const normalized = (value ?? "").replace(/\*/g, " ").replace(/\s+/g, " ").trim();
+  return normalized || null;
+}
+function getLabelNodeText(labelNode) {
+  for (const child of labelNode.children || []) {
+    if (child.type === compilerCore.NodeTypes.TEXT) {
+      const normalized2 = normalizeControlLabelText(child.content);
+      if (normalized2) {
+        return normalized2;
+      }
+      continue;
+    }
+    if (child.type !== compilerCore.NodeTypes.ELEMENT) {
+      continue;
+    }
+    if (getNativeHtmlControlRole(child)) {
+      continue;
+    }
+    const normalized = normalizeControlLabelText(getInnerText(child));
+    if (normalized) {
+      return normalized;
+    }
+  }
+  return normalizeControlLabelText(getInnerText(labelNode));
+}
+function getAssociatedLabelText(element, hierarchyMap2) {
+  let parent = hierarchyMap2.get(element) || null;
+  while (parent) {
+    if (parent.tag === "label") {
+      return getLabelNodeText(parent);
+    }
+    parent = hierarchyMap2.get(parent) || null;
+  }
+  const id = getStaticAttributeContent(element, "id");
+  if (!id) {
+    return null;
+  }
+  const candidates = /* @__PURE__ */ new Set();
+  for (const child of hierarchyMap2.keys()) {
+    candidates.add(child);
+  }
+  for (const maybeParent of hierarchyMap2.values()) {
+    if (maybeParent) {
+      candidates.add(maybeParent);
+    }
+  }
+  for (const candidate of candidates) {
+    if (candidate.tag !== "label") {
+      continue;
+    }
+    if (getStaticAttributeContent(candidate, "for") === id) {
+      return getLabelNodeText(candidate);
+    }
+  }
+  return null;
+}
 function normalizeSearchRoots(wrapperSearchRoots) {
   const normalized = /* @__PURE__ */ new Set();
   for (const root of wrapperSearchRoots) {
@@ -5245,7 +5355,9 @@ function createTestIdTransform(componentName, componentHierarchyMap, nativeWrapp
       }
     }
     const getBestAvailableKeyValue = () => {
-      const vForKey = getKeyDirectiveValue(element, context) || getSelfClosingForDirectiveKeyAttrValue(element) || getContainedInVForDirectiveKeyValue(context, element, hierarchyMap);
+      const parentNode = context.parent && typeof context.parent === "object" ? context.parent : null;
+      const isDirectVForChild = parentNode?.type === compilerCore.NodeTypes.FOR;
+      const vForKey = (isDirectVForChild ? getKeyDirectiveValue(element, context) : null) || getContainedInVForDirectiveKeyValue(context, element, hierarchyMap);
       if (vForKey) return vForKey;
       return getContainedInSlotDataKeyValue(element, hierarchyMap);
     };
@@ -5402,6 +5514,50 @@ Fix: remove the explicit ${attrLabel}, or change existingIdBehavior to "overwrit
       });
       return;
     }
+    const nativeHtmlRole = getNativeHtmlControlRole(element);
+    if (nativeHtmlRole) {
+      const rawIdentifier = getStaticAttributeContent(element, "id") || getStaticAttributeContent(element, "name");
+      const labelText = getAssociatedLabelText(element, hierarchyMap);
+      const { vModel, modelValue } = getModelBindingValues(element);
+      const bindingHint = modelValue || vModel || null;
+      const labelToken = labelText ? toPascalCase(labelText) : "";
+      const bindingToken = bindingHint ? toPascalCase(bindingHint) : "";
+      let identifierToken = null;
+      let semanticNameHint2;
+      if (nativeHtmlRole === "radio" || nativeHtmlRole === "checkbox") {
+        if (rawIdentifier) {
+          identifierToken = rawIdentifier;
+          semanticNameHint2 = rawIdentifier;
+        } else if (bindingToken && labelToken) {
+          identifierToken = `${bindingToken}${labelToken}`;
+          semanticNameHint2 = `${bindingHint || bindingToken} ${labelText || labelToken}`;
+        } else if (labelToken) {
+          identifierToken = labelToken;
+          semanticNameHint2 = labelText || labelToken;
+        } else if (bindingToken) {
+          identifierToken = bindingToken;
+          semanticNameHint2 = bindingHint || bindingToken;
+        }
+      } else if (rawIdentifier) {
+        identifierToken = rawIdentifier;
+        semanticNameHint2 = rawIdentifier;
+      } else if (labelToken) {
+        identifierToken = labelToken;
+        semanticNameHint2 = labelText || labelToken;
+      } else if (bindingToken) {
+        identifierToken = bindingToken;
+        semanticNameHint2 = bindingHint || bindingToken;
+      }
+      if (identifierToken) {
+        const preferredGeneratedValue = bestKeyPlaceholder ? templateAttributeValue(`${componentName}-${bestKeyPlaceholder}-${identifierToken}-${nativeHtmlRole}`) : staticAttributeValue(`${componentName}-${identifierToken}-${nativeHtmlRole}`);
+        applyResolvedDataTestIdForElement({
+          preferredGeneratedValue,
+          nativeRoleOverride: nativeHtmlRole,
+          semanticNameHint: semanticNameHint2 || conditionalHint || void 0
+        });
+        return;
+      }
+    }
     const innerText = getInnerText(element) || null;
     const toDirective = nodeHasToDirective(element);
     if (toDirective) {
@@ -5520,6 +5676,7 @@ function resolveComponentNameFromPath(options) {
   }
   return toPascalCase(path.parse(absFilename).name);
 }
+const devStartupMetricsByOutputKey = /* @__PURE__ */ new Map();
 function createDevProcessorPlugin(options) {
   const {
     nativeWrappers,
@@ -5713,6 +5870,21 @@ function createDevProcessorPlugin(options) {
         logInfo(`initial compile: ${compiledCount}/${totalVueFiles} files in ${formatMs(t1 - t0)} (components=${snapshotHierarchy.size})`);
       };
       const generateAggregatedFromSnapshot = (reason) => {
+        const metrics = getGenerationMetrics(snapshotHierarchy);
+        if (metrics.entryCount <= 0 || metrics.selectorCount <= 0) {
+          logInfo(`generate(${reason}): skipped empty snapshot (components=${metrics.entryCount}, selectors=${metrics.selectorCount})`);
+          return;
+        }
+        const generationMetricsKey = getGenerationMetricsKey(projectRootRef.current, outDir);
+        if (reason === "startup") {
+          const previousMetrics = devStartupMetricsByOutputKey.get(generationMetricsKey);
+          if (previousMetrics && isLessRich(metrics, previousMetrics)) {
+            logInfo(
+              `generate(${reason}): skipped smaller snapshot (components=${metrics.entryCount}, selectors=${metrics.selectorCount})`
+            );
+            return;
+          }
+        }
         const t0 = node_perf_hooks.performance.now();
         generateFiles(snapshotHierarchy, snapshotVuePathMap, normalizedBasePagePath, {
           outDir,
@@ -5729,8 +5901,11 @@ function createDevProcessorPlugin(options) {
           routerEntry: resolvedRouterEntry,
           routerType
         });
+        if (reason === "startup") {
+          devStartupMetricsByOutputKey.set(generationMetricsKey, metrics);
+        }
         const t1 = node_perf_hooks.performance.now();
-        logInfo(`generate(${reason}): components=${snapshotHierarchy.size} in ${formatMs(t1 - t0)}`);
+        logInfo(`generate(${reason}): components=${metrics.entryCount} selectors=${metrics.selectorCount} in ${formatMs(t1 - t0)}`);
       };
       const initialBuildPromise = (async () => {
         const t0 = node_perf_hooks.performance.now();
@@ -6251,20 +6426,21 @@ function createVuePluginWithTestIds(options) {
       }
     ];
   };
+  const runtimeNodeTransform = (node, context) => {
+    const filename = context.filename;
+    if (!filename || !filename.endsWith(".vue") || !isFileInScope(filename)) {
+      return;
+    }
+    const transforms = getNodeTransforms(filename);
+    const ourTransform = transforms[transforms.length - 1];
+    return ourTransform(node, context);
+  };
   const templateCompilerOptions = {
     ...userCompilerOptions,
     prefixIdentifiers: true,
     nodeTransforms: [
       ...userNodeTransforms,
-      (node, context) => {
-        const filename = context.filename;
-        if (!filename || !filename.endsWith(".vue") || !isFileInScope(filename)) {
-          return;
-        }
-        const transforms = getNodeTransforms(filename);
-        const ourTransform = transforms[transforms.length - 1];
-        return ourTransform(node, context);
-      }
+      runtimeNodeTransform
     ]
   };
   const metadataCollectorPlugin = {
@@ -6303,7 +6479,42 @@ function createVuePluginWithTestIds(options) {
     ...vueOptions,
     template
   });
-  return { metadataCollectorPlugin, internalVuePlugin };
+  const nuxtVueBridgePlugin = {
+    name: "vue-pom-generator-nuxt-vue-bridge",
+    apply: "serve",
+    configResolved(config) {
+      const viteVuePlugin = config.plugins.find((plugin) => {
+        return typeof plugin === "object" && plugin !== null && "name" in plugin && plugin.name === "vite:vue" && "api" in plugin;
+      });
+      const api = viteVuePlugin?.api;
+      if (!api) {
+        loggerRef.current.warn("[vue-pom-generator] Nuxt bridge could not find vite:vue plugin to patch.");
+        return;
+      }
+      const currentOptions = api.options ?? {};
+      const currentTemplate = currentOptions.template ?? {};
+      const currentCompilerOptions = currentTemplate.compilerOptions ?? {};
+      const currentNodeTransforms = currentCompilerOptions.nodeTransforms ?? [];
+      if (currentNodeTransforms.includes(runtimeNodeTransform)) {
+        return;
+      }
+      api.options = {
+        ...currentOptions,
+        template: {
+          ...currentTemplate,
+          compilerOptions: {
+            ...currentCompilerOptions,
+            prefixIdentifiers: true,
+            nodeTransforms: [
+              ...currentNodeTransforms,
+              runtimeNodeTransform
+            ]
+          }
+        }
+      };
+    }
+  };
+  return { metadataCollectorPlugin, internalVuePlugin, nuxtVueBridgePlugin };
 }
 function assertNonEmptyString(value, name) {
   if (!value || !value.trim()) {
@@ -6429,7 +6640,7 @@ function createVuePomGeneratorPlugins(options = {}) {
   const semanticNameMap = /* @__PURE__ */ new Map();
   const componentHierarchyMap = /* @__PURE__ */ new Map();
   const vueFilesPathMap = /* @__PURE__ */ new Map();
-  const { metadataCollectorPlugin, internalVuePlugin } = createVuePluginWithTestIds({
+  const { metadataCollectorPlugin, internalVuePlugin, nuxtVueBridgePlugin } = createVuePluginWithTestIds({
     vueOptions,
     existingIdBehavior,
     nameCollisionBehavior,
@@ -6480,7 +6691,7 @@ function createVuePomGeneratorPlugins(options = {}) {
   const resultPlugins = [
     configPlugin,
     metadataCollectorPlugin,
-    ...isNuxt ? [] : [internalVuePlugin],
+    ...isNuxt ? [nuxtVueBridgePlugin] : [internalVuePlugin],
     ...supportPlugins
   ];
   if (!generationEnabled) {
@@ -6488,7 +6699,7 @@ function createVuePomGeneratorPlugins(options = {}) {
     return [
       configPlugin,
       metadataCollectorPlugin,
-      ...isNuxt ? [] : [internalVuePlugin],
+      ...isNuxt ? [nuxtVueBridgePlugin] : [internalVuePlugin],
       virtualModules
     ];
   }