@bridge_gpt/mcp-server 0.2.65 → 0.2.66

package/build/project-root.js

@@ -16,6 +16,7 @@
  * so this stays trivially testable and swappable.
  */
 import path from "path";
+import { renderDiagnosticValue } from "./diagnostic-text.js";
 /**
  * Whether `cwd` looks like a project root: presence of `.git`, checked through the
  * injected `stat` probe. Any probe failure (absent, or an unexpected I/O error) is
@@ -32,3 +33,101 @@ export async function hasProjectRootMarker(cwd, stat) {
         return false;
     }
 }
+/**
+ * Validate an explicit project root: it must exist, be a directory, and contain
+ * `.git` (a directory, or the `.git` FILE a worktree has) or `.bridge/config`.
+ * An absent path is `missing`; anything else that fails — a file, a folder with
+ * neither marker, or an inaccessible path — is `not-a-project`. Raw filesystem
+ * errors never leave this function.
+ *
+ * Deliberately a SIBLING of {@link hasProjectRootMarker}, which stays `.git`-only
+ * because `install` and `--init` use it and the README pins that wording. A
+ * marker proves only that the path looks like a project, not that it is THIS
+ * project; the stderr source line is how a wrong-but-valid root is noticed.
+ */
+export async function validateExplicitProjectRoot(root, stat) {
+    let info;
+    try {
+        info = await stat(root);
+    }
+    catch (error) {
+        const code = error?.code;
+        return { ok: false, reason: code === "ENOENT" || code === "ENOTDIR" ? "missing" : "not-a-project" };
+    }
+    try {
+        if (!info.isDirectory())
+            return { ok: false, reason: "not-a-project" };
+    }
+    catch {
+        return { ok: false, reason: "not-a-project" };
+    }
+    for (const marker of [path.join(root, ".git"), path.join(root, ".bridge", "config")]) {
+        try {
+            await stat(marker);
+            return { ok: true };
+        }
+        catch {
+            /* try the next marker */
+        }
+    }
+    return { ok: false, reason: "not-a-project" };
+}
+/**
+ * Resolve the project root in the fixed order: a VALID `BAPI_PROJECT_ROOT`,
+ * then MCP `roots/list`, then `CLAUDE_PROJECT_DIR`, then `cwd`. Only the explicit
+ * value is validated — it is the one source written into (possibly committed)
+ * config files, so it is the one that can name another machine's path. The later
+ * sources come from the running host and keep today's behavior. An empty or
+ * whitespace `BAPI_PROJECT_ROOT` is treated as unset.
+ *
+ * Pure apart from the injected seams, so `index.ts` (which cannot be imported in
+ * tests) stays a thin wrapper over this.
+ */
+export async function resolveProjectRoot(options) {
+    let rejectedExplicit;
+    const explicit = (options.env.BAPI_PROJECT_ROOT ?? "").trim();
+    if (explicit.length > 0) {
+        const verdict = await validateExplicitProjectRoot(explicit, options.stat);
+        if (verdict.ok)
+            return { root: explicit, source: "BAPI_PROJECT_ROOT" };
+        rejectedExplicit = { path: explicit, reason: verdict.reason };
+    }
+    const withRejection = (root, source) => rejectedExplicit ? { root, source, rejectedExplicit } : { root, source };
+    let fromRoots = null;
+    try {
+        fromRoots = await options.listRoots();
+    }
+    catch {
+        fromRoots = null;
+    }
+    if (fromRoots && fromRoots.length > 0)
+        return withRejection(fromRoots, "roots/list");
+    const claudeDir = (options.env.CLAUDE_PROJECT_DIR ?? "").trim();
+    if (claudeDir.length > 0)
+        return withRejection(claudeDir, "CLAUDE_PROJECT_DIR");
+    return withRejection(options.cwd(), "cwd");
+}
+/** Resolve `BAPI_DOCS_DIR` (default `docs/tmp`) against the chosen project root. */
+export function resolveDocsDirForRoot(root, env) {
+    return path.resolve(root, env.BAPI_DOCS_DIR ?? "docs/tmp");
+}
+/** Human wording for each closed refusal reason. */
+export const PROJECT_ROOT_REFUSAL_TEXT = {
+    missing: "does not exist",
+    "not-a-project": "is not a project folder (no .git or .bridge/config)",
+};
+/**
+ * The one stderr line the server writes when it first resolves the root, e.g.
+ * `Bridge MCP: project root /home/b/app (from roots/list); BAPI_PROJECT_ROOT
+ * /Users/a/app ignored: does not exist`. Paths are rendered on one line with
+ * control characters escaped.
+ */
+export function formatProjectRootSourceLine(resolved) {
+    let line = `Bridge MCP: project root ${renderDiagnosticValue(resolved.root)} (from ${resolved.source})`;
+    if (resolved.rejectedExplicit) {
+        line +=
+            `; BAPI_PROJECT_ROOT ${renderDiagnosticValue(resolved.rejectedExplicit.path)} ignored: ` +
+                PROJECT_ROOT_REFUSAL_TEXT[resolved.rejectedExplicit.reason];
+    }
+    return line;
+}