@immense/vue-pom-generator 1.0.62 → 1.0.64

package/dist/index.d.ts

@@ -5,5 +5,5 @@ export { createVuePomGeneratorPlugins as vuePomGenerator };
 export default createVuePomGeneratorPlugins;
 export declare function defineVuePomGeneratorConfig(options: VuePomGeneratorPluginOptions): VuePomGeneratorPluginOptions;
 export declare function defineNuxtPomGeneratorConfig(options: NuxtPomGeneratorPluginOptions): NuxtPomGeneratorPluginOptions;
-export type { ExistingIdBehavior, NuxtPomGeneratorPluginOptions, PomGeneratorPluginOptions, PomNameCollisionBehavior, VuePomGeneratorPluginOptions } from "./plugin/types";
+export type { ExistingIdBehavior, MissingSemanticNameBehavior, NuxtPomGeneratorPluginOptions, PomGeneratorPluginOptions, PomNameCollisionBehavior, VuePomGeneratorPluginOptions } from "./plugin/types";
 //# sourceMappingURL=index.d.ts.map

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../index.ts"],"names":[],"mappings":"AAAA,OAAO,4BAA4B,MAAM,2CAA2C,CAAC;AAErF,OAAO,KAAK,EAAE,6BAA6B,EAAE,4BAA4B,EAAE,MAAM,gBAAgB,CAAC;AAIlG,OAAO,EAAE,4BAA4B,EAAE,CAAC;AACxC,OAAO,EAAE,4BAA4B,IAAI,eAAe,EAAE,CAAC;AAC3D,eAAe,4BAA4B,CAAC;AAE5C,wBAAgB,2BAA2B,CAAC,OAAO,EAAE,4BAA4B,GAAG,4BAA4B,CAE/G;AAED,wBAAgB,4BAA4B,CAAC,OAAO,EAAE,6BAA6B,GAAG,6BAA6B,CAOlH;AAED,YAAY,EAAE,kBAAkB,EAAE,6BAA6B,EAAE,yBAAyB,EAAE,wBAAwB,EAAE,4BAA4B,EAAE,MAAM,gBAAgB,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../index.ts"],"names":[],"mappings":"AAAA,OAAO,4BAA4B,MAAM,2CAA2C,CAAC;AAErF,OAAO,KAAK,EAAE,6BAA6B,EAAE,4BAA4B,EAAE,MAAM,gBAAgB,CAAC;AAIlG,OAAO,EAAE,4BAA4B,EAAE,CAAC;AACxC,OAAO,EAAE,4BAA4B,IAAI,eAAe,EAAE,CAAC;AAC3D,eAAe,4BAA4B,CAAC;AAE5C,wBAAgB,2BAA2B,CAAC,OAAO,EAAE,4BAA4B,GAAG,4BAA4B,CAE/G;AAED,wBAAgB,4BAA4B,CAAC,OAAO,EAAE,6BAA6B,GAAG,6BAA6B,CAOlH;AAED,YAAY,EAAE,kBAAkB,EAAE,2BAA2B,EAAE,6BAA6B,EAAE,yBAAyB,EAAE,wBAAwB,EAAE,4BAA4B,EAAE,MAAM,gBAAgB,CAAC"}

package/dist/index.mjs

