@bridge_gpt/mcp-server 0.1.17 → 0.2.1

package/build/credential-store.js

@@ -0,0 +1,232 @@
+/**
+ * Resolver for Tier-1 bridge-api credentials.
+ *
+ * Resolution order is env-first, then a user-scoped credentials file:
+ *   1. `BAPI_API_KEY` in the process environment (overrides everything).
+ *   2. `$XDG_CONFIG_HOME/bridge/credentials.json` (or `~/.config/bridge/credentials.json`).
+ *   3. `~/.bridge/credentials.json` (only when the primary path is absent).
+ *
+ * The file is keyed by a logical target `bapi:<repoName>`. This module NEVER
+ * creates or initializes credential files, and NEVER places secret values in
+ * thrown errors, returned error strings, or stderr warnings.
+ */
+import path from "path";
+// ---------------------------------------------------------------------------
+// Path resolution
+// ---------------------------------------------------------------------------
+/** Primary store path: honors a non-empty `XDG_CONFIG_HOME`, else `~/.config`. */
+export function getPrimaryCredentialStorePath(deps) {
+    const xdg = deps.env.XDG_CONFIG_HOME;
+    if (xdg && xdg.trim().length > 0) {
+        return path.join(xdg, "bridge", "credentials.json");
+    }
+    return path.join(deps.homedir(), ".config", "bridge", "credentials.json");
+}
+/** Fallback store path: `~/.bridge/credentials.json`. */
+export function getFallbackCredentialStorePath(deps) {
+    return path.join(deps.homedir(), ".bridge", "credentials.json");
+}
+/**
+ * Choose which store path to read. The primary path is preferred and used
+ * whenever it exists. The fallback is only consulted when the primary is
+ * *absent* (`ENOENT`); a primary that exists but is unreadable does NOT fall
+ * back — the caller surfaces the read error instead.
+ */
+export async function resolveCredentialStorePath(deps) {
+    const primaryPath = getPrimaryCredentialStorePath(deps);
+    const fallbackPath = getFallbackCredentialStorePath(deps);
+    try {
+        await deps.stat(primaryPath);
+        return { found: true, path: primaryPath, isPrimary: true };
+    }
+    catch (err) {
+        const code = err && typeof err === "object" ? err.code : undefined;
+        if (code !== "ENOENT") {
+            // The primary exists but is not statable cleanly — do NOT fall back; let
+            // the subsequent read surface the real error against the primary path.
+            return { found: true, path: primaryPath, isPrimary: true };
+        }
+    }
+    try {
+        await deps.stat(fallbackPath);
+        return { found: true, path: fallbackPath, isPrimary: false };
+    }
+    catch {
+        return { found: false, primaryPath, fallbackPath };
+    }
+}
+// ---------------------------------------------------------------------------
+// Permission warning
+// ---------------------------------------------------------------------------
+/**
+ * On POSIX platforms, warn (but continue) when the credentials file is
+ * group/world accessible. Skipped entirely on Windows. The warning mentions
+ * `0600` but never any secret value.
+ */
+export function warnIfInsecureCredentialFileMode(statResult, platform, filePath, stderr) {
+    if (platform === "win32")
+        return;
+    if ((statResult.mode & 0o077) !== 0) {
+        const write = stderr ?? ((m) => process.stderr.write(`${m}\n`));
+        write(`Warning: credentials file ${filePath} is group/world-accessible; ` +
+            `it should be mode 0600 (run: chmod 600 ${filePath}).`);
+    }
+}
+// ---------------------------------------------------------------------------
+// JSON parsing / validation
+// ---------------------------------------------------------------------------
+/**
+ * Parse and structurally validate the credentials JSON. The root must be an
+ * object whose values are objects of secret-name -> string. Secret *values* are
+ * never echoed into error messages (only logical keys and secret names appear).
+ */
+export function parseCredentialStoreJson(text) {
+    let data;
+    try {
+        data = JSON.parse(text);
+    }
+    catch {
+        return { ok: false, error: "credentials file is not valid JSON" };
+    }
+    if (data === null || typeof data !== "object" || Array.isArray(data)) {
+        return { ok: false, error: "credentials file must be a JSON object" };
+    }
+    for (const [key, value] of Object.entries(data)) {
+        if (value === null || typeof value !== "object" || Array.isArray(value)) {
+            return {
+                ok: false,
+                error: `credentials entry "${key}" must be an object of secret names to strings`,
+            };
+        }
+        for (const [secretName, secretValue] of Object.entries(value)) {
+            if (typeof secretValue !== "string") {
+                return {
+                    ok: false,
+                    error: `credentials entry "${key}" has a non-string value for "${secretName}"`,
+                };
+            }
+        }
+    }
+    return { ok: true, value: data };
+}
+// ---------------------------------------------------------------------------
+// Resolution
+// ---------------------------------------------------------------------------
+/** Collect trimmed, non-empty env values for the requested keys. */
+function collectEnvValues(env, requiredKeys) {
+    const values = {};
+    const missing = [];
+    for (const key of requiredKeys) {
+        const raw = (env[key] ?? "").trim();
+        if (raw.length > 0) {
+            values[key] = raw;
+        }
+        else {
+            missing.push(key);
+        }
+    }
+    return { values, missing };
+}
+/**
+ * Resolve a logical credential bundle env-first, then from the user-scoped
+ * credentials file. `bundleKey` is the logical store key (e.g. `bapi:<repo>` or
+ * `sfcc:<sandbox-id>`); `requiredKeys` is the exact set of secret names that
+ * must all resolve.
+ *
+ * Behavior:
+ *   - Environment is consulted first. If EVERY required key is present in the
+ *     environment (trimmed, non-empty), the store is NEVER read and `source` is
+ *     `"env"`.
+ *   - Only when one or more keys are missing from the environment is the store
+ *     read (exactly once). Env values override store values key-by-key.
+ *   - The result fails unless every required key resolves.
+ *
+ * Error messages reference only the logical bundle key, the missing secret
+ * names, and the expected store path — never a secret value. Never creates
+ * files.
+ */
+export async function resolveCredentialBundle(bundleKey, requiredKeys, deps) {
+    const primaryPath = getPrimaryCredentialStorePath(deps);
+    const fromEnv = collectEnvValues(deps.env, requiredKeys);
+    if (fromEnv.missing.length === 0) {
+        return { ok: true, values: fromEnv.values, source: "env" };
+    }
+    const resolution = await resolveCredentialStorePath(deps);
+    if (!resolution.found) {
+        return {
+            ok: false,
+            kind: "not-found",
+            error: `No credentials found for "${bundleKey}". Set ${fromEnv.missing.join(", ")} in the ` +
+                `environment, or add them under "${bundleKey}" in ${primaryPath}.`,
+        };
+    }
+    // Permission warning is best-effort and must not block resolution.
+    try {
+        const statResult = await deps.stat(resolution.path);
+        warnIfInsecureCredentialFileMode(statResult, deps.platform, resolution.path, deps.stderr);
+    }
+    catch {
+        /* stat failure here is non-fatal; the read below surfaces real errors */
+    }
+    let raw;
+    try {
+        raw = await deps.readFile(resolution.path);
+    }
+    catch {
+        return {
+            ok: false,
+            kind: "read-error",
+            error: `Unable to read credentials file at ${resolution.path}.`,
+        };
+    }
+    const parsed = parseCredentialStoreJson(raw);
+    if (!parsed.ok) {
+        return {
+            ok: false,
+            kind: "parse-error",
+            error: `Invalid credentials file at ${resolution.path}: ${parsed.error}.`,
+        };
+    }
+    const entry = parsed.value[bundleKey] ?? {};
+    const values = { ...fromEnv.values };
+    const stillMissing = [];
+    for (const key of fromEnv.missing) {
+        // Trim store values exactly like the env path (collectEnvValues), so a
+        // credential pasted into credentials.json with a trailing newline/space is
+        // never sent verbatim in an auth header. Assign the TRIMMED value.
+        const storeValue = typeof entry[key] === "string" ? entry[key].trim() : "";
+        if (storeValue.length > 0) {
+            values[key] = storeValue;
+        }
+        else {
+            stillMissing.push(key);
+        }
+    }
+    if (stillMissing.length > 0) {
+        return {
+            ok: false,
+            kind: "missing-key",
+            error: `No usable value(s) for ${stillMissing.join(", ")} in "${bundleKey}". Add them under ` +
+                `"${bundleKey}" in ${primaryPath}, or set them in the environment.`,
+        };
+    }
+    return { ok: true, values, source: "file" };
+}
+/**
+ * Resolve the `BAPI_API_KEY` for `bapi:<repoName>`. Delegates to the generic
+ * {@link resolveCredentialBundle} with bundle key `bapi:${repoName}` and the
+ * single required key `BAPI_API_KEY`, preserving the historical public return
+ * shape (`{ apiKey, source }`) relied on by `mcp-invoke` and `index.ts`. Env
+ * wins (trimmed, non-empty); otherwise the credentials file is read. Never
+ * leaks secret values; never creates files.
+ */
+export async function resolveBapiCredentials(repoName, deps) {
+    const result = await resolveCredentialBundle(`bapi:${repoName}`, ["BAPI_API_KEY"], deps);
+    if (!result.ok) {
+        return { ok: false, kind: result.kind, error: result.error };
+    }
+    return {
+        ok: true,
+        credentials: { apiKey: result.values.BAPI_API_KEY, source: result.source },
+    };
+}

