fix(chat): replace failed tools with thinking by maria-rcks · Pull Request #9165 · pingdotgg/t3code

maria-rcks · 2026-09-02T04:43:10Z

Failed tool calls stayed visible as a terminal activity row while Thinking was appended beneath them. This returns the shared live activity row to Thinking after a failure, while preserving the failed tool in history once the active run moves on.

Verified with the focused timeline test, web typecheck, targeted lint and formatting, and the same seeded browser fixture at 1280x800 in dark mode.

before	after

Implemented by gpt-5.6-sol in the Codex harness.

Note

Low Risk
Scoped to chat timeline row derivation and covered by a focused unit test; no auth, data, or API changes.

Overview
While a turn is still running, a failed latest tool no longer occupies the shared live activity slot as a terminal work-live row with Thinking stacked underneath.

deriveMessagesTimelineRows now treats the latest visible non-declined tool as failed when nothing is still running and workEntryDisplayIndicatesToolFailure applies. In that case it skips building the active work-live row, still consumes those tool timeline entries so they are not duplicated inline, and emits the single live-activity-row as thinking (same pattern as declined tools). In-progress and successful tools keep the existing live tool behavior.

Tests cover the failed lifecycle path and assert there is no work-live row and only one thinking activity row.

^{Reviewed by Cursor Bugbot for commit 75ceea1. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Replace failed tool rows with thinking activity in `deriveMessagesTimelineRows`

When the latest visible tool fails (and is not declined, with no tool running), the timeline now appends a live thinking activity row instead of a work-live row. Failed tools remain in the active-work entry set so they still appear in the timeline. Added test coverage for the failed-tool lifecycle in MessagesTimeline.logic.test.ts.

^{Macroscope summarized 75ceea1.}

cursor

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit ffeac36. Configure here.}

macroscopeapp · 2026-09-02T05:03:50Z

Approvability

Verdict: Approved at 75ceea1

Macroscope's review found this PR approvable — This is a narrowly scoped chat timeline fix that replaces a terminal failed-tool activity row with the existing Thinking indicator while preserving in-progress and successful tool behavior. The production logic change is isolated and covered by focused tests, with no schema, configuration, security, billing, or deployment impact.

^{You can add or adjust custom eligibility rules. Learn more.}

Adopts main's round-19 features into the v2 stack: payload-budgeted orchestration replay (#8992), sidebar row subscription leases (#9052), tool group virtualization and scroll anchoring (#9106), repeated-command and browser-group presentation, inline assistant citations (#9146), per-cwd provider skills discovery (#8778), Claude composer skill dispatch (#9128), grok health probe and model negotiation (#9154), and the failed-tool thinking fallback (#9165). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

maria-rcks · 2026-09-02T04:43:10Z

Failed tool calls stayed visible as a terminal activity row while Thinking was appended beneath them. This returns the shared live activity row to Thinking after a failure, while preserving the failed tool in history once the active run moves on.

Verified with the focused timeline test, web typecheck, targeted lint and formatting, and the same seeded browser fixture at 1280x800 in dark mode.

before	after

Implemented by gpt-5.6-sol in the Codex harness.

Note

Low Risk
Scoped to chat timeline row derivation and covered by a focused unit test; no auth, data, or API changes.

Overview
While a turn is still running, a failed latest tool no longer occupies the shared live activity slot as a terminal work-live row with Thinking stacked underneath.

deriveMessagesTimelineRows now treats the latest visible non-declined tool as failed when nothing is still running and workEntryDisplayIndicatesToolFailure applies. In that case it skips building the active work-live row, still consumes those tool timeline entries so they are not duplicated inline, and emits the single live-activity-row as thinking (same pattern as declined tools). In-progress and successful tools keep the existing live tool behavior.

Tests cover the failed lifecycle path and assert there is no work-live row and only one thinking activity row.

^{Reviewed by Cursor Bugbot for commit 75ceea1. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Replace failed tool rows with thinking activity in `deriveMessagesTimelineRows`

When the latest visible tool fails (and is not declined, with no tool running), the timeline now appends a live thinking activity row instead of a work-live row. Failed tools remain in the active-work entry set so they still appear in the timeline. Added test coverage for the failed-tool lifecycle in MessagesTimeline.logic.test.ts.

^{Macroscope summarized 75ceea1.}

cursor

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit ffeac36. Configure here.}

macroscopeapp · 2026-09-02T05:03:50Z

Approvability

Verdict: Approved at 75ceea1

Macroscope's review found this PR approvable — This is a narrowly scoped chat timeline fix that replaces a terminal failed-tool activity row with the existing Thinking indicator while preserving in-progress and successful tool behavior. The production logic change is isolated and covered by focused tests, with no schema, configuration, security, billing, or deployment impact.

^{You can add or adjust custom eligibility rules. Learn more.}

Adopts main's round-19 features into the v2 stack: payload-budgeted orchestration replay (#8992), sidebar row subscription leases (#9052), tool group virtualization and scroll anchoring (#9106), repeated-command and browser-group presentation, inline assistant citations (#9146), per-cwd provider skills discovery (#8778), Claude composer skill dispatch (#9128), grok health probe and model negotiation (#9154), and the failed-tool thinking fallback (#9165). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

maria-rcks · 2026-09-02T04:43:10Z

Failed tool calls stayed visible as a terminal activity row while Thinking was appended beneath them. This returns the shared live activity row to Thinking after a failure, while preserving the failed tool in history once the active run moves on.

Verified with the focused timeline test, web typecheck, targeted lint and formatting, and the same seeded browser fixture at 1280x800 in dark mode.

before	after

Implemented by gpt-5.6-sol in the Codex harness.

Note

Low Risk
Scoped to chat timeline row derivation and covered by a focused unit test; no auth, data, or API changes.

Overview
While a turn is still running, a failed latest tool no longer occupies the shared live activity slot as a terminal work-live row with Thinking stacked underneath.

deriveMessagesTimelineRows now treats the latest visible non-declined tool as failed when nothing is still running and workEntryDisplayIndicatesToolFailure applies. In that case it skips building the active work-live row, still consumes those tool timeline entries so they are not duplicated inline, and emits the single live-activity-row as thinking (same pattern as declined tools). In-progress and successful tools keep the existing live tool behavior.

Tests cover the failed lifecycle path and assert there is no work-live row and only one thinking activity row.

^{Reviewed by Cursor Bugbot for commit 75ceea1. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Replace failed tool rows with thinking activity in `deriveMessagesTimelineRows`

When the latest visible tool fails (and is not declined, with no tool running), the timeline now appends a live thinking activity row instead of a work-live row. Failed tools remain in the active-work entry set so they still appear in the timeline. Added test coverage for the failed-tool lifecycle in MessagesTimeline.logic.test.ts.

^{Macroscope summarized 75ceea1.}

cursor

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit ffeac36. Configure here.}

macroscopeapp · 2026-09-02T05:03:50Z

Approvability

Verdict: Approved at 75ceea1

Macroscope's review found this PR approvable — This is a narrowly scoped chat timeline fix that replaces a terminal failed-tool activity row with the existing Thinking indicator while preserving in-progress and successful tool behavior. The production logic change is isolated and covered by focused tests, with no schema, configuration, security, billing, or deployment impact.

^{You can add or adjust custom eligibility rules. Learn more.}

Adopts main's round-19 features into the v2 stack: payload-budgeted orchestration replay (#8992), sidebar row subscription leases (#9052), tool group virtualization and scroll anchoring (#9106), repeated-command and browser-group presentation, inline assistant citations (#9146), per-cwd provider skills discovery (#8778), Claude composer skill dispatch (#9128), grok health probe and model negotiation (#9154), and the failed-tool thinking fallback (#9165). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

maria-rcks · 2026-09-02T04:43:10Z

Failed tool calls stayed visible as a terminal activity row while Thinking was appended beneath them. This returns the shared live activity row to Thinking after a failure, while preserving the failed tool in history once the active run moves on.

Verified with the focused timeline test, web typecheck, targeted lint and formatting, and the same seeded browser fixture at 1280x800 in dark mode.

before	after

Implemented by gpt-5.6-sol in the Codex harness.

Note

Low Risk
Scoped to chat timeline row derivation and covered by a focused unit test; no auth, data, or API changes.

Overview
While a turn is still running, a failed latest tool no longer occupies the shared live activity slot as a terminal work-live row with Thinking stacked underneath.

deriveMessagesTimelineRows now treats the latest visible non-declined tool as failed when nothing is still running and workEntryDisplayIndicatesToolFailure applies. In that case it skips building the active work-live row, still consumes those tool timeline entries so they are not duplicated inline, and emits the single live-activity-row as thinking (same pattern as declined tools). In-progress and successful tools keep the existing live tool behavior.

Tests cover the failed lifecycle path and assert there is no work-live row and only one thinking activity row.

^{Reviewed by Cursor Bugbot for commit 75ceea1. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Replace failed tool rows with thinking activity in `deriveMessagesTimelineRows`

When the latest visible tool fails (and is not declined, with no tool running), the timeline now appends a live thinking activity row instead of a work-live row. Failed tools remain in the active-work entry set so they still appear in the timeline. Added test coverage for the failed-tool lifecycle in MessagesTimeline.logic.test.ts.

^{Macroscope summarized 75ceea1.}

cursor

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit ffeac36. Configure here.}

macroscopeapp · 2026-09-02T05:03:50Z

Approvability

Verdict: Approved at 75ceea1

Macroscope's review found this PR approvable — This is a narrowly scoped chat timeline fix that replaces a terminal failed-tool activity row with the existing Thinking indicator while preserving in-progress and successful tool behavior. The production logic change is isolated and covered by focused tests, with no schema, configuration, security, billing, or deployment impact.

^{You can add or adjust custom eligibility rules. Learn more.}

Adopts main's round-19 features into the v2 stack: payload-budgeted orchestration replay (#8992), sidebar row subscription leases (#9052), tool group virtualization and scroll anchoring (#9106), repeated-command and browser-group presentation, inline assistant citations (#9146), per-cwd provider skills discovery (#8778), Claude composer skill dispatch (#9128), grok health probe and model negotiation (#9154), and the failed-tool thinking fallback (#9165). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

maria-rcks · 2026-09-02T04:43:10Z

Failed tool calls stayed visible as a terminal activity row while Thinking was appended beneath them. This returns the shared live activity row to Thinking after a failure, while preserving the failed tool in history once the active run moves on.

Verified with the focused timeline test, web typecheck, targeted lint and formatting, and the same seeded browser fixture at 1280x800 in dark mode.

before	after

Implemented by gpt-5.6-sol in the Codex harness.

Note

Low Risk
Scoped to chat timeline row derivation and covered by a focused unit test; no auth, data, or API changes.

Overview
While a turn is still running, a failed latest tool no longer occupies the shared live activity slot as a terminal work-live row with Thinking stacked underneath.

deriveMessagesTimelineRows now treats the latest visible non-declined tool as failed when nothing is still running and workEntryDisplayIndicatesToolFailure applies. In that case it skips building the active work-live row, still consumes those tool timeline entries so they are not duplicated inline, and emits the single live-activity-row as thinking (same pattern as declined tools). In-progress and successful tools keep the existing live tool behavior.

Tests cover the failed lifecycle path and assert there is no work-live row and only one thinking activity row.

