test: colocate session popup input tests

[zatevakhin]

Scope

This publishes the corrected, independently approved two-commit Phase 10 Session Select popup test-placement range on exact base refactor/components@22e12dcde17812a02465cdc3fbb09759e738644b:

• 6334afe5e3d408fadb555d5991e1fce81f094c5a — test: colocate session popup input tests
• ccff072aadb29835b284382311b2de6a203596a7 — test: preserve session popup toggle routing

The aggregate changes only src/features/sessions/input/popup.rs and src/runtime/mod.rs (+318/-239). It moves pure movement, filtering, group-collapse, delete-no-op, and typed popup-result coverage to the owner while retaining root adapter/routing/effect coverage in runtime. There are no production, seam, command/effect, routing, dependency, workflow, or user-visible behavior changes.

Boundary and metrics

• Owner popup tests: 6 -> 10, with zero App::new() fixtures.
• Runtime Session popup tests: 39 -> 29.
• Runtime popup App::new() fixtures: 39 -> 29.
• Full suite: 1,176 -> 1,170.
• Root coverage remains for RenderState-derived page steps; ToggledGroup; local, known-remote, and missing-remote OpenSession; local Delete and remote Dismiss; Ctrl-O request/collapse; Load More command construction; popup close; disconnected Ctrl-N; delegate-tab and Tab/BackTab routing; global chords; and New Session behavior.

Review correction

The initial Linus review found that ordinary Enter-on-header ToggledGroup adaptation was no longer covered at the root. The separate second commit restores popup_header_toggle_routes_through_root_adapter, covering collapse/expand, exact empty effects, popup retention, cursor clamping, and separation from start-page collapse state. Corrected aggregate Linus re-review is Approved with no findings.

Validation

Publication gates pass at exact head ccff072aadb29835b284382311b2de6a203596a7:

• cargo fmt --all -- --check
• cargo check --all-targets --all-features
• cargo clippy --all-targets --all-features -- -D warnings
• cargo build --all-targets --all-features
• all-target/all-feature debug and release suites: 1,170/1,170, zero failed/ignored
• default suite: 1,170/1,170
• focused owner 10/10, runtime Session popup 29/29, handlers 49/49, application 62/62
• aggregate scope, git diff --check, and protected-path audits

REFACTOR.md remains ignored/local-only and is excluded from this PR. AGENTS.md, CLAUDE.md, and stuff/ are untouched and excluded. Live ACP/backend and interactive terminal UI smoke remain unavailable and are not claimed.

The bounded New Session owner test-placement slice is explicitly out of scope and has not started.

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: dea16552-69eb-4d03-adc2-c3237aef43e6

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

_{Comment @coderabbitai help to get the list of available commands.}