Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[codex] Hot-reload DeepSeek connection config#97

Open
Star-Star66 wants to merge 1 commit into
Yapie0:main from
Star-Star66:codex/issue-95-config-hot-reload
Open

[codex] Hot-reload DeepSeek connection config #97
Star-Star66 wants to merge 1 commit into
Yapie0:main from
Star-Star66:codex/issue-95-config-hot-reload

Conversation

@Star-Star66

@Star-Star66 Star-Star66 commented Jun 12, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

Summary

  • watch ~/.carboncode/config.json for DeepSeek API key and base URL changes
  • apply connection changes only while the loop is idle, preserving the active session and transcript
  • update the main loop, context folding, balance/model refreshes, and future subagents to use the replacement client
  • preserve explicit environment-variable overrides and ignore partially written JSON

Why

Issue #95 notes that changing provider connection settings currently requires restarting Carbon Code. The TUI constructed one DeepSeekClient at startup, and both context folding and the subagent closure retained that original client.

This change atomically replaces the client after the current turn finishes, so dashboard or config-file edits take effect within about one second without losing conversation state.

Addresses the configuration hot-reload suggestion in #95 without closing the broader issue.

Validation

  • npm run verify
  • 340 test files passed
  • 3918 tests passed, 2 skipped

@Star-Star66 Star-Star66 force-pushed the codex/issue-95-config-hot-reload branch from 5c2ebb3 to ae2e6e2 Compare June 13, 2026 02:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

AltStyle によって変換されたページ (->オリジナル) /