^{Reviewed by Cursor Bugbot for commit 75ceea1. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Replace failed tool rows with thinking activity in `deriveMessagesTimelineRows`

When the latest visible tool fails (and is not declined, with no tool running), the timeline now appends a live thinking activity row instead of a work-live row. Failed tools remain in the active-work entry set so they still appear in the timeline. Added test coverage for the failed-tool lifecycle in MessagesTimeline.logic.test.ts.

^{Macroscope summarized 75ceea1.}

cursor

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit ffeac36. Configure here.}

macroscopeapp · 2026-09-02T05:03:50Z

Approvability

Verdict: Approved at 75ceea1

Macroscope's review found this PR approvable — This is a narrowly scoped chat timeline fix that replaces a terminal failed-tool activity row with the existing Thinking indicator while preserving in-progress and successful tool behavior. The production logic change is isolated and covered by focused tests, with no schema, configuration, security, billing, or deployment impact.

^{You can add or adjust custom eligibility rules. Learn more.}

Adopts main's round-19 features into the v2 stack: payload-budgeted orchestration replay (#8992), sidebar row subscription leases (#9052), tool group virtualization and scroll anchoring (#9106), repeated-command and browser-group presentation, inline assistant citations (#9146), per-cwd provider skills discovery (#8778), Claude composer skill dispatch (#9128), grok health probe and model negotiation (#9154), and the failed-tool thinking fallback (#9165). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

maria-rcks · 2026-09-02T04:43:10Z

Failed tool calls stayed visible as a terminal activity row while Thinking was appended beneath them. This returns the shared live activity row to Thinking after a failure, while preserving the failed tool in history once the active run moves on.

Verified with the focused timeline test, web typecheck, targeted lint and formatting, and the same seeded browser fixture at 1280x800 in dark mode.

before	after

Implemented by gpt-5.6-sol in the Codex harness.

Note

Low Risk
Scoped to chat timeline row derivation and covered by a focused unit test; no auth, data, or API changes.

Overview
While a turn is still running, a failed latest tool no longer occupies the shared live activity slot as a terminal work-live row with Thinking stacked underneath.

deriveMessagesTimelineRows now treats the latest visible non-declined tool as failed when nothing is still running and workEntryDisplayIndicatesToolFailure applies. In that case it skips building the active work-live row, still consumes those tool timeline entries so they are not duplicated inline, and emits the single live-activity-row as thinking (same pattern as declined tools). In-progress and successful tools keep the existing live tool behavior.

Tests cover the failed lifecycle path and assert there is no work-live row and only one thinking activity row.

^{Reviewed by Cursor Bugbot for commit 75ceea1. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Replace failed tool rows with thinking activity in `deriveMessagesTimelineRows`

When the latest visible tool fails (and is not declined, with no tool running), the timeline now appends a live thinking activity row instead of a work-live row. Failed tools remain in the active-work entry set so they still appear in the timeline. Added test coverage for the failed-tool lifecycle in MessagesTimeline.logic.test.ts.

^{Macroscope summarized 75ceea1.}

cursor

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit ffeac36. Configure here.}

macroscopeapp · 2026-09-02T05:03:50Z

Approvability

Verdict: Approved at 75ceea1

Macroscope's review found this PR approvable — This is a narrowly scoped chat timeline fix that replaces a terminal failed-tool activity row with the existing Thinking indicator while preserving in-progress and successful tool behavior. The production logic change is isolated and covered by focused tests, with no schema, configuration, security, billing, or deployment impact.

^{You can add or adjust custom eligibility rules. Learn more.}

Adopts main's round-19 features into the v2 stack: payload-budgeted orchestration replay (#8992), sidebar row subscription leases (#9052), tool group virtualization and scroll anchoring (#9106), repeated-command and browser-group presentation, inline assistant citations (#9146), per-cwd provider skills discovery (#8778), Claude composer skill dispatch (#9128), grok health probe and model negotiation (#9154), and the failed-tool thinking fallback (#9165). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

maria-rcks · 2026-09-02T04:43:10Z

Failed tool calls stayed visible as a terminal activity row while Thinking was appended beneath them. This returns the shared live activity row to Thinking after a failure, while preserving the failed tool in history once the active run moves on.

Verified with the focused timeline test, web typecheck, targeted lint and formatting, and the same seeded browser fixture at 1280x800 in dark mode.

before	after

Implemented by gpt-5.6-sol in the Codex harness.

Note

Low Risk
Scoped to chat timeline row derivation and covered by a focused unit test; no auth, data, or API changes.

Overview
While a turn is still running, a failed latest tool no longer occupies the shared live activity slot as a terminal work-live row with Thinking stacked underneath.

deriveMessagesTimelineRows now treats the latest visible non-declined tool as failed when nothing is still running and workEntryDisplayIndicatesToolFailure applies. In that case it skips building the active work-live row, still consumes those tool timeline entries so they are not duplicated inline, and emits the single live-activity-row as thinking (same pattern as declined tools). In-progress and successful tools keep the existing live tool behavior.

Tests cover the failed lifecycle path and assert there is no work-live row and only one thinking activity row.

^{Reviewed by Cursor Bugbot for commit 75ceea1. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Replace failed tool rows with thinking activity in `deriveMessagesTimelineRows`

When the latest visible tool fails (and is not declined, with no tool running), the timeline now appends a live thinking activity row instead of a work-live row. Failed tools remain in the active-work entry set so they still appear in the timeline. Added test coverage for the failed-tool lifecycle in MessagesTimeline.logic.test.ts.

^{Macroscope summarized 75ceea1.}

cursor

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit ffeac36. Configure here.}

macroscopeapp · 2026-09-02T05:03:50Z

Approvability

Verdict: Approved at 75ceea1

Macroscope's review found this PR approvable — This is a narrowly scoped chat timeline fix that replaces a terminal failed-tool activity row with the existing Thinking indicator while preserving in-progress and successful tool behavior. The production logic change is isolated and covered by focused tests, with no schema, configuration, security, billing, or deployment impact.

^{You can add or adjust custom eligibility rules. Learn more.}

Adopts main's round-19 features into the v2 stack: payload-budgeted orchestration replay (#8992), sidebar row subscription leases (#9052), tool group virtualization and scroll anchoring (#9106), repeated-command and browser-group presentation, inline assistant citations (#9146), per-cwd provider skills discovery (#8778), Claude composer skill dispatch (#9128), grok health probe and model negotiation (#9154), and the failed-tool thinking fallback (#9165). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

maria-rcks · 2026-09-02T04:43:10Z

Failed tool calls stayed visible as a terminal activity row while Thinking was appended beneath them. This returns the shared live activity row to Thinking after a failure, while preserving the failed tool in history once the active run moves on.

Verified with the focused timeline test, web typecheck, targeted lint and formatting, and the same seeded browser fixture at 1280x800 in dark mode.

before	after

Implemented by gpt-5.6-sol in the Codex harness.

Note

Low Risk
Scoped to chat timeline row derivation and covered by a focused unit test; no auth, data, or API changes.

Overview
While a turn is still running, a failed latest tool no longer occupies the shared live activity slot as a terminal work-live row with Thinking stacked underneath.

deriveMessagesTimelineRows now treats the latest visible non-declined tool as failed when nothing is still running and workEntryDisplayIndicatesToolFailure applies. In that case it skips building the active work-live row, still consumes those tool timeline entries so they are not duplicated inline, and emits the single live-activity-row as thinking (same pattern as declined tools). In-progress and successful tools keep the existing live tool behavior.

Tests cover the failed lifecycle path and assert there is no work-live row and only one thinking activity row.

