fix: exclude db tests from workspace root and fix TS build errors

- Exclude src/db/tests from workspace vitest config (needs test DB)
- Make global-setup.ts gracefully skip when test DB unavailable
- Fix exactOptionalPropertyTypes issues in proxymodel-endpoint.ts
- Use proper ProxyModelPlugin type for getPluginHooks function

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Michal
2026-03-07 00:39:25 +00:00
parent a22a17f8d3
commit cfe0d99c8f
3 changed files with 24 additions and 15 deletions

View File

@@ -8,13 +8,19 @@ const TEST_DATABASE_URL = process.env['DATABASE_URL'] ??
'postgresql://mcpctl:mcpctl_test@localhost:5433/mcpctl_test';
export function setup(): void {
execSync('npx prisma db push --force-reset --skip-generate', {
cwd: new URL('..', import.meta.url).pathname,
env: {
...process.env,
DATABASE_URL: TEST_DATABASE_URL,
PRISMA_USER_CONSENT_FOR_DANGEROUS_AI_ACTION: 'yes',
},
stdio: 'pipe',
});
try {
execSync('npx prisma db push --force-reset --skip-generate', {
cwd: new URL('..', import.meta.url).pathname,
env: {
...process.env,
DATABASE_URL: TEST_DATABASE_URL,
PRISMA_USER_CONSENT_FOR_DANGEROUS_AI_ACTION: 'yes',
},
stdio: 'pipe',
});
} catch {
// Test DB not available — db-specific tests will fail individually,
// but non-db tests can still run.
console.warn('[global-setup] Test database not available, skipping schema push');
}
}

View File

@@ -7,6 +7,7 @@
import type { FastifyInstance } from 'fastify';
import { loadProxyModels } from '../proxymodel/loader.js';
import { loadPlugins } from '../proxymodel/plugin-loader.js';
import type { ProxyModelPlugin } from '../proxymodel/plugin.js';
import { createGatePlugin } from '../proxymodel/plugins/gate.js';
import { createContentPipelinePlugin } from '../proxymodel/plugins/content-pipeline.js';
import { createDefaultPlugin } from '../proxymodel/plugins/default.js';
@@ -59,9 +60,9 @@ export function registerProxymodelEndpoint(app: FastifyInstance): void {
name: entry.name,
source: entry.source,
type: 'plugin',
extends: entry.plugin.extends,
extends: entry.plugin.extends ?? [],
hooks: getPluginHooks(entry.plugin),
description: entry.plugin.description,
description: entry.plugin.description ?? '',
});
}
@@ -108,11 +109,11 @@ export function registerProxymodelEndpoint(app: FastifyInstance): void {
});
}
function getPluginHooks(plugin: { [key: string]: unknown }): string[] {
function getPluginHooks(plugin: ProxyModelPlugin): string[] {
const hookNames = [
'onSessionCreate', 'onSessionDestroy', 'onInitialize',
'onToolsList', 'onToolCallBefore', 'onToolCallAfter',
'onResourcesList', 'onResourceRead', 'onPromptsList', 'onPromptGet',
];
] as const;
return hookNames.filter((h) => typeof plugin[h] === 'function');
}