PR_26171_040 clean idea board production copy

Author: ToolboxAid

docs_build/dev/reports/codex_changed_files.txt

@@ -1,57 +1,58 @@
-# Codex Changed Files - PR_26171_039-table-first-workspace-governance
+# Codex Changed Files - PR_26171_040-idea-board-production-copy-cleanup
 
 ## Git Workflow
-- Current branch: `codex/pr-26171-039-table-first-workspace-governance`.
+- Current branch: `codex/pr-26171-040-idea-board-production-copy-cleanup`.
 - Expected starting branch: `main` (PASS before branch creation).
-- Created branch: `codex/pr-26171-039-table-first-workspace-governance`.
+- Created branch: `codex/pr-26171-040-idea-board-production-copy-cleanup`.
 - Push result: pending until after commit.
 - PR URL: pending until after push.
 - Merge result: pending until after PR validation/merge.
 - Final main commit: pending until after merge and final pull.
 
 ## Changed Files
-- docs_build/dev/PROJECT_INSTRUCTIONS.md
 - 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/playwright_v8_coverage_report.txt
+- src/dev-runtime/server/local-api-router.mjs
+- src/shared/toolbox/tool-metadata-inventory.js
+- tests/playwright/tools/IdeaBoardTableNotes.spec.mjs
+- tests/playwright/tools/ToolboxRoutePages.spec.mjs
+- toolbox/idea-board/index.html
+- toolbox/idea-board/index.js
 
 ## Requirement Evidence
-- PASS: Added `TABLE-FIRST WORKSPACE GOVERNANCE` section to `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
-- PASS: Section states primary work surface is a table.
-- PASS: Section says to prefer inline table workflows over forms.
-- PASS: Section prohibits detached selected-item context panels.
-- PASS: Section prohibits detached notes/detail panels when child rows can render under the parent row.
-- PASS: Parent/child patterns include Idea 1-* Notes, Character 1-* Dialog, Quest 1-* Objectives, and Audio Profile 1-* Effects.
-- PASS: Section says parent name cell owns accordion behavior.
-- PASS: Section says chevron appears inside parent name cell.
-- PASS: Section says entire parent name cell toggles expansion.
-- PASS: Section says child count columns are informational only.
-- PASS: Section says only one parent row may be expanded at a time.
-- PASS: Section says default state is all collapsed.
-- PASS: Section says child rows are indented.
-- PASS: Section says child Add button aligns with child rows.
-- PASS: Section says metadata remains stored but hidden unless explicitly needed.
-- PASS: Section says Add buttons are left aligned and self-describing.
-- PASS: Section says enumerated fields become dropdowns during edit mode.
-- PASS: Visual examples include default collapsed table, expanded parent with indented child rows, normal Edit/Delete row, edit Save/Cancel row with dropdown status, and add Save/Cancel row.
-- PASS: Explicit row-state rules include Normal row actions: Edit Delete.
-- PASS: Explicit row-state rules include Edit row actions: Save Cancel.
-- PASS: Explicit row-state rules include Add row actions: Save Cancel.
-- PASS: Explicit row-state rules state Save/Cancel is row-level only.
-- PASS: Explicit row-state rules prohibit page-level Save for normal row editing.
-- PASS: Explicit row-state rules state Cancel restores previous row state.
-- PASS: Explicit row-state rules state Save commits only the edited row.
-- PASS: Explicit row-state rules prohibit multiple simultaneous row edits unless explicitly approved.
-- PASS: Runtime code was not changed.
-- PASS: Tests were not changed.
-- PASS: Roadmap status was not changed.
+- PASS: Creator-visible Idea Board page copy no longer uses DB-shaped, in-page data model, ID/flag/metadata/seed/debug/mock/test/internal wording.
+- PASS: Shared Toolbox metadata for Idea Board now uses simple creator-facing language for short description, description, and progress checklist text.
+- PASS: Existing persisted Toolbox metadata rows now source-sync Idea Board copy so the browser-visible registry summary updates from source-controlled text.
+- PASS: Idea Board table workflow, Add Idea, Add Note, Edit/Delete, Save/Cancel, Status dropdown, accordion behavior, notes under idea rows, and existing in-page data structure were preserved.
+- PASS: Targeted Playwright production-copy checks verify the Idea Board main surface does not expose the prohibited creator-visible wording.
+- PASS: Copied/adapted files verified as not applicable for this copy-only PR; no Tool Template V2 file copy was required.
 
 ## Validation
+- PASS: `node --check toolbox/idea-board/index.js`.
+- PASS: `node --check src/dev-runtime/server/local-api-router.mjs`.
+- PASS: `node --check src/shared/toolbox/tool-metadata-inventory.js`.
+- PASS: `node --check tests/playwright/tools/IdeaBoardTableNotes.spec.mjs`.
+- PASS: `node --check tests/playwright/tools/ToolboxRoutePages.spec.mjs`.
+- PASS: `npx playwright test tests/playwright/tools/IdeaBoardTableNotes.spec.mjs --project=playwright --workers=1 --reporter=line`.
+- PASS: `npx playwright test tests/playwright/tools/ToolboxRoutePages.spec.mjs --project=playwright --workers=1 --reporter=line -g "Idea Board launches"`.
 - PASS: `git diff --check`.
-- PASS: Heading confirmation found `## TABLE-FIRST WORKSPACE GOVERNANCE`.
-- PASS: Visual examples confirmation found `Visual examples:` and numbered examples.
-- PASS: Save/Cancel row-state confirmation found edit/add examples and explicit row-state rules.
-- PASS: No Playwright impact; this PR is docs/workflow only.
+- PASS: Playwright V8 coverage report produced because runtime JavaScript changed.
+- WARN: Coverage report marks `src/dev-runtime/server/local-api-router.mjs` and `src/shared/toolbox/tool-metadata-inventory.js` as not collected by browser V8 coverage; advisory only per project instructions.
+- SKIPPED: Full samples smoke was not run per request.
 
 ## ZIP