^{Reviewed by Cursor Bugbot for commit 75ceea1. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Replace failed tool rows with thinking activity in `deriveMessagesTimelineRows`

When the latest visible tool fails (and is not declined, with no tool running), the timeline now appends a live thinking activity row instead of a work-live row. Failed tools remain in the active-work entry set so they still appear in the timeline. Added test coverage for the failed-tool lifecycle in MessagesTimeline.logic.test.ts.

^{Macroscope summarized 75ceea1.}

cursor

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit ffeac36. Configure here.}

macroscopeapp · 2026-09-02T05:03:50Z

Approvability

Verdict: Approved at 75ceea1

Macroscope's review found this PR approvable — This is a narrowly scoped chat timeline fix that replaces a terminal failed-tool activity row with the existing Thinking indicator while preserving in-progress and successful tool behavior. The production logic change is isolated and covered by focused tests, with no schema, configuration, security, billing, or deployment impact.

^{You can add or adjust custom eligibility rules. Learn more.}

Adopts main's round-19 features into the v2 stack: payload-budgeted orchestration replay (#8992), sidebar row subscription leases (#9052), tool group virtualization and scroll anchoring (#9106), repeated-command and browser-group presentation, inline assistant citations (#9146), per-cwd provider skills discovery (#8778), Claude composer skill dispatch (#9128), grok health probe and model negotiation (#9154), and the failed-tool thinking fallback (#9165). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

fix(chat): replace failed tools with thinking
ffeac36

github-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XS 0-9 changed lines (additions + deletions). labels Sep 2, 2026

macroscopeappBot reviewed Sep 2, 2026
View reviewed changes

Comment threadapps/web/src/components/chat/MessagesTimeline.logic.ts
Comment threadapps/web/src/components/chat/MessagesTimeline.logic.ts

cursorBot reviewed Sep 2, 2026
View reviewed changes

Comment threadapps/web/src/components/chat/MessagesTimeline.logic.ts

fix(chat): preserve running tool activity
75ceea1

github-actionsBot added size:S 10-29 changed lines (additions + deletions). and removed size:XS 0-9 changed lines (additions + deletions). labels Sep 2, 2026

macroscopeappBot approved these changes Sep 2, 2026
View reviewed changes

maria-rcks merged commit c15735d into pingdotgg:mainSep 2, 2026
26 checks passed

berghtho mentioned this pull request Sep 2, 2026
chore(upstream): sync 2026-09-02 berghtho/t3code#20
Draft

fix(chat): replace failed tools with thinking
ffeac36

github-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XS 0-9 changed lines (additions + deletions). labels Sep 2, 2026

macroscopeappBot reviewed Sep 2, 2026
View reviewed changes

Comment threadapps/web/src/components/chat/MessagesTimeline.logic.ts
Comment threadapps/web/src/components/chat/MessagesTimeline.logic.ts

cursorBot reviewed Sep 2, 2026
View reviewed changes

Comment threadapps/web/src/components/chat/MessagesTimeline.logic.ts

fix(chat): preserve running tool activity
75ceea1

github-actionsBot added size:S 10-29 changed lines (additions + deletions). and removed size:XS 0-9 changed lines (additions + deletions). labels Sep 2, 2026

macroscopeappBot approved these changes Sep 2, 2026
View reviewed changes

maria-rcks merged commit c15735d into pingdotgg:mainSep 2, 2026
26 checks passed

berghtho mentioned this pull request Sep 2, 2026
chore(upstream): sync 2026-09-02 berghtho/t3code#20
Draft

fix(chat): replace failed tools with thinking
ffeac36

github-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XS 0-9 changed lines (additions + deletions). labels Sep 2, 2026

macroscopeappBot reviewed Sep 2, 2026
View reviewed changes

Comment threadapps/web/src/components/chat/MessagesTimeline.logic.ts
Comment threadapps/web/src/components/chat/MessagesTimeline.logic.ts

cursorBot reviewed Sep 2, 2026
View reviewed changes

Comment threadapps/web/src/components/chat/MessagesTimeline.logic.ts

fix(chat): preserve running tool activity
75ceea1

