@bridge_gpt/mcp-server 0.2.65 → 0.2.67

package/build/doctor.js

@@ -24,8 +24,9 @@ import { spawn } from "child_process";
 import os from "os";
 import path from "path";
 import { createDefaultStartTicketsDeps } from "./start-tickets.js";
+import { defaultNpmEntryFileExists, resolveNpmEntry } from "./windows-npm-entry.js";
 import { VERSION, BUILD_COMMIT } from "./version.generated.js";
-import { collectInstallStatusChecks, formatInstallStatusReport, formatInstallStatusFallbackReport, resolveInstallDoctorTarget, } from "./install-doctor.js";
+import { collectInstallStatusChecks, collectProjectIdentityAndRoot, formatInstallStatusReport, formatInstallStatusFallbackReport, formatProjectIdentityAndRootFallbackReport, formatProjectIdentityAndRootReport, resolveInstallDoctorTarget, } from "./install-doctor.js";
 import { codexHome, inspectBridgeCodexProfile } from "./codex-bridge-profile.js";
 import { resolveRoutingHost } from "./model-routing-host.js";
 import { modelRoutingPreferencesPath, readModelRoutingPreference } from "./model-routing-preferences.js";
@@ -35,7 +36,7 @@ import { parseDefaultOnEnvFlag } from "./env-flags.js";
 import { createBridgeApiUrls } from "./bridge-api-urls.js";
 import { probeToolSurface } from "./tool-surface-gating.js";
 import { readStoredBindingState, resolveBapiCredentials } from "./credential-store.js";
-import { listEpicLaunchProfiles } from "./epic-launch-profile.js";
+import { EPIC_LAUNCH_PROFILE_NAME_PATTERN, listEpicLaunchProfiles } from "./epic-launch-profile.js";
 import { CONDUCTOR_DEFAULT_BASE_URL, resolveConductorBridgeApiAccess } from "./conductor/bridge-api-client.js";
 import { validateHttpBaseUrl } from "./base-url.js";
 import { getLiveRepositoryConductors, getParseDispatcherHealth, } from "./epic-shared/bridge-client.js";
