@bridge_gpt/mcp-server 0.2.9 → 0.2.12

package/build/upgrade-cli.js

@@ -0,0 +1,154 @@
+/**
+ * Bridge API MCP server one-command upgrade (BAPI-430).
+ *
+ * Extracted from `index.ts` so the upgrade logic always runs from the latest
+ * published version (`@bridge_gpt/mcp-server@latest`) rather than the
+ * potentially-stale copy npx resolved at the call site.
+ *
+ * Flow:
+ *  1. Fetch latest version from the npm registry.
+ *  2. If newer and not already re-exec'd, re-spawn from `@latest` with
+ *     `--internal-reexec` so upgrade logic comes from new code.
+ *  3. Remove any competing local `node_modules/@bridge_gpt/mcp-server` install
+ *     (the silent trap that makes npx always prefer the stale local copy).
+ *  4. Rewrite the launcher pin across all per-host configs via `runInit`.
+ *  5. Print `old -> new` and open a fresh reconnect session.
+ */
+import { spawn } from "child_process";
+import { stat } from "fs/promises";
+import path from "path";
+import { VERSION } from "./version.generated.js";
+import { runInit } from "./init.js";
+import { isNewerVersion } from "./update-check.js";
+import { buildGenericAgentShellCommand, detectTerminal, getDefaultSpawnTerminalTabForPlatform, createDefaultStartTicketsDeps, } from "./start-tickets.js";
+import { AGENT_REGISTRY } from "./agent-registry.js";
+async function fetchLatestVersion() {
+    try {
+        const res = await fetch("https://registry.npmjs.org/@bridge_gpt/mcp-server/latest", {
+            signal: AbortSignal.timeout(3000),
+        });
+        if (res.ok) {
+            const data = (await res.json());
+            return data.version || null;
+        }
+    }
+    catch {
+        return null;
+    }
+    return null;
+}
+export async function runUpgradeCli(argv) {
+    const cwd = process.cwd();
+    const isDryRun = argv.includes("--dry-run");
+    const isInternalReexec = argv.includes("--internal-reexec");
+    let latestVersion = VERSION;
+    const fetched = await fetchLatestVersion();
+    if (fetched) {
+        latestVersion = fetched;
+    }
+    const isNewer = isNewerVersion(VERSION, latestVersion);
+    const npxCmd = process.platform === "win32" ? "npx.cmd" : "npx";
+    // Re-exec from @latest to ensure upgrade logic runs from latest code, not
+    // the stale local copy that npx may have resolved.
+    if (isNewer && !isInternalReexec) {
+        if (!isDryRun) {
+            console.log(`Update available: ${VERSION} -> ${latestVersion}. Re-executing from @latest...`);
+            return new Promise((resolve) => {
+                const childArgs = [
+                    "-y",
+                    "@bridge_gpt/mcp-server@latest",
+                    "upgrade",
+                    "--internal-reexec",
+                    "--old-version",
+                    VERSION,
+                ];
+                const child = spawn(npxCmd, childArgs, { stdio: "inherit", cwd });
+                child.on("close", (code) => resolve(code ?? 0));
+                child.on("error", (err) => {
+                    console.error(`Bridge API upgrade failed: could not re-exec npx: ${err.message}`);
+                    resolve(1);
+                });
+            });
+        }
+    }
+    const oldVersionIdx = argv.indexOf("--old-version");
+    const oldVersion = oldVersionIdx !== -1 && oldVersionIdx + 1 < argv.length ? argv[oldVersionIdx + 1] : VERSION;
+    const targetVersion = isDryRun && isNewer ? latestVersion : VERSION;
+    if (isDryRun) {
+        const configTargets = [".mcp.json", ".vscode/mcp.json", ".cursor/mcp.json"];
+        console.log(`\n[Dry Run] Upgrade preview:`);
+        if (isNewer && !isInternalReexec) {
+            console.log(`- Would re-exec from @latest to upgrade ${VERSION} -> ${latestVersion}`);
+        }
+        console.log(`- Would detect and remove competing local install in node_modules/@bridge_gpt/mcp-server if present.`);
+        console.log(`- Would rewrite launcher pin to @bridge_gpt/mcp-server@${targetVersion} across active per-host configs:`);
+        for (const target of configTargets) {
+            try {
+                await stat(path.join(cwd, target));
+                console.log(`    - ${target}`);
+            }
+            catch { }
+        }
+        console.log(`- Would refresh scaffolded artifacts (commands, agents, pipelines).`);
+        const spawnCommand = buildGenericAgentShellCommand(AGENT_REGISTRY.claude, "Bridge API MCP server has been upgraded. Please reload/reconnect the MCP server in your environment.", cwd, process.platform);
+        console.log(`- Would print: ${oldVersion} -> ${targetVersion} and spawn a fresh reconnect session with command:\n    ${spawnCommand}`);
+        return 0;
+    }
+    // Detect and resolve the competing-local-install trap: npx silently prefers
+    // a local node_modules copy over the registry, so the stale version wins even
+    // after an upgrade unless we remove it.
+    try {
+        const localModulePath = path.join(cwd, "node_modules", "@bridge_gpt", "mcp-server");
+        await stat(localModulePath);
+        console.log("Found stale local installation in node_modules. Removing to converge on pinned-npx...");
+        const npmCmd = process.platform === "win32" ? "npm.cmd" : "npm";
+        await new Promise((resolve) => {
+            const child = spawn(npmCmd, ["uninstall", "@bridge_gpt/mcp-server"], {
+                stdio: "inherit",
+                cwd,
+            });
+            child.on("close", (code) => {
+                if (code !== 0)
+                    console.warn("Warning: npm uninstall failed; you may need to remove node_modules/@bridge_gpt/mcp-server manually.");
+                resolve();
+            });
+            child.on("error", () => resolve());
+        });
+    }
+    catch {
+        // Not installed locally — nothing to clean up.
+    }
+    try {
+        console.log(`\nUpgrading @bridge_gpt/mcp-server to ${targetVersion}...\n`);
+        // Scaffold & rewrite the launcher pin on upgrade.
+        console.log("Refreshing scaffolded artifacts and updating launcher pins...\n");
+        await runInit(cwd);
+        console.log(`\nUpgrade complete: ${oldVersion} -> ${targetVersion}`);
+        console.log("Please reconnect your MCP now.");
+    }
+    catch (err) {
+        const msg = err instanceof Error ? err.message : String(err);
+        console.error(`Bridge API upgrade failed: ${msg}`);
+        return 1;
+    }
+    // Open a fresh agent session so the user lands in a reconnected environment
+    // rather than having to manually restart.
+    try {
+        const terminal = detectTerminal(undefined, process.env);
+        const deps = createDefaultStartTicketsDeps();
+        const spawnTerminalTab = getDefaultSpawnTerminalTabForPlatform(process.platform);
+        const spawnCommand = buildGenericAgentShellCommand(AGENT_REGISTRY.claude, "Bridge API MCP server has been upgraded. Please reload/reconnect the MCP server in your environment.", cwd, process.platform);
+        console.log("\nOpening a fresh agent session to reconnect...");
+        const spawnResult = await spawnTerminalTab(deps, terminal, spawnCommand, {
+            key: "reconnect",
+            worktreePath: cwd,
+        });
+        if (!spawnResult.ok) {
+            console.log(`\nTo continue, manually run:\n${spawnCommand}\n`);
+        }
+    }
+    catch {
+        console.log("To continue, please reload your agent session manually.");
+    }
+    return 0;
+}

