Move tool Playwright specs under tools tests and enforce session-based production launch - PR_26126_110-playwright-tools-directory-and-workspace-prod-guard

Author: DavidQ

docs/dev/reports/PR_26126_110_manual_validation_notes.md

@@ -0,0 +1,21 @@
+# PR_26126_110 Manual Validation Notes
+
+## Commands
+- `node -e "JSON.parse(require('fs').readFileSync('package.json','utf8')); console.log('package.json ok')"`
+- `node --check tests\playwright\tools\AssetManagerV2.spec.mjs`
+- `node --check tests\playwright\tools\PreviewGeneratorV2Baseline.spec.mjs`
+- `npm run test:asset-manager-v2`
+- `npm run test:preview-generator-v2`
+- `npm run test:workspace-v2`
+- `git diff --check`
+
+## Results
+- `package.json` parses successfully after script path updates.
+- Asset Manager V2 and Preview Generator V2 moved specs pass JavaScript syntax checks.
+- `npm run test:asset-manager-v2` passed 7/7 Playwright tests.
+- `npm run test:preview-generator-v2` passed 7/7 Playwright tests.
+- `npm run test:workspace-v2` passed 14/14 Playwright tests.
+- `?workspace=prod` hard-fails to the Asset Manager V2 launch guard overlay.
+- Valid Workspace Manager session launch remains the production access path.
+- Full samples smoke test skipped: this PR changes tool Playwright spec locations, npm script paths, and launch-guard test coverage only.
+- No sample JSON was modified.

docs/dev/reports/PR_26126_110_test_directory_ssot_notes.md

@@ -0,0 +1,16 @@
+# PR_26126_110 Test Directory SSoT Notes
+
+## Tool Spec Location
+- Moved Asset Manager V2 Playwright coverage to `tests/playwright/tools/AssetManagerV2.spec.mjs`.
+- Moved Preview Generator V2 Playwright coverage to `tests/playwright/tools/PreviewGeneratorV2Baseline.spec.mjs`.
+- Updated both moved specs to import helpers from `../../helpers/...`.
+
+## Script Ownership
+- `test:asset-manager-v2` now targets `tests/playwright/tools/AssetManagerV2.spec.mjs`.
+- `test:preview-generator-v2` now targets `tests/playwright/tools/PreviewGeneratorV2Baseline.spec.mjs`.
+- `test:workspace-v2` remains the aggregate Workspace V2/tool completion gate and includes both tools-directory specs in one Playwright run.
+
+## Scope
+- No Asset Manager V2 runtime behavior changed for the test move.
+- No Preview Generator V2 runtime behavior changed.
+- No sample JSON was modified.

docs/dev/reports/PR_26126_110_workspace_launch_guard_notes.md

@@ -0,0 +1,15 @@
+# PR_26126_110 Workspace Launch Guard Notes
+
+## Production Access
+- Asset Manager V2 production access remains session/context based through Workspace Manager.
+- The valid Workspace Manager session path is still covered by the Asset Manager V2 Playwright suite.
+- Direct `tools/asset-manager-v2/index.html?workspace=prod` launch is treated as invalid.
+
+## Temporary Access
+- `?workspace=UAT` remains the only temporary URL-based Asset Manager V2 access path.
+- The UAT path remains covered by the Asset Manager V2 Playwright suite.
+
+## Validation
+- Playwright validates `?workspace=prod` hard-fails to the launch guard overlay with `Temporary workspace prod is not supported.`
+- Playwright validates valid Workspace Manager launch still opens Asset Manager V2 through session-backed context.
+- No sample JSON was modified.

docs/dev/reports/codex_changed_files.txt

@@ -2,15 +2,24 @@
 M docs/dev/reports/coverage_changed_js_guardrail.txt
  M docs/dev/reports/playwright_v8_coverage_report.txt
  M package.json
-?? docs/dev/reports/PR_26126_109_manual_validation_notes.md
-?? docs/dev/reports/PR_26126_109_test_script_ssot_notes.md
+ D tests/playwright/AssetManagerV2.spec.mjs
+ D tests/playwright/PreviewGeneratorV2Baseline.spec.mjs
+?? docs/dev/reports/PR_26126_110_manual_validation_notes.md
+?? docs/dev/reports/PR_26126_110_test_directory_ssot_notes.md
+?? docs/dev/reports/PR_26126_110_workspace_launch_guard_notes.md
+?? tests/playwright/tools/
 
 # git ls-files --others --exclude-standard
-docs/dev/reports/PR_26126_109_manual_validation_notes.md
-docs/dev/reports/PR_26126_109_test_script_ssot_notes.md
+docs/dev/reports/PR_26126_110_manual_validation_notes.md
+docs/dev/reports/PR_26126_110_test_directory_ssot_notes.md
+docs/dev/reports/PR_26126_110_workspace_launch_guard_notes.md
+tests/playwright/tools/AssetManagerV2.spec.mjs
+tests/playwright/tools/PreviewGeneratorV2Baseline.spec.mjs
 
 # git diff --stat
-docs/dev/reports/coverage_changed_js_guardrail.txt | 1 -
- docs/dev/reports/playwright_v8_coverage_report.txt | 3 ---
- package.json                                       | 4 +++-
- 3 files changed, 3 insertions(+), 5 deletions(-)
+docs/dev/reports/coverage_changed_js_guardrail.txt |    4 +-
+ docs/dev/reports/playwright_v8_coverage_report.txt |    7 +-
+ package.json                                       |    6 +-
+ tests/playwright/AssetManagerV2.spec.mjs           | 1435 --------------------
+ .../playwright/PreviewGeneratorV2Baseline.spec.mjs |  747 ----------
+ 5 files changed, 9 insertions(+), 2190 deletions(-)