Skip to content

Phone Chats: render Remote Host sections and Pinned list #2571

New issue
New issue
Closed
Closed
Phone Chats: render Remote Host sections and Pinned list#2571
Labels
ready-for-agentFully specified, ready for an AFK agent

Description

@DoodleBears
DoodleBears
opened on Oct 22, 2026

Part of #2517. Depends on #2552 (row actions and canonical Remote Host section creation).

Human screenshot feedback (2026-10-23) supersedes the grid presentation from #2574; the follow-up is #2618. Pinned should be a list like other sections so names and message state are easier to scan.

First runnable slice

Render the selected Remote Host roster as its actual section/top-level arrangement. Pinned is a collapsible, non-sticky list above ordinary sections, using the same full conversation rows with name, preview, timestamp, unread and working indicators. Pin retains original membership and appears only once; unpin returns to the ordinary section. Hide empty Pinned in browsing. Respect Host global/per-scope manual/latest/inherit ordering and loose rows between section blocks. Only collapse preferences are device-local and scoped by Host.

Search remains flat full conversation rows including pinned matches, title matches before pin rank. Clearing restores Host arrangement. All retains the current Host organization with Cloud conversations in their existing separate scope; no multi-Host aggregation or cross-origin writes.

Preserve #2552 row swipe/menu, iOS preview and section creation. Do not change its pin-preserving assignment semantics in this slice. Organize/drag and move-as-unpin-and-assign are later slices after Human visual feedback. #2554 layout/fixture work remains independently owned.

Acceptance

  • Source regressions: pins without custom sections; membership/nonduplication/unpin; Host manual/latest/inherit/topOrder; loose rows; empty/collapsed sections; Host-scoped collapse; search and All/Cloud separation; full pinned-row preview/unread display and long-list order.
  • Virtualize each ordinary and pinned conversation row; keep accessible equivalent operations and scalable text.
  • Carry accepted glossary/target design precisely; distinguish this implementation from pending full target/native acceptance.
  • Native iOS/Android default/intermediate/maximum text × light/dark, real row operations and Human visual feedback through the centralized test center. Source tests are not geometry evidence.

No independent test/build/Host/Metro/device runner. No merge/release.