package/build/version.generated.js

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bridge_gpt/mcp-server",
-  "version": "0.2.9",
+  "version": "0.2.12",
   "description": "Bridge API MCP server — exposes Jira endpoints as MCP tools for Claude Code agents",
   "license": "MIT",
   "type": "module",
@@ -22,23 +22,26 @@
     "LICENSE"
   ],
   "scripts": {
-    "build": "node scripts/bundle-version.js && node scripts/bundle-readme.js && node scripts/bundle-pipelines.js && node scripts/bundle-commands.js && node scripts/bundle-agents.js && tsc",
+    "build": "node scripts/bundle-version.js && node scripts/bundle-readme.js && node scripts/bundle-pipelines.js && node scripts/bundle-commands.js && node scripts/bundle-agents.js && tsc && node scripts/bundle-esbuild.js",
     "check:version-generated": "node scripts/bundle-version.js && node scripts/check-version-generated.js",
     "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 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/index-resolvers.test.js build/index-project-root.test.js build/index-pipelines.test.js build/index.test.js build/bridge-config.test.js build/credential-store.test.js build/agent-config-credential-migration.test.js build/mcp-invoke.test.js build/mcp-provisioning.test.js build/third-party-mcp-targets.test.js build/git-ignore-utils.test.js build/credential-materialization.test.js build/mcp-registration-doctor.test.js build/secret-safety.test.js build/start-tickets.test.js build/review-tickets.test.js build/start-tickets-base-branch.test.js build/agent-registry.test.js build/agent-registry.model-routing.test.js build/start-tickets.shell-model-routing.test.js build/start-tickets.bridge-api-model-routing.test.js build/start-tickets.tier-fetch-model-routing.test.js build/start-tickets.resolve-model-routing.test.js build/start-tickets.orchestrate-model-routing.test.js build/start-tickets.routing-diagnostics.test.js build/start-tickets-repo.test.js build/start-tickets-credential-invariants.static.test.js build/credentials-cli.test.js build/start-tickets-prereqs.test.js build/doctor.test.js build/resolveUploadAttachment.test.js build/package-static.test.js build/chain-utils.test.js build/chain-orchestrator.test.js build/scheduler-backends/types.test.js build/scheduler-backends/escaping.test.js build/scheduler-backends/launchd.test.js build/scheduler-backends/task-scheduler.test.js build/scheduler-backends/systemd-user.test.js build/scheduler-backends/at-fallback.test.js build/scheduler-backends/index.test.js build/command-catalog.test.js build/scheduled-prompt.test.js build/agent-launchers/claude.test.js build/agent-launchers/cursor.test.js build/agent-launchers/index.test.js build/schedule-store.test.js build/schedule-run.test.js build/agent-capabilities/cli.test.js build/agent-capabilities/runner.test.js build/agent-capabilities/probes.test.js build/agent-capabilities/reporter.test.js build/conductor/taxonomy-and-errors.test.js build/conductor/redaction-normalization.test.js build/conductor/claude-hook.test.js build/conductor/git-ci-types.test.js build/conductor/done-gate.test.js build/conductor/git-ci-taxonomy-payload.test.js build/conductor/bridge-api-client.test.js build/conductor/plan.test.js build/conductor/producer-ledger.test.js build/conductor/git-producer.test.js build/conductor/git-hooks.test.js build/conductor/store-migration.test.js build/conductor/pr-discovery.test.js build/conductor/pr-ci-producer.test.js build/conductor/doctor.test.js build/conductor/index-poll-ci-producer.test.js build/start-tickets-conductor.test.js build/start-tickets-conductor.spawn.test.js build/conductor/supervisor-config.test.js build/conductor/supervisor-ledger.test.js build/conductor/supervisor-state-reducer.test.js build/conductor/supervisor-housekeeping-projection.test.js build/conductor/supervisor-escalation.test.js build/conductor/supervisor-judgment.test.js build/conductor/supervisor-judgment-python-adapter.test.js build/conductor/supervisor-runtime.test.js build/conductor/supervisor-store-projection.test.js build/conductor/supervisor-cli.test.js build/conductor/supervisor-start-tickets.test.js build/conductor/supervisor-message-relay.test.js build/conductor/supervisor-state-message-events.test.js build/conductor/store-message-relay.test.js build/start-tickets-message-relay.test.js build/conductor/merge-ledger.test.js build/conductor/supervisor-merge.test.js build/conductor/bridge-api-merge-client.test.js build/conductor/bridge-api-epic-client.test.js build/conductor/supervisor-merge-runtime-state.test.js build/conductor/epic-state.test.js build/conductor/epic-reconcile.test.js build/conductor/epic-runtime.test.js build/conductor/epic-tick-sequence.test.js build/conductor/epic-runtime-post-action.test.js && node --experimental-test-module-mocks --test build/index-heavy-read-truncation.test.js build/index-artifacts.test.js build/index-brainstorm-filenames.test.js build/index-output-path.test.js build/index-generate-decision-page.test.js build/index-generate-decision-page.integration.test.js build/conductor/paths.test.js build/conductor/store-lifecycle.test.js build/conductor/store-queries.test.js build/conductor/tools.test.js build/conductor/cli.test.js build/conductor/security-regressions.test.js build/conductor/git-inspection.test.js build/conductor/tools-done-gate.test.js build/conductor/cli-git-hooks.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/index-resolvers.test.js build/index-project-root.test.js build/index-pipelines.test.js build/index.test.js build/bridge-config.test.js build/credential-store.test.js build/agent-config-credential-migration.test.js build/mcp-invoke.test.js build/mcp-provisioning.test.js build/third-party-mcp-targets.test.js build/git-ignore-utils.test.js build/credential-materialization.test.js build/mcp-registration-doctor.test.js build/secret-safety.test.js build/start-tickets.test.js build/review-tickets.test.js build/start-tickets-base-branch.test.js build/agent-registry.test.js build/agent-registry.model-routing.test.js build/start-tickets.shell-model-routing.test.js build/start-tickets.bridge-api-model-routing.test.js build/start-tickets.tier-fetch-model-routing.test.js build/start-tickets.resolve-model-routing.test.js build/start-tickets.orchestrate-model-routing.test.js build/start-tickets.routing-diagnostics.test.js build/start-tickets-repo.test.js build/start-tickets-credential-invariants.static.test.js build/credentials-cli.test.js build/start-tickets-prereqs.test.js build/doctor.test.js build/install-bridge.test.js build/resolveUploadAttachment.test.js build/package-static.test.js build/chain-utils.test.js build/chain-orchestrator.test.js build/scheduler-backends/types.test.js build/scheduler-backends/escaping.test.js build/scheduler-backends/launchd.test.js build/scheduler-backends/task-scheduler.test.js build/scheduler-backends/systemd-user.test.js build/scheduler-backends/at-fallback.test.js build/scheduler-backends/index.test.js build/command-catalog.test.js build/scheduled-prompt.test.js build/agent-launchers/claude.test.js build/agent-launchers/cursor.test.js build/agent-launchers/index.test.js build/schedule-store.test.js build/schedule-run.test.js build/agent-capabilities/cli.test.js build/agent-capabilities/runner.test.js build/agent-capabilities/probes.test.js build/agent-capabilities/reporter.test.js build/conductor/taxonomy-and-errors.test.js build/conductor/redaction-normalization.test.js build/conductor/claude-hook.test.js build/conductor/git-ci-types.test.js build/conductor/done-gate.test.js build/conductor/git-ci-taxonomy-payload.test.js build/conductor/bridge-api-client.test.js build/conductor/plan.test.js build/conductor/producer-ledger.test.js build/conductor/spec-review-producer.test.js build/conductor/git-producer.test.js build/conductor/git-hooks.test.js build/conductor/store-migration.test.js build/conductor/pr-discovery.test.js build/conductor/pr-ci-producer.test.js build/conductor/doctor.test.js build/conductor/index-poll-ci-producer.test.js build/start-tickets-conductor.test.js build/start-tickets-conductor.spawn.test.js build/conductor/supervisor-config.test.js build/conductor/supervisor-ledger.test.js build/conductor/supervisor-state-reducer.test.js build/conductor/supervisor-housekeeping-projection.test.js build/conductor/supervisor-escalation.test.js build/conductor/supervisor-judgment.test.js build/conductor/supervisor-judgment-python-adapter.test.js build/conductor/supervisor-runtime.test.js build/conductor/supervisor-store-projection.test.js build/conductor/supervisor-cli.test.js build/conductor/supervisor-start-tickets.test.js build/conductor/supervisor-message-relay.test.js build/conductor/supervisor-state-message-events.test.js build/conductor/store-message-relay.test.js build/start-tickets-message-relay.test.js build/conductor/merge-ledger.test.js build/conductor/supervisor-merge.test.js build/conductor/bridge-api-merge-client.test.js build/conductor/bridge-api-epic-client.test.js build/conductor/supervisor-merge-runtime-state.test.js build/conductor/epic-state.test.js build/conductor/epic-reconcile.test.js build/conductor/epic-runtime.test.js build/conductor/epic-tick-sequence.test.js build/conductor/epic-runtime-post-action.test.js build/mcp-profile.test.js build/mcp-profile-registration.test.js build/tools-budget.test.js build/integration/measure-tools.test.js && node --experimental-test-module-mocks --test build/index-heavy-read-truncation.test.js build/index-artifacts.test.js build/index-brainstorm-filenames.test.js build/index-output-path.test.js build/index-generate-decision-page.test.js build/index-generate-decision-page.integration.test.js build/conductor/paths.test.js build/conductor/store-lifecycle.test.js build/conductor/store-queries.test.js build/conductor/tools.test.js build/conductor/cli.test.js build/conductor/security-regressions.test.js build/conductor/git-inspection.test.js build/conductor/tools-done-gate.test.js build/conductor/cli-git-hooks.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 build/integration/agent-capabilities.integration.test.js build/integration/conductor-producer.integration.test.js build/integration/conductor-message-relay.integration.test.js",
     "test:smoke": "node --test build/integration/packaged-cli-smoke.test.js",
     "prepublishOnly": "node scripts/bundle-assets.js && npm run build && node scripts/verify-shebang.cjs"
   },
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.29.0",
-    "better-sqlite3": "^11.8.1",
     "zod": "^4.4.3"
   },
