vultr/openclaw
1
0
Fork 0
mirror of https://github.com/openclaw/openclaw.git synced 2026-05-06 11:20:43 +00:00
Code Issues Packages Projects Releases Wiki Activity

refactor(agents): clarify prompt cache compatibility gates

Browse Source
This commit is contained in:
Peter Steinberger
2026-04-16 14:58:07 -07:00
parent d59604b15e
commit a98754d504
2 changed files with 25 additions and 23 deletions
Download Patch File Download Diff File Expand all files Collapse all files

38
src/agents/provider-attribution.ts
View File

@@ -573,6 +573,23 @@ export function resolveProviderRequestCapabilities(
compatibilityFamily = "moonshot";
}
const isResponsesApi = api !== undefined && OPENAI_RESPONSES_APIS.has(api);
const promptCacheKeySupport = input.compat?.supportsPromptCacheKey;
// Default strip behavior (proxy-like endpoints with responses APIs) is
// preserved as a safety net for providers that reject prompt_cache_key,
// see #48155 (Volcano Engine DeepSeek). Operators running their payload
// through an OpenAI-compatible proxy known to forward the field
// (CLIProxy, LiteLLM, etc.) can opt out via compat.supportsPromptCacheKey
// to recover prompt caching; providers known to reject the field can
// force the strip with compat.supportsPromptCacheKey = false even on
// native endpoints.
const shouldStripResponsesPromptCache =
promptCacheKeySupport === true
? false
: promptCacheKeySupport === false
? isResponsesApi
: isResponsesApi && policy.usesExplicitProxyLikeEndpoint;
return {
...policy,
isKnownNativeEndpoint,
@@ -599,29 +616,14 @@ export function resolveProviderRequestCapabilities(
(endpointClass === "default" || endpointClass === "anthropic-public"),
// This is intentionally the gate for emitting `store: false` on Responses
// transports, not just a statement about vendor support in the abstract.
supportsResponsesStoreField:
input.compat?.supportsStore !== false && api !== undefined && OPENAI_RESPONSES_APIS.has(api),
supportsResponsesStoreField: input.compat?.supportsStore !== false && isResponsesApi,
allowsResponsesStore:
input.compat?.supportsStore !== false &&
provider !== undefined &&
api !== undefined &&
OPENAI_RESPONSES_APIS.has(api) &&
isResponsesApi &&
OPENAI_RESPONSES_PROVIDERS.has(provider) &&
policy.usesKnownNativeOpenAIEndpoint,
// Default strip behavior (proxy-like endpoints with responses APIs) is
// preserved as a safety net for providers that reject prompt_cache_key —
// see #48155 (Volcano Engine DeepSeek). Operators running their payload
// through an OpenAI-compatible proxy known to forward the field
// (CLIProxy, LiteLLM, etc.) can opt out via compat.supportsPromptCacheKey
// to recover prompt caching; providers known to reject the field can
// force the strip with compat.supportsPromptCacheKey = false even on
// native endpoints.
shouldStripResponsesPromptCache:
input.compat?.supportsPromptCacheKey === true
? false
: input.compat?.supportsPromptCacheKey === false
? api !== undefined && OPENAI_RESPONSES_APIS.has(api)
: api !== undefined && OPENAI_RESPONSES_APIS.has(api) && policy.usesExplicitProxyLikeEndpoint,
shouldStripResponsesPromptCache,
// Native endpoint class is the real signal here. Users can point a generic
// provider key at Moonshot or DashScope and still need streaming usage.
supportsNativeStreamingUsageCompat:

10
test/scripts/bundle-a2ui.test.ts
View File

@@ -65,16 +65,16 @@ describe("scripts/bundle-a2ui.mjs", () => {
const repoRoot = process.cwd();
const dependencyPaths = getResolvedBundleDependencyPackageJsonPaths(repoRoot);
const relativeDependencyPaths = dependencyPaths.map((dependencyPath) =>
path.relative(repoRoot, dependencyPath),
path.relative(repoRoot, dependencyPath).replaceAll(path.sep, "/"),
);
expect(
relativeDependencyPaths.map((relativePath) => relativePath.replace(/^ui\//u, "")),
).toEqual([
path.join("node_modules", "lit", "package.json"),
path.join("node_modules", "@lit/context", "package.json"),
path.join("node_modules", "@lit-labs/signals", "package.json"),
path.join("node_modules", "signal-utils", "package.json"),
path.posix.join("node_modules", "lit", "package.json"),
path.posix.join("node_modules", "@lit/context", "package.json"),
path.posix.join("node_modules", "@lit-labs/signals", "package.json"),
path.posix.join("node_modules", "signal-utils", "package.json"),
]);
expect(
relativeDependencyPaths.every((relativePath) => /^(ui\/)?node_modules\//u.test(relativePath)),