Route Playwright artifacts to tests/results and ignore local dependency outputs - PR_11_318C

Author: DavidQ

docs/dev/reports/PR_11_318C_report.md

@@ -13,17 +13,21 @@ Move Playwright outputs from defaults into `tests/results/**` via config-only ch
 
 ## Implementation Summary
 - Config now routes Playwright outputs under `tests/results/**`:
-  - test artifacts: `tests/results/artifacts`
+  - top-level `outputDir`: `tests/results`
+  - project artifact output: `tests/results/artifacts`
   - HTML report: `tests/results/report`
 - Kept existing run options:
   - `headless: false`
   - `slowMo: 500`
   - `trace: "on"`
-- Scoped Playwright discovery to `tests/ui` to run UI specs only when invoking `npx playwright test`.
+- `.gitignore` already included required ignore entries; no `.gitignore` changes were needed:
+  - `node_modules/`
+  - `tests/results/`
+  - `tmp/`
 
 ## Validation Commands
 - `node --check playwright.config.cjs` -> **PASS**
-- `npx playwright test` -> **PASS**
+- `npx playwright test tests/ui/workspace-v2.asset-manager.spec.js` -> **PASS**
 
 ## Output Verification
 - `tests/results/` exists -> **YES**
@@ -33,5 +37,4 @@ Move Playwright outputs from defaults into `tests/results/**` via config-only ch
 - `playwright-report/` created -> **NO**
 
 ## Constraint Note
-- A direct combination of `outputDir = tests/results` and HTML reporter `outputFolder = tests/results/report` is rejected by Playwright due folder-clash protection.
-- Final config keeps all output in `tests/results/**` without using default output folders.
+- Playwright rejects direct HTML/outputDir folder collision. Final config keeps all output in `tests/results/**` by using top-level `outputDir` plus project-level artifact `outputDir`.

docs/pr/PR_11_318C_PLAYWRIGHT_OUTPUT_DIRS/BUILD_PR.md

@@ -2,8 +2,10 @@
 
 ## Implementation
 - Updated `playwright.config.cjs`:
-  - `testDir: "tests/ui"`
-  - `outputDir: "tests/results/artifacts"`
+  - top-level `outputDir: "tests/results"`
+  - project-level UI artifact output:
+    - `projects[0].outputDir: "tests/results/artifacts"`
+    - `projects[0].testDir: "tests/ui"`
   - HTML reporter output:
     - `tests/results/report`
   - Preserved:
@@ -12,9 +14,12 @@
     - `trace: "on"`
 
 ## Notes
-- Playwright rejects a config where HTML output folder is nested under the same root as `outputDir` when both are the same directory tree root (`tests/results`), because HTML reporter clears its output folder and can erase artifacts.
-- This config keeps all outputs under `tests/results/**` while avoiding reporter/outputDir collision.
+- `.gitignore` already includes required entries:
+  - `node_modules/`
+  - `tests/results/`
+  - `tmp/`
+- Playwright rejects a config where HTML output folder and test artifact `outputDir` share a collision path. Project-level `outputDir` avoids that clash while keeping all outputs under `tests/results/**`.
 
 ## Validation
 - `node --check playwright.config.cjs`
-- `npx playwright test`
+- `npx playwright test tests/ui/workspace-v2.asset-manager.spec.js`

playwright.config.cjs

@@ -1,6 +1,12 @@
 module.exports = {
-  testDir: "tests/ui",
-  outputDir: "tests/results/artifacts",
+  outputDir: "tests/results",
+  projects: [
+    {
+      name: "ui",
+      testDir: "tests/ui",
+      outputDir: "tests/results/artifacts"
+    }
+  ],
   reporter: [
     ["list"],
     ["html", { outputFolder: "tests/results/report", open: "never" }]