+  "optionalDependencies": {
+    "better-sqlite3": "^11.8.1"
+  },
   "devDependencies": {
     "@types/better-sqlite3": "^7.6.12",
     "@types/node": "^25.9.1",
+    "esbuild": "^0.25.0",
     "typescript": "^6.0.3"
   },
   "engines": {

package/pipelines/check-ci-ticket.json

@@ -22,8 +22,8 @@
     },
     {
       "type": "mcp_call",
-      "tool": "get_config_field",
-      "params": { "field_name": "ci_followup_config" },
+      "tool": "config_field",
+      "params": { "operation": "get", "field_name": "ci_followup_config" },
       "description": "Fetch repo-specific CI follow-up configuration",
       "on_error": "warn_and_continue"
     },

package/pipelines/implement-ticket.json

@@ -48,8 +48,8 @@
     },
     {
       "type": "mcp_call",
-      "tool": "get_config_field",
-      "params": { "field_name": "ci_followup_config" },
+      "tool": "config_field",
+      "params": { "operation": "get", "field_name": "ci_followup_config" },
       "description": "Fetch repo-specific CI follow-up configuration",
       "on_error": "warn_and_continue"
     },

package/pipelines/learn-repository.json

@@ -1,7 +1,9 @@
 {
   "name": "learn-repository",
   "description": "Learn and document all configuration fields for the repository by running sequential research agents.",
-  "variables": ["docs_dir"],
+  "variables": [
+    "docs_dir"
+  ],
   "steps": [
     {
       "type": "mcp_call",
@@ -16,8 +18,11 @@
     },
     {
       "type": "mcp_call",
-      "tool": "get_config_field",
-      "params": { "field_name": "architecture_instructions" },
+      "tool": "config_field",
+      "params": {
+        "field_name": "architecture_instructions",
+        "operation": "get"
+      },
       "description": "Fetch current architecture_instructions value",
       "on_error": "warn_and_continue"
     },
@@ -28,18 +33,22 @@
     },
     {
       "type": "mcp_call",
-      "tool": "update_config_field",
+      "tool": "config_field",
       "params": {
         "field_name": "architecture_instructions",
-        "file_path": "{docs_dir}/standards/architecture_instructions.md"
+        "file_path": "{docs_dir}/standards/architecture_instructions.md",
+        "operation": "update"
       },
       "description": "Upload architecture_instructions to config",
       "requires_approval": true
     },
     {
       "type": "mcp_call",
-      "tool": "get_config_field",
-      "params": { "field_name": "review_instructions" },
+      "tool": "config_field",
+      "params": {
+        "field_name": "review_instructions",
+        "operation": "get"
+      },
       "description": "Fetch current review_instructions value",
       "on_error": "warn_and_continue"
     },
@@ -50,18 +59,22 @@
     },
     {
       "type": "mcp_call",
-      "tool": "update_config_field",
+      "tool": "config_field",
       "params": {
         "field_name": "review_instructions",
-        "file_path": "{docs_dir}/standards/review_instructions.md"
+        "file_path": "{docs_dir}/standards/review_instructions.md",
+        "operation": "update"
       },
       "description": "Upload review_instructions to config",
       "requires_approval": true
     },
     {
       "type": "mcp_call",
-      "tool": "get_config_field",
-      "params": { "field_name": "documentation_instructions" },
+      "tool": "config_field",
+      "params": {
+        "field_name": "documentation_instructions",
+        "operation": "get"
+      },
       "description": "Fetch current documentation_instructions value",
       "on_error": "warn_and_continue"
     },
