Replace Objects dev copy with production-safe status summary - PR_26161_006-objects-production-copy-and-status

Author: ToolboxAid

docs_build/dev/reports/codex_changed_files.txt

@@ -1,18 +1,22 @@
 # git status --short
 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/ObjectsTool.spec.mjs
+ M tests/playwright/tools/ToolboxAdminMetadataSsot.spec.mjs
  M toolbox/objects/index.html
  M toolbox/objects/objects.js
-?? docs_build/dev/reports/objects-role-type-save-fix-report.md
+?? docs_build/dev/reports/objects-production-copy-and-status-report.md
 
 # git ls-files --others --exclude-standard
-docs_build/dev/reports/objects-role-type-save-fix-report.md
+docs_build/dev/reports/objects-production-copy-and-status-report.md
 
 # git diff --stat
-.../dev/reports/coverage_changed_js_guardrail.txt  |   6 +-
+.../dev/reports/coverage_changed_js_guardrail.txt  |   3 +-
  .../dev/reports/playwright_v8_coverage_report.txt  |  13 +-
- tests/playwright/tools/ObjectsTool.spec.mjs        |  32 +++--
- toolbox/objects/index.html                         |   8 +-
- toolbox/objects/objects.js                         | 142 ++++++++++++---------
- 5 files changed, 111 insertions(+), 90 deletions(-)
+ .../guest-seeds/tool-metadata-inventory.js         |  10 +-
+ tests/playwright/tools/ObjectsTool.spec.mjs        |  66 ++++++--
+ .../tools/ToolboxAdminMetadataSsot.spec.mjs        |   2 +-
+ toolbox/objects/index.html                         |  41 +++--
+ toolbox/objects/objects.js                         | 180 ++++++++++++---------
+ 7 files changed, 190 insertions(+), 125 deletions(-)

docs_build/dev/reports/codex_review.diff

@@ -7,8 +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
-(67%) src/engine/object-model/objectModelRegistry.js - executed lines 139/139; executed functions 4/6
-(94%) toolbox/objects/objects.js - executed lines 762/762; executed functions 80/85
+(93%) toolbox/objects/objects.js - executed lines 790/790; executed functions 75/81
 
 Guardrail warnings:
 (0%) src/dev-runtime/guest-seeds/tool-metadata-inventory.js - WARNING: changed runtime JS file missing from coverage; advisory only

docs_build/dev/reports/coverage_changed_js_guardrail.txt

@@ -0,0 +1,35 @@
+# PR_26161_006 Objects Production Copy And Status Report
+
+## Scope
+- Read `docs_build/dev/PROJECT_INSTRUCTIONS.md` before edits.
+- Verified current branch is `main`.
+- Treated Objects copy as production-facing while preserving table-first editing, wireframe status, Add Object below the table, Add-disable behavior, row Edit/Trash/Cancel, and sprite asset linking.
+- No sample JSON alignment, auth behavior, production DB behavior, or engine runtime behavior was added.
+
+## Implementation Evidence
+- Replaced Objects center copy with creator-facing wording: `Object Builder`, production-safe page description, and neutral object table guidance.
+- Replaced Readiness Checks with an `Object Status` table covering ready objects, render assets, missing hitboxes, and missing events.
+- Removed low-value Object Status rows for object row, object names, and object roles.
+- Preserved wireframe-safe labels for unavailable hitbox/event values with `Not connected yet` messaging.
+- Kept blank/None render rows free of render asset requirements and preserved Sprite render asset create/resolve behavior.
+- Updated the Objects tool metadata copy because the runtime page title consumes the DB/mock metadata record.
+
+## Validation
+- PASS: `node --check toolbox/objects/objects.js`
+- PASS: `node --check tests/playwright/tools/ObjectsTool.spec.mjs`
+- PASS: `node --check src/dev-runtime/guest-seeds/tool-metadata-inventory.js`
+- PASS: `node --check tests/playwright/tools/ToolboxAdminMetadataSsot.spec.mjs`
+- PASS: `npx playwright test tests/playwright/tools/ObjectsTool.spec.mjs --workers=1 --reporter=line` -> 4 passed.
+- PASS: `npx playwright test tests/playwright/tools/ToolboxAdminMetadataSsot.spec.mjs --workers=1 --reporter=line` -> 4 passed after aligning the Objects capability label assertion with the updated metadata copy.
+- PASS: `git diff --check` with line-ending warnings only.
+- PASS: Objects Playwright validates removed internal copy is absent, `Object Status` appears, table editing works, Toolbox clickability remains, and Sprite render asset linking still works.
+
+## Coverage
+- Produced `docs_build/dev/reports/playwright_v8_coverage_report.txt`.
+- Objects runtime coverage: `toolbox/objects/objects.js` at 93% function coverage.
+- Metadata seed coverage is advisory-only at 0% because that seed module is not collected as browser runtime JavaScript; the conditional Toolbox/Admin metadata Playwright lane covers the metadata behavior.
+
+## Skipped Lanes
+- Full samples validation skipped by request; no sample JSON or sample runtime files changed.
+- Production DB/auth lanes skipped; this PR only updates the mock/DB metadata seed copy and Objects UI behavior, with no production DB or auth behavior changes.
+- Engine runtime lanes skipped; no engine runtime files or runtime object execution behavior changed.

docs_build/dev/reports/objects-production-copy-and-status-report.md

@@ -12,14 +12,13 @@ Note: entry percentages use function coverage when available, otherwise line cov
 Note: coverage entries are aggregated across every page/tool where coverageReporter.start(page) and coverageReporter.stop(page) ran.
 
 Exercised tool entry points detected:
-(85%) Toolbox Index - exercised 5 runtime JS files
+(84%) Toolbox Index - exercised 5 runtime JS files
 (0%) Tool Template V2 - not exercised by this Playwright run
 (59%) Theme V2 Shared JS - exercised 2 runtime JS files
 
 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
-(67%) src/engine/object-model/objectModelRegistry.js - executed lines 139/139; executed functions 4/6
-(94%) toolbox/objects/objects.js - executed lines 762/762; executed functions 80/85
+(93%) toolbox/objects/objects.js - executed lines 790/790; executed functions 75/81
 
 Files with executed line/function counts where available:
 (33%) src/engine/api/toolbox-votes-api-client.js - executed lines 46/46; executed functions 2/6
@@ -33,7 +32,7 @@ Files with executed line/function counts where available:
 (75%) src/engine/object-model/objectDefinitionSchema.js - executed lines 58/58; executed functions 3/4
 (77%) toolbox/tools-page-accordions.js - executed lines 965/965; executed functions 79/103
 (86%) toolbox/tool-registry-api-client.js - executed lines 152/152; executed functions 24/28
-(94%) toolbox/objects/objects.js - executed lines 762/762; executed functions 80/85
+(93%) toolbox/objects/objects.js - executed lines 790/790; executed functions 75/81
 (100%) src/engine/object-model/index.js - executed lines 29/29; executed functions 1/1
 (100%) toolbox/project-workspace/project-workspace-api-client.js - executed lines 12/12; executed functions 2/2
 
@@ -42,10 +41,6 @@ Uncovered or low-coverage changed JS files:
 
 Changed JS files considered:
 (0%) src/dev-runtime/guest-seeds/tool-metadata-inventory.js - changed JS file not collected as browser runtime coverage
-(0%) tests/engine/ObjectModelContract.test.mjs - changed JS file not collected as browser runtime coverage
 (0%) tests/playwright/tools/ObjectsTool.spec.mjs - changed JS file not collected as browser runtime coverage
-(0%) tests/playwright/tools/RootToolsFutureState.spec.mjs - changed JS file not collected as browser runtime coverage
 (0%) tests/playwright/tools/ToolboxAdminMetadataSsot.spec.mjs - changed JS file not collected as browser runtime coverage
