Rename asset-browser-v2 to asset-manager-v2 across repo and enforce single tool contract - PR_11_313

Author: DavidQ

docs/dev/codex_commands.md

@@ -131,3 +131,10 @@ PR_11_313 (Workspace V2 launcher + Asset Manager V2 UI labels)
 ```bash
 npx @openai/codex run --model gpt-5.3-codex --reasoning medium "Implement PR_11_313: Workspace V2 tool launcher UI + Asset Manager V2 user-facing rename."
 ```
+
+---
+PR_11_313 (single contract rename: asset-manager-v2)
+
+```bash
+npx @openai/codex run --model gpt-5.3-codex --reasoning medium "Implement PR_11_313: rename legacy asset browser v2 id to asset-manager-v2 across tool ID, paths, payloads, registry links, tests, and docs with zero old references."
+```

docs/dev/commit_comment.txt

@@ -1 +1 @@
-Add explicit Workspace V2 Asset Manager launcher and rename Asset Browser V2 user-facing labels without contract changes - PR 11.313
+Rename legacy asset browser v2 id to asset-manager-v2 as a single repo-wide contract with zero legacy references - PR 11.313

docs/dev/reports/PR_11_193C_validation.md

@@ -6,8 +6,8 @@ Implement the V2 HTML-first batch for Palette Manager V2, SVG Asset Studio V2, V
 ## Files Changed
 - `tools/tilemap-studio-v2/index.html`
 - `tools/tilemap-studio-v2/index.js`
-- `tools/asset-browser-v2/index.html`
-- `tools/asset-browser-v2/index.js`
+- `tools/asset-manager-v2/index.html`
+- `tools/asset-manager-v2/index.js`
 - `docs/dev/reports/PR_11_193C_validation.md`
 
 Existing active V2 tools verified without additional implementation edits in this pass:
@@ -23,7 +23,7 @@ Existing PR source doc was present in the worktree and included in the ZIP:
 
 ## Implementation Summary
 - Added `Tilemap Studio V2` as `tools/tilemap-studio-v2/`.
-- Added `Asset Browser V2` as `tools/asset-browser-v2/`.
+- Added `Asset Browser V2` as `tools/asset-manager-v2/`.
 - Both new tools are HTML-first: `index.html` owns static shell, CSS links, shared header mount, layout, menus, state containers, and module scripts.
 - Both new tools keep `index.js` behavior-only: document title/tool id setup, session read, contract validation, event binding, dynamic rendering into existing DOM nodes, and empty/error states.
 - No legacy implementation files were copied into V2. Legacy files were used only to infer high-level contract names and user-facing data shape.
@@ -34,7 +34,7 @@ Passed for all five scoped V2 tools:
 - `tools/svg-asset-studio-v2/index.html`
 - `tools/vector-map-editor-v2/index.html`
 - `tools/tilemap-studio-v2/index.html`
-- `tools/asset-browser-v2/index.html`
+- `tools/asset-manager-v2/index.html`
 
 Each contains:
 - `<div id="shared-theme-header"></div>`
@@ -84,15 +84,15 @@ node --check tools/palette-manager-v2/index.js
 node --check tools/svg-asset-studio-v2/index.js
 node --check tools/vector-map-editor-v2/index.js
 node --check tools/tilemap-studio-v2/index.js
