@coffeerunhobby/mcp-ha-connect 2.2.1 → 2.3.1

package/tests/haClient/services.test.ts

@@ -1,120 +0,0 @@
-/**
- * ServiceOperations tests
- */
-
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-import { ServiceOperations } from '../../src/haClient/services.js';
-import type { RequestHandler } from '../../src/haClient/request.js';
-
-describe('ServiceOperations', () => {
-  let mockRequest: RequestHandler;
-  let serviceOps: ServiceOperations;
-
-  beforeEach(() => {
-    mockRequest = {
-      post: vi.fn(),
-    } as unknown as RequestHandler;
-
-    serviceOps = new ServiceOperations(mockRequest);
-  });
-
-  describe('callService', () => {
-    it('should call service with domain and service name', async () => {
-      const mockResponse = {
-        context: { id: 'ctx-123', parent_id: null, user_id: 'user-456' },
-      };
-      (mockRequest.post as ReturnType<typeof vi.fn>).mockResolvedValue(mockResponse);
-
-      const result = await serviceOps.callService({
-        domain: 'light',
-        service: 'turn_on',
-      });
-
-      expect(result).toEqual(mockResponse);
-      expect(mockRequest.post).toHaveBeenCalledWith('/services/light/turn_on', {});
-    });
-
-    it('should include target entity_id', async () => {
-      const mockResponse = {
-        context: { id: 'ctx-123', parent_id: null, user_id: null },
-      };
-      (mockRequest.post as ReturnType<typeof vi.fn>).mockResolvedValue(mockResponse);
-
-      await serviceOps.callService({
-        domain: 'light',
-        service: 'turn_on',
-        target: { entity_id: 'light.living_room' },
-      });
-
-      expect(mockRequest.post).toHaveBeenCalledWith('/services/light/turn_on', {
-        entity_id: 'light.living_room',
-      });
-    });
-
-    it('should include service_data', async () => {
-      const mockResponse = {
-        context: { id: 'ctx-123', parent_id: null, user_id: null },
-      };
-      (mockRequest.post as ReturnType<typeof vi.fn>).mockResolvedValue(mockResponse);
-
-      await serviceOps.callService({
-        domain: 'light',
-        service: 'turn_on',
-        target: { entity_id: 'light.living_room' },
-        service_data: { brightness: 255, color_temp: 370 },
-      });
-
-      expect(mockRequest.post).toHaveBeenCalledWith('/services/light/turn_on', {
-        entity_id: 'light.living_room',
-        brightness: 255,
-        color_temp: 370,
-      });
-    });
-
-    it('should handle multiple entity targets', async () => {
-      const mockResponse = {
-        context: { id: 'ctx-123', parent_id: null, user_id: null },
-      };
-      (mockRequest.post as ReturnType<typeof vi.fn>).mockResolvedValue(mockResponse);
-
-      await serviceOps.callService({
-        domain: 'light',
-        service: 'turn_off',
-        target: { entity_id: ['light.living_room', 'light.bedroom'] },
-      });
-
-      expect(mockRequest.post).toHaveBeenCalledWith('/services/light/turn_off', {
-        entity_id: ['light.living_room', 'light.bedroom'],
-      });
-    });
-
-    it('should call climate service with temperature', async () => {
-      const mockResponse = {
-        context: { id: 'ctx-123', parent_id: null, user_id: null },
-      };
-      (mockRequest.post as ReturnType<typeof vi.fn>).mockResolvedValue(mockResponse);
-
-      await serviceOps.callService({
-        domain: 'climate',
-        service: 'set_temperature',
-        target: { entity_id: 'climate.living_room' },
-        service_data: { temperature: 22 },
-      });
-
-      expect(mockRequest.post).toHaveBeenCalledWith('/services/climate/set_temperature', {
-        entity_id: 'climate.living_room',
-        temperature: 22,
-      });
-    });
-  });
-
-  describe('restartServer', () => {
-    it('should call homeassistant restart service', async () => {
-      (mockRequest.post as ReturnType<typeof vi.fn>).mockResolvedValue({});
-
-      await serviceOps.restartServer();
-
-      expect(mockRequest.post).toHaveBeenCalledWith('/services/homeassistant/restart', {});
-    });
-  });
-});

