@coffeerunhobby/mcp-ha-connect 2.2.1 → 2.3.1

package/src/tools/common.ts

@@ -1,442 +0,0 @@
-/**
- * Common tool utilities
- * Shared helpers for all MCP tools
- */
-
-import type { CallToolResult, ServerContext } from '@modelcontextprotocol/server';
-import { z } from 'zod';
-import { logger } from '../utils/logger.js';
-import { redactSecrets } from '../utils/redact.js';
-import { Permission, hasPermission, getPermissionNames } from '../permissions/index.js';
-
-// Re-export Permission for tool files
-export { Permission };
-
-/**
- * Local-trust flag for the stdio transport (L3 / OWASP A04:2021).
- *
- * Tool calls normally carry a permission mask in `authInfo.extra.permissions`,
- * populated by the HTTP auth middleware (bearer mask, or 0xFF for the loopback
- * `none` mode). When that mask is ABSENT we fail closed — deny — instead of the
- * old default-allow `?? 0xFF`.
- *
- * The one legitimate no-auth path is stdio (Claude Desktop): there is no per-request
- * auth and the process owner is the trusted local user. `startStdioServer` opts into
- * full trust by calling `setLocalFullTrust(true)`. HTTP never enables it.
- */
-let localFullTrust = false;
-
-export function setLocalFullTrust(enabled: boolean): void {
-  localFullTrust = enabled;
-}
-
-/**
- * Extra context passed to tool handlers
- * Permissions are extracted from authInfo.extra.permissions (MCP SDK standard)
- */
-export type ToolExtra = ServerContext;
-
-/**
- * Resolve the caller's permission mask from the MCP SDK auth context.
- *
- * L3: fail closed when the mask is absent — only the trusted-local stdio path
- * (setLocalFullTrust) grants full permissions (0xFF) without an explicit mask.
- * HTTP requests always carry a mask (bearer-derived, or 0xFF for loopback `none`).
- *
- * Exposed so tools that gate per-resource (e.g. the Omada resource-graph
- * `omada_read`/`omada_browse`) can apply the same extraction + fail-closed
- * semantics as `wrapToolHandler`, instead of re-reading `authInfo` ad hoc.
- */
-export function getCallerPermissions(extra: ToolExtra): number {
-  return (extra.http?.authInfo?.extra?.permissions as number | undefined) ?? (localFullTrust ? 0xFF : 0);
-}
-
-/**
- * Convert any value to a CallToolResult
- */
-export function toToolResult(value: unknown, isError = false): CallToolResult {
-  // Every tool response passes here: secrets returned by Home Assistant or Omada
-  // (Wi-Fi password, RADIUS/VPN keys, access tokens) never reach the client.
-  const safe = redactSecrets(value);
-  const text = typeof safe === 'string' ? safe : JSON.stringify(safe, null, 2);
-  return {
-    content: text ? [{ type: 'text' as const, text }] : [],
-    isError,
-  };
-}
-
-/**
- * Safely serialize a value for logging
- */
-export function safeSerialize(value: unknown): string {
-  try {
-    return JSON.stringify(value);
-  } catch {
-    return '[unserializable]';
-  }
-}
-
-/**
- * Static permission bit each tool declares via `wrapToolHandler`, keyed by tool
- * name. Lets the server hide tools a caller can never use from `tools/list`.
- * Tools with no static bit (e.g. `omada_read`/`omada_browse`, which gate per
- * resource path) are absent and therefore always listed.
- */
-const toolPermissions = new Map<string, number>();
-
-/** The static permission a tool requires, or `undefined` when it declares none. */
-export function getToolRequiredPermission(name: string): number | undefined {
-  return toolPermissions.get(name);
-}
-
-/**
- * Visibility rules for tools that gate per call rather than with one static bit
- * (e.g. `omada_read`, whose permission depends on the resource path). The rule
- * answers "can this caller use the tool at all?".
- */
-const toolVisibilityRules = new Map<string, (callerPermissions: number) => boolean>();
-
-/** Declare when a tool without a static permission should be listed to a caller. */
-export function setToolVisibilityRule(name: string, isUsable: (callerPermissions: number) => boolean): void {
-  toolVisibilityRules.set(name, isUsable);
-}
-
-/**
- * Whether `tools/list` should offer `name` to a caller: false when the caller lacks
- * the tool's static permission, or when its visibility rule says the caller can use
- * none of it. Tools with neither stay visible (their handlers still enforce RBAC).
- */
-export function isToolVisibleTo(name: string, callerPermissions: number): boolean {
-  const required = toolPermissions.get(name);
-  if (required !== undefined && !hasPermission(callerPermissions, required)) {
-    return false;
-  }
-  const rule = toolVisibilityRules.get(name);
-  return rule ? rule(callerPermissions) : true;
-}
-
-/**
- * Wrap a tool handler with logging, error handling, and permission checking
- * Includes session ID and args in logs for debugging
- *
- * @param name - Tool name for logging
- * @param handler - The actual tool handler function
- * @param requiredPermission - Optional permission mask required to use this tool
- */
-export function wrapToolHandler<T>(
-  name: string,
-  handler: (args: T, extra: ToolExtra) => Promise<CallToolResult>,
-  requiredPermission?: number
-): (args: T, extra: ToolExtra) => Promise<CallToolResult> {
-  if (requiredPermission !== undefined) {
-    toolPermissions.set(name, requiredPermission);
-  }
-  return async (args: T, extra: ToolExtra): Promise<CallToolResult> => {
-    const sessionId = extra.sessionId ?? 'unknown-session';
-    // Extract permissions from MCP SDK authInfo.extra.permissions.
-    // L3: fail closed when the mask is absent — only the trusted-local stdio path
-    // (setLocalFullTrust) grants full permissions without an explicit mask.
-    const userPermissions = getCallerPermissions(extra);
-
-    // Check permissions if required
-    if (requiredPermission !== undefined && !hasPermission(userPermissions, requiredPermission)) {
-      const required = getPermissionNames(requiredPermission);
-      const has = getPermissionNames(userPermissions);
-      logger.warn('Permission denied', { tool: name, sessionId, required, has });
-      return toToolResult({
-        error: 'Permission denied',
-        message: `Tool '${name}' requires permission: ${required.join(', ')}`,
-        required,
-        has,
-      }, true);
-    }
-
-    // Arguments can carry message text, entity names or MAC addresses: they are
-    // logged at debug level only, never in normal (info) operation.
-    logger.info('Tool invoked', { tool: name, sessionId });
-    logger.debug('Tool arguments', { tool: name, sessionId, args: safeSerialize(args) });
-
-    try {
-      const result = await handler(args, extra);
-      logger.info('Tool completed', { tool: name, sessionId });
-      return result;
-    } catch (error) {
-      logger.error('Tool failed', {
-        tool: name,
-        sessionId,
-        error: error instanceof Error ? error.message : String(error),
-      });
-      return toToolResult({
-        error: `Failed to execute ${name}`,
-        message: error instanceof Error ? error.message : String(error),
-      }, true);
-    }
-  };
-}
-
-/**
- * Common Zod schemas for tool inputs
- */
-
-// Empty schema for tools with no inputs
-export const emptySchema = z.object({});
-
-// Entity ID schema
-export const entityIdSchema = z.object({
-  entity_id: z.string().describe('The entity ID (e.g., "light.living_room", "sensor.temperature")'),
-});
-
-// Domain schema
-export const domainSchema = z.object({
-  domain: z.string().describe('Domain name (e.g., "light", "sensor", "switch", "climate")'),
-});
-
-// Search query schema
-export const searchSchema = z.object({
-  query: z.string().describe('Search query (searches in entity_id and friendly_name)'),
-});
-
-// History schema
-export const historySchema = z.object({
-  entity_id: z.string().describe('The entity ID to get history for'),
-  hours: z.number().optional().describe('Number of hours of history to retrieve (default: 24)'),
-});
-
-// Entity action schema
-export const entityActionSchema = z.object({
-  entity_id: z.string().describe('The entity ID to perform the action on'),
-  action: z.enum(['turn_on', 'turn_off', 'toggle']).describe('The action to perform'),
-});
-
-// List entities filter schema
-export const listEntitiesSchema = z.object({
-  domain: z.string().optional().describe('Filter by domain (e.g., "light", "switch", "sensor")'),
-  search: z.string().optional().describe('Search query to filter by entity_id or friendly_name'),
-  state: z.string().optional().describe('Filter by state (e.g., "on", "off", "unavailable")'),
-  limit: z.number().optional().describe('Maximum number of entities to return'),
-});
-
-// Automation state filter schema
-export const automationFilterSchema = z.object({
-  state: z.enum(['on', 'off']).optional().describe('Filter by automation state (enabled/disabled)'),
-});
-
-// System log schema
-export const systemLogSchema = z.object({
-  hours: z.number().optional().describe('Number of hours of history to retrieve (default: 24, max: 168)'),
-  entity_id: z.string().optional().describe('Optional entity ID to filter logs for a specific entity'),
-  limit: z.number().optional().describe('Maximum number of entries to return (default: 100)'),
-});
-
-// Automation ID schema
-export const automationIdSchema = z.object({
-  automation_id: z.string().describe('The unique ID of the automation'),
-});
-
-// Service call schema
-export const serviceCallSchema = z.object({
-  domain: z.string().describe('Service domain (e.g., "light", "switch", "climate", "notify")'),
-  service: z.string().describe('Service name (e.g., "turn_on", "turn_off", "set_temperature")'),
-  service_data: z.record(z.string(), z.unknown()).optional().describe('Optional service data'),
-  target: z.object({
-    entity_id: z.union([z.string(), z.array(z.string())]).optional(),
-    device_id: z.union([z.string(), z.array(z.string())]).optional(),
-    area_id: z.union([z.string(), z.array(z.string())]).optional(),
-  }).optional().describe('Optional target'),
-});
-
-// Analyze sensors schema
-export const analyzeSensorsSchema = z.object({
-  sensors: z.record(z.string(), z.unknown()).describe('Sensor data to analyze. Object with entity_id keys and {state, unit, attributes} values.'),
-});
-
-// Trigger automation schema
-export const triggerAutomationSchema = z.object({
-  entity_id: z.string().describe('The automation entity ID (e.g., "automation.turn_on_lights")'),
-  variables: z.record(z.string(), z.unknown()).optional().describe('Optional variables to pass to the automation'),
-});
-
-// Create automation schema
-export const createAutomationSchema = z.object({
-  alias: z.string().describe('Friendly name for the automation'),
-  description: z.string().optional().describe('Optional description of what the automation does'),
-  mode: z.enum(['single', 'restart', 'queued', 'parallel']).optional().describe('Execution mode (default: single)'),
-  trigger: z.unknown().describe('Trigger(s) that start the automation. Each trigger needs a "platform" property.'),
-  condition: z.unknown().optional().describe('Optional condition(s) that must be true for actions to run.'),
-  action: z.unknown().describe('Action(s) to perform when triggered. Each action typically has "service" and "target" properties.'),
-});
-
-// Control light schema
-export const controlLightSchema = z.object({
-  entity_id: z.string().describe('The light entity ID (e.g., "light.living_room")'),
-  action: z.enum(['turn_on', 'turn_off', 'toggle']).describe('The action to perform'),
-  brightness_pct: z.number().optional().describe('Brightness percentage (0-100)'),
-  color_temp_kelvin: z.number().optional().describe('Color temperature in Kelvin (e.g., 2700 for warm, 6500 for cool)'),
-  rgb_color: z.tuple([z.number(), z.number(), z.number()]).optional().describe('RGB color as [red, green, blue] (0-255 each)'),
-  transition: z.number().optional().describe('Transition time in seconds'),
-});
-
-// Control climate schema
-export const controlClimateSchema = z.object({
-  entity_id: z.string().describe('The climate entity ID (e.g., "climate.living_room")'),
-  temperature: z.number().optional().describe('Target temperature'),
-  hvac_mode: z.enum(['off', 'heat', 'cool', 'heat_cool', 'auto', 'dry', 'fan_only']).optional().describe('HVAC mode'),
-  fan_mode: z.string().optional().describe('Fan mode (device-specific)'),
-  preset_mode: z.string().optional().describe('Preset mode (e.g., "home", "away", "eco")'),
-});
-
-// Control media player schema
-export const controlMediaPlayerSchema = z.object({
-  entity_id: z.string().describe('The media player entity ID'),
-  action: z.enum(['play', 'pause', 'stop', 'next', 'previous', 'volume_set', 'volume_mute']).describe('The action to perform'),
-  volume_level: z.number().optional().describe('Volume level (0.0 to 1.0)'),
-  is_volume_muted: z.boolean().optional().describe('Mute state'),
-});
-
-// Control cover schema
-export const controlCoverSchema = z.object({
-  entity_id: z.string().describe('The cover entity ID (e.g., "cover.living_room_blinds")'),
-  action: z.enum(['open', 'close', 'stop', 'set_position', 'set_tilt']).describe('The action to perform'),
-  position: z.number().optional().describe('Cover position (0 = closed, 100 = open)'),
-  tilt_position: z.number().optional().describe('Tilt position (0-100)'),
-});
-
-// Control fan schema
-export const controlFanSchema = z.object({
-  entity_id: z.string().describe('The fan entity ID'),
-  action: z.enum(['turn_on', 'turn_off', 'toggle', 'set_speed', 'oscillate', 'set_direction']).describe('The action to perform'),
-  percentage: z.number().optional().describe('Speed percentage (0-100)'),
-  oscillating: z.boolean().optional().describe('Oscillation state'),
-  direction: z.enum(['forward', 'reverse']).optional().describe('Fan direction'),
-});
-
-// Run script schema
-export const runScriptSchema = z.object({
-  entity_id: z.string().describe('The script entity ID (e.g., "script.morning_routine")'),
-  variables: z.record(z.string(), z.unknown()).optional().describe('Optional variables to pass to the script'),
-});
-
-// Notification action schema for actionable notifications
-export const notificationActionSchema = z.object({
-  action: z.string().describe('Action identifier returned when tapped'),
-  title: z.string().describe('Button text shown to user'),
-  uri: z.string().optional().describe('URI to open when tapped (e.g., "https://example.com" or "app://settings")'),
-});
-
-// Send notification schema with mobile app support
-export const sendNotificationSchema = z.object({
-  message: z.string().describe('The notification message. Use "clear_notification" to clear a notification by tag.'),
-  title: z.string().optional().describe('Optional notification title'),
-  target: z.string().optional().describe('Notification service target (e.g., "mobile_app_iphone", "mobile_app_pixel"). Defaults to "persistent_notification".'),
-  // Mobile app specific options
-  tag: z.string().optional().describe('Notification tag for grouping/replacing. Use same tag to update existing notification.'),
-  group: z.string().optional().describe('Group notifications together (Android: group key, iOS: thread-id)'),
-  priority: z.enum(['high', 'normal', 'low', 'min']).optional().describe('Notification priority (Android). "high" wakes device.'),
-  importance: z.enum(['high', 'default', 'low', 'min']).optional().describe('Notification importance (Android channel importance)'),
-  ttl: z.number().optional().describe('Time to live in seconds. Notification expires after this time.'),
-  // Actions
-  actions: z.array(notificationActionSchema).max(3).optional().describe('Up to 3 action buttons for actionable notifications'),
-  // Media
-  image: z.string().optional().describe('URL to image to display in notification'),
-  video: z.string().optional().describe('URL to video (iOS only)'),
-  icon_url: z.string().optional().describe('URL to custom notification icon (Android only)'),
-  // Behavior
-  sticky: z.boolean().optional().describe('If true, notification cannot be dismissed by swiping (Android)'),
-  persistent: z.boolean().optional().describe('If true, notification persists across reboots (Android)'),
-  clickAction: z.string().optional().describe('URI to open when notification is tapped'),
-  // Sound
-  channel: z.string().optional().describe('Android notification channel ID'),
-  vibrationPattern: z.array(z.number()).optional().describe('Custom vibration pattern (Android) e.g., [100, 200, 100]'),
-  ledColor: z.string().optional().describe('LED color as hex (Android) e.g., "#FF0000"'),
-  // iOS specific
-  sound: z.string().optional().describe('Sound file name (iOS) or "default"'),
-  badge: z.number().optional().describe('App badge count (iOS)'),
-  interruptionLevel: z.enum(['passive', 'active', 'time-sensitive', 'critical']).optional().describe('iOS interruption level'),
-  // Raw data passthrough for advanced use
-  data: z.record(z.string(), z.unknown()).optional().describe('Additional raw data to pass to notification service'),
-});
-
-// List notification targets schema (empty - no params needed)
-export const listNotificationTargetsSchema = z.object({});
-
-// List calendars schema (empty - no params needed)
-export const listCalendarsSchema = z.object({});
-
-// Get calendar events schema
-export const getCalendarEventsSchema = z.object({
-  entity_id: z.string().optional().describe('Calendar entity ID (e.g., "calendar.my_calendar"). If not provided, gets events from all calendars.'),
-  start_date: z.string().optional().describe('Start date in ISO format (e.g., "2026-01-01"). Defaults to today.'),
-  end_date: z.string().optional().describe('End date in ISO format (e.g., "2026-12-31"). Defaults to 30 days from start.'),
-  days: z.number().optional().describe('Alternative: number of days from start_date (overrides end_date if provided)'),
-});
-
-// =============================================================================
-// Omada-specific schemas
-// =============================================================================
-
-// Site ID input schema (optional site selector)
-export const siteInputSchema = z.object({
-  siteId: z.string().min(1).optional().describe('Site ID (optional, uses default if not provided)'),
-});
-
-// Client ID schema (extends site input)
-export const clientIdSchema = siteInputSchema.extend({
-  clientId: z.string().min(1, 'clientId (MAC or client identifier) is required').describe('Client MAC address or identifier'),
-});
-
-// Device ID schema (extends site input)
-export const deviceIdSchema = siteInputSchema.extend({
-  deviceId: z.string().min(1, 'deviceId (MAC or device identifier) is required').describe('Device MAC address or identifier'),
-});
-
-// Stack ID schema (extends site input)
-export const stackIdSchema = siteInputSchema.extend({
-  stackId: z.string().min(1, 'stackId is required').describe('Switch stack identifier'),
-});
-
-// =============================================================================
-// Home Assistant pagination schemas
-// =============================================================================
-
-/**
- * Pagination schema for Home Assistant entity queries.
- * Reduces response size by returning paginated, lightweight entities by default.
- */
-export const haPaginationSchema = z.object({
-  page: z.number().int().min(1).default(1).describe('Page number (starts at 1)'),
-  pageSize: z.number().int().min(1).max(200).default(50).describe('Entities per page (1-200, default 50)'),
-  includeAttributes: z.boolean().default(false).describe('Include full attributes (increases response size significantly)'),
-});
-
-/**
- * Pagination parameters interface for Home Assistant queries.
- */
-export interface HAPaginationParams {
-  page: number;
-  pageSize: number;
-  includeAttributes: boolean;
-}
-
-/**
- * Apply pagination to an array and return paginated result metadata.
- */
-export function paginateArray<T>(items: T[], page: number, pageSize: number): {
-  page: number;
-  pageSize: number;
-  totalCount: number;
-  totalPages: number;
-  items: T[];
-} {
-  const totalCount = items.length;
-  const totalPages = Math.ceil(totalCount / pageSize);
-  const start = (page - 1) * pageSize;
-  const paginatedItems = items.slice(start, start + pageSize);
-
-  return {
-    page,
-    pageSize,
-    totalCount,
-    totalPages,
-    items: paginatedItems,
-  };
-}