-node --check tools/asset-browser-v2/index.js
+node --check tools/asset-manager-v2/index.js
 ```
 
 Results:
 - `node --check tools/palette-manager-v2/index.js` passed.
 - `node --check tools/svg-asset-studio-v2/index.js` passed.
 - `node --check tools/vector-map-editor-v2/index.js` passed.
 - `node --check tools/tilemap-studio-v2/index.js` passed.
-- `node --check tools/asset-browser-v2/index.js` passed.
+- `node --check tools/asset-manager-v2/index.js` passed.
 
 Targeted direct-entry/HTML validation:
 - Verified all five scoped `index.html` files have the required static shell/header/menu/script markers.

docs/dev/reports/PR_11_193_V2_HTML_FIRST_BATCH_validation.md

@@ -10,7 +10,7 @@ Batch-correct and verify the active V2 tool lane so `index.html` owns static she
 
 ## Listed V2 Folders Not Present
 - `tools/tilemap-studio-v2/` - NOT PRESENT - SKIPPED
-- `tools/asset-browser-v2/` - NOT PRESENT - SKIPPED
+- `tools/asset-manager-v2/` - NOT PRESENT - SKIPPED
 
 No unrelated scaffolding was created for absent V2 tools.
 
@@ -70,7 +70,7 @@ Results:
 
 Skipped commands:
 - `node --check tools/tilemap-studio-v2/index.js` - NOT PRESENT - SKIPPED
-- `node --check tools/asset-browser-v2/index.js` - NOT PRESENT - SKIPPED
+- `node --check tools/asset-manager-v2/index.js` - NOT PRESENT - SKIPPED
 
 ## Additional Guard Checks
 - HTML/JS compliance scan passed for the three active V2 folders.

docs/dev/reports/PR_11_194_v2_html_first_remaining_tools_report.md

@@ -8,7 +8,7 @@ Verify and complete the remaining Tool V2 HTML-first shell batch. Each scoped V2
 - `tools/svg-asset-studio-v2/`
 - `tools/vector-map-editor-v2/`
 - `tools/tilemap-studio-v2/`
-- `tools/asset-browser-v2/`
+- `tools/asset-manager-v2/`
 
 ## Files Changed
 - `docs/dev/reports/PR_11_194_v2_html_first_remaining_tools_report.md`
@@ -74,15 +74,15 @@ node --check tools/palette-manager-v2/index.js
 node --check tools/svg-asset-studio-v2/index.js
 node --check tools/vector-map-editor-v2/index.js
 node --check tools/tilemap-studio-v2/index.js
-node --check tools/asset-browser-v2/index.js
+node --check tools/asset-manager-v2/index.js
 ```
 
 Results:
 - `node --check tools/palette-manager-v2/index.js` passed.
 - `node --check tools/svg-asset-studio-v2/index.js` passed.
 - `node --check tools/vector-map-editor-v2/index.js` passed.
 - `node --check tools/tilemap-studio-v2/index.js` passed.
-- `node --check tools/asset-browser-v2/index.js` passed.
+- `node --check tools/asset-manager-v2/index.js` passed.
 
 Additional targeted validation:
 - `PR_11_194 V2 HTML-first compliance validation passed`
@@ -93,7 +93,7 @@ Manual browser validation was not launched from this terminal session. The per-t
 - Open `tools/svg-asset-studio-v2/index.html`; confirm shared header renders, static shell is visible before session data, missing session shows explicit empty state, valid session renders without legacy coupling.
 - Open `tools/vector-map-editor-v2/index.html`; confirm shared header renders, static shell is visible before session data, missing session shows explicit empty state, valid session renders without legacy coupling.
 - Open `tools/tilemap-studio-v2/index.html`; confirm shared header renders, static shell is visible before session data, missing session shows explicit empty state, valid session renders without legacy coupling.
-- Open `tools/asset-browser-v2/index.html`; confirm shared header renders, static shell is visible before session data, missing session shows explicit empty state, valid session renders without legacy coupling.
+- Open `tools/asset-manager-v2/index.html`; confirm shared header renders, static shell is visible before session data, missing session shows explicit empty state, valid session renders without legacy coupling.
 
 ## Banned Path Check Result
 Scoped status check confirmed no changes under:

docs/dev/reports/PR_11_195_validation.md

@@ -39,13 +39,13 @@ Additional unchanged V2 lane sanity checks were also run:
 ```powershell
 node --check tools/vector-map-editor-v2/index.js
 node --check tools/tilemap-studio-v2/index.js
-node --check tools/asset-browser-v2/index.js
+node --check tools/asset-manager-v2/index.js
 ```
 
 Results:
 - `node --check tools/vector-map-editor-v2/index.js` passed.
 - `node --check tools/tilemap-studio-v2/index.js` passed.
-- `node --check tools/asset-browser-v2/index.js` passed.
+- `node --check tools/asset-manager-v2/index.js` passed.
 
 ## Banned JS Pattern Check
 Command covered changed JS files:

docs/dev/reports/PR_11_197B_v2_asset_browser_validation.md

@@ -4,16 +4,16 @@
 Complete Asset Browser V2 as the focused, testable V2 implementation target while preserving the HTML-first/static-shell and session-only runtime architecture.
 
 ## Files Changed
-- `tools/asset-browser-v2/index.js`
+- `tools/asset-manager-v2/index.js`
 - `docs/dev/reports/PR_11_197B_v2_asset_browser_validation.md`
 - `docs/dev/reports/PR_11_197B_expected_evidence.md`
 
 Unchanged but packaged for review context:
-- `tools/asset-browser-v2/index.html`
+- `tools/asset-manager-v2/index.html`
 
 ## Implementation Summary
