Skip to content

fix(mobile): cherry-pick upstream work-log and outbox fixes - #305

Merged
ronak-guliani merged 6 commits into
mainfrom
chore/cherry-pick-upstream-mobile-perf
Sep 6, 2026
Merged

fix(mobile): cherry-pick upstream work-log and outbox fixes#305
ronak-guliani merged 6 commits into
mainfrom
chore/cherry-pick-upstream-mobile-perf

Conversation

@ronak-guliani

Copy link
Copy Markdown
Owner

Summary

Testing

  • pnpm --filter @t3tools/mobile typecheck clean
  • pnpm --filter @t3tools/mobile test: 155 files, 1379 tests passed
  • pnpm lint: warnings only, no errors
  • APP_VARIANT=preview npx expo export --platform ios: bundles clean (18MB .hbc)
  • Simulator: launched installed Preview build on iPhone 17 Pro sim, confirmed app runs (screenshot); fresh dev-client rebuild blocked locally — Xcode 26.0.1 cannot compile Expo SDK 57 Swift (needs 26.4+, preflight gate). Recommend EAS simulator build or Xcode upgrade for on-device confirmation of new JS.
  • Pre-existing pnpm fmt:check failure in perf-improvements.html untouched by this branch.

juliusmarminge and others added 6 commits September 6, 2026 13:25
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…ing (pingdotgg#10210)

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings September 6, 2026 20:30
@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:L labels Sep 6, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The cherry-picked changes match their upstream fixes and include focused regression coverage.

Pull request overview

Cherry-picks six upstream mobile reliability and UI fixes.

Changes:

  • Improves work-log expansion, shimmer, spacing, sizing, and edge fades.
  • Resumes diff highlighting after oversized lines.
  • Retains queued tasks after transient delivery failures.
File summaries
File Description
apps/mobile/src/state/thread-outbox.test.ts Tests retry classification.
apps/mobile/src/state/thread-outbox-model.ts Classifies transient failures by tag.
apps/mobile/src/lib/threadActivity.ts Adds Thinking rows and improved work-row presentation.
apps/mobile/src/lib/threadActivity.test.ts Covers feed presentation behavior.
apps/mobile/src/features/threads/ThreadFeed.tsx Renders Thinking rows and passes fade colors.
apps/mobile/src/features/threads/thread-work-log.tsx Fixes expansion sizing and edge fades.
apps/mobile/src/features/threads/floating-working-control.tsx Corrects working-pill spacing.
apps/mobile/src/features/diffs/nativeReviewDiffHighlighter.ts Resumes highlighting after long lines.
apps/mobile/src/features/diffs/nativeReviewDiffHighlighter.test.ts Tests long-line highlighting behavior.
apps/mobile/src/components/AppSymbol.tsx Adds Android brain-icon fallback.
Review details
  • Files reviewed: 10/10 changed files
  • Comments generated: 0
  • Review effort level: Balanced

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@ronak-guliani
ronak-guliani merged commit f83d61b into main Sep 6, 2026
7 of 10 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.

3 participants