Skip to content

Commit a6f1edb

Browse files
committed
Add P0 blockers and known manual-test issues
Record findings from the 2026-03-29 manual test session: add an Active Blockers section to IMPLEMENTATION_MASTERPLAN.md describing two P0 bugs (#508 queue scoping, #509 board auto-switch) and related P1 issues (#510#515). Update Next Best Steps to prioritize resolving #508 and #509. Update MANUAL_TEST_CHECKLIST.md with concrete checks and known-bug notes: a fresh-registration data isolation check referencing #508, a WIP warning/disable note for #517, and a deletion confirmation expectation plus a note for the delete-without-confirmation bug #513.
1 parent ba27554 commit a6f1edb

File tree

2 files changed

+15
-2
lines changed

2 files changed

+15
-2
lines changed

docs/IMPLEMENTATION_MASTERPLAN.md

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -950,9 +950,19 @@ Immediate hardening landed in this context:
950950
- opt-in live-provider Playwright probe (`tests/e2e/live-llm.spec.ts`)
951951
- headed local audit shortcuts (`npm run test:e2e:audit:headed`, `npm run test:e2e:live-llm:headed`)
952952

953+
## Active Blockers (2026-03-29 Manual Test Session)
954+
955+
Two P0 bugs discovered in fresh-registration manual testing must be resolved before Phase 4 can be signed off or any external user onboarding begins. These are data correctness/security failures, not UX polish:
956+
957+
- **`#508`** — Queue list endpoint not scoped to the authenticated user: a fresh-registered account sees all historical queue items from other sessions. Add a `userId` predicate to the LLM queue list query and add a cross-user isolation integration test.
958+
- **`#509`** — Board view auto-switches between boards every few seconds: `boardStore` overwrites `activeBoardId` on each `fetchBoards` response. Add a `preserveSelection` guard so the active board is not reset while it still exists in the refreshed list.
959+
960+
Additional P1 issues from the same session (tracked in `#510``#515`) cover excessive board polling, the missing Inbox capture button, chat not emitting proposals, delete-card without confirmation, dark-mode theming gaps on three surfaces, and text-selected cards being non-draggable. Full findings at `docs/analysis/2026-03-29_manual_testing_consolidated_findings.md`.
961+
953962
## Next Best Steps (Immediate)
954963

955-
1. Close remaining unblocked Priority I security/policy work first (`#33`, `#34`, `#44`, `#152`) with regression coverage.
964+
1. **Resolve `#508` and `#509` (P0 blockers above) before any other backlog work.**
965+
2. Close remaining unblocked Priority I security/policy work first (`#33`, `#34`, `#44`, `#152`) with regression coverage.
956966
2. Run the manual-audit follow-through wave in trust-first order: `#364` -> `#365` -> `#368`, then align product truthfulness through `#366` and `#367`, while routing review-readability detail through `#326`; keep `#369` explicitly lower priority.
957967
3. Run the Saul-facing demo alignment wave as the next narrow product-facing slice: `#354` first, then legibility/demoability follow-through through `#326` and `#330`, then lock the recording contract in `#355` and `#216`.
958968
4. Continue the seeded novice-first shell tranche from `#322`, using the shipped `#320` home/workspace-mode foundation rather than reopening it.

docs/MANUAL_TEST_CHECKLIST.md

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,6 +84,7 @@ Manual-only checks (non-automatable in generic local script):
8484

8585
1. Register new user from `/register`.
8686
- Expected: redirected/authenticated into workspace.
87+
- **Data isolation check (fresh registration):** After registering, navigate to `/workspace/automation/queue`. Expected: 0 items. If items from previous dates appear, this is a P0 queue scoping failure (`#508`).
8788
2. Login with valid credentials from `/login`.
8889
- Expected: routed to `/workspace/home`.
8990
3. Attempt workspace route while logged out.
@@ -124,6 +125,7 @@ Manual-only checks (non-automatable in generic local script):
124125
- Expected: `count/limit` indicator visible.
125126
8. Attempt to exceed WIP by adding/moving cards.
126127
- Expected: operation blocked with visible error feedback.
128+
- **Known bug (`#517`):** As of 2026-03-29, the WIP limit is warning-only — the `+ Add Card` affordance is not disabled when the limit is exceeded. Check is marked failing until `#517` is resolved.
127129

128130
9. Create card inline.
129131
- Expected: card appears in target column.
@@ -134,7 +136,8 @@ Manual-only checks (non-automatable in generic local script):
134136
12. Move card to another column via drag/drop using the `Drag Card` handle.
135137
- Expected: card relocates and counts update.
136138
13. Delete card from modal.
137-
- Expected: card removed.
139+
- Expected: confirmation dialog shown first ("Delete this card? This cannot be undone."), then card removed on confirm.
140+
- **Known bug (`#513`):** As of 2026-03-29, deletion is immediate with no confirmation dialog.
138141

139142
14. Open label manager and perform create/update/delete.
140143
- Expected: label list and card chips reflect changes.

0 commit comments

Comments
 (0)