@immense/vue-pom-generator 1.0.35 → 1.0.36

package/RELEASE_NOTES.md

@@ -1,28 +1,33 @@
-● # Release Notes: v1.0.35
+● ## v1.0.36
 
-  ## Highlights
-  - Fixed wrapper collision fallbacks in transform logic
-  - Added comprehensive test coverage for wrapper collision scenarios
-  - Improved PR workflow with release-notes preview comments
+  ### Highlights
 
-  ## Changes
+  - 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
 
-  ### Bug Fixes
-  - Resolved issues with wrapper collision fallback behavior in transform module
+  ### Changes
 
-  ### Testing
-  - Added 41 lines of new tests for wrapper collision scenarios in `transform.test.ts`
+  **Bug Fixes**
+  - 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**
+  - 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
 
-  ### Developer Experience
-  - Integrated automated release-notes preview comments on pull requests
+  ### Pull Requests Included
 
-  ## Breaking Changes
-  None
+  - [#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)
 
-  ## Pull Requests Included
-  - #1 Add PR release-notes preview comments (https://github.com/immense/vue-pom-generator/pull/1)
-   by @dkattan
+  ### Testing
 
-  ## Testing
-  New test cases added to validate wrapper collision fallback fixes. 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;