@@ -88,6 +89,12 @@ export function getDoctorUsage() {
         "bootstrap-field completeness, integration credentials, and repository-indexing",
         "state. It performs read-only GETs only and never affects the exit code.",
         "",
+        "Next comes an advisory 'Project identity and root' section (BAPI-1318): whether",
+        "each BAPI_PROJECT_ROOT (from the environment and the project MCP configs) names",
+        "a real project folder the server will use, and whether .bridge/config's",
+        "repo_name matches each project MCP config's BAPI_REPO_NAME. It only reads",
+        "files and never affects the exit code.",
+        "",
         "After that come the start-tickets prerequisite checks (for the current OS):",
         "the start-tickets preflight prerequisites plus",
         "uv, the selected agent's command, Bridge API credential resolution, and",
@@ -541,10 +548,25 @@ export function parseLauncherPin(args) {
  * spawn error, or timeout is classified `indeterminate` (never silently "warm").
  * `--no-install` guarantees the probe never downloads/warms the bucket.
  */
-function probeNpxNoInstallDefault(spec) {
+export function probeNpxNoInstallDefault(spec, options = {}) {
+    const doSpawn = options.spawn ?? spawn;
     return new Promise((resolve) => {
+        // WINDOWS (BAPI-1319): a bare `npx` is ENOENT without a shell, so the probe
+        // always read "indeterminate". Map it to npm's `npx-cli.js` through Node;
+        // `--no-install` is kept, so doctor still can never warm the cache. Other
+        // platforms pass through unchanged.
+        const resolved = resolveNpmEntry("npx", ["--no-install", spec, "--version"], {
+            platform: options.platform ?? process.platform,
+            env: options.env ?? process.env,
+            execPath: options.execPath ?? process.execPath,
+            fileExists: options.fileExists ?? defaultNpmEntryFileExists,
+        });
+        if (resolved.status !== "resolved") {
+            resolve({ warmed: false, indeterminate: true });
+            return;
+        }
         try {
-            const child = spawn("npx", ["--no-install", spec, "--version"], {
+            const child = doSpawn(resolved.command, resolved.args, {
                 shell: false,
                 stdio: "ignore",
                 timeout: 30_000,
@@ -1663,11 +1685,17 @@ export function formatOperationalAdvisoryReport(diagnostic) {
  * migrates anything.
  */
 export async function collectEpicLaunchProfileDiagnostic(deps) {
+    const selectedName = deps.env.BAPI_EPIC_LAUNCH_PROFILE?.trim();
+    const selected = selectedName
+        ? EPIC_LAUNCH_PROFILE_NAME_PATTERN.test(selectedName)
+            ? { name: selectedName, status: "missing" }
+            : { name: "<invalid>", status: "invalid" }
+        : undefined;
     const listing = await listEpicLaunchProfiles(deps);
     if (!listing.ok)
-        return { store: "unreadable", message: listing.message };
+        return { store: "unreadable", message: listing.message, ...(selected ? { selected } : {}) };
     if (!listing.present)
-        return { store: "absent" };
+        return { store: "absent", ...(selected ? { selected: { ...selected, status: selected.status === "invalid" ? "invalid" : "missing" } } : {}) };
     const host = deps.hostname();
     const production = validateHttpBaseUrl(CONDUCTOR_DEFAULT_BASE_URL);
     const productionBaseUrl = production.ok ? production.baseUrl : null;
@@ -1711,7 +1739,12 @@ export async function collectEpicLaunchProfileDiagnostic(deps) {
             credentialBinding,
         });
     }
-    return { store: "present", profiles };
+    const selectedProfile = selected?.status !== "invalid" ? profiles.find((p) => p.name === selected?.name) : undefined;
+    return {
+        store: "present",
+        profiles,
+        ...(selected ? { selected: { name: selected.name, status: selected.status === "invalid" ? "invalid" : !selectedProfile ? "missing" : selectedProfile.usable ? "usable" : "unusable" } } : {}),
+    };
 }
 /**
  * BAPI-1272 — a target-selection fact, shown whether or not a profile store
@@ -1725,6 +1758,12 @@ export const EPIC_LAUNCH_DEFAULT_TARGET_LINE = `  [info] drive-epic with no prof
  */
 export function formatEpicLaunchProfileDiagnosticReport(diagnostic) {
     const lines = ["", "Epic launch profiles (read-only)", "", EPIC_LAUNCH_DEFAULT_TARGET_LINE];
+    if (diagnostic.selected) {
+        const { name, status } = diagnostic.selected;
+        lines.push(`  [${status === "usable" ? "ok" : "fail"}] BAPI_EPIC_LAUNCH_PROFILE selects ${name}: ${status}.`);
+        if (status !== "usable")
+            lines.push("         Epic launch will refuse this selection; inspect the local profile and credential binding.");
+    }
     if (diagnostic.store === "absent")
         return lines.join("\n");
     if (diagnostic.store === "unreadable") {
@@ -1772,8 +1811,8 @@ export async function runDoctorCli(argv, overrides = {}) {
     const agent = resolveAgentSpec(parsed.options.agentName) ?? resolveAgentSpec(DEFAULT_AGENT_NAME);
     const collection = await collectDoctorResults(deps, parsed.options.agentName);
     // ---- Section order (BAPI-669, U9c) ----
-    // Broad header → Install status → start-tickets prerequisites → launcher cache →
-    // MCP tool surface. Install status renders FIRST and UNCONDITIONALLY: it answers
+    // Broad header → Install status → Project identity and root (BAPI-1318) →
+    // start-tickets prerequisites → launcher cache → MCP tool surface. Install status renders FIRST and UNCONDITIONALLY: it answers
     // "is this project actually set up?", which is what someone running `doctor` after
     // an install is asking. There is deliberately no "invocation context" flag to gate
     // the ordering on — the report has a single caller and carries no context, so
@@ -1809,6 +1848,27 @@ export async function runDoctorCli(argv, overrides = {}) {
             log(formatInstallStatusFallbackReport());
         }
     }
+    // Advisory "Project identity and root" section (BAPI-1318). Read-only: it
+    // validates each explicit BAPI_PROJECT_ROOT the way the server does and
+    // compares .bridge/config's repo_name with each project MCP config. Placed
+    // right after Install status without reordering any existing section; a throw
+    // renders a sanitized line, and the exit code is never affected.
+    if (overrides.projectIdentity !== false) {
+        try {
+            const injectedFs = deps;
+            const identityDeps = {
+                env: overrides.projectIdentity?.env ?? deps.env,
+                cwd: overrides.projectIdentity?.cwd ?? deps.cwd,
+                readFile: overrides.projectIdentity?.readFile ?? injectedFs.readFile ?? ((p) => readFile(p, "utf-8")),
+                statDir: overrides.projectIdentity?.statDir ?? ((p) => stat(p)),
+            };
+            const collect = overrides.projectIdentity?.collect ?? collectProjectIdentityAndRoot;
+            log(formatProjectIdentityAndRootReport(await collect(identityDeps)));
+        }
+        catch {
+            log(formatProjectIdentityAndRootFallbackReport());
+        }
+    }
     if (parsed.options.agentName === "codex") {
         const readCodexFile = deps.readFile ?? ((p) => readFile(p, "utf-8"));
         const profile = await inspectBridgeCodexProfile(codexHome(deps.env), readCodexFile);
@@ -1883,7 +1943,7 @@ export async function runDoctorCli(argv, overrides = {}) {
         const launcherDeps = {
             cwd: overrides.launcherProbe?.cwd ?? deps.cwd,
             readFile: overrides.launcherProbe?.readFile ?? ((p) => readFile(p, "utf-8")),
-            probeNpxNoInstall: overrides.launcherProbe?.probeNpxNoInstall ?? probeNpxNoInstallDefault,
+            probeNpxNoInstall: overrides.launcherProbe?.probeNpxNoInstall ?? ((spec) => probeNpxNoInstallDefault(spec)),
         };
         const launcherInspections = await inspectLauncherCache(launcherDeps);
         log(formatLauncherCacheReport(launcherInspections));

package/build/drive-epic-key.js

@@ -4,9 +4,10 @@
  * It lives in its own module so `plan-discovery.ts` can import it without
  * importing `drive-epic.ts`, which imports discovery — a cycle.
  *
- * This is deliberately the narrower CLI grammar. Bundle finalization accepts some
- * underscore or digit project prefixes, but `drive-epic` already rejects them, so
- * metadata naming such a key can never be discovered through this CLI.
+ * BAPI-1348: this is the canonical ticket-key grammar from `ticket-key-utils.ts`,
+ * so a local-mode epic whose prefix carries digits or underscores (for example
+ * `STOREFRONT2-1` or `STOREFRONT_CLIENTA-1`) is accepted.
  */
-/** Jira epic key shape. Matches the epic-key positional rule exactly. */
-export const EPIC_KEY_PATTERN = /^[A-Z]+-[0-9]+$/;
+import { TICKET_KEY_PATTERN } from "./ticket-key-utils.js";
+/** Epic key shape. The canonical ticket-key pattern, shared with every other reader. */
+export const EPIC_KEY_PATTERN = TICKET_KEY_PATTERN;