package/src/tools/homeassistant/askOwner.ts

@@ -1,133 +0,0 @@
-/**
- * Owner question tools: askOwner / getOwnerAnswer / cancelOwnerQuestion.
- * See src/ownerQuestions/service.ts for the design.
- */
-
-import type { McpServer } from '@modelcontextprotocol/server';
-import { z } from 'zod';
-
-import { LIMITS, type OwnerQuestionService } from '../../ownerQuestions/service.js';
-import { validatePlainText } from '../../utils/plainText.js';
-import { toToolResult, wrapToolHandler, Permission, type ToolExtra } from '../common.js';
-
-const plainText = (field: string, maxLength: number, allowNewlines = false) =>
-    z.string().transform((value, ctx) => {
-        try {
-            return validatePlainText(value, { field, maxLength, allowNewlines });
-        } catch (error) {
-            ctx.addIssue({ code: 'custom', message: error instanceof Error ? error.message : String(error) });
-            return z.NEVER;
-        }
-    });
-
-export const askOwnerSchema = z.object({
-    question: plainText('question', LIMITS.questionChars, true).describe(
-        `The question, plain text only (no emoji), at most ${LIMITS.questionChars} characters; longer text is refused, not cut`
-    ),
-    title: plainText('title', LIMITS.titleChars).optional().describe(`Notification title, at most ${LIMITS.titleChars} characters (default "Approval needed")`),
-    buttons: z
-        .array(plainText('button label', LIMITS.buttonChars))
-        .min(1)
-        .max(LIMITS.maxButtons)
-        .refine((labels) => new Set(labels.map((l) => l.toLowerCase())).size === labels.length, 'Button labels must be unique')
-        .optional()
-        .describe('1-3 button labels; default ["Approve", "Deny"]. The answer reports the tapped index (0 = first).'),
-    target: z
-        .string()
-        .regex(/^mobile_app_[a-z0-9_]{1,64}$/, 'target must be a mobile app notify service, e.g. "mobile_app_pixel_7"')
-        .describe('The phone to ask: its notify service name, e.g. "mobile_app_pixel_7"'),
-    timeoutSeconds: z
-        .number()
-        .int()
-        .min(LIMITS.minTimeoutSeconds)
-        .max(LIMITS.maxTimeoutSeconds)
-        .optional()
-        .describe(`How long the question stays open (default ${LIMITS.defaultTimeoutSeconds} s, max 7 days). No answer by then = timeout, treat as NO.`),
-    channel: plainText('channel', 40).optional().describe('Android notification channel name'),
-    importance: z.enum(['high', 'default', 'low', 'min']).optional().describe('Android notification channel importance'),
-});
-
-const requestIdSchema = z.string().min(1).max(4096).describe('The requestId returned by askOwner');
-
-export const getOwnerAnswerSchema = z.object({
-    requestId: requestIdSchema,
-    waitSeconds: z
-        .number()
-        .int()
-        .min(0)
-        .max(LIMITS.maxWaitSeconds)
-        .optional()
-        .describe(`Wait up to this many seconds for an answer before returning "pending" (default 25, max ${LIMITS.maxWaitSeconds})`),
-});
-
-export const cancelOwnerQuestionSchema = z.object({ requestId: requestIdSchema });
-
-/** The token `sub` of the caller; stdio (local trust) has none. */
-function callerOf(extra: ToolExtra): string {
-    return extra.http?.authInfo?.clientId ?? 'local';
-}
-
-export function registerOwnerQuestionTools(server: McpServer, service: OwnerQuestionService): number {
-    server.registerTool(
-        'askOwner',
-        {
-            description:
-                "Ask the owner a question on their phone with up to 3 buttons (default Approve / Deny) and return immediately with a requestId. " +
-                'Then call getOwnerAnswer(requestId) until the status is not "pending". ' +
-                'Anything except an explicit approving answer must be treated as NO: deny, timeout, cancelled and unknown. ' +
-                'The owner may be away from the phone for hours; use a timeout that fits.',
-            inputSchema: askOwnerSchema,
-        },
-        wrapToolHandler(
-            'askOwner',
-            async (args: z.infer<typeof askOwnerSchema>, extra: ToolExtra) =>
-                toToolResult(
-                    await service.ask(
-                        {
-                            question: args.question,
-                            title: args.title ?? 'Approval needed',
-                            buttons: args.buttons ?? ['Approve', 'Deny'],
-                            target: args.target,
-                            timeoutSeconds: args.timeoutSeconds ?? LIMITS.defaultTimeoutSeconds,
-                            channel: args.channel,
-                            importance: args.importance,
-                        },
-                        callerOf(extra)
-                    )
-                ),
-            Permission.NOTIFY
-        )
-    );
-
-    server.registerTool(
-        'getOwnerAnswer',
-        {
-            description:
-                'Get the answer to an askOwner question, waiting up to waitSeconds. Status: "answered" (with index, button, answeredAt, answeredBy), ' +
-                '"pending" (ask again), "timeout", "cancelled" or "unknown". Only the client that asked can read the answer.',
-            inputSchema: getOwnerAnswerSchema,
-        },
-        wrapToolHandler(
-            'getOwnerAnswer',
-            async (args: z.infer<typeof getOwnerAnswerSchema>, extra: ToolExtra) =>
-                toToolResult(await service.getAnswer(args.requestId, callerOf(extra), args.waitSeconds ?? 25)),
-            Permission.NOTIFY
-        )
-    );
-
-    server.registerTool(
-        'cancelOwnerQuestion',
-        {
-            description: 'Withdraw an open askOwner question; the phone notification changes to "Cancelled". Returns the final status.',
-            inputSchema: cancelOwnerQuestionSchema,
-        },
-        wrapToolHandler(
-            'cancelOwnerQuestion',
-            async (args: z.infer<typeof cancelOwnerQuestionSchema>, extra: ToolExtra) =>
-                toToolResult(await service.cancel(args.requestId, callerOf(extra))),
-            Permission.NOTIFY
-        )
-    );
-
-    return 3;
-}

