poe-code 3.0.268 → 3.0.269

package/dist/providers/poe-agent.js

@@ -44455,6 +44455,246 @@ init_src9();
 import { lstat as lstat8, mkdir as mkdir10, readFile as readFile11, readdir as readdir9, rename as rename6, stat as stat7, unlink as unlink6, writeFile as writeFile6 } from "node:fs/promises";
 import { homedir as homedir4 } from "node:os";
 
+// packages/agent-code-review/src/document-schemas.ts
+import { basename as basename2 } from "node:path";
+import { parse as load, stringify as stringify3 } from "yaml";
+var CODE_REVIEW_PROMPT_ROLES = [
+  "orchestrator",
+  "subagent",
+  "agent",
+  "profile-synthesis"
+];
+var FRONTMATTER_RE = /^---\r?\n([\s\S]*?)\r?\n---(?:\r?\n|$)([\s\S]*)$/;
+var SAFE_SEGMENT_RE = /^[A-Za-z0-9._-]+$/;
+var SAFE_GITHUB_ACTOR_RE = /^[A-Za-z0-9](?:[A-Za-z0-9-]{0,37}[A-Za-z0-9])?$/;
+function requireSafeDocumentSegment(value, field) {
+  if (typeof value !== "string" || value.trim() !== value || value.normalize("NFKC") !== value || !SAFE_SEGMENT_RE.test(value) || value.startsWith(".") || value === "." || value === "..") {
+    throw new Error(`${field} must be a safe path segment.`);
+  }
+  return value;
+}
+function requireGitHubActorName(value, field) {
+  if (typeof value !== "string" || !SAFE_GITHUB_ACTOR_RE.test(value)) {
+    throw new Error(`${field} must be a safe GitHub actor name.`);
+  }
+  return value;
+}
+function parseCodeReviewProfileMarkdown(content, filePath) {
+  const parsed = parseOptionalFrontmatter(content, filePath);
+  if (!parsed.frontmatter) {
+    return { content: parsed.body };
+  }
+  const metadata = requireMapping(parsed.frontmatter, filePath, "frontmatter");
+  requireOnlyFields(metadata, filePath, "frontmatter", ["version", "name"]);
+  requireVersion(metadata.version, filePath, "frontmatter.version");
+  const name = requireSafeDocumentSegment(metadata.name, `${filePath}: frontmatter.name`);
+  if (basename2(filePath, ".md") !== name) {
+    throw invalidField(filePath, "frontmatter.name", "must match the filename");
+  }
+  return { content: parsed.body, metadata: { version: 1, name } };
+}
+function parseCodeReviewPromptMarkdown(content, filePath, role) {
+  const parsed = parseOptionalFrontmatter(content, filePath);
+  if (!parsed.frontmatter) {
+    return { content: parsed.body };
+  }
+  const metadata = requireMapping(parsed.frontmatter, filePath, "frontmatter");
+  requireOnlyFields(metadata, filePath, "frontmatter", ["version", "role"]);
+  requireVersion(metadata.version, filePath, "frontmatter.version");
+  if (!CODE_REVIEW_PROMPT_ROLES.includes(metadata.role)) {
+    throw invalidField(filePath, "frontmatter.role", "is not a supported role");
+  }
+  const promptRole = metadata.role;
+  if (role !== void 0 && promptRole !== role) {
+    throw invalidField(filePath, "frontmatter.role", `must equal ${role}`);
+  }
+  return { content: parsed.body, metadata: { version: 1, role: promptRole } };
+}
+function parseCodeReviewIngestSource(content, filePath = "code-review ingest source.yaml") {
+  try {
+    const source = requireMapping(load(content), filePath, "document");
+    requireOnlyFields(source, filePath, "document", [
+      "version",
+      "username",
+      "repos",
+      "fetched_at",
+      "output_profile_path",
+      "pagination",
+      "rate_limit"
+    ]);
+    requireVersion(source.version, filePath, "version");
+    const username = requireGitHubActorName(source.username, `${filePath}: username`);
+    if (!Array.isArray(source.repos) || source.repos.length === 0) {
+      throw invalidField(filePath, "repos", "must be a non-empty array");
+    }
+    const repos = source.repos.map((repo, index) => requireRepo(repo, filePath, `repos[${index}]`));
+    const fetchedAt = requireDate(source.fetched_at, filePath, "fetched_at");
+    if (typeof source.output_profile_path !== "string" || !source.output_profile_path) {
+      throw invalidField(filePath, "output_profile_path", "must be a non-empty string");
+    }
+    const pagination = requireMapping(source.pagination, filePath, "pagination");
+    requireOnlyFields(pagination, filePath, "pagination", [
+      "partial",
+      "comments_written",
+      "resume_endpoint"
+    ]);
+    if (typeof pagination.partial !== "boolean") {
+      throw invalidField(filePath, "pagination.partial", "must be a boolean");
+    }
+    const commentsWritten = requireNonNegativeInteger(
+      pagination.comments_written,
+      filePath,
+      "pagination.comments_written"
+    );
+    const resumeEndpoint = optionalString(
+      pagination.resume_endpoint,
+      filePath,
+      "pagination.resume_endpoint"
+    );
+    let rateLimit = null;
+    if (source.rate_limit !== null) {
+      const rate = requireMapping(source.rate_limit, filePath, "rate_limit");
+      requireOnlyFields(rate, filePath, "rate_limit", [
+        "repo",
+        "limit",
+        "remaining",
+        "reset_at",
+        "retry_after",
+        "partial_results",
+        "reason"
+      ]);
+      const reason = optionalString(rate.reason, filePath, "rate_limit.reason");
+      if (reason !== void 0 && !["low_remaining", "primary", "secondary"].includes(
+        reason
+      )) {
+        throw invalidField(filePath, "rate_limit.reason", "is not supported");
+      }
+      rateLimit = {
+        repo: requireRepo(rate.repo, filePath, "rate_limit.repo"),
+        limit: rate.limit === null || rate.limit === void 0 ? null : requireNonNegativeInteger(rate.limit, filePath, "rate_limit.limit"),
+        remaining: rate.remaining === null ? null : requireNonNegativeInteger(rate.remaining, filePath, "rate_limit.remaining"),
+        resetAt: rate.reset_at === null ? null : requireDate(rate.reset_at, filePath, "rate_limit.reset_at"),
+        retryAfter: rate.retry_after === null || rate.retry_after === void 0 ? null : optionalString(rate.retry_after, filePath, "rate_limit.retry_after") ?? null,
+        partialResults: requireNonNegativeInteger(
+          rate.partial_results,
+          filePath,
+          "rate_limit.partial_results"
+        ),
+        reason: reason ?? "low_remaining"
+      };
+    }
+    return {
+      version: 1,
+      username,
+      repos,
+      fetchedAt,
+      outputProfilePath: source.output_profile_path,
+      pagination: {
+        partial: pagination.partial,
+        commentsWritten,
+        ...resumeEndpoint === void 0 ? {} : { resumeEndpoint }
+      },
+      rateLimit
+    };
+  } catch (error2) {
+    if (error2 instanceof Error && error2.message.startsWith(`${filePath}:`)) {
+      throw error2;
+    }
+    throw new Error(`${filePath}: invalid YAML: ${errorMessage(error2)}`);
+  }
+}
+function serializeCodeReviewIngestSource(source, filePath = "code-review ingest source.yaml") {
+  const content = stringify3(
+    {
+      version: source.version,
+      username: source.username,
+      repos: source.repos,
+      fetched_at: source.fetchedAt,
+      pagination: {
+        partial: source.pagination.partial,
+        comments_written: source.pagination.commentsWritten,
+        ...source.pagination.resumeEndpoint === void 0 ? {} : { resume_endpoint: source.pagination.resumeEndpoint }
+      },
+      rate_limit: source.rateLimit === null ? null : {
+        repo: source.rateLimit.repo,
+        limit: source.rateLimit.limit,
+        remaining: source.rateLimit.remaining,
+        reset_at: source.rateLimit.resetAt,
+        retry_after: source.rateLimit.retryAfter,
+        partial_results: source.rateLimit.partialResults,
+        reason: source.rateLimit.reason
+      },
+      output_profile_path: source.outputProfilePath
+    },
+    { aliasDuplicateObjects: false, lineWidth: 0 }
+  );
+  parseCodeReviewIngestSource(content, filePath);
+  return content;
+}
+function parseOptionalFrontmatter(content, filePath) {
+  const match = content.match(FRONTMATTER_RE);
+  if (!match) {
+    return { body: content };
+  }
+  try {
+    return { frontmatter: load(match[1]), body: match[2] };
+  } catch (error2) {
+    throw new Error(`${filePath}: invalid frontmatter YAML: ${errorMessage(error2)}`);
+  }
+}
+function requireMapping(value, filePath, field) {
+  if (typeof value !== "object" || value === null || Array.isArray(value)) {
+    throw invalidField(filePath, field, "must be a YAML mapping");
+  }
+  return value;
+}
+function requireVersion(value, filePath, field) {
+  if (value !== 1) {
+    throw invalidField(filePath, field, "must equal 1");
+  }
+}
+function requireOnlyFields(value, filePath, field, allowedFields) {
+  const allowed = new Set(allowedFields);
+  for (const key2 of Object.keys(value)) {
+    if (!allowed.has(key2)) {
+      throw invalidField(filePath, `${field}.${key2}`, "is not supported");
+    }
+  }
+}
+function requireRepo(value, filePath, field) {
+  if (typeof value !== "string" || !/^[A-Za-z0-9._-]+\/[A-Za-z0-9._-]+$/.test(value)) {
+    throw invalidField(filePath, field, "must be a safe owner/repository name");
+  }
+  return value;
+}
+function requireDate(value, filePath, field) {
+  if (typeof value !== "string" || Number.isNaN(Date.parse(value))) {
+    throw invalidField(filePath, field, "must be a valid timestamp");
+  }
+  return value;
+}
+function requireNonNegativeInteger(value, filePath, field) {
+  if (!Number.isSafeInteger(value) || value < 0) {
+    throw invalidField(filePath, field, "must be a non-negative integer");
+  }
+  return value;
+}
+function optionalString(value, filePath, field) {
+  if (value === void 0) {
+    return void 0;
+  }
+  if (typeof value !== "string" || value.length === 0) {
+    throw invalidField(filePath, field, "must be a non-empty string");
+  }
+  return value;
+}
+function invalidField(filePath, field, reason) {
+  return new Error(`${filePath}: ${field} ${reason}.`);
+}
+function errorMessage(error2) {
+  return error2 instanceof Error ? error2.message : String(error2);
+}
+
 // packages/agent-code-review/src/error-codes.ts
 function hasOwnErrorCode14(error2, code) {
   return error2 instanceof Error && Object.prototype.hasOwnProperty.call(error2, "code") && error2.code === code;
@@ -45529,249 +45769,6 @@ async function* fetchReviewHistory(options) {
   }
 }
 