@@ -72,18 +85,22 @@
     },
     {
       "type": "mcp_call",
-      "tool": "update_config_field",
+      "tool": "config_field",
       "params": {
         "field_name": "documentation_instructions",
-        "file_path": "{docs_dir}/standards/documentation_instructions.md"
+        "file_path": "{docs_dir}/standards/documentation_instructions.md",
+        "operation": "update"
       },
       "description": "Upload documentation_instructions to config",
       "requires_approval": true
     },
     {
       "type": "mcp_call",
-      "tool": "get_config_field",
-      "params": { "field_name": "unit_testing_instructions" },
+      "tool": "config_field",
+      "params": {
+        "field_name": "unit_testing_instructions",
+        "operation": "get"
+      },
       "description": "Fetch current unit_testing_instructions value",
       "on_error": "warn_and_continue"
     },
@@ -94,18 +111,22 @@
     },
     {
       "type": "mcp_call",
-      "tool": "update_config_field",
+      "tool": "config_field",
       "params": {
         "field_name": "unit_testing_instructions",
-        "file_path": "{docs_dir}/standards/unit_testing_instructions.md"
+        "file_path": "{docs_dir}/standards/unit_testing_instructions.md",
+        "operation": "update"
       },
       "description": "Upload unit_testing_instructions to config",
       "requires_approval": true
     },
     {
       "type": "mcp_call",
-      "tool": "get_config_field",
-      "params": { "field_name": "e2e_testing_instructions" },
+      "tool": "config_field",
+      "params": {
+        "field_name": "e2e_testing_instructions",
+        "operation": "get"
+      },
       "description": "Fetch current e2e_testing_instructions value",
       "on_error": "warn_and_continue"
     },
