Measure targeted test lane performance and verify monolith-safe routing before pruning slow Playwright paths - PR_26146_038-test-cleanup-performance-measurement-and-routing

ToolboxAid · ToolboxAid · commit 5704b83d2391 · 2026-05-26T17:21:54.000-04:00
diff --git a/docs/dev/reports/dependency_gating_report.md b/docs/dev/reports/dependency_gating_report.md
@@ -1,6 +1,6 @@
 # Dependency Gating Report
 
-Generated: 2026-05-26T20:47:41.392Z
+Generated: 2026-05-26T21:15:52.422Z
 Status: PASS
 
 ## Gate Order
diff --git a/docs/dev/reports/dependency_hydration_reuse_report.md b/docs/dev/reports/dependency_hydration_reuse_report.md
@@ -1,6 +1,6 @@
 # Dependency Hydration Reuse Report
 
-Generated: 2026-05-26T20:47:41.393Z
+Generated: 2026-05-26T21:15:52.423Z
 Status: PASS
 
 ## Summary
diff --git a/docs/dev/reports/execution_graph_reuse_report.md b/docs/dev/reports/execution_graph_reuse_report.md
@@ -1,6 +1,6 @@
 # Execution Graph Reuse Report
 
-Generated: 2026-05-26T20:47:41.393Z
+Generated: 2026-05-26T21:15:52.423Z
 Status: PASS
 
 ## Summary
diff --git a/docs/dev/reports/failure_fingerprint_report.md b/docs/dev/reports/failure_fingerprint_report.md
@@ -1,6 +1,6 @@
 # Failure Fingerprint Report
 
-Generated: 2026-05-26T20:49:40.108Z
+Generated: 2026-05-26T21:18:42.198Z
 Status: WARN
 
 ## Summary
diff --git a/docs/dev/reports/filesystem_scan_reduction_report.md b/docs/dev/reports/filesystem_scan_reduction_report.md
@@ -1,6 +1,6 @@
 # Filesystem Scan Reduction Report
 
-Generated: 2026-05-26T20:47:41.366Z
+Generated: 2026-05-26T21:15:52.390Z
 Status: PASS
 
 ## Scan Enforcement
diff --git a/docs/dev/reports/incremental_validation_report.md b/docs/dev/reports/incremental_validation_report.md
@@ -1,6 +1,6 @@
 # Incremental Validation Report
 
-Generated: 2026-05-26T20:47:41.394Z
+Generated: 2026-05-26T21:15:52.424Z
 Status: PASS
 
 ## Reuse Summary
diff --git a/docs/dev/reports/lane_compilation_report.md b/docs/dev/reports/lane_compilation_report.md
@@ -1,6 +1,6 @@
 # Lane Compilation Report
 
-Generated: 2026-05-26T20:47:41.391Z
+Generated: 2026-05-26T21:15:52.422Z
 Status: PASS
 
 ## Lane Graph
diff --git a/docs/dev/reports/lane_deduplication_report.md b/docs/dev/reports/lane_deduplication_report.md
@@ -1,6 +1,6 @@
 # Lane Deduplication Report
 
-Generated: 2026-05-26T20:47:41.392Z
+Generated: 2026-05-26T21:15:52.422Z
 Status: PASS
 
 ## Summary
diff --git a/docs/dev/reports/lane_input_validation_report.md b/docs/dev/reports/lane_input_validation_report.md
@@ -1,6 +1,6 @@
 # Lane Input Validation Report
 
-Generated: 2026-05-26T20:47:41.394Z
+Generated: 2026-05-26T21:15:52.424Z
 Status: PASS
 
 ## Input Files
diff --git a/docs/dev/reports/lane_runtime_optimization_report.md b/docs/dev/reports/lane_runtime_optimization_report.md
@@ -1,6 +1,6 @@
 # Lane Runtime Optimization Report
 
-Generated: 2026-05-26T20:47:41.392Z
+Generated: 2026-05-26T21:15:52.422Z
 Status: PASS
 
 ## Runtime Cost Summary
diff --git a/docs/dev/reports/lane_snapshot_report.md b/docs/dev/reports/lane_snapshot_report.md
@@ -1,6 +1,6 @@
 # Lane Snapshot Report
 
