@plannotator/ui 0.46.0 → 0.47.0

package/utils/aiProvider.ts

@@ -21,6 +21,10 @@ export interface AIProviderOption {
   id: string;
   name: string;
   models?: AIProviderModel[];
+  /** Where `models` came from, when the server says (`/api/ai/capabilities`). */
+  modelsSource?: 'fallback' | 'discovered';
+  /** The installed CLI's version, when the server reports it. */
+  toolVersion?: string;
 }
 
 export interface AIProviderSettings {

package/utils/decisionSpec.ts

@@ -126,6 +126,15 @@ export interface DecisionPlatformInput {
   mrLabel: string;
   /** The viewer authored this PR/MR: approve paths mute, never disappear. */
   selfAuthored: boolean;
+  /**
+   * #1611: whether the platform has a real request-changes review (GitHub
+   * does, GitLab does not). `false` makes the request-changes rows say they
+   * post as a comment. Absent keeps the original copy, so a host that never
+   * sets it renders exactly what it rendered before. When supported, a
+   * self-authored PR mutes Request changes too (GitHub refuses it on your own
+   * PR) and adds a live "Comment…" row so the empty state is never a dead end.
+   */
+  requestChangesSupported?: boolean;
 }
 
 export const DECISION_NOTE_PLACEHOLDER = 'Add a note...';
@@ -345,6 +354,11 @@ function buildPlatformSpec(input: DecisionSpecInput, platform: DecisionPlatformI
   // Frozen copy (maintainer-approved): the self-approval mute reason.
   const selfReason = `You can't approve your own ${noun} on ${platform.label}.`;
   const selfReasonShort = `You can't approve your own ${platform.mrLabel}`;
+  const requestChangesUnsupported = platform.requestChangesSupported === false;
+  // GitHub refuses REQUEST_CHANGES on your own PR, exactly like APPROVE.
+  const requestChangesMuted = selfAuthored && platform.requestChangesSupported === true;
+  const selfRequestChangesShort = `You can't request changes on your own ${platform.mrLabel}`;
+  const postsAsComment = `Posts as a comment; ${platform.label} has no request-changes review`;
 
   if (!input.hasFeedback) {
     return {
@@ -374,11 +388,27 @@ function buildPlatformSpec(input: DecisionSpecInput, platform: DecisionPlatformI
           id: 'request-changes',
           // Frozen copy (maintainer-approved): 'Request changes…'.
           label: 'Request changes…',
-          subtitle: 'Overall feedback, zero line comments, via the dialog',
+          subtitle: requestChangesMuted
+            ? selfRequestChangesShort
+            : requestChangesUnsupported
+              ? postsAsComment
+              : 'Overall feedback, zero line comments, via the dialog',
           tone: 'primary',
           icon: 'send',
           dividerBefore: true,
+          ...(requestChangesMuted ? { muted: true } : {}),
         },
+        // With Request changes muted on your own PR, a neutral comment is the
+        // live path (the empty state is never a dead end).
+        ...(requestChangesMuted
+          ? [{
+              id: 'note-with-feedback' as const,
+              label: 'Comment…',
+              subtitle: 'Overall feedback as a neutral comment, via the dialog',
+              tone: 'primary' as const,
+              icon: 'send' as const,
+            }]
+          : []),
       ],
     };
   }
@@ -406,7 +436,11 @@ function buildPlatformSpec(input: DecisionSpecInput, platform: DecisionPlatformI
       {
         id: 'note-with-feedback',
         label: 'Post comments, then…',
-        subtitle: 'Request changes or stay neutral, chosen in the dialog',
+        subtitle: requestChangesUnsupported
+          ? postsAsComment
+          : requestChangesMuted
+            ? `Posts as a neutral comment; you can't request changes on your own ${platform.mrLabel}`
+            : 'Request changes or stay neutral, chosen in the dialog',
         tone: 'primary',
         icon: 'send',
         dividerBefore: true,