@copado/agentia-cli 1.1.0 → 1.2.0

package/dist/lib/mcp/server.js

@@ -2,6 +2,8 @@ import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
 import { StdioServerTransport } from '@modelcontextprotocol/sdk/server/stdio.js';
 import { createRequire } from 'node:module';
 import process from 'node:process';
+import { z } from 'zod';
+import { logVerboseDebug } from '../debug.js';
 import { migrateUserSettingsFromProjectConfig } from '../defaults-store.js';
 import { applyProjectRoot, hasExplicitProjectRoot } from '../project-root.js';
 import { runWithMcpRequestSignal } from './request-context.js';
@@ -10,15 +12,71 @@ import { withToolAnnotations } from './tool-annotations.js';
 import { registerAgentiaMcpTools } from './tools.js';
 const require = createRequire(import.meta.url);
 const packageJson = require('../../../package.json');
+/**
+ * SDK ListRootsResultSchema requires every uri to start with file://.
+ * Some clients send empty or non-file URIs; accept them here and filter below.
+ */
+const LenientListRootsResultSchema = z
+    .object({
+    roots: z.array(z.unknown()).optional().default([]),
+})
+    .passthrough();
+/** Keep only usable local file roots; empty / non-file:// entries are ignored. */
+export function collectMcpFileRootUris(roots) {
+    const uris = [];
+    for (const root of roots) {
+        if (root === null || typeof root !== 'object')
+            continue;
+        const { uri } = root;
+        if (typeof uri !== 'string')
+            continue;
+        const trimmed = uri.trim();
+        if (trimmed.startsWith('file://'))
+            uris.push(trimmed);
+    }
+    return uris;
+}
 export function createAgentiaMcpServer(lifecycleSignal, projectContextReady = Promise.resolve()) {
     const server = new McpServer({
         name: packageJson.name ?? 'agentia-cli',
         version: packageJson.version,
     });
+    installMcpClientCallDebugLogging(server.server);
     wrapMcpToolRegistration(server, lifecycleSignal, projectContextReady);
     registerAgentiaMcpTools(server);
     return server;
 }
