@bridge_gpt/mcp-server 0.1.13 → 0.1.16

package/build/version.generated.js

@@ -1,2 +1,2 @@
 // AUTO-GENERATED — do not edit manually. Regenerate with: npm run build
-export const VERSION = "0.1.13";
+export const VERSION = "0.1.16";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bridge_gpt/mcp-server",
-  "version": "0.1.13",
+  "version": "0.1.16",
   "description": "Bridge API MCP server — exposes Jira endpoints as MCP tools for Claude Code agents",
   "license": "MIT",
   "type": "module",
@@ -9,10 +9,8 @@
   },
   "files": [
     "build/",
-    "!build/pipeline-utils.test.js",
-    "!build/decision-page-template.test.js",
-    "!build/update-check.test.js",
-    "!build/cli-upgrade.test.js",
+    "!build/**/*.test.js",
+    "!build/integration/**",
     "pipelines/",
     "README.md",
     "LICENSE"
@@ -21,7 +19,8 @@
     "build": "node scripts/bundle-version.js && node scripts/bundle-pipelines.js && node scripts/bundle-commands.js && node scripts/bundle-agents.js && tsc",
     "postbuild": "node scripts/prepend-shebang.cjs",
     "start": "node build/index.js",
-    "test": "node --test build/pipeline-utils.test.js build/update-check.test.js build/cli-upgrade.test.js",
+    "test": "node --test build/pipeline-utils.test.js build/update-check.test.js build/cli-upgrade.test.js build/decision-page-schema.test.js build/decision-page-template.test.js build/bundle-pipelines.test.js build/instructions-contract.test.js build/pipeline-orchestrator-persistence.test.js build/pipeline-orchestrator-execution.test.js build/pipeline-orchestrator-integration.test.js build/index-static.test.js build/start-tickets.test.js build/agent-registry.test.js build/start-tickets-prereqs.test.js build/doctor.test.js build/package-static.test.js",
+    "test:integration": "node --test build/integration/refresh-main.integration.test.js build/integration/start-tickets.integration.test.js build/integration/doctor.integration.test.js",
     "prepublishOnly": "npm run build && node scripts/verify-shebang.cjs"
   },
   "dependencies": {
@@ -30,7 +29,8 @@
   },
   "devDependencies": {
     "@types/node": "^22.0.0",
-    "typescript": "^5.7.0"
+    "typescript": "^5.7.0",
+    "undici": "^6.25.0"
   },
   "engines": {
     "node": ">=18.0.0"

package/pipelines/check-ci-ticket.json

@@ -18,8 +18,14 @@
     {
       "type": "agent_task",
       "instruction_file": "create-pr.md",
-      "description": "Create a pull request with a title derived from the commit subject",
-      "requires_approval": true
+      "description": "Create a pull request with a title derived from the commit subject"
+    },
+    {
+      "type": "mcp_call",
+      "tool": "get_config_field",
+      "params": { "field_name": "ci_followup_config" },
+      "description": "Fetch repo-specific CI follow-up configuration",
+      "on_error": "warn_and_continue"
     },
     {
       "type": "agent_task",

package/pipelines/implement-ticket.json

@@ -34,8 +34,7 @@
     {
       "type": "agent_task",
       "instruction_file": "create-pr.md",
-      "description": "Create a pull request with a title derived from the commit subject",
-      "requires_approval": true
+      "description": "Create a pull request with a title derived from the commit subject"
     },
     {
       "type": "mcp_call",
@@ -47,6 +46,13 @@
       "description": "Resolve and transition ticket to post-PR status",
       "on_error": "warn_and_continue"
     },
+    {
+      "type": "mcp_call",
+      "tool": "get_config_field",
+      "params": { "field_name": "ci_followup_config" },
+      "description": "Fetch repo-specific CI follow-up configuration",
+      "on_error": "warn_and_continue"
+    },
     {
       "type": "agent_task",
       "instruction_file": "monitor-ci-checks.md",

package/pipelines/pr-ticket.json

@@ -18,8 +18,7 @@
     {
       "type": "agent_task",
       "instruction_file": "create-pr.md",
-      "description": "Create a pull request with a title derived from the commit subject",
-      "requires_approval": true
+      "description": "Create a pull request with a title derived from the commit subject"
     }
   ]
 }

package/build/decision-page-schema.test.js

@@ -1,248 +0,0 @@
-import { describe, it } from "node:test";
-import assert from "node:assert/strict";
-import { ActionableItemSchema } from "./decision-page-schema.js";
-// ---------------------------------------------------------------------------
-// Fixture helpers
-// ---------------------------------------------------------------------------
-function validItemRaw() {
-    return {
-        id: "E-1",
-        question: "E-1: Timeout behavior needs a decision",
-        original_question: "Should the ticket specify timeout behavior for slow upstream calls?",
-        why_it_matters: "Timeout behavior affects retry paths and determines whether users see stale, failed, or delayed responses.",
-        options: ["Keep existing", "Add configurable", "Defer"],
-        option_consequences: [
-            "No new implementation work, but slow calls keep current failure behavior.",
-            "Implementers add configuration and tests for timeout-specific behavior.",
-            "The ticket remains smaller, but timeout ambiguity persists for a later change.",
-        ],
-        recommendation_explanation: "Configurable timeout best matches code paths that already branch on external service latency.",
-        codebase_evidence: "Assessment: ...\n\nCodebase Evidence:\n- src/api/timeouts.ts:42 — current latency-branching helper",
-        source: "Clarifying Q1 (prior round, weak concurrence)",
-        recommendation_index: 1,
-    };
-}
-// ---------------------------------------------------------------------------
-// Schema validation — happy path
-// ---------------------------------------------------------------------------
-describe("ActionableItemSchema — happy path", () => {
-    it("accepts a valid actionable item using the new contract", () => {
-        const result = ActionableItemSchema.safeParse(validItemRaw());
-        assert.equal(result.success, true);
-    });
-    it("does not require a context field", () => {
-        const item = validItemRaw();
-        const result = ActionableItemSchema.safeParse(item);
-        assert.equal(result.success, true);
-        if (result.success) {
-            assert.equal(result.data.context, undefined);
-        }
-    });
-    it("accepts the minimum branch count (2 options, 2 consequences)", () => {
-        const item = {
-            ...validItemRaw(),
-            options: ["A", "B"],
-            option_consequences: ["Consequence A.", "Consequence B."],
-            recommendation_index: 0,
-        };
-        const result = ActionableItemSchema.safeParse(item);
-        assert.equal(result.success, true);
-    });
-    it("accepts the maximum branch count (4 options, 4 consequences)", () => {
-        const item = {
-            ...validItemRaw(),
-            options: ["A", "B", "C", "D"],
-            option_consequences: ["A.", "B.", "C.", "D."],
-            recommendation_index: 0,
-        };
-        const result = ActionableItemSchema.safeParse(item);
-        assert.equal(result.success, true);
-    });
-});
-// ---------------------------------------------------------------------------
-// Schema validation — required fields
-// ---------------------------------------------------------------------------
-describe("ActionableItemSchema — required clarity fields", () => {
-    for (const fieldName of [
-        "original_question",
-        "why_it_matters",
-        "recommendation_explanation",
-        "codebase_evidence",
-    ]) {
-        it(`rejects payloads missing ${fieldName}`, () => {
-            const item = { ...validItemRaw() };
-            delete item[fieldName];
-            const result = ActionableItemSchema.safeParse(item);
-            assert.equal(result.success, false);
-            if (!result.success) {
-                const paths = result.error.issues.map((issue) => issue.path.join("."));
-                assert.ok(paths.includes(fieldName), `expected issue at path ${fieldName}, got ${paths.join(", ")}`);
-            }
-        });
-    }
-    it("rejects payloads missing option_consequences", () => {
-        const item = { ...validItemRaw() };
-        delete item.option_consequences;
-        const result = ActionableItemSchema.safeParse(item);
-        assert.equal(result.success, false);
-        if (!result.success) {
-            const paths = result.error.issues.map((issue) => issue.path.join("."));
-            assert.ok(paths.includes("option_consequences"));
-        }
-    });
-    it("rejects payloads that still use a legacy context field instead of the new clarity fields", () => {
-        const item = { ...validItemRaw(), context: "legacy context blob" };
-        delete item.original_question;
-        delete item.why_it_matters;
-        delete item.recommendation_explanation;
-        delete item.codebase_evidence;
-        const result = ActionableItemSchema.safeParse(item);
-        assert.equal(result.success, false);
-        if (!result.success) {
-            const paths = result.error.issues.map((issue) => issue.path.join("."));
-            assert.ok(paths.includes("original_question"));
-            assert.ok(paths.includes("why_it_matters"));
-            assert.ok(paths.includes("recommendation_explanation"));
-            assert.ok(paths.includes("codebase_evidence"));
-        }
-    });
-});
-// ---------------------------------------------------------------------------
-// Schema validation — branch count boundaries
-// ---------------------------------------------------------------------------
-describe("ActionableItemSchema — 2–4 branch boundary", () => {
-    it("rejects options with only one branch", () => {
-        const item = {
-            ...validItemRaw(),
-            options: ["Only one"],
-            option_consequences: ["Only one."],
-            recommendation_index: 0,
-        };
-        const result = ActionableItemSchema.safeParse(item);
-        assert.equal(result.success, false);
-    });
-    it("rejects options with five branches", () => {
-        const item = {
-            ...validItemRaw(),
-            options: ["A", "B", "C", "D", "E"],
-            option_consequences: ["A.", "B.", "C.", "D.", "E."],
-            recommendation_index: 0,
-        };
-        const result = ActionableItemSchema.safeParse(item);
-        assert.equal(result.success, false);
-    });
-    it("rejects option_consequences with only one entry", () => {
-        const item = {
-            ...validItemRaw(),
-            options: ["A", "B"],
-            option_consequences: ["Only one."],
-            recommendation_index: 0,
-        };
-        const result = ActionableItemSchema.safeParse(item);
-        assert.equal(result.success, false);
-    });
-    it("rejects option_consequences with five entries", () => {
-        const item = {
-            ...validItemRaw(),
-            options: ["A", "B"],
-            option_consequences: ["A.", "B.", "C.", "D.", "E."],
-            recommendation_index: 0,
-        };
-        const result = ActionableItemSchema.safeParse(item);
-        assert.equal(result.success, false);
-    });
-});
-// ---------------------------------------------------------------------------
-// Schema validation — option/consequence parity
-// ---------------------------------------------------------------------------
-describe("ActionableItemSchema — option/consequence parity", () => {
-    it("rejects mismatched lengths and attaches the issue to option_consequences", () => {
-        const item = {
-            ...validItemRaw(),
-            options: ["A", "B", "C"],
-            option_consequences: ["A.", "B."],
-            recommendation_index: 0,
-        };
-        const result = ActionableItemSchema.safeParse(item);
-        assert.equal(result.success, false);
-        if (!result.success) {
-            const matchingIssue = result.error.issues.find((issue) => issue.path.join(".") === "option_consequences" && /must match options length/.test(issue.message));
-            assert.ok(matchingIssue, `expected an option_consequences issue with the parity message; got: ${JSON.stringify(result.error.issues)}`);
-        }
-    });
-    it("rejects mismatched lengths in the other direction", () => {
-        const item = {
-            ...validItemRaw(),
-            options: ["A", "B"],
-            option_consequences: ["A.", "B.", "C."],
-            recommendation_index: 0,
-        };
-        const result = ActionableItemSchema.safeParse(item);
-        assert.equal(result.success, false);
-    });
-});
-// ---------------------------------------------------------------------------
-// Schema validation — recommendation_index upper bound
-// ---------------------------------------------------------------------------
-describe("ActionableItemSchema — recommendation_index upper bound", () => {
-    it("rejects recommendation_index >= options.length and attaches the issue to recommendation_index", () => {
-        const item = {
-            ...validItemRaw(),
-            options: ["A", "B", "C"],
-            option_consequences: ["A.", "B.", "C."],
-            recommendation_index: 99,
-        };
-        const result = ActionableItemSchema.safeParse(item);
-        assert.equal(result.success, false);
-        if (!result.success) {
-            const matchingIssue = result.error.issues.find((issue) => issue.path.join(".") === "recommendation_index" && /out of bounds/.test(issue.message));
-            assert.ok(matchingIssue, `expected a recommendation_index issue with the out-of-bounds message; got: ${JSON.stringify(result.error.issues)}`);
-        }
-    });
-    it("rejects recommendation_index equal to options.length (off-by-one)", () => {
-        const item = {
-            ...validItemRaw(),
-            options: ["A", "B"],
-            option_consequences: ["A.", "B."],
-            recommendation_index: 2,
-        };
-        const result = ActionableItemSchema.safeParse(item);
-        assert.equal(result.success, false);
-    });
-});
-// ---------------------------------------------------------------------------
-// id format — must produce a valid HTML id so getElementById can find the
-// per-item textarea/radio. Whitespace and dots silently break that lookup
-// (and would skip the "no comment selected" validation), so the schema
-// rejects anything outside [A-Za-z0-9_-].
-// ---------------------------------------------------------------------------
-describe("ActionableItemSchema — id format", () => {
-    it("rejects an id containing whitespace", () => {
-        const item = { ...validItemRaw(), id: "E 1" };
-        const result = ActionableItemSchema.safeParse(item);
-        assert.equal(result.success, false);
-        if (!result.success) {
-            assert.ok(result.error.issues.some((issue) => issue.path[0] === "id"));
-        }
-    });
-    it("rejects an id containing only whitespace", () => {
-        const item = { ...validItemRaw(), id: "   " };
-        const result = ActionableItemSchema.safeParse(item);
-        assert.equal(result.success, false);
-    });
-    it("rejects an id containing a dot", () => {
-        const item = { ...validItemRaw(), id: "E.1" };
-        const result = ActionableItemSchema.safeParse(item);
-        assert.equal(result.success, false);
-    });
-    it("accepts an id with hyphens (matches the instruction's generator format)", () => {
-        const item = { ...validItemRaw(), id: "E-1" };
-        const result = ActionableItemSchema.safeParse(item);
-        assert.equal(result.success, true);
-    });
-    it("accepts an id with underscores", () => {
-        const item = { ...validItemRaw(), id: "decision_1" };
-        const result = ActionableItemSchema.safeParse(item);
-        assert.equal(result.success, true);
-    });
-});