Skip to content

Change 'in-memory' to 'in_memory' in prompt cache enums#6187

Merged
IMax153 merged 2 commits intoEffect-TS:mainfrom
alex-dixon:patch-2
Apr 17, 2026
Merged

Change 'in-memory' to 'in_memory' in prompt cache enums#6187
IMax153 merged 2 commits intoEffect-TS:mainfrom
alex-dixon:patch-2

Conversation

@alex-dixon
Copy link
Copy Markdown
Contributor

@alex-dixon alex-dixon commented Apr 17, 2026

https://developers.openai.com/api/docs/guides/prompt-caching#configure-per-request

openai/openai-node#1756

openai/openai-python#2883

lol

Type

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update

Description

We're getting this as a schema error in prod as of 9:14 pacific.

Related

  • Related Issue #
  • Closes #

@alex-dixon alex-dixon requested a review from IMax153 as a code owner April 17, 2026 18:45
@github-project-automation github-project-automation bot moved this to Discussion Ongoing in PR Backlog Apr 17, 2026
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Apr 17, 2026

🦋 Changeset detected

Latest commit: 158e9ee

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@effect/ai-openai Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@IMax153
Copy link
Copy Markdown
Member

IMax153 commented Apr 17, 2026

@alex-dixon needs a changeset (pnpm changeset - can be a patch).

Updated enum values in prompt cache for consistency.
@IMax153 IMax153 merged commit b63fdb8 into Effect-TS:main Apr 17, 2026
11 checks passed
@github-project-automation github-project-automation bot moved this from Discussion Ongoing to Done in PR Backlog Apr 17, 2026
@github-actions github-actions bot mentioned this pull request Apr 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants