@arii/boomtick-mcp 0.3.6 → 0.4.0

package/dist/tools/contract.js

@@ -1179,7 +1179,7 @@ export const CreateBranchInputJsonSchema = {
     "title": "CreateBranchInput",
     "type": "object"
 };
-export const CreateIssueInputSchema = z.object({ "title": z.string().min(1), "body": z.union([z.string().min(1), z.null()]).default(null), "file": z.union([z.string().min(1), z.null()]).default(null) });
+export const CreateIssueInputSchema = z.object({ "title": z.string().min(1), "body": z.union([z.string().min(1), z.null()]).default(null), "file": z.union([z.string().min(1), z.null()]).default(null), "repo": z.union([z.string(), z.null()]).describe("The target repository override (e.g. org/repo).").default(null) });
 export const CreateIssueInputJsonSchema = {
     "properties": {
         "title": {
@@ -1212,6 +1212,19 @@ export const CreateIssueInputJsonSchema = {
             ],
             "default": null,
             "title": "File"
+        },
+        "repo": {
+            "anyOf": [
+                {
+                    "type": "string"
+                },
+                {
+                    "type": "null"
+                }
+            ],
+            "default": null,
+            "description": "The target repository override (e.g. org/repo).",
+            "title": "Repo"
         }
     },
     "required": [
@@ -1336,7 +1349,7 @@ export const CreateJulesSessionInputJsonSchema = {
     "title": "CreateJulesSessionInput",
     "type": "object"
 };
-export const CreatePullRequestInputSchema = z.object({ "title": z.string().describe("PR Title."), "body": z.string().describe("Description of changes."), "head": z.string().describe("The branch containing changes to merge."), "base": z.string().describe("The target branch to merge into.").default("main"), "draft": z.boolean().describe("Whether to create the PR as a draft.").default(false) });
+export const CreatePullRequestInputSchema = z.object({ "title": z.string().describe("PR Title."), "body": z.string().describe("Description of changes."), "head": z.string().describe("The branch containing changes to merge."), "base": z.string().describe("The target branch to merge into.").default("main"), "draft": z.boolean().describe("Whether to create the PR as a draft.").default(false), "repo": z.union([z.string(), z.null()]).describe("The target repository override (e.g. org/repo).").default(null) });
 export const CreatePullRequestInputJsonSchema = {
     "properties": {
         "title": {
@@ -1365,6 +1378,19 @@ export const CreatePullRequestInputJsonSchema = {
             "description": "Whether to create the PR as a draft.",
             "title": "Draft",
             "type": "boolean"
+        },
+        "repo": {
+            "anyOf": [
+                {
+                    "type": "string"
+                },
+                {
+                    "type": "null"
+                }
+            ],
+            "default": null,
+            "description": "The target repository override (e.g. org/repo).",
+            "title": "Repo"
         }
     },
     "required": [

package/dist/tools/github.create_issue.js

@@ -9,6 +9,8 @@ export async function createIssueHandler(args) {
         cmdArgs.push("--body", params.body);
     if (params.file)
         cmdArgs.push("--file", params.file);
+    if (params.repo)
+        cmdArgs.push("--repo", params.repo);
     const result = await runCommand("td-cli", cmdArgs);
     if (result.exitCode !== 0) {
         throw new Error(`Failed to create issue: ${sanitizeError(result.stderr)}`);

package/dist/tools/github.create_issue.test.js

@@ -22,11 +22,47 @@ describe("github.create_issue", () => {
             durationMs: 10,
             command: "td-cli gh create-issue --title 'Test Issue' --body 'Test Body'"
         });
-        const result = await createIssueHandler({ title: "Test Issue", body: "Test Body", file: null });
+        const result = await createIssueHandler({ title: "Test Issue", body: "Test Body", file: null, repo: null });
         expect(result.status).toBe("success");
         expect(result.issue?.number).toBe(123);
         expect(result.issue?.title).toBe("Test Issue");
     });
+    it("should append --repo flag when repo parameter is provided", async () => {
+        const mockResponse = {
+            status: "success",
+            issue: {
+                number: 456,
+                title: "Cross Repo Issue",
+                html_url: "https://github.com/org/other-repo/issues/456",
+                state: "open"
+            }
+        };
+        vi.mocked(shell.runCommand).mockResolvedValue({
+            stdout: JSON.stringify(mockResponse),
+            stderr: "",
+            exitCode: 0,
+            durationMs: 10,
+            command: "td-cli gh create-issue --title 'Cross Repo Issue' --body 'Body' --repo 'org/other-repo'"
+        });
+        const result = await createIssueHandler({
+            title: "Cross Repo Issue",
+            body: "Body",
+            file: null,
+            repo: "org/other-repo"
+        });
+        expect(shell.runCommand).toHaveBeenCalledWith("td-cli", [
+            "gh",
+            "create-issue",
+            "--title",
+            "Cross Repo Issue",
+            "--body",
+            "Body",
+            "--repo",
+            "org/other-repo"
+        ]);
+        expect(result.status).toBe("success");
+        expect(result.issue?.number).toBe(456);
+    });
     it("should throw error on command failure", async () => {
         vi.mocked(shell.runCommand).mockResolvedValue({
             stdout: "",
@@ -35,7 +71,7 @@ describe("github.create_issue", () => {
             durationMs: 10,
             command: "td-cli gh create-issue"
         });
-        await expect(createIssueHandler({ title: "Test Issue", body: "Test Body", file: null })).rejects.toThrow("Failed to create issue: Auth failed");
+        await expect(createIssueHandler({ title: "Test Issue", body: "Test Body", file: null, repo: null })).rejects.toThrow("Failed to create issue: Auth failed");
     });
     it("should handle error status from CLI output", async () => {
         const mockResponse = {
@@ -49,6 +85,6 @@ describe("github.create_issue", () => {
             durationMs: 10,
             command: "td-cli gh create-issue"
         });
-        await expect(createIssueHandler({ title: "Test Issue", body: "Test Body", file: null })).rejects.toThrow("Failed to create issue: Repo not found");
+        await expect(createIssueHandler({ title: "Test Issue", body: "Test Body", file: null, repo: null })).rejects.toThrow("Failed to create issue: Repo not found");
     });
 });

package/dist/tools/github.create_pull_request.js

@@ -1,24 +1,22 @@
-import { z } from "zod";
 import { runCommand } from "../lib/shell.js";
-export const CreatePullRequestInputSchema = z.object({
-    title: z.string(),
-    body: z.string(),
-    head: z.string(),
-    base: z.string().optional().default("main"),
-    draft: z.boolean().optional().default(false),
-});
+import { CreatePullRequestInputSchema } from "./contract.js";
+export { CreatePullRequestInputSchema };
 export async function createPullRequestHandler(args) {
+    const params = CreatePullRequestInputSchema.parse(args);
     const tdArgs = [
         "gh",
         "create-pr",
-        "--title", args.title,
-        "--body", args.body,
-        "--head", args.head,
-        "--base", args.base
+        "--title", params.title,
+        "--body", params.body,
+        "--head", params.head,
+        "--base", params.base
     ];
-    if (args.draft) {
+    if (params.draft) {
         tdArgs.push("--draft");
     }
+    if (params.repo) {
+        tdArgs.push("--repo", params.repo);
+    }
     const result = await runCommand("td-cli", tdArgs);
     if (result.exitCode !== 0) {
         // Attempt to parse stdout as JSON to see if there's a structured error message

package/dist/tools/github.create_pull_request.test.js

@@ -0,0 +1,93 @@
+import { describe, it, expect, vi } from "vitest";
+import { createPullRequestHandler } from "./github.create_pull_request.js";
+import * as shell from "../lib/shell.js";
+vi.mock("../lib/shell.js", () => ({
+    runCommand: vi.fn()
+}));
+describe("github.create_pull_request", () => {
+    it("should create a pull request successfully", async () => {
+        const mockResponse = {
+            status: "success",
+            pr: {
+                number: 789,
+                html_url: "https://github.com/owner/repo/pull/789"
+            }
+        };
+        vi.mocked(shell.runCommand).mockResolvedValue({
+            stdout: JSON.stringify(mockResponse),
+            stderr: "",
+            exitCode: 0,
+            durationMs: 10,
+            command: "td-cli gh create-pr --title 'PR' --body 'Body' --head 'feature' --base 'main'"
+        });
+        const result = await createPullRequestHandler({
+            title: "PR",
+            body: "Body",
+            head: "feature",
+            base: "main",
+            draft: false
+        });
+        expect(result.success).toBe(true);
+        expect(result.url).toBe("https://github.com/owner/repo/pull/789");
+    });
+    it("should append --repo flag when repo parameter is provided", async () => {
+        const mockResponse = {
+            status: "success",
+            pr: {
+                number: 999,
+                html_url: "https://github.com/org/other-repo/pull/999"
+            }
+        };
+        vi.mocked(shell.runCommand).mockResolvedValue({
+            stdout: JSON.stringify(mockResponse),
+            stderr: "",
+            exitCode: 0,
+            durationMs: 10,
+            command: "td-cli gh create-pr --title 'PR' --body 'Body' --head 'feature' --base 'main' --repo 'org/other-repo'"
+        });
+        const result = await createPullRequestHandler({
+            title: "PR",
+            body: "Body",
+            head: "feature",
+            base: "main",
+            draft: false,
+            repo: "org/other-repo"
+        });
+        expect(shell.runCommand).toHaveBeenCalledWith("td-cli", [
+            "gh",
+            "create-pr",
+            "--title",
+            "PR",
+            "--body",
+            "Body",
+            "--head",
+            "feature",
+            "--base",
+            "main",
+            "--repo",
+            "org/other-repo"
+        ]);
+        expect(result.success).toBe(true);
+        expect(result.url).toBe("https://github.com/org/other-repo/pull/999");
+    });
+    it("should throw error on CLI error response", async () => {
+        const mockResponse = {
+            status: "error",
+            message: "Branch not found"
+        };
+        vi.mocked(shell.runCommand).mockResolvedValue({
+            stdout: JSON.stringify(mockResponse),
+            stderr: "",
+            exitCode: 0,
+            durationMs: 10,
+            command: "td-cli gh create-pr"
+        });
+        await expect(createPullRequestHandler({
+            title: "PR",
+            body: "Body",
+            head: "feature",
+            base: "main",
+            draft: false
+        })).rejects.toThrow("Failed to create pull request: Branch not found");
+    });
+});

package/dist/tools/jules/shared.test.js

@@ -0,0 +1,58 @@
+import { describe, it, expect } from "vitest";
+import { parseJulesSession } from "./shared.js";
+describe("jules/shared", () => {
+    describe("parseJulesSession", () => {
+        it("should strip 'sessions/' prefix from session name to form ID", () => {
+            const input = {
+                name: "sessions/12345",
+                state: "IN_PROGRESS",
+            };
+            const result = parseJulesSession(input);
+            expect(result.id).toBe("12345");
+            expect(result.status).toBe("IN_PROGRESS");
+        });
+        it("should use raw name as ID if 'sessions/' prefix is absent", () => {
+            const input = {
+                name: "custom-session-id",
+                state: "SUCCEEDED",
+            };
+            expect(parseJulesSession(input).id).toBe("custom-session-id");
+        });
+        it("should default name/id to empty string if missing", () => {
+            expect(parseJulesSession({}).id).toBe("");
+        });
+        it("should map states to COMPLETED, FAILED, IN_PROGRESS, or fallback PENDING", () => {
+            expect(parseJulesSession({ state: "SUCCEEDED" }).status).toBe("COMPLETED");
+            expect(parseJulesSession({ state: "COMPLETED" }).status).toBe("COMPLETED");
+            expect(parseJulesSession({ state: "FAILED" }).status).toBe("FAILED");
+            expect(parseJulesSession({ state: "CANCELLED" }).status).toBe("FAILED");
+            expect(parseJulesSession({ state: "TERMINATED" }).status).toBe("FAILED");
+            expect(parseJulesSession({ state: "IN_PROGRESS" }).status).toBe("IN_PROGRESS");
+            expect(parseJulesSession({ state: "UNKNOWN_STATE" }).status).toBe("PENDING");
+            expect(parseJulesSession({}).status).toBe("PENDING");
+        });
+        it("should extract pullRequestUrl from outputs array when present", () => {
+            const input = {
+                name: "sessions/abc",
+                outputs: [
+                    { type: "text", text: "logs" },
+                    { pullRequest: { url: "https://github.com/arii/boomtick/pull/100" } },
+                ],
+            };
+            const session = parseJulesSession(input);
+            expect(session.pullRequestUrl).toBe("https://github.com/arii/boomtick/pull/100");
+        });
+        it("should return undefined pullRequestUrl if missing or invalid outputs", () => {
+            expect(parseJulesSession({ outputs: [] }).pullRequestUrl).toBeUndefined();
+            expect(parseJulesSession({ outputs: [{ other: "data" }] }).pullRequestUrl).toBeUndefined();
+            expect(parseJulesSession({ outputs: "not-an-array" }).pullRequestUrl).toBeUndefined();
+        });
+        it("should parse createTime or fallback to current date", () => {
+            const timeStr = "2025-01-01T12:00:00Z";
+            const sessionWithTime = parseJulesSession({ createTime: timeStr });
+            expect(sessionWithTime.createdAt).toEqual(new Date(timeStr));
+            const sessionWithoutTime = parseJulesSession({});
+            expect(sessionWithoutTime.createdAt).toBeInstanceOf(Date);
+        });
+    });
+});

package/dist/tools/repo.get_package_scripts.test.js

@@ -20,6 +20,10 @@ describe("repo.get_package_scripts", () => {
         vi.mocked(fs.readFile).mockRejectedValue(new Error("File not found"));
         await expect(getPackageScriptsHandler({})).rejects.toThrow("Failed to read package.json: File not found");
     });
+    it("should throw error when package.json is malformed", async () => {
+        vi.mocked(fs.readFile).mockResolvedValue("{ invalid json ");
+        await expect(getPackageScriptsHandler({})).rejects.toThrow(/Failed to read package.json: Malformed package.json:/);
+    });
     it("should filter scripts by pattern", async () => {
         const mockPkg = {
             scripts: {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@arii/boomtick-mcp",
-  "version": "0.3.6",
+  "version": "0.4.0",
   "description": "Boomtick MCP Guide: PR Rescue + Merge Conflict Agent",
   "repository": {
     "type": "git",

package/src/config.test.ts

@@ -0,0 +1,146 @@
+import { describe, it, expect, vi, beforeEach, afterEach } from "vitest";
+import { execSync } from "child_process";
+
+vi.mock("child_process", () => ({
+  execSync: vi.fn(),
+}));
+
+describe("config", () => {
+  const originalEnv = process.env;
+
+  beforeEach(() => {
+    vi.resetModules();
+    process.env = { ...originalEnv };
+    vi.mocked(execSync).mockReset();
+    vi.mocked(execSync).mockReturnValue("{}");
+  });
+
+  afterEach(() => {
+    process.env = originalEnv;
+  });
+
+  describe("initializeConfig", () => {
+    it("should load dynamic config from td-cli config view", async () => {
+      const mockConfig = { github_repo: "owner/repo", base_branch: "origin/main", vite_base_path: "/app/" };
+      vi.mocked(execSync).mockReturnValue(JSON.stringify(mockConfig));
+
+      const { initializeConfig } = await import("./config.js");
+      const result = initializeConfig();
+
+      expect(execSync).toHaveBeenCalledWith("td-cli config view", expect.anything());
+      expect(result).toEqual(mockConfig);
+    });
+
+    it("should return cached dynamic config on subsequent calls", async () => {
+      const mockConfig = { github_repo: "owner/repo" };
+      vi.mocked(execSync).mockReturnValue(JSON.stringify(mockConfig));
+
+      const { initializeConfig } = await import("./config.js");
+      const result1 = initializeConfig();
+      const result2 = initializeConfig();
+
+      expect(result1).toBe(result2);
+      expect(execSync).toHaveBeenCalledTimes(1);
+    });
+
+    it("should throw critical error when execSync fails in development or CI", async () => {
+      process.env.NODE_ENV = "development";
+      vi.mocked(execSync).mockImplementation(() => {
+        throw new Error("Command failed");
+      });
+
+      const { initializeConfig } = await import("./config.js");
+      expect(() => initializeConfig()).toThrow(/CRITICAL: Failed to load dynamic config/);
+    });
+
+    it("should log warning and set empty config when execSync fails in production", async () => {
+      process.env.NODE_ENV = "production";
+      process.env.CI = "false";
+      const consoleWarnSpy = vi.spyOn(console, "warn").mockImplementation(() => {});
+
+      vi.mocked(execSync).mockImplementation(() => {
+        throw new Error("Command failed");
+      });
+
+      const { initializeConfig } = await import("./config.js");
+      const result = initializeConfig();
+
+      expect(result).toEqual({});
+      expect(consoleWarnSpy).toHaveBeenCalledWith(expect.stringContaining("CRITICAL: Failed to load dynamic config"));
+      consoleWarnSpy.mockRestore();
+    });
+  });
+
+  describe("config object getters", () => {
+    it("should resolve githubToken from process.env or gh CLI fallback", async () => {
+      process.env.GITHUB_TOKEN = "env-token-123";
+      const { config } = await import("./config.js");
+      expect(config.githubToken).toBe("env-token-123");
+
+      delete process.env.GITHUB_TOKEN;
+      vi.mocked(execSync).mockReturnValue("gh-cli-token\n");
+      expect(config.githubToken).toBe("gh-cli-token");
+
+      vi.mocked(execSync).mockImplementation(() => {
+        throw new Error("gh not found");
+      });
+      expect(config.githubToken).toBeUndefined();
+    });
+
+    it("should resolve githubOwner and githubRepo from env or cached config", async () => {
+      process.env.GITHUB_OWNER = "custom-owner";
+      process.env.GITHUB_REPO = "custom-repo";
+      const { config } = await import("./config.js");
+
+      expect(config.githubOwner).toBe("custom-owner");
+      expect(config.githubRepo).toBe("custom-repo");
+    });
+
+    it("should throw when githubOwner / githubRepo cannot be determined", async () => {
+      delete process.env.GITHUB_OWNER;
+      delete process.env.GITHUB_REPO;
+      vi.mocked(execSync).mockImplementation(() => {
+        throw new Error("td-cli failed");
+      });
+
+      process.env.NODE_ENV = "production";
+      process.env.CI = "false";
+      vi.spyOn(console, "warn").mockImplementation(() => {});
+
+      const { config, initializeConfig } = await import("./config.js");
+      initializeConfig();
+
+      expect(() => config.githubOwner).toThrow("GITHUB_OWNER must be set via environment variable or project_config.json");
+      expect(() => config.githubRepo).toThrow("GITHUB_REPO must be set via environment variable or project_config.json");
+    });
+
+    it("should resolve repoPath from BOOMTICK_REPO_PATH or findRepoRoot fallback", async () => {
+      process.env.BOOMTICK_REPO_PATH = "/custom/repo/path";
+      const { config } = await import("./config.js");
+      expect(config.repoPath).toBe("/custom/repo/path");
+    });
+
+    it("should resolve defaultBaseBranch, viteBasePath, and ghPath", async () => {
+      process.env.DEFAULT_BASE_BRANCH = "develop";
+      process.env.VITE_BASE_PATH = "/base/";
+      process.env.GH_PATH = "/usr/bin/gh";
+
+      const { config } = await import("./config.js");
+      expect(config.defaultBaseBranch).toBe("develop");
+      expect(config.viteBasePath).toBe("/base/");
+      expect(config.ghPath).toBe("/usr/bin/gh");
+    });
+
+    it("should throw when viteBasePath is missing", async () => {
+      delete process.env.VITE_BASE_PATH;
+      process.env.NODE_ENV = "production";
+      process.env.CI = "false";
+      vi.spyOn(console, "warn").mockImplementation(() => {});
+
+      const { config, initializeConfig } = await import("./config.js");
+      initializeConfig();
+
+      expect(() => config.viteBasePath).toThrow("VITE_BASE_PATH must be set via environment variable or project_config.json");
+    });
+  });
+});

package/src/lib/error_utils.test.ts

@@ -0,0 +1,27 @@
+import { describe, it, expect } from "vitest";
+import { sanitizeError } from "./error_utils.js";
+
+describe("error_utils", () => {
+  describe("sanitizeError", () => {
+    it("should return 'Unknown error' when stderr is empty", () => {
+      expect(sanitizeError("")).toBe("Unknown error");
+    });
+
+    it("should return only the first line of stderr", () => {
+      const stderr = "Error: Something failed\n  at StackTrace.line1\n  at StackTrace.line2";
+      expect(sanitizeError(stderr)).toBe("Error: Something failed");
+    });
+
+    it("should truncate lines exceeding 200 characters", () => {
+      const longLine = "A".repeat(250);
+      const result = sanitizeError(longLine);
+      expect(result.length).toBe(200);
+      expect(result).toBe("A".repeat(200));
+    });
+
+    it("should preserve first line if under 200 characters", () => {
+      const line = "Fatal error: unexpected token near line 5";
+      expect(sanitizeError(line)).toBe(line);
+    });
+  });
+});

package/src/lib/result.test.ts

@@ -0,0 +1,48 @@
+import { describe, it, expect } from "vitest";
+import { createSuccessResult, createErrorResult } from "./result.js";
+
+describe("result utils", () => {
+  describe("createSuccessResult", () => {
+    it("should format string data directly without JSON stringifying", () => {
+      const result = createSuccessResult("simple string message");
+      expect(result).toEqual({
+        content: [
+          {
+            type: "text",
+            text: "simple string message",
+          },
+        ],
+      });
+      expect(result.isError).toBeUndefined();
+    });
+
+    it("should format object data as pretty-printed JSON string", () => {
+      const objData = { foo: "bar", count: 42 };
+      const result = createSuccessResult(objData);
+      expect(result).toEqual({
+        content: [
+          {
+            type: "text",
+            text: JSON.stringify(objData, null, 2),
+          },
+        ],
+      });
+      expect(result.isError).toBeUndefined();
+    });
+  });
+
+  describe("createErrorResult", () => {
+    it("should format error result with message and isError set to true", () => {
+      const result = createErrorResult("Something went wrong");
+      expect(result).toEqual({
+        content: [
+          {
+            type: "text",
+            text: "Something went wrong",
+          },
+        ],
+        isError: true,
+      });
+    });
+  });
+});