Skip to content

Set Responses API prompt cache mode - #327960

Open
dileepyavan wants to merge 7 commits into
mainfrom
DileepY/codespaces_investigate
Open

Set Responses API prompt cache mode#327960
dileepyavan wants to merge 7 commits into
mainfrom
DileepY/codespaces_investigate

Conversation

@dileepyavan

Copy link
Copy Markdown
Member

Summary

  • set the request-wide Responses API prompt_cache_options.mode to explicit when prompt cache breakpoints are enabled
  • default the request-wide cache mode to implicit otherwise
  • add coverage for both request-wide cache policy modes

Testing

  • npm run test:unit -- src/platform/endpoint/node/test/responsesApi.spec.ts
  • npm run typecheck

Signed-off-by: Dileep Yavanamandha <dileepy@microsoft.com>
Copilot AI review requested due to automatic review settings July 29, 2026 05:14
dmitrivMS
dmitrivMS previously approved these changes Jul 29, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds request-wide prompt cache modes to Responses API requests.

Changes:

  • Adds prompt_cache_options to the endpoint body type.
  • Selects explicit or implicit caching and adds unit assertions.
Show a summary per file
File Description
networking.ts Defines prompt cache options.
responsesApi.ts Sets the request-wide cache mode.
responsesApi.spec.ts Tests implicit and explicit modes.

Review details

  • Files reviewed: 3/3 changed files
  • Comments generated: 1
  • Review effort level: Medium

top_logprobs: options.postOptions.logprobs ? 3 : undefined,
store: false,
text: verbosity ? { verbosity } : undefined,
prompt_cache_options: { mode: promptCacheBreakpointsEnabled ? 'explicit' : 'implicit' },
@github-actions

github-actions Bot commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Screenshot Changes

Base: d2ba34a7 Current: 6fed48e8

Changed (6)

agentsVoice/voiceModeOnboarding/Voice Mode onboarding (panel)/Dark
Before After
before after
agentsVoice/voiceModeOnboarding/Voice Mode onboarding (panel)/Light
Before After
before after
agentsVoice/voiceModeOnboarding/Voice Mode onboarding (wide)/Dark
Before After
before after
agentsVoice/voiceModeOnboarding/Voice Mode onboarding (wide)/Light
Before After
before after
agentsVoice/voiceModeOnboarding/Voice Mode onboarding (narrow)/Dark
Before After
before after
agentsVoice/voiceModeOnboarding/Voice Mode onboarding (narrow)/Light
Before After
before after

Signed-off-by: Dileep Yavanamandha <dileepy@microsoft.com>
TylerLeonhardt
TylerLeonhardt previously approved these changes Jul 29, 2026
Signed-off-by: Dileep Yavanamandha <dileepy@microsoft.com>
Signed-off-by: Dileep Yavanamandha <dileepy@microsoft.com>
Signed-off-by: Dileep Yavanamandha <dileepy@microsoft.com>
Signed-off-by: Dileep Yavanamandha <dileepy@microsoft.com>
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.

5 participants