releases/ClaudeDo
2
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases 3 Wiki Activity

feat(ui): planning sessions UI (Plan C) #5

Merged
mikakuns merged 9 commits from feat/planning-sessions-ui into main 2026-04-23 17:38:09 +00:00
Conversation 0 Commits 9 Files Changed 20 +823 -126
claude commented 2026-04-23 17:36:24 +00:00
Owner
Copy Link

Summary

Wires the planning-session feature into the UI (Plan C of the planning-sessions design).

  • TaskRowViewModel gains hierarchy flags (ParentTaskId, IsChild, IsPlanningParent, IsExpanded, PlanningBadge, IsDraft) and context-menu gates (CanOpenPlanningSession, CanResumeOrDiscardPlanning).
  • TasksIslandViewModel gets five [RelayCommand] methods (Open/Resume/Discard/Finalize/ToggleExpand) and a hierarchy-aware Regroup() that interleaves children after their planning parent when expanded.
  • WorkerClient adds five hub-method wrappers (StartPlanningSessionAsync, ResumePlanningSessionAsync, DiscardPlanningSessionAsync, FinalizePlanningSessionAsync, GetPendingDraftCountAsync) plus DTOs. Hub method names are string-based, so no build-time dependency on Plan B.
  • TaskRowView grows an indent column, chevron toggle, DRAFT/PLANNING badges, and italic/faded styling for drafts.
  • New UnfinishedPlanningModal (Resume / Finalize now / Discard / Cancel) triggers when the user right-clicks a Planning parent.
  • Delete-with-children now surfaces a friendly error dialog instead of bubbling a DbUpdateException.

Test plan

  • dotnet build for Worker / Ui / App — clean
  • 4 new TaskRowViewModelPlanningTests + 4 new TasksIslandViewModelPlanningTests — 8/8 pass
  • Pre-existing suite state preserved (73 pass / 86 pre-existing no such table: lists failures unchanged, predating Plan C)
  • End-to-end manual smoke (requires Plan B merged): create Manual task -> Open planning Session -> drafts appear under parent -> finalize -> drafts become Manual/Queued; right-click Planning task -> modal -> Resume/Finalize/Discard; delete parent with children -> friendly error.

Known followups (non-blocking, tracked in docs/open.md)

  • Border.badge.planned style defined but unreachable -- Planned status renders with the amber planning class. Either make TaskRowView swap Classes.planned when Status == Planned, or remove the unused style + brush.
  • Dead Instance statics on BoolToItalicConverter / BoolToDraftOpacityConverter.
  • Regroup() covers OverdueItems/OpenItems/CompletedItems correctly, but a Planning parent that lands in OverdueItems will be visually separated from its draft children (which go to OpenItems). Deterministic; spec is silent on this corner.

Co-Authored-By: Claude Opus 4.7 (1M context) noreply@anthropic.com

## Summary Wires the planning-session feature into the UI (Plan C of the planning-sessions design). - `TaskRowViewModel` gains hierarchy flags (`ParentTaskId`, `IsChild`, `IsPlanningParent`, `IsExpanded`, `PlanningBadge`, `IsDraft`) and context-menu gates (`CanOpenPlanningSession`, `CanResumeOrDiscardPlanning`). - `TasksIslandViewModel` gets five `[RelayCommand]` methods (Open/Resume/Discard/Finalize/ToggleExpand) and a hierarchy-aware `Regroup()` that interleaves children after their planning parent when expanded. - `WorkerClient` adds five hub-method wrappers (`StartPlanningSessionAsync`, `ResumePlanningSessionAsync`, `DiscardPlanningSessionAsync`, `FinalizePlanningSessionAsync`, `GetPendingDraftCountAsync`) plus DTOs. Hub method names are string-based, so no build-time dependency on Plan B. - `TaskRowView` grows an indent column, chevron toggle, DRAFT/PLANNING badges, and italic/faded styling for drafts. - New `UnfinishedPlanningModal` (Resume / Finalize now / Discard / Cancel) triggers when the user right-clicks a Planning parent. - Delete-with-children now surfaces a friendly error dialog instead of bubbling a `DbUpdateException`. ## Test plan - [x] `dotnet build` for Worker / Ui / App — clean - [x] 4 new `TaskRowViewModelPlanningTests` + 4 new `TasksIslandViewModelPlanningTests` — 8/8 pass - [x] Pre-existing suite state preserved (73 pass / 86 pre-existing `no such table: lists` failures unchanged, predating Plan C) - [ ] End-to-end manual smoke (requires Plan B merged): create Manual task -> Open planning Session -> drafts appear under parent -> finalize -> drafts become Manual/Queued; right-click Planning task -> modal -> Resume/Finalize/Discard; delete parent with children -> friendly error. ## Known followups (non-blocking, tracked in docs/open.md) - `Border.badge.planned` style defined but unreachable -- Planned status renders with the amber `planning` class. Either make `TaskRowView` swap `Classes.planned` when `Status == Planned`, or remove the unused style + brush. - Dead `Instance` statics on `BoolToItalicConverter` / `BoolToDraftOpacityConverter`. - `Regroup()` covers `OverdueItems`/`OpenItems`/`CompletedItems` correctly, but a Planning parent that lands in `OverdueItems` will be visually separated from its draft children (which go to `OpenItems`). Deterministic; spec is silent on this corner. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
claude added 9 commits 2026-04-23 17:36:24 +00:00
feat(ui): TaskRowViewModel gains planning hierarchy flags 229d4bbb2b 
Adds ParentTaskId, IsExpanded, IsChild, IsPlanningParent, IsDraft, and
PlanningBadge to TaskRowViewModel with property-changed notifications.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
feat(ui): WorkerClient planning-session methods 00608401aa
feat(ui): planning commands and expand/collapse in TasksIslandViewModel 309f84b388 
- Add IWorkerClient interface; WorkerClient implements it
- TasksIslandViewModel accepts IWorkerClient? and gains OpenPlanningSession,
  ResumePlanningSession, DiscardPlanningSession, FinalizePlanningSession,
  and ToggleExpand commands
- Regroup() is hierarchy-aware: children of collapsed planning parents are hidden
- InternalsVisibleTo ClaudeDo.Worker.Tests for Regroup()
- 4 new unit tests covering collapse/expand and guard logic

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
feat(ui): TaskRowView hierarchy indentation, chevron, badges, draft italic 42b208ff28 
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
feat(ui): planning entries in task context menu 388a8c1fae
feat(ui): draft and planning badge styles 506caa2c53
feat(ui): unfinished planning session dialog 47b49743c0 
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
feat(ui): friendly error when deleting task with children 0e116bec7b
docs(open): add planning-session manual verification checklist 450e685580
mikakuns merged commit 993851009b into main 2026-04-23 17:38:09 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
claude
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: releases/ClaudeDo#5
Delete Branch "feat/planning-sessions-ui"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?