Skip to content

fix: Doom-loop guard for repeated reasoning/output#12623

Open
Heinrich-XIAO wants to merge 4 commits intoanomalyco:devfrom
Heinrich-XIAO:dev
Open

fix: Doom-loop guard for repeated reasoning/output#12623
Heinrich-XIAO wants to merge 4 commits intoanomalyco:devfrom
Heinrich-XIAO:dev

Conversation

@Heinrich-XIAO
Copy link

@Heinrich-XIAO Heinrich-XIAO commented Feb 7, 2026

What does this PR do?

The doom loop previously only caught loops where a tool call is repeatedly called. This pr adds that same feature for reasoning and normal output.
Fixes: #12716

Please provide a description of the issue (if there is one), the changes you made to fix it, and why they work. It is expected that you understand why your changes work and if you do not understand why at least say as much so a maintainer knows how much to value the pr.

If you paste a large clearly AI generated description here your PR may be IGNORED or CLOSED!

How did you verify your code works?

I ran the dev cmd and it worked.
Screenshot 2026-02-07 at 13 41 34

@github-actions
Copy link
Contributor

github-actions bot commented Feb 7, 2026

Hey! Your PR title Doom-loop guard for repeated reasoning/output doesn't follow conventional commit format.

Please update it to start with one of:

  • feat: or feat(scope): new feature
  • fix: or fix(scope): bug fix
  • docs: or docs(scope): documentation changes
  • chore: or chore(scope): maintenance tasks
  • refactor: or refactor(scope): code refactoring
  • test: or test(scope): adding or updating tests

Where scope is the package name (e.g., app, desktop, opencode).

See CONTRIBUTING.md for details.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 7, 2026

The following comment was made by an LLM, it may be inaccurate:

No duplicate PRs found

@Heinrich-XIAO Heinrich-XIAO changed the title Doom-loop guard for repeated reasoning/output fix: Doom-loop guard for repeated reasoning/output Feb 8, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 8, 2026

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

  1. Open an issue describing the bug/feature (if one doesn't exist)
  2. Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Doom loop is not caught when during reasoning or output

1 participant