shumengya/sproutclaw
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

perf(coding-agent): cache ModelsConfig validator (#1772)

Browse Source
This commit is contained in:
Aarni Koskela
2026-03-03 15:24:02 +02:00
committed by GitHub
parent 6f4bd814b8
commit 993d916e56
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
packages/coding-agent/src/core/model-registry.ts
View File

@@ -28,6 +28,7 @@ import type { AuthStorage } from "./auth-storage.js";
import { clearConfigValueCache, resolveConfigValue, resolveHeaders } from "./resolve-config-value.js"; import { clearConfigValueCache, resolveConfigValue, resolveHeaders } from "./resolve-config-value.js";
const Ajv = (AjvModule as any).default || AjvModule; const Ajv = (AjvModule as any).default || AjvModule;
const ajv = new Ajv();
// Schema for OpenRouter routing preferences // Schema for OpenRouter routing preferences
const OpenRouterRoutingSchema = Type.Object({ const OpenRouterRoutingSchema = Type.Object({
@@ -120,6 +121,8 @@ const ModelsConfigSchema = Type.Object({
providers: Type.Record(Type.String(), ProviderConfigSchema), providers: Type.Record(Type.String(), ProviderConfigSchema),
}); });
ajv.addSchema(ModelsConfigSchema, "ModelsConfig");
type ModelsConfig = Static<typeof ModelsConfigSchema>; type ModelsConfig = Static<typeof ModelsConfigSchema>;
/** Provider override config (baseUrl, headers, apiKey) without custom models */ /** Provider override config (baseUrl, headers, apiKey) without custom models */
@@ -350,8 +353,7 @@ export class ModelRegistry {
const config: ModelsConfig = JSON.parse(content); const config: ModelsConfig = JSON.parse(content);
// Validate schema // Validate schema
const ajv = new Ajv(); const validate = ajv.getSchema("ModelsConfig")!;
const validate = ajv.compile(ModelsConfigSchema);
if (!validate(config)) { if (!validate(config)) {
const errors = const errors =
validate.errors?.map((e: any) => ` - ${e.instancePath || "root"}: ${e.message}`).join("\n") || validate.errors?.map((e: any) => ` - ${e.instancePath || "root"}: ${e.message}`).join("\n") ||