package/tests/haClient/states.test.ts

@@ -1,119 +0,0 @@
-/**
- * StateOperations tests
- */
-
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-import { StateOperations } from '../../src/haClient/states.js';
-import type { RequestHandler } from '../../src/haClient/request.js';
-import type { Entity } from '../../src/types/index.js';
-import { ApiError } from '../../src/types/index.js';
-
-describe('StateOperations', () => {
-  let mockRequest: RequestHandler;
-  let stateOps: StateOperations;
-
-  const mockEntities: Entity[] = [
-    {
-      entity_id: 'light.living_room',
-      state: 'on',
-      attributes: { friendly_name: 'Living Room Light', brightness: 255 },
-      last_changed: '2026-01-08T10:00:00.000Z',
-      last_updated: '2026-01-08T10:00:00.000Z',
-      context: { id: 'ctx-1', parent_id: null, user_id: null },
-    },
-    {
-      entity_id: 'sensor.temperature',
-      state: '22.5',
-      attributes: { friendly_name: 'Temperature', unit_of_measurement: '°C' },
-      last_changed: '2026-01-08T10:00:00.000Z',
-      last_updated: '2026-01-08T10:00:00.000Z',
-      context: { id: 'ctx-2', parent_id: null, user_id: null },
-    },
-    {
-      entity_id: 'binary_sensor.motion',
-      state: 'off',
-      attributes: { friendly_name: 'Motion Sensor', device_class: 'motion' },
-      last_changed: '2026-01-08T09:00:00.000Z',
-      last_updated: '2026-01-08T10:00:00.000Z',
-      context: { id: 'ctx-3', parent_id: null, user_id: null },
-    },
-  ];
-
-  beforeEach(() => {
-    mockRequest = {
-      get: vi.fn(),
-    } as unknown as RequestHandler;
-
-    stateOps = new StateOperations(mockRequest);
-  });
-
-  describe('getStates', () => {
-    it('should fetch all states', async () => {
-      (mockRequest.get as ReturnType<typeof vi.fn>).mockResolvedValue(mockEntities);
-
-      const states = await stateOps.getStates();
-
-      expect(states).toEqual(mockEntities);
-      expect(mockRequest.get).toHaveBeenCalledWith('/states');
-    });
-
-    it('should return empty array when no entities exist', async () => {
-      (mockRequest.get as ReturnType<typeof vi.fn>).mockResolvedValue([]);
-
-      const states = await stateOps.getStates();
-
-      expect(states).toEqual([]);
-    });
-  });
-
-  describe('getState', () => {
-    it('should fetch specific entity state', async () => {
-      const entity = mockEntities[0];
-      (mockRequest.get as ReturnType<typeof vi.fn>).mockResolvedValue(entity);
-
-      const state = await stateOps.getState('light.living_room');
-
-      expect(state).toEqual(entity);
-      expect(mockRequest.get).toHaveBeenCalledWith('/states/light.living_room');
-    });
-
-    it('should return null when entity not found', async () => {
-      const error = new ApiError('Not found', 404, { path: '/states/nonexistent' });
-      (mockRequest.get as ReturnType<typeof vi.fn>).mockRejectedValue(error);
-
-      const state = await stateOps.getState('nonexistent');
-
-      expect(state).toBeNull();
-    });
-
-    it('should throw error for non-404 errors', async () => {
-      const error = new ApiError('Server error', 500, { path: '/states/test' });
-      (mockRequest.get as ReturnType<typeof vi.fn>).mockRejectedValue(error);
-
-      await expect(stateOps.getState('test')).rejects.toThrow(ApiError);
-    });
-  });
-
-  describe('getAllSensors', () => {
-    it('should return only sensor and binary_sensor entities', async () => {
-      (mockRequest.get as ReturnType<typeof vi.fn>).mockResolvedValue(mockEntities);
-
-      const sensors = await stateOps.getAllSensors();
-
-      expect(sensors).toHaveLength(2);
-      expect(sensors.map(s => s.entity_id)).toContain('sensor.temperature');
-      expect(sensors.map(s => s.entity_id)).toContain('binary_sensor.motion');
-      expect(sensors.map(s => s.entity_id)).not.toContain('light.living_room');
-    });
-
-    it('should return empty array when no sensors exist', async () => {
-      (mockRequest.get as ReturnType<typeof vi.fn>).mockResolvedValue([
-        { ...mockEntities[0] }, // Only light entity
-      ]);
-
-      const sensors = await stateOps.getAllSensors();
-
-      expect(sensors).toEqual([]);
-    });
-  });
-});

