@indigoai-us/hq-cloud 6.14.16 → 6.14.17

package/src/vault-client.test.ts

@@ -365,7 +365,7 @@ describe("API surface", () => {
 
   it("postTelemetryEvents sends action events without personUid", async () => {
     fetchSpy.mockResolvedValueOnce(
-      jsonResponse(200, { ok: true, written: 1, skipped: [] }),
+      jsonResponse(200, { contractVersion: 1, ok: true, written: 1, skipped: [] }),
     );
     const batch: TelemetryEventsBatch = {
       events: [
@@ -397,6 +397,17 @@ describe("API surface", () => {
     expect(JSON.stringify(body)).not.toContain("personUid");
   });
 
+  it("postTelemetryEvents normalizes the compatible pre-v1 accepted response", async () => {
+    fetchSpy.mockResolvedValueOnce(jsonResponse(200, { accepted: 2, deduped: 1 }));
+    const batch: TelemetryEventsBatch = { events: [] };
+
+    await expect(client.postTelemetryEvents(batch)).resolves.toEqual({
+      ok: true,
+      written: 2,
+      skipped: [],
+    });
+  });
+
   it("updateRole sends correct payload", async () => {
     fetchSpy.mockResolvedValueOnce(
       jsonResponse(200, {

package/src/vault-client.ts

@@ -462,7 +462,14 @@ export interface TelemetryEventsBatch {
   events: RawTelemetryEventInput[];
 }
 
-/** Same wire shape as `UsageIngestResult`; aliased for call-site clarity. */
+/**
+ * Normalized response shape for `POST /v1/telemetry/events`.
+ *
+ * hq-pro response contract v1 uses the established ingest fields (`ok`,
+ * `written`, `skipped`). The decoder also accepts the briefly deployed
+ * `{ accepted, deduped }` shape so clients remain compatible while the server
+ * rollout catches up.
+ */
 export type TelemetryEventsIngestResult = UsageIngestResult;
 
 // ---------------------------------------------------------------------------
@@ -649,6 +656,41 @@ const usageIngestResultSchema: VaultResponseSchema<UsageIngestResult> = z
   })
   .strip();
 
+/**
+ * Accept the canonical v1 response and the previous raw-event aliases, then
+ * normalize at this boundary so telemetry remains best-effort during rollout.
+ */
+const telemetryEventsIngestResultSchema: VaultResponseSchema<TelemetryEventsIngestResult> = z
+  .object({
+    contractVersion: z.literal(1).optional(),
+    ok: z.boolean().optional(),
+    written: z.number().nonnegative().optional(),
+    skipped: z
+      .array(
+        z
+          .object({
+            index: z.number(),
+            code: z.string(),
+            error: z.string(),
+          })
+          .strip(),
+      )
+      .optional(),
+    // Compatibility aliases emitted by hq-pro before response contract v1.
+    accepted: z.number().nonnegative().optional(),
+    deduped: z.number().nonnegative().optional(),
+  })
+  .refine(
+    (value) =>
+      value.ok !== undefined || value.written !== undefined || value.accepted !== undefined,
+    { message: "telemetry ingest response has no recognized result fields" },
+  )
+  .transform((value) => ({
+    ok: value.ok ?? true,
+    written: value.written ?? value.accepted ?? 0,
+    skipped: value.skipped ?? [],
+  }));
+
 const createInviteResponseSchema: VaultResponseSchema<CreateInviteResult> = z
   .object({
     membership: membershipSchema,
@@ -1349,7 +1391,7 @@ export class VaultClient {
     batch: TelemetryEventsBatch,
     options: { timeoutMs?: number } = {},
   ): Promise<TelemetryEventsIngestResult> {
-    return this.post("/v1/telemetry/events", batch, usageIngestResultSchema, {
+    return this.post("/v1/telemetry/events", batch, telemetryEventsIngestResultSchema, {
       timeoutMs: options.timeoutMs ?? 1500,
       maxRetries: 0,
     });