Skip to content

fix: preserve system prompts for DashScope - #207

Merged
hetaoBackend merged 3 commits into
mainfrom
fix/dashscope-system-role
Sep 20, 2026
Merged

hetaoBackend merged 3 commits into
mainfrom
fix/dashscope-system-role

Conversation

@SaladDay

@SaladDay SaladDay commented Sep 19, 2026 •

Copy link
Copy Markdown
Collaborator

Change

Preserve system for thinking-enabled OpenAI-compatible connections to official DashScope endpoints: standard regional hosts, Coding Plan, and the documented regional workspace, trial, and Token Plan hosts.

Only the default message-role capability changes. Thinking, other request options, explicit compatibility overrides, and other providers retain their behavior. Kimi (#206) is already in main and remains unchanged. Authentication, thinking-toggle behavior, connection-test redesign, and unknown gateways are outside scope. This PR targets main.

Stack

main -> #207 -> #209 -> #210

Merge #207 first. After each parent lands in main, update the remaining stack and retarget the next PR to main before merging it. Each PR is reviewed against its immediate base.

Evidence

Validation

  • Focused resolver suite: 88 tests passed, including official hosts, misleading host/path matches, and explicit overrides. The 12 newly covered endpoints failed before the matcher update.
  • Full verification: all 14 gates passed on macOS arm64 / Node 26.7.0 at 5027e8e.
  • Fresh independent blind review of the stacked changes found no actionable issues; git diff --check passed.

Publication and maintainer handoff

  • Existing licenses and notices are preserved.
  • No credentials, account data, private history, or user content is included.
  • No source files were added or removed; tests extend an already registered file.
  • Documentation impact is limited to the vendored patch ledger.
  • Shared-source port: pending.

@hetaoBackend
hetaoBackend merged commit 71d0cba into main Sep 20, 2026
13 checks passed
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.

2 participants