Activity

  1. DoodleBears
    added
    ready-for-agentFully specified, ready for an AFK agent
    on Oct 22, 2026

  2. DoodleBears commented on Oct 22, 2026

    @DoodleBears
    DoodleBears
    on Oct 22, 2026
    ContributorAuthor

    Agent work claim

    Existing claims and hub sub-issues inspected. No matching presentation slice existed. All heavy checks and native work are owned by the centralized test center.

  3. DoodleBears
    added a parent issue on Oct 22, 2026
  4. DoodleBears
    added
    agent:activeOne or more coding-agent tasks currently claim this issue; see claim comments for task IDs
    on Oct 22, 2026

  5. DoodleBears commented on Oct 22, 2026

    @DoodleBears
    DoodleBears
    on Oct 22, 2026
    ContributorAuthor

    Implementation status for this task claim: source prepared; awaiting centralized checks and native QA allocation.

    • Draft PR: feat(mobile): render Host sections and responsive Pinned chats #2574, stacked on feat(mobile): swipe Chats rows and create Host sections #2552.
    • Exact head: da0a53c937e5ef8ccd00b0ab573d8d599a05c1e9 (t3/remote-pinned-grid).
    • Exact base: a0a3f17af55af20e48a8c4bb1d47430bda7b7b99 (t3/2519-chat-row-swipe), incorporating main 897e9334768baf26547eb155f9ab1b3e7a0ee61f.
    • Source includes canonical Host sorting/section projection, responsive virtualized Pinned bands, original membership preservation, per-Host collapse, flat search and Cloud separation, regression fixtures, bilingual fragment and the native feature journey.
    • Only source formatting and git diff --check were performed. No install, test, typecheck, build, Host, Metro or device execution occurred. Bounded tests and checks were submitted to the central test center, with parent copied.
    • Native before/after screenshots, iOS/Android interaction and geometry evidence, and Human visual feedback remain outstanding. fix(mobile): keep Chats titles and timestamps inline at large text #2554 layout/fixture follow-up stays independently owned; frozen QA was not touched.
    • An exact-head Chinese HTML explainer was generated and inspected at 728px and 360px. It is a local/T3 review aid, not native evidence; no reviewer-accessible GitHub attachment path is available yet.

    The full accepted target is documented; Organize/drag and the later unpin-plus-assign move operation are outside this first slice.

    Agent-Task: codex/local/01a16238-8539-71a1-8975-89042bd66eb7

  6. DoodleBears commented on Oct 22, 2026

    @DoodleBears
    DoodleBears
    on Oct 22, 2026
    ContributorAuthor

    Agent work claim

    • Agent: Codex
    • Task: codex/local/01a16587-3995-7b22-8c81-7c19caccddef (not a DSH Session)
    • Issue scope: successor integration of feat(mobile): render Host sections and responsive Pinned chats #2574 onto current main; repair conflicts and CI, normal merge after applicable checks, then hand off native qualification. No Organize/drag expansion.
    • Branch: codex/2574-pinned-grid-integration
    • Started: 2026-10-22 21:35 UTC
    • State: active

    The Human explicitly assigned this continuation and approved merging this PR after source checks and CI, before native QA. The original task codex/local/01a16238-8539-71a1-8975-89042bd66eb7 reported source prepared awaiting centralized checks; its durable host could not be confirmed, which is not evidence of abandonment. Its checkout and exact head da0a53c937e5ef8ccd00b0ab573d8d599a05c1e9 will be preserved. Parent #2552 is merged as 265cb6f5c995c689a834e6aeea763ac724844ce1, verified in current main. We will preserve PR history with a normal merge and recheck the remote head before a non-force push.

    Native/Human visual acceptance remains pending with the shared QA owner. Merge is not release acceptance and this issue remains open. No release, deployment, other PR merge, shared device takeover, or real account deletion is authorized by this claim.

  7. DoodleBears
    added a commit that references this issue on Oct 22, 2026

    Merge main integration for Host sections and repair CI fixtures

    05ba0bd

  8. DoodleBears commented on Oct 22, 2026

    @DoodleBears
    DoodleBears
    on Oct 22, 2026
    ContributorAuthor

    Agent work outcome — source merged; native qualification handed off

    • Task: codex/local/01a16587-3995-7b22-8c81-7c19caccddef
    • State: handed-off (source integration complete; native/Human acceptance pending).
    • feat(mobile): render Host sections and responsive Pinned chats #2574 merged normally as 0b71799a5326635b11bd1b102893b44de33b665f at 2026-10-22 21:46:55Z. Original head da0a53c937e5 remains an ancestor of tested head 05ba0bd873c21dddec4bdefd49e8587e04fe7b55 and merged main. No force push, branch deletion, admin bypass or original-checkout edits.
    • Parent feat(mobile): swipe Chats rows and create Host sections #2552 was already merged as 265cb6f5c995; PR base moved from the old stack to main. Two test conflicts preserved current row-menu/platform and Connect refusal coverage. CI fixes removed prohibited new comments and explicitly separated nine mutation calls from nine settings reads. No tests skipped or checks weakened.
    • CI39858838507 passed every applicable job and aggregate: static/type/build, three full test shards, mobile verification, Android module tests, docs and isolated Host smoke. Locally206 focused tests across12 files, root/mobile types, lint/format, ledgers/policy/map/parity passed. CodeRabbit was pending with no reviews/inline findings at merge; repository rules did not require it.
    • Main advanced with independently authorized fix(mobile): isolate Dev and store deep-link registration #2615 (20d00f03) before the merge. Final tree b2ef318c497fadcd58b18a67f69205e536e493c2 matches a fresh merge-tree of 20d00f03 and this tested head. Combined source includes fix(mobile): isolate Dev and store deep-link registration #2615 native scheme inputs: the next batch must qualify/rebuild those inputs, not assume JS-only. Final-main CI39859995264 was still running at handoff.
    • Final merged-main bounded regression:66 tests across HostClient/home-search and scheme/helper files PASS. This checks the overlapping merged source, not native route behavior.
    • Submitted exact merged main to the sole QA owner's next-batch queue, preserving the frozen account/Send batch. Journey: phone-pinned-grid (pins/no-custom/custom sections, membership and unpin return, ordering/collapse/search, two-Host and All/Cloud isolation, real row/card actions, keyboard/accessibility, width and default/intermediate/maximum text × light/dark, shared Web loading). Android remains a separate lane.
    • Native screenshots, device interaction, Human visual feedback and Release acceptance remain pending. The Human explicitly authorized source/CI merge before native checks for this PR. Keep Phone Chats: render Remote Host sections and Pinned list #2571 open. Organize/drag and unpin-plus-assign remain outside this slice.
    • No Host, Metro, Simulator, phone, driver or tunnel was launched by this task. Source test/compiler processes and the CI watcher have exited; shared QA resources were not changed. Source/evidence and exact-head local explainer retained; public explainer hosting and browser/native capture were not claimed.

  9. DoodleBears commented on Oct 23, 2026

    @DoodleBears
    DoodleBears
    on Oct 23, 2026
    ContributorAuthor

    Agent work claim

  10. DoodleBears
    changed the title [-]Phone Chats: render Remote Host sections and responsive Pinned grid[/-] [+]Phone Chats: render Remote Host sections and Pinned list[/+] on Oct 23, 2026

  11. DoodleBears commented on Oct 23, 2026

    @DoodleBears
    DoodleBears
    on Oct 23, 2026
    ContributorAuthor

    Agent work outcome — Pinned list follow-up

    • Task: codex/local/01a16587-3995-7b22-8c81-7c19caccddef
    • State: handed-off for native/Human review.
    • Draft PR fix(mobile): use full conversation rows in Pinned #2618, head c0106bfed0d8dffd1851b56fd3a8f031551f132a, replaces mobile grid cards with the same full conversation rows used by other sections. Keeps canonical pin order/membership, collapse/search and inverse actions. Issue target/design/journey now reflect the Human screenshot feedback.
    • 69 focused local tests and mobile types/policy/format checks PASS. CI39880968069 all applicable jobs/aggregate PASS, including the complete three-shard test suite. CodeRabbit status SUCCESS with no posted reviews/findings.
    • Native iOS/Android screenshots, large-text/accessibility interactions and Human acceptance remain pending in the centralized next-batch queue. Frozen Batch62/grid evidence preserved as baseline. No source-owned runtime started; no merge, device install or release performed. Phone Chats: render Remote Host sections and Pinned list #2571 remains open.
  12. DoodleBears
    removed
    agent:activeOne or more coding-agent tasks currently claim this issue; see claim comments for task IDs
    on Oct 23, 2026

  13. DoodleBears commented on Oct 23, 2026

    @DoodleBears
    DoodleBears
    on Oct 23, 2026
    ContributorAuthor

    #2618 merged and native QA dispatched — 2026-10-23

    Human explicitly authorized merge and native QA/screenshots. PR #2618 is MERGED at 76078cfc7fc551ae5a4f4b057153085355528dd3, from verified head c0106bfed0d8dffd1851b56fd3a8f031551f132a; first parent is unchanged tested base 5be050bef4dc9a07f02f6e3e1fae63c2c893d076. All applicable PR CI39880968069 checks and aggregate passed; CodeRabbit success with no review/inline findings. Normal merge, no admin bypass.

    Native request dispatched to sole existing owner codex/local/01a1653b-3345-7091-8284-34ceddb41e2c with exact merged source, Pinned list journey, matched grid/list screenshot matrix, privacy/publication rules and cleanup requirements. Submit main containing this merge in the next owner-cut batch. Preserve Batch62 evidence as grid baseline and the retained physical-phone Host/output dependencies. Source-only JS/UI change does not establish combined batch binary compatibility. iOS/Android/Human acceptance and screenshot publication remain pending until the owner records actual results; #2571 stays open. No release/deployment/phone install performed by the source task.

  14. DoodleBears
    closed this as completedon Oct 23, 2026
  15. jishi7
    added 2 commits that reference this issue on Oct 24, 2026

    feat(mobile): render Host sections and responsive pinned chats

    da0a53c

    fix(mobile): show pinned chats as full conversation rows

    c0106bf
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    ready-for-agentFully specified, ready for an AFK agent

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions