Move Controls to wireframe and add controller profile planning - PR_26161_018-controls-wireframe-controller-profiles

Author: ToolboxAid

docs_build/dev/reports/codex_changed_files.txt

@@ -1,20 +1,24 @@
 # git status --short
-M docs_build/dev/reports/coverage_changed_js_guardrail.txt
+M docs_build/dev/reports/codex_changed_files.txt
+ M docs_build/dev/reports/codex_review.diff
+ M docs_build/dev/reports/coverage_changed_js_guardrail.txt
  M docs_build/dev/reports/playwright_v8_coverage_report.txt
+ M src/dev-runtime/guest-seeds/tool-metadata-inventory.js
  M tests/playwright/tools/InputMappingV2Tool.spec.mjs
  M toolbox/controls/index.html
  M toolbox/input-mapping-v2/index.html
- M toolbox/input-mapping-v2/input-mapping-v2.js
-?? docs_build/dev/reports/controls-table-first-input-report.md
+?? docs_build/dev/reports/controls-wireframe-controller-profiles-report.md
 
 # git ls-files --others --exclude-standard
-docs_build/dev/reports/controls-table-first-input-report.md
+docs_build/dev/reports/controls-wireframe-controller-profiles-report.md
 
 # git diff --stat
-.../dev/reports/coverage_changed_js_guardrail.txt  |  4 +-
- .../dev/reports/playwright_v8_coverage_report.txt  | 12 ++--
- tests/playwright/tools/InputMappingV2Tool.spec.mjs | 68 ++++++++++++++++++++++
- toolbox/controls/index.html                        | 14 +++--
- toolbox/input-mapping-v2/index.html                | 14 +++--
- toolbox/input-mapping-v2/input-mapping-v2.js       |  5 +-
- 6 files changed, 97 insertions(+), 20 deletions(-)
+docs_build/dev/reports/codex_changed_files.txt     |   26 +-
+ docs_build/dev/reports/codex_review.diff           | 1100 +++++++++++++++-----
+ .../dev/reports/coverage_changed_js_guardrail.txt  |    7 -
+ .../dev/reports/playwright_v8_coverage_report.txt  |   11 -
+ .../guest-seeds/tool-metadata-inventory.js         |    4 +-
+ tests/playwright/tools/InputMappingV2Tool.spec.mjs |   21 +-
+ toolbox/controls/index.html                        |   32 +
+ toolbox/input-mapping-v2/index.html                |   32 +
+ 8 files changed, 927 insertions(+), 306 deletions(-)

docs_build/dev/reports/codex_review.diff

