Add search to the Starred and Today's 5 tabs

[sohamM97]

Search was only reachable from All Tasks, so finding a task from either of the other two tabs meant switching tabs first — friction for what is a global action. All three tabs now open the same picker, and All Tasks was refactored onto the shared helper rather than left as a second implementation that could drift.

Behavior

• Search icon added to the Starred and Today's 5 app bars, in the same slot as All Tasks (right after the profile icon).
• Selecting a result opens the task in All Tasks — in place when already there, otherwise drilling in and sliding to that tab, reusing the onNavigateToTask callback both tabs already had for "Go to task".
• Create-from-empty-search files at root with the Inbox toggle on, deliberately not auto-starring from Starred (unlike that tab's + FAB) nor auto-pinning from Today's 5 (unlike its FAB create flow). Search spans everything, so it makes no assumption about where the result belongs.
• Duplicate match offers Open rather than creating a duplicate.
• Today's 5 search is the global picker — it includes non-leaf parents, unlike the leaves-only "Pin a task to Today's 5" browse picker on the + FAB.

Responsive app bar title

Manual testing surfaced a second problem: the 4th action icon left the 30px "Task Roulette" line ellipsised to "Task Rou…" at phone width. Both tabs built that two-line title independently, so it moved into a shared TabAppBarTitle that steps down to 22px below 420dp, keeping ellipsis as a last-resort fallback. All Tasks is untouched — its title is already 22px and single-line.

Testing

• flutter analyze clean; 1559 tests pass (1 pre-existing skip).
• +14 tests across starred_screen_test.dart, todays_five_screen_test.dart, task_list_screen_test.dart — covering that the pool is global (an unstarred / non-leaf result navigates), that Today's 5 search opens rather than pins, the pin-toggle gate at max pins, and that created tasks land at root un-starred / un-pinned.
• One test intentionally stops at Cancel instead of committing an add: under FakeAsync + databaseFactoryFfiNoIsolate, the reload a full Today's 5 triggers holds sqflite's lock while waiting on timers pumpAsync never advances, so tearDown's db.reset() blocks forever. The same flow completes in ~2s against the real factory, and create-at-root is covered by the neighbouring test.
• Manual testing: 4 sections / 22 steps all passing on the Linux dev build — Starred search, Today's 5 search, All Tasks regression (the refactor risk), and app bar layout at desktop + narrow width. Not verified on-device; the responsive title renders at 22px on phone, un-truncated but not visually reviewed there.

Docs

• UI_VIEWS.md — search entry rewritten for all three tabs, responsive title documented, plus the All Tasks breadcrumb bar, which was missing entirely (the back arrow pops one level; the breadcrumb's "Task Roulette" crumb is the actual jump-to-root control).
• TEST_COVERAGE.md updated.

Second commit (unrelated to the feature)

c1c0dae tightens two skills from feedback during this round: /manual-test gains four wording rules (no imperative-voice summaries, negative outcomes must include the navigation to verify them, "go to root" is the breadcrumb not the back arrow, and always ask before recommending a /cleardb), and /cleardb gains a worktree pre-flight — every checkout shares one DB, and the app running during this round belonged to a different worktree, so killing it would have hit another session's data.

🤖 Generated with Claude Code