Ayuriel/openclaw
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(image): address review feedback - remove redundant regex and dead code

Browse Source
This commit is contained in:
忻役 2026-03-13 14:51:25 +08:00 committed by Jerry-Xin
parent 0a59973284
commit 7f3c2a50e6
1 changed files with 8 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
src/agents/pi-embedded-runner/model.ts
View File

@ -49,8 +49,7 @@ function isLikelyVisionModel(modelId: string): boolean {
// - Pixtral models
const visionPatterns = [
/vision/,
/\bvl\b/, // "vl" as a separate segment (e.g., qwen-vl, MiniMax-VL-01)
/-vl-/, // "vl" as a segment (e.g., qwen2-vl-72b)
/\bvl\b/, // "vl" as a separate segment (e.g., qwen-vl, qwen2-vl-72b, MiniMax-VL-01)
/visual/,
/claude-3/,
/claude-opus-4/,
@ -315,17 +314,9 @@ export function resolveModelWithRegistry(params: {
// OpenRouter is a pass-through proxy - any model ID available on OpenRouter
// should work without being pre-registered in the local catalog.
// This fallback uses heuristics when the plugin-based capability lookup returns nothing.
// Note: configured models with provider-level `api` return early via inlineMatch,
// so we rely on heuristic detection for vision support here.
if (normalizedProvider === "openrouter") {
// Check if the provider config specifies input capabilities for this model
const configuredModel = providerConfig?.models?.find((candidate) => candidate.id === modelId);
const configuredInput = configuredModel?.input;
// Use configured input if available, otherwise detect vision models by ID pattern
const resolvedInput: Array<"text" | "image"> =
Array.isArray(configuredInput) && configuredInput.length > 0
? configuredInput.filter((item): item is "text" | "image" => item === "text" || item === "image")
: isLikelyVisionModel(modelId)
? ["text", "image"]
: ["text"];
return normalizeResolvedModel({
provider,
cfg,
@ -336,12 +327,12 @@ export function resolveModelWithRegistry(params: {
api: "openai-completions",
provider,
baseUrl: "https://openrouter.ai/api/v1",
reasoning: configuredModel?.reasoning ?? false,
input: resolvedInput,
cost: configuredModel?.cost ?? { input: 0, output: 0, cacheRead: 0, cacheWrite: 0 },
contextWindow: configuredModel?.contextWindow ?? DEFAULT_CONTEXT_TOKENS,
reasoning: false,
input: isLikelyVisionModel(modelId) ? ["text", "image"] : ["text"],
cost: { input: 0, output: 0, cacheRead: 0, cacheWrite: 0 },
contextWindow: DEFAULT_CONTEXT_TOKENS,
// Align with OPENROUTER_DEFAULT_MAX_TOKENS in models-config.providers.ts
maxTokens: configuredModel?.maxTokens ?? 8192,
maxTokens: 8192,
} as Model<Api>,
});
}