Skip to content

feat(mobile): favorite models in thread settings - #790

Merged
rynfar merged 1 commit into
pylonfrom
upstream/2026-09-24-mobile-model-favorites
Sep 25, 2026
Merged

rynfar merged 1 commit into
pylonfrom
upstream/2026-09-24-mobile-model-favorites

Conversation

@rynfar

@rynfar rynfar commented Sep 24, 2026 •

Copy link
Copy Markdown
Collaborator

Mobile users could select models but could not keep frequently used models together. Add a separate favorite action to each model row, a Favorites filter on iOS and Android, and favorite-first ordering within provider groups. Favorites are stored per device and keyed by provider instance and model, so separate accounts do not collide.

Selectively ports T3 Code 298f8c95cd7c20ea7943a431f22e836fa637457b plus its required rapid-update correction 82cd1d1aabe8baa4a5ae23f3c7b08b3fce092e08. Pylon's existing search UI, model disabled reasons, provider options, and selection behavior remain in place; unrelated upstream Material layout changes are excluded. This does not synchronize favorites between devices, matching the existing device-local web setting.

Validation: 20 focused tests passed, including a real preference-store write, cold restart and sanitizer round trip; mobile typecheck and scoped lint passed (existing memo-dependency warnings). Independent review passed on the behavior diff 7e0b51e3cf9310a55ee76e15c3b224fdeeffd39a85d1efca4eee4b912ff386f1 and test-only delta 1d8bc517f3252f6c4291743775ac5bb0d6c7fe08a390e3fb1641e976fa0d4848. Native device UI has not been verified.

Model: GPT-6 Sol (low reasoning), via Codex harness; reviewed under GPT-6 Astra oversight.

@vercel

vercel Bot commented Sep 24, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated
pylon-marketing Ignored Ignored Preview Sep 24, 2026 9:50am UTC

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:L labels Sep 24, 2026
@rynfar
rynfar force-pushed the upstream/2026-09-24-mobile-model-favorites branch from 148f37a to 4040e37 Compare September 24, 2026 09:49
@github-actions

Copy link
Copy Markdown
Contributor

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 14.0 KiB 13.9 KiB −29 B (−0.2%) 15.1 KiB ✅
Codex Thread snapshot wire 7.2 KiB 7.2 KiB 0 B (0.0%) 7.3 KiB ✅
Codex Live turn WebSocket wire 6.8 KiB 6.7 KiB −29 B (−0.4%) 7.8 KiB ✅
Codex Live turn WebSocket decoded 58.0 KiB 58.0 KiB −44 B (−0.1%) 66.4 KiB ✅
Codex Live turn messages 10 9 −1 (−10.0%) 21 ✅
Claude Total thread wire 14.0 KiB 14.0 KiB −13 B (−0.1%) 15.1 KiB ✅
Claude Thread snapshot wire 7.2 KiB 7.2 KiB −10 B (−0.1%) 7.3 KiB ✅
Claude Live turn WebSocket wire 6.7 KiB 6.7 KiB −3 B (−0.0%) 7.8 KiB ✅
Claude Live turn WebSocket decoded 58.9 KiB 58.9 KiB −44 B (−0.1%) 66.4 KiB ✅
Claude Live turn messages 10 9 −1 (−10.0%) 21 ✅

Baseline: 7ed3f71 · PR result: 4040e37 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 115.6 KiB
  • Claude decoded thread snapshot: 116.4 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

@rynfar
rynfar merged commit a5a776d into pylon Sep 25, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant