@tinycloud/operations 0.3.2-beta.3 → 0.3.2-beta.6

package/dist/index.js

@@ -3676,6 +3676,7 @@ var init_dist2 = __esm({
             }
             seen.add(path);
           }
+          let requestMayHaveDispatched = false;
           try {
             const body = new FormData();
             for (let index = 0; index < items.length; index++) {
@@ -3693,14 +3694,31 @@ var init_dist2 = __esm({
                 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, {
@@ -3726,19 +3744,73 @@ var init_dist2 = __esm({
                 )
               );
             }
-            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 }
+            });
           }
         });
       }