Polish Project Workspace actions and prepare Game Design rebuild - PR_26155_058-060-project-workspace-polish

Author: ToolboxAid

docs_build/dev/reports/project-workspace-button-style.md

@@ -0,0 +1,38 @@
+# Project Workspace Button Style
+
+Stacked PR:
+- PR_26155_058-project-workspace-button-style
+
+## Summary
+
+Updated the Project Workspace `Project Setup` action buttons so `Create Project` and `Delete Open Project` use the same existing Theme V2 button style.
+
+Changed:
+- `Create Project`: `class="btn primary"` to `class="btn"`
+- `Delete Open Project`: unchanged, already `class="btn"`
+
+No create/delete behavior changed.
+
+## Validation
+
+Impacted lane:
+- `project-workspace`
+
+Skipped lanes:
+- `workspace-contract` / `npm run test:workspace-v2` skipped because no shared launch, navigation, or layout wiring changed.
+- Full suite skipped because this bundle changed one Project Workspace page, its narrow script copy, and targeted Project Workspace assertions only.
+
+Commands run:
+- `node --check toolbox/project-workspace/project-workspace.js`
+- `node --check tests/playwright/tools/ProjectWorkspaceMockRepository.spec.mjs`
+- `npm run test:lane:project-workspace`
+- `git diff --check`
+
+Results:
+- Targeted Project Workspace lane passed: 6 tests.
+- `git diff --check` passed.
+
+Manual notes:
+- Verified `Create Project` and `Delete Open Project` both use `btn`.
+- Verified create/open/delete behavior through the targeted Playwright lane.
+- Verified no inline styles or new CSS were introduced.

docs_build/dev/reports/project-workspace-layout-cleanup.md

@@ -0,0 +1,54 @@
+# Project Workspace Layout Cleanup
+
+Stacked PR:
+- PR_26155_059-project-workspace-layout-cleanup
+
+## Summary
+
+Reviewed Project Workspace after the wide layout rollout and kept the existing wide left/center/right tool shell intact.
+
+No layout CSS was added or modified.
+
+The center panel remains dominant through the existing approved Theme V2 wide layout classes:
+- `container--tool-wide`
+- `tool-workspace--wide`
+
+## User-Facing Copy Cleanup
+
+Reduced prominent implementation wording in user-facing areas:
+- Replaced the meta description with plain Project Workspace purpose text.
+- Replaced the page lede with project action/readiness copy.
+- Replaced the overview paragraph with local project list copy.
+- Renamed the project list aria label from `Mock projects` to `Projects`.
+- Removed the left-column `Mock Repository Contract` accordion from the setup/workflow area.
+- Replaced prominent `Mock` status/kicker/caption wording with plain project/readiness wording.
+- Kept technical table/status information in the center output/report area, including `Repository Tables`.
+
+Kept technical mock repository terminology in implementation/test-facing names where it is still accurate:
+- `project-workspace-mock-repository.js`
+- Project Workspace test lane/spec names
+
+## Validation
+
+Impacted lane:
+- `project-workspace`
+
+Skipped lanes:
+- `workspace-contract` / `npm run test:workspace-v2` skipped because shared launch/layout wiring did not change.
+- Full samples smoke test skipped because no games/samples changed.
+
+Commands run:
+- `node --check toolbox/project-workspace/project-workspace.js`
+- `node --check tests/playwright/tools/ProjectWorkspaceMockRepository.spec.mjs`
+- `npm run test:lane:project-workspace`
+- `git diff --check`
+
+Results:
+- Targeted Project Workspace lane passed: 6 tests.
+- `git diff --check` passed.
+
+Manual notes:
+- Project Workspace still loads without console errors.
+- Create, open, and delete behavior still works.
+- Technical project table details remain available in the output/report area.
+- No new CSS, page-local CSS, inline styles, or tool-local CSS were added.

docs_build/dev/reports/project-workspace-ready-for-game-design.md

