@immense/vue-pom-generator 1.0.57 → 1.0.58

package/README.md

@@ -54,7 +54,7 @@ export class UserEditorPage extends BasePage {
   get SaveButton() { /* Playwright locator */ }
 
   async typeEmailAddress(text: string, annotationText = "") { /* ... */ }
-  async clickSave(wait: boolean = true) { /* ... */ }
+  async clickSave(wait: boolean = true, annotationText = "") { /* ... */ }
 }
 ```
 

package/RELEASE_NOTES.md

@@ -1,30 +1,47 @@
-● ## Highlights
+● I'll fetch the actual commits and PRs between v1.0.57 and HEAD to generate accurate release
+  notes.
 
-  - Fixed router introspection cleanup to properly restore DOM globals after execution
-  - Improved test coverage for router introspection with 83 new test lines
-  - Refactored router-introspection implementation for better DOM shim management
+● Based on the commit range v1.0.57..HEAD, only PR #19 is included in this release. Here are the
+  release notes:
+
+  ---
+
+  ## Highlights
+
+  - Pass annotation text through generated click helpers for better test documentation
+  - Resolve Vue compiler and Nuxt kit dependencies from consuming app setups instead of the
+  plugin's own tree
+  - Centralize Playwright video dimension configuration for consistent local and CI test
+  recordings
 
   ## Changes
 
-  ### Bug Fixes
-  - **Router introspection**: Restored router DOM globals after introspection to prevent state
-  pollution between test runs and avoid interfering with downstream code that depends on these
-  globals
+  **Local integration hardening:**
+  - Generated click helpers now accept and document annotation text parameters
+  - Resolve `@vue/compiler-sfc.parse` and `@nuxt/kit` from real consuming-app setups to avoid
+  version mismatches
+  - Add centralized Playwright video dimensions configuration (`playwright-video-dimensions.json`)
+  - Add script to normalize Playwright video settings across local config and CI environments
 
-  ### Testing
-  - Added comprehensive test coverage for router DOM global restoration behavior
+  **Test coverage improvements:**
+  - Add regression tests for project-local Nuxt kit loading
+  - Enhance Nuxt discovery test coverage with real-world integration scenarios
 
   ## Breaking Changes
 
-  None
+  None.
 
   ## Pull Requests Included
 
-  - #18 fix: restore router DOM globals after introspection
-  (https://github.com/immense/vue-pom-generator/pull/18)
+  - [#19](https://github.com/immense/vue-pom-generator/pull/19) fix: harden local app integration
 
   ## Testing
 
-  Added 83 lines of test coverage for router introspection DOM global handling. All existing tests
-   continue to pass.
+  Validated with:
+  - `npm run lint`
+  - `npm run typecheck`
+  - `npm run build`
+  - `npm test`
+
+  All tests passing with 258 net lines added across 13 files.
 

package/dist/index.cjs

@@ -104,6 +104,21 @@ function createLogger(options) {
   };
 }
 const requireFromModule = node_module.createRequire(typeof document === "undefined" ? require("url").pathToFileURL(__filename).href : _documentCurrentScript && _documentCurrentScript.tagName.toUpperCase() === "SCRIPT" && _documentCurrentScript.src || new URL("index.cjs", document.baseURI).href);
+function resolveNuxtKitEntry(cwd) {
+  const attemptResolvers = [
+    node_module.createRequire(path.resolve(cwd, "package.json")),
+    requireFromModule
+  ];
+  let lastError;
+  for (const resolver of attemptResolvers) {
+    try {
+      return resolver.resolve("@nuxt/kit");
+    } catch (error) {
+      lastError = error instanceof Error ? error : new Error(String(error));
+    }
+  }
+  throw lastError ?? new Error("Unknown module resolution error");
+}
 function toUniqueResolvedPaths(paths) {
   return Array.from(new Set(paths.map((value) => path.resolve(value))));
 }
@@ -214,7 +229,7 @@ async function loadNuxtProjectDiscovery(cwd = process.cwd()) {
   let loadNuxtConfig;
   let getLayerDirectories;
   try {
-    const nuxtKitEntry = requireFromModule.resolve("@nuxt/kit");
+    const nuxtKitEntry = resolveNuxtKitEntry(cwd);
     ({ loadNuxtConfig, getLayerDirectories } = await import(node_url.pathToFileURL(nuxtKitEntry).href));
   } catch (error) {
     throw new TypeError(
@@ -403,7 +418,14 @@ function generateClickMethod(methodName, formattedDataTestId, alternateFormatted
     const candidatesExpr = [formattedDataTestId, ...alternates].map(testIdExpression).join(", ");
     const clickMethod = createAsyncMethod(
       name,
-      hasParam(params, "key") ? [...baseParameters, createInlineParameter("wait", { type: "boolean", initializer: "true" })] : [createInlineParameter("wait", { type: "boolean", initializer: "true" })],
+      hasParam(params, "key") ? [
+        ...baseParameters,
+        createInlineParameter("wait", { type: "boolean", initializer: "true" }),
+        createInlineParameter("annotationText", { type: "string", initializer: '""' })
+      ] : [
+        createInlineParameter("wait", { type: "boolean", initializer: "true" }),
+        createInlineParameter("annotationText", { type: "string", initializer: '""' })
+      ],
       (writer) => {
         writer.writeLine(`const candidates = [${candidatesExpr}] as const;`);
         writer.writeLine("let lastError: unknown;");
@@ -411,7 +433,7 @@ function generateClickMethod(methodName, formattedDataTestId, alternateFormatted
           writer.writeLine("const locator = this.locatorByTestId(testId);");
           writer.write("try ").block(() => {
             writer.write("if (await locator.count()) ").block(() => {
-              writer.writeLine('await this.clickLocator(locator, "", wait);');
+              writer.writeLine("await this.clickLocator(locator, annotationText, wait);");
               writer.writeLine("return;");
             });
           });
@@ -422,10 +444,10 @@ function generateClickMethod(methodName, formattedDataTestId, alternateFormatted
         writer.writeLine(`throw (lastError instanceof Error) ? lastError : new Error("[pom] Failed to click any candidate locator for ${name}.");`);
       }
     );
-    const noWaitArgs = argsForForward ? `${argsForForward}, false` : "false";
+    const noWaitArgs = argsForForward ? `${argsForForward}, false, annotationText` : "false, annotationText";
     const noWaitMethod = createAsyncMethod(
       noWaitName,
-      hasParam(params, "key") ? baseParameters : [],
+      hasParam(params, "key") ? [...baseParameters, createInlineParameter("annotationText", { type: "string", initializer: '""' })] : [createInlineParameter("annotationText", { type: "string", initializer: '""' })],
       (writer) => {
         writer.writeLine(`await this.${name}(${noWaitArgs});`);
       }
@@ -434,21 +456,44 @@ function generateClickMethod(methodName, formattedDataTestId, alternateFormatted
   }
   if (hasParam(params, "key")) {
     return [
-      createAsyncMethod(name, [...baseParameters, createInlineParameter("wait", { type: "boolean", initializer: "true" })], (writer) => {
-        writer.writeLine(`await this.clickByTestId(\`${formattedDataTestId}\`, "", wait);`);
-      }),
-      createAsyncMethod(noWaitName, baseParameters, (writer) => {
-        writer.writeLine(`await this.${name}(${argsForForward}, false);`);
-      })
+      createAsyncMethod(
+        name,
+        [
+          ...baseParameters,
+          createInlineParameter("wait", { type: "boolean", initializer: "true" }),
+          createInlineParameter("annotationText", { type: "string", initializer: '""' })
+        ],
+        (writer) => {
+          writer.writeLine(`await this.clickByTestId(\`${formattedDataTestId}\`, annotationText, wait);`);
+        }
+      ),
+      createAsyncMethod(
+        noWaitName,
+        [...baseParameters, createInlineParameter("annotationText", { type: "string", initializer: '""' })],
+        (writer) => {
+          writer.writeLine(`await this.${name}(${argsForForward}, false, annotationText);`);
+        }
+      )
     ];
   }
   return [
-    createAsyncMethod(name, [createInlineParameter("wait", { type: "boolean", initializer: "true" })], (writer) => {
-      writer.writeLine(`await this.clickByTestId("${formattedDataTestId}", "", wait);`);
-    }),
-    createAsyncMethod(noWaitName, [], (writer) => {
-      writer.writeLine(`await this.${name}(false);`);
-    })
+    createAsyncMethod(
+      name,
+      [
+        createInlineParameter("wait", { type: "boolean", initializer: "true" }),
+        createInlineParameter("annotationText", { type: "string", initializer: '""' })
+      ],
+      (writer) => {
+        writer.writeLine(`await this.clickByTestId("${formattedDataTestId}", annotationText, wait);`);
+      }
+    ),
+    createAsyncMethod(
+      noWaitName,
+      [createInlineParameter("annotationText", { type: "string", initializer: '""' })],
+      (writer) => {
+        writer.writeLine(`await this.${name}(false, annotationText);`);
+      }
+    )
   ];
 }
 function generateRadioMethod(methodName, formattedDataTestId) {
@@ -2893,10 +2938,13 @@ Fix: make the element identifiable (e.g. add id/name/inner text or use a more sp
           formattedDataTestId: formattedDataTestIdForPom
         },
         keyLiteral: rawValue,
-        params: { wait: "boolean = true" }
+        params: { wait: "boolean = true", annotationText: 'string = ""' }
       });
       if (added) {
-        registerGeneratedMethodSignature(generatedName, { params: `wait: boolean = true`, argNames: ["wait"] });
+        registerGeneratedMethodSignature(generatedName, {
+          params: `wait: boolean = true, annotationText: string = ""`,
+          argNames: ["wait", "annotationText"]
+        });
       }
     }
     return;
