Skip to content

Select Bug 234 recent file removal as next task #1089

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
SoundBlaster merged 1 commit into from
Dec 12, 2025
Merged

Select Bug 234 recent file removal as next task #1089

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
29 changes: 29 additions & 0 deletions DOCS/INPROGRESS/234_Remove_Recent_File_From_Sidebar.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,29 @@
# Bug #234 — Remove Recent File from Sidebar

## 🎯 Objective
Deliver an in-app affordance to remove individual entries from the Recent Files sidebar list so users can declutter or protect sensitive filenames without clearing all history.

## 🧩 Context
- Bug report describes missing removal action and clutter/privacy concerns (`DOCS/BUGS/234_Remove_Recent_File_From_Sidebar.md`).
- No blockers noted in current planning artifacts; task marked implementation-ready in `next_tasks.md` and `Current_State.md`.
- Expected touchpoints: sidebar SwiftUI list rendering recent files, persistence layer that maintains MRU history, and any analytics/logging hooks for sidebar interactions.

## ✅ Success Criteria
- Sidebar exposes an accessible “Remove from Recent” action (e.g., context menu or inline button) for the selected entry.
- Removing an entry updates the UI immediately and persists across app relaunches without affecting other recents.
- Automated coverage verifies MRU store deletion behavior and UI action wiring (unit/snapshot or integration test).

## 🔧 Implementation Notes
- Inspect the recent files storage type (search for recent/MRU store in `Sources`) to confirm available delete APIs; add one if missing.
- Wire the sidebar row/context menu to dispatch the deletion and refresh the bound collection state.
- Ensure keyboard/accessibility pathways (Delete key, VoiceOver labels) are considered and logged in tests or notes.
- If telemetry exists for sidebar actions, emit a deletion event consistent with existing analytics patterns.

## 🧠 Source References
- [`DOCS/BUGS/234_Remove_Recent_File_From_Sidebar.md`](../BUGS/234_Remove_Recent_File_From_Sidebar.md)
- [`DOCS/INPROGRESS/next_tasks.md`](./next_tasks.md)
- [`DOCS/INPROGRESS/Current_State.md`](./Current_State.md)
- [`DOCS/RULES`](../RULES)
- [`DOCS/AI/ISOInspector_Execution_Guide/04_TODO_Workplan.md`](../AI/ISOInspector_Execution_Guide/04_TODO_Workplan.md)
- [`DOCS/AI/ISOViewer/ISOInspector_PRD_TODO.md`](../AI/ISOViewer/ISOInspector_PRD_TODO.md)
- Archives under [`DOCS/TASK_ARCHIVE`](../TASK_ARCHIVE)
8 changes: 3 additions & 5 deletions DOCS/INPROGRESS/Current_State.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,15 +1,13 @@
## Project State Report

### Current Active Tasks
- **Bug #234 – Remove Recent File from Sidebar**: implementation PRD stub created; focus on adding the sidebar MRU removal affordance and persistence update.
- **Bug 246 — NavigationSplitView width overflow**: documented reproduction, hypotheses, and diagnostics plan remain open for macOS window sizing constraints when sidebar, detail, and inspector are visible.



### Next Selected Tasks
- Task queue currently lists **no active items**; next selections should follow `SELECT_NEXT.md`.


- Ready candidate: **Bug #234 – Remove Recent File from Sidebar** (implementation-ready).
- Task queue currently lists **no active items** beyond the in-progress Bug #234; future selections should follow `SELECT_NEXT.md` once capacity frees up.


- Recently resolved and no longer active: **Bug #235 – Smoke tests Sendable violations** (fix landed, smoke filters green).
Expand Down Expand Up @@ -42,7 +40,7 @@


### Recommended Updates
- Update `next_tasks.md` once a new unblocked item is selected (Bug 234 is ready) and after resolving Bug 246 or scheduling T5.4 on macOS hardware to keep queue accuracy.
- Keep `next_tasks.md` synchronized as Bug 234 progresses and after resolving Bug 246 or scheduling T5.4 on macOS hardware to maintain queue accuracy.


- Consider refreshing `todo.md` to reflect current prioritization around documentation linting, FoundationUI integrations, NavigationSplitView fixes, and the macOS benchmark so completion metrics track actionable work.
Expand Down
6 changes: 3 additions & 3 deletions DOCS/INPROGRESS/next_tasks.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,9 @@ _No active items in this queue. Select the next task per `DOCS/COMMANDS/SELECT_N

## 1. UI Defects & Experience Fixes

1. **Bug #234 – Remove Recent File from Sidebar** _(Ready for implementation — `DOCS/INPROGRESS/234_Remove_Recent_File_From_Sidebar.md`)_
- Add the MRU removal affordance in the sidebar along with analytics/logging hooks described in the spec.
- Ensure recents persistence updates and DocumentSessionController wiring reflect removals immediately.
1. **Bug #234 – Remove Recent File from Sidebar** _(In Progress — see `DOCS/INPROGRESS/234_Remove_Recent_File_From_Sidebar.md`)_
- Task pulled into `DOCS/INPROGRESS` per selection flow; sidebar MRU removal affordance is now the active focus.
- Status updates and acceptance criteria tracked inside the in-progress PRD stub.
3. **Bug #235 – Smoke tests blocked by Sendable violations** _(Resolved — `DOCS/INPROGRESS/235_Sendable_SmokeTest_Build_Failure.md`)_
- Strict-concurrency build now passes after sendable annotations and document-loading refactor; smoke filters are green.

Expand Down