Skip to content

docs(codex): document mobile remote thread visibility limitation for provider-table form - #6029

Closed
codingbooo wants to merge 1 commit into
lidge-jun:devfrom
codingbooo:feat/issue-5848-mobile-openai-threads
Closed

codingbooo wants to merge 1 commit into
lidge-jun:devfrom
codingbooo:feat/issue-5848-mobile-openai-threads

Conversation

@codingbooo

@codingbooo codingbooo commented Sep 27, 2026 •

Copy link
Copy Markdown
Contributor

Closes #5848

What

With provider-table routing, both openai and opencodex thread identities can exist in the same
Codex home. A user reported that the ChatGPT Android remote client listed only opencodex threads,
so conversations they could still see on Codex Desktop looked deleted on mobile.

Finding

These are Codex app-server thread/list semantics, not a proxy routing bug. The reporter confirmed
against app-server 0.157.0:

  • modelProviders omitted → only the root provider's threads (opencodex)
  • modelProviders: [] → threads from both openai and opencodex

Clients can explicitly create openai threads even when the root default is opencodex, so this
affects new conversations as well as old history. The retained openai_base_url keeps those threads
routed; it does not change the remote list's provider filter.

Change

Documentation only — this repo does not own the RPC.

  • docs-site/src/content/docs/guides/codex-integration.md documents the filter behaviour, the
    modelProviders: [] workaround for remote-client maintainers, and the fact that OpenCodeX does not
    periodically rewrite thread providers to work around it.
  • Explicitly records why direct SQLite retagging is not a durable fix: a loaded thread can write its
    in-memory provider back, and changing only the row can disagree with its rollout metadata.
  • syncResumeHistory: false remains the opt-out from history remapping, and client-compaction-only
    routing preserves existing thread identities.

An earlier revision of this branch also carried a doc-side rewrite of thread providers. That
approach was dropped: it cannot be made durable, and the remaining change is the documentation the
issue actually calls for.

Verification

  • bun run structure:check, bun run privacy:scan — clean
  • docs-site build completes (537 pages)

Review readiness checklist

This PR stays in draft until every box below is ticked. Tick all four boxes once the requirements are met:

  • Required local validation passed; commands, results, and any full-suite exception are documented.

  • I pushed my PR to a recent dev commit (at most 10 behind; a maintainer may still ask for the exact tip before merge).

  • I resolved all correct Codex and CodeRabbit findings.

  • My PR is ready for review.

Summary by CodeRabbit

  • Documentation
    • Added guidance explaining how provider filters affect the visibility of Codex conversations in mobile remote clients, with a recommended filter setting and Desktop as an alternative for accessing affected conversations.

@github-actions

Copy link
Copy Markdown
Contributor

✅ Deterministic PR hygiene checks passed.

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Sep 27, 2026
@github-actions

github-actions Bot commented Sep 27, 2026 •

Copy link
Copy Markdown
Contributor

✅ READY

  • all PR quality gates passed; the review readiness checklist is complete.

Review readiness checklist

  • ✅ Required local validation passed; commands, results, and any full-suite exception are documented.
  • ✅ I pushed my PR to a recent dev commit (at most 10 behind; a maintainer may still ask for the exact tip before merge).
  • ✅ I resolved all correct Codex and CodeRabbit findings.
  • ✅ My PR is ready for review.

✅ 4/4 boxes ticked.

This pull request has been marked Ready for Review.
The review-ready label marks this PR as ready; review automation runs independently.
Maintainers notified: @lidge-jun @Ingwannu

@github-actions
github-actions Bot marked this pull request as draft September 27, 2026 01:49
@coderabbitai

coderabbitai Bot commented Sep 27, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: lidge-jun/opencodex/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 95ba3156-08a8-477d-bf8f-270e70afb19c

📥 Commits

Reviewing files that changed from the base of the PR and between 5744e7a and e62d197.

📒 Files selected for processing (1)
  • docs-site/src/content/docs/guides/codex-integration.md

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 2 remain after this review.


📝 Walkthrough

Walkthrough

The Codex integration guide now explains how thread/list filters affect visibility of openai and opencodex threads. It recommends that remote clients send modelProviders: [] and notes that OpenCodeX does not rewrite thread providers.

Changes

Remote thread visibility

Layer / File(s) Summary
Document thread-list filtering
docs-site/src/content/docs/guides/codex-integration.md
Lines 516–545 describe how omitting modelProviders lists only opencodex threads, while an empty array lists both openai and opencodex threads. The guide recommends that remote clients send an empty filter, notes Desktop as an access option, and states that OpenCodeX does not rewrite providers.

Priority: ➖ Normal

Estimated code review effort: 1 (Trivial) | ~4 minutes

Change: Other · Severity of issue fixed: Medium

Merge Risk: ⚪ Minimal · up to e62d1

This documentation-only change has no established merge-blocking issue; the translated-page request is not required by the supplied guidance.

Architecture Summary

Architecture risk: 🔵 Low · up to e62d1

The change affects 1 system.

Changed systems: docs-site

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

  • observed — docs-site (service) was modified; 1 changed file maps to changed impact.

Before / after behavior

  • observed — Modified behavior in docs-site/src/content/docs/guides/codex-integration.md: Added documentation on Codex app-server thread visibility: omitted modelProviders lists only opencodex threads, while an empty array lists both openai and opencodex threads. The text explains the effect on Desktop and mobile conversations, recommends remote clients pass modelProviders: [], and states that OpenCodeX does not periodically rewrite providers or combine the groups.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Linked Issues check ✅ Passed The PR addresses the directly linked coding objective in issue #5848. It adds a “ChatGPT mobile remote thread visibility” section in docs-site/src/content/docs/guides/codex-integration.md. The secti…
Out of Scope Changes check ✅ Passed The reported PR change is limited to 30 added documentation lines in docs-site/src/content/docs/guides/codex-integration.md. The content directly supports issue #5848 by explaining the visibility fa…
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the documentation change about the Codex mobile remote thread visibility limitation for provider-table routing.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions
github-actions Bot marked this pull request as ready for review September 27, 2026 01:55
@Ingwannu

Copy link
Copy Markdown
Owner

Closing as overlapping work with the earlier #6007 for #5848. #6007 already contains the runtime/CLI and dashboard warnings plus the guide explanation, has resolved review threads, and exact-head Cross-platform CI is green. The version-specific reproduction and note that new client-created threads can also retain openai are useful; please fold those wording details into #6007 rather than ship two competing sections. Neither PR claims to patch the upstream mobile/app-server list behavior.

@Ingwannu Ingwannu closed this Sep 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation review-ready

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants