refactor: extract shared types from tool files to src/types/arcane-types.ts

- Create src/types/arcane-types.ts with 33 interfaces grouped by domain
- Update 21 tool files to import from shared types instead of defining locally
- Eliminates duplicate interface definitions that could drift from API schema
- No behavioral changes: build and all 50 tests pass

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: RandomSynergy17

src/tools/build-tools.ts

@@ -7,32 +7,7 @@ import { z } from "zod";
 import { toolHandler } from "../utils/tool-helpers.js";
 import { validatePath } from "../utils/format.js";
 import { logger } from "../utils/logger.js";
-
-interface Build {
-  id: string;
-  status: string;
-  tag?: string;
-  platform?: string;
-  provider?: string;
-  startedAt?: string;
-  completedAt?: string;
-  error?: string;
-}
-
-interface BuildDetails extends Build {
-  dockerfile?: string;
-  gitUrl?: string;
-  buildArgs?: Record<string, string>;
-  logs?: string;
-}
-
-interface WorkspaceFile {
-  name: string;
-  path: string;
-  size?: number;
-  isDirectory: boolean;
-  modifiedAt?: string;
-}
+import type { Build, BuildDetails, WorkspaceFile } from "../types/arcane-types.js";
 
 export function registerBuildTools(server: McpServer): void {
 

src/tools/container-tools.ts

@@ -7,17 +7,7 @@ import { z } from "zod";
 import { toolHandler } from "../utils/tool-helpers.js";
 import { DOCKER_SHORT_ID_LENGTH, MAX_DISPLAY_LABELS, DEFAULT_PAGINATION_START, DEFAULT_PAGINATION_LIMIT } from "../constants.js";
 import { logger } from "../utils/logger.js";
-
-interface Container {
-  id: string;
-  name: string;
-  image: string;
-  status: string;
-  state: string;
-  created: string;
-  ports?: Array<{ privatePort: number; publicPort?: number; type: string }>;
-  labels?: Record<string, string>;
-}
+import type { Container } from "../types/arcane-types.js";
 
 export function registerContainerTools(server: McpServer): void {
 

src/tools/dashboard-tools.ts

@@ -6,24 +6,7 @@ import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 import { z } from "zod";
 import { toolHandler } from "../utils/tool-helpers.js";
 import { logger } from "../utils/logger.js";
-
-interface DashboardSnapshot {
-  containers: { total: number; running: number; stopped: number };
-  projects: { total: number; running: number; stopped: number };
-  images: { total: number; updatesAvailable: number };
-  volumes: { total: number; totalSize?: string };
-  networks: { total: number };
-  systemInfo?: { dockerVersion?: string; osType?: string; cpus?: number; memoryBytes?: number };
-}
-
-interface ActionItem {
-  type: string;
-  severity: string;
-  title: string;
-  description?: string;
-  resourceId?: string;
-  resourceName?: string;
-}
+import type { DashboardSnapshot, ActionItem } from "../types/arcane-types.js";
 
 export function registerDashboardTools(server: McpServer): void {
 

src/tools/environment-tools.ts

@@ -7,15 +7,7 @@ import { z } from "zod";
 import { toolHandler } from "../utils/tool-helpers.js";
 import { formatSizeGB } from "../utils/format.js";
 import { logger } from "../utils/logger.js";
-
-interface Environment {
-  id: string;
-  name: string;
-  apiUrl?: string;
-  status?: string;
-  createdAt?: string;
-  updatedAt?: string;
-}
+import type { Environment } from "../types/arcane-types.js";
 
 export function registerEnvironmentTools(server: McpServer): void {
   // arcane_environment_list

src/tools/event-tools.ts

@@ -7,19 +7,7 @@ import { z } from "zod";
 import { toolHandler } from "../utils/tool-helpers.js";
 import { MAX_DISPLAY_EVENTS } from "../constants.js";
 import { logger } from "../utils/logger.js";
-
-interface Event {
-  id: string;
-  type: string;
-  message: string;
-  resourceType?: string;
-  resourceId?: string;
-  resourceName?: string;
-  userId?: string;
-  username?: string;
-  createdAt: string;
-  metadata?: Record<string, unknown>;
-}
+import type { Event } from "../types/arcane-types.js";
 
 export function registerEventTools(server: McpServer): void {
 

src/tools/gitops-tools.ts

@@ -8,29 +8,7 @@ import { z } from "zod";
 import { toolHandler } from "../utils/tool-helpers.js";
 import { validatePath } from "../utils/format.js";
 import { logger } from "../utils/logger.js";
-
-interface GitOpsSync {
-  id: string;
-  name: string;
-  repositoryId: string;
-  branch: string;
-  path: string;
-  targetProjectId?: string;
-  lastSyncAt?: string;
-  lastSyncStatus?: string;
-  autoSync: boolean;
-  syncInterval?: number;
-}
-
-interface GitRepository {
-  id: string;
-  name: string;
-  url: string;
-  branch: string;
-  authType: string;
-  lastTestAt?: string;
-  lastTestStatus?: string;
-}
+import type { GitOpsSync, GitRepository } from "../types/arcane-types.js";
 
 export function registerGitopsTools(server: McpServer): void {
   // ============= GitOps Sync =============

src/tools/image-tools.ts

@@ -8,15 +8,7 @@ import { toolHandler } from "../utils/tool-helpers.js";
 import { formatSize, formatSizeMB, formatSizeGB } from "../utils/format.js";
 import { DOCKER_DIGEST_PREFIX_LENGTH, DOCKER_SHORT_ID_LENGTH } from "../constants.js";
 import { logger } from "../utils/logger.js";
-
-interface Image {
-  id: string;
-  repoTags: string[];
-  repoDigests?: string[];
-  created: string;
-  size: number;
-  virtualSize?: number;
-}
+import type { Image } from "../types/arcane-types.js";
 
 export function registerImageTools(server: McpServer): void {
   // arcane_image_list

src/tools/image-update-tools.ts

@@ -6,28 +6,7 @@ import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 import { z } from "zod";
 import { toolHandler } from "../utils/tool-helpers.js";
 import { logger } from "../utils/logger.js";
-
-interface ImageUpdateResponse {
-  imageRef: string;
-  currentDigest?: string;
-  latestDigest?: string;
-  updateAvailable: boolean;
-  currentTag?: string;
-  latestTag?: string;
-}
-
-interface BatchImageUpdateResponse {
-  results: Array<ImageUpdateResponse & { imageId: string }>;
-  total: number;
-  updatesAvailable: number;
-}
-
-interface ImageUpdateSummary {
-  totalImages: number;
-  checkedImages: number;
-  updatesAvailable: number;
-  lastCheckedAt?: string;
-}
+import type { ImageUpdateResponse, BatchImageUpdateResponse, ImageUpdateSummary } from "../types/arcane-types.js";
 
 export function registerImageUpdateTools(server: McpServer): void {
 

src/tools/job-tools.ts

@@ -6,17 +6,7 @@ import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 import { z } from "zod";
 import { toolHandler } from "../utils/tool-helpers.js";
 import { logger } from "../utils/logger.js";
-
-interface Job {
-  id: string;
-  name: string;
-  type: string;
-  status: string;
-  lastRunAt?: string;
-  nextRunAt?: string;
-  schedule?: string;
-  enabled: boolean;
-}
+import type { Job } from "../types/arcane-types.js";
 
 export function registerJobTools(server: McpServer): void {
   // arcane_job_list

src/tools/network-tools.ts

@@ -7,21 +7,7 @@ import { z } from "zod";
 import { toolHandler } from "../utils/tool-helpers.js";
 import { DOCKER_SHORT_ID_LENGTH } from "../constants.js";
 import { logger } from "../utils/logger.js";
-
-interface Network {
-  id: string;
-  name: string;
-  driver: string;
-  scope: string;
-  internal: boolean;
-  attachable: boolean;
-  ipam?: {
-    driver: string;
-    config?: Array<{ subnet?: string; gateway?: string }>;
-  };
-  containers?: Record<string, { name: string; ipv4Address?: string }>;
-  created?: string;
-}
+import type { Network } from "../types/arcane-types.js";
 
 export function registerNetworkTools(server: McpServer): void {