package/tests/haClient/updates.test.ts

@@ -1,193 +0,0 @@
-/**
- * UpdateOperations tests
- */
-
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-import { UpdateOperations } from '../../src/haClient/updates.js';
-import type { EntityOperations } from '../../src/haClient/entities.js';
-import type { Entity } from '../../src/types/index.js';
-
-describe('UpdateOperations', () => {
-  let mockEntityOps: EntityOperations;
-  let updateOps: UpdateOperations;
-
-  const mockUpdateEntities: Entity[] = [
-    {
-      entity_id: 'update.home_assistant_core_update',
-      state: 'on', // Update available
-      attributes: {
-        friendly_name: 'Home Assistant Core Update',
-        title: 'Home Assistant Core',
-        installed_version: '2026.1.0',
-        latest_version: '2026.1.1',
-      },
-      last_changed: '2026-01-08T10:00:00.000Z',
-      last_updated: '2026-01-08T10:00:00.000Z',
-      context: { id: 'ctx-1', parent_id: null, user_id: null },
-    },
-    {
-      entity_id: 'update.home_assistant_supervisor_update',
-      state: 'off', // No update
-      attributes: {
-        friendly_name: 'Home Assistant Supervisor Update',
-        title: 'Home Assistant Supervisor',
-        installed_version: '2026.01.1',
-        latest_version: '2026.01.1',
-      },
-      last_changed: '2026-01-08T09:00:00.000Z',
-      last_updated: '2026-01-08T10:00:00.000Z',
-      context: { id: 'ctx-2', parent_id: null, user_id: null },
-    },
-    {
-      entity_id: 'update.home_assistant_operating_system_update',
-      state: 'on', // Update available
-      attributes: {
-        friendly_name: 'Home Assistant Operating System Update',
-        title: 'Home Assistant OS',
-        installed_version: '11.2',
-        latest_version: '11.3',
-      },
-      last_changed: '2026-01-08T08:00:00.000Z',
-      last_updated: '2026-01-08T10:00:00.000Z',
-      context: { id: 'ctx-3', parent_id: null, user_id: null },
-    },
-    {
-      entity_id: 'update.esphome_update',
-      state: 'on', // Update available
-      attributes: {
-        friendly_name: 'ESPHome Update',
-        title: 'ESPHome',
-        installed_version: '2025.12.1',
-        latest_version: '2026.1.0',
-      },
-      last_changed: '2026-01-08T07:00:00.000Z',
-      last_updated: '2026-01-08T10:00:00.000Z',
-      context: { id: 'ctx-4', parent_id: null, user_id: null },
-    },
-  ];
-
-  beforeEach(() => {
-    mockEntityOps = {
-      getEntitiesByDomain: vi.fn().mockResolvedValue(mockUpdateEntities),
-    } as unknown as EntityOperations;
-
-    updateOps = new UpdateOperations(mockEntityOps);
-  });
-
-  describe('getAvailableUpdates', () => {
-    it('should fetch update entities', async () => {
-      const result = await updateOps.getAvailableUpdates();
-
-      expect(mockEntityOps.getEntitiesByDomain).toHaveBeenCalledWith('update');
-      expect(result.entities).toEqual(mockUpdateEntities);
-    });
-
-    it('should filter to only available updates (state = on)', async () => {
-      const result = await updateOps.getAvailableUpdates();
-
-      expect(result.updates).toHaveLength(3);
-      expect(result.updates.every(u => u.version_current !== u.version_latest)).toBe(true);
-    });
-
-    it('should categorize core updates', async () => {
-      const result = await updateOps.getAvailableUpdates();
-
-      const coreUpdate = result.updates.find(u => u.update_type === 'core');
-      expect(coreUpdate).toEqual({
-        update_type: 'core',
-        name: 'Home Assistant Core',
-        version_current: '2026.1.0',
-        version_latest: '2026.1.1',
-      });
-    });
-
-    it('should categorize OS updates', async () => {
-      const result = await updateOps.getAvailableUpdates();
-
-      const osUpdate = result.updates.find(u => u.update_type === 'os');
-      expect(osUpdate).toEqual({
-        update_type: 'os',
-        name: 'Home Assistant OS',
-        version_current: '11.2',
-        version_latest: '11.3',
-      });
-    });
-
-    it('should categorize addon updates', async () => {
-      const result = await updateOps.getAvailableUpdates();
-
-      const addonUpdate = result.updates.find(u => u.update_type === 'addon');
-      expect(addonUpdate).toEqual({
-        update_type: 'addon',
-        name: 'ESPHome',
-        version_current: '2025.12.1',
-        version_latest: '2026.1.0',
-      });
-    });
-
-    it('should return empty updates when all entities are up to date', async () => {
-      (mockEntityOps.getEntitiesByDomain as ReturnType<typeof vi.fn>).mockResolvedValue([
-        {
-          entity_id: 'update.test',
-          state: 'off', // No update
-          attributes: {
-            title: 'Test',
-            installed_version: '1.0.0',
-            latest_version: '1.0.0',
-          },
-          last_changed: '2026-01-08T10:00:00.000Z',
-          last_updated: '2026-01-08T10:00:00.000Z',
-          context: { id: 'ctx-1', parent_id: null, user_id: null },
-        },
-      ]);
-
-      const result = await updateOps.getAvailableUpdates();
-
-      expect(result.updates).toEqual([]);
-      expect(result.entities).toHaveLength(1);
-    });
-
-    it('should handle entities without version info', async () => {
-      (mockEntityOps.getEntitiesByDomain as ReturnType<typeof vi.fn>).mockResolvedValue([
-        {
-          entity_id: 'update.incomplete',
-          state: 'on',
-          attributes: {
-            title: 'Incomplete',
-            // Missing installed_version and latest_version
-          },
-          last_changed: '2026-01-08T10:00:00.000Z',
-          last_updated: '2026-01-08T10:00:00.000Z',
-          context: { id: 'ctx-1', parent_id: null, user_id: null },
-        },
-      ]);
-
-      const result = await updateOps.getAvailableUpdates();
-
-      // Should not include entities without version info
-      expect(result.updates).toEqual([]);
-    });
-
-    it('should use title or friendly_name for name', async () => {
-      (mockEntityOps.getEntitiesByDomain as ReturnType<typeof vi.fn>).mockResolvedValue([
-        {
-          entity_id: 'update.with_title',
-          state: 'on',
-          attributes: {
-            title: 'Title Name',
-            friendly_name: 'Friendly Name',
-            installed_version: '1.0',
-            latest_version: '2.0',
-          },
-          last_changed: '2026-01-08T10:00:00.000Z',
-          last_updated: '2026-01-08T10:00:00.000Z',
-          context: { id: 'ctx-1', parent_id: null, user_id: null },
-        },
-      ]);
-
-      const result = await updateOps.getAvailableUpdates();
-
-      expect(result.updates[0].name).toBe('Title Name');
-    });
-  });
-});

