Fix Input Mapping V2 device and gesture wiring - PR_26140_115-fix-input-mapping-v2-device-gesture-wiring

Author: ToolboxAid

docs/dev/reports/input_mapping_v2_device_gesture_wiring_report.md

@@ -4,6 +4,7 @@ PR: PR_26140_115-fix-input-mapping-v2-device-gesture-wiring
 
 ## Summary
 - Renamed the center-column Capture accordion to `Input Device(s)` and Gestures to `Capture Gestures`, with no braces in either visible label.
+- Renamed the center-column mapping workspace from `Captured Mappings` to `Action Mappings`, including related aria labels and mapping-deletion status text.
 - Disabled and grayed `Input Device(s)` until the selected action has an existing tile, and disabled `Capture Gestures` until an input device is selected.
 - Enforced device-first capture flow: selecting an input device enables only that device's gesture group, and stale selected gestures no longer auto-arm when a device is clicked after capture completion.
 - Removed visible Mouse Wheel Left/Right gesture descriptors because PR_115 only requires visible/wired Wheel Up and Wheel Down; underlying wheel input support remains available through the engine where used by existing capture/combo paths.
@@ -21,6 +22,9 @@ PR: PR_26140_115-fix-input-mapping-v2-device-gesture-wiring
 - PASS: targeted module import validation for changed engine/input and Input Mapping V2 JS modules.
 - PASS: focused Input Mapping V2 Playwright coverage: 11 passed.
 - PASS: `npm run test:workspace-v2`: 70 passed.
+- PASS: additional label follow-up targeted syntax validation with `node --check tools/input-mapping-v2/js/services/InputMappingState.js` and `node --check tests/playwright/tools/WorkspaceManagerV2.spec.mjs`.
+- PASS: additional label follow-up focused Playwright validation with `npx playwright test tests/playwright/tools/WorkspaceManagerV2.spec.mjs --project=playwright --workers=1 --reporter=list --grep "launches Input Mapping V2"`: 1 passed.
+- PASS: additional label follow-up full Workspace V2 validation with `npm run test:workspace-v2`: 70 passed.
 - SKIPPED as requested: full samples smoke test.
 
 ## Schema/Samples

tests/playwright/tools/WorkspaceManagerV2.spec.mjs

@@ -2586,8 +2586,11 @@ test.describe("Workspace Manager V2 bootstrap", () => {
           timestamp: 78.75
         };
       });
-      await expect(gamepadReleaseToken).toHaveClass(/is-action-active/, { timeout: 2500 });
-      await expect(gamepadReleaseToken).toHaveCSS("border-top-color", "rgb(245, 159, 0)");
+      await page.waitForFunction(() => {
+        const token = document.querySelector(".input-mapping-v2__input-token[data-input-mapping-binding='Pad0:Button2:GameControllerButtonRelease']");
+        return token?.classList.contains("is-action-active")
+          && getComputedStyle(token).borderTopColor === "rgb(245, 159, 0)";
+      }, null, { timeout: 2500 });
 
       await page.evaluate(() => {
         window.__inputMappingV2MockGamepads[0] = {
@@ -2821,7 +2824,7 @@ test.describe("Workspace Manager V2 bootstrap", () => {
       expect(fullscreenLayout.centerWidth).toBeGreaterThan(1000);
       expect(Math.max(...fullscreenLayout.rightAccordionHeights) - Math.min(...fullscreenLayout.rightAccordionHeights)).toBeLessThanOrEqual(3);
       await expect(page.locator(".tool-starter__panel--left > .tool-starter__accordion > .accordion-v2__header > span:first-child")).toHaveText(["Actions", "Devices"]);
-      await expect(page.locator(".tool-starter__panel--center > .tool-starter__accordion > .accordion-v2__header > span:first-child")).toHaveText(["Input Device(s)", "Capture Gestures", "Captured Mappings"]);
+      await expect(page.locator(".tool-starter__panel--center > .tool-starter__accordion > .accordion-v2__header > span:first-child")).toHaveText(["Input Device(s)", "Capture Gestures", "Action Mappings"]);
       const centerAccordionLabels = await page.locator(".tool-starter__panel--center > .tool-starter__accordion > .accordion-v2__header > span:first-child").allTextContents();
       expect(centerAccordionLabels.some((label) => /[{}]/.test(label))).toBe(false);
       await expect(page.locator(".tool-starter__panel--right > .tool-starter__accordion > .accordion-v2__header > span:first-child")).toHaveText(["Diagnostics", "JSON", "Status / Log"]);
@@ -3254,7 +3257,7 @@ test.describe("Workspace Manager V2 bootstrap", () => {
       await page.locator("#inputMappingV2DeleteMappingsButton").click();
       await expect(page.locator(".input-mapping-v2__mapping-card", { hasText: "Confirm" })).toHaveCount(1);
       await expect(page.locator(".input-mapping-v2__mapping-card", { hasText: "Confirm" })).toContainText("No inputs captured.");
-      await expect(page.locator("#statusLog")).toHaveValue(/OK Deleted captured mappings from Confirm\./);
+      await expect(page.locator("#statusLog")).toHaveValue(/OK Deleted action mappings from Confirm\./);
       await page.locator("#inputMappingV2MappingDeleteActionButton").click();
       await expect(page.locator(".input-mapping-v2__mapping-card", { hasText: "Confirm" })).toHaveCount(0);
       await expect(page.locator(".input-mapping-v2__mapping-card", { hasText: "Move Left" })).toHaveCount(1);

tools/input-mapping-v2/index.html

@@ -102,7 +102,7 @@ <h2 class="tools-platform-frame__eyebrow">Input Sources and Action Mapping</h2>
 
       </aside>
 
-      <section class="tool-starter__panel tool-starter__panel--center" aria-label="Captured mappings">
+      <section class="tool-starter__panel tool-starter__panel--center" aria-label="Action mappings">
         <section class="accordion-v2 tool-starter__accordion is-open" data-accordion-v2-open="true">
           <div class="accordion-v2__header input-mapping-v2__capture-header" aria-expanded="true" aria-controls="captureInputContent">
             <span>Input Device(s)</span>
@@ -143,11 +143,11 @@ <h2 class="tools-platform-frame__eyebrow">Input Sources and Action Mapping</h2>
 
         <section class="accordion-v2 tool-starter__accordion tool-starter__accordion--fill is-open" data-accordion-v2-open="true">
           <button class="accordion-v2__header" type="button" aria-expanded="true" aria-controls="mappingPanelContent">
-            <span>Captured Mappings</span>
+            <span>Action Mappings</span>
             <span class="accordion-v2__icon" aria-hidden="true">+</span>
           </button>
           <div id="mappingPanelContent" class="accordion-v2__content">
-            <div id="previewOutput" class="tool-starter__preview input-mapping-v2__mapping-list" aria-label="Captured action input mappings">
+            <div id="previewOutput" class="tool-starter__preview input-mapping-v2__mapping-list" aria-label="Action input mappings">
               <p>No inputs captured yet.</p>
             </div>
             <hr class="input-mapping-v2__mapping-actions-separator">

tools/input-mapping-v2/js/services/InputMappingState.js

@@ -147,23 +147,23 @@ export class InputMappingState {
       return { ok: false, message: `${action.label} does not have an action tile.` };
     }
     if (!action.inputs.length) {
-      return { ok: false, message: `${action.label} has no captured mappings to delete.` };
+      return { ok: false, message: `${action.label} has no action mappings to delete.` };
     }
     action.inputs = [];
     action.tileVisible = true;
     this.syncInputMap();
-    return { ok: true, message: `Deleted captured mappings from ${action.label}.` };
+    return { ok: true, message: `Deleted action mappings from ${action.label}.` };
   }
 
   deleteActionInput(actionId, binding) {
     const action = this.actionEntries.find((candidate) => candidate.id === actionId) ?? null;
     if (!action) {
-      return { ok: false, message: "Select an action before deleting a captured mapping." };
+      return { ok: false, message: "Select an action before deleting an action mapping." };
     }
     const input = action.inputs.find((candidate) => candidate.binding === binding) ?? null;
     if (!input) {
       this.selectedActionId = action.id;
-      return { ok: false, message: `${action.label} does not have that captured mapping.` };
+      return { ok: false, message: `${action.label} does not have that action mapping.` };
     }
 
     action.inputs = action.inputs.filter((candidate) => candidate.binding !== binding);