@immense/vue-pom-generator 1.0.29 → 1.0.31

package/README.md

@@ -35,15 +35,15 @@ Exported entrypoints:
 - `injection`: how `data-testid` (or your chosen attribute) is derived/injected
 - `generation`: how Page Object Models (POMs) and Playwright helpers are generated
 
-The generator emits an aggregated output under `generation.outDir` (default `tests/playwright/generated`):
+The generator emits an aggregated output under `generation.outDir` (default `tests/playwright/__generated__`):
 
-- `tests/playwright/generated/page-object-models.g.ts` (generated; do not edit)
-- `tests/playwright/generated/index.ts` (generated stable barrel)
-- managed `.gitattributes` files alongside generated outputs so GitHub Linguist treats them as generated by default
+- `tests/playwright/__generated__/page-object-models.g.ts` (generated; do not edit)
+- `tests/playwright/__generated__/index.ts` (generated stable barrel)
+- managed `.gitattributes` files only when you emit outside `__generated__`
 
 If `generation.playwright.fixtures` is enabled, it also emits:
 
-- `tests/playwright/generated/fixtures.g.ts` (generated; do not edit)
+- `tests/playwright/__generated__/fixtures.g.ts` (generated; do not edit)
 
 ### Vite config example
 
@@ -71,6 +71,10 @@ export default defineConfig(() => {
       // For Nuxt, you might want ["app", "components", "pages", "layouts"]
       scanDirs: ["src"],
 
+      // Optional: extra directories to search when inferring wrapper-component roles for
+      // components that live outside scanDirs (for example a sibling shared UI package)
+      wrapperSearchRoots: ["../shared/ui/src/components"],
+
       // Optional: wrapper semantics for design-system components
       nativeWrappers: {
         MyButton: { role: "button" },
@@ -94,8 +98,8 @@ export default defineConfig(() => {
         namespace: "MyProject.Tests.Generated",
       },
 
-      // Default: tests/playwright/generated
-      outDir: "tests/playwright/generated",
+      // Default: tests/playwright/__generated__
+      outDir: "tests/playwright/__generated__",
 
       // Controls how to handle duplicate generated member names within a single POM class.
       // - "error": fail compilation

package/RELEASE_NOTES.md

@@ -1,43 +1,57 @@
-● # Release v1.0.29
+● I need to see the actual diff content to understand what the fix addressed. Let me get the file
+  diffs:
+
+● Let me look at specific key changes to understand what was fixed:
+
+● Based on the commit and PR details, here are the release notes:
+
+  ---
 
   ## Highlights
 
-  - Added new ESLint rule to automatically remove existing `data-test-id` attributes from Vue
-  components
-  - Implemented PR release notes preview comment automation
-  - Enhanced router introspection capabilities
-  - Expanded documentation with ESLint rule usage examples
-  - Added comprehensive test coverage for new ESLint rule
+  - Changed default output directory from `tests/playwright/generated` to
+  `tests/playwright/__generated__` for better convention adherence
+  - Improved `.gitattributes` generation: now skips managed entries for paths using
+  `__generated__` heuristic (GitHub Linguist auto-detects these)
+  - Enhanced slot scope key extraction logic for more robust click handler naming in generated
+  POMs
+  - Added comprehensive utility functions for analyzing assignment patterns, object destructuring,
+   and binding identifiers
 
   ## Changes
 
-  **ESLint Rules**
-  - New `remove-existing-test-id-attributes` rule to clean up manually-added test IDs before
-  auto-generation
-  - Updated ESLint plugin exports to include the new rule
+  **Output Directory Convention**
+  - Updated default `generation.outDir` to `tests/playwright/__generated__`
+  - Updated default Playwright fixtures output to use `__generated__` subdirectory
+  - `.gitattributes` entries are now only generated when using custom output directories outside
+  `__generated__`
 
-  **Automation & Tooling**
-  - Added PR release notes preview comment functionality
+  **Click Handler Naming**
+  - Improved slot scope key candidate extraction with priority-based selection
+  - Added support for object destructuring patterns in slot scope analysis
+  - Enhanced fallback key expression generation for slot scoped elements
+  - Better handling of assignment patterns, rest elements, and object properties
 
-  **Documentation**
-  - Added 45 lines of documentation covering the new ESLint cleanup rule
-  - Included usage examples and configuration guidance
+  **Code Quality**
+  - Added test coverage for new utility functions in `class-generation-coverage.test.ts` and
+  `utils-coverage.test.ts`
+  - Expanded Babel type guards usage for more precise AST node detection
+  - Refactored slot scope analysis with dedicated helper functions
 
-  **Code Improvements**
-  - Enhanced router introspection logic (17 line changes)
-  - Updated utility functions (3 line changes)
+  ## Breaking Changes
 
-  **Testing**
-  - Added 58 lines of tests for `remove-existing-test-id-attributes` rule
-  - Updated existing test for test ID error handling
+  - **Default output directory** changed from `tests/playwright/generated` to
+  `tests/playwright/__generated__`. If you're relying on the default, generated files will move to
+   the new location. To preserve the old behavior, explicitly set `generation.outDir:
+  "tests/playwright/generated"` in your config.
 
   ## Pull Requests Included
 
-  - #1 Add PR release-notes preview comments (https://github.com/immense/vue-pom-generator/pull/1)
-   by @dkattan
+  - [#1 Add PR release-notes preview
+  comments](https://github.com/immense/vue-pom-generator/pull/1) by @dkattan
 
   ## Testing
 
-  Comprehensive test suite added for the new ESLint rule with 58 lines of test coverage. Existing
-  tests updated to reflect changes.
+  Added test coverage for new utility functions covering class generation and slot scope analysis
+  edge cases.
 

package/class-generation/index.ts

@@ -524,6 +524,11 @@ function escapeGitAttributesPattern(value: string): string {
   return output;
 }
 
+function pathUsesGeneratedHeuristic(filePath: string): boolean {
+  const normalized = path.normalize(filePath);
+  return normalized.split(path.sep).includes("__generated__");
+}
+
 function buildManagedGitAttributesBlock(entries: string[]): string {
   return [
     GENERATED_GITATTRIBUTES_BLOCK_START,
@@ -581,6 +586,10 @@ function buildGeneratedGitAttributesFiles(generatedFilePaths: string[]): Generat
       continue;
     }
 
+    if (pathUsesGeneratedHeuristic(resolvedFilePath)) {
+      continue;
+    }
+
     const dir = path.dirname(resolvedFilePath);
     const entry = `${escapeGitAttributesPattern(path.basename(resolvedFilePath))} linguist-generated`;
     const entries = entriesByDir.get(dir) ?? new Set<string>();