poe-code 3.0.236 → 3.0.237

package/dist/index.js

@@ -47493,6 +47493,7 @@ var init_src23 = __esm({
 });
 
 // packages/agent-code-review/src/config-scope.ts
+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");
@@ -47509,13 +47510,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`);
     }
   }
@@ -47528,7 +47539,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;
@@ -47552,6 +47564,12 @@ var init_config_scope2 = __esm({
         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."
       }
     });
   }
@@ -49331,14 +49349,14 @@ import {
   stat as stat6,
   unlink as unlink4
 } from "node:fs/promises";
-import { basename as basename3, dirname, isAbsolute as isAbsolute3, parse as parse6, relative as relative2, resolve as resolve2, sep as sep2 } from "node:path";
+import { basename as basename3, dirname, isAbsolute as isAbsolute4, parse as parse6, relative as relative2, resolve as resolve2, sep as sep2 } from "node:path";
 function codeReviewFileName(prUrl) {
   const ref = requirePullRequestRef2(prUrl);
   return `${safeFilePart(ref.owner)}_${safeFilePart(ref.repo)}_PR${ref.number}.yaml`;
 }
 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}`);
@@ -49504,7 +49522,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);
@@ -49932,7 +49950,8 @@ async function loadCodeReviewConfig(options) {
   return {
     ...agent3 ? { agent: agent3 } : {},
     draftStore,
-    humanGate: config.humanGate
+    humanGate: config.humanGate,
+    profileDirectories: config.profileDirectories
   };
 }
 async function validatePersistedCodeReviewConfig(options) {
@@ -49977,6 +49996,9 @@ async function resolveCodeReviewRunOptions(input, configOptions) {
     ...agent3 ? { agent: agent3 } : {},
     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 },
@@ -49984,12 +50006,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 agent3 = await createConfigStore({
@@ -50089,8 +50117,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, {
@@ -50108,40 +50168,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
       };
     })
   );
@@ -50216,6 +50254,16 @@ async function ensureContainedDirectory(cwd, targetDirectory) {
   }
 }
 async function assertContainedAssetDirectoryOrMissing(cwd, targetDirectory) {
+  let rootStatus;
+  try {
+    rootStatus = await lstat8(cwd);
+  } catch (error3) {
+    if (isMissingFileError4(error3)) return;
+    throw error3;
+  }
+  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}`);
@@ -50365,6 +50413,7 @@ var init_assets = __esm({
   "packages/agent-code-review/src/assets.ts"() {
     "use strict";
     init_document_schemas();
+    init_config_scope2();
     CODE_REVIEW_USER_FACING_OUTPUT_CONTRACT = "In user-facing review/profile output, do not mention dry-run, fake-submit, orchestration, subagents, internal tool flow, source GitHub usernames, source URLs, or generation details.";
     BUILT_IN_GENERIC_PROFILE = `# Generic review profile
 
@@ -55167,7 +55216,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];
@@ -55188,6 +55238,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");
   }
@@ -55198,6 +55249,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")))
@@ -55223,6 +55275,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 = {}) {
@@ -55315,7 +55370,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({
@@ -55331,7 +55387,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) {
@@ -55642,6 +55699,7 @@ var init_mcp3 = __esm({
     init_document_schemas();
     init_prompt_transport2();
     init_review_store();
+    init_config_scope2();
     CODE_REVIEW_AGENT_MCP_ROLES = ["agent", "orchestrator", "subagent"];
     inlineCommentSchema = S.Object({
       path: S.String({ description: "Repository-relative path in the PR diff." }),
@@ -55675,7 +55733,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),
@@ -55714,6 +55773,7 @@ async function runCodeReview(input, dependencies = {}) {
     cwd: options.cwd,
     draftStore,
     agent: agent3,
+    profileDirectories: options.profileDirectories,
     ...options.profiles ? { profiles: options.profiles } : {}
   });
   const spawnResult = await (dependencies.spawnAgent ?? spawnWithPoeCode3)(agent3, prompt, {
@@ -55918,6 +55978,7 @@ var init_cli = __esm({
     init_mcp3();
     init_review2();
     init_review_store();
+    init_config4();
     agentMcpCommand = defineCommand({
       name: "agent-mcp",
       description: "Run the stdio MCP server used by code review agents.",
@@ -55930,7 +55991,8 @@ var init_cli = __esm({
         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(
@@ -55946,7 +56008,8 @@ var init_cli = __esm({
           ...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] : []
         ])
       )
     });
@@ -55969,18 +56032,22 @@ var init_cli = __esm({
     });
     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 } : {}
+          })
+        );
+      }
     });
     readCodeReviewDraftCommand = defineCommand({
       name: "drafts",
@@ -72063,9 +72130,9 @@ var init_registry5 = __esm({
 
 // packages/agent-eval/src/source/open.ts
 import nodeFs13 from "node:fs/promises";
-import { isAbsolute as isAbsolute4 } from "node:path";
+import { isAbsolute as isAbsolute5 } from "node:path";
 async function openSource(dir, fs27 = nodeFs13) {
-  if (!isAbsolute4(dir)) {
+  if (!isAbsolute5(dir)) {
     throw new Error(`Eval source path must be absolute, received "${dir}".`);
   }
   let stat33;
@@ -120208,7 +120275,7 @@ var init_fail = __esm({
 // packages/agent-script/src/modules/git.ts
 import { execFile as execFile3 } from "node:child_process";
 import { mkdir as mkdir34, realpath as realpath9, rm as rm12 } from "node:fs/promises";
-import { basename as basename7, dirname as dirname10, isAbsolute as isAbsolute5, relative as relative5, resolve as resolve8, sep as sep6 } from "node:path";
+import { basename as basename7, dirname as dirname10, isAbsolute as isAbsolute6, relative as relative5, resolve as resolve8, sep as sep6 } from "node:path";
 function makeGitModule(cwd) {
   const normalizedCwd = readNonEmptyString6(cwd, "Git module cwd");
   return {
@@ -120424,11 +120491,11 @@ function parseBranchRef(ref) {
 }
 async function resolveWorktreePath(repoRoot, path154, label) {
   const resolvedRoot = resolve8(repoRoot);
-  const resolvedPath = isAbsolute5(path154) ? resolve8(path154) : resolve8(resolvedRoot, path154);
+  const resolvedPath = isAbsolute6(path154) ? resolve8(path154) : resolve8(resolvedRoot, path154);
   const canonicalRoot = await realpath9(resolvedRoot);
   const canonicalPath = await resolveCanonicalPath(resolvedPath);
   const relativePath = relative5(canonicalRoot, canonicalPath);
-  if (relativePath.length === 0 || relativePath === ".." || relativePath.startsWith(`..${sep6}`) || isAbsolute5(relativePath)) {
+  if (relativePath.length === 0 || relativePath === ".." || relativePath.startsWith(`..${sep6}`) || isAbsolute6(relativePath)) {
     throw new Error(`${label} must be inside the git repository.`);
   }
   return canonicalPath;
@@ -122930,7 +122997,7 @@ var init_package2 = __esm({
   "package.json"() {
     package_default2 = {
       name: "poe-code",
-      version: "3.0.236",
+      version: "3.0.237",
       description: "CLI tool to configure Poe API for developer workflows.",
       type: "module",
       main: "./dist/index.js",