@@ -0,0 +1,50 @@
+# Project Workspace Ready For Game Design
+
+Stacked PR:
+- PR_26155_060-project-workspace-ready-for-game-design
+
+## Ready Gate
+
+Project Workspace remains the active root for the next Toolbox rebuild sequence.
+
+Current Project Workspace state:
+- Active route: `toolbox/project-workspace/index.html`
+- Uses the approved Theme V2 tool shell.
+- Supports create/open/delete against the mock SQL-shaped repository.
+- Shows project status, project progress, publishing progress, current focus, and recommended next tool.
+- Keeps admin-oriented Project Data controls outside this page in the Toolbox role banner flow.
+- Keeps real DB/auth/cloud/persistence out of scope.
+
+## Next Rebuild
+
+The next tool rebuild should be:
+
+- Game Design
+
+Game Design should build on Project Workspace state and treat Project Workspace as the source for:
+- active project identity
+- project status
+- project progress
+- publishing progress
+- recommended next tool routing
+
+This PR bundle does not start Game Design implementation.
+
+## Validation Notes
+
+Impacted lane:
+- `project-workspace`
+
+Skipped lanes:
+- `workspace-contract` / `npm run test:workspace-v2` skipped because no shared launch/navigation/layout wiring changed.
+- Full suite skipped because this bundle is limited to Project Workspace presentation, copy, and targeted assertions.
+
+Commands run:
+- `node --check toolbox/project-workspace/project-workspace.js`
+- `node --check tests/playwright/tools/ProjectWorkspaceMockRepository.spec.mjs`
+- `npm run test:lane:project-workspace`
+- `git diff --check`
+
+Results:
+- Targeted Project Workspace lane passed: 6 tests.
+- `git diff --check` passed.

tests/playwright/tools/ProjectWorkspaceMockRepository.spec.mjs

@@ -52,6 +52,8 @@ test("Project Workspace creates, opens, and deletes mock projects", async ({ pag
   try {
     await expect(page.locator(".tool-workspace")).toBeVisible();
     await expect(page.locator("style, [style], script:not([src])")).toHaveCount(0);
+    await expect(page.getByRole("button", { name: "Create Project" })).toHaveClass("btn");
+    await expect(page.getByRole("button", { name: "Delete Open Project" })).toHaveClass("btn");
     await expect(page.locator("[data-active-project-name]")).toHaveText("Demo Project");
     await expect(page.locator("[data-project-list]")).toContainText("Demo Project");
     const demoProjectRow = page.locator("[data-project-row='demo-project']");

toolbox/project-workspace/index.html

@@ -6,7 +6,7 @@
     <base href="/">
     <meta name="viewport" content="width=device-width, initial-scale=1">
     <title>Project Workspace - GameFoundryStudio</title>
-    <meta name="description" content="Project Workspace Toolbox mock repository for project identity, progress, and publish readiness without real database or persistence behavior.">
+    <meta name="description" content="Project Workspace Toolbox page for project identity, progress, and publish readiness.">
     <link rel="icon" href="/favicon.svg">
     <link rel="stylesheet" href="assets/theme-v2/css/theme.css">
 </head>
@@ -18,7 +18,7 @@
             <div class="container">
                 <div class="kicker">Toolbox / Project Workspace</div>
                 <h1>Project Workspace</h1>
-                <p class="lede">Create, open, and delete mock projects against an in-memory SQL-shaped repository while preserving the multi-user-ready table contract.</p>
+                <p class="lede">Create, open, and delete projects while reviewing progress and publish readiness.</p>
             </div>
         </section>
         <section class="section">
@@ -34,24 +34,15 @@ <h2>Project Workspace</h2>
                                 <div class="accordion-body content-stack">
                                     <form class="content-stack content-stack--compact" data-project-form>
                                         <label for="projectNameInput">Project Name <input id="projectNameInput" type="text" placeholder="Untitled game project" data-project-name-input></label>
-                                        <button class="btn primary" type="submit">Create Project</button>
+                                        <button class="btn" type="submit">Create Project</button>
                                     </form>
                                     <button class="btn" type="button" data-project-delete-active>Delete Open Project</button>
                                 </div>
                             </details>
                             <details class="vertical-accordion" open>
                                 <summary>Open Projects</summary>
                                 <div class="accordion-body">
-                                    <div class="content-stack content-stack--compact" aria-label="Mock projects" data-project-list></div>
-                                </div>
-                            </details>
-                            <details class="vertical-accordion">
-                                <summary>Mock Repository Contract</summary>
-                                <div class="accordion-body">
-                                    <p><span class="pill">users</span> id, displayName, email, role.</p>
-                                    <p><span class="pill">projects</span> id, ownerUserId, name, status.</p>
-                                    <p><span class="pill">project_members</span> projectId, userId, permission.</p>
-                                    <p><span class="pill">Permissions</span> Owner, Editor, Viewer, Admin.</p>
+                                    <div class="content-stack content-stack--compact" aria-label="Projects" data-project-list></div>
                                 </div>
                             </details>
                         </div>
@@ -60,7 +51,7 @@ <h2>Project Workspace</h2>
                     <section class="tool-center-panel">
                         <img src="assets/theme-v2/images/image-missing.svg" alt="Project Workspace preview">
                         <h2>Workspace Overview</h2>
-                        <p>Project Workspace now runs against the in-memory mock repository approved for the first real rebuild slice. It does not persist data or connect to a real database.</p>
+                        <p>Project Workspace now manages the local project list for the first real rebuild slice. It does not persist data or connect to external services.</p>
                         <article class="card">
                             <div class="card-body content-stack">
                                 <div>
@@ -90,7 +81,7 @@ <h3>Project Progress</h3>
                                     <article class="callout"><h3>Recommended Next Tool</h3><p data-recommended-next-tool>Game Configuration</p></article>
                                     <article class="callout"><h3>Checklist</h3><ul data-project-progress-checklist><li>Project identity: Complete</li></ul></article>
                                 </div>
-                                <div class="status" role="status" data-project-workspace-log>Mock repository seeded with Demo Project.</div>
+                                <div class="status" role="status" data-project-workspace-log>Demo Project seeded.</div>
                             </div>
                         </article>
                         <div class="accordion-stack" data-project-output-panels>
@@ -99,7 +90,7 @@ <h3>Project Progress</h3>
                                 <div class="accordion-body">
                                     <div class="table-wrapper">
                                         <table class="data-table">
-                                            <caption>Mock readiness output</caption>
+                                            <caption>Readiness output</caption>
                                             <thead>
                                                 <tr><th scope="col">Path</th><th scope="col">Status</th><th scope="col">Next Tool</th></tr>
                                             </thead>
@@ -117,7 +108,7 @@ <h3>Project Progress</h3>
                                 <div class="accordion-body">
                                     <div class="table-wrapper">
                                         <table class="data-table">
-                                            <caption>Mock repository row counts</caption>
+                                            <caption>Project data row counts</caption>
                                             <thead>
                                                 <tr><th scope="col">Table</th><th scope="col">Rows</th></tr>
                                             </thead>
@@ -150,7 +141,7 @@ <h3>Project Progress</h3>
                         <article class="card" role="dialog" aria-labelledby="project-workspace-missing-requirements-title" data-missing-requirements>
                             <div class="card-body content-stack">
                                 <div>
-                                    <div class="kicker">Mock Requirements</div>
+                                    <div class="kicker">Requirements</div>
                                     <h3 id="project-workspace-missing-requirements-title">Missing Requirements</h3>
                                 </div>
                                 <p>This card remains a static review surface for the next required tools. It does not block, save, or validate runtime state.</p>
@@ -159,7 +150,7 @@ <h3 id="project-workspace-missing-requirements-title">Missing Requirements</h3>
                                     <article class="callout"><h3>Game Configuration</h3><p><span class="pill">Planned</span> Choose release profile and debug policy.</p></article>
                                     <article class="callout"><h3>Publish</h3><p><span class="pill">Planned</span> Prepare share metadata later.</p></article>
                                 </div>
-                                <div class="status" role="status">Mock-only: Project Workspace does not run real publish gating yet.</div>
+                                <div class="status" role="status">Project Workspace does not run real publish gating yet.</div>
                             </div>
                         </article>
                     </section>
@@ -172,7 +163,7 @@ <h2>Inspector</h2>
                                 <summary>Status Log</summary>
                                 <div class="accordion-body">
                                     <p>Project Workspace operations report status in the center panel while the right column stays reserved for inspector status and diagnostics.</p>
-                                    <div class="status" role="status">Mock repository ready.</div>
+                                    <div class="status" role="status">Project data ready.</div>
                                 </div>
                             </details>
                         </div>

toolbox/project-workspace/project-workspace.js

@@ -67,7 +67,7 @@ function renderProjectList() {
   if (projects.length === 0) {
     const emptyState = document.createElement("p");
     emptyState.className = "status";
-    emptyState.textContent = "No mock projects. Create or seed a project to continue.";
+    emptyState.textContent = "No projects. Create or seed a project to continue.";
     elements.projectList.append(emptyState);
     return;
   }