Add Project Journey sort indicators compact button width and skipped status - PR_26157_006-project-journey-sort-and-skipped-status

Author: qbytesdq-jpg

admin/notes.js

@@ -2,7 +2,7 @@ const NOTES_DIRECTORY = "docs_build/dev/admin-notes";
 const DEFAULT_NOTE = "index";
 const LINK_CLASS = "btn btn--compact primary";
 const NOTE_NAME_PATTERN = /^[A-Za-z0-9_-]+$/;
-const STATUS_ICON_PATTERN = /^\[([ xX.!?])\]\s*(.*)$/;
+const STATUS_ICON_PATTERN = /^\[([ xX.!?-])\]\s*(.*)$/;
 const NOTE_INDEX_FILE = "index.txt";
 const DIRECTORY_LIST_QUERY = "adminNotesDirectory";
 const STATUS_MARKERS = {
@@ -40,6 +40,13 @@ const STATUS_MARKERS = {
     label: "Complete",
     name: "complete",
     title: "Complete"
+  },
+  "-": {
+    marker: "[-]",
+    icon: "⏭️",
+    label: "Skipped",
+    name: "skipped",
+    title: "Skipped"
   }
 };
 

assets/theme-v2/css/buttons.css

@@ -19,6 +19,7 @@
 
 .btn--compact {
   margin: var(--space-0);
+  min-width: calc(var(--space-44) + var(--space-28));
   padding: var(--space-3) var(--space-8);
   font-size: var(--font-size-xs);
   font-weight: 600;

docs_build/dev/admin-notes/index.txt

@@ -8,6 +8,7 @@ Ideas
 
 Things to Fix
 - [!] Replace placeholder notes with concrete admin follow-up items.
+- [-] Keep intentionally skipped work visible without counting it as open.
 - Track fixes that are not ready for a dedicated BUILD_PR yet.
 
 Undecided Questions

docs_build/dev/reports/playwright_v8_coverage_report.txt

@@ -17,8 +17,8 @@ Exercised tool entry points detected:
 (74%) Theme V2 Shared JS - exercised 2 runtime JS files
 
 Changed runtime JS files covered:
-(94%) toolbox/project-journey/project-journey-mock-repository.js - executed lines 864/864; executed functions 74/79
-(96%) toolbox/project-journey/project-journey.js - executed lines 802/802; executed functions 80/83
+(94%) toolbox/project-journey/project-journey-mock-repository.js - executed lines 882/882; executed functions 74/79
+(96%) toolbox/project-journey/project-journey.js - executed lines 815/815; executed functions 81/84
 
 Files with executed line/function counts where available:
 (67%) assets/theme-v2/js/tool-display-mode.js - executed lines 201/201; executed functions 10/15
@@ -27,16 +27,16 @@ Files with executed line/function counts where available:
 (76%) toolbox/project-workspace/project-workspace-mock-repository.js - executed lines 402/402; executed functions 26/34
 (80%) assets/theme-v2/js/gamefoundry-partials.js - executed lines 207/207; executed functions 16/20
 (84%) toolbox/toolRegistry.js - executed lines 1788/1788; executed functions 31/37
-(94%) toolbox/project-journey/project-journey-mock-repository.js - executed lines 864/864; executed functions 74/79
-(96%) toolbox/project-journey/project-journey.js - executed lines 802/802; executed functions 80/83
+(94%) toolbox/project-journey/project-journey-mock-repository.js - executed lines 882/882; executed functions 74/79
+(96%) toolbox/project-journey/project-journey.js - executed lines 815/815; executed functions 81/84
 
 Uncovered or low-coverage changed JS files:
 (100%) none - no low-coverage changed runtime JS files
 
 Changed JS files considered:
-(0%) admin/db-viewer.js - changed JS file not collected as browser runtime coverage
+(0%) admin/notes.js - changed JS file not collected as browser runtime coverage
 (0%) tests/playwright/tools/AdminDbViewer.spec.mjs - changed JS file not collected as browser runtime coverage
+(0%) tests/playwright/tools/AdminNotesViewer.spec.mjs - changed JS file not collected as browser runtime coverage
 (0%) tests/playwright/tools/ProjectJourneyTool.spec.mjs - changed JS file not collected as browser runtime coverage
-(80%) assets/theme-v2/js/gamefoundry-partials.js - changed JS file with browser V8 coverage
 (94%) toolbox/project-journey/project-journey-mock-repository.js - changed JS file with browser V8 coverage
 (96%) toolbox/project-journey/project-journey.js - changed JS file with browser V8 coverage

docs_build/dev/reports/project-journey-sort-and-skipped-status-report.md

@@ -0,0 +1,44 @@
+# Project Journey Sort And Skipped Status Report
+
+Generated: 2026-06-06
+
+Status: PASS
+
+## Summary
+
+- Added active Summary Table sort indicators using existing sortable headers, `aria-sort`, `primary` button state, and asc/desc arrows.
+- Increased reusable `.btn.btn--compact` minimum width through Theme V2 button CSS.
+- Added the Skipped status to Project Journey as `[-] ⏭️ Skipped` after Complete.
+- Added Skipped counts to Project Journey Summary Table and Statistics while keeping Open limited to Not Started, Blocker, Decide, and In Progress.
+- Added a Skipped navigation filter and seeded one system-created skipped Project Journey item.
+- Added Admin Notes parser and legend support for `[-] ⏭️ Skipped`.
+
+## Status Order
+
+Project Journey and Admin Notes now use this status order where applicable:
+
+- ⬜ Not Started
+- ⛔ Blocker
+- ❓ Decide
+- 🟡 In Progress
+- ✅ Complete
+- ⏭️ Skipped
+
+## Count Rules
+
+- Open = Not Started + Blocker + Decide + In Progress.
+- Total = all statuses, including Complete and Skipped.
+- Skipped is intentionally excluded from Open and included in Total.
+
+## Validation
+
+- Project Journey runtime/UI lane: PASS.
+- Admin Notes parser/UI lane: PASS.
+- Admin DB Viewer relationship-count check: PASS.
+- Changed-file/static validation: PASS.
+- Full samples smoke: SKIP per instructions.
+
+## Design System Note
+
+- The compact button width change is reusable Theme V2 styling in `assets/theme-v2/css/buttons.css`.
+- Sort indicators reuse existing button states and do not add page-local or tool-local CSS.

docs_build/dev/reports/testing_lane_execution_report.md

@@ -7,9 +7,8 @@ Status: PASS
 ## Lanes
 
 - Project Journey runtime/UI lane: PASS.
-- Mock DB Project Journey Tables / DB Viewer UI lane: PASS.
-- Project Workspace handoff coverage: PASS inside the targeted Project Journey spec.
-- Toolbox registration/navigation coverage: PASS inside the targeted Project Journey spec.
+- Admin Notes parser/UI lane: PASS.
+- Admin DB Viewer relationship-count check: PASS because Project Journey mock DB seed counts changed.
 - Changed-file/static validation: PASS.
 - Full samples smoke: SKIP per PR instructions.
 
@@ -19,25 +18,31 @@ Status: PASS
 | --- | --- | --- |
 | `node --check toolbox/project-journey/project-journey.js` | PASS | Project Journey runtime syntax passed. |
 | `node --check toolbox/project-journey/project-journey-mock-repository.js` | PASS | Project Journey mock DB syntax passed. |
-| `node --check admin/db-viewer.js` | PASS | DB Viewer syntax passed. |
+| `node --check admin/notes.js` | PASS | Admin Notes parser syntax passed. |
 | `node --check tests/playwright/tools/ProjectJourneyTool.spec.mjs` | PASS | Project Journey Playwright spec syntax passed. |
-| `node --check tests/playwright/tools/AdminDbViewer.spec.mjs` | PASS | DB Viewer Playwright spec syntax passed. |
-| `npx playwright test tests/playwright/tools/ProjectJourneyTool.spec.mjs tests/playwright/tools/AdminDbViewer.spec.mjs --reporter=list` | PASS | 12 passed in 29.4s. |
+| `node --check tests/playwright/tools/AdminNotesViewer.spec.mjs` | PASS | Admin Notes Playwright spec syntax passed. |
+| `npx playwright test tests/playwright/tools/ProjectJourneyTool.spec.mjs tests/playwright/tools/AdminNotesViewer.spec.mjs tests/playwright/tools/AdminDbViewer.spec.mjs --reporter=list` | PASS | 16 passed in 39.0s. |
 | `npm run test:playwright:static -- --static-report docs_build/dev/reports/static_validation_report.md` | PASS | Static-only lane runner completed; auxiliary static cache report churn was restored out of the PR diff. |
 
 ## Targeted Runtime Coverage
 
-- Verified Mock DB Project Journey Tables uses the reusable tools template page structure.
-- Verified the DB dump remains human-readable and read-only.
-- Verified the Statistics divider appears between Total and Not Started.
-- Verified the existing manual Status Legend layout and dynamic legend wiring remain intact.
-- Verified obsolete `statusLegend` code was not removed because the current legend element still uses it.
-- Verified System Generated filtering shows system-created items and excludes user-created items.
-- Verified Note Tree rows keep status, title, forge-bot, and trashcan controls on one line.
-- Verified Note Tree content is left-justified and constrained to the reusable 90 percent row-width class.
-- Verified Item Details uses the standard tool form table with the label left of `journeyItemDetailsInput`.
+- Verified Project Journey Summary Table sorted column is visually distinct and shows asc/desc arrows.
+- Verified every sortable Summary Table column still toggles asc/desc.
+- Verified reusable `.btn.btn--compact` minimum width increased through Theme V2 button CSS.
+- Verified Skipped appears after Complete in Project Journey navigation, summary headers, statistics, dropdowns, and legend.
+- Verified Skipped does not count as Open and does count in Total.
+- Verified Summary Table displays Skipped after Complete and before Open.
+- Verified Skipped filter returns the expected release note with Skipped counts.
+- Verified Admin Notes parses `[-]` as Skipped and shows `[-] ⏭️ Skipped` in the status legend.
+- Verified Project Journey manual Status Legend layout remains intact while receiving the new Skipped entry from the existing renderer.
+
+## Coverage Notes
+
+- `docs_build/dev/reports/playwright_v8_coverage_report.txt` was regenerated by the targeted Playwright run.
+- Project Journey changed runtime JS was collected by V8 coverage.
+- `admin/notes.js` is listed by the coverage reporter as not collected browser runtime coverage, but the Admin Notes parser/UI lane executed and passed.
 
 ## Skipped Lanes
 
 - Full samples smoke was skipped because samples were explicitly out of scope.
-- Broad runtime lanes were skipped because the targeted Project Journey, DB Viewer, handoff, registration, and static lanes covered the changed surfaces.
+- Broad runtime lanes were skipped because the targeted Project Journey, Admin Notes, DB Viewer count, and static lanes covered the changed surfaces.

tests/playwright/tools/AdminDbViewer.spec.mjs

@@ -96,10 +96,10 @@ test("Admin DB Viewer shows read-only mock DB tables and diagnostics", async ({
     );
     await expect(page.locator("[data-admin-db-bleed-findings]")).toContainText("No table bleed detected.");
     await expect(page.locator("[data-admin-db-relationship-summary]")).toContainText(
-      "project_journey_items.noteId -> project_journey_notes.id: 8/8 records linked."
+      "project_journey_items.noteId -> project_journey_notes.id: 9/9 records linked."
     );
     await expect(page.locator("[data-admin-db-relationship-summary]")).toContainText(
-      "system project_journey_items.templateId -> active project_journey_templates.templateId: 8/8 records linked."
+      "system project_journey_items.templateId -> active project_journey_templates.templateId: 9/9 records linked."
     );
     await expect(page.locator("[data-admin-db-missing-links]")).toContainText("No missing links detected.");
     await expect(page.locator("[data-admin-db-viewer] input, [data-admin-db-viewer] textarea, [data-admin-db-viewer] select, [data-admin-db-viewer] button")).toHaveCount(0);

tests/playwright/tools/AdminNotesViewer.spec.mjs

@@ -103,6 +103,7 @@ test("Admin Notes displays index.txt, parser output, and linked subnotes", async
     await expect(page.locator("[data-admin-notes-status-icon='in-progress']")).toHaveText("🟡");
     await expect(page.locator("[data-admin-notes-status-icon='blocker']")).toHaveText("⛔");
     await expect(page.locator("[data-admin-notes-status-icon='decide']")).toHaveText("❓");
+    await expect(page.locator("[data-admin-notes-status-icon='skipped']")).toHaveText("⏭️");
     await expect(page.locator("[data-admin-notes-status-icon='decide']")).toHaveAttribute(
       "title",
       "Decide which project questions need their own subnote files."
@@ -141,7 +142,8 @@ test("Admin Notes displays index.txt, parser output, and linked subnotes", async
       "[!] ⛔ Blocker",
       "[?] ❓ Decide",
       "[.] 🟡 In Progress",
-      "[x] ✅ Complete"
+      "[x] ✅ Complete",
+      "[-] ⏭️ Skipped"
     ]);
     await expect(page.locator("[data-admin-notes-legend] [title]")).toHaveCount(0);
     const legendLabelBox = await page.locator("[data-admin-notes-legend] strong").boundingBox();