Remove the deepseek-flags default plugin - #275
Merged
Merged
Conversation
deepseek-flags ("Avoid DeepSeek request rejections") replaced a regional
flag emoji that DeepSeek's API refuses. A provider that refuses certain
content is for that provider to fix; core does not rewrite requests to get
around it. Two default plugins remain: reply-language and wsl-paths.
- Delete defaults/deepseek-flags.js, drop it from defaults::ALL, and
regenerate defaults/manifests.json (UPDATE_DEFAULT_MANIFESTS=1).
- Drop its gateway tests and the test-harness helper only they used.
The test that a default able to rewrite tool calls turns on only
through UpdatePluginConfirmed now uses wsl-paths, which also holds
reply_tool_calls.
- release-notes/0.58.0.md: two default plugins.
No published release contained the plugin, so seeded configurations need
no cleanup. Message codes, CONTROL_API_VERSION (34) and SCHEMA (25) do not
change.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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
1 participant
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.
Removes the
deepseek-flagsdefault plugin ("Avoid DeepSeek request rejections"). It replaced a regional flag emoji that DeepSeek's API refuses. A provider that refuses certain content is for that provider to fix; core does not rewrite requests to get around it.Two default plugins remain:
reply-languageandwsl-paths.crates/tw-gateway/src/plugin/defaults/: deletedeepseek-flags.js, drop it fromALL, and regeneratemanifests.json(UPDATE_DEFAULT_MANIFESTS=1).crates/tw-gateway/tests/plugins_defaults.rs: drop its tests, andoutcomes_offrom the plugin test harness, which only they used.crates/tw-control/tests/plugin_defaults.rs: the test that a default able to rewrite tool calls turns on only throughUpdatePluginConfirmednow useswsl-paths, which also holdsreply_tool_calls.release-notes/0.58.0.md: two default plugins.No published release contained the plugin, so seeded configurations need no cleanup. Message codes,
CONTROL_API_VERSION(34) andSCHEMA(25) do not change.🤖 Generated with Claude Code