@@ -217,6 +217,7 @@ function resolveGenerationSupportOptions(options) {
     customPomImportAliases: options.customPomImportAliases,
     customPomImportNameCollisionBehavior: options.customPomImportNameCollisionBehavior ?? "error",
     nameCollisionBehavior: options.nameCollisionBehavior ?? "error",
+    missingSemanticNameBehavior: options.missingSemanticNameBehavior ?? "error",
     existingIdBehavior: options.existingIdBehavior ?? "error",
     testIdAttribute: (options.testIdAttribute ?? "data-testid").trim() || "data-testid",
     accessibilityAudit: options.accessibilityAudit ?? false,
@@ -5452,14 +5453,51 @@ function prepareViewObjectModelClass(componentName, dependencies, componentHiera
     customPomAttachments = [],
     testIdAttribute
   } = options;
+  const normalizeTrackedComponentRef = (value) => {
+    return value.endsWith(".vue") ? value.slice(0, -4) : value;
+  };
+  const resolveTrackedComponentRef = (value) => {
+    const normalizedValue = normalizeTrackedComponentRef(value);
+    if (componentHierarchyMap.has(value)) {
+      return value;
+    }
+    if (componentHierarchyMap.has(normalizedValue)) {
+      return normalizedValue;
+    }
+    let match = null;
+    for (const [candidateName, candidateDeps] of componentHierarchyMap.entries()) {
+      const normalizedCandidate = normalizeTrackedComponentRef(candidateName);
+      const candidateBaseName = path.parse(candidateDeps.filePath).name;
+      if (normalizedCandidate !== normalizedValue && candidateBaseName !== normalizedValue) {
+        continue;
+      }
+      if (match && match !== candidateName) {
+        return null;
+      }
+      match = candidateName;
+    }
+    return match;
+  };
+  const rawComponentRefsForInstances = isView ? usedComponentSet?.size ? usedComponentSet : childrenComponentSet : childrenComponentSet;
+  const componentRefsForInstances = /* @__PURE__ */ new Set();
+  for (const ref of rawComponentRefsForInstances) {
+    componentRefsForInstances.add(resolveTrackedComponentRef(ref) ?? normalizeTrackedComponentRef(ref));
+  }
   const hasChildComponent = (needle) => {
-    const haystack = usedComponentSet?.size ? usedComponentSet : childrenComponentSet;
-    for (const child of haystack) {
-      if (child === needle)
+    const normalizedNeedle = normalizeTrackedComponentRef(needle);
+    for (const child of rawComponentRefsForInstances) {
+      const resolvedChild = resolveTrackedComponentRef(child);
+      if (normalizeTrackedComponentRef(child) === normalizedNeedle)
         return true;
-      if (child === `${needle}.vue`)
+      if (resolvedChild && normalizeTrackedComponentRef(resolvedChild) === normalizedNeedle)
         return true;
-      if (child.endsWith(".vue") && child.slice(0, -4) === needle)
+      if (resolvedChild) {
+        const resolvedDeps = componentHierarchyMap.get(resolvedChild);
+        if (resolvedDeps && path.parse(resolvedDeps.filePath).name === normalizedNeedle) {
+          return true;
+        }
+      }
+      if (child === `${needle}.vue`)
         return true;
     }
     return false;
@@ -5483,7 +5521,6 @@ function prepareViewObjectModelClass(componentName, dependencies, componentHiera
     methodSignatures: a.flatten ? customPomMethodSignaturesByClass.get(a.className) ?? /* @__PURE__ */ new Map() : /* @__PURE__ */ new Map()
   }));
   const widgetInstances = isView ? getWidgetInstancesForView(componentName, dependencies.dataTestIdSet, customPomAvailableClassIdentifiers) : [];
-  const componentRefsForInstances = isView ? usedComponentSet?.size ? usedComponentSet : childrenComponentSet : childrenComponentSet;
   const className = toPascalCaseLocal(componentName);
   const childInstancePropertyNames = Array.from(componentRefsForInstances).filter((child) => componentHierarchyMap.has(child) && componentHierarchyMap.get(child)?.dataTestIdSet.size).map((child) => child.split(".vue")[0]);
   const blockedViewPassthroughMethodNames = new Set(
@@ -6999,6 +7036,7 @@ function createTestIdTransform(componentName, componentHierarchyMap, nativeWrapp
   const existingIdBehavior = options.existingIdBehavior ?? "error";
   const testIdAttribute = (options.testIdAttribute || "data-testid").trim() || "data-testid";
   const nameCollisionBehavior = options.nameCollisionBehavior ?? "error";
+  const missingSemanticNameBehavior = options.missingSemanticNameBehavior ?? "error";
   const warn = options.warn;
   const vueFilesPathMap = options.vueFilesPathMap;
   const wrapperSearchRoots = options.wrapperSearchRoots ?? [];
@@ -7442,13 +7480,16 @@ Fix: remove the explicit ${attrLabel}, or change existingIdBehavior to "overwrit
     }
     const isSubmit = element.props.find((p) => p.type === NodeTypes.ATTRIBUTE && p.name === "type")?.value?.content === "submit";
     if (isSubmit) {
-      const identifier = getStaticIdOrNameHint(element) || innerText;
+      let identifier = getStaticIdOrNameHint(element) || innerText;
       if (!identifier) {
-        const loc = element.loc?.start;
-        const locationHint = loc ? `${loc.line}:${loc.column}` : "unknown";
-        throw new Error(
-          `[vue-pom-generator] submit button appears identifiable but no usable identity could be derived in ${componentName} (${context.filename ?? "unknown"}:${locationHint}) — id/name were missing/empty and innerText was also missing/invalid`
-        );
+        if (missingSemanticNameBehavior === "error") {
+          const loc = element.loc?.start;
+          const locationHint = loc ? `${loc.line}:${loc.column}` : "unknown";
+          throw new Error(
+            `[vue-pom-generator] submit button appears identifiable but no usable identity could be derived in ${componentName} (${context.filename ?? "unknown"}:${locationHint}) — id/name were missing/empty and innerText was also missing/invalid. Fix: give the button a static id/name, a static inner text, or set missingSemanticNameBehavior = "ignore" to fall back to the generic "submit" identifier.`
+          );
+        }
+        identifier = "submit";
       }
       const testId = getSubmitDataTestId(identifier);
       applyResolvedDataTestIdForElement({
@@ -7515,6 +7556,7 @@ function createBuildProcessorPlugin(options) {
     customPomImportNameCollisionBehavior,
     testIdAttribute,
     nameCollisionBehavior,
+    missingSemanticNameBehavior,
     existingIdBehavior,
     routerAwarePoms,
     routerType,
@@ -7610,6 +7652,9 @@ function createBuildProcessorPlugin(options) {
             prefixIdentifiers: true,
             inline: isScriptSetup,
             bindingMetadata,
+            // See dev-plugin.ts — same rationale: enable TS in template
+            // expressions so `(row: RowType) => ...` handlers parse.
+            expressionPlugins: ["typescript"],
             nodeTransforms: [
               createTestIdTransform(
                 componentName,
@@ -7621,6 +7666,7 @@ function createBuildProcessorPlugin(options) {
                   existingIdBehavior: existingIdBehavior ?? "error",
                   testIdAttribute,
                   nameCollisionBehavior,
+                  missingSemanticNameBehavior,
                   warn: (message) => loggerRef.current.warn(message),
                   vueFilesPathMap,
                   wrapperSearchRoots: getWrapperSearchRoots()
@@ -7768,6 +7814,7 @@ function createDevProcessorPlugin(options) {
     customPomImportAliases,
     customPomImportNameCollisionBehavior,
     nameCollisionBehavior,
+    missingSemanticNameBehavior,
     existingIdBehavior,
     testIdAttribute,
     routerAwarePoms,
@@ -7952,6 +7999,12 @@ function createDevProcessorPlugin(options) {
           prefixIdentifiers: true,
           inline: isScriptSetup,
           bindingMetadata,
+          // Vue templates may contain TypeScript type annotations in expressions
+          // (e.g. `@row-click="(row: RowType) => navigateTo(...)"` in Nuxt + TS
+          // apps). Without this flag, Vue's internal processExpression step
+          // delegates to @babel/parser without the TS plugin and crashes on
+          // "Unexpected token, expected ','".
+          expressionPlugins: ["typescript"],
           nodeTransforms: [
             createTestIdTransform(
               componentName,
@@ -7962,6 +8015,7 @@ function createDevProcessorPlugin(options) {
               {
                 existingIdBehavior: existingIdBehavior ?? "error",
                 nameCollisionBehavior,
+                missingSemanticNameBehavior,
                 testIdAttribute,
                 warn: (message) => loggerRef.current.warn(message),
                 vueFilesPathMap: provisionalVuePathMap,
@@ -8709,6 +8763,7 @@ function createVuePluginWithTestIds(options) {
     vueOptions,
     existingIdBehavior,
     nameCollisionBehavior,
+    missingSemanticNameBehavior = "error",
     nativeWrappers,
     elementMetadata,
     semanticNameMap,
@@ -8789,6 +8844,7 @@ function createVuePluginWithTestIds(options) {
                 existingIdBehavior,
                 testIdAttribute,
                 nameCollisionBehavior,
+                missingSemanticNameBehavior,
                 warn: (message) => loggerRef.current.warn(message),
                 vueFilesPathMap,
                 wrapperSearchRoots: getWrapperSearchRoots()
@@ -8843,6 +8899,7 @@ function createVuePluginWithTestIds(options) {
               existingIdBehavior,
               testIdAttribute,
               nameCollisionBehavior,
+              missingSemanticNameBehavior,
               warn: (message) => loggerRef.current.warn(message),
               vueFilesPathMap,
               wrapperSearchRoots: getWrapperSearchRoots()
@@ -8890,9 +8947,16 @@ function createVuePluginWithTestIds(options) {
       const compile = compilerDom2.compile;
       const { descriptor } = parse2(code, { filename: cleanPath });
       if (descriptor.template) {
+        const mergedExpressionPlugins = Array.from(
+          /* @__PURE__ */ new Set([
+            "typescript",
+            ...userCompilerOptions.expressionPlugins ?? []
+          ])
+        );
         compile(descriptor.template.content, {
           ...userCompilerOptions,
           filename: cleanPath,
+          expressionPlugins: mergedExpressionPlugins,
           nodeTransforms: getNodeTransforms(cleanPath, componentName)
         });
         loggerRef.current.debug(`Metadata collected for ${cleanPath}`);
@@ -9189,6 +9253,7 @@ function createVuePomGeneratorPlugins(options = {}) {
     customPomImportAliases: resolvedCustomPomImportAliases,
     customPomImportNameCollisionBehavior: customPoms?.importNameCollisionBehavior,
     nameCollisionBehavior: generationOptions?.nameCollisionBehavior,
+    missingSemanticNameBehavior: generationOptions?.missingSemanticNameBehavior,
     existingIdBehavior: resolvedInjectionOptions.existingIdBehavior,
     testIdAttribute,
     accessibilityAudit: generationOptions?.accessibilityAudit,
@@ -9277,6 +9342,7 @@ function createVuePomGeneratorPlugins(options = {}) {
     vueOptions,
     existingIdBehavior: resolvedGenerationOptions.existingIdBehavior,
     nameCollisionBehavior: resolvedGenerationOptions.nameCollisionBehavior,
+    missingSemanticNameBehavior: resolvedGenerationOptions.missingSemanticNameBehavior,
     nativeWrappers,
     elementMetadata,
     semanticNameMap,