Skip to content

fix: preserve system prompts for Kimi Coding - #206

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

hetaoBackend merged 3 commits into
mainfrom
fix/kimi-coding-system-role

Conversation

@SaladDay

@SaladDay SaladDay commented Sep 19, 2026 •

Copy link
Copy Markdown
Collaborator

Change

Preserve the system role for thinking-enabled custom OpenAI-compatible connections to Kimi Coding (api.kimi.com and api.kimi.ai). Keep thinking, explicit compatibility overrides, other request options, and other providers unchanged.

This PR contains only the Kimi fix. DashScope is being submitted separately. Authentication, thinking-toggle behavior, and connection-test redesign are outside scope.

Validation

  • The two endpoint regressions failed before the fix; the focused resolver suite passed all 62 tests.
  • All 14 full-profile verification gates passed on macOS arm64 / Node 26.7.0 at 77ca68e. Current HEAD e0193bd has an identical file tree after reverting the DashScope addition.
  • The original Kimi-only independent blind review found no actionable issues.

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.

@SaladDay SaladDay changed the title fix: preserve system prompts for Kimi Coding fix: preserve system prompts for Kimi Coding and DashScope Sep 19, 2026
@SaladDay SaladDay changed the title fix: preserve system prompts for Kimi Coding and DashScope fix: preserve system prompts for Kimi Coding Sep 19, 2026
@hetaoBackend
hetaoBackend merged commit ab67683 into main Sep 20, 2026
17 of 19 checks passed
@hetaoBackend
hetaoBackend deleted the fix/kimi-coding-system-role branch September 20, 2026 03:10
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