@aria-cli/tools 1.0.9 → 1.0.11

package/tests/mcp/injectable-logger.test.ts

@@ -1,83 +0,0 @@
-/**
- * @aria/tools - Injectable Logger Tests for MCPClient
- *
- * Verifies that MCPClient uses an injectable logger instead of hardcoded
- * console.warn/error, and that the default behavior (using console) is preserved.
- *
- * aria-rfu.17: Debug cleanup + injectable logger
- */
-
-import { describe, it, expect, vi } from "vitest";
-import { MCPClient } from "../../src/mcp/client.js";
-import type { Logger } from "../../src/types.js";
-
-function createMockLogger(): Logger & {
-  warn: ReturnType<typeof vi.fn>;
-  error: ReturnType<typeof vi.fn>;
-} {
-  return {
-    warn: vi.fn(),
-    error: vi.fn(),
-  };
-}
-
-describe("MCPClient — injectable logger", () => {
-  describe("constructor", () => {
-    it("accepts a custom logger via options", () => {
-      const logger = createMockLogger();
-      const client = new MCPClient({ logger });
-
-      // Client should be created successfully
-      expect(client.getConnectedServers()).toEqual([]);
-    });
-
-    it("defaults to console when no logger is provided", () => {
-      const client = new MCPClient();
-      expect(client.getConnectedServers()).toEqual([]);
-    });
-
-    it("defaults to console when options is empty object", () => {
-      const client = new MCPClient({});
-      expect(client.getConnectedServers()).toEqual([]);
-    });
-  });
-
-  describe("logger is stored and used internally", () => {
-    it("stores custom logger and does not use console for warnings", () => {
-      const logger = createMockLogger();
-      const client = new MCPClient({ logger });
-
-      // Client should be usable — logger is stored internally
-      // We cannot easily trigger a logger.warn without connecting to a real
-      // MCP server, but we verify the client was constructed successfully
-      // with the custom logger and that it is functional.
-      expect(client.getConnectedServers()).toEqual([]);
-    });
-  });
-
-  describe("logger isolation between instances", () => {
-    it("different MCPClient instances can use different loggers", () => {
-      const logger1 = createMockLogger();
-      const logger2 = createMockLogger();
-
-      const client1 = new MCPClient({ logger: logger1 });
-      const client2 = new MCPClient({ logger: logger2 });
-
-      expect(client1).toBeDefined();
-      expect(client2).toBeDefined();
-
-      // They should be independent instances
-      expect(client1).not.toBe(client2);
-    });
-  });
-
-  describe("disconnectAll is safe with custom logger", () => {
-    it("disconnectAll works cleanly with custom logger", async () => {
-      const logger = createMockLogger();
-      const client = new MCPClient({ logger });
-
-      // disconnectAll on empty client should be safe
-      await expect(client.disconnectAll()).resolves.toBeUndefined();
-    });
-  });
-});

package/tests/mcp/jsonrpc.test.ts

@@ -1,109 +0,0 @@
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-import { PassThrough } from 'stream';
-import { JSONRPCClient } from '../../src/mcp/jsonrpc.js';
-
-describe('JSONRPCClient', () => {
-  let client: JSONRPCClient;
-  let stdin: PassThrough;
-  let stdout: PassThrough;
-
-  beforeEach(() => {
-    vi.useFakeTimers();
-    client = new JSONRPCClient({ timeout: 1000 });
-    stdin = new PassThrough();
-    stdout = new PassThrough();
-    client.connect(stdin, stdout);
-  });
-
-  afterEach(() => {
-    vi.useRealTimers();
-  });
-
-  describe('request', () => {
-    it('should send request and receive response', async () => {
-      const requestPromise = client.request('test/method', { foo: 'bar' });
-
-      // Capture sent message
-      const chunks: Buffer[] = [];
-      stdin.on('data', (chunk) => chunks.push(chunk));
-
-      // Wait for message to be sent
-      await vi.advanceTimersByTimeAsync(10);
-
-      // Parse request
-      const sentMessage = JSON.parse(chunks[0].toString().trim());
-      expect(sentMessage.jsonrpc).toBe('2.0');
-      expect(sentMessage.method).toBe('test/method');
-      expect(sentMessage.params).toEqual({ foo: 'bar' });
-
-      // Send response
-      const response = {
-        jsonrpc: '2.0',
-        id: sentMessage.id,
-        result: { success: true },
-      };
-      stdout.write(JSON.stringify(response) + '\n');
-
-      // Verify result
-      const result = await requestPromise;
-      expect(result).toEqual({ success: true });
-    });
-
-    it('should handle error response', async () => {
-      const chunks: Buffer[] = [];
-      stdin.on('data', (chunk) => chunks.push(chunk));
-
-      const requestPromise = client.request('test/error');
-
-      await vi.advanceTimersByTimeAsync(20);
-      const sentMessage = JSON.parse(chunks[0].toString().trim());
-
-      // Send error response
-      const response = {
-        jsonrpc: '2.0',
-        id: sentMessage.id,
-        error: { code: -32600, message: 'Invalid request' },
-      };
-      stdout.write(JSON.stringify(response) + '\n');
-
-      await expect(requestPromise).rejects.toThrow('-32600: Invalid request');
-    });
-
-    it('should timeout if no response', async () => {
-      const requestPromise = client.request('test/timeout');
-      const assertion = expect(requestPromise).rejects.toThrow('Request timeout');
-      await vi.advanceTimersByTimeAsync(1000);
-      await assertion;
-    });
-  });
-
-  describe('notify', () => {
-    it('should send notification without waiting for response', () => {
-      const chunks: Buffer[] = [];
-      stdin.on('data', (chunk) => chunks.push(chunk));
-
-      client.notify('test/notify', { data: 'value' });
-
-      const sent = JSON.parse(chunks[0].toString().trim());
-      expect(sent.jsonrpc).toBe('2.0');
-      expect(sent.method).toBe('test/notify');
-      expect(sent.id).toBeUndefined();
-    });
-  });
-
-  describe('notifications', () => {
-    it('should emit server notifications', async () => {
-      const notificationPromise = new Promise((resolve) => {
-        client.on('tools/list_changed', resolve);
-      });
-
-      stdout.write(JSON.stringify({
-        jsonrpc: '2.0',
-        method: 'tools/list_changed',
-        params: {},
-      }) + '\n');
-
-      await expect(notificationPromise).resolves.toEqual({});
-    });
-  });
-});