style(openai): trim spark suppression comments · openclaw/openclaw@dae2bcf
obviyus
·
2026-06-15
·
via Recent Commits to openclaw:main
File tree
agents/embedded-agent-runner
| Original file line number | Diff line number | Diff line change |
|---|
@@ -163,8 +163,6 @@ describe("resolveModel forward-compat errors and overrides", () => {
|
163 | 163 | }); |
164 | 164 | |
165 | 165 | it("rejects direct openai gpt-5.3-codex-spark with a codex-only hint", () => { |
166 | | -// Spark is intentionally suppressed from direct OpenAI routing; falling back |
167 | | -// would make a removed catalog row appear usable. |
168 | 166 | const result = resolveModelForTest("openai", "gpt-5.3-codex-spark", "/tmp/agent"); |
169 | 167 | |
170 | 168 | expect(result.model).toBeUndefined(); |
|
| Original file line number | Diff line number | Diff line change |
|---|
@@ -880,8 +880,6 @@ function shouldSuppressInlineConfiguredModel(params: {
|
880 | 880 | ) { |
881 | 881 | return false; |
882 | 882 | } |
883 | | -// Spark is a ChatGPT/Codex OAuth-only OpenAI model. Inline custom endpoints |
884 | | -// may still opt in, but direct/default OpenAI API transports must fail here. |
885 | 883 | return shouldSuppressBuiltInModel({ |
886 | 884 | provider: params.provider, |
887 | 885 | id: params.modelId, |
@@ -1213,8 +1211,6 @@ function resolveRuntimePreferredSuppressedModel(params: {
|
1213 | 1211 | ) { |
1214 | 1212 | return undefined; |
1215 | 1213 | } |
1216 | | -// Runtime-preferred providers may restore an auth-bound model while the |
1217 | | -// suppression still blocks stale static catalog rows for other auth modes. |
1218 | 1214 | return resolvePluginDynamicModelWithRegistry({ ...params, runtimeHooks }); |
1219 | 1215 | } |
1220 | 1216 | |
|
| Original file line number | Diff line number | Diff line change |
|---|
@@ -105,8 +105,6 @@ function manifestSuppressionMatchesConditions(params: {
|
105 | 105 | } |
106 | 106 | if (when.baseUrlHosts?.length) { |
107 | 107 | const baseUrlHost = normalizeBaseUrlHost(params.baseUrl ?? configuredProvider?.baseUrl); |
108 | | -// Missing transport metadata usually means a stale/default catalog row. |
109 | | -// Suppress it unless config names a custom base URL to check. |
110 | 108 | if (!baseUrlHost && !params.baseUrl && !configuredProvider?.baseUrl) { |
111 | 109 | return true; |
112 | 110 | } |
|
此内容由惯性聚合(RSS阅读器)自动聚合整理,仅供阅读参考。 原文来自 — 版权归原作者所有。