feat: add elapsed runtime display for in-progress TUI commands #54

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft

ariane-emory wants to merge 143 commits into dev from feat/elapsed-timer

packages/opencode/src/cli/cmd/tui/routes/session/index.tsx

-Original file line number
+Diff line change
@@ Expand Up / @@ -10,6 +10,8 @@ import { @@
       Show,
       Switch,
       useContext,
+      onCleanup,
+      type Component,
     } from "solid-js"
     import { Dynamic } from "solid-js/web"
     import path from "path"
@@ Expand Down Expand Up @@
         return props.message.time.completed - user.time.created
       })
+      // Elapsed time for in-progress messages
+      const [elapsedTime, setElapsedTime] = createSignal(0)
+      createEffect(() => {
+        // Only run timer for the last in-progress message (not final/completed)
+        // This prevents multiple timers from running for older messages
+        if (!props.last || final()) {
+          setElapsedTime(0)
+          return
+        }
+        // Access parts directly from the store for proper reactivity tracking
+        // Using props.parts doesn't trigger updates when individual parts change
+        const parts = sync.data.part[props.message.id] ?? []
+        // Find the start time of the last active part (current action)
+        let startTime: number | undefined
+        // Check parts in reverse order to find the most recent one with timing
+        for (let i = parts.length - 1; i >= 0; i--) {
+          const part = parts[i]
+          if (part.type === "tool") {
+            // Running tool has time.start
+            if (part.state.status === "running" && part.state.time?.start) {
+              startTime = part.state.time.start
+              break
+            }
+          } else if (part.type === "text" || part.type === "reasoning") {
+            // Text/reasoning parts have time.start if in progress (no end time)
+            if (part.time?.start && !part.time?.end) {
+              startTime = part.time.start
+              break
+            }
+          }
+        }
+        // No running part found - don't show elapsed time
+        if (!startTime) {
+          setElapsedTime(0)
+          return
+        }
+        const interval = setInterval(() => {
+          setElapsedTime(Date.now() - startTime!)
+        }, 1000)
+        onCleanup(() => {
+          clearInterval(interval)
+        })
+      })
       return (
         <>
           <For each={props.parts}>
@@ Expand Down Expand Up @@
                   <Show when={duration()}>
                     <span style={{ fg: theme.textMuted }}> · {Locale.duration(duration())}</span>
                   </Show>
+                  <Show when={!final() && elapsedTime()}>
+                    <span style={{ fg: theme.textMuted }}> · running {Locale.duration(elapsedTime())}</span>
+                  </Show>
                   <Show when={props.message.error?.name === "MessageAbortedError"}>
                     <span style={{ fg: theme.textMuted }}> · interrupted</span>
                   </Show>
@@ Expand Down @@

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: add elapsed runtime display for in-progress TUI commands #54

Uh oh!

Diff view

Diff view

There are no files selected for viewing

feat: add elapsed runtime display for in-progress TUI commands #54

Are you sure you want to change the base?

Uh oh!

feat: add elapsed runtime display for in-progress TUI commands #54

Uh oh!

Uh oh!

Diff view

Diff view

There are no files selected for viewing