@immense/vue-pom-generator 1.0.34 → 1.0.36

package/RELEASE_NOTES.md

@@ -1,39 +1,33 @@
-● # v1.0.34
+● ## v1.0.36
 
-  ## Highlights
+  ### Highlights
 
-  - Fixed strict click collision detection to prevent false positives
-  - Improved lazy route component naming for better code generation
-  - Enhanced router introspection with significant improvements to route handling
-  - Added comprehensive test coverage for class generation and router introspection
+  - Fixed handler collision suffixes to prevent naming conflicts in strict error mode
+  - Improved handler suffix derivation by skipping unstable leading arguments
+  - Enhanced detection of constant-like member expressions for more stable naming
+  - Added comprehensive test coverage for collision avoidance scenarios
 
-  ## Changes
+  ### Changes
 
   **Bug Fixes**
-  - Fixed strict click collision detection logic in transform pipeline
-  - Corrected lazy route component naming in class generation
+  - Fixed handler collision suffixes when generating test IDs and POM methods (#27dc566)
+  - Skip unstable leading arguments when deriving handler suffixes from function arguments
+  - Check up to 4 arguments (previously 2) to find stable literal/constant values
+  - Validate constant-like root identifiers (e.g., `RebootPreference.Suppress`) before using
+  member expression names
 
-  **Testing & Quality**
-  - Added 70+ lines of new tests for class-generation coverage
-  - Added 64+ lines of new tests for router introspection
-  - Added 37+ lines of new tests for transform logic
+  **Testing**
+  - Added test for strict-mode collision avoidance with later stable args
+  - Added utility test coverage for `getRootIdentifierFromMemberChain` function
+  - New test validates `clickRunDeploymentActionAssign` method generation without conflicts
 
-  **Internal Improvements**
-  - Enhanced router introspection with 140+ lines of improvements
-  - Updated plugin path utilities with better handling
-  - Improved support plugins for both build and dev modes
+  ### Pull Requests Included
 
-  ## Breaking Changes
+  - [#1](https://github.com/immense/vue-pom-generator/pull/1) - Add PR release-notes preview
+  comments (merged 2026-02-02, not in v1.0.36 window)
 
-  None.
+  ### Testing
 
-  ## Pull Requests Included
-
-  - #1 Add PR release-notes preview comments (https://github.com/immense/vue-pom-generator/pull/1)
-   by @dkattan
-
-  ## Testing
-
-  Comprehensive test coverage added across class generation, router introspection, and transform
-  modules. All tests passing.
+  Added 44 lines of test coverage across `transform.test.ts` and `utils-coverage.test.ts` to
+  validate collision handling improvements.
 

package/dist/index.cjs

@@ -1005,6 +1005,18 @@ function nodeHandlerAttributeInfo(node) {
     }
     return null;
   };
+  const getRootIdentifierFromMemberChain = (node2) => {
+    if (!node2) {
+      return null;
+    }
+    if (isIdentifierNode(node2)) {
+      return node2.name;
+    }
+    if (isMemberExpressionNode(node2)) {
+      return getRootIdentifierFromMemberChain(node2.object);
+    }
+    return null;
+  };
   const getAssignmentTargetName = (lhs) => {
     if (!lhs) {
       return null;
@@ -1059,7 +1071,10 @@ function nodeHandlerAttributeInfo(node) {
     }
     if (isMemberExpressionNode(arg)) {
       const stableName = getLastIdentifierFromMemberChain(arg);
-      if (stableName) {
+      const rootName = getRootIdentifierFromMemberChain(arg);
+      const firstChar = (rootName ?? "").charAt(0);
+      const isConstantLikeRoot = firstChar !== "" && firstChar === firstChar.toUpperCase() && firstChar !== firstChar.toLowerCase();
+      if (stableName && isConstantLikeRoot) {
         return toPascalCase(stableName.slice(0, 24));
       }
     }
@@ -1077,12 +1092,15 @@ function nodeHandlerAttributeInfo(node) {
     const first = args.length > 0 ? args[0] : null;
     if (!isObjectExpressionNode(first)) {
       const parts2 = [];
-      for (const arg of args.slice(0, 2)) {
+      for (const arg of args.slice(0, 4)) {
         const w = stableWordFromValue(arg ?? null);
         if (!w) {
-          return null;
+          continue;
         }
         parts2.push(w);
+        if (parts2.length >= 2) {
+          break;
+        }
       }
       if (parts2.length === 0) {
         return null;
@@ -5642,14 +5660,23 @@ Fix: remove the explicit ${attrLabel}, or change existingIdBehavior to "overwrit
         upsertAttribute(element, "option-data-testid-prefix", optionDataTestIdPrefixValue);
       }
       const nativeRole = nativeWrappers[element.tag]?.role ?? element.tag;
-      const primarySemanticHint = semanticNameHint || conditionalHint || void 0;
-      const alternates = nameCollisionBehavior === "error" && semanticNameHint && conditionalHint ? [`${semanticNameHint} ${conditionalHint}`] : void 0;
+      const wrapperHintCandidates = [
+        semanticNameHint,
+        getStaticAttributeContent(element, "title"),
+        getStaticAttributeContent(element, "label"),
+        getStaticAttributeContent(element, "okTitle"),
+        getStaticAttributeContent(element, "cancelTitle"),
+        getStaticAttributeContent(element, "id") || getStaticAttributeContent(element, "name"),
+        getInnerText(element) || null,
+        nameCollisionBehavior === "error" && semanticNameHint && conditionalHint ? `${semanticNameHint} ${conditionalHint}` : conditionalHint
+      ].map((value) => (value ?? "").trim()).filter(Boolean).filter((value, index, values) => values.indexOf(value) === index);
+      const [primarySemanticHint, ...alternates] = wrapperHintCandidates;
       const pomMergeKey = semanticNameHint && conditionalMergeGroupKey ? `wrapper:ifgroup:${conditionalMergeGroupKey}:model:${semanticNameHint}` : void 0;
       applyResolvedDataTestIdForElement({
         preferredGeneratedValue: nativeWrappersValue,
         nativeRoleOverride: nativeRole,
         semanticNameHint: primarySemanticHint,
-        semanticNameHintAlternates: alternates,
+        semanticNameHintAlternates: alternates.length ? alternates : void 0,
         pomMergeKey
       });
       return;