@@ -116,18 +137,22 @@
     },
     {
       "type": "mcp_call",
-      "tool": "update_config_field",
+      "tool": "config_field",
       "params": {
         "field_name": "e2e_testing_instructions",
-        "file_path": "{docs_dir}/standards/e2e_testing_instructions.md"
+        "file_path": "{docs_dir}/standards/e2e_testing_instructions.md",
+        "operation": "update"
       },
       "description": "Upload e2e_testing_instructions to config",
       "requires_approval": true
     },
     {
       "type": "mcp_call",
-      "tool": "get_config_field",
-      "params": { "field_name": "frontend_correctness_standards" },
+      "tool": "config_field",
+      "params": {
+        "field_name": "frontend_correctness_standards",
+        "operation": "get"
+      },
       "description": "Fetch current frontend_correctness_standards value",
       "on_error": "warn_and_continue"
     },
@@ -138,18 +163,22 @@
     },
     {
       "type": "mcp_call",
-      "tool": "update_config_field",
+      "tool": "config_field",
       "params": {
         "field_name": "frontend_correctness_standards",
-        "file_path": "{docs_dir}/standards/frontend_correctness_standards.md"
+        "file_path": "{docs_dir}/standards/frontend_correctness_standards.md",
+        "operation": "update"
       },
       "description": "Upload frontend_correctness_standards to config",
       "requires_approval": true
     },
     {
       "type": "mcp_call",
-      "tool": "get_config_field",
-      "params": { "field_name": "backend_correctness_standards" },
+      "tool": "config_field",
+      "params": {
+        "field_name": "backend_correctness_standards",
+        "operation": "get"
+      },
       "description": "Fetch current backend_correctness_standards value",
       "on_error": "warn_and_continue"
     },