-- Kept `tools/asset-browser-v2/index.html` as the static shell owner: CSS links, shared header mount, page layout, menu regions, state containers, and module script tags remain in HTML.
-- Kept `tools/asset-browser-v2/index.js` behavior-only: title/tool id setup, session read, validation, DOM population, event binding, dynamic render, empty state, and invalid state.
+- Kept `tools/asset-manager-v2/index.html` as the static shell owner: CSS links, shared header mount, page layout, menu regions, state containers, and module script tags remain in HTML.
+- Kept `tools/asset-manager-v2/index.js` behavior-only: title/tool id setup, session read, validation, DOM population, event binding, dynamic render, empty state, and invalid state.
 - Tightened valid session contract to require explicit `assetCatalog.entries[]` fields: `id`, `label`, `kind`, and `path`.
 - Removed auto-previewing the first asset as an implicit selection. Valid sessions now render list/count and instruct the user to select an entry to inspect session-backed metadata.
 - No legacy Asset Browser implementation code was copied.
@@ -22,15 +22,15 @@ Unchanged but packaged for review context:
 ## Validation Commands Run
 
 ```powershell
-node --check tools/asset-browser-v2/index.js
+node --check tools/asset-manager-v2/index.js
 ```
 
 Result: passed.
 
 Static banned-pattern check:
 
 ```powershell
-rg -n "document\.body\.innerHTML|document\.head\.insertAdjacentHTML|createElement\(|appendChild\(|platformShell|assetUsageIntegration|tools/shared/|\.\.\/shared|Workspace Manager|fallback|default data|sample data|demo data" -- tools/asset-browser-v2/index.js
+rg -n "document\.body\.innerHTML|document\.head\.insertAdjacentHTML|createElement\(|appendChild\(|platformShell|assetUsageIntegration|tools/shared/|\.\.\/shared|Workspace Manager|fallback|default data|sample data|demo data" -- tools/asset-manager-v2/index.js
 ```
 
 Result: passed. No matches.
@@ -39,13 +39,13 @@ HTML shell marker check verified:
 - `id="shared-theme-header"`
 - `src="../../src/engine/theme/mount-shared-header.js"`
 - `src="./index.js"`
-- `data-tool-id="asset-browser-v2"`
+- `data-tool-id="asset-manager-v2"`
 - `<title>Asset Browser V2</title>`
 
 Result: passed.
 
 ## Lightweight Browser Validation Evidence
-A Node VM harness executed `tools/asset-browser-v2/index.js` with minimal browser/session mocks and verified direct empty, invalid session, and valid session states.
+A Node VM harness executed `tools/asset-manager-v2/index.js` with minimal browser/session mocks and verified direct empty, invalid session, and valid session states.
 
 Log snippets:
 

docs/dev/reports/PR_11_199_report.md

@@ -1,15 +1,15 @@
 # PR_11_199 Report
 
 ## V2 Directories Inspected
-- `tools/asset-browser-v2`
+- `tools/asset-manager-v2`
 - `tools/palette-manager-v2`
 - `tools/svg-asset-studio-v2`
 - `tools/tilemap-studio-v2`
 - `tools/vector-map-editor-v2`
 
 ## Files Changed
-- `tools/asset-browser-v2/index.html`
-- `tools/asset-browser-v2/index.js`
+- `tools/asset-manager-v2/index.html`
+- `tools/asset-manager-v2/index.js`
 - `tools/palette-manager-v2/index.html`
 - `tools/palette-manager-v2/index.js`
 - `tools/svg-asset-studio-v2/index.html`
@@ -19,19 +19,19 @@
 - `docs/dev/reports/PR_11_199_report.md`
 
 ## Validation Commands Run
-- `node --check tools/asset-browser-v2/index.js`
+- `node --check tools/asset-manager-v2/index.js`
 - `node --check tools/palette-manager-v2/index.js`
 - `node --check tools/svg-asset-studio-v2/index.js`
 - `node --check tools/vector-map-editor-v2/index.js`
 
 ## Validation Results
-- `node --check tools/asset-browser-v2/index.js`: **PASS** (exit code 0)
+- `node --check tools/asset-manager-v2/index.js`: **PASS** (exit code 0)
 - `node --check tools/palette-manager-v2/index.js`: **PASS** (exit code 0)
 - `node --check tools/svg-asset-studio-v2/index.js`: **PASS** (exit code 0)
 - `node --check tools/vector-map-editor-v2/index.js`: **PASS** (exit code 0)
 
 ## Manual Validation Results By Tool
