refactor: clean manifest catalog mapping

shakkernerd · shakkernerd · commit 6b6f140c42d2 · 2026-05-02T06:21:08.000+01:00
diff --git a/src/agents/model-catalog.ts b/src/agents/model-catalog.ts
@@ -136,16 +136,27 @@ export function loadManifestModelCatalog(params: {
   const plan = planManifestModelCatalogRows({
     registry: { plugins: eligiblePlugins },
   });
-  return plan.rows.map((row) => ({
-    id: row.id,
-    name: row.name,
-    provider: row.provider,
-    ...(row.contextWindow ? { contextWindow: row.contextWindow } : {}),
-    ...(row.contextTokens && !row.contextWindow ? { contextWindow: row.contextTokens } : {}),
-    ...(typeof row.reasoning === "boolean" ? { reasoning: row.reasoning } : {}),
-    ...(row.input?.length ? { input: [...row.input] } : {}),
-    ...(row.compat ? { compat: row.compat } : {}),
-  }));
+  return plan.rows.map((row) => {
+    const entry: ModelCatalogEntry = {
+      id: row.id,
+      name: row.name,
+      provider: row.provider,
+    };
+    const contextWindow = row.contextWindow ?? row.contextTokens;
+    if (contextWindow) {
+      entry.contextWindow = contextWindow;
+    }
+    if (typeof row.reasoning === "boolean") {
+      entry.reasoning = row.reasoning;
+    }
+    if (row.input?.length) {
+      entry.input = [...row.input];
+    }
+    if (row.compat) {
+      entry.compat = row.compat;
+    }
+    return entry;
+  });
 }
 
 export async function loadModelCatalog(params?: {
diff --git a/src/agents/tools/manifest-capability-availability.ts b/src/agents/tools/manifest-capability-availability.ts
@@ -165,6 +165,7 @@ function metadataKeyForCapabilityContract(
     case "mediaUnderstandingProviders":
       return undefined;
   }
+  return undefined;
 }
 
 function normalizeBaseUrlForManifestGuard(value: string): string {

Original file line number	Diff line number	Diff line change
`@@ -165,6 +165,7 @@ function metadataKeyForCapabilityContract(`
`165`	`165`	`case "mediaUnderstandingProviders":`
`166`	`166`	`return undefined;`
`167`	`167`	`}`
	`168`	`+ return undefined;`
`168`	`169`	`}`
`169`	`170`
`170`	`171`	`function normalizeBaseUrlForManifestGuard(value: string): string {`