@@ -160,18 +189,22 @@
     },
     {
       "type": "mcp_call",
-      "tool": "update_config_field",
+      "tool": "config_field",
       "params": {
         "field_name": "backend_correctness_standards",
-        "file_path": "{docs_dir}/standards/backend_correctness_standards.md"
+        "file_path": "{docs_dir}/standards/backend_correctness_standards.md",
+        "operation": "update"
       },
       "description": "Upload backend_correctness_standards to config",
       "requires_approval": true
     },
     {
       "type": "mcp_call",
-      "tool": "get_config_field",
-      "params": { "field_name": "template_correctness_standards" },
+      "tool": "config_field",
+      "params": {
+        "field_name": "template_correctness_standards",
+        "operation": "get"
+      },
       "description": "Fetch current template_correctness_standards value",
       "on_error": "warn_and_continue"
     },
@@ -182,18 +215,22 @@
     },
     {
       "type": "mcp_call",
-      "tool": "update_config_field",
+      "tool": "config_field",
       "params": {
         "field_name": "template_correctness_standards",
-        "file_path": "{docs_dir}/standards/template_correctness_standards.md"
+        "file_path": "{docs_dir}/standards/template_correctness_standards.md",
+        "operation": "update"
       },
       "description": "Upload template_correctness_standards to config",
       "requires_approval": true
     },
     {
       "type": "mcp_call",
-      "tool": "get_config_field",
-      "params": { "field_name": "style_correctness_standards" },
+      "tool": "config_field",
+      "params": {
+        "field_name": "style_correctness_standards",
+        "operation": "get"
+      },
       "description": "Fetch current style_correctness_standards value",
       "on_error": "warn_and_continue"
     },
@@ -204,18 +241,22 @@
     },
     {
       "type": "mcp_call",
-      "tool": "update_config_field",
+      "tool": "config_field",
       "params": {
         "field_name": "style_correctness_standards",
-        "file_path": "{docs_dir}/standards/style_correctness_standards.md"
+        "file_path": "{docs_dir}/standards/style_correctness_standards.md",
+        "operation": "update"
       },
       "description": "Upload style_correctness_standards to config",
       "requires_approval": true
     },
     {
       "type": "mcp_call",
-      "tool": "get_config_field",
-      "params": { "field_name": "design_principles" },
+      "tool": "config_field",
+      "params": {
+        "field_name": "design_principles",
+        "operation": "get"
+      },
       "description": "Fetch current design_principles value",
       "on_error": "warn_and_continue"
     },
