fix: map sparsekernel boolean result

Author: AbrahamGreenman

packages/sparsekernel-client/src/index.ts

@@ -20,6 +20,10 @@ export type SparseKernelInspect = {
   counts: Record<string, number>;
 };
 
+export type SparseKernelBooleanResult = {
+  ok: boolean;
+};
+
 export type SparseKernelResourceBudgets = {
   logical_agents_max: number;
   active_agent_steps_max: number;
@@ -691,7 +695,7 @@ export class SparseKernelClient {
   }
 
   async recordBrowserObservation(input: SparseKernelBrowserObservationInput): Promise<void> {
-    await this.postJson<{ ok: boolean }>("/browser/contexts/observe", input);
+    await this.postJson<SparseKernelBooleanResult>("/browser/contexts/observe", input);
   }
 
   async recordBrowserTarget(
@@ -734,17 +738,17 @@ export class SparseKernelClient {
   }
 
   async heartbeatTask(input: SparseKernelHeartbeatTaskInput): Promise<boolean> {
-    const response = await this.postJson<{ ok: boolean }>("/tasks/heartbeat", input);
+    const response = await this.postJson<SparseKernelBooleanResult>("/tasks/heartbeat", input);
     return response.ok;
   }
 
   async completeTask(input: SparseKernelCompleteTaskInput): Promise<boolean> {
-    const response = await this.postJson<{ ok: boolean }>("/tasks/complete", input);
+    const response = await this.postJson<SparseKernelBooleanResult>("/tasks/complete", input);
     return response.ok;
   }
 
   async failTask(input: SparseKernelFailTaskInput): Promise<boolean> {
-    const response = await this.postJson<{ ok: boolean }>("/tasks/fail", input);
+    const response = await this.postJson<SparseKernelBooleanResult>("/tasks/fail", input);
     return response.ok;
   }
 

scripts/check-sparsekernel-openapi.mjs

@@ -6,6 +6,7 @@ const OPENAPI_PATH = "schemas/sparsekernel.openapi.yaml";
 const DAEMON_PATH = "crates/sparsekernel-cli/src/lib.rs";
 const CLIENT_PATH = "packages/sparsekernel-client/src/index.ts";
 const CLIENT_SCHEMA_MAPPINGS = [
+  mapping("SparseKernelBooleanResult", "BooleanResult"),
   mapping("SparseKernelResourceBudgets", "RuntimeResourceBudgets"),
   mapping("SparseKernelResourceBudgetUpdateInput", "RuntimeResourceBudgetUpdateInput"),
   mapping("SparseKernelNetworkPolicy", "NetworkPolicy"),
@@ -238,6 +239,17 @@ export function checkSparseKernelOpenApi({ openapiText, daemonSource, clientSour
     errors.push(formatList("Unresolved SparseKernel OpenAPI schema refs", unresolvedSchemaRefs));
   }
 
+  const mappedSchemaNames = new Set(CLIENT_SCHEMA_MAPPINGS.map((item) => item.schemaName));
+  const referencedSchemaNames = [...collectSchemaRefs(openapi)]
+    .filter((ref) => ref.startsWith("#/components/schemas/"))
+    .map((ref) => ref.slice("#/components/schemas/".length));
+  pushSetDiff(
+    errors,
+    "SparseKernel OpenAPI referenced schemas missing client parity mapping",
+    new Set(referencedSchemaNames),
+    mappedSchemaNames,
+  );
+
   const inlineRequestBodies = collectOpenApiInlineRequestBodyRoutes(paths);
   if (inlineRequestBodies.size > 0) {
     errors.push(
@@ -247,7 +259,6 @@ export function checkSparseKernelOpenApi({ openapiText, daemonSource, clientSour
       ),
     );
   }
-  const mappedSchemaNames = new Set(CLIENT_SCHEMA_MAPPINGS.map((item) => item.schemaName));
   pushSetDiff(
     errors,
     "SparseKernel OpenAPI request body schemas missing client parity mapping",