-- `asset-browser-v2`: browser-interactive manual run not executed in this CLI-only session (no interactive browser in terminal tooling). Static checks by source inspection: header mount present, no-session empty state region present, malformed-session invalid state region present, valid/render region present, no banned JS `innerHTML` page construction.
+- `asset-manager-v2`: browser-interactive manual run not executed in this CLI-only session (no interactive browser in terminal tooling). Static checks by source inspection: header mount present, no-session empty state region present, malformed-session invalid state region present, valid/render region present, no banned JS `innerHTML` page construction.
 - `palette-manager-v2`: browser-interactive manual run not executed in this CLI-only session (no interactive browser in terminal tooling). Static checks by source inspection: header mount present, no-session empty state region present, malformed-session invalid state region present, valid/render region present, no banned JS `innerHTML` page construction.
 - `svg-asset-studio-v2`: browser-interactive manual run not executed in this CLI-only session (no interactive browser in terminal tooling). Static checks by source inspection: header mount present, no-session empty state region present, malformed-session invalid state region present, valid/render region present, no banned JS `innerHTML` page construction, URL payload writeback path removed.
 - `vector-map-editor-v2`: browser-interactive manual run not executed in this CLI-only session (no interactive browser in terminal tooling). Static checks by source inspection: header mount present, no-session empty state region present, malformed-session invalid state region present, valid/render region present, no banned JS `innerHTML` page construction, vector preview rendered via SVG DOM nodes.

docs/dev/reports/PR_11_200_report.md

@@ -9,7 +9,7 @@
 - `SVG Asset Studio V2` -> `/tools/svg-asset-studio-v2/index.html`
 - `Vector Map Editor V2` -> `/tools/vector-map-editor-v2/index.html`
 - `Tilemap Studio V2` -> `/tools/tilemap-studio-v2/index.html`
-- `Asset Browser V2` -> `/tools/asset-browser-v2/index.html`
+- `Asset Browser V2` -> `/tools/asset-manager-v2/index.html`
 
 All entries explicitly include `V2`, route directly to V2 tool `index.html`, and do not alias to legacy tool paths.
 
@@ -23,7 +23,7 @@ All entries explicitly include `V2`, route directly to V2 tool `index.html`, and
   - `/tools/svg-asset-studio-v2/index.html`
   - `/tools/vector-map-editor-v2/index.html`
   - `/tools/tilemap-studio-v2/index.html`
-  - `/tools/asset-browser-v2/index.html`
+  - `/tools/asset-manager-v2/index.html`
 - No V1 label reuse for these entries (all show explicit `V2` suffix text in entry titles).
 
 ## Full Smoke Decision

docs/dev/reports/PR_11_201_report.md

@@ -1,15 +1,15 @@
 # PR_11_201 Report
 
 ## Tools Tested
-- `tools/asset-browser-v2/index.js`
+- `tools/asset-manager-v2/index.js`
 - `tools/palette-manager-v2/index.js`
 - `tools/svg-asset-studio-v2/index.js`
 - `tools/tilemap-studio-v2/index.js`
 - `tools/vector-map-editor-v2/index.js`
 
 ## Files Changed
 - `tools/index.html`
-- `tools/asset-browser-v2/index.js`
+- `tools/asset-manager-v2/index.js`
 - `tools/palette-manager-v2/index.js`
 - `tools/svg-asset-studio-v2/index.js`
 - `tools/tilemap-studio-v2/index.js`
@@ -24,14 +24,14 @@
   - valid `hostContextId` with valid session payload -> normal render path
 
 ## Validation Commands
-- `node --check tools/asset-browser-v2/index.js`
+- `node --check tools/asset-manager-v2/index.js`
 - `node --check tools/palette-manager-v2/index.js`
 - `node --check tools/svg-asset-studio-v2/index.js`
 - `node --check tools/tilemap-studio-v2/index.js`
 - `node --check tools/vector-map-editor-v2/index.js`
 
 ## Validation Results
-- `node --check tools/asset-browser-v2/index.js`: **PASS**
+- `node --check tools/asset-manager-v2/index.js`: **PASS**
 - `node --check tools/palette-manager-v2/index.js`: **PASS**
 - `node --check tools/svg-asset-studio-v2/index.js`: **PASS**
 - `node --check tools/tilemap-studio-v2/index.js`: **PASS**
@@ -43,7 +43,7 @@
 - Scenario 1 (no parameters): **PASS** (code-path inspection: `renderMissing(...)` when `hostContextId` is absent)
 - Scenario 2 (invalid `hostContextId`): **PASS** (code-path inspection: `renderError(...)` with actionable message when session key is missing)
 - Scenario 3 (valid `hostContextId`): **PASS** (code-path inspection: resolves `toolboxaid.toolHost.context.<id>`, parses JSON, validates `payloadJson.assetCatalog`, renders)
-- Scenario 4 (open from tools index): **PASS** (link points to `./asset-browser-v2/index.html`)
+- Scenario 4 (open from tools index): **PASS** (link points to `./asset-manager-v2/index.html`)
 
 ### Palette Manager V2
 - Scenario 1 (no parameters): **PASS** (code-path inspection: `renderMissing(...)`)