@bridge_gpt/mcp-server 0.2.58 → 0.2.60

package/build/agent-utils.js

@@ -115,10 +115,18 @@ export function translateAgentToCopilot(frontmatter, body) {
  * command catalog.
  */
 function commandDescription(source, commandName) {
-    const line = source
-        .split(/\r?\n/)
-        .map((candidate) => candidate.trim())
-        .find((candidate) => candidate.length > 0 && candidate !== "$ARGUMENTS" && !candidate.startsWith("#"));
+    const lines = source.split(/\r?\n/).map((candidate) => candidate.trim());
+    // Some commands open with host-parsed YAML frontmatter; its delimiters and
+    // keys are metadata, not a summary, so drop the block before scanning.
+    if (lines[0] === "---") {
+        const close = lines.indexOf("---", 1);
+        if (close > 0)
+            lines.splice(0, close + 1);
+    }
+    const line = lines.find((candidate) => candidate.length > 0 &&
+        candidate !== "$ARGUMENTS" &&
+        !candidate.startsWith("#") &&
+        !/^[-*_\s]+$/.test(candidate));
     return line ?? `Run the Bridge ${commandName} workflow.`;
 }
 function commandBodyForPrompt(source) {
@@ -136,17 +144,17 @@ function commandBodyForSkill(source) {
 export function translateCommandToCopilot(filename, source) {
     const commandName = filename.replace(/\.md$/, "");
     const description = commandDescription(source, commandName);
+    // No `tools:` frontmatter: Bridge MCP tools reach Copilot through the
+    // generated `.vscode/mcp.json` registration, not a prompt-level restriction.
     const prompt = [
         "---",
         `description: ${quoteYamlValue(description)}`,
         "argument-hint: \"[task or ticket key]\"",
-        "tools:",
-        "  - bridge/*",
         "---",
         "",
         `# Bridge ${commandName} workflow`,
         "",
-        "The user input is available as `${input:task}`. Follow the canonical workflow below.",
+        "The workflow input is `${input:task}`. When text is supplied after the slash command, that text takes precedence; if neither contains input, ask the user for the task before continuing. Follow the canonical workflow below.",
         "",
         commandBodyForPrompt(source),
     ].join("\n");
@@ -160,7 +168,7 @@ export function translateCommandToCopilot(filename, source) {
         "",
         `# Bridge ${commandName} workflow`,
         "",
-        "Use the text supplied after this slash command as the workflow input. Follow the canonical workflow below.",
+        "Use the text supplied after this slash command as the workflow input; if no text was supplied, ask the user for the task before continuing. Follow the canonical workflow below.",
         "",
         commandBodyForSkill(source),
     ].join("\n");

package/build/claude-user-config-doctor.js

@@ -52,21 +52,6 @@ const CLAUDE_USER_CONFIG_FILENAME = ".claude.json";
 export function resolveClaudeUserConfigPath(homeDirectory, platform) {
     return pathApiForProvisioningPlatform(platform).join(homeDirectory, CLAUDE_USER_CONFIG_FILENAME);
 }
-/**
- * RETIRED (BAPI-790) — this override is now behaviorally inert.
- *
- * It used to downgrade a confirmed shadowing collision from a refusal to a loud
- * warning. There is no refusal left to downgrade: strict flag-scoped MCP loading
- * means an operator registration cannot shadow a worker, so the finding is always
- * advisory and setting this variable changes nothing.
- *
- * The constant is kept exported, and {@link isClaudeMcpShadowingOverrideEnabled}
- * with it, so an operator who still has it set in a service unit or shell profile
- * gets no error — and so the name stays greppable to whoever wonders why it stopped
- * mattering. Nothing in the policy consults it. Do not reintroduce a branch on it
- * without re-establishing that shadowing can affect a worker again.
- */
-export const CONDUCTOR_ALLOW_CLAUDE_MCP_SHADOWING_ENV = "BAPI_CONDUCTOR_ALLOW_CLAUDE_MCP_SHADOWING";
 /** A plain, non-array object. */
 function asRecord(value) {
     if (!value || typeof value !== "object" || Array.isArray(value))
@@ -262,27 +247,8 @@ export async function inspectClaudeUserConfigForMcpShadowing(input, deps) {
     return { findings, diagnostics };
 }
 // ---------------------------------------------------------------------------
-// Override + shared policy
+// Shared policy
 // ---------------------------------------------------------------------------
-/** Documented truthy override values (trimmed, case-insensitive). */
-const OVERRIDE_TRUTHY = new Set(["1", "true", "yes", "on"]);
-/**
- * True only for a trimmed, case-insensitive `1`, `true`, `yes`, or `on`. Absent,
- * blank, `0`, `false`, `no`, `off`, and anything else are disabled.
- *
- * RETIRED (BAPI-790): no policy branches on this any more — see
- * {@link CONDUCTOR_ALLOW_CLAUDE_MCP_SHADOWING_ENV}. Retained as a pure predicate
- * so the parsing rule stays documented and testable rather than being deleted and
- * re-guessed if the override is ever revived.
- */
-export function isClaudeMcpShadowingOverrideEnabled(env) {
-    const raw = env?.[CONDUCTOR_ALLOW_CLAUDE_MCP_SHADOWING_ENV];
-    if (typeof raw !== "string")
-        return false;
-    return OVERRIDE_TRUTHY.has(raw.trim().toLowerCase());
-}
-/** The exact wording every override path emits, pinned by tests. */
-export const CLAUDE_MCP_SHADOWING_OVERRIDE_WARNING = `Proceeding anyway because ${CONDUCTOR_ALLOW_CLAUDE_MCP_SHADOWING_ENV} is set`;
 /** Human-readable scope labels (fixed strings, never derived from the config). */
 const SCOPE_LABELS = {
     user: "user scope (mcpServers)",

package/build/codex-bridge-profile.js

@@ -13,7 +13,7 @@ export const BRIDGE_WORKFLOW_TOOLS = [
     "ping", "get_ticket", "get_pipeline_recipe", "request_ticket_review",
     "wait_for_ticket_review", "fresh_base", "generate_decision_page",
     "capture_review_decisions", "request_plan_generation", "get_plan",
-    "attachment",
+    "attachment", "publish_reviewed_attachment",
     "get_phase_context", "record_phase_result", "create_pull_request",
     "update_jira_status", "config_field", "resolve_ci_checks", "poll_ci_checks",
     "wait_for_ci_checks",