@qaguardian/playwright-score 0.1.13 → 0.1.14

package/README.md

@@ -16,7 +16,7 @@ Lint + score Playwright tests with:
 
 The score never calls an LLM. AI may *generate* or *repair* code using findings; **rules grade code**.
 
-See [METHODOLOGY.md](./METHODOLOGY.md) for the frozen formula, or the full product write-up on the [landing page](https://qaguardian.com/open-source/playwright-score).
+See [METHODOLOGY.md](./METHODOLOGY.md) for the frozen formula, [VALIDATION.md](./VALIDATION.md) for real-world results against public repos (cal.com, Grafana, Mattermost, ...) and a re-runnable script to reproduce them, or the full product write-up on the [landing page](https://qaguardian.com/open-source/playwright-score).
 
 ## Install
 

package/dist/cli.js

@@ -34,6 +34,14 @@ Website: https://qaguardian.com/open-source/playwright-score
 Methodology: METHODOLOGY.md (sqs-v1)
 `);
 }
+// Deliberately never process.exit() right after a console.log/console.error
+// — for a large enough write (verified against a real suite: a JSON report
+// with 1000+ findings, ~370KB) stdout to a pipe (not a TTY) is asynchronous,
+// and process.exit() can terminate the process before the OS has drained
+// the write, silently truncating the output exactly at the pipe buffer
+// size (64KB on macOS/Linux). Setting process.exitCode and returning lets
+// Node exit naturally once pending I/O actually completes, which is the
+// standard fix for this class of bug.
 async function main() {
     const args = process.argv.slice(2);
     if (args.includes('--version') || args.includes('-v')) {
@@ -46,11 +54,13 @@ async function main() {
         catch {
             console.log('0.1.8');
         }
-        process.exit(0);
+        process.exitCode = 0;
+        return;
     }
     if (args.length === 0 || args.includes('--help') || args.includes('-h')) {
         printHelp();
-        process.exit(args.length === 0 ? 2 : 0);
+        process.exitCode = args.length === 0 ? 2 : 0;
+        return;
     }
     let profile = 'standard';
     let threshold;
@@ -63,21 +73,24 @@ async function main() {
             profile = args[++i];
             if (profile !== 'standard' && profile !== 'guardian') {
                 console.error(`Invalid profile: ${profile}`);
-                process.exit(2);
+                process.exitCode = 2;
+                return;
             }
         }
         else if (a === '--threshold') {
             threshold = Number(args[++i]);
             if (Number.isNaN(threshold) || threshold < 0 || threshold > 100) {
                 console.error('threshold must be 0-100');
-                process.exit(2);
+                process.exitCode = 2;
+                return;
             }
         }
         else if (a === '--format') {
             const f = args[++i];
             if (f !== 'text' && f !== 'json' && f !== 'markdown' && f !== 'sarif') {
                 console.error(`Invalid format: ${f}`);
-                process.exit(2);
+                process.exitCode = 2;
+                return;
             }
             format = f;
         }
@@ -89,7 +102,8 @@ async function main() {
         }
         else if (a.startsWith('-')) {
             console.error(`Unknown option: ${a}`);
-            process.exit(2);
+            process.exitCode = 2;
+            return;
         }
         else {
             paths.push(a);
@@ -97,7 +111,8 @@ async function main() {
     }
     if (paths.length === 0) {
         console.error('At least one path is required');
-        process.exit(2);
+        process.exitCode = 2;
+        return;
     }
     try {
         const result = await scorePaths({ paths, profile, threshold });
@@ -122,11 +137,11 @@ async function main() {
         else {
             console.log(body);
         }
-        process.exit(result.pass ? 0 : 1);
+        process.exitCode = result.pass ? 0 : 1;
     }
     catch (err) {
         console.error(err instanceof Error ? err.message : err);
-        process.exit(2);
+        process.exitCode = 2;
     }
 }
 main();

package/dist/index.js

@@ -14,17 +14,17 @@ export { formatJson } from './formatters/json.js';
 export { formatText } from './formatters/text.js';
 export { formatMarkdown } from './formatters/markdown.js';
 export { formatSarif } from './formatters/sarif.js';
-// `.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.
+// `.e2e.`/`.e2e-spec.`/`.e2e-test.` are real, common conventions for
+// disambiguating end-to-end specs from unit tests living in the same tree
+// — verified against two well-known, large real-world repos where a
+// directory scan previously hard-failed with "no files matched" despite a
+// fully healthy, populated suite: cal.com (53 specs, all *.e2e.ts) and
+// Immich (52 specs, all *.e2e-spec.ts). 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,test,e2e,e2e-spec,e2e-test}.{ts,tsx,js,jsx}',
     '**/*.spec.ts',
     '**/*.test.ts',
 ];

package/package.json

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