@tinycloud/sdk-services 2.10.0 → 2.11.0-beta.10

package/dist/kv/index.js

@@ -1192,6 +1192,7 @@ var KVService = class extends BaseService {
         }
         seen.add(path);
       }
+      let requestMayHaveDispatched = false;
       try {
         const body = new FormData();
         for (let index = 0; index < items.length; index++) {
@@ -1209,14 +1210,31 @@ var KVService = class extends BaseService {
             action: KVAction.PUT
           }))
         );
-        const response = await this.context.fetch(`${this.host}/invoke`, {
+        const url = `${this.host}/invoke`;
+        const init = {
           method: "POST",
           headers,
           body,
           signal: this.combineSignals(options?.signal)
-        });
+        };
+        const fetchFn = this.context.fetch;
+        requestMayHaveDispatched = true;
+        const response = await fetchFn(url, init);
         if (!response.ok) {
-          const errorText = await response.text();
+          let errorText;
+          try {
+            errorText = await response.text();
+          } catch (textError) {
+            const cause = textError instanceof Error ? textError : new Error(String(textError));
+            return err(
+              serviceError(
+                ErrorCodes.KV_WRITE_FAILED,
+                `Failed to batch put ${items.length} key(s): ${response.status} - <response body could not be read: ${cause.message}>`,
+                "kv",
+                { cause, meta: { status: response.status, statusText: response.statusText } }
+              )
+            );
+          }
           if (response.status === 401 || response.status === 403) {
             const { resource, action } = parseAuthError(errorText);
             return err(authUnauthorizedError("kv", errorText, {
@@ -1242,19 +1260,73 @@ var KVService = class extends BaseService {
             )
           );
         }
-        const batchResponse = this.normalizeBatchPutResponse(await response.json());
-        if (!batchResponse || batchResponse.count !== batchResponse.written.length) {
+        let rawBody;
+        try {
+          rawBody = await response.json();
+        } catch (jsonError) {
+          const cause = jsonError instanceof Error ? jsonError : new Error(String(jsonError));
+          return err(
+            serviceError(
+              ErrorCodes.NETWORK_ERROR,
+              `KV batchPut response was not valid JSON: ${cause.message}`,
+              "kv",
+              {
+                cause,
+                meta: {
+                  requestMayHaveDispatched: true,
+                  responseReceived: true,
+                  status: response.status,
+                  outcome: "batch-unconfirmed"
+                }
+              }
+            )
+          );
+        }
+        const batchResponse = this.normalizeBatchPutResponse(rawBody);
+        if (!batchResponse) {
           return err(
             serviceError(
               ErrorCodes.NETWORK_ERROR,
               "KV batchPut response did not include matching written keys and count",
-              "kv"
+              "kv",
+              {
+                meta: {
+                  requestMayHaveDispatched: true,
+                  responseReceived: true,
+                  status: response.status,
+                  outcome: "batch-unconfirmed"
+                }
+              }
+            )
+          );
+        }
+        const requestedPaths = new Set(paths);
+        const writtenPaths = new Set(batchResponse.written);
+        const matchesRequest = batchResponse.count === paths.length && batchResponse.written.length === paths.length && writtenPaths.size === paths.length && batchResponse.written.every((key) => requestedPaths.has(key));
+        if (!matchesRequest) {
+          return err(
+            serviceError(
+              ErrorCodes.NETWORK_ERROR,
+              `KV batchPut response did not confirm all ${paths.length} requested key(s) were written`,
+              "kv",
+              {
+                meta: {
+                  requestMayHaveDispatched: true,
+                  responseReceived: true,
+                  status: response.status,
+                  outcome: "batch-unconfirmed"
+                }
+              }
             )
           );
         }
         return ok(batchResponse);
       } catch (error) {
-        return err(wrapError("kv", error));
+        const wrapped = wrapError("kv", error);
+        return err({
+          ...wrapped,
+          meta: { ...wrapped.meta, requestMayHaveDispatched }
+        });
       }
     });
   }