@arii/boomtick-mcp 0.6.0 → 0.6.2

package/dist/mcp/definitions.js

@@ -1,4 +1,4 @@
-import { HealthCheckInputJsonSchema, SearchPRsInputJsonSchema, GetPrDiffInputJsonSchema, CheckoutBranchInputJsonSchema, GetMergeConflictFilesInputJsonSchema, GetChangedFilesInputJsonSchema, GetPackageScriptsInputJsonSchema, ReadCiLogsInputJsonSchema, RepoLogsInputJsonSchema, CreateBranchInputJsonSchema, CreateRepairBranchInputJsonSchema, RunTestsInputJsonSchema, CommitPatchInputJsonSchema, OpenReplacementPrInputJsonSchema, CreatePullRequestInputJsonSchema, CommentTriageSummaryInputJsonSchema, GetPrInputJsonSchema, IssueViewInputJsonSchema, IssueUpdateInputJsonSchema, IssueCommentInputJsonSchema, CreateIssueInputJsonSchema, GetCommandSchemaInputJsonSchema, ReadAgentContextInputJsonSchema as ReadAgentContextSchema, CreateJulesSessionInputJsonSchema, JulesSessionIdInputJsonSchema, JulesSendMessageInputJsonSchema, JulesListSessionsInputJsonSchema, GetPrReviewsInputJsonSchema, GetPrCommitsInputJsonSchema, GetWorkflowRunsInputJsonSchema, } from "../tools/contract.js";
+import { HealthCheckInputJsonSchema, SearchPRsInputJsonSchema, SearchPRsResponseJsonSchema, GetPrDiffInputJsonSchema, PrDiffResponseJsonSchema, CheckoutBranchInputJsonSchema, GetMergeConflictFilesInputJsonSchema, GetMergeConflictFilesResponseJsonSchema, GetChangedFilesInputJsonSchema, GetPackageScriptsInputJsonSchema, ReadCiLogsInputJsonSchema, RepoLogsInputJsonSchema, CreateBranchInputJsonSchema, CreateRepairBranchInputJsonSchema, RunTestsInputJsonSchema, RunTestsResponseJsonSchema, CommitPatchInputJsonSchema, OpenReplacementPrInputJsonSchema, CreatePullRequestInputJsonSchema, CommentTriageSummaryInputJsonSchema, GetPrInputJsonSchema, IssueViewInputJsonSchema, IssueUpdateInputJsonSchema, IssueUpdateResponseJsonSchema, IssueCommentInputJsonSchema, CreateIssueInputJsonSchema, CreateIssueResponseJsonSchema, GetCommandSchemaInputJsonSchema, ReadAgentContextInputJsonSchema as ReadAgentContextSchema, CreateJulesSessionInputJsonSchema, JulesSessionIdInputJsonSchema, JulesSendMessageInputJsonSchema, JulesListSessionsInputJsonSchema, GetPrReviewsInputJsonSchema, GetPrCommitsInputJsonSchema, GetWorkflowRunsInputJsonSchema, } from "../tools/contract.js";
 export const MCP_PROMPTS = [
     {
         name: "conflict-scout",
@@ -44,6 +44,12 @@ export const MCP_RESOURCES = [
         mimeType: "application/json",
         description: "List of changed files in the repository.",
     },
+    {
+        uri: "mcp://ci/latest-run",
+        name: "Latest CI Run",
+        mimeType: "application/json",
+        description: "Recent CI exit statuses, workflow runs, and annotations.",
+    },
     {
         uri: "repo://package-json",
         name: "package.json",
@@ -93,7 +99,7 @@ const BASE_MCP_TOOLS = [
         name: "github.search_open_prs",
         description: "Search for open pull requests in the repository.",
         inputSchema: SearchPRsInputJsonSchema,
-        outputSchema: DEFAULT_OUTPUT_SCHEMA,
+        outputSchema: SearchPRsResponseJsonSchema,
     },
     {
         name: "repo.read_agent_context",
@@ -105,7 +111,7 @@ const BASE_MCP_TOOLS = [
         name: "github.get_pr_diff",
         description: "Get the diff and changed files for a pull request.",
         inputSchema: GetPrDiffInputJsonSchema,
-        outputSchema: DEFAULT_OUTPUT_SCHEMA,
+        outputSchema: PrDiffResponseJsonSchema,
     },
     {
         name: "github.checkout_branch",
@@ -117,7 +123,7 @@ const BASE_MCP_TOOLS = [
         name: "github.get_merge_conflict_files",
         description: "Detect files that conflict when a PR is merged with the base branch.",
         inputSchema: GetMergeConflictFilesInputJsonSchema,
-        outputSchema: DEFAULT_OUTPUT_SCHEMA,
+        outputSchema: GetMergeConflictFilesResponseJsonSchema,
     },
     {
         name: "repo.get_changed_files",
@@ -161,11 +167,27 @@ const BASE_MCP_TOOLS = [
         inputSchema: CreateRepairBranchInputJsonSchema,
         outputSchema: DEFAULT_OUTPUT_SCHEMA,
     },
+    {
+        name: "repo.check_jules_status",
+        description: "Check or poll the status of a Jules remediation session and verify its branch or PR status.",
+        inputSchema: {
+            type: "object",
+            properties: {
+                sessionId: { type: "string", description: "The unique ID or task ID of the Jules session." },
+                commitSha: { type: "string", description: "Commit SHA for idempotency key verification." },
+                failureLog: { type: "string", description: "Failure log content for idempotency key calculation." },
+                poll: { type: "boolean", description: "Whether to poll until session completion or timeout." },
+                timeoutMs: { type: "integer", description: "Timeout in milliseconds when polling." },
+                intervalMs: { type: "integer", description: "Polling interval in milliseconds." },
+            },
+        },
+        outputSchema: DEFAULT_OUTPUT_SCHEMA,
+    },
     {
         name: "repo.run_tests",
         description: "Run repository tests and checks.",
         inputSchema: RunTestsInputJsonSchema,
-        outputSchema: DEFAULT_OUTPUT_SCHEMA,
+        outputSchema: RunTestsResponseJsonSchema,
     },
     {
         name: "repo.commit_patch",
@@ -225,7 +247,7 @@ const BASE_MCP_TOOLS = [
         name: "github.issue_update",
         description: "Update a GitHub issue's body, labels, and/or state.",
         inputSchema: IssueUpdateInputJsonSchema,
-        outputSchema: DEFAULT_OUTPUT_SCHEMA,
+        outputSchema: IssueUpdateResponseJsonSchema,
     },
     {
         name: "github.issue_comment",
@@ -237,7 +259,7 @@ const BASE_MCP_TOOLS = [
         name: "github.create_issue",
         description: "Create a new GitHub issue.",
         inputSchema: CreateIssueInputJsonSchema,
-        outputSchema: DEFAULT_OUTPUT_SCHEMA,
+        outputSchema: CreateIssueResponseJsonSchema,
     },
     {
         name: "jules.create_session",

package/dist/mcp/http.js

@@ -7,7 +7,7 @@ export function createMcpHttpServer(options) {
     const server = http.createServer(async (req, res) => {
         // CORS Headers
         res.setHeader("Access-Control-Allow-Origin", "*");
-        res.setHeader("Access-Control-Allow-Methods", "GET, POST, OPTIONS");
+        res.setHeader("Access-Control-Allow-Methods", "GET, POST, OPTIONS, HEAD");
         res.setHeader("Access-Control-Allow-Headers", "Content-Type, Authorization");
         if (req.method === "OPTIONS") {
             res.writeHead(204);
@@ -15,19 +15,27 @@ export function createMcpHttpServer(options) {
             return;
         }
         const url = new URL(req.url || "/", `http://${req.headers.host || "localhost"}`);
-        // Endpoint 1: OAuth Protected Resource metadata
-        if (url.pathname === "/.well-known/oauth-protected-resource" && req.method === "GET") {
-            res.writeHead(200, { "Content-Type": "application/json" });
-            res.end(JSON.stringify({
-                resource: "mcp://boomtick-mcp",
-                "authorization_servers": [
+        // Endpoint 1: RFC 9728 OAuth Protected Resource Metadata Discovery
+        if (url.pathname === "/.well-known/oauth-protected-resource" && (req.method === "GET" || req.method === "HEAD")) {
+            const metadata = {
+                resource: process.env.MCP_RESOURCE_URI || process.env.BOOMTICK_RESOURCE_URI || "mcp://boomtick-mcp",
+                authorization_servers: [
                     process.env.MCP_OAUTH_AUTH_SERVER || "https://auth.boomtick.dev",
                 ],
                 scopes_supported: ["mcp:read", "mcp:write"],
-            }));
+                bearer_methods_supported: ["header"],
+                resource_documentation: process.env.MCP_RESOURCE_DOCS || "https://github.com/arii/boomtick",
+            };
+            res.writeHead(200, { "Content-Type": "application/json" });
+            if (req.method === "HEAD") {
+                res.end();
+            }
+            else {
+                res.end(JSON.stringify(metadata, null, 2));
+            }
             return;
         }
-        // Bearer Token Validation for SSE & Messages endpoints
+        // Bearer Token / OAuth 2.1 Validation for Protected Routes (/sse & /messages)
         if (url.pathname === "/sse" || url.pathname === "/messages") {
             if (!bearerToken) {
                 res.writeHead(500, { "Content-Type": "application/json" });
@@ -44,7 +52,7 @@ export function createMcpHttpServer(options) {
             if (!isMatch) {
                 res.writeHead(401, {
                     "Content-Type": "application/json",
-                    "WWW-Authenticate": 'Bearer realm="boomtick-mcp", error="invalid_token"',
+                    "WWW-Authenticate": 'Bearer realm="boomtick-mcp", error="invalid_token", error_description="Invalid or missing Bearer token"',
                 });
                 res.end(JSON.stringify({ error: "unauthorized", message: "Invalid or missing Bearer token" }));
                 return;

package/dist/mcp/http.test.js

@@ -1,6 +1,7 @@
 import { describe, it, expect, beforeEach, afterEach } from "vitest";
 import { Server } from "@modelcontextprotocol/sdk/server/index.js";
 import { createMcpHttpServer } from "./http.js";
+import { BoomtickMCPServer } from "./server.js";
 describe("createMcpHttpServer", () => {
     let mcpServer;
     let httpServerObj;
@@ -13,29 +14,54 @@ describe("createMcpHttpServer", () => {
             httpServerObj.server.close();
         }
     });
-    it("should respond with OAuth protected resource metadata on /.well-known/oauth-protected-resource", async () => {
+    it("should respond with RFC 9728 OAuth protected resource metadata on /.well-known/oauth-protected-resource", async () => {
         httpServerObj = createMcpHttpServer({ port: 0, mcpServer });
         port = await httpServerObj.listen(0);
         const res = await fetch(`http://localhost:${port}/.well-known/oauth-protected-resource`);
         expect(res.status).toBe(200);
+        expect(res.headers.get("content-type")).toContain("application/json");
         const body = await res.json();
         expect(body.resource).toBe("mcp://boomtick-mcp");
-        expect(body.authorization_servers).toBeDefined();
+        expect(body.authorization_servers).toEqual(["https://auth.boomtick.dev"]);
         expect(body.scopes_supported).toEqual(["mcp:read", "mcp:write"]);
+        expect(body.bearer_methods_supported).toEqual(["header"]);
+        expect(body.resource_documentation).toBe("https://github.com/arii/boomtick");
     });
-    it("should reject requests when Bearer token is missing or invalid", async () => {
+    it("should handle HEAD requests for /.well-known/oauth-protected-resource", async () => {
+        httpServerObj = createMcpHttpServer({ port: 0, mcpServer });
+        port = await httpServerObj.listen(0);
+        const res = await fetch(`http://localhost:${port}/.well-known/oauth-protected-resource`, {
+            method: "HEAD",
+        });
+        expect(res.status).toBe(200);
+        const text = await res.text();
+        expect(text).toBe("");
+    });
+    it("should handle OPTIONS preflight CORS requests", async () => {
+        httpServerObj = createMcpHttpServer({ port: 0, mcpServer });
+        port = await httpServerObj.listen(0);
+        const res = await fetch(`http://localhost:${port}/sse`, {
+            method: "OPTIONS",
+        });
+        expect(res.status).toBe(204);
+        expect(res.headers.get("access-control-allow-origin")).toBe("*");
+        expect(res.headers.get("access-control-allow-methods")).toContain("GET");
+        expect(res.headers.get("access-control-allow-headers")).toContain("Authorization");
+    });
+    it("should reject protected requests when Bearer token is missing or invalid", async () => {
         httpServerObj = createMcpHttpServer({ port: 0, bearerToken: "secret123", mcpServer });
         port = await httpServerObj.listen(0);
         // Request without token
         const res1 = await fetch(`http://localhost:${port}/sse`);
         expect(res1.status).toBe(401);
         expect(res1.headers.get("www-authenticate")).toContain('Bearer realm="boomtick-mcp"');
+        expect(res1.headers.get("www-authenticate")).toContain('error="invalid_token"');
         // Request with wrong token
         const res2 = await fetch(`http://localhost:${port}/sse`, {
             headers: { Authorization: "Bearer wrong" },
         });
         expect(res2.status).toBe(401);
-        // Request with valid token
+        // Public resource metadata accessible with or without token
         const res3 = await fetch(`http://localhost:${port}/.well-known/oauth-protected-resource`, {
             headers: { Authorization: "Bearer secret123" },
         });
@@ -55,4 +81,28 @@ describe("createMcpHttpServer", () => {
         const body = await res.json();
         expect(body.error).toContain("Server misconfiguration");
     });
+    it("should return 400 for message requests without session or invalid session", async () => {
+        httpServerObj = createMcpHttpServer({ port: 0, bearerToken: "secret123", mcpServer });
+        port = await httpServerObj.listen(0);
+        const res = await fetch(`http://localhost:${port}/messages?sessionId=invalid_session`, {
+            method: "POST",
+            headers: {
+                Authorization: "Bearer secret123",
+                "Content-Type": "application/json",
+            },
+            body: JSON.stringify({ jsonrpc: "2.0", method: "ping", id: 1 }),
+        });
+        expect(res.status).toBe(400);
+        const body = await res.json();
+        expect(body.error).toContain("Session not found or expired");
+    });
+    it("should allow BoomtickMCPServer to start HTTP server via run() when transport is http", async () => {
+        process.env.MCP_TRANSPORT = "http";
+        process.env.PORT = "0";
+        process.env.BOOMTICK_MCP_BEARER_TOKEN = "test-token";
+        const boomtickServer = new BoomtickMCPServer();
+        await boomtickServer.run();
+        // Verify instance initialized properly
+        expect(boomtickServer).toBeInstanceOf(BoomtickMCPServer);
+    });
 });

package/dist/mcp/server.js

@@ -16,14 +16,18 @@ import { getCommandSchemaHandler, GetCommandSchemaInputSchema } from "../tools/r
 import { getPackageScriptsHandler, GetPackageScriptsInputSchema } from "../tools/repo.get_package_scripts.js";
 import { readCiLogsHandler, ReadCiLogsInputSchema } from "../tools/repo.read_ci_logs.js";
 import { repoLogsHandler, RepoLogsInputSchema } from "../tools/repo.logs.js";
+import { getWorkflowRunsHandler, GetWorkflowRunsInputSchema } from "../tools/repo.get_workflow_runs.js";
 import { createRepairBranchHandler, CreateRepairBranchInputSchema } from "../tools/repo.create_repair_branch.js";
 import { createBranchHandler, CreateBranchInputSchema } from "../tools/repo.create_branch.js";
 import { runTestsHandler, RunTestsInputSchema } from "../tools/repo.run_tests.js";
+import { checkJulesStatusHandler, CheckJulesStatusInputSchema } from "../tools/repo.check_jules_status.js";
 import { commitPatchHandler, CommitPatchInputSchema } from "../tools/repo.commit_patch.js";
 import { openReplacementPrHandler, OpenReplacementPrInputSchema } from "../tools/github.open_replacement_pr.js";
 import { commentTriageSummaryHandler, CommentTriageSummaryInputSchema } from "../tools/github.comment_triage_summary.js";
 import { createPullRequestHandler, CreatePullRequestInputSchema } from "../tools/github.create_pull_request.js";
 import { getPrHandler, GetPrInputSchema } from "../tools/github.get_pr.js";
+import { getPrReviewsHandler, GetPrReviewsInputSchema } from "../tools/github.get_pr_reviews.js";
+import { getPrCommitsHandler, GetPrCommitsInputSchema } from "../tools/github.get_pr_commits.js";
 import { issueViewHandler, IssueViewInputSchema } from "../tools/github.issue_view.js";
 import { issueUpdateHandler, IssueUpdateInputSchema } from "../tools/github.issue_update.js";
 import { issueCommentHandler, IssueCommentInputSchema } from "../tools/github.issue_comment.js";
@@ -117,6 +121,14 @@ export class BoomtickMCPServer {
                     contents: [{ uri, mimeType: "application/json", text: JSON.stringify(changedFiles, null, 2) }],
                 };
             }
+            if (uri === "mcp://ci/latest-run" ||
+                uri === "repo://ci/latest-run" ||
+                uri === "repo://ci-latest-run") {
+                const workflowRuns = await getWorkflowRunsHandler(GetWorkflowRunsInputSchema.parse({}));
+                return {
+                    contents: [{ uri, mimeType: "application/json", text: JSON.stringify(workflowRuns, null, 2) }],
+                };
+            }
             if (uri === "repo://package-json" || uri === "mcp://workspace/package-json") {
                 const content = await fs.readFile(path.join(config.repoPath, "package.json"), "utf-8");
                 return {
@@ -189,12 +201,18 @@ export class BoomtickMCPServer {
                         return createSuccessResult(await readCiLogsHandler(ReadCiLogsInputSchema.parse(normArgs)));
                     case "repo.logs":
                         return createSuccessResult(await repoLogsHandler(RepoLogsInputSchema.parse(normArgs)));
+                    case "repo.get.workflow.runs":
+                    case "repo.get_workflow_runs":
+                        return createSuccessResult(await getWorkflowRunsHandler(GetWorkflowRunsInputSchema.parse(normArgs)));
                     case "repo.create.branch":
                     case "repo.create_branch":
                         return createSuccessResult(await createBranchHandler(CreateBranchInputSchema.parse(normArgs)));
                     case "repo.create.repair.branch":
                     case "repo.create_repair_branch":
                         return createSuccessResult(await createRepairBranchHandler(CreateRepairBranchInputSchema.parse(normArgs)));
+                    case "repo.check.jules.status":
+                    case "repo.check_jules_status":
+                        return createSuccessResult(await checkJulesStatusHandler(CheckJulesStatusInputSchema.parse(normArgs)));
                     case "repo.run.tests":
                     case "repo.run_tests":
                         return createSuccessResult(await runTestsHandler(RunTestsInputSchema.parse(normArgs)));
@@ -213,6 +231,12 @@ export class BoomtickMCPServer {
                     case "github.get.pr":
                     case "github.get_pr":
                         return createSuccessResult(await getPrHandler(GetPrInputSchema.parse(normArgs)));
+                    case "github.get.pr.reviews":
+                    case "github.get_pr_reviews":
+                        return createSuccessResult(await getPrReviewsHandler(GetPrReviewsInputSchema.parse(normArgs)));
+                    case "github.get.pr.commits":
+                    case "github.get_pr_commits":
+                        return createSuccessResult(await getPrCommitsHandler(GetPrCommitsInputSchema.parse(normArgs)));
                     case "github.issue.view":
                     case "github.issue_view":
                         return createSuccessResult(await issueViewHandler(IssueViewInputSchema.parse(normArgs)));
@@ -323,20 +347,33 @@ export class BoomtickMCPServer {
             process.exit(1);
         }
     }
-    async startHttp(port = parseInt(process.env.PORT || process.env.HTTP_PORT || "3000", 10), bearerToken) {
-        this.prepareEnvironmentAndPreflight();
-        const mcpHttp = createMcpHttpServer({ port, bearerToken, mcpServer: this.server });
-        return await mcpHttp.listen(port);
-    }
     async run() {
         this.prepareEnvironmentAndPreflight();
+        const args = process.argv.slice(2);
+        let port = parseInt(process.env.PORT || process.env.HTTP_PORT || "3000", 10);
+        let bearerToken = process.env.BOOMTICK_MCP_BEARER_TOKEN || process.env.MCP_BEARER_TOKEN || process.env.AUTH_TOKEN;
+        for (let i = 0; i < args.length; i++) {
+            if (args[i] === "--port" && i + 1 < args.length) {
+                port = parseInt(args[i + 1], 10);
+                i++;
+            }
+            else if (args[i].startsWith("--port=")) {
+                port = parseInt(args[i].split("=")[1], 10);
+            }
+            else if (args[i] === "--bearer-token" && i + 1 < args.length) {
+                bearerToken = args[i + 1];
+                i++;
+            }
+            else if (args[i].startsWith("--bearer-token=")) {
+                bearerToken = args[i].split("=")[1];
+            }
+        }
         const isHttp = process.env.MCP_TRANSPORT === "http" ||
-            process.argv.includes("--http") ||
-            (process.env.PORT !== undefined && !process.argv.includes("--stdio")) ||
-            (process.env.HTTP_PORT !== undefined && !process.argv.includes("--stdio"));
+            args.includes("--http") ||
+            (process.env.PORT !== undefined && !args.includes("--stdio")) ||
+            (process.env.HTTP_PORT !== undefined && !args.includes("--stdio"));
         if (isHttp) {
-            const port = parseInt(process.env.PORT || process.env.HTTP_PORT || "3000", 10);
-            const mcpHttp = createMcpHttpServer({ port, mcpServer: this.server });
+            const mcpHttp = createMcpHttpServer({ port, bearerToken, mcpServer: this.server });
             await mcpHttp.listen(port);
             return;
         }

package/dist/mcp/server.test.js

@@ -4,6 +4,9 @@ import fs from "fs/promises";
 import * as healthModule from "./tools.js";
 import * as prDiffModule from "../tools/github.get_pr_diff.js";
 import * as ciLogsModule from "../tools/repo.read_ci_logs.js";
+import * as prReviewsModule from "../tools/github.get_pr_reviews.js";
+import * as prCommitsModule from "../tools/github.get_pr_commits.js";
+import * as workflowRunsModule from "../tools/repo.get_workflow_runs.js";
 vi.mock("fs/promises");
 vi.mock("../config.js", () => ({
     config: {
@@ -76,12 +79,16 @@ describe("BoomtickMCPServer", () => {
         });
     });
     describe("ListResources", () => {
-        it("should return resources list", async () => {
+        it("should return resources list containing workspace resources", async () => {
             const handler = requestHandlers.get("resources/list");
             expect(handler).toBeDefined();
             const result = await handler({ method: "resources/list" });
             expect(result.resources).toBeDefined();
             expect(Array.isArray(result.resources)).toBe(true);
+            const uris = result.resources.map((r) => r.uri);
+            expect(uris).toContain("mcp://workspace/agent-contract");
+            expect(uris).toContain("mcp://git/changed-files");
+            expect(uris).toContain("mcp://ci/latest-run");
         });
         it("should return resource templates list", async () => {
             const handler = requestHandlers.get("resources/templates/list");
@@ -147,6 +154,29 @@ describe("BoomtickMCPServer", () => {
             expect(result.contents[0].uri).toBe("repo://ci/42");
             expect(JSON.parse(result.contents[0].text)).toEqual({ status: "success" });
         });
+        it("should handle mcp://git/changed-files", async () => {
+            const handler = requestHandlers.get("resources/read");
+            const changedFilesModule = await import("../tools/repo.get_changed_files.js");
+            vi.spyOn(changedFilesModule, "getChangedFilesHandler").mockResolvedValue({ files: ["src/index.ts"] });
+            const result = await handler({
+                method: "resources/read",
+                params: { uri: "mcp://git/changed-files" },
+            });
+            expect(result.contents[0].uri).toBe("mcp://git/changed-files");
+            expect(result.contents[0].mimeType).toBe("application/json");
+            expect(JSON.parse(result.contents[0].text)).toEqual({ files: ["src/index.ts"] });
+        });
+        it("should handle mcp://ci/latest-run", async () => {
+            const handler = requestHandlers.get("resources/read");
+            vi.spyOn(workflowRunsModule, "getWorkflowRunsHandler").mockResolvedValue({ workflowRuns: [{ id: 1, status: "completed", conclusion: "success" }] });
+            const result = await handler({
+                method: "resources/read",
+                params: { uri: "mcp://ci/latest-run" },
+            });
+            expect(result.contents[0].uri).toBe("mcp://ci/latest-run");
+            expect(result.contents[0].mimeType).toBe("application/json");
+            expect(JSON.parse(result.contents[0].text)).toEqual({ workflowRuns: [{ id: 1, status: "completed", conclusion: "success" }] });
+        });
         it("should throw when resource is unknown", async () => {
             const handler = requestHandlers.get("resources/read");
             await expect(handler({
@@ -177,6 +207,27 @@ describe("BoomtickMCPServer", () => {
                 structuredContent: { status: "ok" },
             });
         });
+        it("should execute github_get_pr_reviews and github_get_pr_commits via underscore or dot name", async () => {
+            const handler = requestHandlers.get("tools/call");
+            vi.spyOn(prReviewsModule, "getPrReviewsHandler").mockResolvedValue({ reviews: [] });
+            vi.spyOn(prCommitsModule, "getPrCommitsHandler").mockResolvedValue({ commits: [] });
+            vi.spyOn(workflowRunsModule, "getWorkflowRunsHandler").mockResolvedValue({ runs: [] });
+            const resReviews = await handler({
+                method: "tools/call",
+                params: { name: "github_get_pr_reviews", arguments: { prNumber: 123 } },
+            });
+            expect(resReviews.structuredContent).toEqual({ reviews: [] });
+            const resCommits = await handler({
+                method: "tools/call",
+                params: { name: "github_get_pr_commits", arguments: { prNumber: 123 } },
+            });
+            expect(resCommits.structuredContent).toEqual({ commits: [] });
+            const resWorkflow = await handler({
+                method: "tools/call",
+                params: { name: "repo_get_workflow_runs", arguments: {} },
+            });
+            expect(resWorkflow.structuredContent).toEqual({ runs: [] });
+        });
         it("should return error result for unknown tool", async () => {
             const handler = requestHandlers.get("tools/call");
             const result = await handler({

package/dist/tools/contract.js

@@ -1564,6 +1564,49 @@ export const GetMergeConflictFilesInputJsonSchema = {
     "title": "GetMergeConflictFilesInput",
     "type": "object"
 };
+export const GetMergeConflictFilesResponseSchema = z.object({ "hasConflicts": z.boolean(), "conflictedFiles": z.array(z.string()).optional(), "prNumber": z.union([z.number().int(), z.null()]).default(null), "baseBranch": z.string(), "headRef": z.string() });
+export const GetMergeConflictFilesResponseJsonSchema = {
+    "properties": {
+        "hasConflicts": {
+            "title": "Hasconflicts",
+            "type": "boolean"
+        },
+        "conflictedFiles": {
+            "items": {
+                "type": "string"
+            },
+            "title": "Conflictedfiles",
+            "type": "array"
+        },
+        "prNumber": {
+            "anyOf": [
+                {
+                    "type": "integer"
+                },
+                {
+                    "type": "null"
+                }
+            ],
+            "default": null,
+            "title": "Prnumber"
+        },
+        "baseBranch": {
+            "title": "Basebranch",
+            "type": "string"
+        },
+        "headRef": {
+            "title": "Headref",
+            "type": "string"
+        }
+    },
+    "required": [
+        "hasConflicts",
+        "baseBranch",
+        "headRef"
+    ],
+    "title": "GetMergeConflictFilesResponse",
+    "type": "object"
+};
 export const GetPackageScriptsInputSchema = z.object({ "filter": z.union([z.string(), z.null()]).describe("Optional glob pattern to filter script names.").default(null) });
 export const GetPackageScriptsInputJsonSchema = {
     "properties": {
@@ -3086,6 +3129,40 @@ export const RunTestsInputJsonSchema = {
     "title": "RunTestsInput",
     "type": "object"
 };
+export const RunTestsResponseSchema = z.object({ "passed": z.boolean(), "exitCode": z.number().int(), "failedTests": z.array(z.string()).optional(), "results": z.array(z.record(z.string(), z.any())).optional() });
+export const RunTestsResponseJsonSchema = {
+    "properties": {
+        "passed": {
+            "title": "Passed",
+            "type": "boolean"
+        },
+        "exitCode": {
+            "title": "Exitcode",
+            "type": "integer"
+        },
+        "failedTests": {
+            "items": {
+                "type": "string"
+            },
+            "title": "Failedtests",
+            "type": "array"
+        },
+        "results": {
+            "items": {
+                "additionalProperties": true,
+                "type": "object"
+            },
+            "title": "Results",
+            "type": "array"
+        }
+    },
+    "required": [
+        "passed",
+        "exitCode"
+    ],
+    "title": "RunTestsResponse",
+    "type": "object"
+};
 export const SearchDdgsInputSchema = z.object({ "query": z.string().describe("The search query."), "maxResults": z.union([z.number().int(), z.null()]).describe("Maximum number of results to return.").default(null) });
 export const SearchDdgsInputJsonSchema = {
     "properties": {

package/dist/tools/github.comment_triage_summary.js

@@ -1,23 +1,16 @@
 import { z } from "zod";
-import { runCommand } from "../lib/shell.js";
+import { runTDCli } from "./jules/cli.js";
 export const CommentTriageSummaryInputSchema = z.object({
     prNumber: z.number(),
     body: z.string(),
 });
 export async function commentTriageSummaryHandler(args) {
-    const result = await runCommand("td", [
+    const output = await runTDCli([
         "gh",
         "post-comment",
         "--pr", args.prNumber.toString(),
         "--body", args.body
-    ]);
-    if (result.exitCode !== 0) {
-        throw new Error(`Failed to comment on PR: ${result.stderr}`);
-    }
-    const output = JSON.parse(result.stdout);
-    if (output.status === "error") {
-        throw new Error(`Failed to comment on PR: ${output.message}`);
-    }
+    ], "comment on PR");
     return {
         success: true,
         commentUrl: output.html_url || output.url

package/dist/tools/github.get_merge_conflict_files.js

@@ -9,10 +9,12 @@ export async function getMergeConflictFilesHandler(args) {
     const baseBranch = args.baseBranch || "main";
     const headRef = args.headRef || "HEAD";
     const conflictFiles = await getMergeConflictFiles(baseBranch, headRef);
+    const hasConflicts = conflictFiles.length > 0;
     return {
-        prNumber: args.prNumber,
+        hasConflicts,
+        conflictedFiles: conflictFiles,
+        prNumber: args.prNumber ?? null,
         baseBranch,
         headRef,
-        conflictFiles,
     };
 }

package/dist/tools/github.get_pr_commits.js

@@ -2,7 +2,7 @@ import { z } from "zod";
 import { runCommand } from "../lib/shell.js";
 import { resolveRepo } from "../lib/git.js";
 import { sanitizeError } from "../lib/error_utils.js";
-const GetPrCommitsInputSchema = z.object({
+export const GetPrCommitsInputSchema = z.object({
     prNumber: z.number().describe("The number of the PR to list commits for."),
     repo: z.string().optional().describe("The target repository override (e.g. org/repo)."),
 });

package/dist/tools/github.get_pr_diff.js

@@ -1,6 +1,6 @@
 import { z } from "zod";
-import { runCommand } from "../lib/shell.js";
 import { resolveRepo } from "../lib/git.js";
+import { runTDCli } from "./jules/cli.js";
 export const GetPrDiffInputSchema = z.object({
     prNumber: z.number().describe("The number of the pull request to get the diff for."),
     repo: z.string().optional().describe("The target repository override (e.g. org/repo)."),
@@ -8,17 +8,9 @@ export const GetPrDiffInputSchema = z.object({
 export async function getPrDiffHandler(args) {
     const params = GetPrDiffInputSchema.parse(args);
     const targetRepo = resolveRepo(params.repo);
-    const result = await runCommand("td", [
+    const output = await runTDCli([
         "gh", "pr-diff", params.prNumber.toString(), "--repo", targetRepo
-    ]);
-    if (result.exitCode !== 0) {
-        throw new Error(`Failed to get PR diff: ${result.stderr}`);
-    }
-    const output = JSON.parse(result.stdout);
-    if (output.status === "error") {
-        throw new Error(`Failed to get PR diff: ${output.message}`);
-    }
-    // Every TypeScript tool file must be a pure routing shim
+    ], "get PR diff");
     return {
         prNumber: output.prNumber,
         files: output.files,

package/dist/tools/github.get_pr_reviews.js

@@ -2,7 +2,7 @@ import { z } from "zod";
 import { runCommand } from "../lib/shell.js";
 import { resolveRepo } from "../lib/git.js";
 import { sanitizeError } from "../lib/error_utils.js";
-const GetPrReviewsInputSchema = z.object({
+export const GetPrReviewsInputSchema = z.object({
     prNumber: z.number().describe("The number of the PR to view reviews for."),
     repo: z.string().optional().describe("The target repository override (e.g. org/repo)."),
 });

package/dist/tools/jules/cancel-session.js

@@ -1,17 +1,10 @@
 import { z } from "zod";
-import { runCommand } from "../../lib/shell.js";
+import { runTDCli } from "./cli.js";
 export const CancelJulesSessionInputSchema = z.object({
     sessionId: z.string(),
 });
 export async function cancelJulesSessionHandler(input) {
-    const result = await runCommand("td", ["agent", "cancel", input.sessionId]);
-    if (result.exitCode !== 0) {
-        throw new Error(`Failed to cancel session: ${result.stderr}`);
-    }
-    const output = JSON.parse(result.stdout);
-    if (output.status === "error") {
-        throw new Error(`Failed to cancel session: ${output.message}`);
-    }
+    await runTDCli(["agent", "cancel", input.sessionId], "cancel session");
     return {
         id: input.sessionId.replace("sessions/", ""),
         status: "FAILED",