abelworkflow 1.1.0 → 1.1.2

package/extensions/pi-gpt-responses-compat/index.ts

@@ -1,23 +1,6 @@
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
-import { createNodeInsecureDispatcher, installProviderTlsFetch } from "./tls-fetch.mjs";
-
-let nodeInsecureDispatcher: unknown;
-
-function getNodeInsecureDispatcher() {
-  if (nodeInsecureDispatcher) return nodeInsecureDispatcher;
-  nodeInsecureDispatcher = createNodeInsecureDispatcher();
-  return nodeInsecureDispatcher;
-}
 
 export default function (pi: ExtensionAPI) {
-  if (typeof globalThis.fetch === "function") {
-    const runtime = typeof (globalThis as any).Bun === "undefined" ? "node" : "bun";
-    installProviderTlsFetch({
-      runtime,
-      insecureDispatcher: runtime === "node" ? getNodeInsecureDispatcher : undefined
-    });
-  }
-
   pi.on("before_provider_request", (event, ctx) => {
     const payload = event.payload as any;
     if (!payload || typeof payload !== "object" || Array.isArray(payload)) return;

package/lib/cli/main.mjs

@@ -126,31 +126,32 @@ ${c.bold("Default behavior:")}
 `);
 }
 
-async function configureCodexForPaths(paths) {
+async function configureProviderWithMetadata(paths, configure) {
   const metadata = await readInstallMetadata(paths);
   const packageVersion = await packageVersionFor(paths, metadata);
-  const result = await configureCodexApi(paths, promptApi, {
-    managedAuthKeys: getPreviousManagedCodexAuthKeys(metadata),
-    managedCodexAgentFiles: getPreviousManagedCodexAgentFiles(metadata)
-  });
+  const overrides = await configure(metadata);
   await writeInstallMetadata(paths, finalizeProviderInstallMetadata({
     previousMetadata: metadata,
     packageVersion,
-    overrides: {
+    overrides
+  }));
+}
+
+async function configureCodexForPaths(paths) {
+  await configureProviderWithMetadata(paths, async (metadata) => {
+    const result = await configureCodexApi(paths, promptApi, {
+      managedAuthKeys: getPreviousManagedCodexAuthKeys(metadata),
+      managedCodexAgentFiles: getPreviousManagedCodexAgentFiles(metadata)
+    });
+    return {
       managedCodexAuthKeys: result.managedAuthKeys,
       managedCodexAgentFiles: result.managedCodexAgentFiles
-    }
-  }));
+    };
+  });
 }
 
 async function configureClaudeForPaths(paths) {
-  const metadata = await readInstallMetadata(paths);
-  const packageVersion = await packageVersionFor(paths, metadata);
-  await configureClaudeApi(paths, promptApi);
-  await writeInstallMetadata(paths, finalizeProviderInstallMetadata({
-    previousMetadata: metadata,
-    packageVersion
-  }));
+  await configureProviderWithMetadata(paths, () => configureClaudeApi(paths, promptApi));
 }
 
 async function runFullInit(options) {
@@ -176,13 +177,13 @@ async function runFullInit(options) {
     await installCliTool("pi", promptApi);
   }
   if (await confirmOrCancel({ message: "是否配置 Pi 当前有效 Provider API？", initialValue: commandExists("pi") })) {
-    await configurePiApi(options.paths, ensurePiResourcesLinked, promptApi);
+    await configurePiApi(options.paths, ensurePiResourcesLinkedWithReport, promptApi);
   }
   if (await confirmOrCancel({ message: "是否填写 grok-search 环境变量？", initialValue: false })) {
-    await configureGrokSearchEnv(options.paths, (paths) => ensureSkillPresent(paths, "grok-search"), promptApi);
+    await configureGrokSearchEnv(options.paths, (paths) => ensureSkillPresentWithReport(paths, "grok-search"), promptApi);
   }
   if (await confirmOrCancel({ message: "是否填写 context7-auto-research 环境变量？", initialValue: false })) {
-    await configureContext7Env(options.paths, (paths) => ensureSkillPresent(paths, "context7-auto-research"), promptApi);
+    await configureContext7Env(options.paths, (paths) => ensureSkillPresentWithReport(paths, "context7-auto-research"), promptApi);
   }
 
   const conflictCount = uniqueReportPaths(installReport.conflicts).length;
@@ -205,35 +206,31 @@ async function runInteractiveMenu(options) {
     return opt;
   };
   const cliToolMenus = {
-    "pi-cli": {
-      tool: "pi",
+    pi: {
       title: "Pi",
-      actions: {
-        "pi-install": async () => installCliTool("pi", promptApi),
-        "pi-api": async () => configurePiApi(options.paths, ensurePiResourcesLinkedWithReport, promptApi)
-      }
+      install: () => installCliTool("pi", promptApi),
+      configure: () => configurePiApi(options.paths, ensurePiResourcesLinkedWithReport, promptApi)
     },
-    "codex-cli": {
-      tool: "codex",
+    codex: {
       title: "Codex",
-      actions: {
-        "codex-install": async () => installCliTool("codex", promptApi),
-        "codex-api": async () => configureCodexForPaths(options.paths)
-      }
+      install: () => installCliTool("codex", promptApi),
+      configure: () => configureCodexForPaths(options.paths)
     },
-    "claude-cli": {
-      tool: "claude",
+    claude: {
       title: "Claude Code",
-      actions: {
-        "claude-install": async () => installCliTool("claude", promptApi),
-        "claude-api": async () => configureClaudeForPaths(options.paths)
-      }
+      install: () => installCliTool("claude", promptApi),
+      configure: () => configureClaudeForPaths(options.paths)
     }
   };
-  const runCliToolMenu = async ({ tool, title, actions }) => {
+  const runCliToolMenu = async (tool) => {
+    const menu = cliToolMenus[tool];
+    const actions = {
+      [`${tool}-api`]: menu.configure,
+      [`${tool}-install`]: menu.install
+    };
     while (true) {
       const choice = await p.select({
-        message: `请选择 ${title} 操作`,
+        message: `请选择 ${menu.title} 操作`,
         options: buildCliToolMenuDescriptors(tool).map(buildOption),
         initialValue: `${tool}-api`
       });
@@ -259,9 +256,9 @@ async function runInteractiveMenu(options) {
     }),
     "grok-search": async () => configureGrokSearchEnv(options.paths, (paths) => ensureSkillPresentWithReport(paths, "grok-search"), promptApi),
     context7: async () => configureContext7Env(options.paths, (paths) => ensureSkillPresentWithReport(paths, "context7-auto-research"), promptApi),
-    "pi-cli": async () => runCliToolMenu(cliToolMenus["pi-cli"]),
-    "codex-cli": async () => runCliToolMenu(cliToolMenus["codex-cli"]),
-    "claude-cli": async () => runCliToolMenu(cliToolMenus["claude-cli"])
+    "pi-cli": async () => runCliToolMenu("pi"),
+    "codex-cli": async () => runCliToolMenu("codex"),
+    "claude-cli": async () => runCliToolMenu("claude")
   };
 
   while (true) {
@@ -376,10 +373,4 @@ async function main(argv, runtime = {}) {
   }
 }
 
-export {
-  main,
-  presentInstallReport,
-  printHelp,
-  runFullInit,
-  runInteractiveMenu
-};
+export { main, presentInstallReport };

package/lib/cli/prompts.mjs

@@ -11,22 +11,10 @@ const interactiveMenuDescriptors = [
   { value: "exit", label: "退出", group: "exit" }
 ];
 const interactiveMenuDefaultValue = "full-init";
-const cliToolMenuDescriptorMap = {
-  pi: [
-    { value: "pi-api", label: "配置 Pi API" },
-    { value: "pi-install", label: "安装/更新 Pi" },
-    { value: "back", label: "返回上一级" }
-  ],
-  codex: [
-    { value: "codex-api", label: "配置 Codex API" },
-    { value: "codex-install", label: "安装/更新 Codex" },
-    { value: "back", label: "返回上一级" }
-  ],
-  claude: [
-    { value: "claude-api", label: "配置 Claude Code API" },
-    { value: "claude-install", label: "安装/更新 Claude Code" },
-    { value: "back", label: "返回上一级" }
-  ]
+const cliToolLabels = {
+  pi: "Pi",
+  codex: "Codex",
+  claude: "Claude Code"
 };
 class CancelledError extends Error {
   constructor(message = "用户取消") {
@@ -36,9 +24,13 @@ class CancelledError extends Error {
 }
 
 function buildCliToolMenuDescriptors(tool) {
-  const descriptors = cliToolMenuDescriptorMap[tool];
-  if (!descriptors) throw new Error(`Unknown CLI tool: ${tool}`);
-  return descriptors.map((descriptor) => ({ ...descriptor }));
+  const label = cliToolLabels[tool];
+  if (!label) throw new Error(`Unknown CLI tool: ${tool}`);
+  return [
+    { value: `${tool}-api`, label: `配置 ${label} API` },
+    { value: `${tool}-install`, label: `安装/更新 ${label}` },
+    { value: "back", label: "返回上一级" }
+  ];
 }
 
 function required(message = "此项不能为空") {
@@ -79,6 +71,16 @@ function assertNotCancelled(value) {
   if (p.isCancel(value)) throw new CancelledError();
 }
 
+async function passwordOrExisting({ message, existingValue, requiredMessage = "API Key 不能为空" }) {
+  const input = await p.password(passwordPromptOptions(
+    message,
+    existingValue,
+    requiredMessage ? requiredUnlessExisting(existingValue, requiredMessage) : undefined
+  ));
+  assertNotCancelled(input);
+  return resolvePasswordValue(input, existingValue);
+}
+
 async function confirmOrCancel({ message, initialValue = false }) {
   const value = await p.confirm({ message, initialValue, active: "是", inactive: "否" });
   assertNotCancelled(value);
@@ -98,6 +100,7 @@ export {
   confirmOrCancel,
   interactiveMenuDefaultValue,
   interactiveMenuDescriptors,
+  passwordOrExisting,
   passwordPromptOptions,
   required,
   requiredUnlessExisting,

package/lib/config/dotenv.mjs

@@ -34,14 +34,6 @@ function quoteEnvValue(value) {
   throw new Error("Environment value cannot be represented without changing its meaning");
 }
 
-function renderDotenv(values) {
-  const lines = Object.entries(values)
-    .filter(([, value]) => value !== undefined && value !== null && value !== "")
-    .sort(([left], [right]) => left.localeCompare(right))
-    .map(([key, value]) => `${key}=${quoteEnvValue(String(value))}`);
-  return lines.length ? `${lines.join("\n")}\n` : "";
-}
-
 function getDotenvLines(content) {
   const lines = [];
   for (let start = 0; start < content.length;) {
@@ -99,4 +91,4 @@ function updateDotenvContent(content, updates) {
   return next;
 }
 
-export { parseDotenv, quoteEnvValue, renderDotenv, updateDotenvContent };
+export { parseDotenv, quoteEnvValue, updateDotenvContent };

package/lib/config/store.mjs

@@ -6,7 +6,6 @@ import { stripJsonComments } from "./jsonc.mjs";
 
 const newBackupMarker = ".abelworkflow.bak.";
 const newBackupSuffixPattern = /^\d+-\d+-\d{10,}$/u;
-const chmod = fs.chmod;
 let uniqueFileIndex = 0;
 
 function nextUniqueSuffix() {
@@ -225,21 +224,6 @@ async function updateLockedJson(path, updater, options = {}) {
   }
 }
 
-async function backupExistingPath(targetPath, options = {}) {
-  return copyBackup(targetPath, options);
-}
-
-async function backupPrivateFile(targetPath, content, options = {}) {
-  if (!(await pathExists(targetPath))) return null;
-  const backupLimit = options.backupLimit ?? 3;
-  if (backupLimit === 0) return null;
-  const backupPath = await createBackupPath(targetPath);
-  await fs.writeFile(backupPath, content, { encoding: "utf8", flag: "wx", mode: 0o600 });
-  if (isPosix()) await fs.chmod(backupPath, 0o600);
-  await pruneNewBackups(targetPath, backupLimit);
-  return backupPath;
-}
-
 async function backupIfNeeded(targetPath, options = {}) {
   if (!(await pathExists(targetPath))) return null;
   const backupLimit = options.backupLimit ?? 3;
@@ -281,19 +265,8 @@ async function updateDotenvFile(path, updates, options = {}) {
   return writeText(path, updateDotenvContent(content, updates), options);
 }
 
-async function writeJsonFileSafe(path, data, options = {}) {
-  return writeJson(path, data, { ...options, backupLimit: options.backupLimit ?? 0 });
-}
-
-async function writeJsonFileWithBackup(path, data, options = {}) {
-  return writeJson(path, data, options);
-}
-
 export {
-  backupExistingPath,
   backupIfNeeded,
-  backupPrivateFile,
-  chmod,
   ensurePrivateJsonFile,
   pathExists,
   pathTargetExists,
@@ -304,7 +277,5 @@ export {
   updateDotenvFile,
   updateLockedJson,
   writeJson,
-  writeJsonFileSafe,
-  writeJsonFileWithBackup,
   writeText
 };

package/lib/config/toml.mjs

@@ -46,6 +46,15 @@ function removeTomlSection(content, sectionName) {
   return `${content.slice(0, section.start)}${content.slice(section.end)}`;
 }
 
+function removeTomlSectionField(content, sectionName, field) {
+  const document = parseTomlDocument(content);
+  const section = findTomlSection(document, sectionName);
+  const entry = section && findTomlAssignment(document.assignments, field, section);
+  return entry
+    ? `${content.slice(0, entry.start)}${content.slice(entry.lineEnd)}`
+    : content;
+}
+
 function buildTomlSection(sectionName, values, lineEnding = "\n") {
   const lines = [`[${sectionName}]`];
   for (const [key, value] of Object.entries(values)) {
@@ -399,14 +408,6 @@ function findTomlAssignment(assignments, field, section) {
   ));
 }
 
-function splitTopLevelTomlContent(content) {
-  const { topLevelEnd } = parseTomlDocument(content);
-  return {
-    topLevel: topLevelEnd === -1 ? content : content.slice(0, topLevelEnd),
-    rest: topLevelEnd === -1 ? "" : content.slice(topLevelEnd)
-  };
-}
-
 function extractTopLevelTomlEntries(content) {
   const document = parseTomlDocument(content);
   return document.assignments
@@ -631,8 +632,8 @@ export {
   parseTomlSection,
   readTopLevelTomlString,
   removeTomlSection,
+  removeTomlSectionField,
   removeTopLevelTomlField,
-  splitTopLevelTomlContent,
   updateTomlSectionFields,
   updateTopLevelTomlField
 };

package/lib/installer/assets.mjs

@@ -1,4 +1,3 @@
-import { createHash } from "node:crypto";
 import * as fs from "node:fs/promises";
 import { join, relative, resolve } from "node:path";
 import {
@@ -8,6 +7,7 @@ import {
   readJsonFileSafe,
   writeText
 } from "../config/store.mjs";
+import { hashBytes } from "../utils.mjs";
 import { createInstallReport, readInstallMetadata } from "./state.mjs";
 
 const ignoredSkillPathPatterns = [
@@ -69,10 +69,6 @@ function mapGitignoreTemplate(relativePath) {
   return relativePath.replace(/(^|\/)gitignore\.template$/u, "$1.gitignore");
 }
 
-function hashBytes(content) {
-  return createHash("sha256").update(content).digest("hex");
-}
-
 function shouldCopySkillPath(skillsRoot, sourcePath) {
   const relativePath = normalizeRelativePath(relative(skillsRoot, sourcePath));
   if (!relativePath) return true;
@@ -316,9 +312,9 @@ async function ensureManagedContainerDirectory(targetPath, sourcePath) {
 export {
   collectManagedAssets,
   ensureManagedContainerDirectory,
-  hashBytes,
   pathsReferToSameEntry,
   readManagedFiles,
+  readPackageVersion,
   shouldCopySkillPath,
   syncManagedFiles
 };

package/lib/installer/install.mjs

@@ -1,7 +1,6 @@
-import { join } from "node:path";
-import { pathExists, readJsonFileSafe } from "../config/store.mjs";
-import { createPaths, defaultPaths } from "../paths.mjs";
-import { readManagedFiles, syncManagedFiles } from "./assets.mjs";
+import { pathExists } from "../config/store.mjs";
+import { createPaths } from "../paths.mjs";
+import { readManagedFiles, readPackageVersion, syncManagedFiles } from "./assets.mjs";
 import { linkClaude, linkCodex, linkPi } from "./links.mjs";
 import {
   buildInstallMetadata,
@@ -27,15 +26,6 @@ function validatedPaths(paths) {
   });
 }
 
-function resolveInstallPaths(options = {}) {
-  if (options.paths) return validatedPaths(options.paths);
-  return createPaths({
-    homeDir: options.homeDir ?? defaultPaths.homeDir,
-    packageRoot: options.packageRoot ?? defaultPaths.packageRoot,
-    agentsDir: options.agentsDir ?? defaultPaths.agentsDir
-  });
-}
-
 function reportFromLinkResults(results) {
   const report = createInstallReport();
   for (const result of results) {
@@ -58,12 +48,17 @@ async function packageVersionFor(paths, previousMetadata) {
     && previousMetadata.packageVersion) {
     return previousMetadata.packageVersion;
   }
-  const packagePath = join(paths.packageRoot, "package.json");
-  const packageData = await readJsonFileSafe(packagePath);
-  if (typeof packageData.version !== "string" || !packageData.version) {
-    throw new Error(`Missing package version in ${packagePath}`);
-  }
-  return packageData.version;
+  return readPackageVersion(paths);
+}
+
+async function writeInstallState(paths, assetResult, linkedTargets) {
+  const metadata = buildInstallMetadata({
+    previousMetadata: assetResult.previousMetadata,
+    packageVersion: assetResult.packageVersion,
+    managedFiles: assetResult.managedFiles,
+    linkedTargets
+  });
+  await writeInstallMetadata(paths, metadata);
 }
 
 async function installWorkflow(options) {
@@ -94,20 +89,12 @@ async function installWorkflow(options) {
   const codexResults = await linkCodex(paths, previousLinkedTargets, linkOptions);
   const piResults = await linkPi(paths, previousLinkedTargets, linkOptions);
   const linkResults = [...claudeResults, ...codexResults, ...piResults];
-  const linkedTargets = mergeLinkedTargets(previousLinkedTargets, linkResults);
-
-  const metadata = buildInstallMetadata({
-    previousMetadata,
-    packageVersion: assetResult.packageVersion,
-    managedFiles: assetResult.managedFiles,
-    linkedTargets
-  });
-  await writeInstallMetadata(paths, metadata);
+  await writeInstallState(paths, assetResult, mergeLinkedTargets(previousLinkedTargets, linkResults));
   return mergeInstallReports(assetResult.report, reportFromLinkResults(linkResults));
 }
 
 async function installManagedWorkflow(options = {}) {
-  return installWorkflow({ ...options, paths: resolveInstallPaths(options) });
+  return installWorkflow({ ...options, paths: validatedPaths(options.paths) });
 }
 
 async function ensureSkillPresent(inputPaths, skillName) {
@@ -118,13 +105,7 @@ async function ensureSkillPresent(inputPaths, skillName) {
     force: false,
     pathPrefix: `skills/${skillName}/`
   });
-  const metadata = buildInstallMetadata({
-    previousMetadata,
-    packageVersion: assetResult.packageVersion,
-    managedFiles: assetResult.managedFiles,
-    linkedTargets: previousMetadata.linkedTargets ?? {}
-  });
-  await writeInstallMetadata(paths, metadata);
+  await writeInstallState(paths, assetResult, previousMetadata.linkedTargets ?? {});
   return assetResult.report;
 }
 
@@ -134,13 +115,7 @@ async function ensurePiResourcesLinked(inputPaths) {
   const assetResult = await syncManagedFiles({ paths, force: false });
   const previousLinkedTargets = previousMetadata.linkedTargets ?? {};
   const piResults = await linkPi(paths, previousLinkedTargets, { force: false });
-  const metadata = buildInstallMetadata({
-    previousMetadata,
-    packageVersion: assetResult.packageVersion,
-    managedFiles: assetResult.managedFiles,
-    linkedTargets: mergeLinkedTargets(previousLinkedTargets, piResults)
-  });
-  await writeInstallMetadata(paths, metadata);
+  await writeInstallState(paths, assetResult, mergeLinkedTargets(previousLinkedTargets, piResults));
   return mergeInstallReports(assetResult.report, reportFromLinkResults(piResults));
 }
 

package/lib/installer/links.mjs

@@ -1,12 +1,9 @@
 import * as fs from "node:fs/promises";
 import { basename, dirname, isAbsolute, join, relative, resolve, sep } from "node:path";
 import { backupIfNeeded, pathExists, pathTargetExists } from "../config/store.mjs";
-import { isWindows } from "../paths.mjs";
-import { ensureManagedContainerDirectory, hashBytes } from "./assets.mjs";
-
-function shouldForceFileSymlinkFailure(kind) {
-  return process.env.ABELWORKFLOW_TEST_FORCE_FILE_SYMLINK_EPERM === "1" && isWindows() && kind === "file";
-}
+import { containsPath, isWindows } from "../paths.mjs";
+import { hashBytes } from "../utils.mjs";
+import { ensureManagedContainerDirectory } from "./assets.mjs";
 
 async function createManagedTargetState(targetPath, sourcePath, kind, mode, status) {
   let targetHash;
@@ -16,12 +13,7 @@ async function createManagedTargetState(targetPath, sourcePath, kind, mode, stat
   return { targetPath, sourcePath, kind, mode, status, ...(targetHash ? { targetHash } : {}) };
 }
 
-async function createSymlink(targetPath, sourcePath, linkType, kind) {
-  if (shouldForceFileSymlinkFailure(kind)) {
-    const error = new Error("simulated EPERM");
-    error.code = "EPERM";
-    throw error;
-  }
+async function createSymlink(targetPath, sourcePath, linkType) {
   await fs.symlink(sourcePath, targetPath, linkType);
 }
 
@@ -150,7 +142,7 @@ async function ensureManagedLink(targetPath, sourcePath, kind, previousLinkedTar
 
   const linkType = isWindows() ? (kind === "dir" ? "junction" : "file") : kind;
   try {
-    await createSymlink(targetPath, sourcePath, linkType, kind);
+    await createSymlink(targetPath, sourcePath, linkType);
     return createManagedTargetState(targetPath, sourcePath, kind, "symlink", "linked");
   } catch (error) {
     if (!shouldFallbackToManagedFile(error, kind)) throw error;
@@ -258,11 +250,8 @@ async function getPiExtensionNames(extensionsDir) {
 }
 
 function isWithinManagedRoot(targetPath, managedSourceRoot, pathOps = { isAbsolute, relative, sep }) {
-  const relativePath = pathOps.relative(managedSourceRoot, targetPath);
-  return relativePath !== ""
-    && !pathOps.isAbsolute(relativePath)
-    && relativePath !== ".."
-    && !relativePath.startsWith(`..${pathOps.sep}`);
+  return pathOps.relative(managedSourceRoot, targetPath) !== ""
+    && containsPath(managedSourceRoot, targetPath, pathOps);
 }
 
 async function pruneManagedTargets(targetDir, managedSourceRoot, expectedNames, previousLinkedTargets, options = {}) {

package/lib/installer/state.mjs

@@ -1,5 +1,6 @@
 import { join } from "node:path";
 import { readJsonFileSafe, writeJson } from "../config/store.mjs";
+import { isManagedCodexAgentFileEntry } from "../utils.mjs";
 
 function metadataPathFor(paths) {
   return join(paths.agentsDir, paths.installMetadataName);
@@ -30,11 +31,7 @@ function getPreviousManagedCodexAuthKeys(previousMetadata = {}) {
 function normalizeManagedCodexAgentFiles(value = {}) {
   if (!value || typeof value !== "object" || Array.isArray(value)) return {};
   return sortObject(Object.fromEntries(Object.entries(value).filter(([name, hash]) => (
-    name.endsWith(".toml")
-    && !name.includes("/")
-    && !name.includes("\\")
-    && typeof hash === "string"
-    && /^[a-f0-9]{64}$/u.test(hash)
+    isManagedCodexAgentFileEntry(name, hash)
   ))));
 }
 
@@ -63,7 +60,7 @@ function buildInstallMetadata({
   managedCodexAgentFiles = getPreviousManagedCodexAgentFiles(previousMetadata),
   linkedTargets = {}
 }) {
-  const metadata = {
+  return {
     schemaVersion: 2,
     packageVersion,
     managedFiles: sortObject(managedFiles),
@@ -71,10 +68,6 @@ function buildInstallMetadata({
     managedCodexAgentFiles: normalizeManagedCodexAgentFiles(managedCodexAgentFiles),
     linkedTargets: sortObject(linkedTargets)
   };
-  if (typeof previousMetadata.installedAt === "string") {
-    metadata.installedAt = previousMetadata.installedAt;
-  }
-  return metadata;
 }
 
 function finalizeProviderInstallMetadata({
@@ -100,17 +93,6 @@ function finalizeProviderInstallMetadata({
   });
 }
 
-function linkedTargetsFromResults(results) {
-  return Object.fromEntries(results
-    .filter((result) => result.sourcePath)
-    .map((result) => [result.targetPath, {
-      sourcePath: result.sourcePath,
-      kind: result.kind,
-      mode: result.mode,
-      ...(result.targetHash ? { targetHash: result.targetHash } : {})
-    }]));
-}
-
 function mergeLinkedTargets(previousLinkedTargets = {}, results = []) {
   const nextLinkedTargets = { ...previousLinkedTargets };
   for (const result of results) {
@@ -154,7 +136,6 @@ export {
   finalizeProviderInstallMetadata,
   getPreviousManagedCodexAgentFiles,
   getPreviousManagedCodexAuthKeys,
-  linkedTargetsFromResults,
   mergeInstallReports,
   mergeLinkedTargets,
   metadataPathFor,