Repository navigation
Fix shell replay gaps - #142
Merged
Merged
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 67ef69c8e9
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
adamfgr
pushed a commit
that referenced
this pull request
Jul 21, 2026
* fix(server): resolve Claude SDK executable path on Windows npm installs (pingdotgg#3740) * Fix project action preview settings persistence (pingdotgg#3842) * fix(desktop): allow clipboard writes in the preview browser (pingdotgg#3889) * fix(web): handle sidebar shortcut before editors (pingdotgg#3921) * fix(server): recognize Bedrock-backed Claude as authenticated (pingdotgg#3931) * Fix incorrect pluralization of “entry” (pingdotgg#3933) * feat(server): title background-task work-log rows with the task name (pingdotgg#3751) Co-authored-by: Claude Fable 5 <[email protected]> * fix: delegate OpenCode session titles to provider (pingdotgg#3720) * Archive selected threads from the context menu (pingdotgg#3895) * fix(cli): support force removing projects (pingdotgg#3922) * fix: allow sidebar to be shrunk when wider than viewport (pingdotgg#2456) Co-authored-by: Shoaib Ansari <[email protected]> Co-authored-by: Julius Marminge <[email protected]> * fix(codex): show web search query and url in tool call details (pingdotgg#2093) Co-authored-by: Julius Marminge <[email protected]> * Add Codex launch arguments setting (pingdotgg#2892) Co-authored-by: Julius Marminge <[email protected]> Co-authored-by: Julius Marminge <[email protected]> Co-authored-by: root <[email protected]> * [orchestration] Clear stale active turn when session becomes inactive (pingdotgg#3159) Co-authored-by: Julius Marminge <[email protected]> * Regenerate Codex reset credit protocol bindings (pingdotgg#4173) Co-authored-by: codex <[email protected]> * fix(preview): preserve direct localhost navigation (pingdotgg#3939) Co-authored-by: Julius Marminge <[email protected]> Co-authored-by: codex <[email protected]> * Synchronize mobile threads with authoritative shell snapshots (pingdotgg#4163) Co-authored-by: codex <[email protected]> * Gate iOS glass layout on native support (pingdotgg#4032) Co-authored-by: codex <[email protected]> * fix(opencode): resume the OpenCode session on follow-ups instead of starting an empty one (pingdotgg#3617) Co-authored-by: codex <[email protected]> * fix(server): use CLI for OpenCode health check instead of spawning server (pingdotgg#4153) * fix(web): scope timeline minimap hover target to the side gutter (pingdotgg#3869) Co-authored-by: Claude Fable 5 <[email protected]> * [codex] show complete approval details (pingdotgg#4111) * fix(web): paint text selection over composer chips (pingdotgg#4139) Signed-off-by: Yordis Prieto <[email protected]> * [codex] preserve custom model slugs (pingdotgg#4168) * fix(web): preview workspace images in the file panel (pingdotgg#3996) Co-authored-by: Rhiz3K <[email protected]> Co-authored-by: Julius Marminge <[email protected]> * feat(web): drag files from the explorer into the chat composer (pingdotgg#4140) Co-authored-by: Julius Marminge <[email protected]> Signed-off-by: Yordis Prieto <[email protected]> * fix(desktop): preserve main window bounds (pingdotgg#3851) Co-authored-by: Julius Marminge <[email protected]> Co-authored-by: codex <[email protected]> * perf(orchestration): speed up new-chat propagation and offline catch-up (pingdotgg#4177) Co-authored-by: codex <[email protected]> Co-authored-by: Julius Marminge <[email protected]> * Finale: upgrade changed files card to fix various UI issues (pingdotgg#4113) Co-authored-by: Julius Marminge <[email protected]> * fix(web): always show environment chip for remote projects (pingdotgg#4217) Co-authored-by: Claude Fable 5 <[email protected]> * chore: reconcile upstream sync (20260721) — typecheck/test fixups Post-cherry-pick reconciliation of the shell/thread sync rewrite: - adopt upstream's client shell.ts + shell-sync.test.ts (drops the fork's superseded #142 replay-gap watchdog; server no longer emits 'caught-up') - remove the orphaned 'caught-up' contract member and obsolete fork shell server tests (upstream's coalescing tests cover the behavior) - keep the fork's thread reconciliation, resubscribing via subscribeDynamic (establish base once; foreground/session resubscribe via the live cursor) - add latestSequence to OrchestrationEngine test mocks; thread dispatch authority in bin.test; narrow stream-item unions past the new 'synchronized' Co-Authored-By: Claude Opus 4.8 <[email protected]> --------- Signed-off-by: Yordis Prieto <[email protected]> Co-authored-by: David Whatley <[email protected]> Co-authored-by: coach007 <[email protected]> Co-authored-by: Carlos Rico-Ospina <[email protected]> Co-authored-by: Andrew Barnes <[email protected]> Co-authored-by: Pieter van Zyl <[email protected]> Co-authored-by: mel <[email protected]> Co-authored-by: Theo Browne <[email protected]> Co-authored-by: Claude Fable 5 <[email protected]> Co-authored-by: Tristan Knight <[email protected]> Co-authored-by: Christoph Herzog <[email protected]> Co-authored-by: Shoaib <[email protected]> Co-authored-by: Shoaib Ansari <[email protected]> Co-authored-by: Julius Marminge <[email protected]> Co-authored-by: Guilherme Vieira <[email protected]> Co-authored-by: James <[email protected]> Co-authored-by: Julius Marminge <[email protected]> Co-authored-by: root <[email protected]> Co-authored-by: Andrew Forster <[email protected]> Co-authored-by: codex <[email protected]> Co-authored-by: Chris Michael Guzman <[email protected]> Co-authored-by: Vadym Kotai <[email protected]> Co-authored-by: Utkarsh Patil <[email protected]> Co-authored-by: xxashxx-svg <[email protected]> Co-authored-by: Maxwell Young <[email protected]> Co-authored-by: Yordis Prieto <[email protected]> Co-authored-by: Rhiz3K <[email protected]> Co-authored-by: Rhiz3K <[email protected]> Co-authored-by: Anirudh Coontoor <[email protected]> Co-authored-by: Rusiru Sadathana <[email protected]> Co-authored-by: ss <[email protected]> Co-authored-by: wizzoapp[bot] <254688279+wizzoapp[bot]@users.noreply.github.com>
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.
Fixes ADA-108.
Summary
Verification