package/build/decision-page-schema.js

@@ -19,8 +19,8 @@ export const ActionableItemSchema = z
     question: z.string().min(1),
     original_question: z
         .string()
-        .min(1)
-        .describe("The clarifying question or critique point as originally raised; soft cap ~30 words."),
+        .optional()
+        .describe("Optional display-only field: the clarifying question or critique point as originally raised; soft cap ~30 words. Omit it (or pass an empty string) for non-review callers — the renderer omits the section when it is absent or blank."),
     why_it_matters: z
         .string()
         .min(1)
@@ -31,12 +31,12 @@ export const ActionableItemSchema = z
         .describe("Why the recommended branch is the best choice; soft cap ~60 words."),
     codebase_evidence: z
         .string()
-        .min(1)
-        .describe("Combined Assessment paragraph and Codebase Evidence bullet list. Rendered as escaped plain text inside a closed-by-default <details> block."),
+        .optional()
+        .describe("Optional display-only field: combined Assessment paragraph and Codebase Evidence bullet list. Rendered as escaped plain text inside a closed-by-default <details> block, which is omitted when this field is absent or blank."),
     source: z
         .string()
-        .min(1)
-        .describe("Source reference from the combined review-and-resolution doc, e.g. 'Clarifying Q3 (prior round, weak concurrence)'"),
+        .optional()
+        .describe("Optional source reference from the combined review-and-resolution doc, e.g. 'Clarifying Q3 (prior round, weak concurrence)'. When absent the rendered card emits data-source=\"\"."),
     recommendation_index: z
         .number()
         .int()