@@ -0,0 +1,74 @@
+# PR_26161_018 Controls Wireframe Controller Profiles Report
+
+## Branch Validation
+
+- PASS: Current branch was `main`.
+- Expected branch: `main`.
+- Worktree was clean before PR_26161_018 edits.
+
+## Scope
+
+- Impacted lane: Controls/Input Mapping tool recovery/UAT.
+- Playwright impacted: Yes.
+- Full samples validation: SKIPPED. Samples are out of scope for this Controls wireframe/profile planning change, and no sample JSON alignment was requested.
+- Engine runtime behavior: Unchanged. No production controller runtime, sample runtime, auth behavior, or unrelated rewrite was added.
+
+## Requirement Checklist
+
+- PASS: Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
+- PASS: Reviewed `docs_build/tools/input-mapping-v2/uat.md`.
+- PASS: Verified branch `main` before edits.
+- PASS: Continued from PR_26161_017 table-first Controls behavior.
+- PASS: Controls metadata moved away from beta to Wireframe with `releaseChannel: "wireframe"`.
+- PASS: Controller profile planning was added inside Controls, not as a separate tool.
+- PASS: Wireframe UI includes Device Type, Controller Name, Controller ID, Mapping Profile, Input, and Action.
+- PASS: Unknown controller status gives an actionable Refresh Devices step and future Create Profile path.
+- PASS: Existing mapping table behavior and DB persistence were preserved.
+- PASS: No production controller runtime or engine behavior was added.
+
+## Files Changed
+
+- `src/dev-runtime/guest-seeds/tool-metadata-inventory.js`
+- `toolbox/input-mapping-v2/index.html`
+- `toolbox/controls/index.html`
+- `tests/playwright/tools/InputMappingV2Tool.spec.mjs`
+- `docs_build/dev/reports/controls-wireframe-controller-profiles-report.md`
+- `docs_build/dev/reports/codex_review.diff`
+- `docs_build/dev/reports/codex_changed_files.txt`
+- `docs_build/dev/reports/playwright_v8_coverage_report.txt`
+- `docs_build/dev/reports/coverage_changed_js_guardrail.txt`
+
+## Testing Performed
+
+- PASS: `node --check src/dev-runtime/guest-seeds/tool-metadata-inventory.js`
+- PASS: `node --check tests/playwright/tools/InputMappingV2Tool.spec.mjs`
+- PASS: `node --check toolbox/input-mapping-v2/input-mapping-v2.js`
+- PASS: HTML restriction scan for inline scripts, style blocks, inline styles, and inline event handlers in both Controls routes.
+- PASS: `git diff --check`
+  - Note: Git reported CRLF normalization warnings on touched files only.
+- PASS: `npx playwright test tests/playwright/tools/InputMappingV2Tool.spec.mjs --workers=1 --reporter=line`
+  - Final result: 4 passed.
+  - Initial run found only a test expectation mismatch because the registry API returns normalized `wireframe`; the assertion was corrected and the final run passed.
+
+## Playwright Coverage
+
+- Generated `docs_build/dev/reports/playwright_v8_coverage_report.txt`.
+- Generated `docs_build/dev/reports/coverage_changed_js_guardrail.txt`.
+- Coverage report includes an advisory WARN for `src/dev-runtime/guest-seeds/tool-metadata-inventory.js` because server-side seed metadata is not collected by browser V8 coverage.
+- Controls browser runtime coverage was collected for `toolbox/input-mapping-v2/input-mapping-v2.js`.
+
+## Manual Validation Steps
+
+1. Open `/toolbox/input-mapping-v2/index.html`.
+2. Confirm Controls shows as Wireframe in registry-backed tool metadata.
+3. Confirm Actions, Capture, Controller Profiles, Mappings, Mapping JSON, Devices, and Status panels appear.
+4. Confirm Controller Profiles shows Device Type, Controller Name, Controller ID, Mapping Profile, Input, and Action.
+5. Confirm the unknown controller status recommends Refresh Devices and points to the future Create Profile flow.
+6. Add a mapping row, save it, reload, and confirm it persists.
+7. Capture `KeyA`, confirm the mapping appears in the table and Mapping JSON, then click the token to delete it.
+
+## Skipped Lanes
+
+- Full samples smoke: SKIPPED because samples and sample JSON alignment are explicitly out of scope.
+- Broad workspace suite: SKIPPED because this PR only changes the Controls tool surface, Controls metadata, and targeted Controls Playwright coverage.
+- Engine tests: SKIPPED because no engine runtime files or controller runtime behavior changed.

docs_build/dev/reports/controls-wireframe-controller-profiles-report.md

@@ -7,14 +7,7 @@ Source: Playwright/Chromium built-in V8 coverage from the active Playwright run.
 
 Changed runtime JS files considered:
 (0%) src/dev-runtime/guest-seeds/tool-metadata-inventory.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
-(0%) src/dev-runtime/persistence/mock-db-store.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
-(0%) src/dev-runtime/persistence/tool-repositories/input-mapping-mock-repository.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
-(0%) src/dev-runtime/server/mock-api-router.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
 (92%) toolbox/input-mapping-v2/input-mapping-v2.js - executed lines 703/703; executed functions 80/87
-(100%) toolbox/input-mapping-v2/input-mapping-api-client.js - executed lines 12/12; executed functions 2/2
 
 Guardrail warnings:
 (0%) src/dev-runtime/guest-seeds/tool-metadata-inventory.js - WARNING: changed runtime JS file missing from coverage; advisory only
-(0%) src/dev-runtime/persistence/mock-db-store.js - WARNING: changed runtime JS file missing from coverage; advisory only
-(0%) src/dev-runtime/persistence/tool-repositories/input-mapping-mock-repository.js - WARNING: changed runtime JS file missing from coverage; advisory only
-(0%) src/dev-runtime/server/mock-api-router.mjs - WARNING: changed runtime JS file missing from coverage; advisory only

docs_build/dev/reports/coverage_changed_js_guardrail.txt

@@ -18,11 +18,7 @@ Exercised tool entry points detected:
 
 Changed runtime JS files covered:
 (0%) src/dev-runtime/guest-seeds/tool-metadata-inventory.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
-(0%) src/dev-runtime/persistence/mock-db-store.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
-(0%) src/dev-runtime/persistence/tool-repositories/input-mapping-mock-repository.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
-(0%) src/dev-runtime/server/mock-api-router.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
 (92%) toolbox/input-mapping-v2/input-mapping-v2.js - executed lines 703/703; executed functions 80/87
-(100%) toolbox/input-mapping-v2/input-mapping-api-client.js - executed lines 12/12; executed functions 2/2
 
 Files with executed line/function counts where available:
 (53%) src/engine/api/server-api-client.js - executed lines 159/159; executed functions 10/19
