@plannotator/pi-extension 0.21.1 → 0.21.2

package/generated/agent-jobs.ts

@@ -76,12 +76,23 @@ export interface AgentJobInfo {
   diffScope?: string;
   /** Diff context at launch time (see AgentJobDiffContext). */
   diffContext?: AgentJobDiffContext;
+  /** Resolved review profile id at launch time (e.g. "builtin:default", "user:security"). */
+  reviewProfileId?: string;
+  /** Resolved review profile label — rides on findings so the UI can show a profile tag. */
+  reviewProfileLabel?: string;
 }
 
 export interface AgentCapability {
   id: string;
   name: string;
   available: boolean;
+  /**
+   * Provider-discovered model catalog (currently only Cursor). Best-effort and
+   * account-specific — populated from the provider CLI at capability-detection
+   * time, empty when discovery fails or the CLI is unauthenticated. The UI
+   * drives its model picker from this instead of a hardcoded list.
+   */
+  models?: { id: string; label: string }[];
 }
 
 export interface AgentCapabilities {
@@ -131,3 +142,16 @@ export function isTerminalStatus(status: AgentJobStatus): boolean {
 export function jobSource(id: string): string {
   return "agent-" + id.slice(0, 8);
 }
+
+// ---------------------------------------------------------------------------
+// Review ingestion completion semantics
+// ---------------------------------------------------------------------------
+
+/** Calm, provider-neutral failure reason. Never leak schema/CLI internals. */
+export const REVIEW_OUTPUT_FAILED = "Review finished but produced no usable findings.";
+
+/** Flip a job to failed with a calm one-liner (Code Tour precedent). */
+export function markJobReviewFailed(job: AgentJobInfo, error: string): void {
+  job.status = "failed";
+  job.error = error;
+}

package/generated/agent-review-message.ts

@@ -80,32 +80,45 @@ export function buildWorkspacePromptContextLines(
   ];
 }
 
-export function buildAgentReviewUserMessageForTarget(target: AgentReviewTarget): string {
+export function buildAgentReviewUserMessageForTarget(
+  target: AgentReviewTarget,
+  contextOnly = false,
+): string {
   if (target.kind === "workspace") {
-    return buildWorkspaceReviewUserMessage(target.patch, target.workspace);
+    return buildWorkspaceReviewUserMessage(target.patch, target.workspace, contextOnly);
   }
   return buildAgentReviewUserMessage(
     target.patch,
     target.diffType,
     target.options,
     target.kind === "pr" ? target.prMetadata : undefined,
+    contextOnly,
   );
 }
 
-/** Build the dynamic user message shared by local Claude and Codex review jobs. */
+/**
+ * Build the dynamic user message shared by local Claude and Codex review jobs.
+ *
+ * `contextOnly` strips the "Review… / provide findings" framing prose and keeps
+ * only the git/PR context the agent needs to locate the changes. Used by custom
+ * review skills, which carry their own instructions and must not inherit the
+ * default review's framing. With `contextOnly` off the output is byte-identical
+ * to today's prompt.
+ */
 export function buildAgentReviewUserMessage(
   patch: string,
   diffType: DiffType,
   options?: AgentReviewUserMessageOptions,
   prMetadata?: PRMetadata,
+  contextOnly = false,
 ): string {
   if (prMetadata) {
     if (options?.prDiffScope === "full-stack") {
       return [
-        `Full-stack review of ${prMetadata.url}`,
+        contextOnly ? prMetadata.url : `Full-stack review of ${prMetadata.url}`,
         "",
         "This is a stacked PR. The diff below shows ALL accumulated changes from the repository default branch through this PR's head (not just this PR's own layer).",
-        "Review the complete diff for issues that span the stack.",
+        ...(contextOnly ? [] : ["Review the complete diff for issues that span the stack."]),
         "",
         "```diff",
         patch,
@@ -113,6 +126,9 @@ export function buildAgentReviewUserMessage(
       ].join("\n");
     }
     if (options?.hasLocalAccess) {
+      // Pure context already (where the checkout is, how to diff against the
+      // base) — no "Review… / provide findings" framing to strip, so this is
+      // identical for default and custom reviews regardless of contextOnly.
       return [
         prMetadata.url,
         "",
@@ -126,11 +142,14 @@ export function buildAgentReviewUserMessage(
 
   const instruction = getLocalDiffInstruction(diffType, options?.defaultBranch);
   if (instruction) {
+    if (contextOnly) {
+      return `Changeset: ${instruction.target}.\n${instruction.inspect}`;
+    }
     return `Review ${instruction.target}. ${instruction.inspect} Provide prioritized, actionable findings.`;
   }
 
   return [
-    "Review the following code changes and provide prioritized findings.",
+    contextOnly ? "Code changes:" : "Review the following code changes and provide prioritized findings.",
     "",
     "```diff",
     patch,
@@ -141,10 +160,12 @@ export function buildAgentReviewUserMessage(
 function buildWorkspaceReviewUserMessage(
   patch: string,
   workspace: WorkspaceReviewPromptContext,
+  contextOnly = false,
 ): string {
   return [
-    "Review the local workspace changes across multiple nested VCS repositories.",
-    "",
+    ...(contextOnly
+      ? []
+      : ["Review the local workspace changes across multiple nested VCS repositories.", ""]),
     ...buildWorkspacePromptContextLines(workspace, { includeReportingInstruction: true }),
     "",
     "```diff",

package/generated/ai/providers/codex-sdk.ts

@@ -13,6 +13,8 @@
 
 import { buildSystemPrompt, buildEffectivePrompt } from "../context.ts";
 import { BaseSession } from "../base-session.ts";
+import { execFile } from "node:child_process";
+import { promisify } from "node:util";
 import type {
   AIProvider,
   AIProviderCapabilities,
@@ -29,6 +31,30 @@ import type {
 const PROVIDER_NAME = "codex-sdk";
 const DEFAULT_MODEL = "gpt-5.4";
 
+type GitWorkTreeProbe = (
+  command: string,
+  args: string[],
+  options: { encoding: "utf8" },
+) => Promise<{ stdout?: string | Buffer }>;
+
+const execFileAsync = promisify(execFile);
+
+export async function shouldSkipGitRepoCheck(
+  cwd: string,
+  probe: GitWorkTreeProbe = execFileAsync,
+): Promise<boolean> {
+  try {
+    const result = probe(
+      "git",
+      ["-C", cwd, "rev-parse", "--is-inside-work-tree"],
+      { encoding: "utf8" },
+    );
+    return String((await result).stdout ?? "").trim() !== "true";
+  } catch {
+    return true;
+  }
+}
+
 // ---------------------------------------------------------------------------
 // Provider
 // ---------------------------------------------------------------------------
@@ -58,10 +84,13 @@ export class CodexSDKProvider implements AIProvider {
   }
 
   async createSession(options: CreateSessionOptions): Promise<AISession> {
+    const cwd = options.cwd ?? this.config.cwd ?? process.cwd();
+    const skipGitRepoCheck = await shouldSkipGitRepoCheck(cwd);
     return new CodexSDKSession({
       ...this.baseConfig(options),
       systemPrompt: buildSystemPrompt(options.context),
-      cwd: options.cwd ?? this.config.cwd ?? process.cwd(),
+      cwd,
+      skipGitRepoCheck,
       parentSessionId: null,
     });
   }
@@ -74,10 +103,13 @@ export class CodexSDKProvider implements AIProvider {
   }
 
   async resumeSession(sessionId: string): Promise<AISession> {
+    const cwd = this.config.cwd ?? process.cwd();
+    const skipGitRepoCheck = await shouldSkipGitRepoCheck(cwd);
     return new CodexSDKSession({
       ...this.baseConfig(),
       systemPrompt: null,
-      cwd: this.config.cwd ?? process.cwd(),
+      cwd,
+      skipGitRepoCheck,
       parentSessionId: null,
       resumeThreadId: sessionId,
     });
@@ -124,6 +156,7 @@ interface SessionConfig {
   maxTurns: number;
   sandboxMode: "read-only" | "workspace-write" | "danger-full-access";
   cwd: string;
+  skipGitRepoCheck: boolean;
   parentSessionId: string | null;
   resumeThreadId?: string;
   codexExecutablePath?: string;
@@ -174,6 +207,7 @@ class CodexSDKSession extends BaseSession {
           this._thread = this._codexInstance.resumeThread(this.config.resumeThreadId, {
             model: this.config.model,
             workingDirectory: this.config.cwd,
+            skipGitRepoCheck: this.config.skipGitRepoCheck,
             sandboxMode: this.config.sandboxMode,
             ...(this.config.reasoningEffort && { modelReasoningEffort: this.config.reasoningEffort }),
           });
@@ -181,6 +215,7 @@ class CodexSDKSession extends BaseSession {
           this._thread = this._codexInstance.startThread({
             model: this.config.model,
             workingDirectory: this.config.cwd,
+            skipGitRepoCheck: this.config.skipGitRepoCheck,
             sandboxMode: this.config.sandboxMode,
             ...(this.config.reasoningEffort && { modelReasoningEffort: this.config.reasoningEffort }),
           });

package/generated/claude-review.ts

@@ -1,5 +1,14 @@
 // @generated — DO NOT EDIT. Source: packages/server/claude-review.ts
-import { toRelativePath } from "./path-utils.js";
+import {
+  composeReviewPrompt,
+  type ResolvedReviewProfile,
+} from "./review-profiles.js";
+import {
+  transformSeverityFindings,
+  type ReviewSeverity,
+  type ReviewFinding,
+  type ReviewAnnotationInput,
+} from "./review-findings";
 
 /**
  * Claude Code Review Agent — prompt, command builder, and JSONL output parser.
@@ -17,16 +26,10 @@ import { toRelativePath } from "./path-utils.js";
 // Types
 // ---------------------------------------------------------------------------
 
-export type ClaudeSeverity = "important" | "nit" | "pre_existing";
-
-export interface ClaudeFinding {
-  severity: ClaudeSeverity;
-  file: string;
-  line: number;
-  end_line: number;
-  description: string;
-  reasoning: string;
-}
+// Claude findings ARE review findings — reuse the one shared shape from
+// review-findings.ts rather than keeping a byte-identical copy that can drift.
+export type ClaudeSeverity = ReviewSeverity;
+export type ClaudeFinding = ReviewFinding;
 
 export interface ClaudeReviewOutput {
   findings: ClaudeFinding[];
@@ -50,9 +53,13 @@ export const CLAUDE_REVIEW_SCHEMA_JSON = JSON.stringify({
         type: "object",
         properties: {
           severity: { type: "string", enum: ["important", "nit", "pre_existing"] },
-          file: { type: "string" },
-          line: { type: "integer" },
-          end_line: { type: "integer" },
+          // Nullable, not omitted: keep every property in `required` so the
+          // schema is valid under strict structured-output validators too. A
+          // whole-file finding sets line/end_line null; a general finding also
+          // sets file null.
+          file: { type: ["string", "null"] },
+          line: { type: ["integer", "null"] },
+          end_line: { type: ["integer", "null"] },
           description: { type: "string" },
           reasoning: { type: "string" },
         },
@@ -162,6 +169,10 @@ Step 5: Deduplicate and rank
   - Within each severity, sort by file path and line number
 
 Step 6: Return structured JSON output matching the schema.
+  Place each finding by how specific it is: give file and line for a line-level
+  issue; give file and set line null for a whole-file issue; set file and line
+  null for a general, review-level note. Never invent a line you are unsure of —
+  drop to a file or general placement instead of guessing.
   If no issues are found, return an empty findings array with zeroed summary.
 
 ## Hard constraints
@@ -176,6 +187,23 @@ Step 6: Return structured JSON output matching the schema.
 - Do NOT use gh pr comment or any commenting tool
 - Your only output is the structured JSON findings`;
 
+// ---------------------------------------------------------------------------
+// Prompt composition
+// ---------------------------------------------------------------------------
+
+/**
+ * Compose Claude's review prompt: the immutable system prompt, the resolved
+ * profile's Custom Review Profile section (omitted for builtin:default), then
+ * the user review message. For builtin:default / no profile the output is
+ * byte-identical to today's `CLAUDE_REVIEW_PROMPT + "\n\n---\n\n" + userMessage`.
+ */
+export function composeClaudeReviewPrompt(
+  userMessage: string,
+  reviewProfile?: ResolvedReviewProfile,
+): string {
+  return composeReviewPrompt(CLAUDE_REVIEW_PROMPT, reviewProfile, userMessage);
+}
+
 // ---------------------------------------------------------------------------
 // Command builder
 // ---------------------------------------------------------------------------
@@ -284,36 +312,10 @@ export function transformClaudeFindings(
   source: string,
   cwd?: string,
   pathTransform?: (path: string) => string,
-): Array<{
-  source: string;
-  filePath: string;
-  lineStart: number;
-  lineEnd: number;
-  type: string;
-  side: string;
-  scope: string;
-  text: string;
-  severity: ClaudeSeverity;
-  reasoning: string;
-  author: string;
-}> {
-  return findings
-    .filter(f => f.file && typeof f.line === "number")
-    .map(f => ({
-      source,
-      filePath: pathTransform
-        ? pathTransform(toRelativePath(f.file, cwd))
-        : toRelativePath(f.file, cwd),
-      lineStart: f.line,
-      lineEnd: f.end_line ?? f.line,
-      type: "comment",
-      side: "new",
-      scope: "line",
-      text: `[${f.severity}] ${f.description}`,
-      severity: f.severity,
-      reasoning: f.reasoning,
-      author: "Claude Code",
-    }));
+): ReviewAnnotationInput[] {
+  // Routing (line / whole-file / general) is shared with the marker engines in
+  // review-findings.ts — nothing is dropped; only the author differs.
+  return transformSeverityFindings(findings, source, "Claude Code", cwd, pathTransform);
 }
 
 // ---------------------------------------------------------------------------

package/generated/codex-review.ts

@@ -12,6 +12,11 @@ import { appendFile, mkdir, unlink, writeFile, readFile } from "node:fs/promises
 import { existsSync } from "node:fs";
 import { toRelativePath } from "./path-utils.js";
 import { getPlannotatorDataDir } from "./data-dir";
+import {
+  composeReviewPrompt,
+  type ResolvedReviewProfile,
+} from "./review-profiles.js";
+import { classifyFindingPlacement } from "./external-annotation.js";
 
 // ---------------------------------------------------------------------------
 // Debug log — only active when PLANNOTATOR_DEBUG is set
@@ -39,7 +44,7 @@ async function debugLog(label: string, data?: unknown): Promise<void> {
 // can't read, so we materialize the schema to a real file.
 // ---------------------------------------------------------------------------
 
-const CODEX_REVIEW_SCHEMA = JSON.stringify({
+export const CODEX_REVIEW_SCHEMA = JSON.stringify({
   type: "object",
   properties: {
     findings: {
@@ -51,12 +56,15 @@ const CODEX_REVIEW_SCHEMA = JSON.stringify({
           body: { type: "string" },
           confidence_score: { type: "number" },
           priority: { type: ["integer", "null"] },
+          // Nullable, not omittable: OpenAI strict structured output requires
+          // every property to appear in `required`. A whole-file finding sets
+          // line_range to null; a general finding sets code_location to null.
           code_location: {
-            type: "object",
+            type: ["object", "null"],
             properties: {
               absolute_file_path: { type: "string" },
               line_range: {
-                type: "object",
+                type: ["object", "null"],
                 properties: {
                   start: { type: "integer" },
                   end: { type: "integer" },
@@ -152,6 +160,8 @@ At the beginning of the finding title, tag the bug with priority level. For exam
 
 Additionally, include a numeric priority field in the JSON output for each finding: set "priority" to 0 for P0, 1 for P1, 2 for P2, or 3 for P3. If a priority cannot be determined, omit the field or use null.
 
+Place each finding by how specific it is. For a line-level issue, set code_location with the file and a line_range. For a whole-file issue, set code_location with the file path and line_range null. For a general, review-wide point, set code_location null. Do not invent a line range you are unsure of — drop to a whole-file or general placement instead.
+
 At the end of your findings, output an "overall correctness" verdict of whether or not the patch should be considered "correct".
 Correct implies that existing code and tests will not break, and the patch is free of bugs and other blocking issues.
 Ignore non-blocking issues such as style, formatting, typos, documentation, and other nits.
@@ -159,6 +169,24 @@ Ignore non-blocking issues such as style, formatting, typos, documentation, and
 FORMATTING GUIDELINES:
 The finding description should be one paragraph.`;
 
+// ---------------------------------------------------------------------------
+// Prompt composition
+// ---------------------------------------------------------------------------
+
+/**
+ * Compose Codex's review prompt: the immutable system prompt, the resolved
+ * profile's Custom Review Profile section (omitted for builtin:default), then
+ * the user review message. For builtin:default / no profile the output is
+ * byte-identical to today's
+ * `CODEX_REVIEW_SYSTEM_PROMPT + "\n\n---\n\n" + userMessage`.
+ */
+export function composeCodexReviewPrompt(
+  userMessage: string,
+  reviewProfile?: ResolvedReviewProfile,
+): string {
+  return composeReviewPrompt(CODEX_REVIEW_SYSTEM_PROMPT, reviewProfile, userMessage);
+}
+
 // ---------------------------------------------------------------------------
 // Command builder
 // ---------------------------------------------------------------------------
@@ -213,7 +241,7 @@ export function generateOutputPath(): string {
 
 export interface CodexCodeLocation {
   absolute_file_path: string;
-  line_range: { start: number; end: number };
+  line_range?: { start: number; end: number } | null; // null for a whole-file comment
 }
 
 export interface CodexFinding {
@@ -221,7 +249,7 @@ export interface CodexFinding {
   body: string;
   confidence_score: number;
   priority: number | null;
-  code_location: CodexCodeLocation;
+  code_location?: CodexCodeLocation | null; // null for a general (review-level) comment
 }
 
 export interface CodexReviewOutput {
@@ -296,25 +324,27 @@ export function transformReviewFindings(
   author?: string,
   pathTransform?: (path: string) => string,
 ): ReviewAnnotationInput[] {
-  const annotations = findings
-    .filter((f) =>
-      f.code_location?.absolute_file_path &&
-      typeof f.code_location?.line_range?.start === "number" &&
-      typeof f.code_location?.line_range?.end === "number"
-    )
-    .map((f) => ({
+  // Route every finding by what it carries — nothing is dropped. No usable file
+  // becomes a general comment; a file without a line range, a whole-file comment.
+  const annotations = findings.map((f) => {
+    const loc = f.code_location;
+    const rawFile = loc && typeof loc.absolute_file_path === "string" ? loc.absolute_file_path : "";
+    const filePath = rawFile
+      ? (pathTransform ? pathTransform(toRelativePath(rawFile, cwd)) : toRelativePath(rawFile, cwd))
+      : "";
+    const placement = classifyFindingPlacement(filePath, loc?.line_range?.start, loc?.line_range?.end);
+    return {
       source,
-      filePath: pathTransform
-        ? pathTransform(toRelativePath(f.code_location.absolute_file_path, cwd))
-        : toRelativePath(f.code_location.absolute_file_path, cwd),
-      lineStart: f.code_location.line_range.start,
-      lineEnd: f.code_location.line_range.end,
+      filePath: placement.filePath,
+      lineStart: placement.lineStart,
+      lineEnd: placement.lineEnd,
       type: "comment",
       side: "new",
-      scope: "line",
+      scope: placement.scope,
       text: `${f.title}\n\n${f.body}`.trim(),
       author: author ?? "Review Agent",
-    }));
+    };
+  });
 
   debugLog("TRANSFORM_FINDINGS", {
     inputCount: findings.length,

package/generated/external-annotation.ts

@@ -191,14 +191,53 @@ interface ReviewAnnotation {
   createdAt: number;
   author?: string;
   source?: string;
-  // Agent review metadata (optional — only set by Claude review findings)
+  // Agent review metadata (optional — only set by agent review findings)
   severity?: string; // "important" | "nit" | "pre_existing"
   reasoning?: string; // Validation chain explaining how the issue was confirmed
+  reviewProfileLabel?: string; // Custom review profile that produced this finding
 }
 
 const VALID_REVIEW_TYPES = ["comment", "suggestion", "concern"];
 const VALID_SIDES = ["old", "new"];
-const VALID_SCOPES = ["line", "file"];
+const VALID_SCOPES = ["line", "file", "general"];
+
+/** A review finding's placement, derived from what it carries. */
+export type FindingPlacement = {
+  scope: "line" | "file" | "general";
+  filePath: string;
+  lineStart: number;
+  lineEnd: number;
+};
+
+/**
+ * Classify an agent review finding by what it carries, so nothing is dropped:
+ *   file + a usable line → a line comment
+ *   file, no line        → a whole-file comment
+ *   neither              → a general (review-level) comment
+ *
+ * For file and general placements the line is 0; for general the path is "".
+ * Consumers branch on `scope`, never on the sentinel values.
+ */
+export function classifyFindingPlacement(
+  filePath: string,
+  lineStart: number | null | undefined,
+  lineEnd: number | null | undefined,
+): FindingPlacement {
+  const hasFile = filePath.length > 0;
+  const hasLine = typeof lineStart === "number";
+  if (hasFile && hasLine) {
+    return {
+      scope: "line",
+      filePath,
+      lineStart,
+      lineEnd: typeof lineEnd === "number" ? lineEnd : lineStart,
+    };
+  }
+  if (hasFile) {
+    return { scope: "file", filePath, lineStart: 0, lineEnd: 0 };
+  }
+  return { scope: "general", filePath: "", lineStart: 0, lineEnd: 0 };
+}
 
 export function transformReviewInput(
   body: unknown,
@@ -213,14 +252,40 @@ export function transformReviewInput(
     const source = requireString(obj, "source", i);
     if (typeof source !== "string") return source;
 
-    const filePath = requireString(obj, "filePath", i);
-    if (typeof filePath !== "string") return filePath;
-
-    if (typeof obj.lineStart !== "number") {
-      return { error: `annotations[${i}] missing required "lineStart" field` };
+    // scope: optional, defaults to "line"
+    const scope = typeof obj.scope === "string" ? obj.scope : "line";
+    if (!VALID_SCOPES.includes(scope)) {
+      return {
+        error: `annotations[${i}] invalid scope "${scope}". Must be one of: ${VALID_SCOPES.join(", ")}`,
+      };
     }
-    if (typeof obj.lineEnd !== "number") {
-      return { error: `annotations[${i}] missing required "lineEnd" field` };
+
+    // Location requirements depend on scope:
+    //   line    → filePath + lineStart + lineEnd required. A finding that claims
+    //             a line must carry one, so a broken line finding is rejected
+    //             rather than quietly passing as a vaguer comment.
+    //   file    → filePath required; line ignored (defaults to 0).
+    //   general → no file, no line (review-level; defaults to "" / 0).
+    let filePath = "";
+    let lineStart = 0;
+    let lineEnd = 0;
+    if (scope !== "general") {
+      const fp = requireString(obj, "filePath", i);
+      if (typeof fp !== "string") return fp;
+      filePath = fp;
+      if (scope === "line") {
+        if (typeof obj.lineStart !== "number") {
+          return { error: `annotations[${i}] missing required "lineStart" field` };
+        }
+        if (typeof obj.lineEnd !== "number") {
+          return { error: `annotations[${i}] missing required "lineEnd" field` };
+        }
+        lineStart = obj.lineStart;
+        lineEnd = obj.lineEnd;
+      } else {
+        lineStart = typeof obj.lineStart === "number" ? obj.lineStart : 0;
+        lineEnd = typeof obj.lineEnd === "number" ? obj.lineEnd : 0;
+      }
     }
 
     // side: optional, defaults to "new"
@@ -239,14 +304,6 @@ export function transformReviewInput(
       };
     }
 
-    // scope: optional, defaults to "line"
-    const scope = typeof obj.scope === "string" ? obj.scope : "line";
-    if (!VALID_SCOPES.includes(scope)) {
-      return {
-        error: `annotations[${i}] invalid scope "${scope}". Must be one of: ${VALID_SCOPES.join(", ")}`,
-      };
-    }
-
     // Must have at least text or suggestedCode
     if (typeof obj.text !== "string" && typeof obj.suggestedCode !== "string") {
       return {
@@ -259,8 +316,8 @@ export function transformReviewInput(
       type,
       scope,
       filePath,
-      lineStart: obj.lineStart,
-      lineEnd: obj.lineEnd,
+      lineStart,
+      lineEnd,
       side,
       text: typeof obj.text === "string" ? obj.text : undefined,
       suggestedCode: typeof obj.suggestedCode === "string" ? obj.suggestedCode : undefined,
@@ -268,9 +325,10 @@ export function transformReviewInput(
       createdAt: Date.now(),
       author: typeof obj.author === "string" ? obj.author : undefined,
       source,
-      // Agent review metadata (optional — only set by Claude review findings)
+      // Agent review metadata (optional — only set by agent review findings)
       ...(typeof obj.severity === "string" && { severity: obj.severity }),
       ...(typeof obj.reasoning === "string" && { reasoning: obj.reasoning }),
+      ...(typeof obj.reviewProfileLabel === "string" && { reviewProfileLabel: obj.reviewProfileLabel }),
     });
   }