@djthread is attempting to deploy a commit to the LobeHub OSS Team on Vercel.

A member of the Team first needs to authorize it.

@ONLY-yours @tjx666 - This is a desktop platform fix switching from Node.js fetch() to Electron's net.fetch() for remote server requests (affects auth/OIDC handoff, token refresh, and backend proxy). All changes are in apps/desktop/. Please take a look.

The tests stub global.fetch but the electron mock has no net object. The last commit makes net-fetch.ts fall back to globalThis.fetch when net is unavailable (tests).

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 66.56%. Comparing base (e0f97c4) to head (44ab3f8).
⚠️ Report is 5 commits behind head on canary.

@@            Coverage Diff            @@
##           canary   #13400     +/-   ##
=========================================
  Coverage   66.56%   66.56%             
=========================================
  Files        2022     2022             
  Lines      171262   171262             
  Branches    19915    16654   -3261     
=========================================
+ Hits       113996   113998      +2     
+ Misses      57141    57139      -2     
  Partials      125      125             

@djthread Test failed, plz take a look

Thanks, @Innei !

Two categories of test breakage patched in this commit:

E2E selectors (e2e/src/steps/page/): The "New Page" button in the Pages sidebar is rendered by @lobehub/ui's ActionIcon, which outputs <div role="button"> rather than a native <button>. The old selectors (button:has(svg.lucide-square-pen), input[placeholder*="Search"]) were both dead — the first matched no element, and the search bar is a click-handler NavItem, not an <input>. Fixed to [role="button"]:has(svg.lucide-square-pen).

Unit test mocks (apps/desktop/.../LocalFileCtr.test.ts, RemoteServerConfigCtr.test.ts): The net.fetch change routes HTTP calls through Electron's networking stack instead of globalThis.fetch. The tests were patching global.fetch / using vi.stubGlobal, which no longer intercepted anything. Fixed by hoisting the fetch mock and pointing vi.mock('@/utils/net-fetch', ...) at it directly.

E2E selectors: This is a common error that often occurs; retrying might resolve it. Here, there is no need for any changes; you just need to fix the unit test you are involved with. Thx.

Thanks, @Innei !

Two categories of test breakage patched in this commit:

E2E selectors (e2e/src/steps/page/): The "New Page" button in the Pages sidebar is rendered by @lobehub/ui's ActionIcon, which outputs <div role="button"> rather than a native <button>. The old selectors (button:has(svg.lucide-square-pen), input[placeholder*="Search"]) were both dead — the first matched no element, and the search bar is a click-handler NavItem, not an <input>. Fixed to [role="button"]:has(svg.lucide-square-pen).

Unit test mocks (apps/desktop/.../LocalFileCtr.test.ts, RemoteServerConfigCtr.test.ts): The net.fetch change routes HTTP calls through Electron's networking stack instead of globalThis.fetch. The tests were patching global.fetch / using vi.stubGlobal, which no longer intercepted anything. Fixed by hoisting the fetch mock and pointing vi.mock('@/utils/net-fetch', ...) at it directly.

@Innei Ty again. Done.

@codex Will this modification cause issues with exception handling for requests?

To use Codex here, create an environment for this repo.

❤️ Great PR @djthread ❤️

The growth of project is inseparable from user feedback and contribution, thanks for your contribution! If you are interesting with the lobehub developer community, please join our discord and then dm @arvinxx or @canisminor1990. They will invite you to our private developer channel. We are talking about the lobe-chat development or sharing ai newsletter around the world.