Polish status legend order and Project Journey count labels - PR_26156_190-status-legend-and-journey-counts-polish

ToolboxAid · ToolboxAid · commit 5237675e8818 · 2026-06-06T13:46:46.000-04:00
diff --git a/admin/notes.js b/admin/notes.js
@@ -7,34 +7,39 @@ const STATUS_ICON_PATTERN = /^\[([ xX.!?])\]\s*(.*)$/;
 const NOTE_INDEX_FILE = "index.txt";
 const STATUS_MARKERS = {
   " ": {
+    marker: "[ ]",
     icon: "⬜",
     label: "Not Started",
     name: "not-started",
     title: "Not Started"
   },
-  ".": {
-    icon: "🟡",
-    label: "In Progress",
-    name: "in-progress",
-    title: "In Progress"
-  },
-  x: {
-    icon: "✅",
-    label: "Complete",
-    name: "complete",
-    title: "Complete"
-  },
   "!": {
+    marker: "[!]",
     icon: "⛔",
     label: "Blocker",
     name: "blocker",
     title: "Blocker"
   },
   "?": {
+    marker: "[?]",
     icon: "❓",
     label: "Decide",
     name: "decide",
     title: "Decide which project questions need their own subnote files."
+  },
+  ".": {
+    marker: "[.]",
+    icon: "🟡",
+    label: "In Progress",
+    name: "in-progress",
+    title: "In Progress"
+  },
+  x: {
+    marker: "[x]",
+    icon: "✅",
+    label: "Complete",
+    name: "complete",
+    title: "Complete"
   }
 };
 
