opencode icon indicating copy to clipboard operation
opencode copied to clipboard

feat: add configuration for open ai store option on provider level

Open thebongy opened this issue 3 weeks ago • 3 comments

For organizations with ZDR enabled, currently we need to manually set the "store" option for every open ai model in configuration. This change adds support to specify it once on the provider level instead. Ref: https://github.com/anomalyco/opencode/issues/2966

thebongy avatar Jan 08 '26 10:01 thebongy

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

No duplicate PRs found

github-actions[bot] avatar Jan 08 '26 10:01 github-actions[bot]

A seperate note, but this existing snippet seems to possibly be dead code, or uninentional:

      if (model.providerID.includes("codex")) {
        result["store"] = false
      }

The "provider ID" would not possibly include codex, was this supposed to check against the model with model.api.id? Or is there actually a provider with the keyword "codex" in it? But not sure why this was required in the first place as well 😅

thebongy avatar Jan 08 '26 10:01 thebongy

cc @rekram1-node can I bug you for a review on this one?

need it to use OpenAI models & their zero data retention config in OC.

elithrar avatar Jan 08 '26 12:01 elithrar