chore: refactoring and tests#12468
Merged
Merged
Conversation
adamdotdevin
force-pushed
the
opencode/quick-wizard
branch
from
31c05b0 to
1528ffa
Compare
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>
Colin4k1024
pushed a commit
to Colin4k1024/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
xywsxp
pushed a commit
to xywsxp/opencode
that referenced
this pull request
Apr 24, 2026
Rwanbt
pushed a commit
to Rwanbt/opencode
that referenced
this pull request
May 5, 2026
zoulukuang
pushed a commit
to zoulukuang/deskfox
that referenced
this pull request
May 27, 2026
…test 在 Windows fail [feat: file-tree-test-client-env] 根因:bun test 把 solid-js 解析成服务端构建(无浏览器/development 条件)。file-tree.test.ts 只想测纯函数(shouldListRoot 等),但 import ./file-tree 连带加载 @opencode-ai/ui/context-menu (= Kobalte ContextMenu),@kobalte/core 在模块求值期就调 solid-js client-only API → 服务端构建 直接抛 "Client-only API called on the server side"。其他 707 测试不引 Kobalte 故不触发。 测试已 mock collapsible/file-icon/icon/tooltip,独缺 context-menu。补一条同款 mock(Proxy 透传 任意子组件)阻止真 Kobalte 加载即可 —— 本测试从不渲染组件,mock 形状无所谓。 上游测试(anomalyco#12468),改动加 FORK marker。修后 app 包 711 pass / 0 fail(原 708/1)+ typecheck 17/17。 纯测试环境隔离,不改任何软件功能;macOS/真实 app(WebView2 有浏览器)本就正常。 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.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
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.
No description provided.