Compact Object Type Catalog to template and capability display - PR_26161_009-object-catalog-compact-display

Author: ToolboxAid

docs_build/dev/codex_commands.md

@@ -1294,3 +1294,32 @@ Required reports:
 - `docs_build/dev/reports/coverage_changed_js_guardrail.txt`
 - `docs_build/dev/reports/codex_changed_files.txt`
 - `docs_build/dev/reports/codex_review.diff`
+
+
+## PR_26161_009-object-catalog-compact-display
+
+Changes:
+- Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
+- Verified the current branch is `main` before edits.
+- Continued from the latest Objects PR state.
+- Updated the Object Type Catalog display to show only `Template` and `Capability`.
+- Removed `State` and `Render` columns from the Object Type Catalog display only.
+- Preserved State, Render, State Flow, object registry/config concepts, object validation contracts, render asset linking, and Sprite Editor linking behavior.
+- Preserved table-first input, `Add Object` below the table, disabled Add while adding, Cancel, Edit, Trash, Reset Table, Object Status, and Sprite asset linking.
+- Did not change engine runtime behavior, sample JSON, auth behavior, production DB behavior, or unrelated tool behavior.
+
+Validation:
+- `node --check toolbox/objects/objects.js`
+- `node --check tests/playwright/tools/ObjectsTool.spec.mjs`
+- HTML restriction check for `toolbox/objects/index.html`
+- Objects forbidden wording scan for requested terms
+- `npx playwright test tests/playwright/tools/ObjectsTool.spec.mjs --workers=1 --reporter=line`
+- `git diff --check`
+- Full samples validation skipped by request and because samples/runtime behavior were not changed.
+
+Required reports:
+- `docs_build/dev/reports/object-catalog-compact-display-report.md`
+- `docs_build/dev/reports/playwright_v8_coverage_report.txt`
+- `docs_build/dev/reports/coverage_changed_js_guardrail.txt`
+- `docs_build/dev/reports/codex_changed_files.txt`
+- `docs_build/dev/reports/codex_review.diff`

docs_build/dev/commit_comment.txt

@@ -1 +1 @@
-Add Objects status action links - PR_26161_007-objects-status-action-links
+Compact Objects type catalog display - PR_26161_009-object-catalog-compact-display

docs_build/dev/reports/codex_changed_files.txt

@@ -3,7 +3,7 @@ docs_build/dev/commit_comment.txt
 docs_build/dev/reports/codex_changed_files.txt
 docs_build/dev/reports/codex_review.diff
 docs_build/dev/reports/coverage_changed_js_guardrail.txt
-docs_build/dev/reports/objects-status-action-links-report.md
+docs_build/dev/reports/object-catalog-compact-display-report.md
 docs_build/dev/reports/playwright_v8_coverage_report.txt
 tests/playwright/tools/ObjectsTool.spec.mjs
 toolbox/objects/index.html

docs_build/dev/reports/codex_review.diff

@@ -6,7 +6,7 @@ Missing changed runtime JS files are WARN, not FAIL.
 Source: Playwright/Chromium built-in V8 coverage from the active Playwright run.
 
 Changed runtime JS files considered:
-(95%) toolbox/objects/objects.js - executed lines 987/987; executed functions 94/99
+(95%) toolbox/objects/objects.js - executed lines 985/985; executed functions 94/99
 
 Guardrail warnings:
 (100%) none - no changed runtime JS coverage warnings

docs_build/dev/reports/coverage_changed_js_guardrail.txt

