@immense/vue-pom-generator 1.0.60 → 1.0.62

package/dist/index.mjs

@@ -219,6 +219,7 @@ function resolveGenerationSupportOptions(options) {
     nameCollisionBehavior: options.nameCollisionBehavior ?? "error",
     existingIdBehavior: options.existingIdBehavior ?? "error",
     testIdAttribute: (options.testIdAttribute ?? "data-testid").trim() || "data-testid",
+    accessibilityAudit: options.accessibilityAudit ?? false,
     routerAwarePoms: options.routerAwarePoms ?? false,
     routerEntry: options.routerEntry,
     routerType: options.routerType ?? "vue-router",
@@ -1760,11 +1761,12 @@ function nodeHandlerAttributeInfo(node) {
     return null;
   }
   const exp = handlerDirective.exp;
-  const source = getVueExpressionSource(exp, "content", "compiled");
-  if (!source) {
+  const transformedSource = getVueExpressionSource(exp, "content", "compiled");
+  const authorSource = getVueExpressionSource(exp, "loc", "content", "compiled");
+  if (!authorSource) {
     return null;
   }
-  const mergeKey = `handler:expr:${source}`;
+  const mergeKey = `handler:expr:${authorSource}`;
   const expr = tryGetDirectiveBabelAst(handlerDirective, {
     preferredViews: ["content", "compiled"],
     plugins: ["typescript", "jsx"],
@@ -1772,9 +1774,7 @@ function nodeHandlerAttributeInfo(node) {
     // That is fine for Vue codegen, but our semantic-name extraction needs a normal Babel parse tree.
     preferExistingAst: false
   });
-  if (!expr) {
-    return null;
-  }
+  const fallbackExpr = transformedSource !== authorSource ? tryParseBabelExpressionFromSource(authorSource, ["typescript", "jsx"]) : null;
   const isNodeType2 = (node2, type) => {
     return node2 !== null && node2.type === type;
   };
@@ -2010,59 +2010,73 @@ function nodeHandlerAttributeInfo(node) {
     const limited = parts.slice(0, 2);
     return limited.map((p) => `${toPascalCase(p.key)}${p.value}`).join("");
   };
-  const direct = getLastIdentifierFromMemberChain(expr);
-  if (direct) {
-    return { semanticNameHint: toPascalCase(direct), mergeKey };
-  }
-  if (isArrowFunctionExpressionNode(expr)) {
-    const body = expr.body;
-    const tryFromCallExpression = (call) => {
-      const resolvedCall = isAwaitExpressionNode(call) ? call.argument : call;
-      if (!isCallExpressionNode(resolvedCall)) {
-        return null;
-      }
-      const name = getLastIdentifierFromMemberChain(resolvedCall.callee);
-      if (!name) {
-        return null;
+  const tryFromCallExpression = (call) => {
+    const resolvedCall = isAwaitExpressionNode(call) ? call.argument : call;
+    if (!isCallExpressionNode(resolvedCall)) {
+      return null;
+    }
+    const name = getLastIdentifierFromMemberChain(resolvedCall.callee);
+    if (!name) {
+      return null;
+    }
+    const suffix = getStableSuffixFromCall(resolvedCall);
+    const semanticNameHint2 = suffix ? `${toPascalCase(name)}${suffix}` : toPascalCase(name);
+    return semanticNameHint2;
+  };
+  const resolveSemanticName = (candidateExpr) => {
+    if (!candidateExpr) {
+      return null;
+    }
+    const direct = getLastIdentifierFromMemberChain(candidateExpr);
+    if (direct) {
+      return toPascalCase(direct);
+    }
+    if (isArrowFunctionExpressionNode(candidateExpr)) {
+      const body = candidateExpr.body;
+      const directCall = tryFromCallExpression(body);
+      if (directCall) {
+        return directCall;
+      }
+      if (isAssignmentExpressionNode(body)) {
+        const lhs = getAssignmentTargetName(body.left);
+        if (lhs) {
+          const rhs = stableWordFromValue(body.right);
+          return `Set${toPascalCase(lhs)}${rhs ?? ""}`;
+        }
       }
-      const suffix = getStableSuffixFromCall(resolvedCall);
-      const semanticNameHint = suffix ? `${toPascalCase(name)}${suffix}` : toPascalCase(name);
-      return semanticNameHint;
-    };
-    const directCall = tryFromCallExpression(body);
-    if (directCall) {
-      return { semanticNameHint: directCall, mergeKey };
-    }
-    if (isAssignmentExpressionNode(body)) {
-      const lhs = getAssignmentTargetName(body.left);
-      if (lhs) {
-        const rhs = stableWordFromValue(body.right);
-        const semanticNameHint = `Set${toPascalCase(lhs)}${rhs ?? ""}`;
-        return { semanticNameHint, mergeKey };
-      }
-    }
-    if (isBlockStatementNode(body)) {
-      const stmts = body.body ?? [];
-      if (stmts.length > 0) {
-        const firstStmt = stmts[0];
-        if (isReturnStatementNode(firstStmt)) {
-          const fromReturn = tryFromCallExpression(firstStmt.argument ?? null);
-          if (fromReturn) {
-            return { semanticNameHint: fromReturn, mergeKey };
+      if (isBlockStatementNode(body)) {
+        const stmts = body.body ?? [];
+        if (stmts.length > 0) {
+          const firstStmt = stmts[0];
+          if (isReturnStatementNode(firstStmt)) {
+            const fromReturn = tryFromCallExpression(firstStmt.argument ?? null);
+            if (fromReturn) {
+              return fromReturn;
+            }
           }
-        }
-        if (isExpressionStatementNode(firstStmt)) {
-          const fromExpr = tryFromCallExpression(firstStmt.expression ?? null);
-          if (fromExpr) {
-            return { semanticNameHint: fromExpr, mergeKey };
+          if (isExpressionStatementNode(firstStmt)) {
+            const fromExpr = tryFromCallExpression(firstStmt.expression ?? null);
+            if (fromExpr) {
+              return fromExpr;
+            }
           }
         }
       }
+      const bodyName = getLastIdentifierFromMemberChain(body);
+      if (bodyName) {
+        return toPascalCase(bodyName);
+      }
     }
-    const bodyName = getLastIdentifierFromMemberChain(body);
-    if (bodyName) {
-      return { semanticNameHint: toPascalCase(bodyName), mergeKey };
-    }
+    return null;
+  };
+  const isVueRefValueAccess = (candidate) => {
+    return isMemberExpressionNode(candidate) && candidate.computed === false && isIdentifierNode2(candidate.property) && candidate.property.name === "value";
+  };
+  const transformedSemanticName = resolveSemanticName(expr);
+  const fallbackSemanticName = resolveSemanticName(fallbackExpr);
+  const semanticNameHint = fallbackSemanticName && transformedSource !== authorSource && isVueRefValueAccess(expr) ? fallbackSemanticName : transformedSemanticName ?? fallbackSemanticName;
+  if (semanticNameHint) {
+    return { semanticNameHint, mergeKey };
   }
   return null;
 }
@@ -8173,6 +8187,59 @@ function createDevProcessorPlugin(options) {
     }
   };
 }
+function supportsInlineTextAccessibleName(role) {
+  return role === "button" || role === "radio";
+}
+function buildAccessibilityAudit(metadata, inferredRole) {
+  if (!metadata || !inferredRole) {
+    return void 0;
+  }
+  const role = normalizePomRoleLabel(inferredRole).toLowerCase();
+  const dynamicProps = new Set(metadata.dynamicProps ?? []);
+  const hasDynamicAccessibleNameSignal = dynamicProps.has("aria-label") || dynamicProps.has("title") || !!metadata.hasDynamicText;
+  let accessibleNameSource;
+  const reasons = [];
+  if (metadata.staticAriaLabel) {
+    accessibleNameSource = "aria-label";
+  } else if (supportsInlineTextAccessibleName(role) && metadata.staticTextContent) {
+    accessibleNameSource = "text";
+  } else if (metadata.staticTitle) {
+    accessibleNameSource = "title";
+  } else if (hasDynamicAccessibleNameSignal) {
+    accessibleNameSource = "dynamic";
+  } else if (role === "input" || role === "select") {
+    accessibleNameSource = "unknown";
+    reasons.push("No inline accessible-name signal was found; the element may rely on external markup such as a separate <label>.");
+  } else {
+    accessibleNameSource = "missing";
+    reasons.push("No compile-time accessible-name signal was found.");
+  }
+  return {
+    needsReview: accessibleNameSource === "unknown" || accessibleNameSource === "missing",
+    accessibleNameSource,
+    reasons,
+    ...metadata.staticAriaLabel ? { staticAriaLabel: metadata.staticAriaLabel } : {},
+    ...metadata.staticRole ? { staticRole: metadata.staticRole } : {},
+    ...metadata.staticTitle ? { staticTitle: metadata.staticTitle } : {},
+    ...metadata.staticTextContent ? { staticTextContent: metadata.staticTextContent } : {}
+  };
+}
+function collectAccessibilityReviewWarnings(manifest) {
+  const warnings = [];
+  for (const [componentName, component] of Object.entries(manifest)) {
+    for (const entry of component.entries) {
+      if (!entry.accessibility?.needsReview) {
+        continue;
+      }
+      const entryLabel = entry.generatedPropertyName ?? entry.testId;
+      const reasonText = entry.accessibility.reasons.join(" ");
+      warnings.push(
+        `[vue-pom-generator] Accessibility review suggested for ${componentName}.${entryLabel} (role=${entry.inferredRole ?? "unknown"}, testId=${JSON.stringify(entry.testId)}): ${reasonText}`
+      );
+    }
+  }
+  return warnings;
+}
 function removeByKeySegment(value) {
   const idx = value.indexOf("ByKey");
   if (idx < 0) {
@@ -8231,6 +8298,7 @@ function getManifestEntry(componentName, entry, componentMetadata, extraMethods)
     ...generatedActionName ? [generatedActionName] : [],
     ...extraActionNames.filter((name) => name !== generatedActionName)
   ]));
+  const accessibility = buildAccessibilityAudit(metadata, pom?.nativeRole ?? null);
   return {
     testId,
     selectorPatternKind: entry.selectorValue.patternKind,
@@ -8241,6 +8309,7 @@ function getManifestEntry(componentName, entry, componentMetadata, extraMethods)
       nativeRole: pom.nativeRole
     }) : componentName,
     inferredRole: pom?.nativeRole ?? null,
+    ...accessibility ? { accessibility } : {},
     generatedPropertyName: pom ? getGeneratedPropertyName(pom) : null,
     generatedActionName,
     generatedActionNames,
@@ -8516,6 +8585,33 @@ function parseDynamicProps(dynamicProps) {
   }
   return void 0;
 }
+function findStaticAttributeValue(element, attributeName) {
+  const attribute = element.props.find(
+    (prop) => prop.type === NodeTypes.ATTRIBUTE && prop.name === attributeName
+  );
+  const value = attribute?.value?.content?.trim();
+  return value ? value : void 0;
+}
+function collectStaticTextContent(children) {
+  const parts = [];
+  const visit = (nodes) => {
+    for (const node of nodes) {
+      if (node.type === NodeTypes.TEXT) {
+        const text2 = node.content.replace(/\s+/g, " ").trim();
+        if (text2) {
+          parts.push(text2);
+        }
+        continue;
+      }
+      if (node.type === NodeTypes.ELEMENT) {
+        visit(node.children);
+      }
+    }
+  };
+  visit(children);
+  const text = parts.join(" ").replace(/\s+/g, " ").trim();
+  return text || void 0;
+}
 function tryCreateElementMetadata(args) {
   const { element, semanticNameMap } = args;
   const testIdAttribute = (args.testIdAttribute ?? "data-testid").trim() || "data-testid";
@@ -8539,13 +8635,9 @@ function tryCreateElementMetadata(args) {
       dynamicPropsList = [content];
     }
   }
-  const semanticName = semanticNameMap.get(testId);
-  if (!semanticName) {
-    return null;
-  }
   const metadata = {
     testId,
-    semanticName,
+    semanticName: semanticNameMap.get(testId),
     tag: element.tag,
     tagType: element.tagType,
     patchFlag,
@@ -8554,7 +8646,11 @@ function tryCreateElementMetadata(args) {
     hasClickHandler: patchFlag ? Boolean(patchFlag & 32) : void 0,
     hasDynamicClass: patchFlag ? Boolean(patchFlag & 2) : void 0,
     hasDynamicStyle: patchFlag ? Boolean(patchFlag & 4) : void 0,
-    hasDynamicText: patchFlag ? Boolean(patchFlag & 1) : void 0
+    hasDynamicText: patchFlag ? Boolean(patchFlag & 1) : void 0,
+    staticAriaLabel: findStaticAttributeValue(element, "aria-label"),
+    staticRole: findStaticAttributeValue(element, "role"),
+    staticTitle: findStaticAttributeValue(element, "title"),
+    staticTextContent: collectStaticTextContent(element.children)
   };
   return metadata;
 }
@@ -8606,6 +8702,7 @@ function extractMetadataAfterTransform(ast, componentName, elementMetadata, sema
   if (componentMetadata.size > 0) {
     elementMetadata.set(componentName, componentMetadata);
   }
+  return componentMetadata;
 }
 function createVuePluginWithTestIds(options) {
   const {
@@ -8620,11 +8717,13 @@ function createVuePluginWithTestIds(options) {
     excludedComponents,
     getViewsDirAbs,
     testIdAttribute,
+    accessibilityAudit,
     loggerRef,
     getSourceDirs,
     getWrapperSearchRoots,
     getProjectRoot
   } = options;
+  const lastAccessibilityWarningSignatureByComponent = /* @__PURE__ */ new Map();
   const getComponentNameFromPath = (filename) => {
     return resolveComponentNameFromPath({
       filename,
@@ -8697,13 +8796,37 @@ function createVuePluginWithTestIds(options) {
             )
           );
           return () => {
-            extractMetadataAfterTransform(
+            const componentMetadata = extractMetadataAfterTransform(
               node,
               componentName,
               elementMetadata,
               semanticNameMap,
               testIdAttribute
             );
+            if (!accessibilityAudit) {
+              return;
+            }
+            const dependencies = componentHierarchyMap.get(componentName);
+            if (!dependencies || componentMetadata.size === 0) {
+              return;
+            }
+            const manifest = buildPomManifest(
+              /* @__PURE__ */ new Map([[componentName, dependencies]]),
+              /* @__PURE__ */ new Map([[componentName, componentMetadata]])
+            );
+            const warnings = collectAccessibilityReviewWarnings(manifest);
+            const signature = warnings.join("\n");
+            if (!signature) {
+              lastAccessibilityWarningSignatureByComponent.delete(componentName);
+              return;
+            }
+            if (lastAccessibilityWarningSignatureByComponent.get(componentName) === signature) {
+              return;
+            }
+            lastAccessibilityWarningSignatureByComponent.set(componentName, signature);
+            for (const warning of warnings) {
+              loggerRef.current.warn(warning);
+            }
           };
         }
         let transform = perFileTransform.get(componentName);
@@ -9068,6 +9191,7 @@ function createVuePomGeneratorPlugins(options = {}) {
     nameCollisionBehavior: generationOptions?.nameCollisionBehavior,
     existingIdBehavior: resolvedInjectionOptions.existingIdBehavior,
     testIdAttribute,
+    accessibilityAudit: generationOptions?.accessibilityAudit,
     routerAwarePoms: typeof routerEntry === "string" && routerEntry.trim().length > 0 || routerType === "nuxt",
     routerEntry,
     routerType,
@@ -9161,6 +9285,7 @@ function createVuePomGeneratorPlugins(options = {}) {
     excludedComponents,
     getViewsDirAbs,
     testIdAttribute,
+    accessibilityAudit: resolvedGenerationOptions.accessibilityAudit,
     loggerRef,
     getSourceDirs,
     getWrapperSearchRoots: getWrapperSearchRootsAbs,