pmx-canvas 0.2.7 → 0.3.1

package/src/server/agent-context.ts

@@ -56,11 +56,8 @@ function summarizeWebpageData(data: Record<string, unknown>, maxLength: number):
   const status = typeof data.status === 'string' ? data.status : '';
   const statusCode = typeof data.statusCode === 'number' ? data.statusCode : null;
   const error = typeof data.error === 'string' ? data.error : '';
-  const content = typeof data.content === 'string'
-    ? data.content
-    : typeof data.excerpt === 'string'
-      ? data.excerpt
-      : '';
+  const content =
+    typeof data.content === 'string' ? data.content : typeof data.excerpt === 'string' ? data.excerpt : '';
 
   if (url) parts.push(`URL: ${url}`);
   if (pageTitle) parts.push(`Title: ${pageTitle}`);
@@ -132,9 +129,7 @@ function summarizeWebArtifactData(data: Record<string, unknown>, maxLength: numb
   const sourceFiles = Array.isArray(data.sourceFiles)
     ? data.sourceFiles.filter((file): file is string => typeof file === 'string')
     : [];
-  const deps = Array.isArray(data.deps)
-    ? data.deps.filter((dep): dep is string => typeof dep === 'string')
-    : [];
+  const deps = Array.isArray(data.deps) ? data.deps.filter((dep): dep is string => typeof dep === 'string') : [];
 
   if (content) parts.push(content);
   if (!content && title) parts.push(`Web artifact: ${title}`);
@@ -170,7 +165,16 @@ function metadataForNode(node: CanvasNodeState): Record<string, unknown> | undef
   switch (node.type) {
     case 'webpage': {
       const metadata: Record<string, unknown> = {};
-      for (const key of ['url', 'pageTitle', 'description', 'imageUrl', 'fetchedAt', 'status', 'statusCode', 'contentType']) {
+      for (const key of [
+        'url',
+        'pageTitle',
+        'description',
+        'imageUrl',
+        'fetchedAt',
+        'status',
+        'statusCode',
+        'contentType',
+      ]) {
         const value = node.data[key];
         if (value !== undefined && value !== null && value !== '') metadata[key] = value;
       }
@@ -194,7 +198,19 @@ function metadataForNode(node: CanvasNodeState): Record<string, unknown> | undef
     }
     case 'html': {
       const metadata: Record<string, unknown> = {};
-      for (const key of ['summary', 'description', 'agentSummary', 'contentSummary', 'htmlPrimitive', 'presentation', 'slideCount', 'slideTitles', 'speakerNotes', 'embeddedNodeIds', 'embeddedUrls']) {
+      for (const key of [
+        'summary',
+        'description',
+        'agentSummary',
+        'contentSummary',
+        'htmlPrimitive',
+        'presentation',
+        'slideCount',
+        'slideTitles',
+        'speakerNotes',
+        'embeddedNodeIds',
+        'embeddedUrls',
+      ]) {
         const value = node.data[key];
         if (Array.isArray(value)) {
           if (value.length > 0) metadata[key] = value;
@@ -206,7 +222,22 @@ function metadataForNode(node: CanvasNodeState): Record<string, unknown> | undef
     }
     case 'mcp-app': {
       const metadata: Record<string, unknown> = {};
-      for (const key of ['url', 'path', 'mode', 'hostMode', 'viewerType', 'serverName', 'toolName', 'resourceUri', 'sessionStatus', 'projectPath', 'artifactBytes', 'sourceFiles', 'sourceFileCount', 'deps']) {
+      for (const key of [
+        'url',
+        'path',
+        'mode',
+        'hostMode',
+        'viewerType',
+        'serverName',
+        'toolName',
+        'resourceUri',
+        'sessionStatus',
+        'projectPath',
+        'artifactBytes',
+        'sourceFiles',
+        'sourceFileCount',
+        'deps',
+      ]) {
         const value = node.data[key];
         if (Array.isArray(value)) {
           if (value.length > 0) metadata[key] = value;
@@ -221,10 +252,7 @@ function metadataForNode(node: CanvasNodeState): Record<string, unknown> | undef
   }
 }
 
-export function summarizeNodeForAgentContext(
-  node: CanvasNodeState,
-  options: AgentContextOptions = {},
-): string {
+export function summarizeNodeForAgentContext(node: CanvasNodeState, options: AgentContextOptions = {}): string {
   const defaultTextLength = options.defaultTextLength ?? DEFAULT_CONTEXT_TEXT_LENGTH;
   const webpageTextLength = options.webpageTextLength ?? DEFAULT_WEBPAGE_CONTEXT_TEXT_LENGTH;
 
@@ -241,9 +269,7 @@ export function summarizeNodeForAgentContext(
       if (chartCfg) {
         const chartTitle = (chartCfg.title as string) || 'Untitled chart';
         const chartType = (chartCfg.type as string) || 'unknown';
-        const labels = Array.isArray(chartCfg.labels)
-          ? (chartCfg.labels as string[]).join(', ')
-          : '';
+        const labels = Array.isArray(chartCfg.labels) ? (chartCfg.labels as string[]).join(', ') : '';
         return truncateContextText(`Chart: ${chartTitle} (${chartType}). Labels: ${labels}`, defaultTextLength);
       }
       return summarizeMcpAppData(node.data, defaultTextLength);
@@ -263,14 +289,17 @@ export function summarizeNodeForAgentContext(
       if (typeof node.data.htmlPrimitive === 'string') {
         return summarizeHtmlPrimitiveData(node.data, defaultTextLength);
       }
-      return stringifyContextValue({
-        title: node.data.title,
-        description: node.data.description,
-        summary: node.data.summary,
-        contentSummary: node.data.contentSummary,
-        embeddedNodeIds: node.data.embeddedNodeIds,
-        embeddedUrls: node.data.embeddedUrls,
-      }, defaultTextLength);
+      return stringifyContextValue(
+        {
+          title: node.data.title,
+          description: node.data.description,
+          summary: node.data.summary,
+          contentSummary: node.data.contentSummary,
+          embeddedNodeIds: node.data.embeddedNodeIds,
+          embeddedUrls: node.data.embeddedUrls,
+        },
+        defaultTextLength,
+      );
     }
     case 'prompt':
     case 'response': {
@@ -279,11 +308,12 @@ export function summarizeNodeForAgentContext(
     }
     case 'file': {
       const path = typeof node.data.path === 'string' ? node.data.path : '';
-      const fileContent = typeof node.data.fileContent === 'string'
-        ? node.data.fileContent
-        : typeof node.data.content === 'string'
-          ? node.data.content
-          : '';
+      const fileContent =
+        typeof node.data.fileContent === 'string'
+          ? node.data.fileContent
+          : typeof node.data.content === 'string'
+            ? node.data.content
+            : '';
       const prefix = path ? `Path: ${path}\n\n` : '';
       const remaining = Math.max(0, defaultTextLength - prefix.length);
       return `${prefix}${truncateContextText(fileContent, remaining)}`.trim();
@@ -309,13 +339,10 @@ export function serializeNodeForAgentContext(
   };
 }
 
-export function buildAgentContextPreamble(
-  nodes: CanvasNodeState[],
-  options: AgentContextOptions = {},
-): string {
+export function buildAgentContextPreamble(nodes: CanvasNodeState[], options: AgentContextOptions = {}): string {
   const sections = nodes
     .map((node) => {
-      const title = (typeof node.data.title === 'string' && node.data.title) ? node.data.title : node.id;
+      const title = typeof node.data.title === 'string' && node.data.title ? node.data.title : node.id;
       const content = summarizeNodeForAgentContext(node, options);
       if (!content) return '';
       const kind = getCanvasNodeKind(node);

package/src/server/ax-context.ts

@@ -53,11 +53,13 @@ export function buildCanvasAxSurfaceSnapshot(): PmxAxSurfaceSnapshot {
 }
 
 function serializeNodes(nodes: CanvasNodeState[]) {
-  return nodes.map((node) => serializeNodeForAgentContext(node, {
-    defaultTextLength: 700,
-    webpageTextLength: 1600,
-    includePosition: true,
-  }));
+  return nodes.map((node) =>
+    serializeNodeForAgentContext(node, {
+      defaultTextLength: 700,
+      webpageTextLength: 1600,
+      includePosition: true,
+    }),
+  );
 }
 
 export function buildCanvasAxPinnedContext(): PmxAxPinnedContext {

package/src/server/ax-interaction.ts

@@ -86,11 +86,29 @@ function caps(
  * a node can anchor AX state but only eligible types may EMIT interactions.
  */
 export const DEFAULT_NODE_AX_CAPABILITIES: Record<CanvasNodeType, NodeAxCapabilities> = {
-  markdown: caps(true, ['ax.steer', 'ax.work.create', 'ax.evidence.add', 'ax.command.invoke', 'ax.event.record'], 'notify-agent'),
-  context: caps(true, ['ax.focus.set', 'ax.steer', 'ax.evidence.add', 'ax.command.invoke', 'ax.event.record'], 'notify-agent'),
-  status: caps(true, ['ax.work.create', 'ax.work.update', 'ax.approval.request', 'ax.mode.request', 'ax.event.record'], 'notify-agent'),
+  markdown: caps(
+    true,
+    ['ax.steer', 'ax.work.create', 'ax.evidence.add', 'ax.command.invoke', 'ax.event.record'],
+    'notify-agent',
+  ),
+  context: caps(
+    true,
+    ['ax.focus.set', 'ax.steer', 'ax.evidence.add', 'ax.command.invoke', 'ax.event.record'],
+    'notify-agent',
+  ),
+  status: caps(
+    true,
+    ['ax.work.create', 'ax.work.update', 'ax.approval.request', 'ax.mode.request', 'ax.event.record'],
+    'notify-agent',
+  ),
   file: caps(true, ['ax.evidence.add', 'ax.review.add', 'ax.focus.set', 'ax.event.record']),
-  'json-render': caps(true, ['ax.work.create', 'ax.work.update', 'ax.evidence.add', 'ax.elicitation.request', 'ax.event.record']),
+  'json-render': caps(true, [
+    'ax.work.create',
+    'ax.work.update',
+    'ax.evidence.add',
+    'ax.elicitation.request',
+    'ax.event.record',
+  ]),
   graph: caps(true, ['ax.evidence.add', 'ax.focus.set', 'ax.event.record']),
   ledger: caps(true, ['ax.evidence.add', 'ax.event.record']),
   trace: caps(true, ['ax.evidence.add', 'ax.event.record']),
@@ -123,8 +141,12 @@ export const DEFAULT_NODE_AX_CAPABILITIES: Record<CanvasNodeType, NodeAxCapabili
   // itself, and request human input. Excludes higher-trust types (steer, approval,
   // review, command, mode) which stay native-control / adapter only.
   'mcp-app': caps(false, [
-    'ax.event.record', 'ax.evidence.add', 'ax.work.create', 'ax.work.update',
-    'ax.focus.set', 'ax.elicitation.request',
+    'ax.event.record',
+    'ax.evidence.add',
+    'ax.work.create',
+    'ax.work.update',
+    'ax.focus.set',
+    'ax.elicitation.request',
   ]),
   // Internal thread nodes — anchor only, no human-facing emission by default.
   prompt: caps(false, ['ax.event.record']),
@@ -141,11 +163,13 @@ export function normalizeNodeAxCapabilities(value: unknown): Partial<NodeAxCapab
   if (typeof v.enabled === 'boolean') out.enabled = v.enabled;
   if (Array.isArray(v.allowed)) {
     out.allowed = v.allowed.filter((a): a is AxInteractionType =>
-      AX_INTERACTION_TYPES.includes(a as AxInteractionType));
+      AX_INTERACTION_TYPES.includes(a as AxInteractionType),
+    );
   }
   if (Array.isArray(v.requiresApproval)) {
     out.requiresApproval = v.requiresApproval.filter((a): a is AxInteractionType =>
-      AX_INTERACTION_TYPES.includes(a as AxInteractionType));
+      AX_INTERACTION_TYPES.includes(a as AxInteractionType),
+    );
   }
   if (v.delivery === 'record-only' || v.delivery === 'notify-agent' || v.delivery === 'send-to-agent') {
     out.delivery = v.delivery;
@@ -172,7 +196,15 @@ export function resolveNodeAxCapabilities(node: CanvasNodeState): NodeAxCapabili
 
 // ── Envelope + payload validation ──────────────────────────────
 
-const EVENT_KINDS = ['prompt', 'assistant-message', 'tool-start', 'tool-result', 'failure', 'approval', 'steering'] as const;
+const EVENT_KINDS = [
+  'prompt',
+  'assistant-message',
+  'tool-start',
+  'tool-result',
+  'failure',
+  'approval',
+  'steering',
+] as const;
 const EVIDENCE_KINDS = ['logs', 'tool-result', 'screenshot', 'file', 'diff', 'test-output'] as const;
 const WORK_STATUSES = ['todo', 'in-progress', 'blocked', 'done', 'cancelled'] as const;
 const REVIEW_KINDS = ['comment', 'finding'] as const;
@@ -284,7 +316,13 @@ const PAYLOAD_SCHEMAS: Record<string, z.ZodType> = {
 export interface AxInteractionManager {
   getNode(id: string): CanvasNodeState | undefined;
   recordAxEvent(
-    input: { kind: PmxAxEventKind; summary: string; detail?: string | null; nodeIds?: string[]; data?: Record<string, unknown> | null },
+    input: {
+      kind: PmxAxEventKind;
+      summary: string;
+      detail?: string | null;
+      nodeIds?: string[];
+      data?: Record<string, unknown> | null;
+    },
     options?: { source?: PmxAxSource },
   ): PmxAxEvent;
   recordSteeringMessage(message: string, options?: { source?: PmxAxSource }): PmxAxSteeringMessage;
@@ -298,7 +336,14 @@ export interface AxInteractionManager {
     options?: { source?: PmxAxSource },
   ): PmxAxWorkItem | null;
   addEvidence(
-    input: { kind: PmxAxEvidenceKind; title: string; body?: string | null; ref?: string | null; nodeIds?: string[]; data?: Record<string, unknown> | null },
+    input: {
+      kind: PmxAxEvidenceKind;
+      title: string;
+      body?: string | null;
+      ref?: string | null;
+      nodeIds?: string[];
+      data?: Record<string, unknown> | null;
+    },
     options?: { source?: PmxAxSource },
   ): PmxAxEvidence;
   requestApproval(
@@ -332,7 +377,11 @@ export interface AxInteractionManager {
     input: { mode: PmxAxMode; reason?: string | null; nodeIds?: string[] },
     options?: { source?: PmxAxSource },
   ): PmxAxModeRequest;
-  invokeCommand(name: string, args?: Record<string, unknown> | null, options?: { source?: PmxAxSource }): PmxAxEvent | null;
+  invokeCommand(
+    name: string,
+    args?: Record<string, unknown> | null,
+    options?: { source?: PmxAxSource },
+  ): PmxAxEvent | null;
 }
 
 export interface AxInteractionEvent {
@@ -353,13 +402,7 @@ function outcomeEvent(extra: Record<string, unknown>): AxInteractionEvent {
   return { event: 'ax-interaction', payload: extra };
 }
 
-function reject(
-  type: string,
-  sourceNodeId: string,
-  status: number,
-  code: string,
-  error: string,
-): AxInteractionResult {
+function reject(type: string, sourceNodeId: string, status: number, code: string, error: string): AxInteractionResult {
   return {
     result: { ok: false, status, code, error },
     events: [outcomeEvent({ ok: false, type, sourceNodeId, code, error })],
@@ -375,10 +418,7 @@ function accept(
 ): AxInteractionResult {
   return {
     result: { ok: true, type, sourceNodeId, primitive },
-    events: [
-      outcomeEvent({ ok: true, type, sourceNodeId }),
-      { event: stateEvent, payload: statePayload },
-    ],
+    events: [outcomeEvent({ ok: true, type, sourceNodeId }), { event: stateEvent, payload: statePayload }],
   };
 }
 
@@ -396,10 +436,14 @@ export function applyAxInteraction(
   const parsed = InteractionEnvelopeSchema.safeParse(rawBody);
   if (!parsed.success) {
     const error = parsed.error.issues.map((i) => `${i.path.join('.') || 'envelope'}: ${i.message}`).join('; ');
-    const type = typeof (rawBody as { type?: unknown })?.type === 'string' ? String((rawBody as { type?: unknown }).type) : 'unknown';
-    const sourceNodeId = typeof (rawBody as { sourceNodeId?: unknown })?.sourceNodeId === 'string'
-      ? String((rawBody as { sourceNodeId?: unknown }).sourceNodeId)
-      : '';
+    const type =
+      typeof (rawBody as { type?: unknown })?.type === 'string'
+        ? String((rawBody as { type?: unknown }).type)
+        : 'unknown';
+    const sourceNodeId =
+      typeof (rawBody as { sourceNodeId?: unknown })?.sourceNodeId === 'string'
+        ? String((rawBody as { sourceNodeId?: unknown }).sourceNodeId)
+        : '';
     return reject(type, sourceNodeId, 400, 'invalid-envelope', error);
   }
   const interaction = parsed.data;
@@ -410,7 +454,13 @@ export function applyAxInteraction(
 
   const capabilities = resolveNodeAxCapabilities(node);
   if (!capabilities.enabled) {
-    return reject(type, sourceNodeId, 403, 'ax-disabled', `AX interactions are not enabled for node "${sourceNodeId}".`);
+    return reject(
+      type,
+      sourceNodeId,
+      403,
+      'ax-disabled',
+      `AX interactions are not enabled for node "${sourceNodeId}".`,
+    );
   }
   if (!capabilities.allowed.includes(type)) {
     return reject(type, sourceNodeId, 403, 'not-allowed', `Node type "${node.type}" cannot emit "${type}".`);
@@ -418,7 +468,13 @@ export function applyAxInteraction(
   // Fail closed: approval-gated interaction types are rejected until approval
   // routing lands, rather than dispatched without the gate they require.
   if (capabilities.requiresApproval.includes(type)) {
-    return reject(type, sourceNodeId, 403, 'requires-approval', `"${type}" requires approval routing, which is not yet available.`);
+    return reject(
+      type,
+      sourceNodeId,
+      403,
+      'requires-approval',
+      `"${type}" requires approval routing, which is not yet available.`,
+    );
   }
 
   const schema = PAYLOAD_SCHEMAS[type];
@@ -446,9 +502,21 @@ export function applyAxInteraction(
 
   switch (type) {
     case 'ax.event.record': {
-      const p = payloadParsed.data as { kind: PmxAxEventKind; summary: string; detail?: string | null; nodeIds?: string[]; data?: Record<string, unknown> | null };
+      const p = payloadParsed.data as {
+        kind: PmxAxEventKind;
+        summary: string;
+        detail?: string | null;
+        nodeIds?: string[];
+        data?: Record<string, unknown> | null;
+      };
       const event = manager.recordAxEvent(
-        { kind: p.kind, summary: p.summary, detail: p.detail ?? null, nodeIds: scopedNodeIds(p.nodeIds), data: p.data ?? null },
+        {
+          kind: p.kind,
+          summary: p.summary,
+          detail: p.detail ?? null,
+          nodeIds: scopedNodeIds(p.nodeIds),
+          data: p.data ?? null,
+        },
         opts,
       );
       return accept(type, sourceNodeId, event, 'ax-event-created', { event });
@@ -459,15 +527,31 @@ export function applyAxInteraction(
       return accept(type, sourceNodeId, steering, 'ax-event-created', { steering });
     }
     case 'ax.work.create': {
-      const p = payloadParsed.data as { title: string; status?: PmxAxWorkItemStatus; detail?: string | null; nodeIds?: string[] };
+      const p = payloadParsed.data as {
+        title: string;
+        status?: PmxAxWorkItemStatus;
+        detail?: string | null;
+        nodeIds?: string[];
+      };
       const workItem = manager.addWorkItem(
-        { title: p.title, ...(p.status ? { status: p.status } : {}), ...(p.detail !== undefined ? { detail: p.detail } : {}), nodeIds: scopedNodeIds(p.nodeIds) },
+        {
+          title: p.title,
+          ...(p.status ? { status: p.status } : {}),
+          ...(p.detail !== undefined ? { detail: p.detail } : {}),
+          nodeIds: scopedNodeIds(p.nodeIds),
+        },
         opts,
       );
       return accept(type, sourceNodeId, workItem, 'ax-state-changed', { workItem });
     }
     case 'ax.work.update': {
-      const p = payloadParsed.data as { id: string; title?: string; status?: PmxAxWorkItemStatus; detail?: string | null; nodeIds?: string[] };
+      const p = payloadParsed.data as {
+        id: string;
+        title?: string;
+        status?: PmxAxWorkItemStatus;
+        detail?: string | null;
+        nodeIds?: string[];
+      };
       const { id, ...patch } = p;
       if (scoped && patch.nodeIds !== undefined) patch.nodeIds = [sourceNodeId];
       const workItem = manager.updateWorkItem(id, patch, opts);
@@ -475,29 +559,71 @@ export function applyAxInteraction(
       return accept(type, sourceNodeId, workItem, 'ax-state-changed', { workItem });
     }
     case 'ax.evidence.add': {
-      const p = payloadParsed.data as { kind: PmxAxEvidenceKind; title: string; body?: string | null; ref?: string | null; nodeIds?: string[]; data?: Record<string, unknown> | null };
+      const p = payloadParsed.data as {
+        kind: PmxAxEvidenceKind;
+        title: string;
+        body?: string | null;
+        ref?: string | null;
+        nodeIds?: string[];
+        data?: Record<string, unknown> | null;
+      };
       const evidence = manager.addEvidence(
-        { kind: p.kind, title: p.title, body: p.body ?? null, ref: p.ref ?? null, nodeIds: scopedNodeIds(p.nodeIds), data: p.data ?? null },
+        {
+          kind: p.kind,
+          title: p.title,
+          body: p.body ?? null,
+          ref: p.ref ?? null,
+          nodeIds: scopedNodeIds(p.nodeIds),
+          data: p.data ?? null,
+        },
         opts,
       );
       return accept(type, sourceNodeId, evidence, 'ax-event-created', { evidence });
     }
     case 'ax.approval.request': {
-      const p = payloadParsed.data as { title: string; detail?: string | null; action?: string | null; nodeIds?: string[] };
+      const p = payloadParsed.data as {
+        title: string;
+        detail?: string | null;
+        action?: string | null;
+        nodeIds?: string[];
+      };
       const approvalGate = manager.requestApproval(
-        { title: p.title, ...(p.detail !== undefined ? { detail: p.detail } : {}), ...(p.action !== undefined ? { action: p.action } : {}), nodeIds: scopedNodeIds(p.nodeIds) },
+        {
+          title: p.title,
+          ...(p.detail !== undefined ? { detail: p.detail } : {}),
+          ...(p.action !== undefined ? { action: p.action } : {}),
+          nodeIds: scopedNodeIds(p.nodeIds),
+        },
         opts,
       );
       return accept(type, sourceNodeId, approvalGate, 'ax-state-changed', { approvalGate });
     }
     case 'ax.approval.resolve': {
       const p = payloadParsed.data as { id: string; decision: 'approved' | 'rejected'; resolution?: string };
-      const approvalGate = manager.resolveApproval(p.id, p.decision, { ...(p.resolution !== undefined ? { resolution: p.resolution } : {}), source });
-      if (!approvalGate) return reject(type, sourceNodeId, 404, 'approval-not-found', `Approval "${p.id}" not found or already resolved.`);
+      const approvalGate = manager.resolveApproval(p.id, p.decision, {
+        ...(p.resolution !== undefined ? { resolution: p.resolution } : {}),
+        source,
+      });
+      if (!approvalGate)
+        return reject(
+          type,
+          sourceNodeId,
+          404,
+          'approval-not-found',
+          `Approval "${p.id}" not found or already resolved.`,
+        );
       return accept(type, sourceNodeId, approvalGate, 'ax-state-changed', { approvalGate });
     }
     case 'ax.review.add': {
-      const p = payloadParsed.data as { body: string; kind?: PmxAxReviewKind; severity?: PmxAxReviewSeverity; anchorType?: PmxAxReviewAnchorType; nodeId?: string; file?: string; author?: string };
+      const p = payloadParsed.data as {
+        body: string;
+        kind?: PmxAxReviewKind;
+        severity?: PmxAxReviewSeverity;
+        anchorType?: PmxAxReviewAnchorType;
+        nodeId?: string;
+        file?: string;
+        author?: string;
+      };
       // Sandboxed surfaces may only review their own node; trusted surfaces may
       // anchor to a file/region or another node.
       // A node-interaction review carries a sourceNodeId, so it defaults to a node
@@ -510,13 +636,20 @@ export function applyAxInteraction(
           ...(p.kind ? { kind: p.kind } : {}),
           ...(p.severity ? { severity: p.severity } : {}),
           anchorType,
-          nodeId: scoped ? sourceNodeId : (anchorType === 'node' ? (p.nodeId ?? sourceNodeId) : (p.nodeId ?? null)),
+          nodeId: scoped ? sourceNodeId : anchorType === 'node' ? (p.nodeId ?? sourceNodeId) : (p.nodeId ?? null),
           ...(!scoped && p.file !== undefined ? { file: p.file } : {}),
           ...(p.author !== undefined ? { author: p.author } : {}),
         },
         opts,
       );
-      if (!reviewAnnotation) return reject(type, sourceNodeId, 400, 'invalid-review-anchor', 'Node-anchored review requires a nodeId that exists on the canvas.');
+      if (!reviewAnnotation)
+        return reject(
+          type,
+          sourceNodeId,
+          400,
+          'invalid-review-anchor',
+          'Node-anchored review requires a nodeId that exists on the canvas.',
+        );
       return accept(type, sourceNodeId, reviewAnnotation, 'ax-state-changed', { reviewAnnotation });
     }
     case 'ax.focus.set': {