fix(search): polish inline action review interactions

Author: waleedlatif1

apps/docs/content/docs/search/ask-questions.mdx

@@ -127,13 +127,13 @@ Copy the reviewed draft into another app, or ask Assistant to create it using yo
 
 When integration actions are enabled, Assistant can answer questions and take supported actions in the same conversation. For example, ask it to draft an email in Gmail or create a Jira issue using your own connected account.
 
-Reads run without a prompt by default. Writes wait for review by default. Each proposal appears inline in the conversation with its app, account, destination, and proposed content. Edit the supported fields, then select **Allow** or **Cancel** for each action. For several proposals, **Allow N shown** approves the actions currently displayed in the review.
+Reads run without a prompt by default. Writes wait for review by default. Each proposal appears inline in the conversation with its app, account, destination, and proposed content. Edit the supported fields, then select **Allow** or **Cancel** for each action. After several proposals, **Allow N shown** or **Cancel N shown** applies your choice to that displayed batch. You can approve individual actions first, then cancel the remaining batch.
 
 <Image
   src="/static/search/assistant-action-review.png"
   alt="Inline Assistant action review showing the connected Jira account, exact site and project, editable issue summary and description, and Allow or Cancel controls"
   width={705}
-  height={535}
+  height={489}
   variant="detail"
 />
 

apps/docs/public/static/search/assistant-action-review.png

@@ -953,6 +953,8 @@ interface MessageContentProps {
    * place without ever changing height.
    */
   isLast?: boolean
+  /** Pending review controls own the tail until the user decides. */
+  awaitingInteraction?: boolean
   /** Transcript-derived answers for this message's question card (renders the recap). */
   questionAnswers?: string[]
   /** Transcript-derived status payload for this message's credential card. */
@@ -980,6 +982,7 @@ function MessageContentInner({
   requestMode,
   isStreaming = false,
   isLast = false,
+  awaitingInteraction = false,
   questionAnswers,
   credentialSubmission,
   credentialAbandoned,
@@ -1082,7 +1085,8 @@ function MessageContentInner({
   // trailing text keeps visually revealing on a timer after the network stream
   // closes, and collapsing under a still-growing reveal reads as the blob
   // winking out early while everything shifts.
-  const thinkingExpanded = phase !== 'settled' && lastSegment?.type !== 'stopped'
+  const thinkingExpanded =
+    !awaitingInteraction && phase !== 'settled' && lastSegment?.type !== 'stopped'
 
   if (segments.length === 0 && !isLast) return null
 

apps/sim/app/workspace/[workspaceId]/home/components/message-content/message-content.tsx

@@ -22,7 +22,10 @@ import {
 } from '@sim/emcn'
 import { useQueryClient } from '@tanstack/react-query'
 import { defaultRangeExtractor, type Range, useVirtualizer } from '@tanstack/react-virtual'
-import { AssistantActionReviewMessage } from '@/components/assistant-actions'
+import {
+  AssistantActionReviewMessage,
+  useHasAssistantActionReview,
+} from '@/components/assistant-actions'
 import { ComputerUseActivity } from '@/components/computer-use/activity'
 import { SMOOTH_CHASE_RATE } from '@/lib/core/utils/smooth-bottom-chase'
 import type { TurnOrigin } from '@/lib/mothership/generated/protocol'
@@ -282,6 +285,7 @@ const AssistantMessageRow = memo(function AssistantMessageRow({
   onOptionSelect,
   onAnimatingChange,
 }: AssistantMessageRowProps) {
+  const hasActionReview = useHasAssistantActionReview()
   const permissions = useOptionalWorkspacePermissionsContext()
   const canEdit = permissions?.userPermissions.canEdit ?? false
   const blocks = message.contentBlocks ?? EMPTY_BLOCKS
@@ -325,15 +329,18 @@ const AssistantMessageRow = memo(function AssistantMessageRow({
   // Settle timing lives in MessageContent (the actions take the thinking
   // slot's place in the same render), so eligibility here is phase-free:
   // `phase: 'settled'` asks the helper "would a settled turn show them?".
-  const actionsEligible = shouldShowAssistantMessageActions({
-    phase: 'settled',
-    hasContent: Boolean(message.content) || hasAnyBlocks,
-    endsWithInteraction: endsWithQuestion || showsCredentialCard,
-    questionDismissed,
-  })
+  const actionsEligible =
+    !hasActionReview &&
+    shouldShowAssistantMessageActions({
+      phase: 'settled',
+      hasContent: Boolean(message.content) || hasAnyBlocks,
+      endsWithInteraction: endsWithQuestion || showsCredentialCard,
+      questionDismissed,
+    })
 
   /** Match the 16px prose/card gap when the card is the message's last visible content. */
-  const showsInteractionCard = (endsWithQuestion && !questionDismissed) || showsCredentialCard
+  const showsInteractionCard =
+    (endsWithQuestion && !questionDismissed) || showsCredentialCard || hasActionReview
 
   return (
     <div className={cn(rowClassName, showsInteractionCard && 'pb-4')}>
@@ -345,6 +352,7 @@ const AssistantMessageRow = memo(function AssistantMessageRow({
         fallbackContent={message.content}
         isStreaming={isStreaming}
         isLast={isLast}
+        awaitingInteraction={hasActionReview}
         questionAnswers={questionAnswers}
         credentialSubmission={credentialSubmission}
         credentialAbandoned={credentialAbandoned}

apps/sim/app/workspace/[workspaceId]/home/components/mothership-chat/mothership-chat.tsx

@@ -2,4 +2,5 @@ export {
   AssistantActionReviewMessage,
   AssistantActionReviewProvider,
   AssistantActionReviewStatus,
+  useHasAssistantActionReview,
 } from './review'