-Generated: 2026-05-26T20:47:41.393Z
+Generated: 2026-05-26T21:15:52.423Z
 Status: PASS
 Snapshot directory: docs/dev/reports/lane_snapshots
 
diff --git a/docs/dev/reports/lane_warm_start_report.md b/docs/dev/reports/lane_warm_start_report.md
@@ -1,6 +1,6 @@
 # Lane Warm-Start Report
 
-Generated: 2026-05-26T20:47:41.392Z
+Generated: 2026-05-26T21:15:52.422Z
 Status: PASS
 Warm-start directory: docs/dev/reports/lane_warm_starts
 
diff --git a/docs/dev/reports/persistent_lane_manifest_report.md b/docs/dev/reports/persistent_lane_manifest_report.md
@@ -1,6 +1,6 @@
 # Persistent Lane Manifest Report
 
-Generated: 2026-05-26T20:47:41.394Z
+Generated: 2026-05-26T21:15:52.424Z
 Status: PASS
 Manifest directory: docs/dev/reports/lane_manifests
 
diff --git a/docs/dev/reports/playwright_discovery_ownership_report.md b/docs/dev/reports/playwright_discovery_ownership_report.md
@@ -1,6 +1,6 @@
 # Playwright Discovery Ownership Report
 
-Generated: 2026-05-26T20:47:41.361Z
+Generated: 2026-05-26T21:15:52.384Z
 Status: PASS
 
 ## Discovery-Time Ownership
diff --git a/docs/dev/reports/playwright_discovery_scope_report.md b/docs/dev/reports/playwright_discovery_scope_report.md
@@ -1,6 +1,6 @@
 # Playwright Discovery Scope Report
 
-Generated: 2026-05-26T20:47:41.364Z
+Generated: 2026-05-26T21:15:52.388Z
 Status: PASS
 Scoped discovery: Yes
 
diff --git a/docs/dev/reports/playwright_structure_audit.md b/docs/dev/reports/playwright_structure_audit.md
@@ -1,6 +1,6 @@
 # Playwright Structure Audit
 
-Generated: 2026-05-26T20:47:41.348Z
+Generated: 2026-05-26T21:15:52.369Z
 Status: PASS
 
 ## Lane Directories
diff --git a/docs/dev/reports/retry_suppression_report.md b/docs/dev/reports/retry_suppression_report.md
@@ -1,6 +1,6 @@
 # Retry Suppression Report
 
-Generated: 2026-05-26T20:49:40.108Z
+Generated: 2026-05-26T21:18:42.199Z
 Status: WARN
 
 ## Summary
diff --git a/docs/dev/reports/static_validation_report.md b/docs/dev/reports/static_validation_report.md
@@ -1,6 +1,6 @@
 # Static Validation Report
 
-Generated: 2026-05-26T20:47:41.384Z
+Generated: 2026-05-26T21:15:52.409Z
 Status: PASS
 Static only: No
 Dry run: No
diff --git a/docs/dev/reports/targeted_file_manifest_report.md b/docs/dev/reports/targeted_file_manifest_report.md
@@ -1,6 +1,6 @@
 # Targeted File Manifest Report
 
-Generated: 2026-05-26T20:47:41.394Z
+Generated: 2026-05-26T21:15:52.424Z
 Status: PASS
 
 ## Manifest-Generated Lane Inputs