@@ -35,15 +31,8 @@ Files with executed line/function counts where available:
 
 Uncovered or low-coverage changed JS files:
 (0%) src/dev-runtime/guest-seeds/tool-metadata-inventory.js - WARNING: uncovered changed runtime JS file; advisory only
-(0%) src/dev-runtime/persistence/mock-db-store.js - WARNING: uncovered changed runtime JS file; advisory only
-(0%) src/dev-runtime/persistence/tool-repositories/input-mapping-mock-repository.js - WARNING: uncovered changed runtime JS file; advisory only
-(0%) src/dev-runtime/server/mock-api-router.mjs - WARNING: uncovered changed runtime JS file; advisory only
 
 Changed JS files considered:
 (0%) src/dev-runtime/guest-seeds/tool-metadata-inventory.js - changed JS file not collected as browser runtime coverage
-(0%) src/dev-runtime/persistence/mock-db-store.js - changed JS file not collected as browser runtime coverage
-(0%) src/dev-runtime/persistence/tool-repositories/input-mapping-mock-repository.js - changed JS file not collected as browser runtime coverage
-(0%) src/dev-runtime/server/mock-api-router.mjs - changed JS file not collected as browser runtime coverage
 (0%) tests/playwright/tools/InputMappingV2Tool.spec.mjs - changed JS file not collected as browser runtime coverage
 (92%) toolbox/input-mapping-v2/input-mapping-v2.js - changed JS file with browser V8 coverage
-(100%) toolbox/input-mapping-v2/input-mapping-api-client.js - changed JS file with browser V8 coverage

docs_build/dev/reports/playwright_v8_coverage_report.txt

