Fix Project Journey selection counts ownership and status layout - PR_26157_001-project-journey-selection-counts-and-ownership

ToolboxAid · ToolboxAid · commit 29d49d2e26d3 · 2026-06-06T14:26:00.000-04:00
diff --git a/docs_build/dev/reports/coverage_changed_js_guardrail.txt b/docs_build/dev/reports/coverage_changed_js_guardrail.txt
@@ -6,9 +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:
-(0%) toolbox/project-journey/project-journey-mock-repository.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
-(0%) toolbox/project-journey/project-journey.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
+(92%) toolbox/project-journey/project-journey-mock-repository.js - executed lines 524/524; executed functions 58/63
+(96%) toolbox/project-journey/project-journey.js - executed lines 520/520; executed functions 55/57
 
 Guardrail warnings:
-(0%) toolbox/project-journey/project-journey-mock-repository.js - WARNING: changed runtime JS file missing from coverage; advisory only
-(0%) toolbox/project-journey/project-journey.js - WARNING: changed runtime JS file missing from coverage; advisory only
+(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-06T18:08:38.813Z
+Generated: 2026-06-06T18:18:46.366Z
 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-06T18:08:38.813Z
+Generated: 2026-06-06T18:18:46.367Z
 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-06T18:08:38.813Z
+Generated: 2026-06-06T18:18:46.367Z
 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-06T18:08:38.813Z
+Generated: 2026-06-06T18:18:46.367Z
 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-06T18:08:38.814Z
+Generated: 2026-06-06T18:18:46.368Z
 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-06T18:08:38.813Z
+Generated: 2026-06-06T18:18:46.366Z
 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-06T18:08:38.813Z
+Generated: 2026-06-06T18:18:46.366Z
 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-06T18:08:38.814Z
+Generated: 2026-06-06T18:18:46.368Z
 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-06T18:08:38.813Z
+Generated: 2026-06-06T18:18:46.367Z
 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-06T18:08:38.813Z
+Generated: 2026-06-06T18:18:46.367Z
 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-06T18:08:38.813Z
+Generated: 2026-06-06T18:18:46.367Z
 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-06T18:08:38.816Z
+Generated: 2026-06-06T18:18:46.370Z
 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-06T18:08:38.814Z
+Generated: 2026-06-06T18:18:46.367Z
 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
@@ -12,26 +12,31 @@ Note: entry percentages use function coverage when available, otherwise line cov
 Note: coverage entries are aggregated across every page/tool where coverageReporter.start(page) and coverageReporter.stop(page) ran.
 
 Exercised tool entry points detected:
-(0%) Toolbox Index - not exercised by this Playwright run
+(83%) Toolbox Index - exercised 6 runtime JS files
 (0%) Tool Template V2 - not exercised by this Playwright run
-(80%) Theme V2 Shared JS - exercised 1 runtime JS files
+(74%) Theme V2 Shared JS - exercised 2 runtime JS files
 
 Changed runtime JS files covered:
-(0%) toolbox/project-journey/project-journey-mock-repository.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
-(0%) toolbox/project-journey/project-journey.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
+(92%) toolbox/project-journey/project-journey-mock-repository.js - executed lines 524/524; executed functions 58/63
+(96%) toolbox/project-journey/project-journey.js - executed lines 520/520; executed functions 55/57
 
 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
+(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
+(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
-(100%) admin/notes.js - executed lines 488/488; executed functions 48/48
+(84%) toolbox/toolRegistry.js - executed lines 1788/1788; executed functions 31/37
+(92%) toolbox/project-journey/project-journey-mock-repository.js - executed lines 524/524; executed functions 58/63
+(96%) toolbox/project-journey/project-journey.js - executed lines 520/520; executed functions 55/57
 
 Uncovered or low-coverage changed JS files:
-(0%) toolbox/project-journey/project-journey-mock-repository.js - WARNING: uncovered changed runtime JS file; advisory only
-(0%) toolbox/project-journey/project-journey.js - WARNING: uncovered changed runtime JS file; advisory only
+(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/helpers/playwrightRepoServer.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
-(0%) toolbox/project-journey/project-journey-mock-repository.js - changed JS file not collected as browser runtime coverage
-(0%) toolbox/project-journey/project-journey.js - changed JS file not collected as browser runtime coverage
-(100%) admin/notes.js - changed JS file with browser V8 coverage
+(92%) 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
diff --git a/docs_build/dev/reports/project-journey-selection-counts-and-ownership-report.md b/docs_build/dev/reports/project-journey-selection-counts-and-ownership-report.md
@@ -0,0 +1,43 @@
+# Project Journey Selection Counts And Ownership Report
+
+Task: PR_26157_001-project-journey-selection-counts-and-ownership
+
+## Scope
+
+- Moved Project Journey Status Legend into the Statistics tile and placed it after the statistics grid.
+- Added explicit Statistics scope text:
+  - selected summary row: counts for that selected note only
+  - navigation filter mode: aggregate counts for the filtered result set
+- Navigation buttons now deselect the selected Summary Table Name row.
+- Summary Table column order is now:
+  `Name | Type | ⬜ | ⛔ | ❓ | 🟡 | ✅ | Open | Total | Updated`
+- Open = ⬜ Not Started + ⛔ Blocker + ❓ Decide + 🟡 In Progress.
+- Total = all entries, including ✅ Complete.
+- Selected note rows and selected tree entries use existing `btn primary` styling for visible selection.
+
+## Ownership
+
+- Project Journey entries now track:
+  - `createdBy`
+  - `createdByType`
+  - `originalSystemMeaning`
+- Seeded mock entries are system-created and preserve `originalSystemMeaning` through edits.
+- User-added entries are user-created.
+- System-created entries can be edited but cannot be deleted.
+- User-created entries can be edited and deleted through the Project Journey row editor.
+
+## Constraint Checks
+
+- No Project Journey display of Admin Notes Current Folder, Open folder, Return to root index, or filesystem listing diagnostics.
+- 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
+
+- `node --check toolbox/project-journey/project-journey.js` - PASS.
+- `node --check toolbox/project-journey/project-journey-mock-repository.js` - PASS.
+- `node --check tests/playwright/tools/ProjectJourneyTool.spec.mjs` - PASS.
+- `npx playwright test tests/playwright/tools/ProjectJourneyTool.spec.mjs --workers=1` - PASS, 7 tests.
+- `npm run test:playwright:static` - PASS.
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-06T18:08:38.814Z
+Generated: 2026-06-06T18:18:46.367Z
 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-06T18:08:38.816Z
+Generated: 2026-06-06T18:18:46.370Z
 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-06T18:08:38.808Z
+Generated: 2026-06-06T18:18:46.361Z
 Status: PASS
 Static only: Yes
 Dry run: No
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-06T18:08:38.814Z
+Generated: 2026-06-06T18:18:46.367Z
 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-06T18:08:38.815Z
+Generated: 2026-06-06T18:18:46.369Z
 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-06T18:08:38.816Z
+Generated: 2026-06-06T18:18:46.369Z
 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
@@ -195,6 +195,21 @@ This section records the full targeted validation set for `PR_26156_189-project-
 
 Full samples smoke was not run, per PR instructions.
 
+## PR_26157_001 Project Journey Selection Counts And Ownership Manual Lane Summary
+
+This section records the targeted validation set for `PR_26157_001-project-journey-selection-counts-and-ownership`.
+
+| Command | Result | Notes |
+| --- | --- | --- |
+| `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/ProjectJourneyTool.spec.mjs` | PASS | Targeted spec syntax check. |
+| `rg --pcre2 ... toolbox/project-journey tests/playwright/tools/ProjectJourneyTool.spec.mjs` | PASS | No inline CSS/script handlers, archive/start_of_day references, or Admin Notes coupling in touched Project Journey files. |
+| `npx playwright test tests/playwright/tools/ProjectJourneyTool.spec.mjs --workers=1` | PASS | 7 tests; status legend placement, selected-note stats, filtered aggregate stats, column order, Open/Total formulas, ownership/delete rules, selected entry styling, and Admin Notes UI absence. |
+| `npm run test:playwright:static` | PASS | Changed-file/static zero-browser validation. |
+
+Full samples smoke was not run, per PR instructions.
+
 ## PR_26156_191 Admin Notes Live Folder Listing Manual Lane Summary
 
 This section records the targeted validation set for `PR_26156_191-admin-notes-live-folder-listing`.
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-06T18:08:38.815Z
+Generated: 2026-06-06T18:18:46.369Z
 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-06T18:08:38.815Z
+Generated: 2026-06-06T18:18:46.368Z
 Status: PASS
 
 ## Prevented Browser Launches
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