diff --git a/docs/dev/reports/test_cleanup_performance_report.md b/docs/dev/reports/test_cleanup_performance_report.md
@@ -0,0 +1,59 @@
+# Test Cleanup Performance Report
+
+Generated: 2026-05-26T21:18:42.199Z
+Status: WARN
+
+## Cost Summary
+
+Total measured lane elapsed time: 169.71s
+Actual browser launch count: 4
+Scheduled browser launch count: 4
+Baseline browser launch count: 8
+Skipped lanes: 2
+Reused manifests: 4
+Reused snapshots: 4
+Cached validations reused: 18
+Prevented broad execution: 3
+Prevented reruns: 0
+Prevented redundant browser launches: 4
+Prevented graph rebuilds: 4
+Prevented redundant dependency traversal: 4
+
+## Lane Elapsed Time
+
+| Lane | Status | Elapsed | Browser Launches | Reason |
+| --- | --- | --- | --- | --- |
+| workspace-contract | SKIP | 0ms | 0 | Lane was not selected for this targeted run. |
+| tool-runtime | PASS | 98.86s | 2 | Tool runtime lane validates focused tool behavior without treating unrelated stale product assertions as blockers. |
+| game-runtime | FAIL | 37.40s | 1 | Game runtime lane validates explicit game-owned Playwright behavior only. |
+| integration | PASS | 31.19s | 1 | Integration lane validates explicit cross-surface handoffs only; broad all-game thumbnail coverage is outside the default targeted lane. |
+| engine-src | PASS | 2.26s | 0 | Engine/src lane validates reusable runtime surfaces through targeted node tests. |
+| samples | SKIP | 0ms | 0 | Lane was not selected for this targeted run. |
+
+## Slowest Tests
+
+| Lane | Duration | Test | Command |
+| --- | --- | --- | --- |
+| tool-runtime | 19.10s | tests\playwright\tools\AssetManagerV2.spec.mjs:342:3 > Asset Manager V2 > launches Asset Manager V2 with temporary UAT context and schema-complete asset controls | C:\nvm4w\nodejs\node.exe C:\Users\davidq\Documents\GitHub\HTML-JavaScript-Gaming\node_modules\@playwright\test\cli.js test tests/playwright/tools/AssetManagerV2.spec.mjs --grep "launch guard|temporary UAT context|rejects non-Workspace" --project=playwright --workers=1 --reporter=list |
+| integration | 14.50s | tests\playwright\integration\GameIndexPreviewManifestResolution.spec.mjs:144:1 > games index resolves Pong thumbnail from manifest preview role | C:\nvm4w\nodejs\node.exe C:\Users\davidq\Documents\GitHub\HTML-JavaScript-Gaming\node_modules\@playwright\test\cli.js test tests/playwright/integration/GameIndexPreviewManifestResolution.spec.mjs --grep Pong --project=playwright --workers=1 --reporter=list |
+| tool-runtime | 10.10s | tests\playwright\tools\PreviewGeneratorV2Baseline.spec.mjs:488:3 > Preview Generator V2 baseline > generates real batch output with skip, failure, status, and summary assertions | C:\nvm4w\nodejs\node.exe C:\Users\davidq\Documents\GitHub\HTML-JavaScript-Gaming\node_modules\@playwright\test\cli.js test tests/playwright/tools/PreviewGeneratorV2Baseline.spec.mjs tests/playwright/tools/CollisionInspectorV2.spec.mjs --project=playwright --workers=1 --reporter=list |
+| tool-runtime | 8.10s | tests\playwright\tools\CollisionInspectorV2.spec.mjs:257:3 > Collision Inspector V2 > loads a game manifest and reports live vector, pixel, bounds, and hybrid collisions | C:\nvm4w\nodejs\node.exe C:\Users\davidq\Documents\GitHub\HTML-JavaScript-Gaming\node_modules\@playwright\test\cli.js test tests/playwright/tools/PreviewGeneratorV2Baseline.spec.mjs tests/playwright/tools/CollisionInspectorV2.spec.mjs --project=playwright --workers=1 --reporter=list |
+| tool-runtime | 7.90s | tests\playwright\tools\PreviewGeneratorV2Baseline.spec.mjs:352:3 > Preview Generator V2 baseline > exercises controls, required-field gating, accordions, paths layout, and status clear | C:\nvm4w\nodejs\node.exe C:\Users\davidq\Documents\GitHub\HTML-JavaScript-Gaming\node_modules\@playwright\test\cli.js test tests/playwright/tools/PreviewGeneratorV2Baseline.spec.mjs tests/playwright/tools/CollisionInspectorV2.spec.mjs --project=playwright --workers=1 --reporter=list |
+| tool-runtime | 6.80s | tests\playwright\tools\PreviewGeneratorV2Baseline.spec.mjs:447:3 > Preview Generator V2 baseline > phase folder input enumerates existing sample folders only | C:\nvm4w\nodejs\node.exe C:\Users\davidq\Documents\GitHub\HTML-JavaScript-Gaming\node_modules\@playwright\test\cli.js test tests/playwright/tools/PreviewGeneratorV2Baseline.spec.mjs tests/playwright/tools/CollisionInspectorV2.spec.mjs --project=playwright --workers=1 --reporter=list |
+| game-runtime | 5.80s | tests\playwright\games\AsteroidsBeatTiming.spec.mjs:14:1 > uses weighted active asteroid objects for beat cadence | C:\nvm4w\nodejs\node.exe C:\Users\davidq\Documents\GitHub\HTML-JavaScript-Gaming\node_modules\@playwright\test\cli.js test tests/playwright/games/AsteroidsBackgroundAssetResolution.spec.mjs tests/playwright/games/AsteroidsBeatTiming.spec.mjs tests/playwright/games/AsteroidsGameSceneCleanup.spec.mjs tests/playwright/games/AsteroidsShipStateVisuals.spec.mjs --project=playwright --workers=1 --reporter=list |
+| game-runtime | 4.30s | tests\playwright\games\AsteroidsBackgroundAssetResolution.spec.mjs:158:1 > omits optional Asteroids bezel image when Asset Manager bezel role is absent | C:\nvm4w\nodejs\node.exe C:\Users\davidq\Documents\GitHub\HTML-JavaScript-Gaming\node_modules\@playwright\test\cli.js test tests/playwright/games/AsteroidsBackgroundAssetResolution.spec.mjs tests/playwright/games/AsteroidsBeatTiming.spec.mjs tests/playwright/games/AsteroidsGameSceneCleanup.spec.mjs tests/playwright/games/AsteroidsShipStateVisuals.spec.mjs --project=playwright --workers=1 --reporter=list |
+| tool-runtime | 4.10s | tests\playwright\tools\PreviewGeneratorV2Baseline.spec.mjs:586:3 > Preview Generator V2 baseline > launches Tool Template V2 with runtime-valid controls | C:\nvm4w\nodejs\node.exe C:\Users\davidq\Documents\GitHub\HTML-JavaScript-Gaming\node_modules\@playwright\test\cli.js test tests/playwright/tools/PreviewGeneratorV2Baseline.spec.mjs tests/playwright/tools/CollisionInspectorV2.spec.mjs --project=playwright --workers=1 --reporter=list |
+| game-runtime | 3.70s | tests\playwright\games\AsteroidsShipStateVisuals.spec.mjs:14:1 > validates Asteroids ship visual states from manifest runtime rendering | C:\nvm4w\nodejs\node.exe C:\Users\davidq\Documents\GitHub\HTML-JavaScript-Gaming\node_modules\@playwright\test\cli.js test tests/playwright/games/AsteroidsBackgroundAssetResolution.spec.mjs tests/playwright/games/AsteroidsBeatTiming.spec.mjs tests/playwright/games/AsteroidsGameSceneCleanup.spec.mjs tests/playwright/games/AsteroidsShipStateVisuals.spec.mjs --project=playwright --workers=1 --reporter=list |
+
+## Prevented Broad Execution
+
+- Workspace V2 lane was not scheduled without explicit selection.
+- Full samples smoke stayed skipped/on-request.
+- Unselected lane directories stayed outside targeted discovery.
+
+## Runtime Savings Observations
+
+- Performance reporting is generated from the targeted lane runner without launching additional broad suites.
+- Browser launch counts are counted from scheduled Playwright command groups, not from recursive test discovery.
+- Manifest, snapshot, and validation-cache reuse are reported from deterministic pre-runtime state.
+- Full samples smoke remains skipped unless an explicit samples scope is active.
diff --git a/docs/dev/reports/test_cleanup_routing_report.md b/docs/dev/reports/test_cleanup_routing_report.md
@@ -0,0 +1,57 @@
+# Test Cleanup Routing Report
+
+Generated: 2026-05-26T21:18:42.200Z
+Status: PASS
+
+## Representative Routing Cases
+
+| Case | Changed Files | Expected Lanes | Actual Lanes | Status | Reason |
+| --- | --- | --- | --- | --- | --- |
+| docs-only change | docs/dev/PROJECT_INSTRUCTIONS.md | none | none | PASS | Docs/workflow-only changes use static review evidence; runtime lanes, Workspace V2, and samples stay explicit/on-request. |
+| tool change | tools/audio-sfx-playground-v2/index.js | tool-runtime | tool-runtime | PASS | Tool-owned runtime/UI changes route to the affected tool-runtime lane only. |
+| game change | games/asteroids/asteroids.js | game-runtime | game-runtime | PASS | Game-owned runtime changes route to game-runtime without treating tool lanes as blockers. |
+| src change | src/input/InputMap.js | engine-src | engine-src | PASS | Reusable src/ capability changes route to engine-src validation first. |
+| integration change | tests/playwright/integration/GameIndexPreviewManifestResolution.spec.mjs | integration | integration | PASS | Cross-surface handoff coverage routes to the integration lane only. |
+
+## Explicit Broad-Lane Guards
+
+Workspace V2 explicit/on-request only: PASS
+Full samples smoke explicit/on-request only: PASS
+Misplaced test preflight fast-fail: PASS
+Scheduled runtime lanes: tool-runtime, game-runtime, integration, engine-src
+Full samples smoke decision: SKIP - Skipped because changed files do not modify sample JSON or shared sample loader/framework behavior.
+
+## Lane Script Routing
+
+| Script | Status | Command |
+| --- | --- | --- |
+| test:lane:workspace-contract | PASS | node ./scripts/run-targeted-test-lanes.mjs --lane workspace-contract |
+| test:lane:tool-runtime | PASS | node ./scripts/run-targeted-test-lanes.mjs --lane tool-runtime |
+| test:lane:game-runtime | PASS | node ./scripts/run-targeted-test-lanes.mjs --lane game-runtime |
+| test:lane:integration | PASS | node ./scripts/run-targeted-test-lanes.mjs --lane integration |
+| test:lane:engine-src | PASS | node ./scripts/run-targeted-test-lanes.mjs --lane engine-src |
+| test:lane:samples | PASS | node ./scripts/run-targeted-test-lanes.mjs --lane samples --include-samples |
+
+## Legacy Direct Playwright Scripts
+
+| Script | Status | Command |
+| --- | --- | --- |
+| test:asset-manager-v2 | INFO | playwright test tests/playwright/tools/AssetManagerV2.spec.mjs --project=playwright --workers=1 --reporter=list |
+| test:preview-generator-v2 | INFO | playwright test tests/playwright/tools/PreviewGeneratorV2Baseline.spec.mjs --project=playwright --workers=1 --reporter=list |
+| test:workspace-v2 | INFO | playwright test tests/playwright/tools/WorkspaceManagerV2.spec.mjs --project=playwright --workers=1 --reporter=list |
+
+## Misplaced Test Probe
+
+- Lane tool-runtime has no scoped manifest target.
+- Scoped discovery target is outside selected lane ownership: tests/playwright/games/AsteroidsShipStateVisuals.spec.mjs.
+
+## Routing Findings
+
+No routing findings. Targeted lanes execute only expected representative lanes, and broad Workspace/samples paths remain explicit.
+
+## Enforcement Notes
+
+- Representative docs, tool, game, src, and integration cases route through deterministic lane rules.
+- Old direct Playwright scripts may remain available, but test:lane:* scripts route through the targeted Node lane runner.
+- Misplaced ownership probes fail in zero-browser scoped discovery before Playwright/browser launch.
+- Full Workspace and full samples smoke are not used as accidental fallback lanes.
diff --git a/docs/dev/reports/testing_lane_execution_report.md b/docs/dev/reports/testing_lane_execution_report.md
diff --git a/docs/dev/reports/validation_cache_report.md b/docs/dev/reports/validation_cache_report.md
diff --git a/docs/dev/reports/zero_browser_preflight_report.md b/docs/dev/reports/zero_browser_preflight_report.md
diff --git a/scripts/run-targeted-test-lanes.mjs b/scripts/run-targeted-test-lanes.mjs
