Fix json issue

Author: owjs3901

bun.lock

@@ -20,6 +20,7 @@
       "dependencies": {
         "@devup-api/fetch": "workspace:*",
         "@devup-api/next-plugin": "workspace:*",
+        "@devup-api/react-query": "workspace:*",
         "@devup-ui/react": "^1",
         "next": "^16.0.4",
         "react": "^19.2.0",
@@ -88,15 +89,15 @@
     },
     "packages/core": {
       "name": "@devup-api/core",
-      "version": "0.1.5",
+      "version": "0.1.6",
       "devDependencies": {
         "@types/node": "^24.10",
         "typescript": "^5.9",
       },
     },
     "packages/fetch": {
       "name": "@devup-api/fetch",
-      "version": "0.1.5",
+      "version": "0.1.7",
       "dependencies": {
         "@devup-api/core": "workspace:*",
       },
@@ -139,7 +140,7 @@
     },
     "packages/react-query": {
       "name": "@devup-api/react-query",
-      "version": "0.0.0",
+      "version": "0.1.0",
       "dependencies": {
         "@devup-api/fetch": "workspace:*",
         "@tanstack/react-query": ">=5.90",

examples/next/app/page.tsx

@@ -1,18 +1,49 @@
 'use client'
 
 import { createApi } from '@devup-api/fetch'
+import { createQueryClient } from '@devup-api/react-query'
 import { Box, Text } from '@devup-ui/react'
 import { useEffect } from 'react'
 
-const api = createApi('https://api.example.com')
-const api2 = createApi('https://api.example2.com', undefined, 'openapi2.json')
+const api = createApi({
+  baseUrl: 'https://api.example.com',
+})
+const api2 = createApi({
+  baseUrl: 'https://api.example2.com',
+  serverName: 'openapi2.json',
+})
+
+const queryClient = createQueryClient(api)
 
 export default function Home() {
+  const { data, isLoading, error } = queryClient.useQuery(
+    'GET',
+    'getUserById',
+    {
+      params: { id: 1 },
+      query: {
+        name: 'John Doe',
+      },
+    },
+  )
+
+  console.info(data, isLoading, error)
+
+  const {
+    data: data2,
+    isLoading: isLoading2,
+    error: error2,
+  } = queryClient.useQuery('GET', '/users', {
+    params: { id: 1 },
+  })
+
+  console.info(data2, isLoading2, error2)
+
   useEffect(() => {
-    api2.get('getUsers2', {}).then((res) => {
+    api2.get('getUsers2').then((res) => {
       console.log(res)
     })
-    api.get('getUsers', {}).then((res) => {
+    api.get('getUsers').then((res) => {
       console.log(res)
     })
 

examples/next/package.json

@@ -14,6 +14,7 @@
     "react-dom": "^19.2.0",
     "@devup-api/next-plugin": "workspace:*",
     "@devup-api/fetch": "workspace:*",
+    "@devup-api/react-query": "workspace:*",
     "@devup-ui/react": "^1"
   },
   "devDependencies": {

packages/core/src/additional.ts

@@ -7,7 +7,13 @@ export type Additional<
 
 export type RequiredOptions<T extends object> = keyof T extends undefined
   ? never
-  : T
+  : 'params' extends keyof T
+    ? T
+    : 'query' extends keyof T
+      ? T
+      : 'body' extends keyof T
+        ? T
+        : never
 export type DevupApiRequestInit = Omit<RequestInit, 'body'> & {
   body?: object | RequestInit['body']
   params?: Record<string, string | number | boolean | null | undefined>

packages/core/src/api-struct.ts

@@ -24,28 +24,48 @@ export interface DevupRequestComponentStruct {}
 // biome-ignore lint/suspicious/noEmptyInterface: empty interface
 export interface DevupResponseComponentStruct {}
 
-export type DevupApiStruct = DevupGetApiStruct &
-  DevupPostApiStruct &
-  DevupPutApiStruct &
-  DevupDeleteApiStruct &
-  DevupPatchApiStruct
+export type DevupGetApiStructScope<O extends string> = ConditionalScope<
+  DevupGetApiStruct,
+  O
+>
+export type DevupPostApiStructScope<O extends string> = ConditionalScope<
+  DevupPostApiStruct,
+  O
+>
+export type DevupPutApiStructScope<O extends string> = ConditionalScope<
+  DevupPutApiStruct,
+  O
+>
+export type DevupDeleteApiStructScope<O extends string> = ConditionalScope<
+  DevupDeleteApiStruct,
+  O
+>
+export type DevupPatchApiStructScope<O extends string> = ConditionalScope<
+  DevupPatchApiStruct,
+  O
+>
 
 export type DevupGetApiStructKey<O extends string> = ConditionalKeys<
-  ConditionalScope<DevupGetApiStruct, O>
+  DevupGetApiStructScope<O>
 >
 export type DevupPostApiStructKey<O extends string> = ConditionalKeys<
-  ConditionalScope<DevupPostApiStruct, O>
+  DevupPostApiStructScope<O>
 >
 export type DevupPutApiStructKey<O extends string> = ConditionalKeys<
-  ConditionalScope<DevupPutApiStruct, O>
+  DevupPutApiStructScope<O>
 >
 export type DevupDeleteApiStructKey<O extends string> = ConditionalKeys<
-  ConditionalScope<DevupDeleteApiStruct, O>
+  DevupDeleteApiStructScope<O>
 >
 export type DevupPatchApiStructKey<O extends string> = ConditionalKeys<
-  ConditionalScope<DevupPatchApiStruct, O>
+  DevupPatchApiStructScope<O>
 >
+export type DevupApiStructScope<O extends string> = DevupGetApiStructScope<O> &
+  DevupPostApiStructScope<O> &
+  DevupPutApiStructScope<O> &
+  DevupDeleteApiStructScope<O> &
+  DevupPatchApiStructScope<O>
 
 export type DevupApiStructKey<O extends string> = ConditionalKeys<
-  ConditionalScope<DevupApiStruct, O>
+  DevupApiStructScope<O>
 >

packages/core/src/middleware.ts

@@ -11,13 +11,13 @@ export interface MiddlewareCallbackParams {
 }
 
 type MiddlewareOnRequest = (
-  params: MiddlewareCallbackParams,
+  params: Readonly<MiddlewareCallbackParams>,
 ) => PromiseOr<undefined | Request | Response>
 type MiddlewareOnResponse = (
-  params: MiddlewareCallbackParams & { response: Response },
+  params: Readonly<MiddlewareCallbackParams & { response: Response }>,
 ) => PromiseOr<undefined | Error | Response>
 type MiddlewareOnError = (
-  params: MiddlewareCallbackParams & { error: unknown },
+  params: Readonly<MiddlewareCallbackParams & { error: unknown }>,
 ) => PromiseOr<undefined | Error | Response>
 
 export type Middleware =

packages/fetch/src/__tests__/api.test.ts

@@ -131,6 +131,28 @@ test('request does not serialize non-plain object body', async () => {
   }
 })
 
+test('request serializes plain object body to JSON with custom headers', async () => {
+  const api = new DevupApi('https://api.example.com', undefined, 'openapi.json')
+  const mockFetch = globalThis.fetch as unknown as ReturnType<typeof mock>
+
+  await api.post(
+    '/test' as never,
+    {
+      body: { name: 'test', value: 123 },
+    } as never,
+  )
+
+  expect(mockFetch).toHaveBeenCalledTimes(1)
+  const call = mockFetch.mock.calls[0]
+  expect(call).toBeDefined()
+  if (call) {
+    const request = call[0] as Request
+    const body = await request.text()
+    expect(body).toBe(JSON.stringify({ name: 'test', value: 123 }))
+    expect(request.headers.get('Content-Type')).toBe('application/json')
+  }
+})
+
 test('request merges defaultOptions with request options', async () => {
   const api = new DevupApi(
     'https://api.example.com',