@@ -226,13 +267,14 @@
     },
     {
       "type": "mcp_call",
-      "tool": "update_config_field",
+      "tool": "config_field",
       "params": {
         "field_name": "design_principles",
-        "file_path": "{docs_dir}/standards/design_principles.md"
+        "file_path": "{docs_dir}/standards/design_principles.md",
+        "operation": "update"
       },
       "description": "Upload design_principles to config",
       "requires_approval": true
     }
   ]
-}
+}

package/smoke-test/SMOKE-TEST.md

@@ -84,13 +84,12 @@ Ask the user for the following before making any tool calls:
    `BAPI_API_KEY` can read (used for the Tier 1 read-only checks). It only needs
    to be readable — comments, attachments, tracked state, and generated
    artifacts are all optional.
-2. **Optional:** a **config field name** (for `get_config_field`).
+2. **Optional:** a **config field name** (for `config_field` with operation `"get"`).
 3. **Optional:** a **pipeline name** (for `get_pipeline_recipe`) — either a
    no-variable pipeline name (e.g. `learn-repository`) or, for a
    variable-declaring pipeline, the variable values it needs (e.g. a
    `ticket_key`).
-4. **Optional:** an **attachment identifier or filename** (for
-   `download_attachment` / `list_attachments`).
+4. **Optional:** an **attachment identifier or filename** (for `attachment` with operation `"download"` or `"list"`).
 5. **Optional:** any **artifact or task IDs** (e.g. a `task_id`, `brainstorm_id`,
    or commit ref) for retriever and CI tools.
 6. **Required decision:** explicit **Tier 3 consent** (Tier 3 spends tokens and
@@ -186,7 +185,7 @@ all local writes:
 
 There are **five heavy-read tools** whose successful oversized output is saved to
 disk before a truncated inline preview is returned: `get_project_standards`,
-`get_tickets`, `get_ticket`, `get_comments`, and `list_attachments`.
+`get_tickets`, `get_ticket`, `get_comments`, and `attachment` (operation: `"list"`).
 
 - When a heavy-read response begins with `[Response truncated. Full response
   saved locally.]` and carries a `Saved to <path>` note, grade it **`PASS`** as
@@ -210,8 +209,7 @@ default plus the read-only ticket checks when a readable ticket was supplied.
 ### All Tier 0 tools run by default
 
 `ping`, `get_my_role`, `get_docs_dir`, `get_project_standards`,
-`list_config_fields`, `list_pipelines`, `get_parse_status`,
-`list_pipeline_runs`.
+`list_pipelines`, `get_parse_status`, `list_pipeline_runs`.
 
 (The "fast-pass" list is the minimum release-blocking subset; the tier
 definition is "always run", so do not exclude the other Tier 0 tools.)
@@ -221,7 +219,7 @@ definition is "always run", so do not exclude the other Tier 0 tools.)
 Run `get_ticket`, `get_comments`, and `get_plan` against the user-supplied
 ticket.
 
-- **`get_comments` is preferred**; **`list_attachments` is the fallback** if
+- **`get_comments` is preferred**; **`attachment` (operation: `"list"`) is the fallback** if
   comments are unavailable for the selected ticket.
 - **`get_plan` must be called with a real boolean `save_locally: false`** (a true
   JSON boolean, not the string `"false"`).
@@ -246,7 +244,7 @@ supplied the relevant input; otherwise record `SKIP_USER_INPUT`.
 - `get_tickets` — a query/filter (a default project query is acceptable).
 - `get_ticket_state`, `get_jira_transitions`, `resolve_target_status` — the
   ticket key.
-- `get_config_field` — the optional config field name.
+- `config_field` (operation: `"get"`) — the optional config field name.
 - `get_pipeline_recipe` — a pipeline name. When no variables are supplied, use a
   no-variable pipeline such as `learn-repository` as the smoke target. A
   variable-declaring pipeline (e.g. a ticket-based pipeline) requires a