package/src/tools/homeassistant/automationControls.ts

@@ -1,66 +0,0 @@
-/**
- * Automation control tools - Enable, disable, toggle, reload automations
- */
-
-import type { McpServer } from '@modelcontextprotocol/server';
-import type { HaClient } from '../../haClient/index.js';
-import { entityIdSchema, emptySchema, toToolResult, wrapToolHandler, Permission } from '../common.js';
-import type { z } from 'zod';
-
-type EntityIdArgs = z.infer<typeof entityIdSchema>;
-
-export function registerEnableAutomationTool(server: McpServer, client: HaClient): void {
-  server.registerTool(
-    'enableAutomation',
-    {
-      description: 'Enable a disabled Home Assistant automation.',
-      inputSchema: entityIdSchema,
-    },
-    wrapToolHandler('enableAutomation', async ({ entity_id }: EntityIdArgs) => {
-      const result = await client.enableAutomation(entity_id);
-      return toToolResult({ success: true, entity_id, action: 'enabled', context: result.context });
-    }, Permission.CONFIGURE)
-  );
-}
-
-export function registerDisableAutomationTool(server: McpServer, client: HaClient): void {
-  server.registerTool(
-    'disableAutomation',
-    {
-      description: 'Disable a Home Assistant automation.',
-      inputSchema: entityIdSchema,
-    },
-    wrapToolHandler('disableAutomation', async ({ entity_id }: EntityIdArgs) => {
-      const result = await client.disableAutomation(entity_id);
-      return toToolResult({ success: true, entity_id, action: 'disabled', context: result.context });
-    }, Permission.CONFIGURE)
-  );
-}
-
-export function registerToggleAutomationTool(server: McpServer, client: HaClient): void {
-  server.registerTool(
-    'toggleAutomation',
-    {
-      description: 'Toggle a Home Assistant automation (enable if disabled, disable if enabled).',
-      inputSchema: entityIdSchema,
-    },
-    wrapToolHandler('toggleAutomation', async ({ entity_id }: EntityIdArgs) => {
-      const result = await client.toggleAutomation(entity_id);
-      return toToolResult({ success: true, entity_id, action: 'toggled', context: result.context });
-    }, Permission.CONFIGURE)
-  );
-}
-
-export function registerReloadAutomationsTool(server: McpServer, client: HaClient): void {
-  server.registerTool(
-    'reloadAutomations',
-    {
-      description: 'Reload all automations from the Home Assistant configuration. Useful after editing automation YAML files.',
-      inputSchema: emptySchema,
-    },
-    wrapToolHandler('reloadAutomations', async () => {
-      await client.reloadAutomations();
-      return toToolResult({ success: true, message: 'Automations reloaded from configuration' });
-    }, Permission.CONFIGURE)
-  );
-}