Skip to content
This repository was archived by the owner on Aug 6, 2026. It is now read-only.

Remove "Looping" from agent thinking messages - #3896

Merged
trunk-io[bot] merged 1 commit into
mainfrom
posthog-code/remove-looping-thinking-message
Jul 28, 2026
Merged

Remove "Looping" from agent thinking messages#3896
trunk-io[bot] merged 1 commit into
mainfrom
posthog-code/remove-looping-thinking-message

Conversation

@rafaeelaudibert

Copy link
Copy Markdown
Member

Problem

The word "Looping" showing while the agent is thinking is misleading — it collides with the "loops" feature and implies the agent is running a loop when it isn't. Raised in a Slack thread.

Changes

Removed "Looping" from the THINKING_MESSAGES list in both places it lives (the mobile chat util and the shared GeneratingIndicator component). All other status words are unchanged.

How did you test this?

No tests run — this is a one-word deletion from a static string array.

Automatic notifications

  • Publish to changelog?
  • Alert Sales and Marketing teams?

Created with PostHog from a Slack thread

"Looping" collides with the "loops" feature and is misleading since the agent isn't running a loop while thinking. Removed from both copies of THINKING_MESSAGES.

Generated-By: PostHog Code
Task-Id: 5488fce5-0445-49d8-8521-74b8eb681683
@trunk-io

trunk-io Bot commented Jul 28, 2026

Copy link
Copy Markdown

😎 Merged successfully - details.

@posthog

posthog Bot commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

es, we control those — they live in a THINKING_MESSAGES list (duplicated in the mobile chat util and the shared GeneratingIndicator component). "Looping" was in there, sandwiched between "Nesting" and "Wiring".

Removed it from both copies and opened a draft PR: #3896.

@rafaeelaudibert
rafaeelaudibert marked this pull request as ready for review July 28, 2026 20:04
@rafaeelaudibert rafaeelaudibert added the Stamphog This will request an autostamp by stamphog on small changes label Jul 28, 2026
@rafaeelaudibert
rafaeelaudibert enabled auto-merge (squash) July 28, 2026 20:04
@github-actions

Copy link
Copy Markdown

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit 4d63cdf.

@greptile-apps

greptile-apps Bot commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Reviews (1): Last reviewed commit: "Remove "Looping" from agent thinking mes..." | Re-trigger Greptile

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Trivial cosmetic change removing one word from a static UI string array in two files; no risky territory, no behavior change beyond display text.

  • 👍 on the PR from greptile-apps[bot].
Gate mechanics and policy version
Gate Result
prerequisites all clear
deny-list no deny categories matched
size 2L, 2F substantive — within ceiling
tier T1-agent / T1b-small (2L, 2F, two-areas, unknown)
stamphog 2.0.0b3 .stamphog/policy.yml @ 608b6d5 · reviewed head 4d63cdf

@trunk-io
trunk-io Bot merged commit 3e5f136 into main Jul 28, 2026
40 of 41 checks passed
@trunk-io
trunk-io Bot deleted the posthog-code/remove-looping-thinking-message branch July 28, 2026 20:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Stamphog This will request an autostamp by stamphog on small changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant