fix(server): own provider generation lifecycle through shutdown - #2734
Closed
timigod wants to merge 18 commits into
Closed
fix(server): own provider generation lifecycle through shutdown#2734timigod wants to merge 18 commits into
timigod wants to merge 18 commits into
Conversation
# Conflicts: # packages/server/src/server/agent/providers/opencode-server-manager.test.ts
…review/v025-provider-availability-native-2
…review/v025-provider-availability-native-2
timigod
added a commit
to timigod/paseo
that referenced
this pull request
Aug 1, 2026
# Conflicts: # packages/server/src/server/agent/agent-manager.ts
Author
|
Closed as superseded by focused successor #2758 at independently accepted head |
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.
Outcome
Makes provider generations, snapshots, managed loads, session startup, and provider-owned processes retain truthful ownership through refresh, timeout, replacement, abort, and daemon shutdown.
Changes
Startup cancellation amendment
OpenCode teardown amendment
session.createrequestFinal lifecycle amendment
provider.listsignal-aware and bounded while retaining the raw SDK operation until actual settlementsession.deletebefore releasing the server while retaining the raw delete operationCurrent-main integration
mainat70ed70d36e2eabd3876a22931954319ec202c902with an ordinary two-parent mergePR #2744 integration guidance
Do not combine this branch with draft PR #2744. During later integration, normalize all OpenCode acquisition methods to one options-object shape:
acquireCurrent(options?: { signal?: AbortSignal })acquireNew(options?: { signal?: AbortSignal })acquireDedicated(env, options?: { signal?: AbortSignal })Translate this branch's positional startup calls to
{ signal }, including dedicated acquisition, while retaining #2744's waiter-aware cancellation and shutdown implementation.Verification
npm run build:servernpm run typechecknpm run lint: 0 warnings and 0 errorsnpm run format:checkgit diff --checkpatches/checksExact receipt
70ed70d36e2eabd3876a22931954319ec202c9020facad3e0535991a32e72055b5e0ab9e861b02c6833a60dabb8d1681b7478defe5b5abf86abc00f0584471d82b0aec902cf0d249b4e26479ce0ac7a50d8fbd9b2cff87a2bf980a4c49589747dff2401676d72382253ad17e43eaec6811893e91dac881c5653b8b9c0f4bd19fdf10bd6aaeb2782cca5008d876d72382253ad17e43eaec6811893e91dac881c50ea257185b384ece2dd65a814a7ef3d1df420a28This PR remains draft pending exact-tip rereview.