From 76ad697dcb277bc468e2249e1a11eb6686406c4d Mon Sep 17 00:00:00 2001 From: Brooklyn Nicholson Date: Tue, 21 Apr 2026 22:51:30 -0500 Subject: [PATCH] fix(tui): don't force-open Activity on every error MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Reverts the auto-expand-on-new-error effect added in 93b47d96. The effect overrode the user's chosen detailsMode and visually interrupted every turn. Red/yellow chevron tint remains as the passive signal — click to read, just like Thinking and Tool calls. --- ui-tui/src/components/thinking.tsx | 13 +------------ 1 file changed, 1 insertion(+), 12 deletions(-) diff --git a/ui-tui/src/components/thinking.tsx b/ui-tui/src/components/thinking.tsx index 2c741caadc..37c9598f81 100644 --- a/ui-tui/src/components/thinking.tsx +++ b/ui-tui/src/components/thinking.tsx @@ -1,5 +1,5 @@ import { Box, NoSelect, Text } from '@hermes/ink' -import { memo, type ReactNode, useEffect, useMemo, useState } from 'react' +import { memo, useEffect, useMemo, useState, type ReactNode } from 'react' import spinners, { type BrailleSpinnerName } from 'unicode-animations' import { THINKING_COT_MAX } from '../config/limits.js' @@ -596,17 +596,6 @@ export const ToolTrail = memo(function ToolTrail({ } }, [detailsMode]) - const latestErrorId = useMemo( - () => activity.reduce((max, i) => (i.tone === 'error' && i.id > max ? i.id : max), -1), - [activity] - ) - - useEffect(() => { - if (latestErrorId >= 0) { - setOpenMeta(true) - } - }, [latestErrorId]) - const cot = useMemo(() => thinkingPreview(reasoning, 'full', THINKING_COT_MAX), [reasoning]) if (