Repository navigation
fix(server): a tool call with a parameter named -i is no longer dropped - #837
Merged
Merged
Conversation
Claude Code's Grep names its flags -i, -n, -A, -B and -C. The DSML and XML parameter patterns took names that start with a letter or underscore, so a DeepSeek call passing one of those flags left text the parser could not account for, and the whole call was dropped: Claude Code ended the task after one sentence. Parameter names may now start with '-'. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
davide221
commented
Oct 11, 2026
davide221
left a comment
Contributor
Author
There was a problem hiding this comment.
Code review
No issues found. Checked for bugs and logic errors in the changed code.
Reviewed 50aa0d05 · 0 candidate issues checked, kept at ≥70% confidence · add the code-review label to review again
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
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.
Claude Code's Grep tool names its flags
-i,-n,-A,-Band-C. When DeepSeek V4.1 Flash passed one of them, the DSML parameter pattern did not match the name, the call body had text left over, and the parser dropped the whole call. Claude Code then got a reply with one sentence and no tool use, and ended the task.Change
Parameter names in the DSML and XML attribute patterns may start with
-. Tool names are unchanged.Measured
DeepSeek V4.1 Flash (
ds41-gorgon, R9700 + Strix Halo, 192 GB), Claude Code 2.1.296 on a clean config (22 tools), asked to explain the prefix cache's eviction policy in this repository:end_turn: the model'sGrepcall with-iwas droppedGrepcalls with-i,tool_useTests
test_server_unitGrepcall with-iand-nBuilt for gfx1151 + gfx1201.
🤖 Generated with Claude Code