Skip to content

[WIP] Rename DeepSeekClient and internal types to provider-neutral types - #5106

Closed
Hmbown with Copilot wants to merge 2 commits into
mainfrom
copilot/rename-deepseekclient-legacy-internals
Closed

[WIP] Rename DeepSeekClient and internal types to provider-neutral types#5106
Hmbown with Copilot wants to merge 2 commits into
mainfrom
copilot/rename-deepseekclient-legacy-internals

Conversation

Copilot AI commented Aug 3, 2026

Copy link
Copy Markdown
Contributor
  • Audit current DeepSeekClient / deepseek_client / deepseek_api_key usage and identify provider-neutral rename targets
  • Rename shared client type and engine wiring to provider-neutral names without behavior changes
  • Update related docs/comments and active-route API key helper naming where appropriate (keep DeepSeek-specific helpers where truly provider-specific)
  • Add/adjust focused tests for renamed helpers and wiring references if needed
  • Run targeted tui client/engine/config tests plus grep verification for legacy symbols
  • Run security checks (secret scan + CodeQL) and report final status

@Hmbown

Hmbown commented Aug 13, 2026

Copy link
Copy Markdown
Owner

Triaging this long-stale draft: closing it to keep the board honest. The intent is preserved in #5103 (moved to v0.9.8) — a fresh, smaller PR from current main is very welcome whenever you want to pick it back up.

@Hmbown Hmbown closed this Aug 13, 2026
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.

Rename DeepSeekClient / deepseek_client legacy internals to provider-neutral types

2 participants