-// packages/agent-code-review/src/document-schemas.ts
-import { basename as basename2 } from "node:path";
-import { parse as load, stringify as stringify3 } from "yaml";
-var CODE_REVIEW_PROMPT_ROLES = [
-  "orchestrator",
-  "subagent",
-  "agent",
-  "profile-synthesis"
-];
-var FRONTMATTER_RE = /^---\r?\n([\s\S]*?)\r?\n---(?:\r?\n|$)([\s\S]*)$/;
-var SAFE_SEGMENT_RE = /^[A-Za-z0-9._-]+$/;
-var SAFE_GITHUB_ACTOR_RE = /^[A-Za-z0-9](?:[A-Za-z0-9-]{0,38})$/;
-function requireSafeDocumentSegment(value, field) {
-  if (typeof value !== "string" || value.trim() !== value || value.normalize("NFKC") !== value || !SAFE_SEGMENT_RE.test(value) || value.startsWith(".") || value === "." || value === "..") {
-    throw new Error(`${field} must be a safe path segment.`);
-  }
-  return value;
-}
-function requireGitHubActorName(value, field) {
-  if (typeof value !== "string" || !SAFE_GITHUB_ACTOR_RE.test(value)) {
-    throw new Error(`${field} must be a safe GitHub actor name.`);
-  }
-  return value;
-}
-function parseCodeReviewProfileMarkdown(content, filePath) {
-  const parsed = parseOptionalFrontmatter(content, filePath);
-  if (!parsed.frontmatter) {
-    return { content: parsed.body };
-  }
-  const metadata = requireMapping(parsed.frontmatter, filePath, "frontmatter");
-  requireOnlyFields(metadata, filePath, "frontmatter", ["version", "name"]);
-  requireVersion(metadata.version, filePath, "frontmatter.version");
-  const name = requireSafeDocumentSegment(metadata.name, `${filePath}: frontmatter.name`);
-  if (basename2(filePath, ".md") !== name) {
-    throw invalidField(filePath, "frontmatter.name", "must match the filename");
-  }
-  return { content: parsed.body, metadata: { version: 1, name } };
-}
-function parseCodeReviewPromptMarkdown(content, filePath, role) {
-  const parsed = parseOptionalFrontmatter(content, filePath);
-  if (!parsed.frontmatter) {
-    return { content: parsed.body };
-  }
-  const metadata = requireMapping(parsed.frontmatter, filePath, "frontmatter");
-  requireOnlyFields(metadata, filePath, "frontmatter", ["version", "role"]);
-  requireVersion(metadata.version, filePath, "frontmatter.version");
-  if (!CODE_REVIEW_PROMPT_ROLES.includes(metadata.role)) {
-    throw invalidField(filePath, "frontmatter.role", "is not a supported role");
-  }
-  const promptRole = metadata.role;
-  if (role !== void 0 && promptRole !== role) {
-    throw invalidField(filePath, "frontmatter.role", `must equal ${role}`);
-  }
-  return { content: parsed.body, metadata: { version: 1, role: promptRole } };
-}
-function parseCodeReviewIngestSource(content, filePath = "code-review ingest source.yaml") {
-  try {
-    const source = requireMapping(load(content), filePath, "document");
-    requireOnlyFields(source, filePath, "document", [
-      "version",
-      "username",
-      "repos",
-      "fetched_at",
-      "output_profile_path",
-      "pagination",
-      "rate_limit"
-    ]);
-    requireVersion(source.version, filePath, "version");
-    const username = requireGitHubActorName(source.username, `${filePath}: username`);
-    if (!Array.isArray(source.repos) || source.repos.length === 0) {
-      throw invalidField(filePath, "repos", "must be a non-empty array");
-    }
-    const repos = source.repos.map((repo, index) => requireRepo(repo, filePath, `repos[${index}]`));
-    const fetchedAt = requireDate(source.fetched_at, filePath, "fetched_at");
-    if (typeof source.output_profile_path !== "string" || !source.output_profile_path) {
-      throw invalidField(filePath, "output_profile_path", "must be a non-empty string");
-    }
-    const pagination = requireMapping(source.pagination, filePath, "pagination");
-    requireOnlyFields(pagination, filePath, "pagination", [
-      "partial",
-      "comments_written",
-      "resume_endpoint"
-    ]);
-    if (typeof pagination.partial !== "boolean") {
-      throw invalidField(filePath, "pagination.partial", "must be a boolean");
-    }
-    const commentsWritten = requireNonNegativeInteger(
-      pagination.comments_written,
-      filePath,
-      "pagination.comments_written"
-    );
-    const resumeEndpoint = optionalString(
-      pagination.resume_endpoint,
-      filePath,
-      "pagination.resume_endpoint"
-    );
-    let rateLimit = null;
-    if (source.rate_limit !== null) {
-      const rate = requireMapping(source.rate_limit, filePath, "rate_limit");
-      requireOnlyFields(rate, filePath, "rate_limit", [
-        "repo",
-        "limit",
-        "remaining",
-        "reset_at",
-        "retry_after",
-        "partial_results",
-        "reason"
-      ]);
-      const reason = optionalString(rate.reason, filePath, "rate_limit.reason");
-      if (reason !== void 0 && !["low_remaining", "primary", "secondary"].includes(
-        reason
-      )) {
-        throw invalidField(filePath, "rate_limit.reason", "is not supported");
-      }
-      rateLimit = {
-        repo: requireRepo(rate.repo, filePath, "rate_limit.repo"),
-        limit: rate.limit === null || rate.limit === void 0 ? null : requireNonNegativeInteger(rate.limit, filePath, "rate_limit.limit"),
-        remaining: rate.remaining === null ? null : requireNonNegativeInteger(rate.remaining, filePath, "rate_limit.remaining"),
-        resetAt: rate.reset_at === null ? null : requireDate(rate.reset_at, filePath, "rate_limit.reset_at"),
-        retryAfter: rate.retry_after === null || rate.retry_after === void 0 ? null : optionalString(rate.retry_after, filePath, "rate_limit.retry_after") ?? null,
-        partialResults: requireNonNegativeInteger(
-          rate.partial_results,
-          filePath,
-          "rate_limit.partial_results"
-        ),
-        reason: reason ?? "low_remaining"
-      };
-    }
-    return {
-      version: 1,
-      username,
-      repos,
-      fetchedAt,
-      outputProfilePath: source.output_profile_path,
-      pagination: {
-        partial: pagination.partial,
-        commentsWritten,
-        ...resumeEndpoint === void 0 ? {} : { resumeEndpoint }
-      },
-      rateLimit
-    };
-  } catch (error2) {
-    if (error2 instanceof Error && error2.message.startsWith(`${filePath}:`)) {
-      throw error2;
-    }
-    throw new Error(`${filePath}: invalid YAML: ${errorMessage(error2)}`);
-  }
-}
-function serializeCodeReviewIngestSource(source, filePath = "code-review ingest source.yaml") {
-  const content = stringify3(
-    {
-      version: source.version,
-      username: source.username,
-      repos: source.repos,
-      fetched_at: source.fetchedAt,
-      pagination: {
-        partial: source.pagination.partial,
-        comments_written: source.pagination.commentsWritten,
-        ...source.pagination.resumeEndpoint === void 0 ? {} : { resume_endpoint: source.pagination.resumeEndpoint }
-      },
-      rate_limit: source.rateLimit === null ? null : {
-        repo: source.rateLimit.repo,
-        limit: source.rateLimit.limit,
-        remaining: source.rateLimit.remaining,
-        reset_at: source.rateLimit.resetAt,
-        retry_after: source.rateLimit.retryAfter,
-        partial_results: source.rateLimit.partialResults,
-        reason: source.rateLimit.reason
-      },
-      output_profile_path: source.outputProfilePath
-    },
-    { aliasDuplicateObjects: false, lineWidth: 0 }
-  );
-  parseCodeReviewIngestSource(content, filePath);
-  return content;
-}
-function parseOptionalFrontmatter(content, filePath) {
-  const match = content.match(FRONTMATTER_RE);
-  if (!match) {
-    if (/^---\r?\n/.test(content)) {
-      throw new Error(`${filePath}: frontmatter is missing a closing --- delimiter.`);
-    }
-    return { body: content };
-  }
-  try {
-    return { frontmatter: load(match[1]), body: match[2] };
-  } catch (error2) {
-    throw new Error(`${filePath}: invalid frontmatter YAML: ${errorMessage(error2)}`);
-  }
-}
-function requireMapping(value, filePath, field) {
-  if (typeof value !== "object" || value === null || Array.isArray(value)) {
-    throw invalidField(filePath, field, "must be a YAML mapping");
-  }
-  return value;
-}
-function requireVersion(value, filePath, field) {
-  if (value !== 1) {
-    throw invalidField(filePath, field, "must equal 1");
-  }
-}
-function requireOnlyFields(value, filePath, field, allowedFields) {
-  const allowed = new Set(allowedFields);
-  for (const key2 of Object.keys(value)) {
-    if (!allowed.has(key2)) {
-      throw invalidField(filePath, `${field}.${key2}`, "is not supported");
-    }
-  }
-}
-function requireRepo(value, filePath, field) {
-  if (typeof value !== "string" || !/^[A-Za-z0-9._-]+\/[A-Za-z0-9._-]+$/.test(value)) {
-    throw invalidField(filePath, field, "must be a safe owner/repository name");
-  }
-  return value;
-}
-function requireDate(value, filePath, field) {
-  if (typeof value !== "string" || Number.isNaN(Date.parse(value))) {
-    throw invalidField(filePath, field, "must be a valid timestamp");
-  }
-  return value;
-}
-function requireNonNegativeInteger(value, filePath, field) {
-  if (!Number.isSafeInteger(value) || value < 0) {
-    throw invalidField(filePath, field, "must be a non-negative integer");
-  }
-  return value;
-}
-function optionalString(value, filePath, field) {
-  if (value === void 0) {
-    return void 0;
-  }
-  if (typeof value !== "string" || value.length === 0) {
-    throw invalidField(filePath, field, "must be a non-empty string");
-  }
-  return value;
-}
-function invalidField(filePath, field, reason) {
-  return new Error(`${filePath}: ${field} ${reason}.`);
-}
-function errorMessage(error2) {
-  return error2 instanceof Error ? error2.message : String(error2);
-}
-
 // packages/agent-code-review/src/review-state.ts
 import { parse as load2, stringify as stringify4 } from "yaml";
 function parseCodeReviewState(content, filePath = "code-review review YAML") {
@@ -46583,8 +46580,8 @@ async function withFileLock2(lockPath, lockTimeoutMs, operation) {
 }
 async function isStaleLock(lockPath, lockTimeoutMs) {
   try {
-    const ownerPid = Number.parseInt((await readFile10(lockPath, "utf8")).trim(), 10);
-    if (Number.isSafeInteger(ownerPid) && ownerPid > 0) {
+    const ownerPid = parseLockOwnerPid((await readFile10(lockPath, "utf8")).trim());
+    if (ownerPid !== void 0) {
       return !isRunningProcess(ownerPid);
     }
     return Date.now() - (await stat6(lockPath)).mtimeMs >= lockTimeoutMs;
@@ -46595,6 +46592,13 @@ async function isStaleLock(lockPath, lockTimeoutMs) {
     throw error2;
   }
 }
+function parseLockOwnerPid(value) {
+  const processId = Number(value);
+  if (!Number.isSafeInteger(processId) || processId <= 0) {
+    return void 0;
+  }
+  return String(processId) === value ? processId : void 0;
+}
 function isRunningProcess(processId) {
   try {
     process.kill(processId, 0);
@@ -46839,7 +46843,7 @@ async function resolveCodeReviewRunOptions(input, configOptions) {
     ),
     ...inputProfilePath === void 0 ? {} : { profilePath: inputProfilePath },
     ...inputPromptPath === void 0 ? {} : { promptPath: inputPromptPath },
-    ...inputProfiles === void 0 ? {} : { profiles: inputProfiles },
+    ...inputProfiles === void 0 ? {} : { profiles: requireProfileFilters(inputProfiles) },
     ...inputAdditionalFeedback === void 0 ? {} : { additionalFeedback: inputAdditionalFeedback }
   };
 }
@@ -46892,6 +46896,18 @@ function requireNonEmptyString(value, field) {
   }
   return normalized;
 }
+function requireProfileFilters(value) {
+  if (!Array.isArray(value)) {
+    throw new Error("profiles must be an array of safe profile names.");
+  }
+  return value.map((profile) => {
+    try {
+      return requireSafeDocumentSegment(profile, "profiles");
+    } catch {
+      throw new Error("profiles must be an array of safe profile names.");
+    }
+  });
+}
 function isMissingFileError5(error2) {
   return hasOwnErrorCode14(error2, "ENOENT");
 }
@@ -52506,11 +52522,17 @@ function shouldUseTextStdinForCodeReview(agent2) {
 var CODE_REVIEW_AGENT_MCP_ROLES = ["agent", "orchestrator", "subagent"];
 var inlineCommentSchema = S.Object({
   path: S.String({ description: "Repository-relative path in the PR diff." }),
-  line: S.Number({ description: "Right-side line number in the PR diff." }),
+  line: S.Number({
+    description: "Right-side line number in the PR diff.",
+    jsonType: "integer",
+    minimum: 1
+  }),
   body: S.String({ description: "Inline review comment body." })
 });
 var inlineCommentIndexSchema = S.Number({
-  description: "Zero-based merged review inline comment index."
+  description: "Zero-based merged review inline comment index.",
+  jsonType: "integer",
+  minimum: 0
 });
 var prParam = S.String({ description: "GitHub pull request URL." });
 function parseCodeReviewAgentMcpArgs(argv) {
@@ -52632,7 +52654,7 @@ function createCodeReviewAgentMcpGroup(context, dependencies = {}) {
     handler: async ({ params }) => {
       const pr = canonicalPullRequestUrl(params.pr);
       const draft = validateDraft2(params);
-      const currentState = await ensureState(store, context, pr);
+      const currentState = context.role === "orchestrator" ? await ensureState(store, context, pr) : await requireState(store, context, pr);
       if (context.role === "orchestrator") {
         const unfinishedProfiles = Object.values(currentState.subagents).filter(({ status }) => status === "pending" || status === "running").map(({ profile }) => profile);
         if (unfinishedProfiles.length > 0) {
@@ -52827,7 +52849,11 @@ function createCodeReviewAgentMcpGroup(context, dependencies = {}) {
       pr: prParam,
       index: inlineCommentIndexSchema,
       path: S.String({ description: "Repository-relative path in the PR diff." }),
-      line: S.Number({ description: "Right-side line number in the PR diff." }),
+      line: S.Number({
+        description: "Right-side line number in the PR diff.",
+        jsonType: "integer",
+        minimum: 1
+      }),
       body: S.String({ description: "Inline review comment body." })
     }),
     handler: async ({ params }) => {