feat(mobile): add Codex task parity#3654
Open
richardsolomou wants to merge 7 commits into
Open
Conversation
|
React Doctor found 2 issues in 2 files · 2 warnings. 2 warnings
Reviewed by React Doctor for commit |
Contributor
Prompt To Fix All With AIFix the following 2 code review issues. Work through them one at a time, proposing concise fixes.
---
### Issue 1 of 2
packages/core/src/sessions/cloudTaskSessionService.ts:402
**Legacy Resume Drops Task Controls**
When both the legacy run and its persisted composer config lack an adapter, this remains `undefined`. The mobile API client then omits the resolved model, reasoning, and permission mode with the adapter, so retry can restart an older Claude task using server defaults instead of its previous controls.
```suggestion
const adapter =
composerConfig?.adapter ?? previousRun?.adapter ?? "claude";
```
### Issue 2 of 2
packages/core/src/sessions/cloudTaskSessionService.ts:425-428
**Invalid Run Reasoning Reused**
When the composer reasoning is unsupported and the adapter has not changed, this fallback reuses `previousRun.reasoningEffort` without validating it. If that stored value is also unsupported by the resolved model, the API client rejects the payload and sandbox-inactive retry fails instead of resuming.
Reviews (1): Last reviewed commit: "feat(mobile): add Codex task controls" | Re-trigger Greptile |
richardsolomou
force-pushed
the
posthog-code/finalize-mobile-portability-boundary
branch
from
July 23, 2026 13:19
e917fbd to
0f867aa
Compare
richardsolomou
force-pushed
the
posthog-code/feat-mobile-codex-parity
branch
from
July 23, 2026 13:24
44c9001 to
515fcec
Compare
richardsolomou
force-pushed
the
posthog-code/finalize-mobile-portability-boundary
branch
from
July 23, 2026 13:44
0f867aa to
c01f356
Compare
richardsolomou
force-pushed
the
posthog-code/feat-mobile-codex-parity
branch
from
July 23, 2026 13:46
515fcec to
132174b
Compare
This was referenced Jul 23, 2026
Member
Author
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
This was referenced Jul 23, 2026
richardsolomou
force-pushed
the
posthog-code/finalize-mobile-portability-boundary
branch
from
July 23, 2026 16:34
e809724 to
ff81e50
Compare
richardsolomou
force-pushed
the
posthog-code/feat-mobile-codex-parity
branch
from
July 23, 2026 16:34
29a77af to
01d941c
Compare
richardsolomou
force-pushed
the
posthog-code/finalize-mobile-portability-boundary
branch
from
July 23, 2026 17:28
ff81e50 to
c630011
Compare
richardsolomou
force-pushed
the
posthog-code/feat-mobile-codex-parity
branch
from
July 23, 2026 17:28
01d941c to
d4a99d1
Compare
Generated-By: PostHog Code Task-Id: 11daf8d3-7830-436a-ab8f-99123586070d
Generated-By: PostHog Code Task-Id: 11daf8d3-7830-436a-ab8f-99123586070d
Generated-By: PostHog Code Task-Id: 11daf8d3-7830-436a-ab8f-99123586070d
Generated-By: PostHog Code Task-Id: c1bbe3cf-742b-4b24-bf96-d11a18b4cf22
Generated-By: PostHog Code Task-Id: c1bbe3cf-742b-4b24-bf96-d11a18b4cf22
Keep task creation, in-task changes, and terminal retries on the same adapter configuration policy. Generated-By: PostHog Code Task-Id: c1bbe3cf-742b-4b24-bf96-d11a18b4cf22
Generated-By: PostHog Code Task-Id: c1bbe3cf-742b-4b24-bf96-d11a18b4cf22
richardsolomou
force-pushed
the
posthog-code/feat-mobile-codex-parity
branch
from
July 23, 2026 18:55
d4a99d1 to
5e2a064
Compare
richardsolomou
force-pushed
the
posthog-code/finalize-mobile-portability-boundary
branch
from
July 23, 2026 18:55
c630011 to
498fc78
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Problem
Mobile task creation hardcoded Claude, and its new-task and conversation screens had drifted from desktop. Runtime choices could also be lost or mapped to Claude-specific configuration when retrying or resuming Codex tasks.
Changes
I added Claude and Codex controls across task creation, retry, and resume using the shared model and reasoning policies. Adapter-specific execution modes, defaults, and cloud reasoning keys now come from core, while unrestricted modes remain hidden pending an explicit mobile opt-in. Native rendering remains in mobile.
How did you test this?
Automatic notifications
Created with PostHog Code