style(openai): trim spark suppression comments

This commit is contained in:
Ayaan Zaidi
2026-06-15 12:57:15 +05:30
parent aaa5ce6280
commit dae2bcf31b
3 changed files with 0 additions and 8 deletions
@@ -163,8 +163,6 @@ describe("resolveModel forward-compat errors and overrides", () => {
});
it("rejects direct openai gpt-5.3-codex-spark with a codex-only hint", () => {
// Spark is intentionally suppressed from direct OpenAI routing; falling back
// would make a removed catalog row appear usable.
const result = resolveModelForTest("openai", "gpt-5.3-codex-spark", "/tmp/agent");
expect(result.model).toBeUndefined();
@@ -880,8 +880,6 @@ function shouldSuppressInlineConfiguredModel(params: {
) {
return false;
}
// Spark is a ChatGPT/Codex OAuth-only OpenAI model. Inline custom endpoints
// may still opt in, but direct/default OpenAI API transports must fail here.
return shouldSuppressBuiltInModel({
provider: params.provider,
id: params.modelId,
@@ -1213,8 +1211,6 @@ function resolveRuntimePreferredSuppressedModel(params: {
) {
return undefined;
}
// Runtime-preferred providers may restore an auth-bound model while the
// suppression still blocks stale static catalog rows for other auth modes.
return resolvePluginDynamicModelWithRegistry({ ...params, runtimeHooks });
}
@@ -105,8 +105,6 @@ function manifestSuppressionMatchesConditions(params: {
}
if (when.baseUrlHosts?.length) {
const baseUrlHost = normalizeBaseUrlHost(params.baseUrl ?? configuredProvider?.baseUrl);
// Missing transport metadata usually means a stale/default catalog row.
// Suppress it unless config names a custom base URL to check.
if (!baseUrlHost && !params.baseUrl && !configuredProvider?.baseUrl) {
return true;
}