@@ -8060,6 +8108,13 @@ function tryCreateElementMetadata(args) {
   };
   return metadata;
 }
+function resolveCompilerSfcParse(compilerSfc2) {
+  const parse = compilerSfc2.parse ?? compilerSfc2.default?.parse;
+  if (typeof parse !== "function") {
+    throw new TypeError("[vue-pom-generator] Failed to resolve @vue/compiler-sfc.parse.");
+  }
+  return parse;
+}
 function extractMetadataAfterTransform(ast, componentName, elementMetadata, semanticNameMap, testIdAttribute) {
   const componentMetadata = /* @__PURE__ */ new Map();
   function traverseNode(node) {
@@ -8256,7 +8311,8 @@ function createVuePluginWithTestIds(options) {
       }
       const componentName = getComponentNameFromPath(cleanPath);
       loggerRef.current.debug(`Collecting metadata for ${cleanPath} (component: ${componentName})`);
-      const { parse } = await import("@vue/compiler-sfc");
+      const compilerSfc2 = await import("@vue/compiler-sfc");
+      const parse = resolveCompilerSfcParse(compilerSfc2);
       const compilerDom2 = await import("@vue/compiler-dom");
       const compile = compilerDom2.compile;
       const { descriptor } = parse(code, { filename: cleanPath });