refactor: use @constructive-io/fetch package instead of inline wrapper

Replace the inlined localhostFetch wrapper with an import from
@constructive-io/fetch (dev-utils#81). The package handles:
- *.localhost DNS rewriting via node:http (fixes ENOTFOUND)
- Host header preservation (fixes silent drop by undici)
- Automatic passthrough for non-localhost URLs and browsers

This moves the fetch logic to a shared library so it can be
iterated on in one place rather than being baked into every
generated client file.

Added @constructive-io/fetch ^0.1.0 as a dependency to all SDK
packages (constructive-sdk, constructive-react, constructive-cli,
migrate-client).

Author: pyramation

graphql/codegen/src/__tests__/codegen/__snapshots__/client-generator.test.ts.snap

@@ -104,6 +104,7 @@ exports[`client-generator generateOrmClientFile generates OrmClient class with e
  * @generated by @constructive-io/graphql-codegen
  * DO NOT EDIT - changes will be overwritten
  */
+import { createFetch } from '@constructive-io/fetch';
 import type {
   GraphQLAdapter,
   GraphQLError,
@@ -116,49 +117,13 @@ export type {
   QueryResult,
 } from '@constructive-io/graphql-query/runtime';
 
-/**
- * Rewrite *.localhost URLs so they resolve in Node.js.
- *
- * Node's undici-backed fetch cannot resolve *.localhost subdomains
- * (ENOTFOUND) and silently drops the Host header (forbidden by spec).
- * This wrapper rewrites the URL to plain localhost and injects the
- * original Host header as a best-effort (works with node:http-based
- * fetch implementations; Node's built-in fetch may still drop it).
- *
- * In browsers *.localhost resolves natively, so this is a no-op.
- */
-function localhostFetch(
-  input: RequestInfo | URL,
-  init?: RequestInit,
-): Promise<Response> {
-  const url = new URL(
-    typeof input === 'string'
-      ? input
-      : input instanceof URL
-        ? input.href
-        : input.url,
-  );
-
-  if (url.hostname.endsWith('.localhost') && url.hostname !== 'localhost') {
-    const originalHost = url.host;
-    url.hostname = 'localhost';
-    const headers = new Headers(init?.headers);
-    if (!headers.has('host')) {
-      headers.set('host', originalHost);
-    }
-    return fetch(url.toString(), { ...init, headers });
-  }
-
-  return fetch(input, init);
-}
-
 /**
  * Default adapter that uses fetch for HTTP requests.
  *
- * When no custom fetch is provided, uses localhostFetch which rewrites
- * *.localhost URLs to plain localhost for Node.js DNS compatibility.
- * Pass a custom fetch to override (e.g. for Node.js Host header
- * preservation via node:http, or for test mocking).
+ * When no custom fetch is provided, uses @constructive-io/fetch which
+ * handles *.localhost DNS rewriting and Host header preservation in
+ * Node.js. Pass a custom fetch to override for test mocking or custom
+ * proxy/credentials.
  */
 export class FetchAdapter implements GraphQLAdapter {
   private headers: Record<string, string>;
@@ -170,7 +135,7 @@ export class FetchAdapter implements GraphQLAdapter {
     fetchFn?: typeof globalThis.fetch,
   ) {
     this.headers = headers ?? {};
-    this.fetchFn = fetchFn ?? localhostFetch;
+    this.fetchFn = fetchFn ?? createFetch();
   }
 
   async execute<T>(
@@ -240,10 +205,9 @@ export interface OrmClientConfig {
   /** Default headers for HTTP requests (only used with endpoint) */
   headers?: Record<string, string>;
   /**
-   * Custom fetch implementation. Defaults to a wrapper that rewrites
-   * *.localhost URLs for Node.js compatibility. Pass your own fetch
-   * for full Node.js Host header support (e.g. via node:http), test
-   * mocking, or custom proxy/credentials.
+   * Custom fetch implementation. Defaults to @constructive-io/fetch
+   * which handles *.localhost DNS and Host headers in Node.js.
+   * Pass your own for test mocking or custom proxy/credentials.
    */
   fetch?: typeof globalThis.fetch;
   /** Custom adapter for GraphQL execution (overrides endpoint/headers/fetch) */

graphql/codegen/src/__tests__/codegen/client-generator.test.ts

@@ -68,13 +68,13 @@ describe('client-generator', () => {
       expect(result.content).toContain('config.fetch');
     });
 
-    it('includes localhostFetch wrapper for *.localhost DNS rewriting', () => {
+    it('imports createFetch from @constructive-io/fetch', () => {
       const result = generateOrmClientFile();
 
-      expect(result.content).toContain('function localhostFetch');
-      expect(result.content).toContain(".endsWith('.localhost')");
-      expect(result.content).toContain("url.hostname = 'localhost'");
-      expect(result.content).toContain("headers.set('host', originalHost)");
+      expect(result.content).toContain(
+        "import { createFetch } from '@constructive-io/fetch'",
+      );
+      expect(result.content).toContain('createFetch()');
     });
   });
 

graphql/codegen/src/core/codegen/templates/orm-client.ts

@@ -8,6 +8,7 @@
  * Any changes here will affect all generated ORM clients.
  */
 
+import { createFetch } from '@constructive-io/fetch';
 import type {
   GraphQLAdapter,
   GraphQLError,
@@ -20,49 +21,13 @@ export type {
   QueryResult,
 } from '@constructive-io/graphql-query/runtime';
 
-/**
- * Rewrite *.localhost URLs so they resolve in Node.js.
- *
- * Node's undici-backed fetch cannot resolve *.localhost subdomains
- * (ENOTFOUND) and silently drops the Host header (forbidden by spec).
- * This wrapper rewrites the URL to plain localhost and injects the
- * original Host header as a best-effort (works with node:http-based
- * fetch implementations; Node's built-in fetch may still drop it).
- *
- * In browsers *.localhost resolves natively, so this is a no-op.
- */
-function localhostFetch(
-  input: RequestInfo | URL,
-  init?: RequestInit,
-): Promise<Response> {
-  const url = new URL(
-    typeof input === 'string'
-      ? input
-      : input instanceof URL
-        ? input.href
-        : input.url,
-  );
-
-  if (url.hostname.endsWith('.localhost') && url.hostname !== 'localhost') {
-    const originalHost = url.host;
-    url.hostname = 'localhost';
-    const headers = new Headers(init?.headers);
-    if (!headers.has('host')) {
-      headers.set('host', originalHost);
-    }
-    return fetch(url.toString(), { ...init, headers });
-  }
-
-  return fetch(input, init);
-}
-
 /**
  * Default adapter that uses fetch for HTTP requests.
  *
- * When no custom fetch is provided, uses localhostFetch which rewrites
- * *.localhost URLs to plain localhost for Node.js DNS compatibility.
- * Pass a custom fetch to override (e.g. for Node.js Host header
- * preservation via node:http, or for test mocking).
+ * When no custom fetch is provided, uses @constructive-io/fetch which
+ * handles *.localhost DNS rewriting and Host header preservation in
+ * Node.js. Pass a custom fetch to override for test mocking or custom
+ * proxy/credentials.
  */
 export class FetchAdapter implements GraphQLAdapter {
   private headers: Record<string, string>;
@@ -74,7 +39,7 @@ export class FetchAdapter implements GraphQLAdapter {
     fetchFn?: typeof globalThis.fetch,
   ) {
     this.headers = headers ?? {};
-    this.fetchFn = fetchFn ?? localhostFetch;
+    this.fetchFn = fetchFn ?? createFetch();
   }
 
   async execute<T>(
@@ -144,10 +109,9 @@ export interface OrmClientConfig {
   /** Default headers for HTTP requests (only used with endpoint) */
   headers?: Record<string, string>;
   /**
-   * Custom fetch implementation. Defaults to a wrapper that rewrites
-   * *.localhost URLs for Node.js compatibility. Pass your own fetch
-   * for full Node.js Host header support (e.g. via node:http), test
-   * mocking, or custom proxy/credentials.
+   * Custom fetch implementation. Defaults to @constructive-io/fetch
+   * which handles *.localhost DNS and Host headers in Node.js.
+   * Pass your own for test mocking or custom proxy/credentials.
    */
   fetch?: typeof globalThis.fetch;
   /** Custom adapter for GraphQL execution (overrides endpoint/headers/fetch) */

sdk/constructive-cli/package.json

@@ -46,6 +46,7 @@
   ],
   "dependencies": {
     "@0no-co/graphql.web": "^1.1.2",
+    "@constructive-io/fetch": "^0.1.0",
     "@constructive-io/graphql-query": "workspace:^",
     "@constructive-io/graphql-types": "workspace:^",
     "appstash": "^0.7.0",

sdk/constructive-cli/src/admin/orm/client.ts

@@ -3,6 +3,7 @@
  * @generated by @constructive-io/graphql-codegen
  * DO NOT EDIT - changes will be overwritten
  */
+import { createFetch } from '@constructive-io/fetch';
 import type {
   GraphQLAdapter,
   GraphQLError,
@@ -15,49 +16,13 @@ export type {
   QueryResult,
 } from '@constructive-io/graphql-query/runtime';
 
-/**
- * Rewrite *.localhost URLs so they resolve in Node.js.
- *
- * Node's undici-backed fetch cannot resolve *.localhost subdomains
- * (ENOTFOUND) and silently drops the Host header (forbidden by spec).
- * This wrapper rewrites the URL to plain localhost and injects the
- * original Host header as a best-effort (works with node:http-based
- * fetch implementations; Node's built-in fetch may still drop it).
- *
- * In browsers *.localhost resolves natively, so this is a no-op.
- */
-function localhostFetch(
-  input: RequestInfo | URL,
-  init?: RequestInit,
-): Promise<Response> {
-  const url = new URL(
-    typeof input === 'string'
-      ? input
-      : input instanceof URL
-        ? input.href
-        : input.url,
-  );
-
-  if (url.hostname.endsWith('.localhost') && url.hostname !== 'localhost') {
-    const originalHost = url.host;
-    url.hostname = 'localhost';
-    const headers = new Headers(init?.headers);
-    if (!headers.has('host')) {
-      headers.set('host', originalHost);
-    }
-    return fetch(url.toString(), { ...init, headers });
-  }
-
-  return fetch(input, init);
-}
-
 /**
  * Default adapter that uses fetch for HTTP requests.
  *
- * When no custom fetch is provided, uses localhostFetch which rewrites
- * *.localhost URLs to plain localhost for Node.js DNS compatibility.
- * Pass a custom fetch to override (e.g. for Node.js Host header
- * preservation via node:http, or for test mocking).
+ * When no custom fetch is provided, uses @constructive-io/fetch which
+ * handles *.localhost DNS rewriting and Host header preservation in
+ * Node.js. Pass a custom fetch to override for test mocking or custom
+ * proxy/credentials.
  */
 export class FetchAdapter implements GraphQLAdapter {
   private headers: Record<string, string>;
@@ -69,7 +34,7 @@ export class FetchAdapter implements GraphQLAdapter {
     fetchFn?: typeof globalThis.fetch,
   ) {
     this.headers = headers ?? {};
-    this.fetchFn = fetchFn ?? localhostFetch;
+    this.fetchFn = fetchFn ?? createFetch();
   }
 
   async execute<T>(
@@ -139,10 +104,9 @@ export interface OrmClientConfig {
   /** Default headers for HTTP requests (only used with endpoint) */
   headers?: Record<string, string>;
   /**
-   * Custom fetch implementation. Defaults to a wrapper that rewrites
-   * *.localhost URLs for Node.js compatibility. Pass your own fetch
-   * for full Node.js Host header support (e.g. via node:http), test
-   * mocking, or custom proxy/credentials.
+   * Custom fetch implementation. Defaults to @constructive-io/fetch
+   * which handles *.localhost DNS and Host headers in Node.js.
+   * Pass your own for test mocking or custom proxy/credentials.
    */
   fetch?: typeof globalThis.fetch;
   /** Custom adapter for GraphQL execution (overrides endpoint/headers/fetch) */

sdk/constructive-cli/src/auth/orm/client.ts

@@ -3,6 +3,7 @@
  * @generated by @constructive-io/graphql-codegen
  * DO NOT EDIT - changes will be overwritten
  */
+import { createFetch } from '@constructive-io/fetch';
 import type {
   GraphQLAdapter,
   GraphQLError,
@@ -15,49 +16,13 @@ export type {
   QueryResult,
 } from '@constructive-io/graphql-query/runtime';
 
-/**
- * Rewrite *.localhost URLs so they resolve in Node.js.
- *
- * Node's undici-backed fetch cannot resolve *.localhost subdomains
- * (ENOTFOUND) and silently drops the Host header (forbidden by spec).
- * This wrapper rewrites the URL to plain localhost and injects the
- * original Host header as a best-effort (works with node:http-based
- * fetch implementations; Node's built-in fetch may still drop it).
- *
- * In browsers *.localhost resolves natively, so this is a no-op.
- */
-function localhostFetch(
-  input: RequestInfo | URL,
-  init?: RequestInit,
-): Promise<Response> {
-  const url = new URL(
-    typeof input === 'string'
-      ? input
-      : input instanceof URL
-        ? input.href
-        : input.url,
-  );
-
-  if (url.hostname.endsWith('.localhost') && url.hostname !== 'localhost') {
-    const originalHost = url.host;
-    url.hostname = 'localhost';
-    const headers = new Headers(init?.headers);
-    if (!headers.has('host')) {
-      headers.set('host', originalHost);
-    }
-    return fetch(url.toString(), { ...init, headers });
-  }
-
-  return fetch(input, init);
-}
-
 /**
  * Default adapter that uses fetch for HTTP requests.
  *
- * When no custom fetch is provided, uses localhostFetch which rewrites
- * *.localhost URLs to plain localhost for Node.js DNS compatibility.
- * Pass a custom fetch to override (e.g. for Node.js Host header
- * preservation via node:http, or for test mocking).
+ * When no custom fetch is provided, uses @constructive-io/fetch which
+ * handles *.localhost DNS rewriting and Host header preservation in
+ * Node.js. Pass a custom fetch to override for test mocking or custom
+ * proxy/credentials.
  */
 export class FetchAdapter implements GraphQLAdapter {
   private headers: Record<string, string>;
@@ -69,7 +34,7 @@ export class FetchAdapter implements GraphQLAdapter {
     fetchFn?: typeof globalThis.fetch,
   ) {
     this.headers = headers ?? {};
-    this.fetchFn = fetchFn ?? localhostFetch;
+    this.fetchFn = fetchFn ?? createFetch();
   }
 
   async execute<T>(
@@ -139,10 +104,9 @@ export interface OrmClientConfig {
   /** Default headers for HTTP requests (only used with endpoint) */
   headers?: Record<string, string>;
   /**
-   * Custom fetch implementation. Defaults to a wrapper that rewrites
-   * *.localhost URLs for Node.js compatibility. Pass your own fetch
-   * for full Node.js Host header support (e.g. via node:http), test
-   * mocking, or custom proxy/credentials.
+   * Custom fetch implementation. Defaults to @constructive-io/fetch
+   * which handles *.localhost DNS and Host headers in Node.js.
+   * Pass your own for test mocking or custom proxy/credentials.
    */
   fetch?: typeof globalThis.fetch;
   /** Custom adapter for GraphQL execution (overrides endpoint/headers/fetch) */