Task management: reachable delete, one-click complete, live dependency release

[Frailrain]

Two entwined stories from real use, plus a small usability follow-up after the first two menus landed on my instance.

The reachability story

Delete already existed end-to-end — DELETE /api/tasks/[id] aborts the turn, tears down the worktree and uploads, and cascades the DB — but the only UI path was TaskHero → Edit → EditTaskModal → Delete, and TaskHero unmounts once a task has a session. Every started task (including the seeded tutorial after its intended first Start) was un-deletable from the UI.

The dependency-trap story

Nothing in the UI ever set status='done' on its own. Turns settle, merge stamps merged_at, PRs open — but the only way to reach 'done' was digging into the status dropdown. With task dependencies in play this is a trap: blockerTitles clears a dependent's "Blocked by …" chip only for 'done' / 'cancelled', so a chain of tasks sits permanently blocked on a state nobody knew to produce. Hit in real use.

What's added

Commit 1 — Task delete: kebab in task rows, overflow menu in session header

• Kebab (…) on each non-suggested task row in TasksColumn.
• Overflow (…) in SessionView's header, next to the existing pickers.
• Both open a shared DeleteTaskModal mirroring EditTaskModal's two-step confirm + copy verbatim.
• Card converted from <button> to a keyboard-activatable <div> so the kebab can be a real nested <button>.

Commit 2 — Task completion: reachable "done", offered at merge & PR

• "Mark complete" button in the session header, next to the existing pickers. Hidden once done.
• After a successful Merge or first Create PR, TaskChanges surfaces an inline dismissible "Mark this task complete?" strip. Never modal. "Not yet" hides it. Suppressed on already-merged merges and PR updates (no new signal).

Commit 3 — Task management: card affordances + kebab contrast (usability feedback from actually using the first two)

• Circle-check on the task card — small check control in the bottom-right of each non-suggested, non-done card. One click completes via a new task-id-aware completeTask helper (setStatus is scoped to the selected task). Legal markup now that commit 1 already turned the card into a keyboard-activatable div.
• "Mark complete" in the kebab, above "Delete task…". Hidden for done tasks.
• Kebab contrast bump — the row kebab was opacity:0 at rest (visible only on hover) so I couldn't find it looking straight at it. Now opacity:.7 with --ink-3 at rest, full --ink-2 on hover. Still quieter than the title, but findable.
• Session-header overflow contrast — the iconless dots pill vanished next to labelled pickers. Bumped ink and tightened padding via a scoped .sh-more class so the neighbouring pickers stay unchanged.

Coordination with upstream 0001fe4

The wire-level "dependency release across tabs" plumbing (publish task_updated on status PATCH + task_deleted on DELETE, with useGlobalEvents consuming both and patching project counts locally) already exists on main via commit 0001fe4 ("Make the 'N need you' badge and its dropdown agree"). This PR is the UI half depending on it — every "Mark complete" affordance here PATCHes status and rides 0001fe4's existing task_updated publish path; every delete rides its task_deleted publish path. tests/needsYou.test.ts on main already pins that behavior, so no new server test comes with this PR.

Superseded branches

feat/task-delete (91f7d09) and feat/task-completion (d097a72) exist on the fork but are superseded by this combined PR. Both branches carried an early wire-level plumbing attempt (status_changed, refetch /api/projects after delete) that would have duplicated / conflicted with 0001fe4; that layer has been dropped from the combined branch, keeping only the UI pieces.

Verification

• npm test — 271/271 (37 files) including tests/needsYou.test.ts and everything else on main.
• tsc --noEmit clean for touched files.
• HTTP smoke against a booted server: subscribed to GET /api/events, PATCHed a task's status — captured data: {"type":"task","event":"task_updated",…,"status":"in_progress","awaiting_count":0} (upstream's coarse event).

Test plan (browser)

• Task-row circle-check on a non-done, non-suggested card marks the task done; the row moves to Done immediately.
• Row kebab → Mark complete does the same; Delete task… opens the confirm modal (two-step).
• Session header Mark complete button flips status and disappears; status pill reflects done.
• Session header … overflow → Delete task… opens the same confirm modal.
• Merge a task → "Mark this task complete?" strip appears in the Changes toolbar → clicking Mark complete flips status and hides the strip. "Not yet" also hides it.
• First-time Create PR → same strip. Update PR on an already-open PR → no strip.
• Dependency unblock: create task B depending on task A; mark A done from the row circle-check — task B's "Blocked by A" chip clears without a page reload.
• Open two tabs on the same project; mark done in one — the other's row moves to Done and any dependents unblock live (verifies useGlobalEvents picks up upstream's task_updated).
• Row kebab is now visible at rest (was invisible in commit 1); session-header … button reads as a picker even without a label.