@edspencer/paddock 0.61.1 → 0.63.0

package/packages/server/dist/claude-home.js

@@ -15,7 +15,8 @@
  * READ-ONLY source. This module owns everything that follows from that:
  *
  *  - **{@link ensureClaudeHome}** — create the home, bridge the user-level config
- *    that lives in `~/.claude`, and report anything an operator needs to know.
+ *    that lives in `~/.claude`, put `claude.credentials` into the environment
+ *    Claude Code runs in, and report anything an operator needs to know.
  *  - **{@link mirrorLegacyTranscriptFolders}** — keep adoption (#588) able to see
  *    the user's own CLI history, which by definition is in `~/.claude`.
  *
@@ -30,39 +31,74 @@
  * what a user wants; it is not paddock relocating their data behind their back.
  * The thing #620 exists to stop — `ensureProjectChats` copying a user's
  * transcripts out and then `fs.rm`-ing the originals, unprompted, inside a bare
- * `catch {}` — is gone, gated on `PaddockConfig.ownsClaudeHome`.
+ * `catch {}` — is gone: paddock's home is never the user's, so the directory it
+ * migrates can only be its own (#691, `resolveClaudeHome`).
+ *
+ * The one thing that DOES write under `~/.claude` is `claude.transcripts: host`,
+ * which creates the encoded folder its symlink points at. That is the user
+ * asking for their transcripts to be shared, by name, in a config file — see
+ * `ensureProjectChats`.
  */
 import { promises as fs } from "node:fs";
+import { createHash } from "node:crypto";
 import path from "node:path";
 import { encodePathForCli } from "@herdctl/core";
+import { applyCredentialsMode, SECURE_STORAGE_DIR_VAR } from "./claude-credentials.js";
+import { INSTRUCTION_ENTRIES } from "./claude-instructions.js";
+import { planHostSettings, GENERATED_MARKER_FILE, SETTINGS_ENTRY, } from "./claude-settings.js";
+/**
+ * The user's file-based Claude Code login, bridged in only under
+ * `claude.credentials: host` (#691).
+ *
+ * This is what the lever means on Linux and in the Docker image, where `claude
+ * login` writes a file rather than using a keychain: on darwin `credentials: host`
+ * is the `CLAUDE_SECURESTORAGE_CONFIG_DIR` empty-string trick, here it is this
+ * symlink, and one config key has to mean the same thing on both.
+ *
+ * Symlinked rather than copied — a copy would duplicate a live secret onto disk
+ * AND diverge the moment Claude Code refreshed the token.
+ *
+ * It was bridged unconditionally before the lever existed, which is why
+ * `credentials` defaulting to `host` keeps every existing install working
+ * unchanged; `own` is the new behaviour, and it has to also REMOVE a link a
+ * previous boot planted, or "isolated" would be a claim that only holds for
+ * instances that were never anything else.
+ */
+export const CREDENTIAL_ENTRY = ".credentials.json";
 /**
- * Entries of a Claude home that are USER-LEVEL configuration rather than
- * per-instance state, symlinked from the user's home into paddock's own when
- * paddock's does not already have its own.
- *
- * Without this, relocating the home silently changes agent behaviour: the user's
- * `~/.claude/CLAUDE.md` is auto-loaded into every session, `settings.json`
- * carries permissions and hooks, and `agents/` `commands/` `plugins/` are
- * capability. Dropping them is not "a clean home", it is a behaviour regression
- * nobody asked for.
- *
- * `.credentials.json` is the one that matters most: on Linux (and in the Docker
- * image) it is where `claude login` puts the OAuth token, so without the bridge
- * every file-authenticated install would come back logged out. Symlinked rather
- * than copied — a copy would duplicate a live secret onto disk AND diverge the
- * moment Claude Code refreshed the token.
- *
- * Deliberately NOT bridged: `projects/` (that's the whole point), and everything
- * that is per-instance runtime state — `todos/`, `shell-snapshots/`, `statsig/`,
- * `sessions/`. Those are exactly what should now be per-instance.
+ * Entries of the user's Claude home that paddock can bridge into its own — the
+ * complete list, for callers that want to reason about the surface rather than
+ * about one lever.
+ *
+ * There is no longer an "always bridged" set: since #691 step 4, every entry here
+ * belongs to exactly one key of the `claude:` block, and the mapping is the whole
+ * point of the block.
+ *
+ * | Entry | Lever |
+ * |---|---|
+ * | `CLAUDE.md`, `agents/`, `commands/`, `plugins/` | `claude.instructions` |
+ * | `settings.json` (its `hooks` key) | `claude.hooks` |
+ * | `.credentials.json` | `claude.credentials` |
+ *
+ * `settings.json` appears under `hooks` because that is the half of it a lever
+ * governs; its OTHER keys — permissions, model, statusline, `enabledPlugins` —
+ * are inherited in every mode, which is why `hooks: own` writes a filtered file
+ * rather than declining to bridge one. See `claude-settings.ts`.
+ *
+ * Deliberately NOT bridged in any mode: `projects/` (that's the whole point), and
+ * everything that is per-instance runtime state — `todos/`, `shell-snapshots/`,
+ * `statsig/`, `sessions/`. Those are exactly what should be per-instance.
+ *
+ * The fifth lever, `claude.mcpServers`, has NO row here on purpose: MCP servers
+ * are declared in `~/.claude.json`, which is a sibling of the home rather than an
+ * entry inside it, so it is not bridgeable at all — paddock reads it and passes
+ * the servers programmatically (`claude-mcp.ts`). That asymmetry is the reason
+ * MCP inheritance broke separately from everything in this table.
  */
-export const BRIDGED_ENTRIES = [
-    ".credentials.json",
-    "settings.json",
-    "CLAUDE.md",
-    "agents",
-    "commands",
-    "plugins",
+export const BRIDGEABLE_ENTRIES = [
+    SETTINGS_ENTRY,
+    ...INSTRUCTION_ENTRIES,
+    CREDENTIAL_ENTRY,
 ];
 /** Env vars that authenticate Claude Code without any on-disk credential. */
 const TOKEN_ENV_VARS = ["CLAUDE_CODE_OAUTH_TOKEN", "ANTHROPIC_API_KEY", "ANTHROPIC_AUTH_TOKEN"];
@@ -74,9 +110,13 @@ const TOKEN_ENV_VARS = ["CLAUDE_CODE_OAUTH_TOKEN", "ANTHROPIC_API_KEY", "ANTHROP
  * home can already see. What we are looking for is the login made before paddock
  * was involved.
  *
- * More than one candidate because the name is Claude Code's private detail, not
- * a contract — see {@link probeMacosKeychainLogin} for why guessing wrong here
- * is designed to be harmless.
+ * The first name is now **confirmed exact**, not guessed (#691): the SDK bundle
+ * builds it as `` `Claude Code${OAUTH_FILE_SUFFIX}${"-credentials"}${hash}` ``,
+ * with an empty production suffix and no hash when the config dir is unscoped —
+ * see `claude-credentials.ts`, which quotes the function. The second is kept as a
+ * cheap hedge against an older or future spelling; it costs one `security` call
+ * that fails fast, and {@link probeMacosKeychainLogin} is built so a miss is
+ * harmless either way.
  */
 const KEYCHAIN_SERVICE_NAMES = ["Claude Code-credentials", "Claude Code"];
 /**
@@ -90,15 +130,24 @@ const KEYCHAIN_SERVICE_NAMES = ["Claude Code-credentials", "Claude Code"];
  * ## The asymmetry, which is deliberate
  *
  * A positive result is trustworthy: an entry under that service name is a login.
- * A negative result is NOT — the service name is Claude Code's internal detail
- * and can change, and `security` can fail for a dozen reasons that have nothing
- * to do with the user. So a miss returns `unknown`, and the caller falls back to
- * the existing, correct, generic warning. Guessing the name wrong therefore
- * costs nothing beyond the improvement not happening.
+ * A negative result is NOT — `security` can fail for a dozen reasons that have
+ * nothing to do with the user, and the service name, though now read straight out
+ * of the SDK bundle rather than guessed, is still Claude Code's internal detail
+ * and can change. So a miss returns `unknown`, and the caller falls back to the
+ * existing, correct, generic warning. Being wrong about the name therefore costs
+ * nothing beyond the improvement not happening.
  *
  * Never prompts and never reads a secret: `find-generic-password` WITHOUT `-w`
  * reports existence only (attributes, which are discarded), so nothing sensitive
  * reaches a log, a transcript, or this process's memory.
+ *
+ * One known imprecision, left in on purpose: Claude Code's own lookup also passes
+ * `-a` (`process.env.USER`, falling back to the OS username), and this does not.
+ * So an entry filed under a DIFFERENT account still reads as `found` here while
+ * Claude Code would miss it — a service running under a different `USER` than the
+ * terminal that logged in. Matching that exactly would trade a rare false
+ * positive for a rarer false negative, and a false negative is the one that costs
+ * something: it downgrades a specific instruction back to the generic warning.
  */
 export async function probeMacosKeychainLogin(platform = process.platform) {
     if (platform !== "darwin")
@@ -124,18 +173,251 @@ async function exists(p) {
         .catch(() => false);
 }
 /**
- * Create paddock's Claude home and bridge the user-level config into it.
+ * Symlink one entry of the user's home into paddock's, if paddock's does not
+ * already have that name. Returns whether a link was planted THIS call.
+ *
+ * Non-clobbering by `lstat`, not `stat`: a bridge symlink planted on a previous
+ * boot whose target has since been removed still occupies the name, and
+ * re-linking it would be a no-op churn at best. Never throws — a race with
+ * another instance, or a read-only home, is not a reason to fail a boot.
+ */
+async function bridgeEntry(cfg, entry) {
+    const target = path.join(cfg.legacyClaudeHome, entry);
+    const link = path.join(cfg.claudeHome, entry);
+    const occupied = await fs
+        .lstat(link)
+        .then(() => true)
+        .catch(() => false);
+    if (occupied)
+        return false;
+    if (!(await exists(target)))
+        return false;
+    return fs.symlink(target, link).then(() => true, () => false);
+}
+/**
+ * Undo {@link bridgeEntry} for one entry: remove it IF it is a symlink pointing
+ * into the user's home. Returns whether anything was removed.
+ *
+ * Deliberately narrow. A real file at that name is this instance's own — an
+ * operator's, or the product of a `CLAUDE_CONFIG_DIR=<own home> claude login` —
+ * and removing it would be paddock deleting a credential nobody asked it to
+ * touch. A symlink pointing somewhere OTHER than the user's home is not ours
+ * either. Only a link paddock itself would have planted is withdrawn, and the
+ * withdrawal happens inside paddock's own home, so the "never write to
+ * `~/.claude`" invariant is untouched.
+ */
+async function unbridgeEntry(cfg, entry) {
+    const link = path.join(cfg.claudeHome, entry);
+    const target = await fs.readlink(link).catch(() => null);
+    if (target === null)
+        return false; // absent, or a real file: not ours
+    const resolved = path.resolve(path.dirname(link), target);
+    if (resolved !== path.resolve(cfg.legacyClaudeHome, entry))
+        return false;
+    return fs.unlink(link).then(() => true, () => false);
+}
+const sha256 = (content) => createHash("sha256").update(content, "utf8").digest("hex");
+/** Read the entry → sha256 map of files paddock generated. Never throws. */
+async function readGeneratedHashes(home) {
+    const raw = await fs.readFile(path.join(home, GENERATED_MARKER_FILE), "utf8").catch(() => null);
+    if (raw === null)
+        return {};
+    try {
+        const parsed = JSON.parse(raw);
+        if (parsed === null || typeof parsed !== "object" || Array.isArray(parsed))
+            return {};
+        return parsed;
+    }
+    catch {
+        return {};
+    }
+}
+/** Record (or, with `null`, forget) the hash of a file paddock generated. */
+async function recordGenerated(home, entry, hash) {
+    const hashes = await readGeneratedHashes(home);
+    if (hash === null) {
+        if (!(entry in hashes))
+            return;
+        delete hashes[entry];
+    }
+    else {
+        if (hashes[entry] === hash)
+            return;
+        hashes[entry] = hash;
+    }
+    const marker = path.join(home, GENERATED_MARKER_FILE);
+    await fs.writeFile(marker, `${JSON.stringify(hashes, null, 2)}\n`, "utf8").catch(() => { });
+}
+/**
+ * Decide whether paddock may replace `<ownHome>/<entry>`.
+ *
+ * The `generated` case is the one that is new (#691 step 4) and the one the
+ * whole copy-and-filter mechanism rests on. A symlink needs no such question —
+ * paddock can always recognise its own by where it points — but a real file it
+ * wrote is indistinguishable from a real file a human wrote, and getting that
+ * wrong in either direction is bad: refuse to regenerate and the filtered copy
+ * is stale forever, regenerate anyway and paddock silently overwrites an
+ * operator's settings. Hashing what we wrote and comparing on the next boot
+ * answers it exactly, and answers `foreign` — the safe way — for any file we
+ * did not write or that has been edited since.
+ */
+async function classifyOwnEntry(cfg, entry) {
+    const own = path.join(cfg.claudeHome, entry);
+    const st = await fs.lstat(own).catch(() => null);
+    if (st === null)
+        return "absent";
+    if (st.isSymbolicLink()) {
+        const raw = await fs.readlink(own).catch(() => null);
+        if (raw === null)
+            return "foreign";
+        const resolved = path.resolve(path.dirname(own), raw);
+        return resolved === path.resolve(cfg.legacyClaudeHome, entry) ? "bridge" : "foreign";
+    }
+    const content = await fs.readFile(own, "utf8").catch(() => null);
+    if (content === null)
+        return "foreign";
+    const recorded = (await readGeneratedHashes(cfg.claudeHome))[entry];
+    return recorded !== undefined && recorded === sha256(content) ? "generated" : "foreign";
+}
+/** Remove a file paddock generated, and forget it. Returns whether it went. */
+async function removeGenerated(home, entry) {
+    const ok = await fs.rm(path.join(home, entry), { force: true }).then(() => true, () => false);
+    if (ok)
+        await recordGenerated(home, entry, null);
+    return ok;
+}
+/**
+ * Put the right `settings.json` in paddock's home for the `claude.hooks` mode
+ * (#691 step 4).
+ *
+ * Four states have to compose, and all four are reachable by editing one config
+ * key on a running instance:
+ *
+ * | own home has | `hooks: host` | `hooks: own` |
+ * |---|---|---|
+ * | nothing | symlink the user's | symlink, or write the filtered copy |
+ * | paddock's bridge symlink | leave it | replace with the filtered copy |
+ * | paddock's generated file | delete it, symlink | regenerate from the user's |
+ * | a human's file | leave it | leave it, and say so if hooks were in play |
  *
- * Idempotent and non-clobbering: an entry paddock's home already has (whether a
- * real file the user put there or a symlink from a previous boot) is left
- * completely alone, so this is safe to run on every start.
+ * The last row is the existing non-clobbering bridge behaviour, extended to a
+ * file rather than a name — with one addition, because silence there is
+ * dangerous: if the user's settings define hooks and paddock is not allowed to
+ * write the filtered copy, `hooks: own` is NOT in force for whatever that file
+ * says, so it warns rather than leaving an operator to believe a guarantee that
+ * is not holding.
+ *
+ * Never throws; a home it cannot write just keeps whatever it had.
+ */
+async function materializeSettings(cfg, hooks, report) {
+    const entry = SETTINGS_ENTRY;
+    const own = path.join(cfg.claudeHome, entry);
+    const state = await classifyOwnEntry(cfg, entry);
+    const raw = await fs.readFile(path.join(cfg.legacyClaudeHome, entry), "utf8").catch(() => null);
+    if (raw === null) {
+        // The user has no settings.json at all — including the case where they
+        // deleted one paddock had already bridged or filtered. Withdraw ours.
+        if (state === "bridge" && (await unbridgeEntry(cfg, entry)))
+            report.withdrawn.push(entry);
+        else if (state === "generated" && (await removeGenerated(cfg.claudeHome, entry)))
+            report.withdrawn.push(entry);
+        return;
+    }
+    const plan = planHostSettings(hooks, raw);
+    if (state === "foreign") {
+        if (plan.action === "write") {
+            report.notices.push({
+                level: "warn",
+                message: `\`claude.hooks: own\` cannot take effect: ${own} is a file paddock did not write, ` +
+                    `so it is left alone (as any entry of your own always is). Your ` +
+                    `~/.claude/${entry} defines hooks, and whatever THAT file says is what runs. ` +
+                    `Move or delete it to let paddock generate the filtered copy.`,
+            });
+        }
+        return;
+    }
+    if (plan.action === "skip") {
+        // Fail closed: a settings.json that will not parse cannot be filtered, and
+        // linking it instead would hand over exactly the hooks `own` withholds.
+        if (state === "bridge" && (await unbridgeEntry(cfg, entry)))
+            report.withdrawn.push(entry);
+        else if (state === "generated" && (await removeGenerated(cfg.claudeHome, entry)))
+            report.withdrawn.push(entry);
+        report.notices.push({
+            level: "warn",
+            message: `not using your ~/.claude/${entry}: ${plan.reason}. Paddock could not filter it, and ` +
+                `\`claude.hooks: own\` means it must not be used unfiltered — so this instance runs ` +
+                `with no user-level settings at all. Fix the JSON and restart.`,
+        });
+        return;
+    }
+    if (plan.action === "link") {
+        // Nothing to filter (no hooks, or `hooks: host`): the symlink is strictly
+        // better than a copy, so prefer it even under `own`. Only users who actually
+        // have hooks pay the staleness of a copy.
+        if (state === "generated")
+            await removeGenerated(cfg.claudeHome, entry);
+        if (await bridgeEntry(cfg, entry))
+            report.bridged.push(entry);
+        return;
+    }
+    if (state === "bridge")
+        await fs.unlink(own).catch(() => { });
+    const existing = state === "generated" ? await fs.readFile(own, "utf8").catch(() => null) : null;
+    if (existing !== plan.content) {
+        const ok = await fs.writeFile(own, plan.content, "utf8").then(() => true, () => false);
+        if (!ok) {
+            report.notices.push({
+                level: "warn",
+                message: `could not write ${own}, so \`claude.hooks: own\` is not in force — this instance ` +
+                    `has no user-level settings rather than filtered ones.`,
+            });
+            return;
+        }
+        report.generated.push(entry);
+    }
+    await recordGenerated(cfg.claudeHome, entry, sha256(plan.content));
+    report.notices.push({
+        level: "info",
+        message: `Claude hooks: your ~/.claude/${entry} defines ${plan.dropped.join(", ")}, which this ` +
+            `instance does NOT run (\`claude.hooks: own\`). Paddock writes its own ${entry} carrying ` +
+            `every other key of yours — permissions, model, statusline — and regenerates it at ` +
+            `startup, so restart paddock after editing yours. Set \`claude.hooks: host\` to run them.`,
+    });
+}
+/**
+ * Create paddock's Claude home and put the user-level config into it that this
+ * instance's `claude:` levers say it should have (#691).
+ *
+ * Idempotent and non-clobbering: a real file the user put in paddock's home is
+ * left completely alone, so this is safe to run on every start. What it DOES
+ * maintain is what paddock itself planted — bridge symlinks, withdrawn when a
+ * lever flips to `own`, and the generated `settings.json`, regenerated from the
+ * user's current one. Ownership of a generated file is decided by hash
+ * ({@link classifyOwnEntry}), never by "it is at a name we use".
  *
  * Never throws. A home that cannot be created is not a reason to refuse to boot
  * — Claude Code falls back to its own default and chat still works, which is the
  * same failure mode `ensureProjectChats` has always had.
+ *
+ * **Mutates `env`** (`process.env` by default) to carry `claude.credentials` into
+ * the environment Claude Code runs in — see {@link applyCredentialsMode} for why
+ * that is the only seam available and why it is safe process-wide. Done here
+ * because this is already the one call that runs before the fleet starts, and
+ * because the credential warning below has to describe the environment it just
+ * set rather than the one it found.
  */
 export async function ensureClaudeHome(cfg, env = process.env, probeKeychain = probeMacosKeychainLogin) {
-    const report = { bridged: [], notices: [] };
+    const report = { bridged: [], withdrawn: [], generated: [], notices: [] };
+    const credentials = cfg.claude.credentials;
+    const decision = applyCredentialsMode(credentials, env);
+    if (decision.action === "deferred") {
+        report.notices.push({
+            level: "info",
+            message: `${SECURE_STORAGE_DIR_VAR} is already set to "${decision.value}"; honouring it and ` +
+                `ignoring \`claude.credentials: ${credentials}\`. Unset it to let paddock choose.`,
+        });
+    }
     try {
         await fs.mkdir(path.join(cfg.claudeHome, "projects"), { recursive: true });
     }
@@ -148,78 +430,122 @@ export async function ensureClaudeHome(cfg, env = process.env, probeKeychain = p
         });
         return report;
     }
-    // Running in the user's own home is the pre-#620 layout: nothing to bridge
-    // (the files are already there) and nothing to warn about.
-    if (!cfg.ownsClaudeHome) {
+    // No "is this home ours?" branch any more (#691): it always is, and a config
+    // that resolves the home to `~/.claude` never gets this far — `resolveClaudeHome`
+    // refuses to start. What remains is one lever per entry.
+    // The login — `claude.credentials`, step 3. Its file half on Linux/Docker.
+    if (credentials === "host") {
+        if (await bridgeEntry(cfg, CREDENTIAL_ENTRY))
+            report.bridged.push(CREDENTIAL_ENTRY);
+    }
+    else if (await unbridgeEntry(cfg, CREDENTIAL_ENTRY)) {
+        report.withdrawn.push(CREDENTIAL_ENTRY);
         report.notices.push({
             level: "info",
-            message: `Claude home: ${cfg.claudeHome} (the user's own — paddock is not managing it)`,
+            message: `removed the bridged ${CREDENTIAL_ENTRY} symlink into ${cfg.legacyClaudeHome}: ` +
+                `\`claude.credentials: own\` means this instance uses only its own login. A real ` +
+                `${CREDENTIAL_ENTRY} in ${cfg.claudeHome} is left alone.`,
         });
-        return report;
     }
-    for (const entry of BRIDGED_ENTRIES) {
-        const target = path.join(cfg.legacyClaudeHome, entry);
-        const link = path.join(cfg.claudeHome, entry);
-        // lstat, not stat: a bridge symlink planted on a previous boot whose target
-        // has since been removed still occupies the name, and re-linking it would be
-        // a no-op churn at best.
-        const occupied = await fs
-            .lstat(link)
-            .then(() => true)
-            .catch(() => false);
-        if (occupied)
-            continue;
-        if (!(await exists(target)))
-            continue;
-        try {
-            await fs.symlink(target, link);
-            report.bridged.push(entry);
+    // CLAUDE.md, agents, commands, plugins — `claude.instructions`, step 4.
+    const instructions = cfg.claude.instructions;
+    for (const entry of INSTRUCTION_ENTRIES) {
+        if (instructions === "host") {
+            if (await bridgeEntry(cfg, entry))
+                report.bridged.push(entry);
         }
-        catch {
-            /* a race with another instance, or a read-only home — not fatal */
+        else if (await unbridgeEntry(cfg, entry)) {
+            report.withdrawn.push(entry);
         }
     }
+    if (instructions === "own") {
+        // Named rather than left to be discovered. `own` is the default and it is a
+        // real behaviour change for anyone with a curated `~/.claude/CLAUDE.md`:
+        // their agents quietly stop knowing things, with no error to search for. So
+        // paddock says which of their files it is not using, and which key turns
+        // them back on. Only when they HAVE some — otherwise it is noise.
+        const present = [];
+        for (const entry of INSTRUCTION_ENTRIES) {
+            if (await exists(path.join(cfg.legacyClaudeHome, entry)))
+                present.push(entry);
+        }
+        if (present.length > 0) {
+            report.notices.push({
+                level: "info",
+                message: `Claude instructions: this instance's own only (\`claude.instructions: own\`). Your ` +
+                    `~/.claude ${present.join(", ")} ${present.length === 1 ? "is" : "are"} NOT loaded — ` +
+                    `set \`claude.instructions: host\` (or PADDOCK_CLAUDE_INSTRUCTIONS=host) to share ` +
+                    `them. Each project's own CLAUDE.md is unaffected either way.`,
+            });
+        }
+    }
+    // settings.json — `claude.hooks`, step 4. Not a bridge decision: the file is a
+    // mixed bag, so `own` writes a filtered copy. See `materializeSettings`.
+    await materializeSettings(cfg, cfg.claude.hooks, report);
     report.notices.push({
         level: "info",
         message: `Claude home: ${cfg.claudeHome} (paddock-owned; ~/.claude is read-only)` +
-            (report.bridged.length > 0 ? ` — bridged from ~/.claude: ${report.bridged.join(", ")}` : ""),
+            (report.bridged.length > 0 ? ` — bridged from ~/.claude: ${report.bridged.join(", ")}` : "") +
+            (report.generated.length > 0 ? ` — generated: ${report.generated.join(", ")}` : "") +
+            (report.withdrawn.length > 0 ? ` — withdrawn: ${report.withdrawn.join(", ")}` : ""),
     });
-    // The credential question (#620). Claude Code derives its secure-storage
+    // The credential question (#620, #683). Claude Code derives its secure-storage
     // service name from whether `CLAUDE_CONFIG_DIR` is SET AT ALL — unset gets the
-    // plain name, set gets a path-hash suffix. Pointing paddock at its own home
-    // means it is always set, so a login held in the macOS Keychain under the plain
-    // name will not be found. A token in the environment is unaffected, and a
-    // file-based login is bridged above; if neither applies we cannot see the
-    // credential at all, so say so BEFORE the first turn fails with "Not logged in"
-    // rather than leaving the operator to work it out from a chat that never
-    // replies.
+    // plain name, set gets a path-hash suffix. Paddock always sets it, so a login
+    // held in the macOS Keychain under the plain name is invisible unless
+    // `credentials: host` un-scopes it above. A token in the environment is
+    // unaffected, and a file-based login is bridged (under `host`); if none of that
+    // applies we cannot see a credential at all, so say so BEFORE the first turn
+    // fails with "Not logged in" rather than leaving the operator to work it out
+    // from a chat that never replies.
     const hasTokenEnv = TOKEN_ENV_VARS.some((v) => (env[v] ?? "").trim().length > 0);
-    const hasCredentialFile = await exists(path.join(cfg.claudeHome, ".credentials.json"));
+    const hasCredentialFile = await exists(path.join(cfg.claudeHome, CREDENTIAL_ENTRY));
     if (!hasTokenEnv && !hasCredentialFile) {
         // #683: "you have no login" and "your login exists but I changed the service
-        // name it is filed under" are the same message today, and only one of them is
-        // the user's problem to solve. On darwin we can tell them apart, and a hit
-        // upgrades a paragraph of explanation into an instruction. A miss is
-        // inconclusive by design (see probeMacosKeychainLogin), so it falls through
-        // to the generic wording rather than asserting anything.
+        // name it is filed under" were the same message, and only one of them is the
+        // user's problem to solve. On darwin we can tell them apart, and a hit either
+        // resolves the question outright (`host`) or upgrades a paragraph of
+        // explanation into an instruction (`own`). A miss is inconclusive by design
+        // (see probeMacosKeychainLogin), so it falls through to the generic wording
+        // rather than asserting anything.
         const keychain = await probeKeychain();
-        report.notices.push({
-            level: "warn",
-            message: keychain === "found"
-                ? `you ARE logged in to Claude Code, but paddock cannot see it. The login is in the ` +
-                    `macOS Keychain, and Claude Code files it under a service name derived from whether ` +
-                    `CLAUDE_CONFIG_DIR is set — paddock sets it, to ${cfg.claudeHome}, so it looks under ` +
-                    `a different name. A Keychain entry cannot be bridged the way ~/.claude files are. ` +
-                    `Fix it either way: run \`CLAUDE_CONFIG_DIR=${cfg.claudeHome} claude login\` to log ` +
-                    `in for this home too, or set CLAUDE_HOME=${cfg.legacyClaudeHome} (what \`paddock\` ` +
-                    `does by default) to run against your own home and use the login you already have.`
-                : `no Claude credentials found for ${cfg.claudeHome}: no .credentials.json and no ` +
-                    `${TOKEN_ENV_VARS.join("/")} in the environment. Claude Code scopes its credential ` +
-                    `store to CLAUDE_CONFIG_DIR, so a login stored in the OS keychain against the ` +
-                    `default home will NOT be found here. If turns fail with "Not logged in", either ` +
-                    `re-run \`claude login\` with CLAUDE_CONFIG_DIR=${cfg.claudeHome}, or set ` +
-                    `CLAUDE_HOME=${cfg.legacyClaudeHome} to keep using the previous layout.`,
-        });
+        if (keychain === "found" && credentials === "host") {
+            // Not a problem: this is the lever working. Said out loud anyway, because
+            // "paddock found no credentials of its own and is using yours" is exactly
+            // the kind of thing an operator should not have to deduce.
+            report.notices.push({
+                level: "info",
+                message: `Claude login: this machine's own, from the macOS Keychain ` +
+                    `(\`claude.credentials: host\`). Paddock sets ${SECURE_STORAGE_DIR_VAR}="" so Claude ` +
+                    `Code reads the same unsuffixed entry your own \`claude\` does, while its config dir ` +
+                    `stays at ${cfg.claudeHome}. Set \`claude.credentials: own\` to use only a login of ` +
+                    `this instance's.`,
+            });
+        }
+        else {
+            report.notices.push({
+                level: "warn",
+                message: keychain === "found"
+                    ? `you ARE logged in to Claude Code, but paddock cannot see it. The login is in the ` +
+                        `macOS Keychain, and Claude Code files it under a service name derived from ` +
+                        `whether CLAUDE_CONFIG_DIR is set — paddock sets it, to ${cfg.claudeHome}, so it ` +
+                        `looks under a different name. \`claude.credentials: own\` is what keeps it that ` +
+                        `way. Fix it either way: drop that key (the default is \`host\`, which shares this ` +
+                        `machine's login and writes nothing), or log in for this home too with ` +
+                        `\`CLAUDE_CONFIG_DIR=${cfg.claudeHome} claude login\`.`
+                    : `no Claude credentials found for ${cfg.claudeHome}: no ${CREDENTIAL_ENTRY} and no ` +
+                        `${TOKEN_ENV_VARS.join("/")} in the environment` +
+                        (credentials === "host"
+                            ? ` — and \`claude.credentials: host\` found nothing on this machine to share ` +
+                                `either (no ~/.claude/${CREDENTIAL_ENTRY}, and no macOS Keychain login under ` +
+                                `the name Claude Code uses). `
+                            : `, and \`claude.credentials: own\` means this instance shares none of yours. ` +
+                                `Claude Code scopes its credential store to CLAUDE_CONFIG_DIR, so a login in ` +
+                                `the OS keychain against the default home will NOT be found here. `) +
+                        `If turns fail with "Not logged in", re-run \`claude login\` with ` +
+                        `CLAUDE_CONFIG_DIR=${cfg.claudeHome}, or put a token in the environment.`,
+            });
+        }
     }
     return report;
 }
@@ -274,6 +600,14 @@ export function legacySourcePath(legacyHome, folderName) {
  * mirroring them would re-offer paddock's own chats as "adoptable". That also
  * makes the leftovers from before this change inert rather than confusing.
  *
+ * A folder paddock's own home ALREADY links to directly is skipped for the same
+ * reason. That is the `claude.transcripts: host` case (#691): there the folder in
+ * the user's home is a real directory that paddock itself writes through, so
+ * mirroring it would offer every project's own live chats back as importable —
+ * the #658 mistake with the arrow reversed. (Adoption de-duplicates sources by
+ * resolved real path, so this was noise rather than breakage; it is skipped here
+ * because a link paddock plants and then mirrors is confusing to find on disk.)
+ *
  * Idempotent and never throws. Returns every mirror it maintains, not only the
  * ones planted this call, so a caller can rely on the mapping after any call.
  */
@@ -293,6 +627,12 @@ export async function mirrorLegacyTranscriptFolders(claudeHome, legacyHome) {
             const st = await fs.lstat(legacyDir).catch(() => null);
             if (!st?.isDirectory())
                 continue; // skips files AND paddock's old symlinks
+            // Already reachable directly from paddock's own home (`transcripts: host`)?
+            // Then it is not an outside source to import; it is where this instance's
+            // own chats live.
+            const direct = await fs.readlink(path.join(to, folderName)).catch(() => null);
+            if (direct !== null && path.resolve(to, direct) === path.resolve(legacyDir))
+                continue;
             const engineCwd = legacySourcePath(legacyHome, folderName);
             // encodePathForCli, not a local encoder: this name must be byte-identical
             // to what the engine derives from `engineCwd`, including the truncate+hash