+/**
+ * Log every server→client MCP request/notification when DEBUG=verbose.
+ * Covers roots/list and any future client calls made through the protocol.
+ */
+export function installMcpClientCallDebugLogging(protocol) {
+    const target = protocol;
+    const originalRequest = target.request.bind(target);
+    target.request = async (request, resultSchema, options) => {
+        logVerboseDebug('MCP client request', request);
+        try {
+            const result = await originalRequest(request, resultSchema, options);
+            logVerboseDebug('MCP client response', {
+                method: typeof request?.method === 'string' ? request.method : undefined,
+                result,
+            });
+            return result;
+        }
+        catch (error) {
+            logVerboseDebug('MCP client error', {
+                error: error instanceof Error ? error.message : String(error),
+                method: typeof request?.method === 'string' ? request.method : undefined,
+            });
+            throw error;
+        }
+    };
+    const originalNotification = target.notification.bind(target);
+    target.notification = async (notification, options) => {
+        logVerboseDebug('MCP client notification', notification);
+        await originalNotification(notification, options);
+    };
+}
 function combineSignals(requestSignal, lifecycleSignal) {
     if (requestSignal === undefined || lifecycleSignal === undefined) {
         return { dispose() { }, signal: requestSignal ?? lifecycleSignal };
@@ -61,8 +119,19 @@ function wrapMcpToolRegistration(server, lifecycleSignal, projectContextReady =
 function clientSupportsRoots(server) {
     return (server.server._clientCapabilities?.roots !== undefined);
 }
+async function listMcpFileRootUris(server) {
+    try {
+        const result = await server.server.request({ method: 'roots/list' }, LenientListRootsResultSchema);
+        return collectMcpFileRootUris(result.roots);
+    }
+    catch (error) {
+        const message = error instanceof Error ? error.message : String(error);
+        process.stderr.write(`[agentia-mcp] ignoring invalid MCP roots/list response: ${message}\n`);
+        return undefined;
+    }
+}
 async function applyMcpProjectRoot(server, includeMcpRoots) {
-    const rootUris = includeMcpRoots ? (await server.server.listRoots()).roots.map((root) => root.uri) : undefined;
+    const rootUris = includeMcpRoots ? await listMcpFileRootUris(server) : undefined;
     applyProjectRoot(undefined, rootUris);
     await migrateUserSettingsFromProjectConfig().catch(() => {
         // Best-effort; MCP tools still fall back to reading legacy project keys.

package/dist/lib/mcp/testing-live-tools.js

@@ -2,15 +2,22 @@ import { mkdtemp } from 'node:fs/promises';
 import { tmpdir } from 'node:os';
 import { join } from 'node:path';
 import { z } from 'zod';
-import { controlTestingLive, executeTestingLive, getTestingLiveHistory, getTestingLiveReports, getTestingLiveStatus, inspectTestingLive, saveTestingLiveScreenshot, setTestingLiveOptions, startTestingLive, stopTestingLive, } from '../testing-live-operations.js';
+import { controlTestingLive, executeTestingLive, getTestingLiveHistory, getTestingLiveReports, getTestingLiveStatus, inspectTestingLive, parseTestingLiveScreen, saveTestingLiveScreenshot, setTestingLiveOptions, startTestingLive, stopTestingLive, } from '../testing-live-operations.js';
 import { TESTING_LIVE_LIBRARIES } from '../testing-live-types.js';
 import { safelyHandleTool } from './result.js';
 async function testingLiveStart(args) {
-    const screenValues = [args.screenWidth, args.screenHeight, args.screenDepth];
-    const supplied = screenValues.filter((value) => value !== undefined);
-    if (supplied.length > 0 && supplied.length !== 3) {
-        throw new Error('Provide screenWidth, screenHeight, and screenDepth together.');
+    const hasScreenParts = args.screenDepth !== undefined || args.screenHeight !== undefined || args.screenWidth !== undefined;
+    if (hasScreenParts && (args.screen !== undefined || args.viewport !== undefined)) {
+        throw new Error('Use either screen/viewport or screenWidth/screenHeight/screenDepth, not both.');
     }
+    let screenInput = args.screen;
+    if (hasScreenParts) {
+        if (args.screenDepth === undefined || args.screenHeight === undefined || args.screenWidth === undefined) {
+            throw new Error('Provide screenWidth, screenHeight, and screenDepth together.');
+        }
+        screenInput = { depth: args.screenDepth, height: args.screenHeight, width: args.screenWidth };
+    }
+    const screen = parseTestingLiveScreen(screenInput, args.viewport);
     return {
         result: await startTestingLive({
             description: args.description,
@@ -21,9 +28,7 @@ async function testingLiveStart(args) {
             noBrowser: args.noBrowser,
             pollIntervalMs: args.pollIntervalSeconds === undefined ? undefined : args.pollIntervalSeconds * 1000,
             projectId: args.project,
-            screen: supplied.length === 0
-                ? undefined
-                : { depth: args.screenDepth, height: args.screenHeight, width: args.screenWidth },
+            screen,
             section: args.section,
             sectionIndex: args.sectionIndex,
             sessionVersion: args.sessionVersion,
@@ -58,7 +63,9 @@ async function testingLiveExecute(args) {
     };
 }
 async function testingLiveControl(action, args) {
-    return { result: await controlTestingLive(action, { jobId: args.job, projectId: args.project, sessionId: args.sessionId }) };
+    return {
+        result: await controlTestingLive(action, { jobId: args.job, projectId: args.project, sessionId: args.sessionId }),
+    };
 }
 async function testingLiveInspect(args) {
     return {
@@ -100,7 +107,9 @@ async function testingLiveSetOptions(args) {
     };
 }
 async function testingLiveHistory(args) {
-    return { result: getTestingLiveHistory({ jobId: args.job, limit: args.limit, projectId: args.project, status: args.status }) };
+    return {
+        result: getTestingLiveHistory({ jobId: args.job, limit: args.limit, projectId: args.project, status: args.status }),
+    };
 }
 export const testingLiveMcpHandlers = {
     testingLiveContinue: (args) => testingLiveControl('continue', args),
@@ -116,7 +125,7 @@ export const testingLiveMcpHandlers = {
     testingLiveStop,
 };
 const liveSessionSchema = {
-    job: z.number().int().positive().optional().describe('Optional CRT job/test ID guard.'),
+    job: z.number().int().positive().optional().describe('Optional CRT job ID guard.'),
     project: z.number().int().positive().describe('CRT project ID.'),
     sessionId: z.number().int().positive().optional().describe('Optional active live session ID guard.'),
 };
@@ -135,12 +144,13 @@ export function registerTestingLiveMcpTools(server) {
         inputSchema: {
             description: z.string().max(500).optional(),
             japIme: z.boolean().optional(),
-            job: z.number().int().positive().describe('CRT job/test ID.'),
+            job: z.number().int().positive().describe('CRT job ID.'),
             library: z.enum(TESTING_LIVE_LIBRARIES).optional(),
             noBootstrap: z.boolean().optional(),
             noBrowser: z.boolean().optional(),
             pollIntervalSeconds: z.number().int().min(1).max(30).optional(),
             project: z.number().int().positive().describe('CRT project ID.'),
+            screen: z.string().max(100).optional(),
             screenDepth: z.number().int().min(8).max(32).optional(),
             screenHeight: z.number().int().min(240).max(4320).optional(),
             screenWidth: z.number().int().min(320).max(7680).optional(),
@@ -148,6 +158,7 @@ export function registerTestingLiveMcpTools(server) {
             sectionIndex: z.number().int().positive().optional(),
             sessionVersion: z.string().max(100).optional(),
             timeoutSeconds: z.number().int().min(1).max(600).optional(),
+            viewport: z.string().max(100).optional(),
         },
         title: 'Agentia Testing Live Start',
     }, (args) => safelyHandleTool(() => testingLiveStart(args)));
@@ -204,7 +215,7 @@ export function registerTestingLiveMcpTools(server) {
     server.registerTool('agentia_testing_live_history', {
         description: 'Read bounded sanitized local CRT live execution history.',
         inputSchema: {
-            job: z.number().int().positive().describe('CRT job/test ID.'),
+            job: z.number().int().positive().describe('CRT job ID.'),
             limit: z.number().int().min(1).max(500).optional(),
             project: z.number().int().positive().describe('CRT project ID.'),
             status: z.enum(['completed', 'failed', 'recorded']).optional(),