Remove cross-fetch usage

[Copilot]

fetch is a global API as of Node 18, which is now the minimum supported version. This removes the cross-fetch polyfill dependency.

Changes

• Source files: Replace cross-fetch imports with native fetch in:

  • packages/connected-solid/src/util/guaranteeFetch.ts
  • packages/solid-react/src/UnauthenticatedSolidLdoProvider.tsx
  • packages/test-solid-server/src/authFetch.ts

• Dependencies: Remove cross-fetch from 5 package.json files:

  • connected-solid, solid-react, connected-nextgraph, ldo, react

• TypeScript config: Add "lib": ["ES2021", "DOM"] to packages/connected-solid/tsconfig.esm.json to provide DOM types (HeadersInit, window, etc.) that were previously included transitively via cross-fetch's type reference

• Tests: Update guaranteeFetch.test.ts to assert against global fetch instead of crossFetch

Example

// Before
import crossFetch from "cross-fetch";
export function guaranteeFetch(fetchInput?: typeof fetch): typeof fetch {
  return fetchInput || crossFetch;
}

// After
export function guaranteeFetch(fetchInput?: typeof fetch): typeof fetch {
  return fetchInput || fetch;
}

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• www.w3.org
  • Triggering command: /usr/local/bin/node /usr/local/bin/node /home/REDACTED/work/ldo/ldo/node_modules/jest-worker/build/workers/processChild.js mcli/run-script/--coverage node er/b�� config.esm.json tsconfig.cjs.json (dns block)
  • Triggering command: /usr/local/bin/node /usr/local/bin/node /home/REDACTED/work/ldo/ldo/node_modules/jest-worker/build/workers/processChild.js odules/.bin/sh node er/b�� config.esm.json tsconfig.cjs.json (dns block)
  • Triggering command: /usr/local/bin/node /usr/local/bin/node /home/REDACTED/work/ldo/ldo/node_modules/jest-worker/build/workers/processChild.js es/ldo/node_modu--coverage sh k/ld�� tsc --project tsconfig.esm.json n es/.bin/sh config.cjs.json watchman node mcli/run-script/lib/node-gyp-binjest --coverage sh -c config.cjs.json && node ../../scripts/rename-to-cjs.js sh (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Remove cross-fetch usage</issue_title>
<issue_description>fetch is a global object as of Node 18; which is now the oldest supported version of Node. Thus it is safe to remove the cross-fetch global and just use fetch in the next major release.</issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes Remove cross-fetch usage #99

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[Copilot]

Pull request overview

This PR removes the cross-fetch polyfill dependency from the codebase in favor of native fetch, which is now globally available as of Node 18 (the minimum supported version). The changes eliminate an unnecessary dependency and modernize the codebase.

• Replaced all cross-fetch imports with native global fetch
• Removed cross-fetch from 5 package.json files
• Updated TypeScript configuration to include DOM types previously provided transitively

Reviewed changes

Copilot reviewed 12 out of 13 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
packages/connected-solid/src/util/guaranteeFetch.ts	Removed cross-fetch import and updated to use global fetch; also fixed typo "Guantees" → "Guarantees"
packages/solid-react/src/UnauthenticatedSolidLdoProvider.tsx	Removed cross-fetch import and changed from libraryFetch to global fetch
packages/test-solid-server/src/authFetch.ts	Removed unused cross-fetch import (already using global fetch)
packages/connected-solid/test/guaranteeFetch.test.ts	Updated test assertion from crossFetch to global fetch and updated test description
packages/solid-type-index/test/fileData.ts	Removed unused cross-fetch type import
packages/solid-react/test/fileData.ts	Removed unused cross-fetch import
packages/connected-solid/tsconfig.esm.json	Added "lib": ["ES2021", "DOM"] to provide DOM types like HeadersInit
packages/connected-solid/package.json	Removed cross-fetch dependency
packages/solid-react/package.json	Removed cross-fetch dependency
packages/react/package.json	Removed cross-fetch dependency
packages/ldo/package.json	Removed cross-fetch dev dependency
packages/connected-nextgraph/package.json	Removed cross-fetch dependency
package-lock.json	Updated lockfile to reflect removed dependencies

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[jeswr]

Note that https://github.com/o-development/ldo/pull/122/files also adds a minimum version number to the package.json