Clean up Object Vector Studio V2 control layout, shape selection, palette, preview, and JSON copy behavior - PR_26132_018-object-vector-studio-v2-control-layout-cleanup

Author: DavidQ

docs/dev/reports/playwright_v8_coverage.txt

@@ -22,9 +22,7 @@ Exercised tool entry points detected:
 Changed runtime JS files covered:
 (80%) tools/object-vector-studio-v2/js/bootstrap.js - executed lines 101/101; executed functions 4/5
 (91%) tools/object-vector-studio-v2/js/controls/ActionNavControl.js - executed lines 78/78; executed functions 10/11
-(92%) tools/object-vector-studio-v2/js/ToolStarterApp.js - executed lines 2862/2862; executed functions 328/356
-(95%) tools/object-vector-studio-v2/js/services/ObjectVectorStudioV2SchemaService.js - executed lines 406/406; executed functions 52/55
-(98%) src/engine/rendering/ObjectVectorRuntimeAssetService.js - executed lines 919/919; executed functions 105/107
+(91%) tools/object-vector-studio-v2/js/ToolStarterApp.js - executed lines 2879/2879; executed functions 334/366
 
 Files with executed line/function counts where available:
 (2%) src/engine/input/ActionInputService.js - executed lines 397/397; executed functions 1/51
@@ -200,10 +198,10 @@ Files with executed line/function counts where available:
 (90%) tools/text2speech-V2/js/controls/ActionNavControl.js - executed lines 117/117; executed functions 19/21
 (90%) tools/text2speech-V2/js/TextToSpeechToolApp.js - executed lines 807/807; executed functions 62/69
 (91%) tools/object-vector-studio-v2/js/controls/ActionNavControl.js - executed lines 78/78; executed functions 10/11
+(91%) tools/object-vector-studio-v2/js/ToolStarterApp.js - executed lines 2879/2879; executed functions 334/366
 (91%) tools/toolRegistry.js - executed lines 526/526; executed functions 10/11
 (91%) tools/workspace-manager-v2/js/services/WorkspaceManagerV2ContextService.js - executed lines 1598/1598; executed functions 145/159
 (92%) tools/object-vector-studio-v2/js/controls/ToolStarterShellControl.js - executed lines 112/112; executed functions 11/12
-(92%) tools/object-vector-studio-v2/js/ToolStarterApp.js - executed lines 2862/2862; executed functions 328/356
 (93%) tools/asset-manager-v2/js/services/WorkspaceBridge.js - executed lines 305/305; executed functions 25/27
 (93%) tools/session-inspector-v2/js/SessionInspectorV2App.js - executed lines 337/337; executed functions 42/45
 (93%) tools/text2speech-V2/js/controls/QueueControl.js - executed lines 122/122; executed functions 26/28
@@ -296,10 +294,7 @@ Uncovered or low-coverage changed JS files:
 
 Changed JS files considered:
 (0%) tests/playwright/tools/WorkspaceManagerV2.spec.mjs - changed JS file not collected as browser runtime coverage
-(50%) games/Asteroids/index.js - changed JS file with browser V8 coverage
-(52%) games/Asteroids/game/AsteroidsGameScene.js - changed JS file with browser V8 coverage
+(0%) tools/object-vector-studio-v2/tests/playwright/FirstClassToolStarter.spec.mjs - changed JS file not collected as browser runtime coverage
 (80%) tools/object-vector-studio-v2/js/bootstrap.js - changed JS file with browser V8 coverage
 (91%) tools/object-vector-studio-v2/js/controls/ActionNavControl.js - changed JS file with browser V8 coverage
-(92%) tools/object-vector-studio-v2/js/ToolStarterApp.js - changed JS file with browser V8 coverage
-(95%) tools/object-vector-studio-v2/js/services/ObjectVectorStudioV2SchemaService.js - changed JS file with browser V8 coverage
-(98%) src/engine/rendering/ObjectVectorRuntimeAssetService.js - changed JS file with browser V8 coverage
+(91%) tools/object-vector-studio-v2/js/ToolStarterApp.js - changed JS file with browser V8 coverage

