@skyhook-io/radar-app 1.13.2 → 1.13.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@skyhook-io/radar-app",
-  "version": "1.13.2",
+  "version": "1.13.3",
   "description": "Radar's full web UI as a reusable React component. Used by Radar's own binary and by external consumers like Radar Cloud.",
   "repository": {
     "type": "git",

package/src/api/diagnose.ts

@@ -13,6 +13,8 @@ export interface AgentInfo {
   profiles?: ExecutionProfile[];
   consentSurfaces?: Partial<Record<ExecutionProfile, string>>;
   hosted?: boolean;
+  /** Hosted backends opt in only when they implement assessment-bound, tool-free explanations. */
+  assessmentExplanations?: boolean;
 }
 
 export interface AgentsResponse {

package/src/components/diagnose/InvestigationView.tsx

@@ -8,7 +8,10 @@ import {
   prefersReducedMotion,
   useDisclosureReveal,
 } from "./useDisclosureReveal";
-import { investigationExplanation } from "./investigationExplanation";
+import {
+  investigationExplanation,
+  supportsAssessmentExplanation,
+} from "./investigationExplanation";
 import {
   initialInvestigationPane,
   investigationEvidenceShouldMarkUnread,
@@ -160,8 +163,17 @@ export function InvestigationView({
   const { kind, namespace, name } = run;
   // Apply is off for hosted agents (read-only server-side). Keyed on the selected
   // agent, which matches run.agent unless a deployment mixes hosted + local agents.
-  const { refreshRuns, openInvestigation, startError, dismissError, hosted } =
-    useDiagnose();
+  const {
+    refreshRuns,
+    openInvestigation,
+    startError,
+    dismissError,
+    hosted,
+    agents,
+  } = useDiagnose();
+  const explanationEnabled = supportsAssessmentExplanation(
+    agents.find((agent) => agent.name === run.agent),
+  );
   // Investigate again means look again, so it asks for a new session explicitly and only
   // carries the issue forward — being handed the previous answer is the one
   // thing someone clicking this doesn't want.
@@ -754,15 +766,14 @@ export function InvestigationView({
     const sequence = assessment.resultSequence;
     if (!sequence || !assessment.diagnosis?.rootCause) return undefined;
     const saved = investigationExplanation(turns, sequence);
-    // This intent is implemented by the local run manager; hosted continuation
-    // alone does not imply support for assessment-bound explanation turns.
-    if ((readOnly || hosted) && saved.status === "idle") return undefined;
+    if ((readOnly || !explanationEnabled) && saved.status === "idle")
+      return undefined;
     const state =
       explanationRequest?.sequence === sequence ? explanationRequest : saved;
     return {
       ...state,
       onGenerate:
-        !hosted && !interactionsBlocked
+        explanationEnabled && !interactionsBlocked
           ? () => askExplanation(sequence)
           : undefined,
       openRequest:

package/src/components/diagnose/investigationExplanation.test.ts

@@ -1,5 +1,9 @@
 import { describe, expect, it } from "vitest";
-import { investigationExplanation } from "./investigationExplanation";
+import {
+  investigationExplanation,
+  supportsAssessmentExplanation,
+} from "./investigationExplanation";
+import type { AgentInfo } from "../../api/diagnose";
 import type { Turn } from "./parts";
 
 const turn = (overrides: Partial<Turn> = {}): Turn => ({
@@ -10,6 +14,31 @@ const turn = (overrides: Partial<Turn> = {}): Turn => ({
   ...overrides,
 });
 describe("assessment-local explanation", () => {
+  it("requires explicit hosted support without enabling mutations or changing local support", () => {
+    const agent: AgentInfo = {
+      name: "hub",
+      label: "Managed agent",
+      path: "",
+      version: "",
+      supported: true,
+      present: true,
+      hosted: true,
+    };
+    expect(supportsAssessmentExplanation(undefined)).toBe(false);
+    expect(supportsAssessmentExplanation(agent)).toBe(false);
+    expect(
+      supportsAssessmentExplanation({
+        ...agent,
+        assessmentExplanations: false,
+      }),
+    ).toBe(false);
+    expect(
+      supportsAssessmentExplanation({ ...agent, assessmentExplanations: true }),
+    ).toBe(true);
+    expect(supportsAssessmentExplanation({ ...agent, hosted: false })).toBe(
+      true,
+    );
+  });
   it("does not mistake an ordinary answer for an explanation", () => {
     expect(
       investigationExplanation([turn({ question: "Explain simply" })], 2),

package/src/components/diagnose/investigationExplanation.ts

@@ -1,4 +1,11 @@
 import type { AssessmentExplanation, Turn } from "./parts";
+import type { AgentInfo } from "../../api/diagnose";
+
+export function supportsAssessmentExplanation(
+  agent: AgentInfo | undefined,
+): boolean {
+  return !!agent && (!agent.hosted || agent.assessmentExplanations === true);
+}
 
 export function investigationExplanation(
   turns: readonly Turn[],