@@ -254,7 +252,7 @@ supplied the relevant input; otherwise record `SKIP_USER_INPUT`.
   BAD_REQUEST` with `Missing required variable(s)` is **working as designed**, not
   a server fault — rerun with the required variables or pick a no-variable
   pipeline.
-- `list_attachments` / `download_attachment` — the optional attachment input.
+- `attachment` (operation: `"list"` or `"download"`) — the optional attachment input.
 - The clean-empty artifact retrievers `get_architecture`,
   `get_clarifying_questions`, `get_ticket_critique` — grade a clean empty result
   as `PASS_EXPECTED_404` and valid content as `PASS`.
@@ -489,12 +487,12 @@ and raw protocol noise.** Sensitive values are redacted.
 
 ---
 
-## Full 62-Tool Smoke Tier Matrix
+## Full 58-Tool Smoke Tier Matrix
 
 Every registered MCP tool appears **exactly once** below with its smoke tier.
 Tier 4 (mutating/hazardous) tools are **deferred in v1**.
 
-Coverage reconciliation: **8 + 22 + 1 + 12 + 19 = 62**.
+Coverage reconciliation: **7 + 19 + 1 + 12 + 19 = 58** (BAPI-443 Phase 3a: `list_config_fields` merged into `config_field`; `get_config_field`/`list_attachments`/`download_attachment` merged into `config_field`/`attachment`).
 
 | Tier | Tool | Default action / expected-empty / special handling |
 |---|---|---|
@@ -502,7 +500,6 @@ Coverage reconciliation: **8 + 22 + 1 + 12 + 19 = 62**.
 | Tier 0 | `get_my_role` | Zero-input read; always run. |
 | Tier 0 | `get_docs_dir` | Zero-input read; always run; call first to anchor writes. |
 | Tier 0 | `get_project_standards` | Zero-input heavy read; always run; may save/truncate oversized output under `project-standards/`. |
-| Tier 0 | `list_config_fields` | Zero-input read; always run. |
 | Tier 0 | `list_pipelines` | Zero-input read; always run. |
 | Tier 0 | `get_parse_status` | Zero-input read; always run. |
 | Tier 0 | `list_pipeline_runs` | Zero-input read; always run. |
@@ -523,9 +520,6 @@ Coverage reconciliation: **8 + 22 + 1 + 12 + 19 = 62**.
 | Tier 1 | `get_ticket_state` | Read-only; needs ticket key; v1 read-only reaches the clean "not tracked" 404; full state requires deferred Tier 4 `track_ticket`. |
 | Tier 1 | `get_jira_transitions` | Read-only; needs ticket key. |
 | Tier 1 | `resolve_target_status` | Read-only; needs ticket key. |
-| Tier 1 | `get_config_field` | Read-only; needs a config field name. |
-| Tier 1 | `list_attachments` | Read-only; needs ticket key; comments-fallback tool. |
-| Tier 1 | `download_attachment` | Tier 1 network read; secondary **path-canary** — saves locally only when an attachment input is supplied; omit `file_path` or keep any override under `get_docs_dir`. |
 | Tier 1 | `poll_ci_checks` | `available:true` with empty or populated checks is `PASS`; `SKIP_NOT_APPLICABLE` when disabled with `TOOL_DISABLED`/503, or when `resolve_ci_checks` has not populated CI configuration. |
 | Tier 1 | `get_pipeline_recipe` | Read-only; needs a pipeline name plus required variables for variable-declaring pipelines; use `learn-repository` as the no-variable smoke target. |
 | Tier 2 | `generate_decision_page` | Local-only file-write canary; always run with the verbatim payload above. |
@@ -544,11 +538,11 @@ Coverage reconciliation: **8 + 22 + 1 + 12 + 19 = 62**.
 | Tier 4 | `create_ticket` | Mutating; deferred in v1. |
 | Tier 4 | `add_comment` | Mutating; deferred in v1. |
 | Tier 4 | `update_ticket_description` | Mutating; deferred in v1. |
-| Tier 4 | `upload_attachment` | Mutating; deferred in v1. |
+| Tier 4 | `attachment` | Discriminated-union: `upload`/`download`/`list`; classified Tier 4 (upload is mutating); deferred in v1. |
 | Tier 4 | `track_ticket` | Mutating; deferred in v1. |
 | Tier 4 | `update_ticket_state` | Mutating; deferred in v1. |
 | Tier 4 | `update_jira_status` | Mutating; deferred in v1. |
-| Tier 4 | `update_config_field` | Mutating; deferred in v1. |
+| Tier 4 | `config_field` | Discriminated-union: `get`/`update`/`list`; classified Tier 4 (update is mutating); deferred in v1. |
 | Tier 4 | `create_pull_request` | Mutating; deferred in v1. |
 | Tier 4 | `resolve_ci_checks` | Cache-mutating/orchestration; deferred in v1. |
 | Tier 4 | `run_pipeline` | Orchestration (can dispatch mutating tools); deferred in v1. |