@@ -332,7 +337,7 @@ class AdminNotesViewer {
       item.dataset.adminNotesLegendItem = marker.name;
       const icon = this.documentRef.createElement("span");
       icon.dataset.adminNotesLegendIcon = marker.name;
-      icon.textContent = marker.icon;
+      icon.textContent = `${marker.marker} ${marker.icon}`;
       item.append(icon);
       item.append(this.documentRef.createTextNode(` ${marker.label}`));
       this.legendList?.append(item);
diff --git a/docs_build/dev/reports/coverage_changed_js_guardrail.txt b/docs_build/dev/reports/coverage_changed_js_guardrail.txt
@@ -6,11 +6,8 @@ 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:
-(71%) toolbox/project-workspace/project-workspace.js - executed lines 264/264; executed functions 17/24
-(71%) toolbox/tools-page-accordions.js - executed lines 826/826; executed functions 53/75
-(84%) toolbox/toolRegistry.js - executed lines 1788/1788; executed functions 31/37
-(91%) toolbox/project-journey/project-journey-mock-repository.js - executed lines 478/478; executed functions 52/57
-(93%) toolbox/project-journey/project-journey.js - executed lines 433/433; executed functions 43/46
+(91%) toolbox/project-journey/project-journey-mock-repository.js - executed lines 480/480; executed functions 52/57
+(94%) toolbox/project-journey/project-journey.js - executed lines 447/447; executed functions 45/48
 
 Guardrail warnings:
 (100%) none - no changed runtime JS coverage warnings
diff --git a/docs_build/dev/reports/dependency_gating_report.md b/docs_build/dev/reports/dependency_gating_report.md
@@ -1,6 +1,6 @@
 # Dependency Gating Report
 
-Generated: 2026-06-06T17:05:49.467Z
+Generated: 2026-06-06T17:40:56.189Z
 Status: PASS
 
 ## Gate Order
diff --git a/docs_build/dev/reports/dependency_hydration_reuse_report.md b/docs_build/dev/reports/dependency_hydration_reuse_report.md
@@ -1,6 +1,6 @@
 # Dependency Hydration Reuse Report
 
-Generated: 2026-06-06T17:05:49.468Z
+Generated: 2026-06-06T17:40:56.189Z
 Status: PASS
 
 ## Summary
diff --git a/docs_build/dev/reports/execution_graph_reuse_report.md b/docs_build/dev/reports/execution_graph_reuse_report.md
@@ -1,6 +1,6 @@
 # Execution Graph Reuse Report
 
-Generated: 2026-06-06T17:05:49.468Z
+Generated: 2026-06-06T17:40:56.189Z
 Status: PASS
 
 ## Summary
diff --git a/docs_build/dev/reports/failure_fingerprint_report.md b/docs_build/dev/reports/failure_fingerprint_report.md
@@ -1,6 +1,6 @@
 # Failure Fingerprint Report
 
-Generated: 2026-06-06T17:05:49.468Z
+Generated: 2026-06-06T17:40:56.189Z
 Status: PASS
 
 ## Summary
diff --git a/docs_build/dev/reports/incremental_validation_report.md b/docs_build/dev/reports/incremental_validation_report.md
@@ -1,6 +1,6 @@
 # Incremental Validation Report
 
-Generated: 2026-06-06T17:05:49.469Z
+Generated: 2026-06-06T17:40:56.190Z
 Status: PASS
 
 ## Reuse Summary
diff --git a/docs_build/dev/reports/lane_compilation_report.md b/docs_build/dev/reports/lane_compilation_report.md
@@ -1,6 +1,6 @@
 # Lane Compilation Report
 
-Generated: 2026-06-06T17:05:49.467Z
+Generated: 2026-06-06T17:40:56.189Z
 Status: PASS
 
 ## Lane Graph
diff --git a/docs_build/dev/reports/lane_deduplication_report.md b/docs_build/dev/reports/lane_deduplication_report.md
@@ -1,6 +1,6 @@
 # Lane Deduplication Report
 
-Generated: 2026-06-06T17:05:49.468Z
+Generated: 2026-06-06T17:40:56.189Z
 Status: PASS
 
 ## Summary
diff --git a/docs_build/dev/reports/lane_input_validation_report.md b/docs_build/dev/reports/lane_input_validation_report.md
@@ -1,6 +1,6 @@
 # Lane Input Validation Report
 
-Generated: 2026-06-06T17:05:49.469Z
+Generated: 2026-06-06T17:40:56.190Z
 Status: PASS
 
 ## Input Files
diff --git a/docs_build/dev/reports/lane_runtime_optimization_report.md b/docs_build/dev/reports/lane_runtime_optimization_report.md
@@ -1,6 +1,6 @@
 # Lane Runtime Optimization Report
 
-Generated: 2026-06-06T17:05:49.468Z
+Generated: 2026-06-06T17:40:56.189Z
 Status: PASS
 
 ## Runtime Cost Summary
diff --git a/docs_build/dev/reports/lane_snapshot_report.md b/docs_build/dev/reports/lane_snapshot_report.md
@@ -1,6 +1,6 @@
 # Lane Snapshot Report
 
-Generated: 2026-06-06T17:05:49.468Z
+Generated: 2026-06-06T17:40:56.189Z
 Status: PASS
 Snapshot directory: docs_build/dev/reports/lane_snapshots
 
diff --git a/docs_build/dev/reports/lane_warm_start_report.md b/docs_build/dev/reports/lane_warm_start_report.md
@@ -1,6 +1,6 @@
 # Lane Warm-Start Report
 
-Generated: 2026-06-06T17:05:49.468Z
+Generated: 2026-06-06T17:40:56.189Z
 Status: PASS
 Warm-start directory: docs_build/dev/reports/lane_warm_starts
 
diff --git a/docs_build/dev/reports/monolith_trigger_removal_report.md b/docs_build/dev/reports/monolith_trigger_removal_report.md
@@ -1,6 +1,6 @@
 # Monolith Trigger Removal Report
 
-Generated: 2026-06-06T17:05:49.471Z
+Generated: 2026-06-06T17:40:56.192Z
 Status: PASS
 
 ## Removed Broad Execution Triggers
diff --git a/docs_build/dev/reports/persistent_lane_manifest_report.md b/docs_build/dev/reports/persistent_lane_manifest_report.md
@@ -1,6 +1,6 @@
 # Persistent Lane Manifest Report
 
-Generated: 2026-06-06T17:05:49.469Z
+Generated: 2026-06-06T17:40:56.190Z
 Status: PASS
 Manifest directory: docs_build/dev/reports/lane_manifests
 
diff --git a/docs_build/dev/reports/playwright_v8_coverage_report.txt b/docs_build/dev/reports/playwright_v8_coverage_report.txt
@@ -17,11 +17,8 @@ Exercised tool entry points detected:
 (74%) Theme V2 Shared JS - exercised 2 runtime JS files
 
 Changed runtime JS files covered:
-(71%) toolbox/project-workspace/project-workspace.js - executed lines 264/264; executed functions 17/24
-(71%) toolbox/tools-page-accordions.js - executed lines 826/826; executed functions 53/75
-(84%) toolbox/toolRegistry.js - executed lines 1788/1788; executed functions 31/37
-(91%) toolbox/project-journey/project-journey-mock-repository.js - executed lines 478/478; executed functions 52/57
-(93%) toolbox/project-journey/project-journey.js - executed lines 433/433; executed functions 43/46
+(91%) toolbox/project-journey/project-journey-mock-repository.js - executed lines 480/480; executed functions 52/57
+(94%) toolbox/project-journey/project-journey.js - executed lines 447/447; executed functions 45/48
 
 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
@@ -30,20 +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 206/206; executed functions 16/20
 (84%) toolbox/toolRegistry.js - executed lines 1788/1788; executed functions 31/37
-(91%) toolbox/project-journey/project-journey-mock-repository.js - executed lines 478/478; executed functions 52/57
-(93%) toolbox/project-journey/project-journey.js - executed lines 433/433; executed functions 43/46
+(91%) toolbox/project-journey/project-journey-mock-repository.js - executed lines 480/480; executed functions 52/57
+(94%) toolbox/project-journey/project-journey.js - executed lines 447/447; executed functions 45/48
+(100%) admin/notes.js - executed lines 435/435; executed functions 44/44
 
 Uncovered or low-coverage changed JS files:
 (100%) none - no low-coverage changed runtime JS files
 
 Changed JS files considered:
-(0%) admin/notes.js - 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
-(0%) tests/playwright/tools/ProjectWorkspaceMockRepository.spec.mjs - changed JS file not collected as browser runtime coverage
-(0%) tests/playwright/tools/ToolNavigationPrevNext.spec.mjs - changed JS file not collected as browser runtime coverage
-(71%) toolbox/project-workspace/project-workspace.js - changed JS file with browser V8 coverage
-(71%) toolbox/tools-page-accordions.js - changed JS file with browser V8 coverage
-(84%) toolbox/toolRegistry.js - changed JS file with browser V8 coverage
 (91%) toolbox/project-journey/project-journey-mock-repository.js - changed JS file with browser V8 coverage
-(93%) toolbox/project-journey/project-journey.js - changed JS file with browser V8 coverage
+(94%) toolbox/project-journey/project-journey.js - changed JS file with browser V8 coverage
+(100%) admin/notes.js - changed JS file with browser V8 coverage
diff --git a/docs_build/dev/reports/project-journey-tool-v1-report.md b/docs_build/dev/reports/project-journey-tool-v1-report.md
@@ -18,7 +18,8 @@ Task: PR_26156_189-project-journey-tool-v1
   - `[x]` Complete / `✅`
   - `[!]` Blocker / `⛔`
   - `[?]` Decide / `❓`
-- Open count is computed as Not Started + In Progress + Blocker + Decide.
+- Total count is every entry in the note.
+- Open count is computed as Not Started + Blocker + Decide + In Progress. Complete entries are not counted as Open.
 
 ## UI Coverage
 
diff --git a/docs_build/dev/reports/retry_suppression_report.md b/docs_build/dev/reports/retry_suppression_report.md
@@ -1,6 +1,6 @@
 # Retry Suppression Report
 
-Generated: 2026-06-06T17:05:49.469Z
+Generated: 2026-06-06T17:40:56.190Z
 Status: PASS
 
 ## Summary
diff --git a/docs_build/dev/reports/slow_path_pruning_report.md b/docs_build/dev/reports/slow_path_pruning_report.md
@@ -1,6 +1,6 @@
 # Slow Path Pruning Report
 
-Generated: 2026-06-06T17:05:49.471Z
+Generated: 2026-06-06T17:40:56.192Z
 Status: PASS
 Source timing evidence: docs_build/dev/reports/test_cleanup_performance_report.md (2026-05-26T21:18:42.199Z)
 
diff --git a/docs_build/dev/reports/static_validation_report.md b/docs_build/dev/reports/static_validation_report.md
@@ -1,6 +1,6 @@
 # Static Validation Report
 
-Generated: 2026-06-06T17:05:49.463Z
+Generated: 2026-06-06T17:40:56.184Z
 Status: PASS
 Static only: Yes
 Dry run: No
diff --git a/docs_build/dev/reports/status-legend-and-journey-counts-polish-report.md b/docs_build/dev/reports/status-legend-and-journey-counts-polish-report.md
@@ -0,0 +1,34 @@
+# Status Legend And Journey Counts Polish Report
+
+Task: PR_26156_190-status-legend-and-journey-counts-polish
+
+## Scope
+
+- Updated Admin Notes status legend order to the requested execution sequence:
+  `[ ] ⬜ Not Started`, `[!] ⛔ Blocker`, `[?] ❓ Decide`, `[.] 🟡 In Progress`, `[x] ✅ Complete`.
+- Preserved Admin Notes parsing behavior and status icon rendering for note content.
+- Updated Project Journey status order to:
+  `⬜ Not Started`, `⛔ Blocker`, `❓ Decide`, `🟡 In Progress`, `✅ Complete`.
+- Removed raw bracket markers from Project Journey status dropdown labels.
+- Added a Project Journey Status Legend using existing Theme V2 tool classes.
+- Added `Total` and `Open` as separate Project Journey summary table columns.
+
+## Count Definitions
+
+- Total = every entry in the selected Project Journey note.
+- Open = Not Started + Blocker + Decide + In Progress.
+- Complete is not counted as Open.
+
+## Constraints
+
+- No new CSS was added.
+- No page-local CSS, tool-local CSS, inline styles, `<style>` blocks, `<script>` blocks, or inline event handlers were added.
+- No archived V1/V2 files or `start_of_day` folders were modified.
+- Full samples smoke was not run, per request.
+
+## Validation
+
+- Targeted Admin Notes parser/UI lane: `npx playwright test tests/playwright/tools/AdminNotesViewer.spec.mjs --workers=1`.
+- Targeted Project Journey runtime/UI lane: `npx playwright test tests/playwright/tools/ProjectJourneyTool.spec.mjs --workers=1`.
+- Combined coverage-producing targeted pass: `npx playwright test tests/playwright/tools/AdminNotesViewer.spec.mjs tests/playwright/tools/ProjectJourneyTool.spec.mjs --workers=1`.
+- Changed-file/static validation: `npm run test:playwright:static`.
diff --git a/docs_build/dev/reports/targeted_file_manifest_report.md b/docs_build/dev/reports/targeted_file_manifest_report.md
@@ -1,6 +1,6 @@
 # Targeted File Manifest Report
 
-Generated: 2026-06-06T17:05:49.469Z
+Generated: 2026-06-06T17:40:56.190Z
 Status: PASS
 
 ## Manifest-Generated Lane Inputs
diff --git a/docs_build/dev/reports/test_cleanup_performance_report.md b/docs_build/dev/reports/test_cleanup_performance_report.md
@@ -1,6 +1,6 @@
 # Test Cleanup Performance Report
 
-Generated: 2026-06-06T17:05:49.470Z
+Generated: 2026-06-06T17:40:56.192Z
 Status: PASS
 
 ## Cost Summary
diff --git a/docs_build/dev/reports/test_cleanup_routing_report.md b/docs_build/dev/reports/test_cleanup_routing_report.md
@@ -1,6 +1,6 @@
 # Test Cleanup Routing Report
 
-Generated: 2026-06-06T17:05:49.470Z
+Generated: 2026-06-06T17:40:56.192Z
 Status: PASS
 
 ## Representative Routing Cases
diff --git a/docs_build/dev/reports/testing_lane_execution_report.md b/docs_build/dev/reports/testing_lane_execution_report.md
@@ -194,3 +194,22 @@ This section records the full targeted validation set for `PR_26156_189-project-
 | `npm run test:playwright:static` | PASS | Changed-file/static zero-browser validation. |
 
 Full samples smoke was not run, per PR instructions.
+
+## PR_26156_190 Status Legend And Journey Counts Manual Lane Summary
+
+This section records the targeted validation set for `PR_26156_190-status-legend-and-journey-counts-polish`.
+
+| Command | Result | Notes |
+| --- | --- | --- |
+| `node --check admin/notes.js` | PASS | Admin Notes parser/legend runtime syntax check. |
+| `node --check toolbox/project-journey/project-journey.js` | PASS | Project Journey runtime syntax check. |
+| `node --check toolbox/project-journey/project-journey-mock-repository.js` | PASS | Project Journey mock repository syntax check. |
+| `node --check tests/playwright/tools/AdminNotesViewer.spec.mjs` | PASS | Admin Notes targeted spec syntax check. |
+| `node --check tests/playwright/tools/ProjectJourneyTool.spec.mjs` | PASS | Project Journey targeted spec syntax check. |
+| `rg --pcre2 ... admin/notes.js admin/notes.html toolbox/project-journey ...` | PASS | No inline CSS/script handlers and no archive/start_of_day references in touched active files. |
+| `npx playwright test tests/playwright/tools/AdminNotesViewer.spec.mjs --workers=1` | PASS | 3 tests; Admin Notes parser/UI lane and raw-marker legend order. |
+| `npx playwright test tests/playwright/tools/ProjectJourneyTool.spec.mjs --workers=1` | PASS | 6 tests; Project Journey dropdown labels, status legend order, Total/Open summary counts, row editing, filters, indent/outdent, and movement behavior. |
+| `npx playwright test tests/playwright/tools/AdminNotesViewer.spec.mjs tests/playwright/tools/ProjectJourneyTool.spec.mjs --workers=1` | PASS | 9 tests; combined targeted run used for final V8 coverage report. |
+| `npm run test:playwright:static` | PASS | Changed-file/static zero-browser validation. |
+
+Full samples smoke was not run, per PR instructions.
diff --git a/docs_build/dev/reports/validation_cache_report.md b/docs_build/dev/reports/validation_cache_report.md
@@ -1,6 +1,6 @@
 # Validation Cache Report
 
-Generated: 2026-06-06T17:05:49.470Z
+Generated: 2026-06-06T17:40:56.191Z
 Status: PASS
 
 ## Cache Summary
diff --git a/docs_build/dev/reports/zero_browser_preflight_report.md b/docs_build/dev/reports/zero_browser_preflight_report.md
@@ -1,6 +1,6 @@
 # Zero-Browser Preflight Report
 
-Generated: 2026-06-06T17:05:49.470Z
+Generated: 2026-06-06T17:40:56.191Z
 Status: PASS
 
 ## Prevented Browser Launches
diff --git a/tests/playwright/tools/AdminNotesViewer.spec.mjs b/tests/playwright/tools/AdminNotesViewer.spec.mjs
@@ -102,11 +102,11 @@ test("Admin Notes displays index.txt, parser output, and linked subnotes", async
     await expect(page.locator("[data-admin-notes-legend]")).toContainText("Status Legend");
     const legendItems = page.locator("[data-admin-notes-legend-item]");
     await expect(legendItems).toHaveText([
-      "⬜ Not Started",
-      "🟡 In Progress",
-      "✅ Complete",
-      "⛔ Blocker",
-      "❓ Decide"
+      "[ ] ⬜ Not Started",
+      "[!] ⛔ Blocker",
+      "[?] ❓ Decide",
+      "[.] 🟡 In Progress",
+      "[x] ✅ Complete"
     ]);
     await expect(page.locator("[data-admin-notes-legend] [title]")).toHaveCount(0);
     const legendLabelBox = await page.locator("[data-admin-notes-legend] strong").boundingBox();
diff --git a/tests/playwright/tools/ProjectJourneyTool.spec.mjs b/tests/playwright/tools/ProjectJourneyTool.spec.mjs
diff --git a/toolbox/project-journey/index.html b/toolbox/project-journey/index.html
diff --git a/toolbox/project-journey/project-journey-mock-repository.js b/toolbox/project-journey/project-journey-mock-repository.js
diff --git a/toolbox/project-journey/project-journey.js b/toolbox/project-journey/project-journey.js
