@immense/vue-pom-generator 1.0.65 → 1.0.66

package/RELEASE_NOTES.md

@@ -1,31 +1,27 @@
-● ## v1.0.65
+● ## Highlights
 
-  ### Highlights
+  - Fixed component child POM attachment bug
+  - Added comprehensive test coverage for class generation
 
-  - Fixed parsing of guarded handler expressions (e.g., `@click="condition && handler"`)
-  - Improved semantic name resolution for wrapped and conditional event handlers
-  - Enhanced Vue compiler AST traversal to handle logical expressions (`&&` operators)
+  ## Changes
 
-  ### Changes
-
-  **Parser Improvements**
-  - Added support for `LogicalExpression` nodes in handler attribute parsing
-  - Introduced `unwrapSemanticHelperCall` to handle Vue compiler helpers (`_unref`,
-  `_withModifiers`, etc.)
-  - Refactored `resolveSemanticName` to recursively unwrap nested expressions and guarded calls
-  - Handler names now correctly extracted from conditional expressions like `isValid &&
-  submitForm`
+  **Bug Fixes**
+  - Corrected logic for attaching child page object models to component classes
+  - Improved reliability of generated POM hierarchies
 
   **Testing**
-  - Added test coverage for guarded handler scenarios
-  - Updated transform and utils test suites to validate new parsing logic
+  - Added 53 lines of test coverage for class generation scenarios
+
+  ## Breaking Changes
+
+  None
 
-  ### Pull Requests Included
+  ## Pull Requests Included
 
-  No pull request was created for this fix; it was committed directly to `main`.
+  Unable to determine specific PR for commit `e1f328c` from available metadata.
 
-  ### Testing
+  ## Testing
 
-  Validated with expanded unit test coverage in `tests/transform.test.ts` and
-  `tests/utils-coverage.test.ts`.
+  Added new test cases in `tests/class-generation-coverage.test.ts` covering component child POM
+  attachment scenarios.
 

package/class-generation/index.ts

@@ -1710,9 +1710,9 @@ function prepareViewObjectModelClass(
     members.push(...getComponentInstances(componentRefsForInstances, componentHierarchyMap, attachmentsForThisClass, widgetInstances));
     members.push(getConstructor(componentRefsForInstances, componentHierarchyMap, attachmentsForThisClass, widgetInstances, { testIdAttribute }));
   }
-  if (!isView && attachmentsForThisClass.length > 0) {
-    members.push(...getComponentInstances(new Set(), componentHierarchyMap, attachmentsForThisClass));
-    members.push(getConstructor(new Set(), componentHierarchyMap, attachmentsForThisClass, [], { testIdAttribute }));
+  if (!isView && (componentRefsForInstances.size > 0 || attachmentsForThisClass.length > 0)) {
+    members.push(...getComponentInstances(componentRefsForInstances, componentHierarchyMap, attachmentsForThisClass));
+    members.push(getConstructor(componentRefsForInstances, componentHierarchyMap, attachmentsForThisClass, [], { testIdAttribute }));
   }
 
   members.push(
@@ -1771,7 +1771,10 @@ function generateViewObjectModelContent(
   const basePageImport = path.relative(fromAbs, toAbs).replace(/\\/g, "/");
   const basePageImportNoExt = stripExtension(basePageImport).replace(/\\/g, "/");
   const basePageImportSpecifier = basePageImportNoExt.startsWith(".") ? basePageImportNoExt : `./${basePageImportNoExt}`;
-  const needsPlaywrightPageImport = prepared.isView || prepared.attachmentsForThisClass.length > 0;
+  const needsPlaywrightPageImport = prepared.isView
+    || prepared.attachmentsForThisClass.length > 0
+    || prepared.componentRefsForInstances.size > 0
+    || prepared.widgetInstances.length > 0;
   const customPomImportSpecifiersByClass = options.customPomImportSpecifiersByClass ?? {};
 
   const customImports = Array.from(

package/dist/index.cjs

@@ -5601,9 +5601,9 @@ function prepareViewObjectModelClass(componentName, dependencies, componentHiera
     members.push(...getComponentInstances(componentRefsForInstances, componentHierarchyMap, attachmentsForThisClass, widgetInstances));
     members.push(getConstructor(componentRefsForInstances, componentHierarchyMap, attachmentsForThisClass, widgetInstances, { testIdAttribute }));
   }
-  if (!isView && attachmentsForThisClass.length > 0) {
-    members.push(...getComponentInstances(/* @__PURE__ */ new Set(), componentHierarchyMap, attachmentsForThisClass));
-    members.push(getConstructor(/* @__PURE__ */ new Set(), componentHierarchyMap, attachmentsForThisClass, [], { testIdAttribute }));
+  if (!isView && (componentRefsForInstances.size > 0 || attachmentsForThisClass.length > 0)) {
+    members.push(...getComponentInstances(componentRefsForInstances, componentHierarchyMap, attachmentsForThisClass));
+    members.push(getConstructor(componentRefsForInstances, componentHierarchyMap, attachmentsForThisClass, [], { testIdAttribute }));
   }
   members.push(
     ...getAttachmentPassthroughMethods(componentName, dependencies, attachmentsForThisClass, reservedAttachmentPassthroughNames)
@@ -5647,7 +5647,7 @@ function generateViewObjectModelContent(componentName, dependencies, componentHi
   const basePageImport = path.relative(fromAbs, toAbs).replace(/\\/g, "/");
   const basePageImportNoExt = stripExtension(basePageImport).replace(/\\/g, "/");
   const basePageImportSpecifier = basePageImportNoExt.startsWith(".") ? basePageImportNoExt : `./${basePageImportNoExt}`;
-  const needsPlaywrightPageImport = prepared.isView || prepared.attachmentsForThisClass.length > 0;
+  const needsPlaywrightPageImport = prepared.isView || prepared.attachmentsForThisClass.length > 0 || prepared.componentRefsForInstances.size > 0 || prepared.widgetInstances.length > 0;
   const customPomImportSpecifiersByClass = options.customPomImportSpecifiersByClass ?? {};
   const customImports = Array.from(
     /* @__PURE__ */ new Set([