-- Path: `tmp/PR_26171_039-table-first-workspace-governance_delta.zip`.
-- Size and contents: reported after final ZIP generation.
+- Path: `tmp/PR_26171_040-idea-board-production-copy-cleanup_delta.zip`.
+- Size: `86984` bytes before final report refresh; final size reported in the delivery summary.
+- Contents:
+  - 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/playwright_v8_coverage_report.txt
+  - src/dev-runtime/server/local-api-router.mjs
+  - src/shared/toolbox/tool-metadata-inventory.js
+  - tests/playwright/tools/IdeaBoardTableNotes.spec.mjs
+  - tests/playwright/tools/ToolboxRoutePages.spec.mjs
+  - toolbox/idea-board/index.html
+  - toolbox/idea-board/index.js

docs_build/dev/reports/codex_review.diff

@@ -1,182 +1,21 @@
-diff --git a/docs_build/dev/PROJECT_INSTRUCTIONS.md b/docs_build/dev/PROJECT_INSTRUCTIONS.md
-index 8ec5d3681..6a2014f2f 100644
---- a/docs_build/dev/PROJECT_INSTRUCTIONS.md
-+++ b/docs_build/dev/PROJECT_INSTRUCTIONS.md
-@@ -1660,6 +1660,81 @@ For tools that present Summary Table or Notes Table note-management surfaces:
- - System-created notes may only have status changed when the tool explicitly allows status editing.
- - Selected Note Metadata must be shown as table columns, not as a separate detached panel.
-
-+## TABLE-FIRST WORKSPACE GOVERNANCE
-+
-+Primary tool work surfaces must be table-first unless a PR explicitly approves another interaction model.
-+
-+Rules:
-+- The primary work surface is a table.
-+- Prefer inline table workflows over forms.
-+- No detached selected-item context panels.
-+- No detached notes/detail panels when child rows can render under the parent row.
-+- Parent name cells own accordion behavior.
-+- The chevron appears inside the parent name cell.
-+- The entire parent name cell toggles expansion.
-+- Child count columns are informational only.
-+- Only one parent row may be expanded at a time.
-+- The default state is all collapsed.
-+- Child rows are indented.
-+- The child Add button aligns with child rows.
-+- Metadata remains stored but hidden unless explicitly needed.
-+- Add buttons are left aligned and self-describing.
-+- Enumerated fields become dropdowns during edit mode.
-+
-+Parent/child table pattern examples:
-+- Idea 1-* Notes
-+- Character 1-* Dialog
-+- Quest 1-* Objectives
-+- Audio Profile 1-* Effects
-+
-+Visual examples:
-+
-+1. Default collapsed table.
-+
-+| Idea | Pitch | Status | Updated | Notes | Actions |
-+| --- | --- | --- | --- | --- | --- |
-+| > Sky Orchard | Grow floating islands... | Exploring | 2026-06-20 | 3 Notes | Edit Delete |
-+| > Clockwork Courier | Deliver messages through looping city... | New | 2026-06-20 | 0 Notes | Edit Delete |
-+| Add Idea | | | | | |
-+
-+2. Expanded parent with child rows indented.
-+
-+| Idea | Pitch | Status | Updated | Notes | Actions |
-+| --- | --- | --- | --- | --- | --- |
-+| v Sky Orchard | Grow floating islands... | Exploring | 2026-06-20 | 3 Notes | Edit Delete |
-+|   Note | Actions | | | | |
-+|   Wind test needs traversal risks. | Edit Delete | | | | |
-+|   Island height should read from camera. | Edit Delete | | | | |
-+|   Add Note | | | | | |
-+
-+3. Normal row with Edit/Delete.
-+
-+| Idea | Pitch | Status | Updated | Notes | Actions |
-+| --- | --- | --- | --- | --- | --- |
-+| > Sky Orchard | Grow floating islands... | Exploring | 2026-06-20 | 3 Notes | Edit Delete |
-+
-+4. Edit row with Save/Cancel and dropdown status.
-+
-+| Idea | Pitch | Status | Updated | Notes | Actions |
-+| --- | --- | --- | --- | --- | --- |
-+| [Sky Orchard] | [Grow floating islands...] | [Exploring v] | 2026-06-20 | 3 Notes | Save Cancel |
-+
-+5. Add row with Save/Cancel.
-+
-+| Idea | Pitch | Status | Updated | Notes | Actions |
-+| --- | --- | --- | --- | --- | --- |
-+| [New idea] | [Pitch] | [New v] | auto | 0 Notes | Save Cancel |
-+
-+Explicit row-state rules:
-+- Normal row actions: Edit Delete.
-+- Edit row actions: Save Cancel.
-+- Add row actions: Save Cancel.
-+- Save/Cancel is row-level only.
-+- No page-level Save for normal row editing.
-+- Cancel restores previous row state.
-+- Save commits only the edited row.
-+- Multiple simultaneous row edits are prohibited unless explicitly approved.
-+
- ## INPUT RESOLUTION RULES
-
- - Discover real files and directories.
-diff --git a/docs_build/dev/reports/codex_changed_files.txt b/docs_build/dev/reports/codex_changed_files.txt
-index ac3c628b0..4a954a997 100644
+diff --git a/docs_build/dev/reports/codex_changed_files.txt b/docs_build/dev/reports/codex_changed_files.txt
+index 3c2821db9..105ba5ca3 100644
 --- a/docs_build/dev/reports/codex_changed_files.txt
 +++ b/docs_build/dev/reports/codex_changed_files.txt
