poe-code 3.0.236 → 3.0.238

package/dist/providers/poe-agent.js

@@ -40385,6 +40385,7 @@ function capitalize(value) {
 
 // packages/agent-code-review/src/config-scope.ts
 init_src7();
+import { isAbsolute as isAbsolute3 } from "node:path";
 function parseHumanGate(value) {
   if (typeof value !== "object" || value === null || Array.isArray(value)) {
     throw new Error("humanGate must be an object");
@@ -40401,13 +40402,23 @@ function parseHumanGate(value) {
   }
   return { provider: provider2 };
 }
+function parseCodeReviewProfileDirectories(value) {
+  if (!Array.isArray(value) || value.some((directory) => typeof directory !== "string")) {
+    throw new Error("codeReview.profileDirectories must be an array of strings");
+  }
+  const directories = value.map((directory) => directory.trim());
+  if (directories.some((directory) => !directory || !isAbsolute3(directory))) {
+    throw new Error("codeReview.profileDirectories entries must be absolute paths");
+  }
+  return [...new Set(directories)];
+}
 function parseCodeReviewConfigDocument(value) {
   if (typeof value !== "object" || value === null || Array.isArray(value)) {
     throw new Error("codeReview must be an object");
   }
   const config = value;
   for (const key2 of Object.keys(config)) {
-    if (!["agent", "draftStore", "humanGate"].includes(key2)) {
+    if (!["agent", "draftStore", "humanGate", "profileDirectories"].includes(key2)) {
       throw new Error(`codeReview.${key2} is not supported`);
     }
   }
@@ -40420,7 +40431,8 @@ function parseCodeReviewConfigDocument(value) {
   return {
     ...config.agent === void 0 ? {} : { agent: config.agent },
     ...config.draftStore === void 0 ? {} : { draftStore: config.draftStore },
-    ...config.humanGate === void 0 ? {} : { humanGate: parseHumanGate(config.humanGate) }
+    ...config.humanGate === void 0 ? {} : { humanGate: parseHumanGate(config.humanGate) },
+    ...config.profileDirectories === void 0 ? {} : { profileDirectories: parseCodeReviewProfileDirectories(config.profileDirectories) }
   };
 }
 var codeReviewConfigScope = defineScope("codeReview", {
@@ -40439,6 +40451,12 @@ var codeReviewConfigScope = defineScope("codeReview", {
     default: { provider: "none" },
     parse: parseHumanGate,
     doc: "External human-gate configuration for code review runs."
+  },
+  profileDirectories: {
+    type: "json",
+    default: [],
+    parse: parseCodeReviewProfileDirectories,
+    doc: "Absolute external reviewer profile directories, in precedence order after repo-local profiles."
   }
 });
 
@@ -40462,7 +40480,7 @@ import {
   stat as stat6,
   unlink as unlink4
 } from "node:fs/promises";
-import { basename as basename2, dirname, isAbsolute as isAbsolute3, parse as parse6, relative as relative2, resolve as resolve2, sep as sep2 } from "node:path";
+import { basename as basename2, dirname, isAbsolute as isAbsolute4, parse as parse6, relative as relative2, resolve as resolve2, sep as sep2 } from "node:path";
 
 // packages/github-review/src/command.ts
 import { spawnSync } from "node:child_process";
@@ -42144,7 +42162,7 @@ function codeReviewFileName(prUrl) {
 }
 function resolveCodeReviewStoreDirectory(cwd, directory = DEFAULT_CODE_REVIEW_REVIEWS_DIRECTORY) {
   const root = resolve2(cwd, CODE_REVIEW_DIRECTORY);
-  const target = isAbsolute3(directory) ? resolve2(directory) : resolve2(cwd, directory);
+  const target = isAbsolute4(directory) ? resolve2(directory) : resolve2(cwd, directory);
   const fromRoot = relative2(root, target);
   if (fromRoot.startsWith("..") || fromRoot.startsWith(sep2)) {
     throw new Error(`Code review draft store must stay under ${root}: ${target}`);
@@ -42643,7 +42661,7 @@ async function assertDirectoryPath(directory) {
   }
 }
 function containedPath(directory, name) {
-  if (!name || isAbsolute3(name)) {
+  if (!name || isAbsolute4(name)) {
     throw new Error(`Code review path name is unsafe: ${name}`);
   }
   const root = resolve2(directory);
@@ -42742,7 +42760,8 @@ async function loadCodeReviewConfig(options) {
   return {
     ...agent2 ? { agent: agent2 } : {},
     draftStore,
-    humanGate: config.humanGate
+    humanGate: config.humanGate,
+    profileDirectories: config.profileDirectories
   };
 }
 async function validatePersistedCodeReviewConfig(options) {
@@ -42787,6 +42806,9 @@ async function resolveCodeReviewRunOptions(input, configOptions) {
     ...agent2 ? { agent: agent2 } : {},
     draftStore,
     humanGate: input.humanGate ?? config.humanGate,
+    profileDirectories: parseCodeReviewProfileDirectories(
+      input.profileDirectories ?? config.profileDirectories
+    ),
     ...input.profilePath === void 0 ? {} : { profilePath: input.profilePath },
     ...input.promptPath === void 0 ? {} : { promptPath: input.promptPath },
     ...input.profiles === void 0 ? {} : { profiles: input.profiles },
@@ -42794,12 +42816,18 @@ async function resolveCodeReviewRunOptions(input, configOptions) {
   };
 }
 async function resolveCodeReviewRuntimeOptions(input) {
-  return resolveCodeReviewRunOptions(input, {
+  return resolveCodeReviewRunOptions(input, runtimeConfigOptions(input.cwd));
+}
+async function loadCodeReviewRuntimeConfig(cwd) {
+  return loadCodeReviewConfig(runtimeConfigOptions(cwd));
+}
+function runtimeConfigOptions(cwd) {
+  return {
     fs: nativeConfigFs,
     filePath: resolveConfigPath(homedir4()),
-    projectFilePath: resolveProjectConfigPath(input.cwd),
+    projectFilePath: resolveProjectConfigPath(cwd),
     env: process.env
-  });
+  };
 }
 async function loadDefaultPoeCodeAgent(input) {
   const agent2 = await createConfigStore({
@@ -42896,8 +42924,40 @@ async function loadCodeReviewPrompt(filePath, reader) {
   return requireMarkdownBody(parsed.content, filePath);
 }
 async function discoverCodeReviewProfiles(input) {
-  const profilesDirectory = join2(codeReviewAssetsDirectory(input.cwd), "profiles");
-  await assertContainedAssetDirectoryOrMissing(resolve3(input.cwd), profilesDirectory);
+  const cwd = resolve3(input.cwd);
+  const directories = [
+    { path: join2(codeReviewAssetsDirectory(cwd), "profiles"), source: "repo", root: cwd },
+    ...parseCodeReviewProfileDirectories(input.profileDirectories ?? []).map((directory) => ({
+      path: resolve3(directory),
+      source: "external",
+      root: resolve3(directory)
+    }))
+  ];
+  const profiles = /* @__PURE__ */ new Map();
+  const normalizedNames = /* @__PURE__ */ new Map();
+  for (const directory of directories) {
+    for (const profile of await discoverProfilesInDirectory(directory)) {
+      const normalized = profile.name.normalize("NFKC").toLowerCase();
+      const existingName = normalizedNames.get(normalized);
+      if (existingName && existingName !== profile.name) {
+        throw new Error(`Code review profile filenames normalize to the same name: ${profile.name}`);
+      }
+      normalizedNames.set(normalized, profile.name);
+      if (!profiles.has(profile.name)) profiles.set(profile.name, profile);
+    }
+  }
+  const availableNames = [...profiles.keys()];
+  if (availableNames.length === 0) {
+    validateProfileFilters(input.filters, ["generic"]);
+    return [{ name: "generic", content: BUILT_IN_GENERIC_PROFILE, source: "built-in" }];
+  }
+  validateProfileFilters(input.filters, availableNames);
+  const filterSet = input.filters?.length ? new Set(input.filters) : void 0;
+  return [...profiles.values()].filter((profile) => !filterSet || filterSet.has(profile.name));
+}
+async function discoverProfilesInDirectory(input) {
+  const profilesDirectory = input.path;
+  await assertContainedAssetDirectoryOrMissing(input.root, profilesDirectory);
   let profileFileNames;
   try {
     const profileEntries = await readdir10(profilesDirectory, {
@@ -42915,40 +42975,18 @@ async function discoverCodeReviewProfiles(input) {
     }
     profileFileNames = [];
   }
-  const availableNames = profileFileNames.map(
-    (fileName) => requireSafeDocumentSegment(
-      profileNameFromFile(fileName),
-      `${join2(profilesDirectory, fileName)}: filename`
-    )
-  );
-  const normalizedNames = /* @__PURE__ */ new Set();
-  for (const name of availableNames) {
-    const normalized = name.normalize("NFKC").toLowerCase();
-    if (normalizedNames.has(normalized)) {
-      throw new Error(`Code review profile filenames normalize to the same name: ${name}`);
-    }
-    normalizedNames.add(normalized);
-  }
-  if (availableNames.length === 0) {
-    validateProfileFilters(input.filters, ["generic"]);
-    return [
-      {
-        name: "generic",
-        content: BUILT_IN_GENERIC_PROFILE,
-        source: "built-in"
-      }
-    ];
-  }
-  validateProfileFilters(input.filters, availableNames);
-  const filterSet = input.filters?.length ? new Set(input.filters) : void 0;
   return Promise.all(
-    profileFileNames.filter((fileName) => !filterSet || filterSet.has(profileNameFromFile(fileName))).map(async (fileName) => {
+    profileFileNames.map(async (fileName) => {
+      const name = requireSafeDocumentSegment(
+        profileNameFromFile(fileName),
+        `${join2(profilesDirectory, fileName)}: filename`
+      );
       const filePath = join2(profilesDirectory, fileName);
       return {
-        name: profileNameFromFile(fileName),
+        name,
         content: await loadCodeReviewProfile(filePath),
         filePath,
-        source: "repo"
+        source: input.source
       };
     })
   );
@@ -43023,6 +43061,16 @@ async function ensureContainedDirectory(cwd, targetDirectory) {
   }
 }
 async function assertContainedAssetDirectoryOrMissing(cwd, targetDirectory) {
+  let rootStatus;
+  try {
+    rootStatus = await lstat8(cwd);
+  } catch (error2) {
+    if (isMissingFileError4(error2)) return;
+    throw error2;
+  }
+  if (!rootStatus.isDirectory()) {
+    throw new Error(`Code review asset directory is not a regular directory: ${cwd}`);
+  }
   const pathFromCwd = relative3(cwd, targetDirectory);
   if (pathFromCwd.startsWith("..") || pathFromCwd.startsWith(sep3)) {
     throw new Error(`Code review asset directory escapes repository: ${targetDirectory}`);
@@ -47608,7 +47656,8 @@ function parseCodeReviewAgentMcpArgs(argv) {
     "--cwd",
     "--draft-store",
     "--agent",
-    "--profiles"
+    "--profiles",
+    "--profile-directories"
   ]);
   for (let index = 0; index < argv.length; index += 2) {
     const flag = argv[index];
@@ -47629,6 +47678,7 @@ function parseCodeReviewAgentMcpArgs(argv) {
     throw new Error(`Invalid code-review MCP role: ${role}`);
   }
   const profiles = values.get("--profiles")?.split(",");
+  const profileDirectories = values.has("--profile-directories") ? parseCodeReviewProfileDirectories(JSON.parse(requiredValue(values, "--profile-directories"))) : void 0;
   if (values.has("--profiles") && profiles?.length === 0) {
     throw new Error("--profiles requires at least one profile");
   }
@@ -47639,6 +47689,7 @@ function parseCodeReviewAgentMcpArgs(argv) {
     cwd: requiredValue(values, "--cwd"),
     ...values.has("--draft-store") ? { draftStore: requiredValue(values, "--draft-store") } : {},
     agent: requiredValue(values, "--agent"),
+    ...profileDirectories ? { profileDirectories } : {},
     ...profiles && profiles.length > 0 ? {
       profiles: [
         ...new Set(profiles.map((profile) => requireSafeDocumentSegment(profile, "--profiles")))
@@ -47664,6 +47715,9 @@ function createCodeReviewAgentMcpConfig(context) {
   if (context.profiles?.length) {
     args.push("--profiles", context.profiles.join(","));
   }
+  if (context.profileDirectories?.length) {
+    args.push("--profile-directories", JSON.stringify(context.profileDirectories));
+  }
   return { transport: "stdio", command: "poe-code", args: ["code-review", ...args] };
 }
 function createCodeReviewAgentMcpGroup(context, dependencies = {}) {
@@ -47756,7 +47810,8 @@ function createCodeReviewAgentMcpGroup(context, dependencies = {}) {
     params: S.Object({}),
     handler: async () => discoverCodeReviewProfiles({
       cwd: context.cwd,
-      filters: context.profiles
+      filters: context.profiles,
+      profileDirectories: context.profileDirectories
     })
   });
   const agentSpawnCommand = defineCommand({
@@ -47772,7 +47827,8 @@ function createCodeReviewAgentMcpGroup(context, dependencies = {}) {
       const pr = canonicalPullRequestUrl(params.pr);
       const profiles = await discoverCodeReviewProfiles({
         cwd: context.cwd,
-        filters: context.profiles
+        filters: context.profiles,
+        profileDirectories: context.profileDirectories
       });
       const profile = profiles.find((candidate) => candidate.name === params.profile);
       if (!profile) {
@@ -48093,7 +48149,8 @@ async function runCodeReview(input, dependencies = {}) {
   }
   const profiles = await discoverCodeReviewProfiles({
     cwd: options.cwd,
-    filters: options.profiles
+    filters: options.profiles,
+    profileDirectories: options.profileDirectories
   });
   const [profile, promptTemplate, prDetails, diff2, priorActivity] = await Promise.all([
     resolveProfile(input, options, profiles),
@@ -48132,6 +48189,7 @@ async function runCodeReview(input, dependencies = {}) {
     cwd: options.cwd,
     draftStore,
     agent: agent2,
+    profileDirectories: options.profileDirectories,
     ...options.profiles ? { profiles: options.profiles } : {}
   });
   const spawnResult = await (dependencies.spawnAgent ?? spawnWithPoeCode3)(agent2, prompt, {
@@ -48249,7 +48307,8 @@ var agentMcpCommand = defineCommand({
     cwd: S.String({ description: "Repository working directory." }),
     draftStore: S.Optional(S.String({ description: "Absolute YAML review state directory." })),
     agent: S.String({ description: "Poe Code agent used for subagents." }),
-    profiles: S.Optional(S.String({ description: "Comma-separated allowed profile names." }))
+    profiles: S.Optional(S.String({ description: "Comma-separated allowed profile names." })),
+    profileDirectories: S.Optional(S.String({ description: "JSON array of external profile directories." }))
   }),
   scope: ["cli"],
   handler: async ({ params }) => runCodeReviewAgentMcp(
@@ -48265,7 +48324,8 @@ var agentMcpCommand = defineCommand({
       ...params.draftStore ? ["--draft-store", params.draftStore] : [],
       "--agent",
       params.agent,
-      ...params.profiles ? ["--profiles", params.profiles] : []
+      ...params.profiles ? ["--profiles", params.profiles] : [],
+      ...params.profileDirectories ? ["--profile-directories", params.profileDirectories] : []
     ])
   )
 });
@@ -48288,18 +48348,22 @@ var installCodeReviewAssetsCommand = defineCommand({
 });
 var listCodeReviewProfilesCommand = defineCommand({
   name: "profiles",
-  description: "List repo-local code review profiles.",
+  description: "List configured code review profiles.",
   params: S.Object({
     cwd: S.Optional(S.String({ description: "Repository root directory." }))
   }),
   scope: ["cli"],
-  handler: async ({ params }) => (await discoverCodeReviewProfiles({ cwd: params.cwd?.trim() || process.cwd() })).map(
-    ({ name, source, filePath }) => ({
-      name,
-      source,
-      ...filePath ? { filePath } : {}
-    })
-  )
+  handler: async ({ params }) => {
+    const cwd = params.cwd?.trim() || process.cwd();
+    const config = await loadCodeReviewRuntimeConfig(cwd);
+    return (await discoverCodeReviewProfiles({ cwd, profileDirectories: config.profileDirectories })).map(
+      ({ name, source, filePath }) => ({
+        name,
+        source,
+        ...filePath ? { filePath } : {}
+      })
+    );
+  }
 });
 var readCodeReviewDraftCommand = defineCommand({
   name: "drafts",