@taiga-ai/mcp-server 0.4.0-dev.383 → 0.4.0-dev.387

package/dist/index.js

@@ -28,7 +28,7 @@ var TaigaClient = class {
 		const response = await fetch(url, {
 			method,
 			headers,
-			body: body ? JSON.stringify(body) : void 0
+			body: body ? JSON.stringify(body) : null
 		});
 		if (!response.ok) {
 			const text = await response.text().catch(() => "");
@@ -167,6 +167,21 @@ var TaigaClient = class {
 	}
 };
 //#endregion
+//#region src/config.ts
+const envSchema = z.object({
+	TAIGA_API_KEY: z.string().min(1),
+	TAIGA_API_URL: z.string().default("https://api.taiga.app")
+});
+/** Null when TAIGA_API_KEY is missing or empty: it is the one setting with no default. */
+function readConfig() {
+	const parsed = envSchema.safeParse(process.env);
+	if (!parsed.success) return null;
+	return {
+		apiKey: parsed.data.TAIGA_API_KEY,
+		apiUrl: parsed.data.TAIGA_API_URL
+	};
+}
+//#endregion
 //#region src/tools/summarize.ts
 /**
 * Strip heavy content from list responses to avoid context window bloat.
@@ -357,17 +372,16 @@ function registerAllTools(server, client) {
 }
 //#endregion
 //#region src/index.ts
-const DEFAULT_API_URL = "https://api.taiga.app";
 async function main() {
-	const apiKey = process.env.TAIGA_API_KEY;
-	if (!apiKey) {
+	const config = readConfig();
+	if (!config) {
 		console.error("Error: TAIGA_API_KEY environment variable is required.");
 		console.error("Create an API key at https://app.taiga.app/settings/api-keys");
 		process.exit(1);
 	}
 	const client = new TaigaClient({
-		baseUrl: process.env.TAIGA_API_URL ?? DEFAULT_API_URL,
-		apiKey
+		baseUrl: config.apiUrl,
+		apiKey: config.apiKey
 	});
 	const introspection = await client.introspect().catch((error) => {
 		const message = error instanceof Error ? error.message : String(error);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@taiga-ai/mcp-server",
-  "version": "0.4.0-dev.383",
+  "version": "0.4.0-dev.387",
   "description": "MCP server for Taiga -- gives AI assistants access to documents and products",
   "type": "module",
   "main": "./dist/index.js",
@@ -26,6 +26,7 @@
     "zod": "^4.4.3"
   },
   "devDependencies": {
+    "@tsconfig/strictest": "^2.0.8",
     "@types/node": "^22.20.1",
     "rolldown": "^1.2.3",
     "typescript": "^7.0.2",

package/src/client.test.ts

@@ -57,7 +57,7 @@ describe('TaigaClient', () => {
       await client.listProducts();
 
       // Assert
-      const headers = spy.mock.calls[0][1].headers;
+      const headers = spy.mock.calls[0]![1].headers;
       expect(headers.Authorization).toBe('Bearer taiga_sk_live_secret42');
     });
 
@@ -66,7 +66,7 @@ describe('TaigaClient', () => {
 
       await makeClient().listProducts();
 
-      expect(spy.mock.calls[0][1].headers['User-Agent']).toBe(`taiga-ai-mcp-server/${pkg.version}`);
+      expect(spy.mock.calls[0]![1].headers['User-Agent']).toBe(`taiga-ai-mcp-server/${pkg.version}`);
     });
   });
 
@@ -113,7 +113,7 @@ describe('TaigaClient', () => {
 
       await client.listProducts();
 
-      expect(spy.mock.calls[0][0]).toBe('https://api.test.com/v1/products');
+      expect(spy.mock.calls[0]![0]).toBe('https://api.test.com/v1/products');
     });
 
     it('should omit query string when no optional params are provided', async () => {
@@ -121,7 +121,7 @@ describe('TaigaClient', () => {
 
       await makeClient().listProducts();
 
-      expect(spy.mock.calls[0][0]).toBe('https://api.test.com/v1/products');
+      expect(spy.mock.calls[0]![0]).toBe('https://api.test.com/v1/products');
     });
 
     it('should include all optional filters as query params', async () => {
@@ -129,7 +129,7 @@ describe('TaigaClient', () => {
 
       await makeClient().listProducts({ factoryId: 't1', status: 'active', limit: 10 });
 
-      const url = spy.mock.calls[0][0] as string;
+      const url = spy.mock.calls[0]![0] as string;
       expect(url).toContain('factoryId=t1');
       expect(url).toContain('status=active');
       expect(url).toContain('limit=10');
@@ -140,7 +140,7 @@ describe('TaigaClient', () => {
 
       await makeClient().listBacklogItems('proj-1', { limit: 20 });
 
-      const url = spy.mock.calls[0][0] as string;
+      const url = spy.mock.calls[0]![0] as string;
       expect(url).toContain('/v1/products/proj-1/backlog');
       expect(url).toContain('limit=20');
     });
@@ -150,7 +150,7 @@ describe('TaigaClient', () => {
 
       await makeClient().listEnvironments('proj-1');
 
-      expect(spy.mock.calls[0][0]).toBe('https://api.test.com/v1/products/proj-1/environments');
+      expect(spy.mock.calls[0]![0]).toBe('https://api.test.com/v1/products/proj-1/environments');
     });
   });
 
@@ -211,7 +211,7 @@ describe('TaigaClient', () => {
 
       const result = await makeClient().getProduct('p1');
 
-      expect(spy.mock.calls[0][0]).toBe('https://api.test.com/v1/products/p1');
+      expect(spy.mock.calls[0]![0]).toBe('https://api.test.com/v1/products/p1');
       expect(result.data).toEqual({ id: 'p1', name: 'Alpha' });
     });
 
@@ -220,7 +220,7 @@ describe('TaigaClient', () => {
 
       await makeClient().getTenantDocument('security_policy');
 
-      expect(spy.mock.calls[0][0]).toBe('https://api.test.com/v1/tenants/current/documents/security_policy');
+      expect(spy.mock.calls[0]![0]).toBe('https://api.test.com/v1/tenants/current/documents/security_policy');
     });
 
     it('should PUT a document draft with the provided body', async () => {
@@ -228,9 +228,9 @@ describe('TaigaClient', () => {
 
       await makeClient().saveTenantDocumentDraft('security_policy', { data: { title: 'New' } });
 
-      expect(spy.mock.calls[0][1].method).toBe('PUT');
-      expect(spy.mock.calls[0][0]).toContain('/tenants/current/documents/security_policy/draft');
-      expect(JSON.parse(spy.mock.calls[0][1].body)).toEqual({
+      expect(spy.mock.calls[0]![1].method).toBe('PUT');
+      expect(spy.mock.calls[0]![0]).toContain('/tenants/current/documents/security_policy/draft');
+      expect(JSON.parse(spy.mock.calls[0]![1].body)).toEqual({
         data: { title: 'New' },
       });
     });
@@ -240,7 +240,7 @@ describe('TaigaClient', () => {
 
       await makeClient().getBacklogItem('item-1');
 
-      expect(spy.mock.calls[0][0]).toBe('https://api.test.com/v1/backlog/item-1');
+      expect(spy.mock.calls[0]![0]).toBe('https://api.test.com/v1/backlog/item-1');
     });
 
     it('should GET factories with optional limit', async () => {
@@ -248,7 +248,7 @@ describe('TaigaClient', () => {
 
       await makeClient().listFactories({ limit: 50 });
 
-      expect(spy.mock.calls[0][0]).toContain('/v1/factories?limit=50');
+      expect(spy.mock.calls[0]![0]).toContain('/v1/factories?limit=50');
     });
 
     it('should GET documents from tenant path with optional limit', async () => {
@@ -256,7 +256,7 @@ describe('TaigaClient', () => {
 
       await makeClient().listTenantDocuments({ limit: 5 });
 
-      expect(spy.mock.calls[0][0]).toContain('/v1/tenants/current/documents?limit=5');
+      expect(spy.mock.calls[0]![0]).toContain('/v1/tenants/current/documents?limit=5');
     });
   });
 });

package/src/client.ts

@@ -26,7 +26,7 @@ export class TaigaClient {
     const response = await fetch(url, {
       method,
       headers,
-      body: body ? JSON.stringify(body) : undefined,
+      body: body ? JSON.stringify(body) : null,
     });
 
     if (!response.ok) {
@@ -60,7 +60,11 @@ export class TaigaClient {
   // Products
   // ---------------------------------------------------------------------------
 
-  async listProducts(opts?: { factoryId?: string; status?: string; limit?: number }) {
+  async listProducts(opts?: {
+    factoryId?: string | undefined;
+    status?: string | undefined;
+    limit?: number | undefined;
+  }) {
     const params = new URLSearchParams();
     if (opts?.factoryId) params.set('factoryId', opts.factoryId);
     if (opts?.status) params.set('status', opts.status);
@@ -77,7 +81,7 @@ export class TaigaClient {
   // Documents (tenant-level policies)
   // ---------------------------------------------------------------------------
 
-  async listTenantDocuments(opts?: { limit?: number }) {
+  async listTenantDocuments(opts?: { limit?: number | undefined }) {
     const params = new URLSearchParams();
     if (opts?.limit) params.set('limit', String(opts.limit));
     const qs = params.toString();
@@ -99,7 +103,7 @@ export class TaigaClient {
   // Factory Documents
   // ---------------------------------------------------------------------------
 
-  async listFactoryDocuments(factoryId: string, opts?: { limit?: number }) {
+  async listFactoryDocuments(factoryId: string, opts?: { limit?: number | undefined }) {
     const params = new URLSearchParams();
     if (opts?.limit) params.set('limit', String(opts.limit));
     const qs = params.toString();
@@ -121,7 +125,7 @@ export class TaigaClient {
   // Product Documents
   // ---------------------------------------------------------------------------
 
-  async listProductDocuments(productId: string, opts?: { limit?: number }) {
+  async listProductDocuments(productId: string, opts?: { limit?: number | undefined }) {
     const params = new URLSearchParams();
     if (opts?.limit) params.set('limit', String(opts.limit));
     const qs = params.toString();
@@ -214,7 +218,7 @@ export class TaigaClient {
   // Backlog
   // ---------------------------------------------------------------------------
 
-  async listBacklogItems(productId: string, opts?: { limit?: number }) {
+  async listBacklogItems(productId: string, opts?: { limit?: number | undefined }) {
     const params = new URLSearchParams();
     if (opts?.limit) params.set('limit', String(opts.limit));
     const qs = params.toString();
@@ -240,7 +244,7 @@ export class TaigaClient {
   // Factories
   // ---------------------------------------------------------------------------
 
-  async listFactories(opts?: { limit?: number }) {
+  async listFactories(opts?: { limit?: number | undefined }) {
     const params = new URLSearchParams();
     if (opts?.limit) params.set('limit', String(opts.limit));
     const qs = params.toString();

package/src/config.test.ts

@@ -0,0 +1,24 @@
+import { afterEach, describe, expect, it, vi } from 'vitest';
+
+import { readConfig } from './config.js';
+
+afterEach(() => {
+  vi.unstubAllEnvs();
+});
+
+describe('readConfig', () => {
+  it('needs an API key', () => {
+    vi.stubEnv('TAIGA_API_KEY', undefined);
+    expect(readConfig()).toBeNull();
+    vi.stubEnv('TAIGA_API_KEY', '');
+    expect(readConfig()).toBeNull();
+  });
+
+  it('defaults the API URL', () => {
+    vi.stubEnv('TAIGA_API_KEY', 'taiga_sk_live_test');
+    vi.stubEnv('TAIGA_API_URL', undefined);
+    expect(readConfig()).toEqual({ apiKey: 'taiga_sk_live_test', apiUrl: 'https://api.taiga.app' });
+    vi.stubEnv('TAIGA_API_URL', 'http://localhost:3001');
+    expect(readConfig()?.apiUrl).toBe('http://localhost:3001');
+  });
+});

package/src/config.ts

@@ -0,0 +1,25 @@
+import { z } from 'zod';
+
+/**
+ * The one place the MCP server reads its environment (PRO-2511), parsed once
+ * at startup.
+ */
+
+const DEFAULT_API_URL = 'https://api.taiga.app';
+
+const envSchema = z.object({
+  TAIGA_API_KEY: z.string().min(1),
+  TAIGA_API_URL: z.string().default(DEFAULT_API_URL),
+});
+
+export interface ServerConfig {
+  readonly apiKey: string;
+  readonly apiUrl: string;
+}
+
+/** Null when TAIGA_API_KEY is missing or empty: it is the one setting with no default. */
+export function readConfig(): ServerConfig | null {
+  const parsed = envSchema.safeParse(process.env);
+  if (!parsed.success) return null;
+  return { apiKey: parsed.data.TAIGA_API_KEY, apiUrl: parsed.data.TAIGA_API_URL };
+}

package/src/index.ts

@@ -3,22 +3,19 @@
 import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
 import { StdioServerTransport } from '@modelcontextprotocol/sdk/server/stdio.js';
 import { TaigaClient } from './client.js';
+import { readConfig } from './config.js';
 import { registerAllTools } from './tools/index.js';
 import { SERVER_VERSION } from './version.js';
 
-const DEFAULT_API_URL = 'https://api.taiga.app';
-
 async function main() {
-  const apiKey = process.env.TAIGA_API_KEY;
-  if (!apiKey) {
+  const config = readConfig();
+  if (!config) {
     console.error('Error: TAIGA_API_KEY environment variable is required.');
     console.error('Create an API key at https://app.taiga.app/settings/api-keys');
     process.exit(1);
   }
 
-  const apiUrl = process.env.TAIGA_API_URL ?? DEFAULT_API_URL;
-
-  const client = new TaigaClient({ baseUrl: apiUrl, apiKey });
+  const client = new TaigaClient({ baseUrl: config.apiUrl, apiKey: config.apiKey });
 
   // Introspect the API key to discover available capabilities
   const introspection = await client.introspect().catch((error) => {

package/src/server.integration.test.ts

@@ -110,7 +110,7 @@ function toolNames(tools: { name: string }[]) {
 /** Extract parsed JSON from the first text content block of a tool call result. */
 function parseToolResult(result: Record<string, unknown>) {
   const content = result.content as { type: string; text: string }[];
-  return JSON.parse(content[0].text);
+  return JSON.parse(content[0]!.text);
 }
 
 // ---------------------------------------------------------------------------

package/tsconfig.json

@@ -1,17 +1,19 @@
 {
+  "extends": "@tsconfig/strictest/tsconfig.json",
   "compilerOptions": {
     "target": "ES2022",
     "module": "ESNext",
     "moduleResolution": "bundler",
     "declaration": true,
     "declarationMap": true,
-    "strict": true,
-    "skipLibCheck": true,
     "outDir": "./dist",
     "rootDir": "./src",
-    "esModuleInterop": true,
     "resolveJsonModule": true,
-    "types": ["node"]
+    "types": ["node"],
+
+    // Preset flags this package still fails (PRO-2509). Each line is deleted
+    // by the issue named beside it; no new line is added here.
+    "noPropertyAccessFromIndexSignature": false // PRO-2519
   },
   "include": ["src"],
   "exclude": ["node_modules", "dist"],