Skip to content

fix(mobile): show missing Android symbol icons - #864

Merged
rynfar merged 1 commit into
pylonfrom
upstream/2026-09-26-android-agent-icon
Sep 27, 2026
Merged

rynfar merged 1 commit into
pylonfrom
upstream/2026-09-26-android-agent-icon

Conversation

@rynfar

@rynfar rynfar commented Sep 27, 2026 •

Copy link
Copy Markdown
Collaborator

Several symbols already used by Pylon Mobile rendered no icon on Android because AppSymbol had no mapping. This adds only the current Pylon callers: SF Symbols text.alignleft, mic, circle, and exclamationmark.circle, plus Material names merge and smartphone. Existing iOS behavior and shared symbol types remain unchanged.

This is a partial adaptation of T3 #12316. Its Agent behavior settings row does not exist in Pylon; that row remains a separate product choice. The text.alignleft mapping fixes Pylon's current file views. The upstream broad symbol-map/type rewrite is not needed for these callers. Revisit the Agent behavior row if Pylon adds that setting.

Validation: mobile typecheck, focused lint and format check, and a static six-symbol mapping assertion. Android visual verification is unavailable because device_list reports no Android SDK or emulator on this host; no device rendering is claimed.

Model: GPT-6 Sol. Harness: Pylon.

@vercel

vercel Bot commented Sep 27, 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 27, 2026 4:21am UTC

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S labels Sep 27, 2026
@rynfar
rynfar merged commit f0559dd into pylon Sep 27, 2026
19 checks passed
@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 14.0 KiB +7 B (+0.0%) 15.1 KiB ✅
Codex Thread snapshot wire 7.2 KiB 7.3 KiB +6 B (+0.1%) 7.3 KiB ✅
Codex Live turn WebSocket wire 6.7 KiB 6.7 KiB +1 B (+0.0%) 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 8 9 +1 (+12.5%) 21 ✅
Claude Total thread wire 14.0 KiB 14.0 KiB −17 B (−0.1%) 15.1 KiB ✅
Claude Thread snapshot wire 7.3 KiB 7.3 KiB −5 B (−0.1%) 7.3 KiB ✅
Claude Live turn WebSocket wire 6.8 KiB 6.7 KiB −12 B (−0.2%) 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 9 9 0 (0.0%) 21 ✅

Baseline: dd4e28b · PR result: 8941490 · 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.4 KiB

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

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

Labels

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