Skip to content

fix(opencode): skip writing blank model id into Windows config - #2647

Open
patil2001 wants to merge 1 commit into
Osmantic:mainfrom
patil2001:oc2-fix/win-blank-modelid-guard
Open

fix(opencode): skip writing blank model id into Windows config#2647
patil2001 wants to merge 1 commit into
Osmantic:mainfrom
patil2001:oc2-fix/win-blank-modelid-guard

Conversation

@patil2001

Copy link
Copy Markdown
Contributor

Summary

When the Lemonade model id resolved to empty, the Windows config write set model and small_model to an empty value and masked the served model.

Change

The write is now skipped on null or whitespace, mirroring the existing empty value skip behavior in the function.

Test plan

  • Windows install with a blank Lemonade model id no longer produces an empty model entry in the opencode config.

When the Lemonade model id resolved to empty, the Windows config write set
model and small_model to an empty value and masked the served model. The
write is now skipped on null or whitespace, mirroring the existing empty
value skip behavior in the function.
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.

1 participant