@bridge_gpt/mcp-server 0.1.16 → 0.1.17

package/build/index.js

@@ -16,7 +16,7 @@ import { writeFile, mkdir, readFile, stat } from "fs/promises";
 import path from "path";
 import { execSync } from "child_process";
 import { createRequire } from "module";
-import { PIPELINES as BUNDLED_PIPELINES, INSTRUCTIONS as BUNDLED_INSTRUCTIONS } from "./pipelines.generated.js";
+import { PIPELINES as BUNDLED_PIPELINES, INSTRUCTIONS as BUNDLED_INSTRUCTIONS, CHAIN_RECIPES } from "./pipelines.generated.js";
 import { COMMANDS } from "./commands.generated.js";
 import { AGENTS } from "./agents.generated.js";
 import { VERSION } from "./version.generated.js";
@@ -25,9 +25,11 @@ import { reconstructAgentMarkdown, translateAgentToCopilot } from "./agent-utils
 import { resolveRecipe, loadCustomPipelines } from "./pipeline-utils.js";
 import { runStartTicketsCli } from "./start-tickets.js";
 import { runDoctorCli } from "./doctor.js";
+import { runScheduleRunCli } from "./schedule-run.js";
 import { generateDecisionPageHtml } from "./decision-page-template.js";
 import { DecisionPageInputShape } from "./decision-page-schema.js";
-import { runPipeline, resumePipeline, listPipelineRuns, deletePipelineRun, } from "./pipeline-orchestrator.js";
+import { runPipeline, resumePipeline, listPipelineRuns, deletePipelineRun, deriveIdeaHash, } from "./pipeline-orchestrator.js";
+import { runFullAutomation, resumeFullAutomation, } from "./chain-orchestrator.js";
 // Mutable pipeline/instruction state — starts with bundled, merged with user at startup
 const PIPELINES = { ...BUNDLED_PIPELINES };
 const INSTRUCTIONS = { ...BUNDLED_INSTRUCTIONS };
@@ -127,6 +129,8 @@ async function createTicketRequest(params) {
         payload.labels = params.labels;
     if (params.assignee)
         payload.assignee = params.assignee;
+    if (params.parent_key)
+        payload.parent_key = params.parent_key;
     const resp = await fetch(buildUrl("/create-ticket"), {
         method: "POST",
         headers: POST_HEADERS,
@@ -704,6 +708,12 @@ async function dispatchCliSubcommand(argv) {
     if (argv[0] === "doctor") {
         return runDoctorCli(argv.slice(1));
     }
+    // The local-only `schedule-run` subcommand (BAPI-327) is likewise a positional
+    // subcommand that owns the rest of argv, routed before the --init / --upgrade
+    // flag guards and never starts the MCP server.
+    if (argv[0] === "schedule-run") {
+        return runScheduleRunCli(argv.slice(1));
+    }
     // --init takes precedence over --upgrade; both are position-independent flags.
     if (argv.includes("--init")) {
         return runInitCli(cwd);
@@ -854,7 +864,7 @@ registerTool("get_project_standards", {
 });
 registerTool("get_tickets", {
     description: "Search for and list Jira tickets from the configured project. " +
-        "Filters by query text, status name, or date. Returns up to 'limit' tickets ordered by most recently updated. " +
+        "Filters by query text, status name, label, or date. Returns up to 'limit' tickets ordered by most recently updated. " +
         "All data is fetched live from Jira. Use get_ticket to retrieve full details for a specific ticket.",
     inputSchema: {
         query: z
@@ -867,6 +877,12 @@ registerTool("get_tickets", {
             .string()
             .optional()
             .describe("Filter by Jira status name (e.g. 'To Do', 'In Progress', 'Done')"),
+        labels: z
+            .string()
+            .optional()
+            .describe("Comma-separated Jira labels. Filters tickets via JQL labels in (...) " +
+            "(matches tickets carrying any of the given labels). Labels cannot contain spaces. " +
+            "Example: \"bapi-idea-to-ticket-fa-1a2b3c\""),
         limit: z
             .number()
             .optional()
@@ -882,12 +898,14 @@ registerTool("get_tickets", {
             .optional()
             .describe("ISO date string (YYYY-MM-DD). Only return tickets updated on or after this date"),
     },
-}, async ({ query, status, limit, offset, updated_since }) => {
+}, async ({ query, status, labels, limit, offset, updated_since }) => {
     const params = { repo_name: REPO_NAME };
     if (query)
         params.query = query;
     if (status)
         params.status = status;
+    if (labels)
+        params.labels = labels;
     if (limit !== undefined)
         params.limit = String(limit);
     if (offset !== undefined && offset > 0)
@@ -936,7 +954,8 @@ registerTool("create_ticket", {
     description: "Create a new Jira ticket in the configured project. Requires either description or file_path (or both — file_path takes precedence). " +
         "Returns JSON with {ticket_key: 'PROJ-123', url: 'https://...'}. " +
         "The ticket is created immediately in Jira — confirm details with the user before calling. " +
-        "The description field supports Jira markdown formatting.",
+        "The description field supports Jira markdown formatting. " +
+        "Pass parent_key ONLY when creating a child ticket under an existing Jira Epic; omit it for standalone tickets and for Epic parent creation itself.",
     inputSchema: {
         summary: z.string().describe("Ticket title — keep under 100 characters"),
         description: z
@@ -966,12 +985,17 @@ registerTool("create_ticket", {
             .string()
             .optional()
             .describe("Jira username or account ID of the assignee. Omit to leave unassigned"),
+        parent_key: z
+            .string()
+            .optional()
+            .describe("Optional Jira Epic key to set as the parent of the newly created child issue. " +
+            "Omit for standalone tickets and Epic parent creation."),
     },
-}, async ({ summary, description, file_path, issue_type, priority, labels, assignee }) => {
+}, async ({ summary, description, file_path, issue_type, priority, labels, assignee, parent_key }) => {
     const resolved = await resolveTextOrFile(description, file_path, "description");
     if (!resolved.ok)
         return resolved.errorResponse;
-    const text = await createTicketRequest({ summary, description: resolved.text, issue_type, priority, labels, assignee });
+    const text = await createTicketRequest({ summary, description: resolved.text, issue_type, priority, labels, assignee, parent_key });
     return { content: [{ type: "text", text: text + resolved.note }] };
 });
 registerTool("get_plan", {
@@ -2044,6 +2068,7 @@ const VALID_CONFIG_FIELDS = [
     "post_pr_target_status", "ci_check_config", "ci_followup_config",
     "allow_mutating_smoke_ops",
     "selected_mcp_slugs",
+    "base_branch",
 ].join(", ");
 registerTool("list_config_fields", {
     description: "List all configurable fields available for reading and updating via the Bridge API. " +
@@ -2094,6 +2119,9 @@ registerTool("update_config_field", {
             "The selected_mcp_slugs field takes a JSON array of supported MCP validation manual slug strings " +
             "(e.g. [\"b2c-commerce-developer\", \"playwright-mcp\", \"pwa-kit-mcp\"]) — pass an array of strings, " +
             "not a comma-delimited string; an empty array clears the selection. " +
+            "The base_branch field is a string/null field controlling the development base branch used by PR " +
+            "creation (/create-pr) and start-tickets worktree creation; an empty/null value clears it and " +
+            "automations fall back to 'main'. " +
             "For string fields, omit both value and file_path to set the field to NULL (clearing it). " +
             "Scalar boolean fields are NOT NULL and have no clear/null state: omitting the value writes false " +
             "(matching the API-layer coercion), so pass true/false explicitly."),
@@ -2791,7 +2819,7 @@ registerTool("list_pipelines", {
     const list = Object.entries(PIPELINES).map(([key, pipeline]) => ({
         name: key,
         description: pipeline.description ?? "",
-        variables: (pipeline.variables ?? []).filter((v) => v !== "docs_dir"),
+        variables: (pipeline.variables ?? []).filter((v) => v !== "docs_dir" && v !== "idea_hash"),
         source: userPipelineKeys.has(key) ? "user" : "bundled",
     }));
     return {
@@ -2809,7 +2837,7 @@ registerTool("get_pipeline_recipe", {
             .string()
             .describe("Pipeline name (e.g. 'review-ticket', 'implement-ticket')"),
         variables: z
-            .record(z.string())
+            .record(z.string(), z.string())
             .optional()
             .describe("Key-value pairs for variable substitution (e.g. { ticket_key: 'BAPI-123' })"),
         skip_steps: z
@@ -2860,6 +2888,11 @@ registerTool("get_pipeline_recipe", {
             auto_approve: auto_approve ? "true" : "",
             ...(variables ?? {}),
         };
+        // Auto-inject the stable idea-hash (like docs_dir) for the standalone
+        // /idea-to-ticket path, mirroring runPipeline.
+        if ("idea" in mergedVariables) {
+            mergedVariables.idea_hash = deriveIdeaHash(mergedVariables.idea);
+        }
         const recipe = resolveRecipe(pipelineDef, INSTRUCTIONS, mergedVariables, skip_steps, !!auto_approve);
         return {
             content: [{
@@ -2912,6 +2945,21 @@ function buildPipelineOrchestratorDeps() {
         toolHandlers: TOOL_HANDLERS,
     };
 }
+// BAPI-326: dependency injection for the full-automation chain orchestrator.
+// Same shape as the pipeline deps plus the bundled chain-recipe registry. The
+// chain orchestrator never imports index.ts; everything flows through here.
+function buildChainOrchestratorDeps() {
+    return {
+        baseUrl: BASE_URL,
+        apiKey: API_KEY,
+        repoName: REPO_NAME,
+        docsDir: BAPI_DOCS_DIR,
+        pipelines: PIPELINES,
+        chainRecipes: CHAIN_RECIPES,
+        instructions: INSTRUCTIONS,
+        toolHandlers: TOOL_HANDLERS,
+    };
+}
 registerTool("run_pipeline", {
     description: "Execute a Bridge API pipeline by name. The orchestrator runs steps sequentially, " +
         "dispatching mcp_call steps in-process and pausing on agent_task steps with a " +
@@ -2929,7 +2977,7 @@ registerTool("run_pipeline", {
             .string()
             .describe("Pipeline name (e.g. 'review-ticket', 'implement-ticket')"),
         variables: z
-            .record(z.string())
+            .record(z.string(), z.string())
             .optional()
             .describe("Key-value pairs for variable substitution (e.g. { ticket_key: 'BAPI-123' }). " +
             "Do NOT pass `auto_approve` here — use the top-level parameter."),
@@ -3021,6 +3069,85 @@ registerTool("delete_pipeline_run", {
     };
 });
 // ---------------------------------------------------------------------------
+// Full Automation Chain Tools (BAPI-326)
+// ---------------------------------------------------------------------------
+//
+// run_full_automation / resume_full_automation drive an idea through three
+// chained stages (idea-to-ticket -> review-ticket fan-out -> start-tickets CLI
+// seam) via the existing pipeline runner. They share the chain envelope shape
+// (status: needs_agent_task | completed | failed) with a strict error_code
+// enum (VALIDATION | NOT_FOUND | EXPIRED | REPO_MISMATCH | TOOL_ERROR). Chain
+// state is persisted server-side via /jira/chain-runs/runs. Registered AFTER
+// TOOL_HANDLERS and the pipeline tools so the orchestrator can dispatch the
+// existing child pipelines in-process.
+registerTool("run_full_automation", {
+    description: "Run the full-automation chain for an idea: create ticket(s) " +
+        "(idea-to-ticket), review each created ticket (review-ticket fan-out), " +
+        "then emit the exact `/start-tickets ...` command for you to invoke in " +
+        "this same session. Returns the chain envelope keyed on `status`: " +
+        "`needs_agent_task` (perform the `next_action.instruction`, then call " +
+        "`resume_full_automation` with the result as `agent_result`), `completed`, " +
+        "or `failed` (check `error_code`: VALIDATION | NOT_FOUND | EXPIRED | " +
+        "REPO_MISMATCH | TOOL_ERROR). Provide the idea via `idea` or `idea_file` " +
+        "(mutually exclusive).",
+    inputSchema: {
+        idea: z.string().optional(),
+        idea_file: z.string().optional(),
+        auto_approve: z
+            .union([z.boolean(), z.literal("true"), z.literal("false")])
+            .optional(),
+        scheduled_at: z.string().optional(),
+        max_children: z.number().int().positive().optional(),
+        allow_duplicate: z.boolean().optional(),
+        agent: z.enum(["claude"]).optional(),
+        ttl_seconds: z.number().int().positive().optional(),
+    },
+}, async (input) => {
+    const { idea, idea_file, ...rest } = input;
+    if (idea !== undefined && idea_file !== undefined) {
+        return {
+            content: [{
+                    type: "text",
+                    text: JSON.stringify({
+                        error: "BAD_REQUEST",
+                        status: 400,
+                        message: "Provide exactly one of `idea` or `idea_file`, not both.",
+                    }),
+                }],
+        };
+    }
+    const resolved = await resolveTextOrFile(idea, idea_file, "idea");
+    if (!resolved.ok) {
+        return resolved.errorResponse;
+    }
+    const result = await runFullAutomation(buildChainOrchestratorDeps(), {
+        idea: resolved.text,
+        ...rest,
+    });
+    return {
+        content: [
+            { type: "text", text: JSON.stringify(result, null, 2) },
+        ],
+    };
+});
+registerTool("resume_full_automation", {
+    description: "Resume a paused full-automation chain run. Provide the `chain_run_id` " +
+        "returned by the prior needs_agent_task envelope and the string the " +
+        "instruction asked you to produce as `agent_result`. Returns the same " +
+        "chain envelope shape as `run_full_automation`.",
+    inputSchema: {
+        chain_run_id: z.string(),
+        agent_result: z.string(),
+    },
+}, async (input) => {
+    const result = await resumeFullAutomation(buildChainOrchestratorDeps(), input);
+    return {
+        content: [
+            { type: "text", text: JSON.stringify(result, null, 2) },
+        ],
+    };
+});
+// ---------------------------------------------------------------------------
 // generate_decision_page
 // ---------------------------------------------------------------------------
 registerTool("generate_decision_page", {

package/build/pipeline-orchestrator.js

@@ -16,7 +16,21 @@
  *     are inlined into the instruction string so the agent can review and
  *     confirm before resuming. (BAPI-275 decisions E-4 / E-36.)
  */
+import { createHash } from "node:crypto";
 import { resolveRecipe } from "./pipeline-utils.js";
+/**
+ * Derive a STABLE, content-addressed hash of an idea — the basis for the
+ * cross-run idempotency label ``bapi-idea-hash-{idea_hash}``. Normalizes
+ * (trim + lowercase + collapse internal whitespace) so cosmetic edits don't
+ * change the hash, then takes the first 12 hex chars of the SHA-256. Unlike the
+ * per-run ``run_id`` label, this is identical every time the same idea is run,
+ * so a re-run of the same idea is reliably caught by label (not just fuzzy
+ * keyword search). Auto-injected as the ``idea_hash`` pipeline variable.
+ */
+export function deriveIdeaHash(idea) {
+    const normalized = String(idea ?? "").trim().toLowerCase().replace(/\s+/g, " ");
+    return createHash("sha256").update(normalized).digest("hex").slice(0, 12);
+}
 class PipelinePersistenceError extends Error {
     code;
     constructor(code, message) {
@@ -175,6 +189,10 @@ const ORCHESTRATION_TOOLS = new Set([
     "resume_pipeline",
     "list_pipeline_runs",
     "delete_pipeline_run",
+    // BAPI-326: chain orchestration tools must never be invoked from within a
+    // declarative pipeline step (avoids accidental orchestration recursion).
+    "run_full_automation",
+    "resume_full_automation",
 ]);
 // ---------------------------------------------------------------------------
 // runPipeline
@@ -196,6 +214,12 @@ export async function runPipeline(deps, input) {
         auto_approve: autoApprove ? "true" : "",
         ...(input.variables ?? {}),
     };
+    // Auto-inject the stable idea-hash (like docs_dir) whenever the pipeline was
+    // given an `idea`, so both the chain and standalone callers get the cross-run
+    // idempotency label without computing it themselves.
+    if ("idea" in mergedVariables) {
+        mergedVariables.idea_hash = deriveIdeaHash(mergedVariables.idea);
+    }
     let recipe;
     try {
         recipe = resolveRecipe(pipelineDef, deps.instructions, mergedVariables, undefined, autoApprove);
@@ -227,6 +251,39 @@ export async function runPipeline(deps, input) {
     }
     return continuePipelineExecution(deps, persistence, row, recipe, autoApprove);
 }
+/**
+ * Read-only snapshot of a pipeline run's current persisted state — performs no
+ * execution and no state transition. Used by the chain orchestrator to decide,
+ * at a stage boundary, whether the inner pipeline still needs resuming
+ * (``paused``), has already reached a terminal state (``completed`` / ``failed``)
+ * so the stage can be advanced/failed idempotently, or is in an indeterminate
+ * state that cannot be safely resumed or recovered.
+ */
+export async function peekPipelineRun(deps, pipeline_run_id) {
+    const persistence = createPipelinePersistenceClient({
+        baseUrl: deps.baseUrl,
+        apiKey: deps.apiKey,
+        repoName: deps.repoName,
+    });
+    let row;
+    try {
+        row = await persistence.getRun(pipeline_run_id);
+    }
+    catch (err) {
+        if (err instanceof PipelinePersistenceError) {
+            return failedEnvelope(err.code, err.message, { pipeline_run_id });
+        }
+        return failedEnvelope("TOOL_ERROR", err instanceof Error ? err.message : String(err), { pipeline_run_id });
+    }
+    return {
+        status: row.status,
+        pipeline_run_id: row.pipeline_run_id,
+        pipeline: row.pipeline_name,
+        total_steps: row.resolved_recipe.total_steps,
+        current_step_index: row.current_step_index,
+        results: row.results,
+    };
+}
 // ---------------------------------------------------------------------------
 // resumePipeline
 // ---------------------------------------------------------------------------