@@ -69,12 +69,45 @@ export const ActionableItemSchema = z
         });
     }
 });
+// Optional presentation labels that override the page's review-flavored copy.
+// Every field is optional and may be an explicitly empty string (no .min(1)) so
+// a caller can intentionally render an empty heading/title segment. The renderer
+// merges these with raw default constants using nullish coalescing, so an
+// explicit "" is preserved rather than falling back to a default.
+export const DecisionPageLabelsSchema = z.object({
+    title: z
+        .string()
+        .optional()
+        .describe('Overrides the page <title>/<h1> lead text (default "Review Decisions").'),
+    intro: z
+        .string()
+        .optional()
+        .describe("Overrides the actionable-page intro copy shown when there are decisions."),
+    section_heading: z
+        .string()
+        .optional()
+        .describe('Overrides the decision cards <h2> (default "Review Decisions").'),
+    improvements_heading: z
+        .string()
+        .optional()
+        .describe('Overrides the confirmed-improvements <h2> (default "Confirmed Improvements").'),
+});
 // Raw input shape for the `generate_decision_page` tool registration.
 // MCP's registerTool expects a shape object (which the SDK wraps in z.object),
 // not a pre-built z.object. Exporting the shape lets index.ts and the schema
 // share a single source of truth for the input contract.
 export const DecisionPageInputShape = {
     ticket_key: z.string().describe("Jira ticket key, e.g. BAPI-123"),
+    output_subdir: z
+        .string()
+        .optional()
+        .default("review")
+        .describe('Optional docs-relative subdirectory to write the page under (default "review"). Validated strictly: no absolute paths, backslashes, ".." segments, null bytes, or encoded path tokens.'),
+    output_filename: z
+        .string()
+        .optional()
+        .describe('Optional output filename (default "${ticket_key}-decisions.html"). Must end with .html and contain no path separators; the .html suffix is required and never auto-appended.'),
+    labels: DecisionPageLabelsSchema.optional().describe("Optional presentation-label overrides (title, intro, section_heading, improvements_heading). Presentation-only; does not change data-testid hooks or the submitted JSON shape."),
     actionable_items: z
         .array(ActionableItemSchema)
         .optional()

package/build/decision-page-template.js

@@ -31,6 +31,30 @@ const COPY_SUCCESS_LABEL = "Copied!";
 const COPY_FALLBACK_PROMPT_LABEL = "Auto-copy unavailable. Press Ctrl+C / Cmd+C to copy.";
 const PAGE_INTRO_ASK_COPY = "Have a question about an item? Choose 'Ask about this' for that card; we'll discuss before the changes are made.";
 const PAGE_INTRO_NO_DECISIONS = "All suggestions were confirmed as improvements. No decisions are needed from you.";
+// Raw (unescaped) default label constants. They reproduce the legacy review copy
+// exactly so omitting `labels` yields byte-identical output. Escaping happens at
+// each render site, so these must stay raw to avoid double-escaping.
+const DEFAULT_LABEL_TITLE = "Review Decisions";
+const DEFAULT_LABEL_INTRO = PAGE_INTRO_ASK_COPY;
+const DEFAULT_LABEL_SECTION_HEADING = "Review Decisions";
+const DEFAULT_LABEL_IMPROVEMENTS_HEADING = "Confirmed Improvements";
+/**
+ * Merge caller-supplied labels with the raw default constants using nullish
+ * coalescing (`??`), NOT logical OR — an explicit empty-string label is an
+ * intentional override and must be preserved rather than replaced by a default.
+ */
+function resolveDecisionPageLabels(labels) {
+    return {
+        title: labels?.title ?? DEFAULT_LABEL_TITLE,
+        intro: labels?.intro ?? DEFAULT_LABEL_INTRO,
+        section_heading: labels?.section_heading ?? DEFAULT_LABEL_SECTION_HEADING,
+        improvements_heading: labels?.improvements_heading ?? DEFAULT_LABEL_IMPROVEMENTS_HEADING,
+    };
+}
+/** True only when a string is present and not whitespace-only. */
+function hasDisplayText(value) {
+    return typeof value === "string" && value.trim().length > 0;
+}
 // ---------------------------------------------------------------------------
 // Template
 // ---------------------------------------------------------------------------
@@ -38,17 +62,21 @@ const DEFAULT_ASSETS = { faviconBase64: "", logoBase64: "", fontsRelPath: "" };
 export function generateDecisionPageHtml(data, assets = DEFAULT_ASSETS) {
     const { ticket_key, actionable_items, clear_improvements } = data;
     const hasDecisions = actionable_items.length > 0;
+    const effectiveLabels = resolveDecisionPageLabels(data.labels);
     const faviconLink = assets.faviconBase64
         ? `<link rel="icon" type="image/png" sizes="32x32" href="data:image/png;base64,${assets.faviconBase64}">`
         : "";
     const fontFaces = assets.fontsRelPath ? renderFontFaces(assets.fontsRelPath) : "";
-    const pageIntro = hasDecisions ? PAGE_INTRO_ASK_COPY : PAGE_INTRO_NO_DECISIONS;
+    // The has-decisions intro is an overridable label; the no-decisions copy is a
+    // fixed constant because the MCP handler returns before rendering in that path
+    // (this branch is only reachable from direct renderer callers/tests).
+    const pageIntro = hasDecisions ? effectiveLabels.intro : PAGE_INTRO_NO_DECISIONS;
     return `<!DOCTYPE html>
 <html lang="en">
 <head>
   <meta charset="UTF-8">
   <meta name="viewport" content="width=device-width, initial-scale=1.0">
-  <title>Review Decisions: ${escapeHtml(ticket_key)}</title>
+  <title>${escapeHtml(effectiveLabels.title)}: ${escapeHtml(ticket_key)}</title>
   ${faviconLink}
   <style>
 ${fontFaces}
@@ -407,12 +435,12 @@ ${fontFaces}
 ${renderHeader(assets)}
   <main class="main-content">
     <div class="container">
-      <h1>Review Decisions: ${escapeHtml(ticket_key)}</h1>
+      <h1>${escapeHtml(effectiveLabels.title)}: ${escapeHtml(ticket_key)}</h1>
       <p class="page-intro">${escapeHtml(pageIntro)}</p>
 
-${hasDecisions ? renderForm(data) : renderNoDecisions()}
+${hasDecisions ? renderForm(data, effectiveLabels) : renderNoDecisions()}
 
-${renderImprovements(clear_improvements)}
+${renderImprovements(clear_improvements, effectiveLabels)}
 
     </div>
   </main>
@@ -466,13 +494,13 @@ function renderNoDecisions() {
         <p>No decisions needed. All suggestions were confirmed as improvements.</p>
       </div>`;
 }
-function renderForm(data) {
+function renderForm(data, labels) {
     const { actionable_items } = data;
     let html = `      <div id="form-container">
         <form id="decision-form">`;
     if (actionable_items.length > 0) {
         html += `
-          <h2>Review Decisions</h2>`;
+          <h2>${escapeHtml(labels.section_heading)}</h2>`;
         for (const item of actionable_items) {
             html += renderDecisionItem(item);
         }
@@ -547,13 +575,25 @@ function renderDecisionItem(item) {
               </div>`;
     // Encode option labels for client-side JS to resolve chosen_label
     const labelsJson = escapeHtml(JSON.stringify(item.options));
-    return `
-          <section class="card" data-item-id="${id}" data-item-type="decision" data-source="${escapeHtml(item.source)}" data-labels="${labelsJson}" data-testid="decision-card" aria-labelledby="${titleId}">
-            <h3 class="card-title" id="${titleId}" data-testid="decision-card-title">${escapeHtml(item.question)}</h3>
+    // Optional review-flavored display sections: omit entirely when the field is
+    // missing, empty, or whitespace-only so non-review callers can skip them.
+    const originalQuestionSection = hasDisplayText(item.original_question)
+        ? `
             <section class="card-section" data-testid="original-question-section" aria-labelledby="${originalQuestionLabelId}">
               <div class="card-section-label" id="${originalQuestionLabelId}">Original question</div>
               <div class="card-section-body" data-testid="original-question-body">${escapeHtml(item.original_question)}</div>
-            </section>
+            </section>`
+        : "";
+    const codebaseEvidenceSection = hasDisplayText(item.codebase_evidence)
+        ? `
+            <details class="codebase-evidence" data-testid="codebase-evidence">
+              <summary>Codebase evidence</summary>
+              <div class="codebase-evidence-body" data-testid="codebase-evidence-body">${escapeHtml(item.codebase_evidence)}</div>
+            </details>`
+        : "";
+    return `
+          <section class="card" data-item-id="${id}" data-item-type="decision" data-source="${escapeHtml(item.source ?? "")}" data-labels="${labelsJson}" data-testid="decision-card" aria-labelledby="${titleId}">
+            <h3 class="card-title" id="${titleId}" data-testid="decision-card-title">${escapeHtml(item.question)}</h3>${originalQuestionSection}
             <section class="card-section" data-testid="why-it-matters-section" aria-labelledby="${whyItMattersLabelId}">
               <div class="card-section-label" id="${whyItMattersLabelId}">Why it matters</div>
               <div class="card-section-body" data-testid="why-it-matters-body">${escapeHtml(item.why_it_matters)}</div>
@@ -568,14 +608,10 @@ function renderDecisionItem(item) {
             <div class="comment-area">
               <label for="${textareaId}">Comment</label>
               <textarea id="${textareaId}" name="${textareaId}" placeholder="Required for None of these selections..." data-testid="decision-comment"></textarea>
-            </div>
-            <details class="codebase-evidence" data-testid="codebase-evidence">
-              <summary>Codebase evidence</summary>
-              <div class="codebase-evidence-body" data-testid="codebase-evidence-body">${escapeHtml(item.codebase_evidence)}</div>
-            </details>
+            </div>${codebaseEvidenceSection}
           </section>`;
 }
-function renderImprovements(improvements) {
+function renderImprovements(improvements, labels) {
     if (improvements.length === 0)
         return "";
     let items = "";
@@ -593,7 +629,7 @@ function renderImprovements(improvements) {
           </li>`;
     }
     return `      <section data-testid="confirmed-improvements">
-      <h2>Confirmed Improvements</h2>
+      <h2>${escapeHtml(labels.improvements_heading)}</h2>
       <p>These improvements have been confirmed and will be applied. No action needed from you.</p>
       <ul class="improvements-list">${items}
       </ul>

package/build/doctor.js

@@ -14,6 +14,8 @@
  * spawning, no MCP server startup. It only ever runs read-only PATH probes
  * (`which`/`where`, `bash --version`, `git rev-parse`) through the injected deps.
  */
+import { readFile, stat } from "fs/promises";
+import os from "os";
 import { createDefaultStartTicketsDeps } from "./start-tickets.js";
 import { DEFAULT_AGENT_NAME, resolveAgentSpec, isAgentName, formatValidAgentNames, } from "./agent-registry.js";
 import { getDoctorPrereqDescriptors, probePrerequisite, } from "./start-tickets-prereqs.js";
@@ -32,7 +34,8 @@ export function getDoctorUsage() {
         "  -h, --help                    Show this help",
         "",
         "Checks (for the current OS): the start-tickets preflight prerequisites plus",
-        "uv plus the selected agent's command.",
+        "uv, the selected agent's command, Bridge API credential resolution, and",
+        "worktree MCP registration reachability.",
         "",
         "Exit code: 0 when all required prerequisites are present, non-zero otherwise.",
     ].join("\n");
@@ -100,9 +103,22 @@ export async function collectDoctorResults(deps, agentName) {
     if (!descriptorsResult.ok) {
         return { ok: false, unsupported: true, error: descriptorsResult.error };
     }
+    // Augment with read-only filesystem deps so the doctor-only credential and
+    // worktree-MCP-registration probes can run. These are intentionally NOT part
+    // of StartTicketsDeps (live preflight never reads files); doctor injects them
+    // here, and they remain strictly read-only (readFile/stat/homedir only). If a
+    // caller (e.g. a unit test) already supplied them on `deps`, honor those so
+    // the probes are deterministic.
+    const injected = deps;
+    const probeDeps = {
+        ...deps,
+        readFile: injected.readFile ?? ((p) => readFile(p, "utf-8")),
+        stat: injected.stat ?? ((p) => stat(p)),
+        homedir: injected.homedir ?? os.homedir,
+    };
     const results = [];
     for (const descriptor of descriptorsResult.descriptors) {
-        results.push(await probePrerequisite(deps, descriptor));
+        results.push(await probePrerequisite(probeDeps, descriptor));
     }
     return { ok: true, results };
 }

package/build/git-ignore-utils.js

@@ -0,0 +1,63 @@
+/**
+ * Local-only ignore helpers.
+ *
+ * Two idempotent, exact-line appenders:
+ *   - `ensureGitignored` writes the tracked `<cwd>/.gitignore` (shared with the
+ *     existing `--init` scaffolding behavior).
+ *   - `ensureGitInfoExcluded` writes the UNtracked `<worktreeRoot>/.git/info/exclude`,
+ *     used for Tier-3 file credentials that must never enter version control and
+ *     must never touch the tracked `.gitignore`.
+ *
+ * Both are dependency-injected so they are unit-testable without real I/O, and
+ * neither ever includes credential file contents in an error.
+ */
+import path from "path";
+/** True when `content` already contains `entry` as its own exact (trimmed) line. */
+function hasExactLine(content, entry) {
+    return content.split("\n").some((line) => line.trim() === entry);
+}
+/** Append `entry` to `content`, inserting a separating newline only if needed. */
+function appendLine(content, entry) {
+    const separator = content.length > 0 && !content.endsWith("\n") ? "\n" : "";
+    return content + separator + entry + "\n";
+}
+/**
+ * Idempotently ensure `filePath` is listed in `<cwd>/.gitignore`. An absolute
+ * `filePath` is made relative to `cwd`; an already-relative path is used as-is.
+ * No-op when the exact line already exists.
+ */
+export async function ensureGitignored(cwd, filePath, deps) {
+    const gitignorePath = path.join(cwd, ".gitignore");
+    const entry = path.isAbsolute(filePath) ? path.relative(cwd, filePath) : filePath;
+    let content = "";
+    try {
+        content = await deps.readFile(gitignorePath);
+    }
+    catch {
+        /* .gitignore doesn't exist yet */
+    }
+    if (hasExactLine(content, entry))
+        return;
+    await deps.writeFile(gitignorePath, appendLine(content, entry));
+}
+/**
+ * Idempotently ensure `relativePath` is listed in the worktree's local-only
+ * `<worktreeRoot>/.git/info/exclude`. Creates `<worktreeRoot>/.git/info` if it
+ * does not exist. Matching is exact-line (so `dw.json` is not treated as present
+ * just because `dw.json.bak` exists). Never edits the tracked `.gitignore`.
+ */
+export async function ensureGitInfoExcluded(worktreeRoot, relativePath, deps) {
+    const infoDir = path.join(worktreeRoot, ".git", "info");
+    const excludePath = path.join(infoDir, "exclude");
+    let content = "";
+    try {
+        content = await deps.readFile(excludePath);
+    }
+    catch {
+        /* exclude file (or .git/info) doesn't exist yet */
+    }
+    if (hasExactLine(content, relativePath))
+        return;
+    await deps.mkdir(infoDir, { recursive: true });
+    await deps.writeFile(excludePath, appendLine(content, relativePath));
+}