@startsimpli/api 0.2.5 → 0.3.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@startsimpli/api",
-  "version": "0.2.5",
+  "version": "0.3.0",
   "description": "Type-safe Django REST API client for StartSimpli apps",
   "main": "./src/index.ts",
   "types": "./src/index.ts",

package/src/lib/fetch-wrapper.ts

@@ -75,12 +75,15 @@ export class FetchWrapper {
       const headers = await this.buildHeaders(customHeaders);
 
       // Execute request
+      const hasAuth = headers.has('Authorization');
+      console.log(`[FetchWrapper] ${method} ${url} (auth=${hasAuth})`);
       let response = await fetch(url, {
         method,
         headers,
         credentials: 'include',
         ...fetchOptions,
       });
+      console.log(`[FetchWrapper] ${method} ${url} → ${response.status} ${response.statusText}`);
 
       // Handle 401 Unauthorized - attempt token refresh
       if (response.status === 401) {

package/src/utils/url-builder.ts

@@ -17,25 +17,39 @@ export function buildUrl({ baseUrl, endpoint, params, id }: UrlBuilderOptions):
   const cleanBase = baseUrl.replace(/\/$/, '');
   const cleanEndpoint = endpoint.replace(/^\//, '');
 
+  // Separate any inline query string from the endpoint path
+  const [pathPart, inlineQuery] = `${cleanBase}/${cleanEndpoint}`.split('?', 2);
+
   // Build path with optional ID
-  let path = `${cleanBase}/${cleanEndpoint}`;
+  let path = pathPart;
   if (id !== undefined) {
     path += `/${id}`;
   }
 
-  // Add trailing slash (Django convention)
-  if (!path.endsWith('/')) {
+  // Add trailing slash (Django convention) — only when calling Django
+  // directly (baseUrl is set). When proxied through Next.js (baseUrl empty),
+  // the rewrite rule adds the slash; adding it here causes a 308 redirect
+  // that drops the Authorization header.
+  if (baseUrl && !path.endsWith('/')) {
     path += '/';
   }
 
-  // Add query parameters
-  if (params && Object.keys(params).length > 0) {
-    const queryString = buildQueryString(params);
-    if (queryString) {
-      path += `?${queryString}`;
+  // Merge inline query params with explicit params
+  const mergedParams = new URLSearchParams(inlineQuery || '');
+  if (params) {
+    const explicit = buildQueryString(params);
+    if (explicit) {
+      for (const [k, v] of new URLSearchParams(explicit)) {
+        mergedParams.append(k, v);
+      }
     }
   }
 
+  const qs = mergedParams.toString();
+  if (qs) {
+    path += `?${qs}`;
+  }
+
   return path;
 }