-(0%) tests/playwright/tools/ToolboxRoutePages.spec.mjs - changed JS file not collected as browser runtime coverage
-(67%) src/engine/object-model/objectModelRegistry.js - changed JS file with browser V8 coverage
-(94%) toolbox/objects/objects.js - changed JS file with browser V8 coverage
+(93%) toolbox/objects/objects.js - changed JS file with browser V8 coverage

docs_build/dev/reports/playwright_v8_coverage_report.txt

@@ -410,14 +410,14 @@ export const TOOL_REGISTRY = Object.freeze([
     "id": "objects",
     "name": "Objects",
     "displayName": "Objects",
-    "shortDescription": "Define game object setup with broad roles, body types, and render asset handoff.",
+    "shortDescription": "Create game objects with roles, state, and sprite assets.",
     "shortLabel": "Objects",
     "path": "objects",
     "folderName": "objects",
     "entryPoint": "objects/index.html",
     "badge": "/assets/theme-v2/images/badges/objects.png",
     "tool": "/assets/theme-v2/images/tools/objects.png",
-    "description": "Define game object setup with broad roles, body types, and render asset handoff.",
+    "description": "Create game objects with roles, state, and sprite assets.",
     "category": "Design",
     "colorGroup": "tool-group-design",
     "active": true,
@@ -429,16 +429,16 @@ export const TOOL_REGISTRY = Object.freeze([
     "status": "Wireframe",
     "releaseChannel": "wireframe",
     "progressChecklist": [
-      "Table input wireframe adapted",
+      "Editable object table available",
       "Inline row add, save, cancel, edit, and trash actions available",
-      "Sprite asset handoff preserved for saved Sprite rows"
+      "Sprite asset linking available for saved Sprite rows"
     ],
     "deferred": false,
     "hidden": false,
     "adminOnly": false,
     "visibleInToolsList": true,
     "toolboxGroup": "Create",
-    "capabilityLabel": "Object setup types",
+    "capabilityLabel": "Object role types",
     "childCapabilities": [
       "Static",
       "Dynamic",

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

@@ -5,6 +5,23 @@ import { workspaceV2CoverageReporter } from "../../helpers/workspaceV2CoverageRe
 
 const ROLE_OPTIONS = ["Collectible", "Custom", "Enemy", "Goal", "Hazard", "Hero", "Platform", "Projectile", "Spawner", "UI", "Wall"];
 const OLD_SAMPLE_PATH_PATTERN = new RegExp(["M" + "VP", "Padd" + "le", "Ba" + "ll"].join("|"), "i");
+const OLD_INTERNAL_COPY_PATTERN = new RegExp([
+  ["page", " session only"].join(""),
+  ["hand", "off"].join(""),
+  ["Setup ", "readiness"].join(""),
+  ["Session ", "\\+ Assets"].join(""),
+  ["Readiness ", "Checks"].join(""),
+  ["Runtime ", "Scope"].join(""),
+  ["authoring ", "hand", "off"].join(""),
+  ["Game Configuration is ", "not ready"].join(""),
+  ["Object ", "setup ", "table"].join(""),
+  ["Object ", "setup ", "rows"].join(""),
+].join("|"), "i");
+const LOW_VALUE_STATUS_CHECK_PATTERN = new RegExp([
+  ["Object ", "row"].join(""),
+  ["Object ", "names"].join(""),
+  ["Object ", "roles"].join(""),
+].join("|"), "i");
 
 test.beforeEach(async ({ page }) => {
   await installPlaywrightStorageIsolation(page, {
@@ -75,13 +92,29 @@ async function fillActiveRow(page, { name, renderType = "None", role, state = "A
   await page.locator("[data-objects-row-render-type]").selectOption(renderType);
 }
 
-test("Objects exposes broad table input without sample-path wording", async ({ page }) => {
+test("Objects exposes production copy, Object Status, and broad table input", async ({ page }) => {
   const failures = await openObjectsPage(page);
 
   try {
     await expect(page.locator(".tool-workspace")).toBeVisible();
     await expect(page.locator("style, [style], script:not([src])")).toHaveCount(0);
     await expect(page.locator("main")).not.toContainText(OLD_SAMPLE_PATH_PATTERN);
+    await expect(page.locator("main")).not.toContainText(OLD_INTERNAL_COPY_PATTERN);
+    await expect(page.getByRole("heading", { level: 2, name: "Object Builder" })).toBeVisible();
+    await expect(page.getByRole("heading", { level: 3, name: "Object Status" })).toBeVisible();
+    await expect(page.locator("[aria-label='Object status summary'] th")).toHaveText([
+      "Area",
+      "Status",
+      "Next Step",
+    ]);
+    await expect(page.locator("[data-objects-status-summary] tr")).toHaveCount(4);
+    await expect(page.locator("[data-objects-status-summary] td:first-child")).toHaveText([
+      "Ready Objects",
+      "Render Assets",
+      "Missing Hitboxes",
+      "Missing Events",
+    ]);
+    await expect(page.locator("[data-objects-status-summary]")).not.toContainText(LOW_VALUE_STATUS_CHECK_PATTERN);
     await expect(page.locator("[data-objects-list-table] th")).toHaveText([
       "Name",
       "Role",
@@ -94,9 +127,12 @@ test("Objects exposes broad table input without sample-path wording", async ({ p
     await expect(page.locator("[data-objects-row-type]")).toHaveCount(0);
     await expect(page.getByText("Object Types", { exact: true })).toHaveCount(0);
     await expect(page.locator("[data-objects-role-basics] li")).toHaveText(ROLE_OPTIONS);
-    await expect(page.locator("[data-objects-readiness]")).toHaveText("Needs Input");
-    await expect(page.locator("[data-objects-validation-overlay]")).toBeVisible();
-    await expect(page.locator("[data-objects-validation-list]")).toContainText("Object row: Add at least one object row.");
+    await expect(page.locator("[data-objects-readiness]")).toHaveText("Needs Objects");
+    await expect(page.locator("[data-objects-output-readiness]")).toHaveText("Needs Objects");
+    await expect(page.locator("[data-objects-asset-status]")).toHaveText("No sprite render selected");
+    await expect(page.locator("[data-objects-output-setup]")).toHaveText("Add objects to begin the object list.");
+    await expect(page.locator("[data-objects-validation-overlay]")).toBeHidden();
+    await expect(page.locator("[data-objects-validation-list]")).toHaveText("PASS: Object details are valid.");
     await expect(page.locator("[data-objects-trait-basics]")).toContainText("movable");
     await expect(page.locator("[data-objects-trait-basics]")).toContainText("playerControlled");
     await expect(page.locator("[data-objects-trait-basics]")).toContainText("collides");
@@ -109,11 +145,12 @@ test("Objects exposes broad table input without sample-path wording", async ({ p
 
     await page.getByRole("button", { name: "Seed Starter Objects" }).click();
     await expect(page.locator("[data-objects-log]")).toHaveText("Seeded starter objects: Hero, Projectile, and Wall.");
-    await expect(page.locator("[data-objects-readiness]")).toHaveText("Ready");
-    await expect(page.locator("[data-objects-output-readiness]")).toHaveText("Ready");
+    await expect(page.locator("[data-objects-readiness]")).toHaveText("Objects Ready");
+    await expect(page.locator("[data-objects-output-readiness]")).toHaveText("Objects Ready");
     await expect(page.locator("[data-objects-count]")).toHaveText("3");
-    await expect(page.locator("[data-objects-output-static]")).toHaveText("1");
-    await expect(page.locator("[data-objects-output-dynamic]")).toHaveText("2");
+    await expect(page.locator("[data-objects-output-count]")).toHaveText("3");
+    await expect(page.locator("[data-objects-status-summary]")).toContainText("3/3");
+    await expect(page.locator("[data-objects-status-summary]")).toContainText("No sprite render selected");
     await expect(page.locator("[data-objects-validation-overlay]")).toBeHidden();
     await expect(page.locator("[data-objects-list] tr")).toHaveCount(3);
     await expect(page.locator("[data-objects-list]")).toContainText("Hero");
@@ -123,11 +160,12 @@ test("Objects exposes broad table input without sample-path wording", async ({ p
     await expect(page.locator("[data-objects-list] [data-objects-trash-row]")).toHaveCount(3);
     await expect(page.locator("[data-objects-output-render-asset]")).toHaveText("None");
     await expect(page.locator("[data-objects-edit-sprite]")).toBeHidden();
-    await expect(page.locator("[data-objects-output-setup]")).toHaveText("Object setup table has the required row details.");
+    await expect(page.locator("[data-objects-output-setup]")).toHaveText("Objects have saved table details.");
 
     await page.getByRole("button", { name: "Validate Setup" }).click();
-    await expect(page.locator("[data-objects-log]")).toHaveText("Validation PASS: Object setup rows are ready for authoring handoff.");
-    await expect(page.locator("[data-objects-validation-list]")).toHaveText("PASS: Object setup rows are ready.");
+    await expect(page.locator("[data-objects-log]")).toHaveText("Validation PASS: Object details are ready for review.");
+    await expect(page.locator("[data-objects-validation-list]")).toHaveText("PASS: Object details are valid.");
+    await expect(page.locator("main")).not.toContainText(OLD_INTERNAL_COPY_PATTERN);
 
     await expectNoPageFailures(failures);
   } finally {
@@ -201,8 +239,9 @@ test("Objects table save preserves linked sprite asset create and resolve behavi
 
     await expect(page.locator("[data-objects-log]")).toContainText("Added Bolt.");
     await expect(page.locator("[data-objects-log]")).toContainText("Created editable default sprite asset sprite_bolt for Bolt.");
-    await expect(page.locator("[data-objects-log]")).toContainText("Game Configuration is not ready");
     await expect(page.locator("[data-objects-list]")).toContainText("sprite_bolt");
+    await expect(page.locator("[data-objects-asset-status]")).toHaveText("Linked");
+    await expect(page.locator("[data-objects-status-summary]")).toContainText("Linked");
     await expect(page.locator("[data-objects-output-render-asset]")).toHaveText("sprite_bolt");
     await expect(page.locator("[data-objects-output-sprite-preview]")).toContainText("sprite_bolt");
     await expect(page.locator("[data-objects-output-sprite-preview]")).toContainText("projects/");
@@ -216,8 +255,9 @@ test("Objects table save preserves linked sprite asset create and resolve behavi
     await expect(page.locator("[data-objects-row-render-asset-preview]")).toHaveText("sprite_bolt");
     await page.locator("[data-objects-save-row]").click();
     await expect(page.locator("[data-objects-log]")).toHaveText("Saved Bolt.");
-    await expect(page.locator("[data-objects-log]")).not.toContainText("Sprite asset handoff blocked");
+    await expect(page.locator("[data-objects-log]")).not.toContainText("Sprite asset link blocked");
     await expect(page.locator("[data-objects-list]")).toContainText("sprite_bolt");
+    await expect(page.locator("main")).not.toContainText(OLD_INTERNAL_COPY_PATTERN);
 
     await page.getByRole("button", { name: "Reset Table" }).click();
     await page.getByRole("button", { name: "Add Object" }).click();

tests/playwright/tools/ObjectsTool.spec.mjs

@@ -200,7 +200,7 @@ test("Toolbox and Admin Tool Votes share the same 43-tool DB-backed metadata and
     expect(Array.isArray(registryColors.progressChecklist)).toBe(true);
     expect(Array.isArray(registryColors.requires)).toBe(true);
     expect(registrySnapshot.activeTools.find((tool) => tool.id === "objects")).toEqual(expect.objectContaining({
-      capabilityLabel: "Object setup types",
+      capabilityLabel: "Object role types",
       childCapabilities: ["Static", "Dynamic", "Collectible", "Hazard", "Goal"],
       releaseChannel: "wireframe",
       status: "wireframe",