Promote Playwright Workspace V2 tests to required validation gate with fail-fast execution - PR_11_319

Author: DavidQ

docs/dev/codex_commands.md

@@ -208,3 +208,10 @@ PR_11_318E
 ```bash
 npx @openai/codex run --model gpt-5.3-codex --reasoning medium "Implement PR_11_318E: Configure Playwright HTML reporter to open automatically after targeted UI test runs."
 ```
+
+---
+PR_11_319
+
+```bash
+npx @openai/codex run --model gpt-5.3-codex --reasoning medium "Implement PR_11_319: Add required Workspace V2 Playwright gate command with explicit pass/fail summary and non-zero failure handling."
+```

docs/dev/commit_comment.txt

@@ -1 +1 @@
-Enable automatic Playwright HTML report opening while preserving existing test output settings - PR 11.318E
+Promote Workspace V2 Playwright validation into required gate script with explicit non-zero failure handling - PR 11.319

docs/dev/reports/PR_11_319_report.md

@@ -0,0 +1,33 @@
+# PR_11_319 Report
+
+## Purpose
+Promote Workspace V2 Playwright UI validation into a required gate command with explicit failure handling and summary output.
+
+## Files Changed
+- `package.json`
+- `scripts/run-workspace-v2-playwright-gate.mjs`
+- `docs/pr/PR_11_319_WORKSPACE_V2_PLAYWRIGHT_GATE/PLAN_PR.md`
+- `docs/pr/PR_11_319_WORKSPACE_V2_PLAYWRIGHT_GATE/BUILD_PR.md`
+- `docs/dev/reports/PR_11_319_report.md`
+- `docs/dev/codex_commands.md`
+- `docs/dev/commit_comment.txt`
+
+## Implementation Summary
+- Added `npm run test:workspace-v2` as a dedicated Workspace V2 Playwright gate.
+- Added deterministic gate runner script that:
+  - runs Playwright via local dependency CLI
+  - prints pass/fail summary counts
+  - exits non-zero on any gate failure condition
+- Existing Playwright output contract remains in `tests/results/**` and HTML report generation remains enabled via current config.
+
+## Validation Commands
+- `node --check scripts/run-workspace-v2-playwright-gate.mjs` -> **PASS**
+- `node --check playwright.config.cjs` -> **PASS**
+- `npm run test:workspace-v2` -> **PASS**
+
+## Console Summary Example
+- `Workspace V2 Playwright Gate Summary: passed=1 failed=0`
+
+## Full Samples Smoke Decision
+- **Skipped** full samples smoke test.
+- Reason: this PR only adds test runner integration and does not modify runtime/sample behavior.

docs/pr/PR_11_319_WORKSPACE_V2_PLAYWRIGHT_GATE/BUILD_PR.md

@@ -0,0 +1,21 @@
+# BUILD_PR_11_319
+
+## Implementation
+- Added script entry in `package.json`:
+  - `test:workspace-v2`: `node ./scripts/run-workspace-v2-playwright-gate.mjs`
+- Added new gate runner:
+  - `scripts/run-workspace-v2-playwright-gate.mjs`
+  - Executes Playwright UI spec through local Playwright CLI:
+    - `node node_modules/@playwright/test/cli.js test tests/ui/workspace-v2.asset-manager.spec.js`
+  - Streams Playwright stdout/stderr.
+  - Parses and prints clear summary:
+    - `Workspace V2 Playwright Gate Summary: passed=<n> failed=<n>`
+  - Returns non-zero exit when:
+    - Playwright process fails to execute
+    - Playwright exits non-zero
+    - parsed failed count is greater than zero
+
+## Validation
+- `node --check scripts/run-workspace-v2-playwright-gate.mjs`
+- `node --check playwright.config.cjs`
+- `npm run test:workspace-v2`

docs/pr/PR_11_319_WORKSPACE_V2_PLAYWRIGHT_GATE/PLAN_PR.md

@@ -0,0 +1,18 @@
+# PLAN_PR_11_319
+
+## Purpose
+Promote Workspace V2 Playwright UI coverage into a required validation gate command with explicit pass/fail handling.
+
+## Scope
+- `package.json`
+- `scripts/run-workspace-v2-playwright-gate.mjs`
+- `docs/dev/reports/PR_11_319_report.md`
+- `docs/dev/codex_commands.md`
+- `docs/dev/commit_comment.txt`
+
+## Steps
+1. Add `npm run test:workspace-v2` script entry.
+2. Implement a dedicated gate runner that executes Playwright and prints clear pass/fail summary counts.
+3. Enforce non-zero exit on any execution or test failure.
+4. Keep Playwright output/report behavior using existing `tests/results/**` configuration.
+5. Run targeted syntax checks for changed JS/CJS files.

package.json

@@ -11,6 +11,7 @@
     "check:phase24-closeout-guard": "node tools/dev/checkPhase24CloseoutExecutionGuard.mjs",
     "check:style-system-guard": "node tools/dev/checkStyleSystemGuard.mjs",
     "check:games-template-contract": "node ./scripts/validate-games-template-contract.mjs",
+    "test:workspace-v2": "node ./scripts/run-workspace-v2-playwright-gate.mjs",
     "test:launch-smoke": "node ./tests/runtime/LaunchSmokeAllEntries.test.mjs",
     "test:launch-smoke:games": "node ./tests/runtime/LaunchSmokeAllEntries.test.mjs --games",
     "test:workspace-manager:games": "node ./tests/runtime/GamesIndexWorkspaceManagerOpen.test.mjs",

scripts/run-workspace-v2-playwright-gate.mjs

@@ -0,0 +1,45 @@
+/*
+Toolbox Aid
+David Quesenberry
+05/03/2026
+run-workspace-v2-playwright-gate.mjs
+*/
+import path from "node:path";
+import { spawnSync } from "node:child_process";
+import { fileURLToPath } from "node:url";
+
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = path.dirname(__filename);
+const repoRoot = path.resolve(__dirname, "..");
+const command = process.execPath;
+const args = [
+  path.join(repoRoot, "node_modules", "@playwright", "test", "cli.js"),
+  "test",
+  "tests/ui/workspace-v2.asset-manager.spec.js"
+];
+
+const result = spawnSync(command, args, {
+  cwd: repoRoot,
+  encoding: "utf8",
+  stdio: ["ignore", "pipe", "pipe"]
+});
+
+const stdout = typeof result.stdout === "string" ? result.stdout : "";
+const stderr = typeof result.stderr === "string" ? result.stderr : "";
+if (stdout) process.stdout.write(stdout);
+if (stderr) process.stderr.write(stderr);
+
+const combinedOutput = `${stdout}\n${stderr}`;
+const passedMatch = combinedOutput.match(/(^|\n)\s*(\d+)\s+passed\b/);
+const failedMatch = combinedOutput.match(/(^|\n)\s*(\d+)\s+failed\b/);
+const passedCount = passedMatch ? Number.parseInt(passedMatch[2], 10) : 0;
+const failedCount = failedMatch ? Number.parseInt(failedMatch[2], 10) : 0;
+
+console.log(`Workspace V2 Playwright Gate Summary: passed=${passedCount} failed=${failedCount}`);
+
+if (result.error) {
+  console.error(`Workspace V2 Playwright gate execution failed: ${result.error.message}`);
+  process.exitCode = 1;
+} else if (result.status !== 0 || failedCount > 0) {
+  process.exitCode = typeof result.status === "number" && result.status !== 0 ? result.status : 1;
+}