Skip to content

Fix: My Work listed tasks from draft/completed/cancelled plans - #53

Merged
SylonZero merged 1 commit into
masterfrom
claude/roadmap-plans-review-8t8gri
Aug 8, 2026
Merged

Fix: My Work listed tasks from draft/completed/cancelled plans#53
SylonZero merged 1 commit into
masterfrom
claude/roadmap-plans-review-8t8gri

Conversation

@SylonZero

Copy link
Copy Markdown
Owner

Bug

"My Open Tasks" on /my-work filtered only by task status (!== 'done'), never checking the owning plan's status. Result: a not-started task on a draft plan you never activated — or a leftover open task on a completed/cancelled plan — showed up in your personal to-do list as if it were live work.

Fix

getTasks already inner-joins code_plans (needed for planTitle), so codePlans.status was one field away — added it to the select and the returned TaskWithContext as planStatus. /my-work now filters myTasks to planStatus === 'active' in addition to the existing status !== 'done'.

The /tasks page is untouched — it's the comprehensive review surface and is meant to show tasks across every plan status; only the personal "what should I actually be doing" view needed the filter.

Tests

Two regression assertions in getTasks coverage: planStatus is present and correct for a task on the active fixture plan and for one on the completed fixture plan (uses existing seed data, no new fixtures needed). 207 tests passing (up from 206); typecheck and production build clean.

🤖 Generated with Claude Code

https://claude.ai/code/session_01Y7FhGxYjnJHJ7m9MpH2dgJ


Generated by Claude Code

'My Open Tasks' filtered only by task status, not by the owning plan's
status — so a not-started task on a draft plan (never activated) or a
leftover open task on a completed/cancelled plan showed up as if it
were live work.

getTasks now selects codePlans.status as TaskWithContext.planStatus
(the query already joins code_plans; this was just missing from the
select/return), and My Work filters to planStatus === 'active'. The
Tasks page is unaffected — it's the comprehensive review surface and
intentionally shows tasks across all plan statuses.

Regression tests: planStatus is present and correct for tasks on
active and completed plans (207 total, up from 206).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Y7FhGxYjnJHJ7m9MpH2dgJ
@SylonZero
SylonZero merged commit 6e12250 into master Aug 8, 2026
1 check passed
@SylonZero
SylonZero deleted the claude/roadmap-plans-review-8t8gri branch August 8, 2026 04:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants