fix(app): make openapi-effect a drop-in openapi-fetch wrapper

Author: Codex

package.json

@@ -8,18 +8,18 @@
     "packages/*"
   ],
   "scripts": {
-    "build": "pnpm --filter @effect-template/app build",
-    "check": "pnpm --filter @effect-template/app check",
+    "build": "pnpm --filter openapi-effect build",
+    "check": "pnpm --filter openapi-effect check",
     "changeset": "changeset",
     "changeset-publish": "node -e \"if (!process.env.NPM_TOKEN) { console.log('Skipping publish: NPM_TOKEN is not set'); process.exit(0); }\" && changeset publish",
     "changeset-version": "changeset version",
-    "dev": "pnpm --filter @effect-template/app dev",
-    "lint": "pnpm --filter @effect-template/app lint",
-    "lint:tests": "pnpm --filter @effect-template/app lint:tests",
-    "lint:effect": "pnpm --filter @effect-template/app lint:effect",
-    "test": "pnpm --filter @effect-template/app test",
-    "typecheck": "pnpm --filter @effect-template/app typecheck",
-    "start": "pnpm --filter @effect-template/app start"
+    "dev": "pnpm --filter openapi-effect dev",
+    "lint": "pnpm --filter openapi-effect lint",
+    "lint:tests": "pnpm --filter openapi-effect lint:tests",
+    "lint:effect": "pnpm --filter openapi-effect lint:effect",
+    "test": "pnpm --filter openapi-effect test",
+    "typecheck": "pnpm --filter openapi-effect typecheck",
+    "start": "pnpm --filter openapi-effect start"
   },
   "devDependencies": {
     "@changesets/changelog-github": "^0.5.2",

packages/app/package.json

@@ -1,42 +1,31 @@
 {
-  "name": "@effect-template/app",
-  "version": "1.0.12",
-  "description": "Minimal Vite-powered TypeScript console starter using Effect",
-  "main": "dist/main.js",
-  "directories": {
-    "doc": "doc"
+  "name": "openapi-effect",
+  "version": "0.0.0",
+  "description": "Drop-in replacement for openapi-fetch with an opt-in Effect API",
+  "type": "module",
+  "main": "dist/index.js",
+  "exports": {
+    ".": {
+      "types": "./src/index.ts",
+      "import": "./dist/index.js",
+      "default": "./dist/index.js"
+    }
   },
   "scripts": {
-    "build": "vite build --ssr src/app/main.ts",
-    "dev": "vite build --watch --ssr src/app/main.ts",
+    "build": "vite build",
+    "dev": "vite build --watch",
     "lint": "npx @ton-ai-core/vibecode-linter src/",
     "lint:tests": "npx @ton-ai-core/vibecode-linter tests/",
     "lint:effect": "npx eslint --config eslint.effect-ts-check.config.mjs .",
     "lint:types": "./scripts/lint-types.sh",
     "check": "pnpm run typecheck",
+    "prepare": "pnpm run build",
     "prestart": "pnpm run build",
     "start": "node dist/main.js",
     "test": "pnpm run lint:tests && vitest run",
     "typecheck": "tsc --noEmit",
     "gen:strict-api": "npx ts-node --esm scripts/gen-strict-api.ts"
   },
-  "repository": {
-    "type": "git",
-    "url": "git+https://github.com/ProverCoderAI/effect-template.git"
-  },
-  "keywords": [
-    "effect",
-    "typescript",
-    "vite",
-    "console"
-  ],
-  "author": "",
-  "license": "ISC",
-  "type": "module",
-  "bugs": {
-    "url": "https://github.com/ProverCoderAI/effect-template/issues"
-  },
-  "homepage": "https://github.com/ProverCoderAI/effect-template#readme",
   "packageManager": "pnpm@10.28.2",
   "dependencies": {
     "@effect/cli": "^0.73.1",
@@ -52,6 +41,7 @@
     "@effect/typeclass": "^0.38.0",
     "@effect/workflow": "^0.16.0",
     "effect": "^3.19.15",
+    "openapi-fetch": "^0.15.2",
     "openapi-typescript-helpers": "^0.0.15",
     "ts-morph": "^27.0.2"
   },

packages/app/src/index.ts

@@ -1,48 +1,35 @@
-// CHANGE: Main entry point for openapi-effect package with Effect-native error handling
-// WHY: Enable default import of createClient function with proper error channel design
-// QUOTE(ТЗ): "import createClient from \"openapi-effect\""
-// REF: PR#3 comment from skulidropek about Effect representation
+// CHANGE: Make openapi-effect a drop-in replacement for openapi-fetch (Promise API), with an opt-in Effect API.
+// WHY: LeadForgeAI must be able to swap openapi-fetch -> openapi-effect with near-zero code changes.
+// QUOTE(ТЗ): "openapi-effect должен почти 1 в 1 заменяться с openapi-fetch" / "Просто добавлять effect поведение"
+// REF: user-msg-2026-02-12
 // SOURCE: n/a
 // PURITY: SHELL (re-exports)
 // COMPLEXITY: O(1)
 
-// High-level API (recommended for most users)
-export { createClient as default } from "./shell/api-client/create-client.js"
+// Promise-based client (openapi-fetch compatible)
+export { default } from "openapi-fetch"
+export { default as createClient } from "openapi-fetch"
+export * from "openapi-fetch"
+
+// Effect-based client (opt-in)
+export * as FetchHttpClient from "@effect/platform/FetchHttpClient"
+
+// Strict Effect client (advanced)
+export type * from "./core/api-client/index.js"
+export { assertNever } from "./core/api-client/index.js"
+
 export type {
-  ClientOptions,
   DispatchersFor,
   StrictApiClient,
   StrictApiClientWithDispatchers
 } from "./shell/api-client/create-client.js"
-export { createClientEffect, registerDefaultDispatchers } from "./shell/api-client/create-client.js"
 
-// Core types (for advanced type manipulation)
-// Effect Channel Design:
-// - ApiSuccess<Responses>: 2xx responses → success channel
-// - ApiFailure<Responses>: HttpError (4xx, 5xx) + BoundaryError → error channel
-export type {
-  ApiFailure,
-  ApiSuccess,
-  BodyFor,
-  BoundaryError,
-  ContentTypesFor,
-  DecodeError,
-  HttpError,
-  HttpErrorResponseVariant,
-  HttpErrorVariants,
-  OperationFor,
-  ParseError,
-  PathsForMethod,
-  ResponsesFor,
-  ResponseVariant,
-  StatusCodes,
-  SuccessVariants,
-  TransportError,
-  UnexpectedContentType,
-  UnexpectedStatus
-} from "./core/api-client/index.js"
-
-// Shell utilities (for custom implementations)
+export {
+  createClient as createClientStrict,
+  createClientEffect,
+  registerDefaultDispatchers
+} from "./shell/api-client/create-client.js"
+
 export type { Decoder, Dispatcher, RawResponse, StrictClient, StrictRequestInit } from "./shell/api-client/index.js"
 
 export {

packages/app/src/shell/api-client/create-client-types.ts

@@ -10,6 +10,7 @@
 
 import type * as HttpClient from "@effect/platform/HttpClient"
 import type { Effect } from "effect"
+import type { ClientOptions as OpenapiFetchClientOptions } from "openapi-fetch"
 import type { HttpMethod } from "openapi-typescript-helpers"
 
 import type {
@@ -27,12 +28,7 @@ import type { Dispatcher } from "../../core/axioms.js"
  *
  * @pure - immutable configuration
  */
-export type ClientOptions = {
-  readonly baseUrl: string
-  readonly credentials?: RequestCredentials
-  readonly headers?: HeadersInit
-  readonly fetch?: typeof globalThis.fetch
-}
+export type ClientOptions = OpenapiFetchClientOptions
 
 // CHANGE: Add dispatcher map type for auto-dispatching clients
 // WHY: Enable creating clients that infer dispatcher from path+method without per-call parameter

packages/app/src/shell/api-client/create-client.ts

@@ -92,7 +92,7 @@ const resolveDefaultDispatchers = <Paths extends object>(): DispatchersFor<Paths
  * @complexity O(n + m) where n = |params|, m = |query|
  */
 const buildUrl = (
-  baseUrl: string,
+  baseUrl: string | undefined,
   path: string,
   params?: Record<string, ParamValue>,
   query?: Record<string, QueryValue>
@@ -105,23 +105,32 @@ const buildUrl = (
     }
   }
 
-  // Construct full URL
-  const fullUrl = new URL(url, baseUrl)
-
-  // Add query parameters
+  // Add query parameters without requiring an absolute baseUrl
   if (query) {
+    const search = new URLSearchParams()
     for (const [key, value] of Object.entries(query)) {
       if (Array.isArray(value)) {
         for (const item of value) {
-          fullUrl.searchParams.append(key, String(item))
+          search.append(key, String(item))
         }
       } else {
-        fullUrl.searchParams.set(key, String(value))
+        search.set(key, String(value))
       }
     }
+
+    const queryString = search.toString()
+    if (queryString.length > 0) {
+      url = url.includes("?") ? `${url}&${queryString}` : `${url}?${queryString}`
+    }
   }
 
-  return fullUrl.toString()
+  // If no baseUrl is provided, keep the URL relative (browser-compatible)
+  if (baseUrl === undefined || baseUrl === "") {
+    return url
+  }
+
+  // Resolve relative paths against baseUrl (Node-compatible)
+  return new URL(url, baseUrl).toString()
 }
 
 /**
@@ -174,16 +183,50 @@ const needsJsonContentType = (body: BodyInit | object | undefined): boolean =>
  * @pure true
  * @complexity O(n) where n = number of headers
  */
+type HeaderOption = ClientOptions["headers"] | undefined
+
+const toHeaders = (input: HeaderOption): Headers => {
+  const headers = new Headers()
+
+  if (input === undefined) {
+    return headers
+  }
+
+  if (input instanceof Headers) {
+    return new Headers(input)
+  }
+
+  if (Array.isArray(input)) {
+    for (const pair of input) {
+      if (Array.isArray(pair) && pair.length === 2) {
+        headers.set(String(pair[0]), String(pair[1]))
+      }
+    }
+    return headers
+  }
+
+  for (const [key, value] of Object.entries(input)) {
+    if (value === undefined || value === null) {
+      continue
+    }
+    if (Array.isArray(value)) {
+      headers.set(key, value.map(String).join(","))
+      continue
+    }
+    headers.set(key, String(value))
+  }
+
+  return headers
+}
+
 const mergeHeaders = (
-  clientHeaders: HeadersInit | undefined,
-  requestHeaders: HeadersInit | undefined
+  clientHeaders: HeaderOption,
+  requestHeaders: HeaderOption
 ): Headers => {
-  const headers = new Headers(clientHeaders)
-  if (requestHeaders) {
-    const optHeaders = new Headers(requestHeaders)
-    for (const [key, value] of optHeaders.entries()) {
-      headers.set(key, value)
-    }
+  const headers = toHeaders(clientHeaders)
+  const optHeaders = toHeaders(requestHeaders)
+  for (const [key, value] of optHeaders.entries()) {
+    headers.set(key, value)
   }
   return headers
 }
@@ -197,7 +240,7 @@ type MethodHandlerOptions = {
   params?: Record<string, ParamValue> | undefined
   query?: Record<string, QueryValue> | undefined
   body?: BodyInit | object | undefined
-  headers?: HeadersInit | undefined
+  headers?: ClientOptions["headers"] | undefined
   signal?: AbortSignal | undefined
 }
 

packages/app/vite.config.ts

@@ -1,3 +1,4 @@
+import fs from "node:fs"
 import path from "node:path"
 import { fileURLToPath } from "node:url"
 import { defineConfig } from "vite"
@@ -6,6 +7,24 @@ import tsconfigPaths from "vite-tsconfig-paths"
 const __filename = fileURLToPath(import.meta.url)
 const __dirname = path.dirname(__filename)
 
+type Pkg = {
+  dependencies?: Record<string, string> | undefined
+  peerDependencies?: Record<string, string> | undefined
+}
+
+// CHANGE: Build both the library entry (src/index.ts) and the CLI entry (src/app/main.ts).
+// WHY: Consumers need a JS entrypoint in dist for `import "openapi-effect"`, while we keep the template CLI working.
+// SOURCE: n/a
+const pkgPath = path.resolve(__dirname, "package.json")
+const pkg = JSON.parse(fs.readFileSync(pkgPath, "utf8")) as Pkg
+const dependencies = [
+  ...Object.keys(pkg.dependencies ?? {}),
+  ...Object.keys(pkg.peerDependencies ?? {})
+]
+
+const isExternal = (id: string): boolean =>
+  dependencies.some((dep) => id === dep || id.startsWith(`${dep}/`))
+
 export default defineConfig({
   plugins: [tsconfigPaths()],
   publicDir: false,
@@ -18,15 +37,16 @@ export default defineConfig({
     target: "node20",
     outDir: "dist",
     sourcemap: true,
-    ssr: "src/app/main.ts",
     rollupOptions: {
+      input: {
+        index: path.resolve(__dirname, "src/index.ts"),
+        main: path.resolve(__dirname, "src/app/main.ts")
+      },
+      external: isExternal,
       output: {
         format: "es",
-        entryFileNames: "main.js"
+        entryFileNames: "[name].js"
       }
     }
-  },
-  ssr: {
-    target: "node"
   }
 })