@arii/boomtick-mcp 0.4.1 → 0.5.0

package/CHANGELOG.md

@@ -1,5 +1,21 @@
 # Changelog
 
+## [0.5.0](https://github.com/arii/boomtick/compare/mcp-v0.4.2...mcp-v0.5.0) (2026-10-19)
+
+
+### Features
+
+* **tooling:** add PR reviews, PR commits, workflow runs inspection, and repo overrides ([#834](https://github.com/arii/boomtick/issues/834)) ([42a8638](https://github.com/arii/boomtick/commit/42a8638add4ce8278dedacd6d8d14acc9e0a3bb9))
+
+## [0.4.2](https://github.com/arii/boomtick/compare/mcp-v0.4.1...mcp-v0.4.2) (2026-10-12)
+
+
+### Bug Fixes
+
+* **mcp:** map tool JSON Schema to parameters object in sync-mcp-schemas ([#802](https://github.com/arii/boomtick/issues/802)) ([a57a9ae](https://github.com/arii/boomtick/commit/a57a9aef32109242882d682fcbd1a9ff0a81cffe))
+* **mcp:** resolve identifier regex, commit patch, timeout, and parameter casing bugs ([#815](https://github.com/arii/boomtick/issues/815)) ([1f3b232](https://github.com/arii/boomtick/commit/1f3b232d8550fa87eac2dd974bfd67252a8f4da1))
+* **mcp:** support dynamic repo context resolution and --repo forwarding across workspaces ([#811](https://github.com/arii/boomtick/issues/811)) ([f7e1665](https://github.com/arii/boomtick/commit/f7e166594f170843b7f2e267fab119c631bf3c83))
+
 ## [0.4.1](https://github.com/arii/boomtick/compare/mcp-v0.4.0...mcp-v0.4.1) (2026-09-22)
 
 

package/README.md

@@ -12,7 +12,6 @@ A Model Context Protocol (MCP) server designed to empower AI agents with structu
 
 ### 📄 Resources
 - `repo://package-json`: Access root package manifest.
-- `repo://routes`: Access application route-to-content mapping.
 - `repo://design-tokens`: Access UI design tokens.
 - `repo://diff/{prNumber}`: Access full PR diff text.
 - `repo://ci/{prNumber}`: Access detailed CI check results and logs.

package/actions/impact-analysis/action.yml

@@ -306,22 +306,10 @@ runs:
         PR_NUMBER: ${{ inputs.pr_number }}
         GITHUB_REPOSITORY: ${{ github.repository }}
       run: |
-        TARGET_PATH="${{ github.action_path }}/../../../scripts/send-jules-impact.py"
-        if [ -f "$TARGET_PATH" ]; then
-          RESOLVED_PATH=$(realpath "$TARGET_PATH")
-          echo "Executing send-jules-impact.py at: $RESOLVED_PATH"
-          export PYTHONPATH="$(realpath "${{ github.action_path }}/../../../cli"):$PYTHONPATH"
-          python3 "$RESOLVED_PATH"
-        else
-          # Fallback check for docker run execution or direct python script in checkout workspace
-          # Inside container execution, scripts/send-jules-impact.py should exist in workspace
-          if [ -f "scripts/send-jules-impact.py" ]; then
-            echo "Executing send-jules-impact.py from current workspace."
-            export PYTHONPATH="cli:$PYTHONPATH"
-            python3 scripts/send-jules-impact.py
-          else
-            echo "⚠️ Skipping: send-jules-impact.py script not found."
-          fi
+        if command -v pnpm &> /dev/null; then
+          pnpm exec tsx -e "import { sendJulesImpactReport } from './lib/julesReviewClient'; (async () => { await sendJulesImpactReport(process.env.TASK_ID); })();"
+        elif command -v npx &> /dev/null; then
+          npx tsx -e "import { sendJulesImpactReport } from './lib/julesReviewClient'; (async () => { await sendJulesImpactReport(process.env.TASK_ID); })();"
         fi
       shell: bash
 

package/actions/run-project-gate/action.yml

@@ -17,19 +17,12 @@ runs:
       env:
         TARGET: ${{ inputs.target }}
       run: |
-        # Use centralized script for CLI path resolution
-        if [ -f "scripts/resolve-cli.sh" ]; then
-          CLI_PATH=$(bash scripts/resolve-cli.sh | sed "s|$(pwd)/||")
-        elif [ -f "boomtick-pkg/scripts/resolve-cli.sh" ]; then
-          CLI_PATH=$(bash boomtick-pkg/scripts/resolve-cli.sh | sed "s|$(pwd)/||")
-        elif [ -f "boomtick/scripts/resolve-cli.sh" ]; then
-          CLI_PATH=$(bash boomtick/scripts/resolve-cli.sh | sed "s|$(pwd)/||")
+        if [ -d "cli" ]; then
+          CLI_PATH="cli"
         elif [ -d "boomtick-pkg/cli" ]; then
           CLI_PATH="boomtick-pkg/cli"
         elif [ -d "boomtick/cli" ]; then
           CLI_PATH="boomtick/cli"
-        elif [ -d "cli" ]; then
-          CLI_PATH="cli"
         else
           CLI_PATH="cli"
         fi

package/actions/test-build/action.yml

@@ -1,13 +1,9 @@
 name: BoomTick Build & Test
-description: Encapsulates build, bundle size check, Playwright E2E, and Lighthouse CI
+description: Encapsulates build, bundle size check, and Playwright E2E
 inputs:
   github_token:
     description: 'GitHub token'
     required: true
-  run_lighthouse:
-    description: 'Whether to run Lighthouse CI'
-    required: false
-    default: 'true'
 runs:
   using: composite
   steps:
@@ -36,20 +32,6 @@ runs:
         VITE_BASE_PATH=/ pnpm run test:e2e
       shell: bash
 
-    - name: Run Lighthouse CI
-      if: inputs.run_lighthouse == 'true'
-      run: |
-        set -eu
-        VITE_BASE_PATH="/"
-        CHROME_PATH="$(node -e "console.log(require('playwright').chromium.executablePath())")"
-        export VITE_BASE_PATH
-        export CHROME_PATH
-        echo "CHROME_PATH=$CHROME_PATH"
-        pnpm run lighthouse --collect.url="http://localhost:4173${VITE_BASE_PATH}"
-      env:
-        LHCI_GITHUB_TOKEN: ${{ inputs.github_token }}
-      shell: bash
-
     - name: Check Playwright Report Exists
       id: check_report
       if: always()

package/dist/lib/git.js

@@ -1,6 +1,36 @@
 import { runCommand } from "./shell.js";
+import { config } from "../config.js";
 import path from "path";
 import fs from "fs/promises";
+import { execSync } from "child_process";
+const repoCache = new Map();
+export function resolveRepo(explicitRepo, cwd) {
+    if (explicitRepo && explicitRepo.trim().length > 0) {
+        return explicitRepo.trim();
+    }
+    const targetCwd = cwd || process.cwd();
+    if (repoCache.has(targetCwd)) {
+        return repoCache.get(targetCwd);
+    }
+    try {
+        const url = execSync("git remote get-url origin", {
+            cwd: targetCwd,
+            encoding: "utf-8",
+            stdio: ["ignore", "pipe", "ignore"],
+        }).trim();
+        const match = url.match(/[:/]([^/]+\/[^/.]+)(\.git)?$/);
+        if (match) {
+            repoCache.set(targetCwd, match[1]);
+            return match[1];
+        }
+    }
+    catch (e) {
+        // Ignore error and fall back to static config
+    }
+    const fallback = `${config.githubOwner}/${config.githubRepo}`;
+    repoCache.set(targetCwd, fallback);
+    return fallback;
+}
 function validatePrNumber(value) {
     const prNumber = Number(value);
     if (!Number.isInteger(prNumber) || prNumber <= 0) {
@@ -29,3 +59,47 @@ export async function removeWorktree(worktreePath) {
     await runCommand("git", ["worktree", "remove", "--force", worktreePath]);
     await runCommand("git", ["worktree", "prune"]);
 }
+export async function getChangedFiles(base = "main", head = "HEAD", cwd) {
+    const result = await runCommand("git", ["diff", "--name-status", "-z", `${base}...${head}`], { cwd });
+    if (result.exitCode !== 0) {
+        throw new Error(`Failed to get changed files: ${result.stderr}`);
+    }
+    const tokens = result.stdout.split("\0").filter((t) => t.length > 0);
+    const files = [];
+    let i = 0;
+    while (i < tokens.length) {
+        const status = tokens[i];
+        if (status.startsWith("R") || status.startsWith("C")) {
+            const oldPath = tokens[i + 1];
+            const newPath = tokens[i + 2];
+            if (oldPath && newPath) {
+                files.push({ status, path: newPath, oldPath });
+            }
+            i += 3;
+        }
+        else {
+            const filePath = tokens[i + 1];
+            if (filePath) {
+                files.push({ status, path: filePath });
+            }
+            i += 2;
+        }
+    }
+    return files;
+}
+export async function getMergeConflictFiles(baseBranch = "main", headRef = "HEAD", cwd) {
+    const result = await runCommand("git", ["merge-tree", "--write-tree", "--name-only", baseBranch, headRef], { cwd });
+    // git merge-tree returns exit code 1 if conflicts are present, 0 if clean merge
+    if (result.exitCode !== 0 && result.exitCode !== 1) {
+        throw new Error(`Git merge-tree failed: ${result.stderr}`);
+    }
+    const lines = result.stdout
+        .split("\n")
+        .map((l) => l.trim())
+        .filter((l) => l.length > 0);
+    // First line is tree hash if successful or conflict output
+    // When conflicts exist, git merge-tree --write-tree --name-only outputs conflicted filenames
+    // Skip tree hash line if present (40 hex chars)
+    const files = lines.filter((line) => !/^[0-9a-f]{40}$/i.test(line));
+    return files;
+}

package/dist/lib/git.test.js

@@ -0,0 +1,82 @@
+import { describe, expect, it, vi, beforeEach } from "vitest";
+import { getChangedFiles, getMergeConflictFiles } from "./git.js";
+import * as shell from "./shell.js";
+vi.mock("./shell.js", () => ({
+    runCommand: vi.fn(),
+}));
+describe("Git Helpers", () => {
+    beforeEach(() => {
+        vi.resetAllMocks();
+    });
+    describe("getChangedFiles", () => {
+        it("should parse null-delimited git diff output correctly", async () => {
+            // Mock git diff output with null character delimiters:
+            // Status M -> file1.txt
+            // Status A -> file2.txt
+            // Status R100 -> old.txt -> new.txt
+            const mockStdout = "M\0file1.txt\0A\0file2.txt\0R100\0old.txt\0new.txt\0";
+            vi.mocked(shell.runCommand).mockResolvedValueOnce({
+                exitCode: 0,
+                stdout: mockStdout,
+                stderr: "",
+                durationMs: 10,
+                command: "git diff --name-status -z main...HEAD",
+            });
+            const result = await getChangedFiles("main", "HEAD");
+            expect(shell.runCommand).toHaveBeenCalledWith("git", ["diff", "--name-status", "-z", "main...HEAD"], { cwd: undefined });
+            expect(result).toEqual([
+                { status: "M", path: "file1.txt" },
+                { status: "A", path: "file2.txt" },
+                { status: "R100", path: "new.txt", oldPath: "old.txt" },
+            ]);
+        });
+        it("should throw error if git diff fails", async () => {
+            vi.mocked(shell.runCommand).mockResolvedValueOnce({
+                exitCode: 1,
+                stdout: "",
+                stderr: "fatal: bad revision",
+                durationMs: 10,
+                command: "git diff --name-status -z invalid...HEAD",
+            });
+            await expect(getChangedFiles("invalid", "HEAD")).rejects.toThrow("Failed to get changed files: fatal: bad revision");
+        });
+    });
+    describe("getMergeConflictFiles", () => {
+        it("should parse git merge-tree output when conflicts exist", async () => {
+            // Output with 40-char tree hash followed by conflicting filenames
+            const mockStdout = "5d71843ed2e051b2be91361db7bea0fc4ee1e5a0\nfile1.txt\nsrc/file2.ts\n";
+            vi.mocked(shell.runCommand).mockResolvedValueOnce({
+                exitCode: 1, // merge-tree exits 1 on conflicts
+                stdout: mockStdout,
+                stderr: "",
+                durationMs: 10,
+                command: "git merge-tree --write-tree --name-only main feature",
+            });
+            const conflicts = await getMergeConflictFiles("main", "feature");
+            expect(shell.runCommand).toHaveBeenCalledWith("git", ["merge-tree", "--write-tree", "--name-only", "main", "feature"], { cwd: undefined });
+            expect(conflicts).toEqual(["file1.txt", "src/file2.ts"]);
+        });
+        it("should return empty array when no conflicts exist", async () => {
+            const mockStdout = "5d71843ed2e051b2be91361db7bea0fc4ee1e5a0\n";
+            vi.mocked(shell.runCommand).mockResolvedValueOnce({
+                exitCode: 0,
+                stdout: mockStdout,
+                stderr: "",
+                durationMs: 10,
+                command: "git merge-tree --write-tree --name-only main feature",
+            });
+            const conflicts = await getMergeConflictFiles("main", "feature");
+            expect(conflicts).toEqual([]);
+        });
+        it("should throw error if git merge-tree exits with unexpected exit code", async () => {
+            vi.mocked(shell.runCommand).mockResolvedValueOnce({
+                exitCode: 128,
+                stdout: "",
+                stderr: "fatal: not a valid object name",
+                durationMs: 10,
+                command: "git merge-tree --write-tree --name-only invalid feature",
+            });
+            await expect(getMergeConflictFiles("invalid", "feature")).rejects.toThrow("Git merge-tree failed: fatal: not a valid object name");
+        });
+    });
+});

package/dist/lib/param_utils.js

@@ -0,0 +1,35 @@
+export function normalizeArguments(args) {
+    if (!args || typeof args !== "object") {
+        return {};
+    }
+    const result = {};
+    for (const [key, value] of Object.entries(args)) {
+        // Convert snake_case to camelCase
+        const camelKey = key.replace(/_([a-z])/g, (_, letter) => letter.toUpperCase());
+        // Normalize value recursively if it's an object/array
+        let normalizedValue = value;
+        if (Array.isArray(value)) {
+            normalizedValue = value.map((item) => item && typeof item === "object" ? normalizeArguments(item) : item);
+        }
+        else if (value && typeof value === "object" && !(value instanceof RegExp)) {
+            normalizedValue = normalizeArguments(value);
+        }
+        // Coerce numeric string to integer for prNumber / issueNumber if applicable
+        if ((camelKey === "prNumber" || camelKey === "issueNumber") && typeof normalizedValue === "string") {
+            const parsedNum = parseInt(normalizedValue, 10);
+            if (!isNaN(parsedNum)) {
+                normalizedValue = parsedNum;
+            }
+        }
+        result[camelKey] = normalizedValue;
+    }
+    // Parameter aliases harmonization
+    // branch vs branchName harmonization
+    if (result.branch !== undefined && result.branchName === undefined) {
+        result.branchName = result.branch;
+    }
+    else if (result.branchName !== undefined && result.branch === undefined) {
+        result.branch = result.branchName;
+    }
+    return result;
+}

package/dist/lib/param_utils.test.js

@@ -0,0 +1,44 @@
+import { describe, it, expect } from "vitest";
+import { normalizeArguments } from "./param_utils.js";
+describe("param_utils.normalizeArguments", () => {
+    it("should convert snake_case keys to camelCase", () => {
+        const input = {
+            pr_number: 123,
+            issue_number: 456,
+            worktree_path: "/tmp/worktree",
+            base_branch: "main",
+        };
+        const output = normalizeArguments(input);
+        expect(output).toEqual({
+            prNumber: 123,
+            issueNumber: 456,
+            worktreePath: "/tmp/worktree",
+            baseBranch: "main",
+        });
+    });
+    it("should coerce numeric strings for prNumber and issueNumber", () => {
+        const input = {
+            pr_number: "789",
+            issue_number: "101",
+        };
+        const output = normalizeArguments(input);
+        expect(output).toEqual({
+            prNumber: 789,
+            issueNumber: 101,
+        });
+    });
+    it("should synchronize branch and branchName", () => {
+        expect(normalizeArguments({ branch: "feat-x" })).toEqual({
+            branch: "feat-x",
+            branchName: "feat-x",
+        });
+        expect(normalizeArguments({ branch_name: "feat-y" })).toEqual({
+            branchName: "feat-y",
+            branch: "feat-y",
+        });
+    });
+    it("should handle null or empty arguments safely", () => {
+        expect(normalizeArguments(null)).toEqual({});
+        expect(normalizeArguments(undefined)).toEqual({});
+    });
+});

package/dist/mcp/definitions.js

@@ -1,4 +1,4 @@
-import { HealthCheckInputJsonSchema, SearchPRsInputJsonSchema, GetPrDiffInputJsonSchema, CheckoutBranchInputJsonSchema, GetMergeConflictFilesInputJsonSchema, GetChangedFilesInputJsonSchema, GetPackageScriptsInputJsonSchema, GetRouteMapInputJsonSchema, ReadCiLogsInputJsonSchema, RepoLogsInputJsonSchema, CreateBranchInputJsonSchema, CreateRepairBranchInputJsonSchema, RunTestsInputJsonSchema, RunLighthouseInputJsonSchema, RunPlaywrightInputJsonSchema, CommitPatchInputJsonSchema, OpenReplacementPrInputJsonSchema, CreatePullRequestInputJsonSchema, CommentTriageSummaryInputJsonSchema, GetPrInputJsonSchema, IssueViewInputJsonSchema, IssueUpdateInputJsonSchema, IssueCommentInputJsonSchema, CreateIssueInputJsonSchema, GetCommandSchemaInputJsonSchema, ReadAgentContextInputJsonSchema as ReadAgentContextSchema, CreateJulesSessionInputJsonSchema, JulesSessionIdInputJsonSchema, JulesSendMessageInputJsonSchema, JulesListSessionsInputJsonSchema, SearchDdgsInputJsonSchema, } from "../tools/contract.js";
+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";
 export const MCP_PROMPTS = [
     {
         name: "conflict-scout",
@@ -32,12 +32,6 @@ export const MCP_RESOURCES = [
         mimeType: "application/json",
         description: "The root package.json file of the repository.",
     },
-    {
-        uri: "repo://routes",
-        name: "Route Map",
-        mimeType: "application/json",
-        description: "The mapping of application routes to content files.",
-    },
     {
         uri: "repo://design-tokens",
         name: "Design Tokens",
@@ -50,20 +44,8 @@ export const MCP_RESOURCES = [
         mimeType: "application/json",
         description: "The validation report for a specific repair branch.",
     },
-    {
-        uri: "repo://lighthouse/{branch}",
-        name: "Lighthouse Report",
-        mimeType: "application/json",
-        description: "Lighthouse CI report for a specific branch.",
-    },
-    {
-        uri: "repo://playwright/{branch}",
-        name: "Playwright Report",
-        mimeType: "application/json",
-        description: "Playwright test report for a specific branch.",
-    },
 ];
-export const MCP_TOOLS = [
+const BASE_MCP_TOOLS = [
     {
         name: "boomtick.health",
         description: "Check the health and configuration of the MCP server.",
@@ -109,11 +91,6 @@ export const MCP_TOOLS = [
         description: "Get the scripts defined in package.json.",
         inputSchema: GetPackageScriptsInputJsonSchema,
     },
-    {
-        name: "repo.get_route_map",
-        description: "Get the mapping of routes to content files.",
-        inputSchema: GetRouteMapInputJsonSchema,
-    },
     {
         name: "repo.read_ci_logs",
         description: "Read CI logs for a given pull request.",
@@ -139,16 +116,6 @@ export const MCP_TOOLS = [
         description: "Run repository tests and checks.",
         inputSchema: RunTestsInputJsonSchema,
     },
-    {
-        name: "repo.run_lighthouse",
-        description: "Run Lighthouse CI audits.",
-        inputSchema: RunLighthouseInputJsonSchema,
-    },
-    {
-        name: "repo.run_playwright",
-        description: "Run Playwright E2E tests.",
-        inputSchema: RunPlaywrightInputJsonSchema,
-    },
     {
         name: "repo.commit_patch",
         description: "Commit verified repair changes in a worktree.",
@@ -174,6 +141,21 @@ export const MCP_TOOLS = [
         description: "Get details of a GitHub PR including title, state, and URLs.",
         inputSchema: GetPrInputJsonSchema,
     },
+    {
+        name: "github.get_pr_reviews",
+        description: "Get top-level reviews, verdicts, states, and bodies for a GitHub PR.",
+        inputSchema: GetPrReviewsInputJsonSchema,
+    },
+    {
+        name: "github.get_pr_commits",
+        description: "List commits and commit metadata (SHAs, authors, messages) for a GitHub PR.",
+        inputSchema: GetPrCommitsInputJsonSchema,
+    },
+    {
+        name: "repo.get_workflow_runs",
+        description: "List recent GitHub Actions workflow runs for the repository.",
+        inputSchema: GetWorkflowRunsInputJsonSchema,
+    },
     {
         name: "github.issue_view",
         description: "View details of a GitHub issue including title, body, and state.",
@@ -234,9 +216,14 @@ export const MCP_TOOLS = [
         description: "Automatically collect CI status/logs for the PR associated with a Jules session and send them back as feedback.",
         inputSchema: JulesSessionIdInputJsonSchema,
     },
-    {
-        name: "agent.search_ddgs",
-        description: "Search the web using DuckDuckGo (via ddgs python library).",
-        inputSchema: SearchDdgsInputJsonSchema,
-    },
 ];
+export const MCP_TOOLS = BASE_MCP_TOOLS.flatMap((tool) => {
+    const underscoreName = tool.name.replace(/\./g, "_");
+    if (underscoreName !== tool.name) {
+        return [
+            { ...tool, name: underscoreName },
+            tool,
+        ];
+    }
+    return [tool];
+});

package/dist/mcp/schemas.test.js

@@ -0,0 +1,28 @@
+import { describe, it, expect } from "vitest";
+import * as fs from "fs";
+import * as path from "path";
+import { execSync } from "child_process";
+describe("MCP tool schemas export", () => {
+    it("should generate valid schema files with non-null parameters object", () => {
+        const schemasDir = path.resolve(process.cwd(), ".mcp/schemas");
+        if (!fs.existsSync(schemasDir)) {
+            execSync("npx tsx scripts/sync-mcp-schemas.ts", { cwd: process.cwd() });
+        }
+        expect(fs.existsSync(schemasDir)).toBe(true);
+        const files = fs.readdirSync(schemasDir).filter((f) => f.endsWith(".json"));
+        expect(files.length).toBeGreaterThan(0);
+        for (const file of files) {
+            const filePath = path.join(schemasDir, file);
+            const content = JSON.parse(fs.readFileSync(filePath, "utf-8"));
+            expect(content).toHaveProperty("name");
+            expect(typeof content.name).toBe("string");
+            expect(content).toHaveProperty("description");
+            expect(typeof content.description).toBe("string");
+            expect(content).toHaveProperty("parameters");
+            expect(content.parameters).not.toBeNull();
+            expect(typeof content.parameters).toBe("object");
+            expect(content.parameters.type).toBe("object");
+            expect(content.parameters.properties).toBeDefined();
+        }
+    });
+});