[olafura]

features/timeline/plans-and-subagents.feature still had 77 scenarios tagged @backlog or @backlog-<surface>. They cover what upstream added in pingdotgg/t3code#2829 and #13572: plan follow-ups, saving plans, long-plan previews, the task summary, and subagent titles and counts. Neither the desktop nor the phone had these.

Fix

• Desktop timeline: a plan longer than 10 lines shows its first 10 lines, an ellipsis and Expand plan. Collapse plan brings the preview back. Plans that fit in the preview show whole, with no button (TimelineModel::planPreview, and the PreviewRole and ExpandedRole roles).
• Above the desktop composer:
  • a Plan ready banner. Implement sends the plan; typing feedback turns it into refining the plan instead. The overflow menu has implement in a new thread, copy, download and save.
  • a task summary (done count, current step), which opens to every step with its status and how long it took.
• Plan save:
  • asks for a path inside the workspace;
  • keeps the dialog open with the MC's reason when the save is refused;
  • says where the file went when it succeeds.
• Phone: subagent group counts and titles use the same display rules as the desktop (subagents::displayTitle). Steps were added to the phone's tst_Features for the shared bricks.
• Ledger: the plan-preview scenarios are @desktop @mobile. Two delegated-task scenarios stay @backlog-mobile (one also @backlog-tui) until those surfaces gain them (Phone and TUI: delegated-task results waiting on the parent #1070). Two expectations changed to match upstream and the implementation: a plan of 1,000 characters on 5 lines shows whole (the preview counts lines), and a saved plan's file name follows upstream's sanitizePlanFileSegment (fix-cartts-finally.md).

Several commits carry over upstream code, so they credit its authors (Julius Marminge, maria, Ymit24) with Co-authored-by. One gap: Theo Browne should also have been credited on a TUI commit that already merged through #1053, and that history can't be rewritten now.

Screenshots

On main, none of these exist. After this PR, taken from the desktop tst_Features bricks offscreen:

Long plan preview	Plan banner above the composer	Task summary, opened

Verification

• Desktop tst_Features on timeline/plans-and-subagents.feature: 130 passed, 0 failed.
• Phone tst_Features: 57 passed, 0 failed. The phone's terminal feature also passes: 95 passed, 0 failed.
• TUI runner: 34 passed, 0 failed.
• mix features timeline/plans-and-subagents.feature: 11 passed.

Code changes are in the desktop and the phone. The TUI and MC runs confirm their scenarios in this file still pass, with no changes to either. No contract changes. The plan actions are reachable from both the banner and the plan card.

Done by Claude Opus 5.5 (1M context) in Claude Code, running in HAL-C2.

🤖 Generated with Claude Code

[olafura]

Approved

1 finding: 1 low.

Brings the desktop plan card's long-plan preview with Expand/Collapse, the Plan ready banner with implement/refine, the task summary above the composer, a stricter plan save to the workspace, and phone subagent titles and counts in line with the ledger. The code does what it says, and I found no correctness bugs. The main risk is the description, not the code. It says ThreadStore now passes pending background tasks to the timeline, and it lists the TUI and the MC as surfaces. The diff touches no ThreadStore, TUI or MC code. Either that change is missing from the branch, or the description should drop the claim. The PR also rewrites two ledger expectations without mentioning it: the 1,000-characters-on-5-lines example now shows the whole plan, and the file name is fix-cartts-finally.md. Both match the implementation and upstream's sanitizePlanFileSegment, but they belong in the description.

Not run

• desktop and phone tst_Features, and tst_PlanSaveProp: This checkout has no Qt build directory, and building the Qt clients for a review is too heavy. The PR reports these passing.

Ran 4 checks

• git diff b4a1d153..HEAD (read every source, QML and feature change): Reviewed TimelineModel planPreview, PreviewRole and ExpandedRole, toggle, PlanController save and workspaceRoot, ComposerController turnState tasks and planAction, the TurnRequests banner, task card and save dialog, and the phone TurnAgents and steps
• grep of the MC's turn_writer.ex and the contracts' OrchestrationV2PlanStep: The todo_list plan's runId, status, steps text/status and durationMs match what turnState reads
• grep of McClient::call: A disconnect or not-ready state fails pending calls, so PlanController's m_saving always clears
• git diff --stat b4a1d153..HEAD: No ThreadStore, apps/tui or apps/server-ex changes, unlike the description says

[olafura]

Approved

No findings.

The PR adds the plan preview, Plan ready banner, task summary and plan save to the desktop, and gives the phone the desktop's subagent display rules. The one new commit since my last review (bdaf6ea) fixes my previous finding. The two delegated-task scenarios at features/timeline/plans-and-subagents.feature:317 and :343 are tagged @backlog-mobile again, and :343 also @backlog-tui, so the ledger still records that the phone and TUI lack them. The tags now name @mobile and @tui as surfaces, which follows the @desktop @mobile @backlog-mobile form in features/README.md. I found no new problems in this commit.

Not run

• tst_Features, TUI runner and mix features on plans-and-subagents.feature: The new commit only changes tags. The @backlog-* tags keep the phone and TUI runners from running these scenarios, and the desktop and MC tagging did not change, so a run would not settle anything a read doesn't.

Ran 2 checks

• git diff 46dee58f8 HEAD: One commit; it changes only the tags on two scenarios in features/timeline/plans-and-subagents.feature.
• git show b4a1d1531:features/timeline/plans-and-subagents.feature (original tags) and grep features/README.md for the backlog tag form: The new tags keep the base's backlog markers for the phone and TUI and follow the form features/README.md documents.