package/tests/index.test.ts

@@ -1,160 +0,0 @@
-/**
- * Index/Main entry point tests
- */
-
-import { describe, it, expect } from 'vitest';
-import type { EnvironmentConfig } from '../src/config.js';
-
-describe('Main entry point', () => {
-  describe('Configuration loading', () => {
-    it('should require baseUrl and token', () => {
-      const validConfig: EnvironmentConfig = {
-        baseUrl: 'http://homeassistant.10.0.0.19.nip.io:8123',
-        token: 'test-token',
-        strictSsl: false,
-        timeout: 30000,
-        aiProvider: 'ollama',
-        aiUrl: 'http://ollama.10.0.0.17.nip.io:11434',
-        aiModel: 'qwen3:14b',
-        aiTimeout: 60000,
-        logLevel: 'info',
-        logFormat: 'plain',
-        useHttp: false,
-        stateful: false,
-        
-        httpEnableHealthcheck: true,
-        httpAllowCors: true,
-      };
-
-      expect(validConfig.baseUrl).toBeDefined();
-      expect(validConfig.token).toBeDefined();
-    });
-  });
-
-  describe('Logger initialization', () => {
-    it('should use stderr in stdio mode to avoid MCP protocol interference', () => {
-      const stdioConfig = {
-        useHttp: false,
-      };
-
-      const useStderr = !stdioConfig.useHttp;
-      expect(useStderr).toBe(true);
-    });
-
-    it('should use stdout in HTTP mode', () => {
-      const httpConfig = {
-        useHttp: true,
-      };
-
-      const useStderr = !httpConfig.useHttp;
-      expect(useStderr).toBe(false);
-    });
-  });
-
-  describe('Server mode selection', () => {
-    it('should start stdio server when useHttp is false', () => {
-      const config = {
-        useHttp: false,
-      };
-
-      expect(config.useHttp).toBe(false);
-      // Would call startStdioServer
-    });
-
-    it('should start HTTP server when useHttp is true', () => {
-      const config = {
-        useHttp: true,
-      };
-
-      expect(config.useHttp).toBe(true);
-      // Would call startHttpServer
-    });
-  });
-
-  describe('Configuration logging', () => {
-    it('should log configuration without exposing token', () => {
-      const config = {
-        baseUrl: 'http://homeassistant.10.0.0.19.nip.io:8123',
-        token: 'super-secret-token',
-        strictSsl: false,
-        timeout: 30000,
-        useHttp: false,
-      };
-
-      // Log data should mask the token
-      const logData = {
-        baseUrl: config.baseUrl,
-        hasToken: !!config.token,
-        strictSsl: config.strictSsl,
-        timeout: config.timeout,
-        useHttp: config.useHttp,
-      };
-
-      expect(logData.hasToken).toBe(true);
-      expect(logData).not.toHaveProperty('token');
-      expect(JSON.stringify(logData)).not.toContain('super-secret-token');
-    });
-
-    it('should include useHttp in logs', () => {
-      const config = {
-        useHttp: true,
-      };
-
-      const logData = {
-        useHttp: config.useHttp,
-      };
-
-      expect(logData.useHttp).toBe(true);
-    });
-
-    it('should show useHttp as false in stdio mode logs', () => {
-      const config = {
-        useHttp: false,
-      };
-
-      const logData = {
-        useHttp: config.useHttp,
-      };
-
-      expect(logData.useHttp).toBe(false);
-    });
-  });
-
-  describe('Error handling', () => {
-    it('should set exit code on configuration error', () => {
-      let exitCode: number | undefined;
-
-      try {
-        throw new Error('Invalid configuration');
-      } catch (error) {
-        exitCode = 1;
-      }
-
-      expect(exitCode).toBe(1);
-    });
-
-    it('should set exit code on API connection error', () => {
-      let exitCode: number | undefined;
-
-      try {
-        throw new Error('Connection failed');
-      } catch (error) {
-        exitCode = 1;
-      }
-
-      expect(exitCode).toBe(1);
-    });
-
-    it('should set exit code on server start error', () => {
-      let exitCode: number | undefined;
-
-      try {
-        throw new Error('Server start failed');
-      } catch (error) {
-        exitCode = 1;
-      }
-
-      expect(exitCode).toBe(1);
-    });
-  });
-});