Skip to content

feat(mobile): add iPad keyboard command palette - #840

Draft
rynfar wants to merge 2 commits into
pylonfrom
feat/ipad-keyboard-command-palette-20260924
Draft

rynfar wants to merge 2 commits into
pylonfrom
feat/ipad-keyboard-command-palette-20260924

Conversation

@rynfar

@rynfar rynfar commented Sep 24, 2026 •

Copy link
Copy Markdown
Collaborator

On iPad, Pylon lacked a hardware-keyboard command palette and visible-thread shortcuts. Add a native palette for existing Pylon actions, Cmd-K and Cmd-1–9 ownership for the visible list, and an iOS Return-key preference backed by the native composer. The implementation keeps both Pylon thread lists, current routes and editor behavior, and guards late modal focus when closing.

Adapted upstream sources 26b8f985d85773e41a2ee637785f7c87d5c1a01e (#11679) and e33b710d5f1147993e52076a9e033d97ef4106ad (palette colors). Upstream hover/gesture-handler patches, legacy-list retirement, app-link schemes, and broad lockfile churn are excluded. Android can reach the keyboard settings route through a deep link, but it does not show the iOS-only Return controls. No Oh My Pi integration is included.

Validation: 98 focused mobile tests, mobile typecheck, scoped lint/format/diff checks, and an unsigned iPad simulator build with the pinned Xcode tooling. Independent review passed on exact head 03ff2a68be27152c1a5d2a627c4a3966c382d466. Browser/device interaction and before/after captures were not performed; the native compile is not runtime acceptance.

Implemented by GPT-6 Sol through the Codex harness.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@vercel

vercel Bot commented Sep 24, 2026

Copy link
Copy Markdown

Deployment failed for project pylon-marketing with the following error:

Resource is limited - try again in 24 hours (more than 100, code: "api-deployments-free-per-day").

Learn More: https://vercel.com/pylon-code?upgradeToPro=build-rate-limit

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XXL labels Sep 24, 2026
@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 −33 B (−0.2%) 15.1 KiB ✅
Codex Thread snapshot wire 7.2 KiB 7.2 KiB +2 B (+0.0%) 7.3 KiB ✅
Codex Live turn WebSocket wire 6.8 KiB 6.7 KiB −35 B (−0.5%) 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 +15 B (+0.1%) 15.1 KiB ✅
Claude Thread snapshot wire 7.2 KiB 7.2 KiB −11 B (−0.1%) 7.3 KiB ✅
Claude Live turn WebSocket wire 6.7 KiB 6.8 KiB +26 B (+0.4%) 7.8 KiB ✅
Claude Live turn WebSocket decoded 58.9 KiB 58.9 KiB 0 B (0.0%) 66.4 KiB ✅
Claude Live turn messages 10 10 0 (0.0%) 21 ✅

Baseline: 7ed3f71 · PR result: 03ff2a6 · 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.7 KiB
  • Claude decoded thread snapshot: 116.3 KiB

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

This branch has not been deployed

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

Labels

size:XXL 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