Auto-Reply: relax regex for reply tags to allow whitespace#560
Merged
Conversation
mcinteerj
force-pushed
the
fix/reply-tags-whitespace
branch
from
January 9, 2026 09:00
e96ae07 to
dbccba5
Compare
steipete
force-pushed
the
fix/reply-tags-whitespace
branch
from
January 9, 2026 16:05
dbccba5 to
7d9300e
Compare
Contributor
|
Landed via temp rebase onto main.\n\n- Gate: pnpm lint && pnpm build && pnpm test\n- Land commit: 7d9300e\n- Merge commit: 67af3c3\n\nThanks @mcinteerj! |
heatherstew44-maker
pushed a commit
to heatherstew44-maker/openclaw
that referenced
this pull request
Mar 8, 2026
) * Fix race condition when toggling transcription via SIGUSR2 The toggle state was being set to false immediately when stop was called, but the actual transcription runs asynchronously. If another signal arrived before transcription completed, a new recording could start while the old transcription was still running, causing the old text to paste unexpectedly. Now the toggle state stays true until the async transcription task completes. * remove import --------- Co-authored-by: CJ Pais <[email protected]>
lovewanwan
pushed a commit
to lovewanwan/openclaw
that referenced
this pull request
Apr 28, 2026
lovewanwan
pushed a commit
to lovewanwan/openclaw
that referenced
this pull request
Apr 28, 2026
Auto-Reply: relax regex for reply tags to allow whitespace
25 tasks
github-actions Bot
pushed a commit
to Desicool/openclaw
that referenced
this pull request
May 9, 2026
github-actions Bot
pushed a commit
to Desicool/openclaw
that referenced
this pull request
May 9, 2026
Auto-Reply: relax regex for reply tags to allow whitespace
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Overview
Relaxed the regex patterns for
[[reply_to_current]]and[[reply_to:...]]tags to allow for whitespace characters.Context
Some models (e.g.,
gemini-flash-3-preview) may output tags with internal spaces, such as[[ reply_to_current ]]or[[ reply_to: ... ]]. The previous strict regex failed to detect and strip these tags, causing them to appear in the final message sent to users.Changes
extractReplyToTaginsrc/auto-reply/reply/reply-tags.tsto use\s*in regex patterns.Testing
vitestwith reproduction test cases covering:[[ reply_to_current ]](spaces inside)[[reply_to: 123]](spaces around colon)