From 1e8847bb63f374acd0624ae890c9072f08aa5dfa Mon Sep 17 00:00:00 2001 From: Michal Date: Sun, 22 Feb 2026 18:43:32 +0000 Subject: [PATCH] fix: remove unused variables from profile cleanup Co-Authored-By: Claude Opus 4.6 --- src/mcpd/src/main.ts | 2 +- src/mcpd/src/services/backup/restore-service.ts | 2 +- src/mcpd/src/services/project.service.ts | 2 -- src/mcpd/tests/project-service.test.ts | 17 +---------------- 4 files changed, 3 insertions(+), 20 deletions(-) diff --git a/src/mcpd/src/main.ts b/src/mcpd/src/main.ts index 6455557..4a28f94 100644 --- a/src/mcpd/src/main.ts +++ b/src/mcpd/src/main.ts @@ -63,7 +63,7 @@ async function main(): Promise { const instanceService = new InstanceService(instanceRepo, serverRepo, orchestrator, secretRepo); serverService.setInstanceService(instanceService); const secretService = new SecretService(secretRepo); - const projectService = new ProjectService(projectRepo, serverRepo); + const projectService = new ProjectService(projectRepo); const auditLogService = new AuditLogService(auditLogRepo); const metricsCollector = new MetricsCollector(); const healthAggregator = new HealthAggregator(metricsCollector, orchestrator); diff --git a/src/mcpd/src/services/backup/restore-service.ts b/src/mcpd/src/services/backup/restore-service.ts index 072a353..ff0235d 100644 --- a/src/mcpd/src/services/backup/restore-service.ts +++ b/src/mcpd/src/services/backup/restore-service.ts @@ -114,7 +114,7 @@ export class RestoreService { if (server.packageName) createData.packageName = server.packageName; if (server.dockerImage) createData.dockerImage = server.dockerImage; if (server.repositoryUrl) createData.repositoryUrl = server.repositoryUrl; - const created = await this.serverRepo.create(createData); + await this.serverRepo.create(createData); result.serversCreated++; } catch (err) { result.errors.push(`Failed to restore server "${server.name}": ${err instanceof Error ? err.message : String(err)}`); diff --git a/src/mcpd/src/services/project.service.ts b/src/mcpd/src/services/project.service.ts index 298fbca..72a88bf 100644 --- a/src/mcpd/src/services/project.service.ts +++ b/src/mcpd/src/services/project.service.ts @@ -1,13 +1,11 @@ import type { Project } from '@prisma/client'; import type { IProjectRepository } from '../repositories/project.repository.js'; -import type { IMcpServerRepository } from '../repositories/interfaces.js'; import { CreateProjectSchema, UpdateProjectSchema } from '../validation/project.schema.js'; import { NotFoundError, ConflictError } from './mcp-server.service.js'; export class ProjectService { constructor( private readonly projectRepo: IProjectRepository, - private readonly serverRepo: IMcpServerRepository, ) {} async list(ownerId?: string): Promise { diff --git a/src/mcpd/tests/project-service.test.ts b/src/mcpd/tests/project-service.test.ts index b2870f7..5dbad2d 100644 --- a/src/mcpd/tests/project-service.test.ts +++ b/src/mcpd/tests/project-service.test.ts @@ -2,8 +2,6 @@ import { describe, it, expect, vi, beforeEach } from 'vitest'; import { ProjectService } from '../src/services/project.service.js'; import { NotFoundError, ConflictError } from '../src/services/mcp-server.service.js'; import type { IProjectRepository } from '../src/repositories/project.repository.js'; -import type { IMcpServerRepository } from '../src/repositories/interfaces.js'; - function mockProjectRepo(): IProjectRepository { return { findAll: vi.fn(async () => []), @@ -26,26 +24,13 @@ function mockProjectRepo(): IProjectRepository { }; } -function mockServerRepo(): IMcpServerRepository { - return { - findAll: vi.fn(async () => []), - findById: vi.fn(async () => null), - findByName: vi.fn(async () => null), - create: vi.fn(async () => ({} as never)), - update: vi.fn(async () => ({} as never)), - delete: vi.fn(async () => {}), - }; -} - describe('ProjectService', () => { let projectRepo: ReturnType; - let serverRepo: ReturnType; let service: ProjectService; beforeEach(() => { projectRepo = mockProjectRepo(); - serverRepo = mockServerRepo(); - service = new ProjectService(projectRepo, serverRepo); + service = new ProjectService(projectRepo); }); describe('create', () => {