-@@ -1,52 +1,57 @@
--# Codex Changed Files - PR_26171_033-idea-board-inline-accordion-cell
-+# Codex Changed Files - PR_26171_039-table-first-workspace-governance
-
- ## Git Workflow
--- Current branch: `codex/pr-26171-033-idea-board-inline-accordion-cell`.
-+- Current branch: `codex/pr-26171-039-table-first-workspace-governance`.
- - Expected starting branch: `main` (PASS before branch creation).
--- Created branch: `codex/pr-26171-033-idea-board-inline-accordion-cell`.
-+- Created branch: `codex/pr-26171-039-table-first-workspace-governance`.
- - Push result: pending until after commit.
- - PR URL: pending until after push.
- - Merge result: pending until after PR validation/merge.
- - Final main commit: pending until after merge and final pull.
-
- ## Changed Files
--- assets/theme-v2/images/gfs-chevron-down.svg
--- assets/theme-v2/images/gfs-chevron-up.svg
--- docs_build/dev/codex_commands.md
--- docs_build/dev/commit_comment.txt
-+- docs_build/dev/PROJECT_INSTRUCTIONS.md
- - 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/playwright_v8_coverage_report.txt
--- docs_build/pr/APPLY_PR_26171_033-idea-board-inline-accordion-cell.md
--- docs_build/pr/BUILD_PR_26171_033-idea-board-inline-accordion-cell.md
--- docs_build/pr/PLAN_PR_26171_033-idea-board-inline-accordion-cell.md
--- tests/playwright/tools/IdeaBoardTableNotes.spec.mjs
--- tests/playwright/tools/ToolboxRoutePages.spec.mjs
--- toolbox/idea-board/index.js
-
- ## Requirement Evidence
--- PASS: Separate `>` expand/collapse button removed from `toolbox/idea-board/index.js`; no `data-idea-board-notes-chevron` control remains.
--- PASS: No separate expand/collapse control column was added; the visible Idea table remains Idea, Pitch, Status, Updated, Notes, Actions.
--- PASS: Notes count is rendered as a plain informational span and no longer carries `data-idea-board-toggle-notes`.
--- PASS: Chevron image is rendered inside the Idea row header cell.
--- PASS: Collapsed rows use `assets/theme-v2/images/gfs-chevron-down.svg`.
--- PASS: Expanded rows use `assets/theme-v2/images/gfs-chevron-up.svg`.
--- PASS: The whole Idea cell carries the toggle target and handles click plus Enter/Space keyboard activation.
--- PASS: Expanded notes continue to render directly under the owning idea row.
--- PASS: Targeted Playwright verifies clicking Notes count does not expand from a collapsed state.
--- PASS: Targeted Playwright verifies clicking Notes count does not collapse from an expanded state.
--- PASS: Targeted Playwright verifies clicking the Idea cell expands/collapses and the chevron switches down/up.
--- PASS: Theme V2 chevron SVG assets were added under `assets/theme-v2/images/`.
-+- PASS: Added `TABLE-FIRST WORKSPACE GOVERNANCE` section to `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
-+- PASS: Section states primary work surface is a table.
-+- PASS: Section says to prefer inline table workflows over forms.
-+- PASS: Section prohibits detached selected-item context panels.
-+- PASS: Section prohibits detached notes/detail panels when child rows can render under the parent row.
-+- PASS: Parent/child patterns include Idea 1-* Notes, Character 1-* Dialog, Quest 1-* Objectives, and Audio Profile 1-* Effects.
-+- PASS: Section says parent name cell owns accordion behavior.
-+- PASS: Section says chevron appears inside parent name cell.
-+- PASS: Section says entire parent name cell toggles expansion.
-+- PASS: Section says child count columns are informational only.
-+- PASS: Section says only one parent row may be expanded at a time.
-+- PASS: Section says default state is all collapsed.
-+- PASS: Section says child rows are indented.
-+- PASS: Section says child Add button aligns with child rows.
-+- PASS: Section says metadata remains stored but hidden unless explicitly needed.
-+- PASS: Section says Add buttons are left aligned and self-describing.
-+- PASS: Section says enumerated fields become dropdowns during edit mode.
-+- PASS: Visual examples include default collapsed table, expanded parent with indented child rows, normal Edit/Delete row, edit Save/Cancel row with dropdown status, and add Save/Cancel row.
-+- PASS: Explicit row-state rules include Normal row actions: Edit Delete.
-+- PASS: Explicit row-state rules include Edit row actions: Save Cancel.
-+- PASS: Explicit row-state rules include Add row actions: Save Cancel.
-+- PASS: Explicit row-state rules state Save/Cancel is row-level only.
-+- PASS: Explicit row-state rules prohibit page-level Save for normal row editing.
-+- PASS: Explicit row-state rules state Cancel restores previous row state.
-+- PASS: Explicit row-state rules state Save commits only the edited row.
-+- PASS: Explicit row-state rules prohibit multiple simultaneous row edits unless explicitly approved.
-+- PASS: Runtime code was not changed.
-+- PASS: Tests were not changed.
-+- PASS: Roadmap status was not changed.
-
- ## Validation
--- PASS: `node --check toolbox/idea-board/index.js`.
--- PASS: `node --check tests/playwright/tools/IdeaBoardTableNotes.spec.mjs`.
--- PASS: `node --check tests/playwright/tools/ToolboxRoutePages.spec.mjs`.
--- PASS: `npx playwright test tests/playwright/tools/IdeaBoardTableNotes.spec.mjs --project=playwright --workers=1 --reporter=line` (1 passed).
--- PASS: `npx playwright test tests/playwright/tools/ToolboxRoutePages.spec.mjs --project=playwright --workers=1 --reporter=line -g "Idea Board launches"` (1 passed).
--- SKIP: Full samples smoke was not run; it is not in scope for this PR.
-+- PASS: `git diff --check`.
-+- PASS: Heading confirmation found `## TABLE-FIRST WORKSPACE GOVERNANCE`.
-+- PASS: Visual examples confirmation found `Visual examples:` and numbered examples.
-+- PASS: Save/Cancel row-state confirmation found edit/add examples and explicit row-state rules.
-+- PASS: No Playwright impact; this PR is docs/workflow only.
+@@ -44,4 +44,15 @@
 
  ## ZIP
--- Path: `tmp/PR_26171_033-idea-board-inline-accordion-cell_delta.zip`.
-+- Path: `tmp/PR_26171_039-table-first-workspace-governance_delta.zip`.
- - Size and contents: reported after final ZIP generation.
+ - Path: `tmp/PR_26171_040-idea-board-production-copy-cleanup_delta.zip`.
+-- Size and contents: reported after final ZIP generation.
++- Size: `86984` bytes before final report refresh; final size reported in the delivery summary.
++- Contents:
++  - 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/playwright_v8_coverage_report.txt
++  - src/dev-runtime/server/local-api-router.mjs
++  - src/shared/toolbox/tool-metadata-inventory.js
++  - tests/playwright/tools/IdeaBoardTableNotes.spec.mjs
++  - tests/playwright/tools/ToolboxRoutePages.spec.mjs
++  - toolbox/idea-board/index.html
++  - toolbox/idea-board/index.js

docs_build/dev/reports/coverage_changed_js_guardrail.txt

@@ -6,7 +6,10 @@ 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:
-(70%) toolbox/idea-board/index.js - executed lines 481/481; executed functions 23/33
+(0%) src/dev-runtime/server/local-api-router.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
+(0%) src/shared/toolbox/tool-metadata-inventory.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
+(70%) toolbox/idea-board/index.js - executed lines 480/480; executed functions 23/33
 
 Guardrail warnings:
-(100%) none - no changed runtime JS coverage warnings
+(0%) src/dev-runtime/server/local-api-router.mjs - WARNING: changed runtime JS file missing from coverage; advisory only
+(0%) src/shared/toolbox/tool-metadata-inventory.js - WARNING: changed runtime JS file missing from coverage; advisory only