@arii/boomtick-mcp 0.3.7 → 0.4.1

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.7",
+  "version": "0.4.1",
   "description": "Boomtick MCP Guide: PR Rescue + Merge Conflict Agent",
   "repository": {
     "type": "git",

package/src/config.test.ts

@@ -0,0 +1,165 @@
+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 using active cwd", 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.objectContaining({
+        cwd: process.cwd()
+      }));
+      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 prioritize cachedDynamicConfig over environment variables for githubOwner and githubRepo", async () => {
+      process.env.GITHUB_OWNER = "env-owner";
+      process.env.GITHUB_REPO = "env-repo";
+      process.env.GITHUB_REPOSITORY = "env-owner/env-repo";
+
+      const mockConfig = { github_repo: "workspace-owner/workspace-repo" };
+      vi.mocked(execSync).mockReturnValue(JSON.stringify(mockConfig));
+
+      const { config, initializeConfig } = await import("./config.js");
+      initializeConfig();
+
+      expect(config.githubOwner).toBe("workspace-owner");
+      expect(config.githubRepo).toBe("workspace-repo");
+    });
+
+    it("should fallback to environment variables when cachedDynamicConfig is empty", 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;
+      delete process.env.GITHUB_REPOSITORY;
+      delete process.env.GH_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/config.ts

@@ -1,4 +1,3 @@
-import dotenv from "dotenv";
 import path from "path";
 import fs from "fs";
 import { fileURLToPath } from "url";
@@ -6,10 +5,14 @@ import { execSync } from "child_process";
 
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
 
-dotenv.config({
-  path: path.resolve(__dirname, "../.env"),
-  quiet: true
-});
+const envPath = path.resolve(__dirname, "../.env");
+if (fs.existsSync(envPath) && typeof process.loadEnvFile === "function") {
+  try {
+    process.loadEnvFile(envPath);
+  } catch {
+    // Ignore if already loaded or invalid
+  }
+}
 
 function getGithubToken(): string | undefined {
   if (process.env.GITHUB_TOKEN) {
@@ -36,9 +39,11 @@ export function initializeConfig() {
   }
 
   try {
-    // Attempt to load core properties from the Python CLI to avoid duplication
+    // Attempt to load core properties from the Python CLI in the active workspace
+    const cwd = process.env.BOOMTICK_REPO_PATH || process.cwd();
     const cmd = `td-cli config view`;
     const output = execSync(cmd, {
+      cwd,
       encoding: "utf-8",
       stdio: ["ignore", "pipe", "ignore"]
     });
@@ -64,10 +69,27 @@ function findRepoRoot() {
     return process.env.BOOMTICK_REPO_PATH;
   }
 
-  // Traverse up to find workspace.json or .git
-  let current = __dirname;
+  // Traverse up from process.cwd() to find project_config.json, workspace.json, or .git
+  let current = process.cwd();
   while (current !== path.parse(current).root) {
-    if (fs.existsSync(path.join(current, "workspace.json")) || fs.existsSync(path.join(current, ".git"))) {
+    if (
+      fs.existsSync(path.join(current, "project_config.json")) ||
+      fs.existsSync(path.join(current, "workspace.json")) ||
+      fs.existsSync(path.join(current, ".git"))
+    ) {
+      return current;
+    }
+    current = path.dirname(current);
+  }
+
+  // Fallback to __dirname search if discovery from cwd fails
+  current = __dirname;
+  while (current !== path.parse(current).root) {
+    if (
+      fs.existsSync(path.join(current, "project_config.json")) ||
+      fs.existsSync(path.join(current, "workspace.json")) ||
+      fs.existsSync(path.join(current, ".git"))
+    ) {
       return current;
     }
     current = path.dirname(current);
@@ -79,20 +101,28 @@ function findRepoRoot() {
 export const config = {
   get githubToken() { return getGithubToken(); },
   get githubOwner() {
-    if (process.env.GITHUB_OWNER) return process.env.GITHUB_OWNER;
     const repoString = cachedDynamicConfig?.github_repo;
-    if (typeof repoString !== "string" || !repoString.includes("/")) {
-      throw new Error("GITHUB_OWNER must be set via environment variable or project_config.json");
+    if (typeof repoString === "string" && repoString.includes("/")) {
+      return repoString.split("/")[0];
+    }
+    if (process.env.GITHUB_OWNER) return process.env.GITHUB_OWNER;
+    const envRepo = process.env.GITHUB_REPOSITORY || process.env.GH_REPO;
+    if (envRepo && envRepo.includes("/")) {
+      return envRepo.split("/")[0];
     }
-    return repoString.split("/")[0];
+    throw new Error("GITHUB_OWNER must be set via environment variable or project_config.json");
   },
   get githubRepo() {
-    if (process.env.GITHUB_REPO) return process.env.GITHUB_REPO;
     const repoString = cachedDynamicConfig?.github_repo;
-    if (typeof repoString !== "string" || !repoString.includes("/")) {
-      throw new Error("GITHUB_REPO must be set via environment variable or project_config.json");
+    if (typeof repoString === "string" && repoString.includes("/")) {
+      return repoString.split("/")[1];
+    }
+    if (process.env.GITHUB_REPO) return process.env.GITHUB_REPO;
+    const envRepo = process.env.GITHUB_REPOSITORY || process.env.GH_REPO;
+    if (envRepo && envRepo.includes("/")) {
+      return envRepo.split("/")[1];
     }
-    return repoString.split("/")[1];
+    throw new Error("GITHUB_REPO must be set via environment variable or project_config.json");
   },
   get repoPath() {
     return findRepoRoot();

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,
+      });
+    });
+  });
+});