Polish Input Mapping V2 layout and manifest registration - PR_26140_088-polish-input-mapping-v2-layout-and-manifest

Author: DavidQ

docs/dev/reports/input_mapping_v2_layout_manifest_report.md

@@ -0,0 +1,42 @@
+# Input Mapping V2 Layout and Manifest Report
+
+Task: PR_26140_088-polish-input-mapping-v2-layout-and-manifest
+
+## Summary
+- Updated Input Mapping V2 to use the full available horizontal workspace with fixed left/right columns and an expanding center mapping area.
+- Added vertical scrolling and fixed 175px by 175px grid tiles for captured mappings.
+- Kept the captured mapping list empty until an input is captured, then renders captured mappings only.
+- Set capture buttons to 150px wide.
+- Updated the right column accordions to share vertical space evenly.
+- Added `Start`, `Select`, and `Pause` to default actions and sorted all default/custom actions alphabetically.
+
+## Manifest and Schema Registration
+- Added `tools.input-mapping-v2` to `games/Asteroids/game.manifest.json` as a manifest-owned V2 tool payload with empty inputs.
+- Added the minimal `tools/schemas/tools/input-mapping-v2.schema.json` payload schema because the game manifest schema rejects unknown tool payload keys.
+- Updated `tools/schemas/game.manifest.schema.json` to allow `tools.input-mapping-v2`.
+- Updated Workspace Manager V2 payload schema registration so Input Mapping V2 validates through the same manifest/toolState path as other active V2 tools.
+
+## Playwright Impact
+Playwright impacted: Yes.
+
+Validated behavior:
+- Input Mapping V2 launches from the current V2 template shell.
+- Full-width layout pins left/right columns and expands the center workspace.
+- Mapping list starts empty, has vertical scroll behavior, and renders 175px grid tiles after capture.
+- Action dropdown includes `Pause`, `Select`, and `Start` and is sorted alphabetically.
+- Keyboard capture records `Keyboard KeyA` and clicking the token deletes it.
+- Gamepad capture logs an actionable WARN when live browser gamepad data is unavailable.
+- Asteroids game manifest includes the Input Mapping V2 payload and Workspace Manager validates it through the new schema.
+
+## Validation
+- PASS: targeted `node --check` syntax/import validation for changed Input Mapping V2 and Workspace Manager files.
+- PASS: JSON parse validation for `games/Asteroids/game.manifest.json`, `tools/schemas/game.manifest.schema.json`, and `tools/schemas/tools/input-mapping-v2.schema.json`.
+- PASS: HTML inline script/style/event-handler scan for Input Mapping V2 HTML files returned no matches.
+- PASS: focused Playwright validation for Input Mapping V2 and Workspace Manager tile launch paths.
+- PASS: `npm run test:workspace-v2` with 60 passing tests.
+- PASS: Playwright V8 coverage guardrail lists changed runtime JavaScript files with no WARN entries.
+
+## Scope Notes
+- No sample JSON files were changed.
+- Full samples smoke test was skipped because the PR is limited to Input Mapping V2 layout, focused manifest registration, and Workspace V2 validation.
+- No commits were created.

docs/tools/input-mapping-v2/uat.md

@@ -18,7 +18,8 @@
 ## Default Actions
 
 - Confirm the default actions include movement, confirm/cancel, fire, thrust, and rotation actions.
-- Confirm `Pause` is not present in the default action dropdown.
+- Confirm `Pause`, `Select`, and `Start` are present in the default action dropdown.
+- Confirm default actions are sorted alphabetically.
 
 ## Gamepad Capture
 

games/Asteroids/game.manifest.json

@@ -212,6 +212,104 @@
         }
       }
     },
+    "input-mapping-v2": {
+      "$schema": "tools/schemas/tools/input-mapping-v2.schema.json",
+      "toolId": "input-mapping-v2",
+      "version": 1,
+      "engineInputModel": "src/engine/input/InputMap",
+      "actions": [
+        {
+          "action": "cancel",
+          "label": "Cancel",
+          "inputs": []
+        },
+        {
+          "action": "confirm",
+          "label": "Confirm",
+          "inputs": []
+        },
+        {
+          "action": "fire",
+          "label": "Fire",
+          "inputs": []
+        },
+        {
+          "action": "interact",
+          "label": "Interact",
+          "inputs": []
+        },
+        {
+          "action": "jump",
+          "label": "Jump",
+          "inputs": []
+        },
+        {
+          "action": "menu",
+          "label": "Menu",
+          "inputs": []
+        },
+        {
+          "action": "moveDown",
+          "label": "Move Down",
+          "inputs": []
+        },
+        {
+          "action": "moveLeft",
+          "label": "Move Left",
+          "inputs": []
+        },
+        {
+          "action": "moveRight",
+          "label": "Move Right",
+          "inputs": []
+        },
+        {
+          "action": "moveUp",
+          "label": "Move Up",
+          "inputs": []
+        },
+        {
+          "action": "pause",
+          "label": "Pause",
+          "inputs": []
+        },
+        {
+          "action": "primaryAction",
+          "label": "Primary Action",
+          "inputs": []
+        },
+        {
+          "action": "rotateLeft",
+          "label": "Rotate Left",
+          "inputs": []
+        },
+        {
+          "action": "rotateRight",
+          "label": "Rotate Right",
+          "inputs": []
+        },
+        {
+          "action": "secondaryAction",
+          "label": "Secondary Action",
+          "inputs": []
+        },
+        {
+          "action": "select",
+          "label": "Select",
+          "inputs": []
+        },
+        {
+          "action": "start",
+          "label": "Start",
+          "inputs": []
+        },
+        {
+          "action": "thrust",
+          "label": "Thrust",
+          "inputs": []
+        }
+      ]
+    },
     "object-vector-studio-v2": {
       "version": 1,
       "toolId": "object-vector-studio-v2",