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

Appearance settings

Comments

Close side panel

fix: Persist loaded model and mode on ACP session load#9829

Merged
rekram1-node merged 1 commit intoanomalyco:devanomalyco/opencode:devfrom
noamzbr:fix-9828-set-model-mode-on-loadsessionnoamzbr/opencode:fix-9828-set-model-mode-on-loadsessionCopy head branch name to clipboard
Jan 21, 2026
Merged

fix: Persist loaded model and mode on ACP session load#9829
rekram1-node merged 1 commit intoanomalyco:devanomalyco/opencode:devfrom
noamzbr:fix-9828-set-model-mode-on-loadsessionnoamzbr/opencode:fix-9828-set-model-mode-on-loadsessionCopy head branch name to clipboard

Conversation

@noamzbr
Copy link
Contributor

@noamzbr noamzbr commented Jan 21, 2026

Fixes #9828

Fixes my pervious #7809 by also using set model and set mode to change the actual model and mode of the loaded session (verified working this time)

@github-actions
Copy link
Contributor

The following comment was made by an LLM, it may be inaccurate:

Based on my search, I found the following related PRs that might address similar functionality:

Related PRs:

  1. PR fix(acp): persist session state to disk and add isReplay metadata #6563 - fix(acp): persist session state to disk and add isReplay metadata

    • This PR also deals with ACP session persistence, which is related to the current PR's goal of persisting model and mode on session load.
  2. PR feat(acp): add session persistence and history replay #5153 - feat(acp): add session persistence and history replay

    • This PR addresses session persistence and replay functionality for ACP, which overlaps with the current PR's persistence concerns.

These PRs are related to session state persistence in ACP, though the current PR (#9829) appears to be addressing a specific issue (#9828) about persisting the loaded model and mode. The previous attempt in PR #7809 is referenced in the description as being fixed by this PR.

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.

ACP loadSessionMode doesn't set model and mode

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.