Skip to content

Commit cdca138

Browse files
committed
cli: Remove unnecessary comments in suggest-followups
1 parent 3b1085b commit cdca138

File tree

2 files changed

+0
-10
lines changed

2 files changed

+0
-10
lines changed

cli/src/components/ask-user/index.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22
* Ask User Tool - Multiple choice form with accordion-style FAQ layout
33
*
44
* Shows all questions at once, each expandable to reveal options.
5-
* Auto-submits when all questions are answered.
65
*/
76

87
import { TextAttributes } from '@opentui/core'

cli/src/components/tools/suggest-followups.tsx

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ import { Button } from '../button'
99
import type { ToolRenderConfig } from './types'
1010
import type { SuggestedFollowup } from '../../state/chat-store'
1111

12-
// Stable empty set to avoid creating new references on each render
1312
const EMPTY_CLICKED_SET = new Set<number>()
1413
const MIN_LABEL_COLUMN_WIDTH = 12
1514
const MAX_LABEL_COLUMN_WIDTH = 60
@@ -121,9 +120,6 @@ interface SuggestFollowupsItemProps {
121120
onSendFollowup: (prompt: string, index: number) => void
122121
}
123122

124-
/**
125-
* Static display of a followup for past messages (non-interactive)
126-
*/
127123
interface PastFollowupItemProps {
128124
followup: SuggestedFollowup
129125
isClicked: boolean
@@ -156,9 +152,6 @@ const PastFollowupItem = ({ followup, isClicked }: PastFollowupItemProps) => {
156152
)
157153
}
158154

159-
/**
160-
* Collapsed toggle view for past messages showing "Previously suggested followups"
161-
*/
162155
interface PastFollowupsToggleProps {
163156
toolCallId: string
164157
followups: SuggestedFollowup[]
@@ -219,8 +212,6 @@ const SuggestFollowupsItem = ({
219212
const latestFollowupToolCallId = useChatStore((state) =>
220213
getLatestFollowupToolCallId(state.messages),
221214
)
222-
// Get clicked indices from the persistent map
223-
// Use stable EMPTY_CLICKED_SET to avoid creating new references that cause infinite re-renders
224215
const clickedIndices = useChatStore(
225216
(state) => state.clickedFollowupsMap.get(toolCallId) ?? EMPTY_CLICKED_SET,
226217
)

0 commit comments

Comments
 (0)