Rollback unstable Workspace V2 communication/runtime changes while preserving Session Inspector and cosmetic UI improvements - PR_26128_001-workspace-recovery-baseline

Author: DavidQ

docs/dev/reports/playwright_workspace_recovery.md

@@ -0,0 +1,17 @@
+# PR_26128_001 Playwright Workspace Recovery
+
+## Validation Results
+- `npm run test:workspace-v2`: Pass, 10 passed.
+- Targeted Workspace Manager V2 launch validation: Pass.
+- Targeted Preview Generator V2 workspace launch validation: Pass.
+- Repo selection without `showDirectoryPicker` dependency: Pass. The targeted probe disabled `window.showDirectoryPicker`, selected Asteroids in Workspace Manager V2, launched Preview Generator V2 from workspace context, hydrated `repoPath` from the session context, and wrote `games/Asteroids/assets/images/preview.svg` through the restored baseline write path.
+- Accordion controls: Pass. The targeted probe collapsed and reopened the Active Game accordion and verified `hidden` plus `aria-expanded` state changes.
+- Session Inspector launch: Not run. `tools/session-inspector/index.html` is not present in this checkout, so there was no Session Inspector page to launch after rollback.
+
+## Removed Coupling Checks
+- `selectRepoButton` was absent from Workspace Manager V2 after rollback.
+- Node-backed repo endpoints and persisted repo handle stores were absent from the recovered Workspace Manager V2 path.
+- Preview Generator V2 workspace launch kept the repo picker hidden and hydrated from workspace session context.
+
+## Full Samples Smoke Test
+- Full samples smoke test was skipped by BUILD instruction. This rollback is scoped to Workspace Manager V2, Preview Generator V2, and shared shell rollback validation, and the requested validation explicitly says to skip the full samples smoke test.

docs/dev/reports/rollback_preserved_items.md

@@ -0,0 +1,12 @@
+# PR_26128_001 Rollback Preserved Items
+
+## Preserved Baseline
+- Preserved the committed Workspace Manager V2 accordion behavior in `tools/workspace-manager-v2/js/controls/AccordionSection.js`.
+- Preserved committed Workspace Manager V2 layout and non-runtime UI consistency updates in `tools/workspace-manager-v2/index.html` and `tools/workspace-manager-v2/styles/workspaceManagerV2.css`.
+- Preserved the committed Preview Generator V2 baseline where Workspace Manager launches hydrate from session context and tools-mode repo picking remains local to Preview Generator V2.
+- Preserved existing schema files exactly; no `tools/schemas/**` files were changed by this rollback.
+- Preserved sample JSON exactly; no `games/**`, `samples/**`, or sample manifest JSON files were changed by this rollback.
+- Preserved roadmap content exactly; no roadmap files were changed by this rollback.
+
+## Session Inspector Note
+- `tools/session-inspector/**` is not present in this checkout after rollback, and no files under that path were available to preserve or modify.

docs/dev/reports/rollback_removed_items.md

@@ -0,0 +1,14 @@
+# PR_26128_001 Rollback Removed Items
+
+## Removed Runtime Rewrites
+- Removed the uncommitted Workspace Manager V2 repo scan/select rewrite that added `selectRepoButton`, `activeRepoSummary`, `selectRepoRoot`, repo handle persistence, and `/__repo/games` coupling.
+- Removed the uncommitted Preview Generator V2 node write mode rewrite that added `/__preview-generator-v2/write-preview`, `/__repo/status`, persisted repo handle restore, and node endpoint write selection.
+- Removed the uncommitted shared `tools/common/RepoHandleStore.js` coupling and `tools/preview-generator-v2/PreviewGeneratorV2RepoHandleStore.js` wrapper.
+- Removed the uncommitted `scripts/workspace-v2-dev-server.mjs` dev server and the related `package.json` `dev-tools` / `dev:workspace-v2` script additions.
+- Removed the uncommitted storage debug page replacement under `tests/index.html`, `tests/storage-debug.css`, and `tests/storage-debug.js`.
+- Removed the uncommitted Workspace V2 test rewrites that depended on node-backed repo selection, storage debug persistence, and persisted browser directory handles.
+
+## Restored Baseline Behavior
+- Workspace Manager V2 now uses the committed static game list and sessionStorage-based workspace context flow.
+- Preview Generator V2 workspace launch uses the committed session-context repoPath hydration path and no longer depends on the removed node dev-tools server.
+- Preview Generator V2 tools-mode repo picking remains scoped to Preview Generator V2 and is not shared through IndexedDB or Workspace Manager V2 auto-wiring.