fix: Bazarr CLI uses wrong API base path (#104) (#105)

Fixes double-prefixed URLs by stripping /api from base URL, since generated SDK paths already include /api/ prefix. Also corrects doctor check and response unwrapping for Bazarr's {"data": {...}} response format.

Co-authored-by: Claude Haiku 4.5 <noreply@anthropic.com>

Author: robbeverhelst

src/clients/bazarr.ts

@@ -4,8 +4,9 @@ import { client as bazarrClient } from '../generated/bazarr/client.gen.js';
 import * as BazarrApi from '../generated/bazarr/index.js';
 
 function getBazarrApiBaseUrl(baseUrl: string): string {
-  const normalizedBaseUrl = baseUrl.replace(/\/$/, '');
-  return normalizedBaseUrl.endsWith('/api') ? normalizedBaseUrl : `${normalizedBaseUrl}/api`;
+  // Generated SDK paths already include /api/ prefix, so strip it from the base URL
+  // to avoid double-prefixing (e.g. /api/api/system/status)
+  return baseUrl.replace(/\/+$/, '').replace(/\/api$/, '');
 }
 
 function getBazarrHeaders(config: ReturnType<typeof createServarrClient>) {

tests/clients-unit.test.ts

@@ -424,18 +424,18 @@ describe('Client Unit Tests', () => {
       expect(client).toBeInstanceOf(BazarrClient);
     });
 
-    it('should configure the raw client with the /api prefix', () => {
+    it('should configure the raw client without /api prefix (SDK paths include it)', () => {
       new BazarrClient(validConfig);
-      expect(bazarrApiClient.getConfig().baseUrl).toBe('http://localhost:6767/api');
+      expect(bazarrApiClient.getConfig().baseUrl).toBe('http://localhost:6767');
     });
 
-    it('should not duplicate the /api prefix when already present', () => {
+    it('should strip /api suffix when user includes it in base URL', () => {
       new BazarrClient({
         baseUrl: 'http://localhost:6767/api',
         apiKey: 'valid-api-key',
       });
 
-      expect(bazarrApiClient.getConfig().baseUrl).toBe('http://localhost:6767/api');
+      expect(bazarrApiClient.getConfig().baseUrl).toBe('http://localhost:6767');
     });
 
     it('should configure generated auth for Bazarr requests', () => {