Skip to content

fix: Config update endpoint - #4205

Closed
veracioux wants to merge 5 commits into
anomalyco:devfrom
veracioux:fix/config-update-endpoint
Closed

fix: Config update endpoint#4205
veracioux wants to merge 5 commits into
anomalyco:devfrom
veracioux:fix/config-update-endpoint

Conversation

@veracioux

Copy link
Copy Markdown
Contributor

Fixes #4194.


export async function update(config: Info) {
const filepath = path.join(Instance.directory, "config.json")
let filepath = path.join(Instance.directory, "opencode.json")

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So something I was thinking is that the config.ts state should track all the files it loaded configs for and it wouldn't be that useful in most places but it would allow opencode debug config to have some mode where we state where every file came from and what it had to help users debug

maybe if we had that we wouldn't need to do a file system search each time and then we could also use it to power the config debug command later

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh I think I know the perfect solution for this. I think I'll have an update later today.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rekram1-node I did some fiddling around. Turns out the idea I had in mind might involve a pretty significant refactor. Therefore it's probably best to leave this to you.

@github-actions

github-actions Bot commented Feb 2, 2026

Copy link
Copy Markdown
Contributor

Closing this pull request because it has had no updates for more than 60 days. If you plan to continue working on it, feel free to reopen or open a new 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.

PATCH /config uses wrong config file

3 participants