Refine Object Vector Studio frame controls palette paint modes and shape action placement - PR_26133_046-object-vector-frame-palette-and-shape-action-cleanup

Author: DavidQ

docs/dev/reports/playwright_v8_coverage_report.md

@@ -1,6 +1,6 @@
-# PR_26133_045 Playwright V8 Coverage Report
+# PR_26133_046 Playwright V8 Coverage Report
 
-Task: PR_26133_045-object-preview-pan-direction-and-strict-schema-fix
+Task: PR_26133_046-object-vector-frame-palette-and-shape-action-cleanup
 Date: 2026-05-15
 
 ## Result
@@ -24,6 +24,7 @@ PASS - Coverage reporting was generated during `npm run test:workspace-v2`.
 ## Relevant Runtime Coverage
 
 ```text
+(80%) tools/object-vector-studio-v2/js/bootstrap.js - executed lines 97/97; executed functions 4/5
 (90%) tools/object-vector-studio-v2/js/ToolStarterApp.js - executed lines 3187/3187; executed functions 331/369
 (95%) tools/object-vector-studio-v2/js/services/ObjectVectorStudioV2SchemaService.js - executed lines 409/409; executed functions 53/56
 ```
@@ -36,4 +37,4 @@ PASS - Coverage reporting was generated during `npm run test:workspace-v2`.
 
 ## PR-Specific Note
 
-The Workspace V2 run exercised Object Vector Studio V2 preview pan controls, center-dot visibility, coordinate/origin display updates, transform-origin pivot rendering, and strict schema rejection for unknown runtime/workspace fields.
+The Workspace V2 run exercised Object Vector Studio V2 frame creation/duplication/deletion/reordering, palette sort controls, Paint/Stroke mode application through canvas clicks, and icon-only shape order/group controls under Objects.

docs/dev/reports/playwright_workspace_v2_results.md

@@ -1,6 +1,6 @@
-# PR_26133_045 Workspace V2 Playwright Results
+# PR_26133_046 Workspace V2 Playwright Results
 
-Task: PR_26133_045-object-preview-pan-direction-and-strict-schema-fix
+Task: PR_26133_046-object-vector-frame-palette-and-shape-action-cleanup
 Date: 2026-05-15
 
 ## Result
@@ -10,22 +10,25 @@ PASS - `npm run test:workspace-v2` completed successfully.
 - Command: `npm run test:workspace-v2`
 - Playwright target: `tests/playwright/tools/WorkspaceManagerV2.spec.mjs --project=playwright --workers=1 --reporter=list`
 - Final result: 51 passed, 0 failed.
-- Runtime/console guard: Workspace Manager V2 and Object Vector Studio V2 scenarios completed with no reported page errors.
+- Runtime/console guard: Workspace Manager V2, Object Vector Studio V2, and Asteroids runtime scenarios completed with no reported page errors.
 
 ## PR-Specific Coverage
 
-- Verified the Center control toggles the center dot without recentering or changing the viewport.
-- Verified Up/Down/Left/Right pan only the viewport origin and update the Canvas origin display.
-- Verified pan controls do not mutate selected shape geometry or transform state.
-- Verified pointer X/Y coordinate display remains logical after the viewport-origin display change.
-- Verified the transform origin/pivot marker reflects the applied origin used for rotation/scale behavior.
-- Verified `game.gameData.objectVectorRuntime` and unknown `gameData` fields fail strict schema validation.
-- Verified the workspace manifest schema rejects an unknown root field.
+- Verified Delete Frame removes a selected frame and refuses to delete the final remaining frame.
+- Verified generated and canonical Asteroids frame IDs use `frame-x` instead of state-prefixed frame IDs.
+- Verified Left/Right frame controls and Frame Earlier/Frame Later reorder frames through the same sequence-safe path.
+- Verified palette sort buttons show ascending/descending caret indicators and toggle sort direction.
+- Verified Paint/Stroke buttons switch action mode without changing selected shape style.
+- Verified canvas clicks still apply the currently selected Paint/Stroke color.
+- Verified shape order/group actions moved under Objects and render as icon-only controls with titles.
 
 ## Additional Validation
 
-- `node` JSON parse check passed for `tools/schemas/game.manifest.schema.json`, `tools/schemas/workspace.manifest.schema.json`, and `games/Asteroids/game.manifest.json`.
-- `node` schema-service check: current Asteroids manifest returned `{ ok: true }`.
-- `node` schema-service check: injected `game.gameData.objectVectorRuntime` returned `{ ok: false }`.
-- `node` schema-service check: injected `game.gameData.debugUnknown` returned `{ ok: false }`.
-- `node` schema-service check: injected workspace root `debugUnknown` returned `{ ok: false }`.
+- Targeted Playwright check passed for Object Vector Studio layout, animation frame controls, and Asteroids runtime rendering.
+- Node syntax checks passed for `tools/object-vector-studio-v2/js/ToolStarterApp.js` and `tools/object-vector-studio-v2/js/bootstrap.js`.
+- Node schema-service check passed for `games/Asteroids/game.manifest.json` and its workspace manifest.
+- `git diff --check` passed.
+
+## Frame Control Note
+
+`Left` and `Right` are directional aliases for the same frame-order mutation used by `Frame Earlier` and `Frame Later`; all four controls preserve the selected state linkage and only change frame order.

games/Asteroids/game.manifest.json

@@ -336,7 +336,7 @@
                   "name": "Idle",
                   "frames": [
                     {
-                      "id": "idle-frame-1",
+                      "id": "frame-1",
                       "order": 0,
                       "durationFrames": 1,
                       "shapeOverrides": [
@@ -357,7 +357,7 @@
                   "name": "Thrust",
                   "frames": [
                     {
-                      "id": "thrust-frame-1",
+                      "id": "frame-1",
                       "order": 0,
                       "durationFrames": 1,
                       "shapeOverrides": [
@@ -466,7 +466,7 @@
                   "name": "Active",
                   "frames": [
                     {
-                      "id": "active-frame-1",
+                      "id": "frame-1",
                       "order": 0,
                       "durationFrames": 1,
                       "shapeOverrides": []
@@ -542,7 +542,7 @@
                   "name": "Active",
                   "frames": [
                     {
-                      "id": "active-frame-1",
+                      "id": "frame-1",
                       "order": 0,
                       "durationFrames": 1,
                       "shapeOverrides": []
@@ -626,7 +626,7 @@
                   "name": "Active",
                   "frames": [
                     {
-                      "id": "active-frame-1",
+                      "id": "frame-1",
                       "order": 0,
                       "durationFrames": 1,
                       "shapeOverrides": []
@@ -710,7 +710,7 @@
                   "name": "Active",
                   "frames": [
                     {
-                      "id": "active-frame-1",
+                      "id": "frame-1",
                       "order": 0,
                       "durationFrames": 1,
                       "shapeOverrides": []
@@ -790,7 +790,7 @@
                   "name": "Active",
                   "frames": [
                     {
-                      "id": "active-frame-1",
+                      "id": "frame-1",
                       "order": 0,
                       "durationFrames": 1,
                       "shapeOverrides": []
@@ -870,7 +870,7 @@
                   "name": "Active",
                   "frames": [
                     {
-                      "id": "active-frame-1",
+                      "id": "frame-1",
                       "order": 0,
                       "durationFrames": 1,
                       "shapeOverrides": []