docs/pr/PR_26132_018-object-vector-studio-v2-control-layout-cleanup.md

@@ -0,0 +1,79 @@
+# PR_26132_018-object-vector-studio-v2-control-layout-cleanup
+
+## Scope
+
+Updates Object Vector Studio V2 UI layout only. No schema changes, no sample JSON changes, and no new feature systems.
+
+## Changes
+
+- Reduced stacked accordion spacing and kept removed useful help/status text in Status Log.
+- Moved Shape/Tools to the top of the right column, removed its visible hint paragraph, kept its content scrollable, and made tool buttons square.
+- Moved Words/Icons beside Grid and preserved icon mode for preview/frame action buttons.
+- Added a selectable shape list inside Object Details for the selected object.
+- Moved Object Details under Object in the left column and moved object/shape counts into the Object Details header.
+- Moved Dependency Graph into its own right-column accordion.
+- Reworked Palette as an accordion with Paint, Stroke, Current Color, Stroke Width, header swatch count, square swatches, and internal scrolling.
+- Moved Object Preview coordinate/status text into the Object Preview header and removed the old visible selection/render summary blocks.
+- Added Copy JSON inside JSON Details without changing accordion open/closed state.
+
+## Validation
+
+Playwright impacted: Yes.
+
+Commands run:
+
+- `node --check tools/object-vector-studio-v2/js/ToolStarterApp.js`
+- `node --check tools/object-vector-studio-v2/js/bootstrap.js`
+- `node --check tests/playwright/tools/WorkspaceManagerV2.spec.mjs`
+- `node --check tools/object-vector-studio-v2/tests/playwright/FirstClassToolStarter.spec.mjs`
+- `npx playwright test tests/playwright/tools/WorkspaceManagerV2.spec.mjs --project=playwright --grep "Object Vector Studio V2" --reporter=list`
+- `npm run test:workspace-v2`
+
+Result:
+
+- Targeted Object Vector Studio V2 validation passed: 6 passed.
+- Full Workspace Manager V2 suite passed: 45 passed.
+- Playwright V8 coverage generated at `docs/dev/reports/playwright_v8_coverage.txt`.
+- Full samples smoke test skipped per request; this PR only changes Object Vector Studio V2 UI layout and workspace-v2 coverage validates the impacted surface.
+
+## Playwright Coverage
+
+Validates:
+
+- Shape/Tools is in the right column.
+- Object Details is under Object in the left column.
+- Words/Icons is beside Grid.
+- Shape/tool buttons are square.
+- Object tiles do not overlap on load.
+- Shapes inside the selected object can be selected from the new shape list.
+- JSON Details Copy JSON preserves accordion state.
+- Removed useful status/help text appears in Status Log.
+
+Expected pass behavior:
+
+- Layout controls move to the requested columns, compact content scrolls internally, selectable shape rows are clickable, and JSON copy does not toggle the accordion.
+
+Expected fail behavior:
+
+- Invalid payloads, missing payloads, missing palette, or missing selected object still log visible/actionable failures and block invalid rendering or mutations.
+
+## Manual Validation
+
+1. Open `tools/object-vector-studio-v2/index.html`.
+2. Confirm left column order: Object, Object Details, Objects.
+3. Confirm right column order: Shape/Tools, Palette, JSON Details, Dependency Graph, Status Log.
+4. Load a valid Object Vector payload with a runtime palette.
+5. Create multiple shapes, then select shapes from the Object Details shape list.
+6. Confirm Paint/Stroke, Current Color, Stroke Width, and palette swatches sit at the top of Palette.
+7. Click JSON Details `Copy JSON` and confirm the accordion stays open.
+
+Expected outcome:
+
+- Object Vector Studio V2 has tighter accordions, usable right-column Shape/Tools, non-overlapping object tiles, clickable shape selection, palette controls at the top, and stable JSON copy behavior.
+
+## Out Of Scope
+
+- No schema changes.
+- No sample JSON changes.
+- No new feature systems.
+- No full samples smoke test.