github-actionsBot added size:S 10-29 changed lines (additions + deletions). and removed size:XS 0-9 changed lines (additions + deletions). labels Sep 2, 2026

macroscopeappBot approved these changes Sep 2, 2026
View reviewed changes

maria-rcks merged commit c15735d into pingdotgg:mainSep 2, 2026
26 checks passed

berghtho mentioned this pull request Sep 2, 2026
chore(upstream): sync 2026-09-02 berghtho/t3code#20
Draft

fix(chat): replace failed tools with thinking
ffeac36

github-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XS 0-9 changed lines (additions + deletions). labels Sep 2, 2026

macroscopeappBot reviewed Sep 2, 2026
View reviewed changes

Comment threadapps/web/src/components/chat/MessagesTimeline.logic.ts
Comment threadapps/web/src/components/chat/MessagesTimeline.logic.ts

cursorBot reviewed Sep 2, 2026
View reviewed changes

Comment threadapps/web/src/components/chat/MessagesTimeline.logic.ts

fix(chat): preserve running tool activity
75ceea1

github-actionsBot added size:S 10-29 changed lines (additions + deletions). and removed size:XS 0-9 changed lines (additions + deletions). labels Sep 2, 2026

macroscopeappBot approved these changes Sep 2, 2026
View reviewed changes

maria-rcks merged commit c15735d into pingdotgg:mainSep 2, 2026
26 checks passed

berghtho mentioned this pull request Sep 2, 2026
chore(upstream): sync 2026-09-02 berghtho/t3code#20
Draft

fix(chat): replace failed tools with thinking
ffeac36

github-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XS 0-9 changed lines (additions + deletions). labels Sep 2, 2026

macroscopeappBot reviewed Sep 2, 2026
View reviewed changes

Comment threadapps/web/src/components/chat/MessagesTimeline.logic.ts
Comment threadapps/web/src/components/chat/MessagesTimeline.logic.ts

cursorBot reviewed Sep 2, 2026
View reviewed changes

Comment threadapps/web/src/components/chat/MessagesTimeline.logic.ts

fix(chat): preserve running tool activity
75ceea1

github-actionsBot added size:S 10-29 changed lines (additions + deletions). and removed size:XS 0-9 changed lines (additions + deletions). labels Sep 2, 2026

macroscopeappBot approved these changes Sep 2, 2026
View reviewed changes

maria-rcks merged commit c15735d into pingdotgg:mainSep 2, 2026
26 checks passed

berghtho mentioned this pull request Sep 2, 2026
chore(upstream): sync 2026-09-02 berghtho/t3code#20
Draft

fix(chat): replace failed tools with thinking
ffeac36

github-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XS 0-9 changed lines (additions + deletions). labels Sep 2, 2026

macroscopeappBot reviewed Sep 2, 2026
View reviewed changes

Comment threadapps/web/src/components/chat/MessagesTimeline.logic.ts
Comment threadapps/web/src/components/chat/MessagesTimeline.logic.ts

cursorBot reviewed Sep 2, 2026
View reviewed changes

Comment threadapps/web/src/components/chat/MessagesTimeline.logic.ts

fix(chat): preserve running tool activity
75ceea1

github-actionsBot added size:S 10-29 changed lines (additions + deletions). and removed size:XS 0-9 changed lines (additions + deletions). labels Sep 2, 2026

macroscopeappBot approved these changes Sep 2, 2026
View reviewed changes

maria-rcks merged commit c15735d into pingdotgg:mainSep 2, 2026
26 checks passed

berghtho mentioned this pull request Sep 2, 2026
chore(upstream): sync 2026-09-02 berghtho/t3code#20
Draft

fix(chat): replace failed tools with thinking
ffeac36

github-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XS 0-9 changed lines (additions + deletions). labels Sep 2, 2026

macroscopeappBot reviewed Sep 2, 2026
View reviewed changes

Comment threadapps/web/src/components/chat/MessagesTimeline.logic.ts
Comment threadapps/web/src/components/chat/MessagesTimeline.logic.ts

cursorBot reviewed Sep 2, 2026
View reviewed changes

Comment threadapps/web/src/components/chat/MessagesTimeline.logic.ts

fix(chat): preserve running tool activity
75ceea1

github-actionsBot added size:S 10-29 changed lines (additions + deletions). and removed size:XS 0-9 changed lines (additions + deletions). labels Sep 2, 2026

macroscopeappBot approved these changes Sep 2, 2026
View reviewed changes

maria-rcks merged commit c15735d into pingdotgg:mainSep 2, 2026
26 checks passed

berghtho mentioned this pull request Sep 2, 2026
chore(upstream): sync 2026-09-02 berghtho/t3code#20
Draft

fix(chat): replace failed tools with thinking
ffeac36

github-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XS 0-9 changed lines (additions + deletions). labels Sep 2, 2026

macroscopeappBot reviewed Sep 2, 2026
View reviewed changes

Comment threadapps/web/src/components/chat/MessagesTimeline.logic.ts
Comment threadapps/web/src/components/chat/MessagesTimeline.logic.ts

cursorBot reviewed Sep 2, 2026
View reviewed changes

Comment threadapps/web/src/components/chat/MessagesTimeline.logic.ts

fix(chat): preserve running tool activity
75ceea1

github-actionsBot added size:S 10-29 changed lines (additions + deletions). and removed size:XS 0-9 changed lines (additions + deletions). labels Sep 2, 2026

macroscopeappBot approved these changes Sep 2, 2026
View reviewed changes

maria-rcks merged commit c15735d into pingdotgg:mainSep 2, 2026
26 checks passed

berghtho mentioned this pull request Sep 2, 2026
chore(upstream): sync 2026-09-02 berghtho/t3code#20
Draft

Uh oh!

Conversation

maria-rcks commented Sep 2, 2026• edited by macroscopeapp BotLoading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Replace failed tool rows with thinking activity in deriveMessagesTimelineRows

Uh oh!

Uh oh!

Uh oh!

cursorBot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

macroscopeappBot commented Sep 2, 2026

Approvability

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

Conversation

maria-rcks commented Sep 2, 2026• edited by macroscopeapp BotLoading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Replace failed tool rows with thinking activity in deriveMessagesTimelineRows

Uh oh!

Uh oh!

Uh oh!

cursorBot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

macroscopeappBot commented Sep 2, 2026

Approvability

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

Conversation

maria-rcks commented Sep 2, 2026• edited by macroscopeapp BotLoading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Replace failed tool rows with thinking activity in deriveMessagesTimelineRows

Uh oh!

Uh oh!

Uh oh!

cursorBot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

macroscopeappBot commented Sep 2, 2026

Approvability

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

Conversation

maria-rcks commented Sep 2, 2026• edited by macroscopeapp BotLoading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Replace failed tool rows with thinking activity in deriveMessagesTimelineRows

Uh oh!

Uh oh!

Uh oh!

maria-rcks commented Sep 2, 2026•
edited by macroscopeapp Bot
Loading

Replace failed tool rows with thinking activity in `deriveMessagesTimelineRows`

maria-rcks commented Sep 2, 2026•
edited by macroscopeapp Bot
Loading

Replace failed tool rows with thinking activity in `deriveMessagesTimelineRows`

maria-rcks commented Sep 2, 2026•
edited by macroscopeapp Bot
Loading

Replace failed tool rows with thinking activity in `deriveMessagesTimelineRows`

maria-rcks commented Sep 2, 2026•
edited by macroscopeapp Bot
Loading

Replace failed tool rows with thinking activity in `deriveMessagesTimelineRows`

maria-rcks commented Sep 2, 2026•
edited by macroscopeapp Bot
Loading

Replace failed tool rows with thinking activity in `deriveMessagesTimelineRows`