@@ -730,8 +730,10 @@ export const TOOL_REGISTRY = Object.freeze([
     "requiredForTestable": true,
     "requiredForPublish": true,
     "requires": [],
-    "status": "beta",
+    "status": "Wireframe",
+    "releaseChannel": "wireframe",
     "progressChecklist": [
+      "Plan controller profiles",
       "Map object actions",
       "Capture keyboard, mouse, and gamepad inputs",
       "Persist mappings through shared DB adapter"

src/dev-runtime/guest-seeds/tool-metadata-inventory.js

@@ -129,11 +129,25 @@ test("Controls Input Mapping launch panels, defaults, diagnostics, and workspace
     await expect(page.locator("style, [style], script:not([src])")).toHaveCount(0);
     await expect(page.locator("summary").filter({ hasText: "Actions" })).toBeVisible();
     await expect(page.locator("summary").filter({ hasText: "Capture" })).toBeVisible();
+    await expect(page.locator("summary").filter({ hasText: "Controller Profiles" })).toBeVisible();
     await expect(page.getByRole("heading", { name: "Mappings" })).toBeVisible();
     await expect(page.getByText("Mapping JSON", { exact: true })).toBeVisible();
     await expect(page.getByText("Devices", { exact: true })).toBeVisible();
     await expect(page.getByText("Status", { exact: true })).toBeVisible();
     await expect(page.locator("[data-input-return-workspace]")).toBeVisible();
+    await expect(page.locator("[data-controller-profile-table] th")).toHaveText([
+      "Device Type",
+      "Controller Name",
+      "Controller ID",
+      "Mapping Profile",
+      "Input",
+      "Action",
+    ]);
+    await expect(page.locator("[data-controller-profile-table]")).toContainText("Unknown Controller");
+    await expect(page.locator("[data-controller-profile-table]")).toContainText("Create Profile Needed");
+    await expect(page.locator("[data-controller-profile-status]")).toContainText("WARN: Unknown controller.");
+    await expect(page.locator("[data-controller-profile-status]")).toContainText("future Create Profile flow");
+    await expect(page.locator("[data-controller-profile-planning]")).toContainText("Future game launch will match Controller ID to a saved Mapping Profile");
     await expect(page.locator("[data-input-mapping-table] th")).toHaveText([
       "Object",
       "Action",
@@ -175,7 +189,10 @@ test("Controls Input Mapping launch panels, defaults, diagnostics, and workspace
 
     const registryEntry = await controlsRegistryEntry(page);
     expect(registryEntry.path).toBe("toolbox/input-mapping-v2/index.html");
-    expect(registryEntry.status).toBe("beta");
+    expect(registryEntry.status).toBe("wireframe");
+    expect(registryEntry.status).not.toBe("beta");
+    expect(registryEntry.releaseChannel).toBe("wireframe");
+    expect(registryEntry.releaseChannelLabel).toBe("Wireframe");
 
     await expectNoPageFailures(failures);
   } finally {
@@ -303,6 +320,8 @@ test("Controls compatibility route uses rebuilt Input Mapping surface", async ({
 
   try {
     await expect(page.getByRole("heading", { name: "Input Mapping" })).toBeVisible();
+    await expect(page.locator("summary").filter({ hasText: "Controller Profiles" })).toBeVisible();
+    await expect(page.locator("[data-controller-profile-table]")).toContainText("Mapping Profile");
     await expect(page.locator("[data-input-mapping-table]")).toBeVisible();
     await expect(page.locator("main")).not.toContainText(/Static wireframe only|Not implemented yet|no database|no runtime behavior/i);
     await expectNoPageFailures(failures);

tests/playwright/tools/InputMappingV2Tool.spec.mjs

@@ -56,6 +56,38 @@ <h2>Controls</h2>
                                     <p class="status" role="status" data-input-capture-status>Choose an action and capture an input.</p>
                                 </div>
                             </details>
+                            <details class="vertical-accordion" open>
+                                <summary>Controller Profiles</summary>
+                                <div class="accordion-body content-stack" data-controller-profile-planning>
+                                    <p>Plan how gamepad identities will match saved mapping profiles.</p>
+                                    <div class="table-wrapper">
+                                        <table class="data-table" aria-label="Controller profile planning" data-controller-profile-table>
+                                            <thead>
+                                                <tr>
+                                                    <th>Device Type</th>
+                                                    <th>Controller Name</th>
+                                                    <th>Controller ID</th>
+                                                    <th>Mapping Profile</th>
+                                                    <th>Input</th>
+                                                    <th>Action</th>
+                                                </tr>
+                                            </thead>
+                                            <tbody>
+                                                <tr>
+                                                    <td>Gamepad</td>
+                                                    <td>Unknown Controller</td>
+                                                    <td>Awaiting Device</td>
+                                                    <td>Create Profile Needed</td>
+                                                    <td>Button or Axis</td>
+                                                    <td>Select Action</td>
+                                                </tr>
+                                            </tbody>
+                                        </table>
+                                    </div>
+                                    <p>Future game launch will match Controller ID to a saved Mapping Profile before applying inputs.</p>
+                                    <p class="status" role="status" data-controller-profile-status>WARN: Unknown controller. Refresh Devices after connecting a controller; future Create Profile flow will save the controller identity.</p>
+                                </div>
+                            </details>
                         </div>
                     </aside>
                     <div data-tool-display-mode data-asset-root="assets/theme-v2/images" data-tool-slug="controls" data-tool-icon-src="assets/theme-v2/images/badges/controls.png" data-tool-character-src="assets/theme-v2/images/tools/controls.png"></div>

toolbox/controls/index.html

@@ -56,6 +56,38 @@ <h2>Controls</h2>
                                     <p class="status" role="status" data-input-capture-status>Choose an action and capture an input.</p>
                                 </div>
                             </details>
+                            <details class="vertical-accordion" open>
+                                <summary>Controller Profiles</summary>
+                                <div class="accordion-body content-stack" data-controller-profile-planning>
+                                    <p>Plan how gamepad identities will match saved mapping profiles.</p>
+                                    <div class="table-wrapper">
+                                        <table class="data-table" aria-label="Controller profile planning" data-controller-profile-table>
+                                            <thead>
+                                                <tr>
+                                                    <th>Device Type</th>
+                                                    <th>Controller Name</th>
+                                                    <th>Controller ID</th>
+                                                    <th>Mapping Profile</th>
+                                                    <th>Input</th>
+                                                    <th>Action</th>
+                                                </tr>
+                                            </thead>
+                                            <tbody>
+                                                <tr>
+                                                    <td>Gamepad</td>
+                                                    <td>Unknown Controller</td>
+                                                    <td>Awaiting Device</td>
+                                                    <td>Create Profile Needed</td>
+                                                    <td>Button or Axis</td>
+                                                    <td>Select Action</td>
+                                                </tr>
+                                            </tbody>
+                                        </table>
+                                    </div>
+                                    <p>Future game launch will match Controller ID to a saved Mapping Profile before applying inputs.</p>
+                                    <p class="status" role="status" data-controller-profile-status>WARN: Unknown controller. Refresh Devices after connecting a controller; future Create Profile flow will save the controller identity.</p>
+                                </div>
+                            </details>
                         </div>
                     </aside>
                     <div data-tool-display-mode data-asset-root="assets/theme-v2/images" data-tool-slug="controls" data-tool-icon-src="assets/theme-v2/images/badges/controls.png" data-tool-character-src="assets/theme-v2/images/tools/controls.png"></div>