@@ -0,0 +1,70 @@
+# PR_26161_009 Object Catalog Compact Display Report
+
+## Branch Guard
+- Current branch: `main`
+- Expected branch: `main`
+- Local branches found: `* main`
+- Branch validation: PASS
+
+## Source Of Truth
+- PASS: Read `docs_build/dev/PROJECT_INSTRUCTIONS.md` before branch validation and edits.
+- PASS: Used the user-provided `PR_26161_009-object-catalog-compact-display` request as the active BUILD source.
+- PASS: Confirmed the PR purpose is singular: compact the Object Type Catalog display.
+
+## Requirement Checklist
+- PASS: Object Type Catalog now displays only `Template` and `Capability`.
+- PASS: Removed `State` and `Render` columns from the Object Type Catalog display only.
+- PASS: Kept State, Render, State Flow, registry/config concepts, and object validation contracts in the underlying model.
+- PASS: Kept Render available in the Objects table/editor where object render configuration belongs.
+- PASS: Kept table-first input, `Add Object` below the table, Add disabled while adding, Cancel, Edit, Trash, Reset Table, Object Status, and Sprite asset linking.
+- PASS: Preserved render asset linking and Sprite Editor linking behavior.
+- PASS: Did not add sample JSON alignment, auth behavior, production DB behavior, or unrelated tool rewrites.
+- PASS: Kept Theme V2 only and preserved HTML restrictions.
+
+## Implementation Evidence
+- `toolbox/objects/index.html`: compacted the Object Type Catalog table headers to `Template` and `Capability`.
+- `toolbox/objects/objects.js`: compacted Object Type Catalog row rendering to type plus capability labels only.
+- `tests/playwright/tools/ObjectsTool.spec.mjs`: verifies the compact catalog headers, absence of catalog State/Render columns, Render availability in the Objects table/editor, table input behavior, and Sprite asset linking.
+
+## Impacted Lane
+- Impacted lane: Objects tool UI/runtime.
+- Playwright impacted: Yes.
+- Changed runtime JavaScript: `toolbox/objects/objects.js`.
+
+## Testing Performed
+- PASS: `node --check toolbox/objects/objects.js`
+- PASS: `node --check tests/playwright/tools/ObjectsTool.spec.mjs`
+- PASS: HTML restriction check for `toolbox/objects/index.html`
+- PASS: Objects forbidden wording scan for requested terms.
+- PASS: `npx playwright test tests/playwright/tools/ObjectsTool.spec.mjs --workers=1 --reporter=line` (5 passed)
+- PASS: `docs_build/dev/reports/playwright_v8_coverage_report.txt` was produced by targeted Objects Playwright.
+- PASS: `docs_build/dev/reports/coverage_changed_js_guardrail.txt` reports no changed runtime JS coverage warnings.
+- PASS: `git diff --check` exited 0 with line-ending warnings only.
+
+## Playwright Behavior Coverage
+- PASS: Object Type Catalog shows `Template` and `Capability` only.
+- PASS: `State` and `Render` no longer appear in the Object Type Catalog headers.
+- PASS: `Render` still appears in the Objects table where object render configuration belongs.
+- PASS: Object Type Catalog selection still prefills Type, State, Render, and Capabilities in the active table row.
+- PASS: Table input and Sprite asset linking still work.
+
+## Skipped Lanes
+- Full samples validation: SKIP. Safe because no sample JSON, sample launch path, or sample runtime behavior changed.
+- Engine runtime validation beyond changed-file syntax/import coverage: SKIP. Safe because no engine runtime files or behavior changed.
+- Production DB/auth validation: SKIP. Safe because no production DB or auth behavior changed.
+- Broad Toolbox/Admin validation: SKIP. Safe because no shared navigation, registration, metadata, or admin surface changed.
+
+## Manual Validation Steps
+- Open `/toolbox/objects/index.html`.
+- Confirm Object Type Catalog table headers show only `Template` and `Capability`.
+- Confirm the Object Type Catalog does not display `State` or `Render` columns.
+- Confirm the Objects Table still shows `State` and `Render` columns.
+- Select a catalog template, add an object row, and confirm Type, State, Render, and Capabilities still prefill in the Objects table/editor.
+- Save a Sprite object and confirm the Sprite asset is created/resolved and the Sprite Editor link remains available.
+
+## Required Artifacts
+- PASS: `docs_build/dev/reports/object-catalog-compact-display-report.md`
+- PASS: `docs_build/dev/reports/codex_review.diff`
+- PASS: `docs_build/dev/reports/codex_changed_files.txt`
+- PASS: `docs_build/dev/reports/playwright_v8_coverage_report.txt`
+- PASS: `tmp/PR_26161_009-object-catalog-compact-display_delta.zip`

docs_build/dev/reports/object-catalog-compact-display-report.md

@@ -17,7 +17,7 @@ Exercised tool entry points detected:
 (59%) Theme V2 Shared JS - exercised 2 runtime JS files
 
 Changed runtime JS files covered:
-(95%) toolbox/objects/objects.js - executed lines 987/987; executed functions 94/99
+(95%) toolbox/objects/objects.js - executed lines 985/985; executed functions 94/99
 
 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
@@ -31,7 +31,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
-(95%) toolbox/objects/objects.js - executed lines 987/987; executed functions 94/99
+(95%) toolbox/objects/objects.js - executed lines 985/985; executed functions 94/99
 (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
 

docs_build/dev/reports/playwright_v8_coverage_report.txt

@@ -119,11 +119,15 @@ test("Objects exposes production copy, setup status, and broad table input", asy
     await expect(page.getByRole("heading", { level: 2, name: "Object Builder" })).toBeVisible();
     await expect(page.getByText("Object Type Catalog", { exact: true })).toBeVisible();
     await expect(page.locator("[data-objects-template-select] option")).toHaveText(["Select template", ...TYPE_OPTIONS]);
+    await expect(page.locator("[aria-label='Object type catalog'] th")).toHaveText(["Template", "Capability"]);
+    await expect(page.locator("[aria-label='Object type catalog'] thead")).not.toContainText("State");
+    await expect(page.locator("[aria-label='Object type catalog'] thead")).not.toContainText("Render");
+    await expect(page.locator("[data-objects-template-catalog] tr").first().locator("td")).toHaveCount(2);
     await expect(page.locator("[data-objects-template-catalog] tr")).toHaveCount(TYPE_OPTIONS.length);
     await expect(page.locator("[data-objects-template-catalog] td:first-child")).toHaveText(TYPE_OPTIONS);
     await expect(page.locator("[data-objects-template-catalog]")).toContainText("Spawn Point");
     await expect(page.locator("[data-objects-template-catalog]")).toContainText("Decoration");
-    await expect(page.locator("[data-objects-template-catalog]")).toContainText("Sprite");
+    await expect(page.locator("[data-objects-template-catalog]")).not.toContainText("Sprite");
     await expect(page.locator("[data-objects-template-catalog]")).toContainText("Can Move");
     await expect(page.locator("[data-objects-template-catalog]")).toContainText("Scores Points");
     await expect(page.getByRole("heading", { level: 3, name: "Object Status" })).toBeVisible();

tests/playwright/tools/ObjectsTool.spec.mjs

@@ -50,9 +50,7 @@ <h2>Objects</h2>
                                             <thead>
                                                 <tr>
                                                     <th>Template</th>
-                                                    <th>State</th>
-                                                    <th>Render</th>
-                                                    <th>Capabilities</th>
+                                                    <th>Capability</th>
                                                 </tr>
                                             </thead>
                                             <tbody data-objects-template-catalog></tbody>

toolbox/objects/index.html

@@ -826,8 +826,6 @@ function renderTemplateCatalog() {
       const row = document.createElement("tr");
       row.append(
         tableCell(template.type),
-        tableCell(template.state),
-        tableCell(template.renderType),
         tableCell(capabilityText(template.capabilities))
       );
       elements.templateCatalog.append(row);