Keep keyless mode with a trailing slash in the cloud API URL

src/__tests__/commands/parse.test.ts

@@ -45,7 +45,7 @@ describe('executeParse', () => {
   });
 
   it('posts to /v2/parse without auth when using the default cloud API with no key', async () => {
-    initializeConfig({ apiUrl: 'https://api.firecrawl.dev' });
+    initializeConfig({ apiUrl: 'https://api.firecrawl.dev///' });
 
     const result = await executeParse({ file: filePath });
 

src/__tests__/utils/client.test.ts

@@ -0,0 +1,45 @@
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
+import { keylessGet, keylessRequest } from '../../utils/client';
+import { initializeConfig, resetConfig } from '../../utils/config';
+
+vi.mock('../../utils/credentials', () => ({
+  loadCredentials: vi.fn(() => null),
+}));
+
+describe('keyless requests', () => {
+  const fetchMock = vi.fn();
+
+  beforeEach(() => {
+    resetConfig();
+    initializeConfig({ apiUrl: 'https://api.firecrawl.dev///' });
+    fetchMock.mockResolvedValue({
+      ok: true,
+      json: vi.fn().mockResolvedValue({ success: true }),
+    });
+    vi.stubGlobal('fetch', fetchMock);
+  });
+
+  afterEach(() => {
+    vi.unstubAllGlobals();
+    vi.clearAllMocks();
+    resetConfig();
+  });
+
+  it('joins POST paths after every trailing slash is removed', async () => {
+    await keylessRequest('/v2/scrape', { url: 'https://example.com' });
+
+    expect(fetchMock).toHaveBeenCalledWith(
+      'https://api.firecrawl.dev/v2/scrape',
+      expect.objectContaining({ method: 'POST' })
+    );
+  });
+
+  it('joins GET paths after every trailing slash is removed', async () => {
+    await keylessGet('/v2/search/research');
+
+    expect(fetchMock).toHaveBeenCalledWith(
+      'https://api.firecrawl.dev/v2/search/research',
+      expect.objectContaining({ method: 'GET' })
+    );
+  });
+});

src/__tests__/utils/config.test.ts

@@ -11,7 +11,7 @@ import {
   validateConfig,
   isCustomApiUrl,
 } from '../../utils/config';
-import { getClient, resetClient } from '../../utils/client';
+import { getClient, isKeylessMode, resetClient } from '../../utils/client';
 import * as credentials from '../../utils/credentials';
 
 // Mock credentials module
@@ -240,6 +240,12 @@ describe('Config Fallback Priority', () => {
       expect(isCustomApiUrl()).toBe(false);
     });
 
+    it('should ignore trailing slashes on the default cloud API URL', () => {
+      initializeConfig({ apiUrl: 'https://api.firecrawl.dev/' });
+      expect(isCustomApiUrl()).toBe(false);
+      expect(isCustomApiUrl('https://api.firecrawl.dev///')).toBe(false);
+    });
+
     it('should return true for custom API URLs', () => {
       initializeConfig({ apiUrl: 'http://localhost:3002' });
       expect(isCustomApiUrl()).toBe(true);
@@ -254,6 +260,16 @@ describe('Config Fallback Priority', () => {
       initializeConfig({ apiUrl: 'https://api.firecrawl.dev' });
       expect(isCustomApiUrl('http://localhost:3002')).toBe(true);
     });
+
+    it('should keep custom API URLs custom when they have trailing slashes', () => {
+      expect(isCustomApiUrl('http://localhost:3002/')).toBe(true);
+    });
+  });
+
+  describe('keyless mode with cloud API URLs', () => {
+    it('should allow a trailing slash on the default cloud API URL', () => {
+      expect(isKeylessMode(undefined, 'https://api.firecrawl.dev/')).toBe(true);
+    });
   });
 
   describe('validateConfig with custom API URLs', () => {
@@ -268,6 +284,11 @@ describe('Config Fallback Priority', () => {
       expect(() => validateConfig()).toThrow('API key is required');
     });
 
+    it('should require API key for cloud API URL with a trailing slash', () => {
+      initializeConfig({ apiUrl: 'https://api.firecrawl.dev/' });
+      expect(() => validateConfig()).toThrow('API key is required');
+    });
+
     it('should not throw when API key is provided for cloud API', () => {
       initializeConfig({
         apiUrl: 'https://api.firecrawl.dev',

src/commands/interact.ts

@@ -4,7 +4,7 @@
  */
 
 import { getClient, isKeylessMode } from '../utils/client';
-import { getConfig, validateConfig } from '../utils/config';
+import { getConfig, normalizeApiUrl, validateConfig } from '../utils/config';
 import {
   getScrapeId,
   loadInteractSession,
@@ -46,7 +46,7 @@ function resolveApiConfig(options: { apiKey?: string; apiUrl?: string }) {
     validateConfig(apiKey);
   }
   const apiUrl = options.apiUrl || config.apiUrl || 'https://api.firecrawl.dev';
-  return { apiKey, apiUrl: apiUrl.replace(/\/$/, ''), keyless };
+  return { apiKey, apiUrl: normalizeApiUrl(apiUrl), keyless };
 }
 
 /**

src/commands/parse.ts

@@ -12,7 +12,7 @@ import type { FormatOption } from 'firecrawl';
 import type { ParseOptions, ParseResult } from '../types/parse';
 import type { ScrapeFormat } from '../types/scrape';
 import { getClient, isKeylessMode } from '../utils/client';
-import { getConfig, validateConfig } from '../utils/config';
+import { getConfig, normalizeApiUrl, validateConfig } from '../utils/config';
 import { handleScrapeOutput } from '../utils/output';
 
 const DEFAULT_API_URL = 'https://api.firecrawl.dev';
@@ -162,9 +162,8 @@ export async function executeParse(
     validateConfig(apiKey);
   }
 
-  const apiUrl = (options.apiUrl || config.apiUrl || DEFAULT_API_URL).replace(
-    /\/$/,
-    ''
+  const apiUrl = normalizeApiUrl(
+    options.apiUrl || config.apiUrl || DEFAULT_API_URL
   );
 
   const buffer = fs.readFileSync(filePath);

src/utils/client.ts

@@ -11,6 +11,7 @@ import {
   isCustomApiUrl,
   validateConfig,
   updateConfig,
+  normalizeApiUrl,
   type GlobalConfig,
 } from './config';
 
@@ -33,7 +34,7 @@ export async function keylessRequest(
   path: string,
   body: Record<string, unknown>
 ): Promise<any> {
-  const apiUrl = (getConfig().apiUrl || DEFAULT_API_URL).replace(/\/$/, '');
+  const apiUrl = normalizeApiUrl(getConfig().apiUrl || DEFAULT_API_URL);
   const response = await fetch(`${apiUrl}${path}`, {
     method: 'POST',
     headers: { 'Content-Type': 'application/json' },
@@ -49,7 +50,7 @@ export async function keylessRequest(
 }
 
 export async function keylessGet(path: string): Promise<any> {
-  const apiUrl = (getConfig().apiUrl || DEFAULT_API_URL).replace(/\/$/, '');
+  const apiUrl = normalizeApiUrl(getConfig().apiUrl || DEFAULT_API_URL);
   const response = await fetch(`${apiUrl}${path}`, {
     method: 'GET',
     headers: { 'Content-Type': 'application/json' },

src/utils/config.ts

@@ -74,12 +74,17 @@ export function getApiKey(providedKey?: string): string | undefined {
 
 const DEFAULT_API_URL = 'https://api.firecrawl.dev';
 
+/** Remove trailing slashes before comparing or joining an API base URL. */
+export function normalizeApiUrl(apiUrl: string): string {
+  return apiUrl.replace(/\/+$/, '');
+}
+
 /**
  * Check if using a custom (non-cloud) API URL
  */
 export function isCustomApiUrl(apiUrl?: string): boolean {
   const url = apiUrl || globalConfig.apiUrl;
-  return !!url && url !== DEFAULT_API_URL;
+  return !!url && normalizeApiUrl(url) !== DEFAULT_API_URL;
 }
 
 /**