@qaguardian/playwright-score 0.1.12 → 0.1.13

package/dist/formatters/markdown.js

@@ -24,7 +24,7 @@ export function formatMarkdown(result) {
     }
     if (result.skippedFiles?.length) {
         lines.push('');
-        lines.push(`<details><summary>Skipped — looks like a non-Playwright test (Jest/Vitest/RTL) (${result.skippedFiles.length})</summary>`);
+        lines.push(`<details><summary>Skipped — looks like a non-Playwright test (Jest/Vitest/RTL/Cypress) (${result.skippedFiles.length})</summary>`);
         lines.push('');
         for (const f of result.skippedFiles) {
             lines.push(`- \`${f}\``);

package/dist/formatters/text.js

@@ -26,7 +26,7 @@ export function formatText(result) {
     }
     if (result.skippedFiles?.length) {
         lines.push('');
-        lines.push(`Skipped (looks like a non-Playwright test — Jest/Vitest/RTL — ${result.skippedFiles.length}):`);
+        lines.push(`Skipped (looks like a non-Playwright test — Jest/Vitest/RTL/Cypress — ${result.skippedFiles.length}):`);
         for (const f of result.skippedFiles.slice(0, 20)) {
             lines.push(`  ${f}`);
         }

package/dist/index.js

@@ -14,7 +14,20 @@ export { formatJson } from './formatters/json.js';
 export { formatText } from './formatters/text.js';
 export { formatMarkdown } from './formatters/markdown.js';
 export { formatSarif } from './formatters/sarif.js';
-const SPEC_GLOBS = ['**/*.{spec,test}.{ts,tsx,js,jsx}', '**/*.spec.ts', '**/*.test.ts'];
+// `.e2e.` is a real, common convention for disambiguating end-to-end specs
+// from unit tests living in the same tree — verified against a
+// well-known, large real-world repo (cal.com: 53 real Playwright specs,
+// all named *.e2e.ts, zero *.spec.ts/*.test.ts) where a directory scan
+// previously hard-failed with "no files matched" despite a fully healthy,
+// populated suite. looksLikeNonPlaywrightTest (see metrics.ts) is the
+// safety net for a repo that uses `.e2e.` for a *different* framework —
+// Cypress specifically was common enough here to get its own explicit
+// check alongside the existing Jest/Vitest/RTL ones.
+const SPEC_GLOBS = [
+    '**/*.{spec,test,e2e}.{ts,tsx,js,jsx}',
+    '**/*.spec.ts',
+    '**/*.test.ts',
+];
 // Directory/glob expansion must never sweep in vendored or generated code —
 // verified as a real bug: scanning a project root with a plain node_modules
 // dependency that ships its own *.spec.ts files silently included them
@@ -43,7 +56,7 @@ const DEFAULT_IGNORE_GLOBS = [
  * `expanded`: everything else (matched via a directory scan or a glob
  * pattern) — subject to the looksLikePlaywrightSpec filter in scorePaths,
  * since a broad `**\/*.test.ts`-style match can just as easily sweep in
- * unrelated Jest/Vitest/RTL unit tests sitting in the same repo.
+ * unrelated Jest/Vitest/RTL/Cypress unit tests sitting in the same repo.
  */
 function expandPaths(inputs, cwd) {
     const explicit = new Set();
@@ -166,7 +179,7 @@ export async function scorePaths(options) {
             {
                 rule: 'playwright-score/no-files',
                 severity: 'error',
-                message: `${expanded.length} file(s) matched but all look like non-Playwright tests (Jest/Vitest/RTL) — nothing to score.`,
+                message: `${expanded.length} file(s) matched but all look like non-Playwright tests (Jest/Vitest/RTL/Cypress) — nothing to score.`,
                 file: cwd,
                 dimension: 'structure',
             },

package/dist/metrics.d.ts

@@ -1,11 +1,11 @@
 import type { Finding, LocatorCounts } from './types.js';
 /**
  * Heuristic: does this file show positive evidence of being a *non*-
- * Playwright test (Jest/Vitest/Mocha/Jasmine/React Testing Library), as
- * opposed to a Playwright spec that a broad `**\/*.test.ts` glob would
- * otherwise sweep in alongside real e2e specs? Used only to decide whether
- * to auto-exclude a file discovered via directory/glob expansion — never
- * applied to a file path the caller named explicitly.
+ * Playwright test (Jest/Vitest/Mocha/Jasmine/React Testing Library/
+ * Cypress), as opposed to a Playwright spec that a broad `**\/*.test.ts`
+ * glob would otherwise sweep in alongside real e2e specs? Used only to
+ * decide whether to auto-exclude a file discovered via directory/glob
+ * expansion — never applied to a file path the caller named explicitly.
  */
 export declare function looksLikeNonPlaywrightTest(source: string): boolean;
 export declare function countLocators(source: string, file?: string): LocatorCounts;

package/dist/metrics.js

@@ -13,18 +13,27 @@ import { countSloc } from './sloc.js';
 // in one real suite). Denylisting known non-Playwright frameworks is much
 // safer — a missed rare/unusual file just scores harmlessly, where a
 // falsely-skipped real spec silently stops being scored at all.
-const NON_PLAYWRIGHT_IMPORT_RE = /from\s+['"](?:vitest|jest|@jest\/[a-z-]+|mocha|jasmine|chai|@testing-library\/[a-z-]+)['"]/;
+const NON_PLAYWRIGHT_IMPORT_RE = /from\s+['"](?:vitest|jest|@jest\/[a-z-]+|mocha|jasmine|chai|@testing-library\/[a-z-]+|cypress)['"]/;
 const RTL_API_RE = /\bscreen\.(?:getBy|queryBy|findBy|getAllBy|queryAllBy|findAllBy)\w*\(|\btoBeInTheDocument\(/;
+// Cypress's `cy` command object is globally available without an import in
+// the framework's default setup, the same way RTL's `screen` often is —
+// matched on specific, common Cypress command names (not a bare `cy.`
+// prefix) to avoid colliding with an unrelated local variable named `cy`.
+// Added alongside SPEC_GLOBS picking up `*.e2e.*` by default (see
+// index.ts): `.e2e.` is Cypress's own scaffolding convention too, so
+// broadening spec discovery to that suffix needed this as a companion
+// safety net, not just Playwright's own conventions.
+const CYPRESS_API_RE = /\bcy\.(?:visit|get|contains|click|type|should|intercept|wait|request|fixture)\(/;
 /**
  * Heuristic: does this file show positive evidence of being a *non*-
- * Playwright test (Jest/Vitest/Mocha/Jasmine/React Testing Library), as
- * opposed to a Playwright spec that a broad `**\/*.test.ts` glob would
- * otherwise sweep in alongside real e2e specs? Used only to decide whether
- * to auto-exclude a file discovered via directory/glob expansion — never
- * applied to a file path the caller named explicitly.
+ * Playwright test (Jest/Vitest/Mocha/Jasmine/React Testing Library/
+ * Cypress), as opposed to a Playwright spec that a broad `**\/*.test.ts`
+ * glob would otherwise sweep in alongside real e2e specs? Used only to
+ * decide whether to auto-exclude a file discovered via directory/glob
+ * expansion — never applied to a file path the caller named explicitly.
  */
 export function looksLikeNonPlaywrightTest(source) {
-    return NON_PLAYWRIGHT_IMPORT_RE.test(source) || RTL_API_RE.test(source);
+    return NON_PLAYWRIGHT_IMPORT_RE.test(source) || RTL_API_RE.test(source) || CYPRESS_API_RE.test(source);
 }
 const NATIVE_METHODS = new Set([
     'getByRole',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@qaguardian/playwright-score",
-  "version": "0.1.12",
+  "version": "0.1.13",
   "description": "Deterministic, AI-free Playwright spec quality score (sqs-v1). Lint + score with standard and guardian profiles.",
   "type": "module",
   "main": "./dist/index.js",