fix(session): add cross-project session lookup with fallback search #9474
+60
−5
Conversation
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
Session.get() was failing when sessions were accessed from a different project context (subagents, nested PTYs, SDK clients with different dir). - Add resolveProjectID() helper that searches all project directories - LRU cache (1000 entries) for resolved projectIDs - Update get(), update(), remove() to use the helper Fixes anomalyco#8538 Related to anomalyco#7773 Builds on approach from anomalyco#3018 by @zenyr
Contributor
|
The following comment was made by an LLM, it may be inaccurate: No duplicate PRs found The search confirmed that PR #9474 (the current PR) is the only active PR addressing the cross-project session lookup issue. While PR #3018 appears in results as a related historical PR that implemented an earlier version of the fallback approach (as credited in the description), it is a distinct, prior implementation and not a duplicate. |
This was referenced Jan 19, 2026
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
What does this PR do?
Fixes session lookup failing when sessions are accessed from a different project context (e.g., SDK clients connecting to
opencode servefrom different directories).Problem:
Session.get()usesInstance.project.iddirectly, causingNotFoundErrorwhen sessions are stored in a different project directory than the current context.Solution:
resolveProjectID()helper that searches project directories to find the sessionget(),update(),remove()to use the helperFixes #8538
Related to #7773
Credit: This builds on the approach identified by @zenyr in #3018. That PR falls back to
[current_project, "global"]only. This PR extends the fallback to search all project directories and adds caching for performance.How did you verify your code works?
opencode servein project ANotFoundError./packages/opencode/script/build.ts --single