@immense/vue-pom-generator 1.0.34 → 1.0.35

package/RELEASE_NOTES.md

@@ -1,39 +1,28 @@
-● # v1.0.34
+● # Release Notes: v1.0.35
 
   ## 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 wrapper collision fallbacks in transform logic
+  - Added comprehensive test coverage for wrapper collision scenarios
+  - Improved PR workflow with release-notes preview comments
 
   ## Changes
 
-  **Bug Fixes**
-  - Fixed strict click collision detection logic in transform pipeline
-  - Corrected lazy route component naming in class generation
+  ### Bug Fixes
+  - Resolved issues with wrapper collision fallback behavior in transform module
 
-  **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 41 lines of new tests for wrapper collision scenarios in `transform.test.ts`
 
-  **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
+  ### Developer Experience
+  - Integrated automated release-notes preview comments on pull requests
 
   ## Breaking Changes
-
-  None.
+  None
 
   ## 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.
+  New test cases added to validate wrapper collision fallback fixes. All tests passing.
 

package/dist/index.cjs

@@ -5642,14 +5642,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;