Skip to content

fix(mobile): render bracket-delimited LaTeX when math is enabled - #29708

Closed
shivamhwp wants to merge 1 commit into
mainfrom
issue-watch/29599
Closed

shivamhwp wants to merge 1 commit into
mainfrom
issue-watch/29599

Conversation

@shivamhwp

Copy link
Copy Markdown
Collaborator

Refs: #29599

What's fixed: With Render math enabled, mobile showed \(...\) and \[...\] formulas as raw text because its native parser only recognized dollar delimiters. This updates the vendored Enriched Markdown archive to parse all four delimiters, including the reported matrix and determinant. The parser change and regression fixtures are in pingdotgg/enriched-markdown#13.

Evidence: Android 13, POCO M3 Pro. Baseline 60aec28 shows raw bracket math while dollar math renders. The fixed build at 3b4c208 renders the same matrix, determinant and inline fraction after toggling Render math off and on; code and citations remain literal.

Before:

Android before

android-before.mp4

After:

Android after

android-after.mp4

Not shown: iPad rendering. The before/after iPad run is queued; this PR stays draft until it completes.

Implemented with Opus 5.5 / Claude Code. Integrated and verified with gpt-6.1-sol / Codex.

The native Markdown parser only recognized dollar delimiters, so formulas
written with \(...\) and \[...\] remained text on iOS and Android.

Use Enriched Markdown source 75316dbe, which recognizes all four forms,
keeps link destinations and titles out of math scanning, and includes the
rebuilt WASM parser. Code, escaped delimiters, citations and math disabled
keep their existing behavior.

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S 10-29 changed lines (additions + deletions). 📱 Native Change Changes the native fingerprint; merging blocks production OTAs until a new store build ships. labels Nov 2, 2026
@github-actions

github-actions Bot commented Nov 2, 2026

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 5.0 KiB 5.0 KiB 0 B (0.0%) 6.8 KiB ✅
Codex Thread snapshot wire 3.8 KiB 3.8 KiB 0 B (0.0%) 4.9 KiB ✅
Codex Live turn WebSocket wire 1.2 KiB 1.2 KiB 0 B (0.0%) 2.0 KiB ✅
Codex Live turn WebSocket decoded 21.6 KiB 21.6 KiB 0 B (0.0%) 24.4 KiB ✅
Codex Live turn messages 2 2 0 (0.0%) 8 ✅
Claude Total thread wire 5.0 KiB 5.0 KiB +29 B (+0.6%) 6.8 KiB ✅
Claude Thread snapshot wire 3.8 KiB 3.8 KiB 0 B (0.0%) 4.9 KiB ✅
Claude Live turn WebSocket wire 1.2 KiB 1.3 KiB +29 B (+2.3%) 2.0 KiB ✅
Claude Live turn WebSocket decoded 21.9 KiB 21.9 KiB +41 B (+0.2%) 24.4 KiB ✅
Claude Live turn messages 1 2 +1 (+100.0%) 8 ✅

Baseline: a036489 · PR result: 3b4c208 · 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: 108.5 KiB
  • Claude decoded thread snapshot: 108.8 KiB

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

@shivamhwp

Copy link
Copy Markdown
Collaborator Author

Closing as superseded by #32180 (merged), whose Enriched Markdown sync covers bracket-delimited LaTeX rendering.

@shivamhwp shivamhwp closed this Nov 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📱 Native Change Changes the native fingerprint; merging blocks production OTAs until a new store build ships. size:S 10-29 changed lines (additions + deletions). 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