Skip to content

1#12095

Closed
baixiangcpp wants to merge 7 commits into
anomalyco:devfrom
baixiangcpp:dev
Closed

1#12095
baixiangcpp wants to merge 7 commits into
anomalyco:devfrom
baixiangcpp:dev

Conversation

@baixiangcpp

Copy link
Copy Markdown

What does this PR do?

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?

The previous system reminder logic wrapped every queued user message with XML tags and duplicated the user's text, causing significant input token amplification (O(N * Length)).

This commit refactors the reminder mechanism to:
1. Stop modifying user messages directly.
2. Inject a concise system prompt only when necessary.
3. Implement exponential backoff (15s to 5min) for reminders to prevent excessive interruption during long tasks.
4. Add state management to deduplicate reminders for the same queued message.
@github-actions

github-actions Bot commented Feb 4, 2026

Copy link
Copy Markdown
Contributor

Hey! Your PR title 1 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

github-actions Bot commented Feb 4, 2026

Copy link
Copy Markdown
Contributor

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

The search returned PR #12095 (the current PR) with the title "1", but no meaningful description to search against. Since the PR title is just "1" and the description is only a template placeholder with no actual content describing what the PR does, there's not enough information to identify genuine duplicates.

No duplicate PRs found - The PR lacks substantive title and description content needed for duplicate detection. Please provide actual PR details with a meaningful title and description of the changes.

@baixiangcpp baixiangcpp closed this Feb 4, 2026
@baixiangcpp

Copy link
Copy Markdown
Author

1

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant