ソースを参照

fix(desktop): animating too much

Adam 3 ヶ月 前
コミット
3068e7dcf7
1 ファイル変更4 行追加5 行削除
  1. 4 5
      packages/desktop/src/pages/session.tsx

+ 4 - 5
packages/desktop/src/pages/session.tsx

@@ -459,8 +459,6 @@ export default function Page() {
                               ) as AssistantMessageType[]
                               ) as AssistantMessageType[]
                             })
                             })
                             const error = createMemo(() => assistantMessages().find((m) => m?.error)?.error)
                             const error = createMemo(() => assistantMessages().find((m) => m?.error)?.error)
-                            const initialCompleted = !!message.summary?.body || !!error()
-                            const [completed, setCompleted] = createSignal(initialCompleted)
                             const [detailsExpanded, setDetailsExpanded] = createSignal(false)
                             const [detailsExpanded, setDetailsExpanded] = createSignal(false)
                             const parts = createMemo(() => sync.data.part[message.id])
                             const parts = createMemo(() => sync.data.part[message.id])
                             const hasToolPart = createMemo(() =>
                             const hasToolPart = createMemo(() =>
@@ -471,6 +469,8 @@ export default function Page() {
                             const working = createMemo(
                             const working = createMemo(
                               () => message.id === session.messages.last()?.id && session.working(),
                               () => message.id === session.messages.last()?.id && session.working(),
                             )
                             )
+                            const initialCompleted = !(message.id === session.messages.last()?.id && session.working())
+                            const [completed, setCompleted] = createSignal(initialCompleted)
 
 
                             // allowing time for the animations to finish
                             // allowing time for the animations to finish
                             createEffect(() => {
                             createEffect(() => {
@@ -478,9 +478,8 @@ export default function Page() {
                               setTimeout(() => setTitled(!!title), 10_000)
                               setTimeout(() => setTitled(!!title), 10_000)
                             })
                             })
                             createEffect(() => {
                             createEffect(() => {
-                              const summary = message.summary?.body
-                              const complete = !!summary || !!error()
-                              setTimeout(() => setCompleted(complete), 1200)
+                              const completed = !working()
+                              setTimeout(() => setCompleted(completed), 1200)
                             })
                             })
 
 
                             return (
                             return (