npm - @yuuko1410/feishu-bitable - Versions diffs - 0.0.3 → 0.0.4 - Mend

@yuuko1410/feishu-bitable 0.0.3 → 0.0.4

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (4) hide show

package/lib/client.d.ts CHANGED Viewed

@@ -37,4 +37,5 @@ export declare class Bitable {
     private logWarn;
     private logError;
     private getErrorMeta;
+    private logRequest;
 }

package/lib/index.cjs CHANGED Viewed

@@ -285,7 +285,7 @@ class Bitable {
     }, async () => {
       const token = this.resolveAppToken(appToken);
       const pageSize = Math.max(1, Math.min(options.pageSize ?? FEISHU_BATCH_LIMIT, FEISHU_BATCH_LIMIT));
-      const iterator = await this.client.bitable.v1.appTableRecord.searchWithIterator({
+      const request = {
         path: {
           app_token: token,
           table_id: tableId
@@ -301,7 +301,8 @@ class Bitable {
           sort: options.sort,
           automatic_fields: options.automaticFields
         }
-      });
+      };
+      const iterator = await this.client.bitable.v1.appTableRecord.searchWithIterator(this.logRequest("fetchAllRecords request", request));
       const allRecords = [];
       for await (const page of iterator) {
         const items = page?.items ?? [];
@@ -326,12 +327,15 @@ class Bitable {
       }
       const token = this.resolveAppToken(options.appToken);
       const chunks = chunkArray(records, options.chunkSize ?? FEISHU_BATCH_LIMIT);
-      const responses = await runWithConcurrency(chunks, options.concurrency ?? this.defaultConcurrency, async (chunk) => this.withRetry("insert records", async () => assertFeishuResponse(await this.client.bitable.v1.appTableRecord.batchCreate({
-        path: { app_token: token, table_id: tableId },
-        data: {
-          records: chunk.map((fields) => ({ fields }))
-        }
-      }), "insert records")));
+      const responses = await runWithConcurrency(chunks, options.concurrency ?? this.defaultConcurrency, async (chunk) => this.withRetry("insert records", async () => {
+        const request = {
+          path: { app_token: token, table_id: tableId },
+          data: {
+            records: chunk.map((fields) => ({ fields }))
+          }
+        };
+        return assertFeishuResponse(await this.client.bitable.v1.appTableRecord.batchCreate(this.logRequest("insertList request", request)), "insert records");
+      }));
       this.logInfo("insertList completed", {
         tableId,
         chunkCount: chunks.length
@@ -358,8 +362,18 @@ class Bitable {
       const token = this.resolveAppToken(options.appToken);
       const chunks = chunkArray(records, options.chunkSize ?? FEISHU_BATCH_LIMIT);
       const responses = await runWithConcurrency(chunks, options.concurrency ?? this.defaultConcurrency, async (chunk, index) => {
-        const batchRecords = chunk.map((record) => {
+        const batchRecords = chunk.map((record, recordIndex) => {
           const { recordId, fields } = splitUpdateRecord(record);
+          if (!recordId || !recordId.trim()) {
+            throw new FeishuBitableError(`updateRecords failed: record_id is required for chunk ${index}, item ${recordIndex}`, {
+              details: {
+                tableId,
+                chunkIndex: index,
+                recordIndex,
+                record
+              }
+            });
+          }
           if (Object.keys(fields).length === 0) {
             return null;
           }
@@ -418,12 +432,15 @@ class Bitable {
       }
       const token = this.resolveAppToken(options.appToken);
       const chunks = chunkArray(recordIds, options.chunkSize ?? FEISHU_BATCH_LIMIT);
-      const responses = await runWithConcurrency(chunks, options.concurrency ?? this.defaultConcurrency, async (chunk) => this.withRetry("delete records", async () => assertFeishuResponse(await this.client.bitable.v1.appTableRecord.batchDelete({
-        path: { app_token: token, table_id: tableId },
-        data: {
-          records: chunk
-        }
-      }), "delete records")));
+      const responses = await runWithConcurrency(chunks, options.concurrency ?? this.defaultConcurrency, async (chunk) => this.withRetry("delete records", async () => {
+        const request = {
+          path: { app_token: token, table_id: tableId },
+          data: {
+            records: chunk
+          }
+        };
+        return assertFeishuResponse(await this.client.bitable.v1.appTableRecord.batchDelete(this.logRequest("deleteList request", request)), "delete records");
+      }));
       this.logInfo("deleteList completed", {
         tableId,
         chunkCount: chunks.length
@@ -445,25 +462,37 @@ class Bitable {
         mode: buffer.byteLength <= FEISHU_SIMPLE_UPLOAD_LIMIT ? "simple" : "multipart"
       });
       if (buffer.byteLength <= FEISHU_SIMPLE_UPLOAD_LIMIT) {
-        return await this.withRetry("upload file", async () => this.client.drive.v1.media.uploadAll({
+        return await this.withRetry("upload file", async () => {
+          const request = {
+            data: {
+              file_name: fileName,
+              parent_type: options.parentType,
+              parent_node: options.parentNode,
+              size: buffer.byteLength,
+              extra: options.extra,
+              file: buffer
+            }
+          };
+          this.logRequest("uploadFile request", {
+            data: {
+              ...request.data,
+              file: `[Buffer ${buffer.byteLength} bytes]`
+            }
+          });
+          return this.client.drive.v1.media.uploadAll(request);
+        }) ?? {};
+      }
+      const prepare = assertFeishuResponse(await this.withRetry("prepare multipart upload", async () => {
+        const request = {
           data: {
             file_name: fileName,
             parent_type: options.parentType,
             parent_node: options.parentNode,
-            size: buffer.byteLength,
-            extra: options.extra,
-            file: buffer
+            size: buffer.byteLength
           }
-        })) ?? {};
-      }
-      const prepare = assertFeishuResponse(await this.withRetry("prepare multipart upload", async () => this.client.drive.v1.media.uploadPrepare({
-        data: {
-          file_name: fileName,
-          parent_type: options.parentType,
-          parent_node: options.parentNode,
-          size: buffer.byteLength
-        }
-      })), "prepare multipart upload");
+        };
+        return this.client.drive.v1.media.uploadPrepare(this.logRequest("uploadPrepare request", request));
+      }), "prepare multipart upload");
       const uploadId = prepare.data?.upload_id;
       const blockSize = prepare.data?.block_size;
       const blockNum = prepare.data?.block_num;
@@ -476,21 +505,33 @@ class Bitable {
         const start = index * blockSize;
         const end = Math.min(start + blockSize, buffer.byteLength);
         const chunk = buffer.subarray(start, end);
-        await this.withRetry(`upload file chunk ${index + 1}/${blockNum}`, async () => this.client.drive.v1.media.uploadPart({
+        await this.withRetry(`upload file chunk ${index + 1}/${blockNum}`, async () => {
+          const request = {
+            data: {
+              upload_id: uploadId,
+              seq: index,
+              size: chunk.byteLength,
+              file: chunk
+            }
+          };
+          this.logRequest("uploadPart request", {
+            data: {
+              ...request.data,
+              file: `[Buffer ${chunk.byteLength} bytes]`
+            }
+          });
+          return this.client.drive.v1.media.uploadPart(request);
+        });
+      }
+      const finish = assertFeishuResponse(await this.withRetry("finish multipart upload", async () => {
+        const request = {
           data: {
             upload_id: uploadId,
-            seq: index,
-            size: chunk.byteLength,
-            file: chunk
+            block_num: blockNum
           }
-        }));
-      }
-      const finish = assertFeishuResponse(await this.withRetry("finish multipart upload", async () => this.client.drive.v1.media.uploadFinish({
-        data: {
-          upload_id: uploadId,
-          block_num: blockNum
-        }
-      })), "finish multipart upload");
+        };
+        return this.client.drive.v1.media.uploadFinish(this.logRequest("uploadFinish request", request));
+      }), "finish multipart upload");
       return {
         file_token: finish.data?.file_token
       };
@@ -501,14 +542,17 @@ class Bitable {
       fileToken,
       hasExtra: Boolean(extra)
     }, async () => {
-      const response = await this.withRetry("download file", async () => this.client.drive.v1.media.download({
-        path: {
-          file_token: fileToken
-        },
-        params: {
-          extra
-        }
-      }));
+      const response = await this.withRetry("download file", async () => {
+        const request = {
+          path: {
+            file_token: fileToken
+          },
+          params: {
+            extra
+          }
+        };
+        return this.client.drive.v1.media.download(this.logRequest("downloadFile request", request));
+      });
       const buffer = await readableToBuffer(response.getReadableStream());
       this.logInfo("downloadFile completed", {
         fileToken,
@@ -550,7 +594,7 @@ class Bitable {
     return token;
   }
   async executeBatchUpdateRecords(payload) {
-    return this.withRetry("batch update records", async () => assertFeishuResponse(await this.client.bitable.v1.appTableRecord.batchUpdate(payload), "batch update records"));
+    return this.withRetry("batch update records", async () => assertFeishuResponse(await this.client.bitable.v1.appTableRecord.batchUpdate(this.logRequest("batchUpdateRecords request", payload)), "batch update records"));
   }
   async withRetry(label, task) {
     let lastError;
@@ -623,6 +667,12 @@ class Bitable {
       errorMessage: String(error)
     };
   }
+  logRequest(message, payload) {
+    this.logInfo(message, {
+      payload
+    });
+    return payload;
+  }
 }
 // src/index.ts

package/lib/index.js CHANGED Viewed

@@ -211,7 +211,7 @@ class Bitable {
     }, async () => {
       const token = this.resolveAppToken(appToken);
       const pageSize = Math.max(1, Math.min(options.pageSize ?? FEISHU_BATCH_LIMIT, FEISHU_BATCH_LIMIT));
-      const iterator = await this.client.bitable.v1.appTableRecord.searchWithIterator({
+      const request = {
         path: {
           app_token: token,
           table_id: tableId
@@ -227,7 +227,8 @@ class Bitable {
           sort: options.sort,
           automatic_fields: options.automaticFields
         }
-      });
+      };
+      const iterator = await this.client.bitable.v1.appTableRecord.searchWithIterator(this.logRequest("fetchAllRecords request", request));
       const allRecords = [];
       for await (const page of iterator) {
         const items = page?.items ?? [];
@@ -252,12 +253,15 @@ class Bitable {
       }
       const token = this.resolveAppToken(options.appToken);
       const chunks = chunkArray(records, options.chunkSize ?? FEISHU_BATCH_LIMIT);
-      const responses = await runWithConcurrency(chunks, options.concurrency ?? this.defaultConcurrency, async (chunk) => this.withRetry("insert records", async () => assertFeishuResponse(await this.client.bitable.v1.appTableRecord.batchCreate({
-        path: { app_token: token, table_id: tableId },
-        data: {
-          records: chunk.map((fields) => ({ fields }))
-        }
-      }), "insert records")));
+      const responses = await runWithConcurrency(chunks, options.concurrency ?? this.defaultConcurrency, async (chunk) => this.withRetry("insert records", async () => {
+        const request = {
+          path: { app_token: token, table_id: tableId },
+          data: {
+            records: chunk.map((fields) => ({ fields }))
+          }
+        };
+        return assertFeishuResponse(await this.client.bitable.v1.appTableRecord.batchCreate(this.logRequest("insertList request", request)), "insert records");
+      }));
       this.logInfo("insertList completed", {
         tableId,
         chunkCount: chunks.length
@@ -284,8 +288,18 @@ class Bitable {
       const token = this.resolveAppToken(options.appToken);
       const chunks = chunkArray(records, options.chunkSize ?? FEISHU_BATCH_LIMIT);
       const responses = await runWithConcurrency(chunks, options.concurrency ?? this.defaultConcurrency, async (chunk, index) => {
-        const batchRecords = chunk.map((record) => {
+        const batchRecords = chunk.map((record, recordIndex) => {
           const { recordId, fields } = splitUpdateRecord(record);
+          if (!recordId || !recordId.trim()) {
+            throw new FeishuBitableError(`updateRecords failed: record_id is required for chunk ${index}, item ${recordIndex}`, {
+              details: {
+                tableId,
+                chunkIndex: index,
+                recordIndex,
+                record
+              }
+            });
+          }
           if (Object.keys(fields).length === 0) {
             return null;
           }
@@ -344,12 +358,15 @@ class Bitable {
       }
       const token = this.resolveAppToken(options.appToken);
       const chunks = chunkArray(recordIds, options.chunkSize ?? FEISHU_BATCH_LIMIT);
-      const responses = await runWithConcurrency(chunks, options.concurrency ?? this.defaultConcurrency, async (chunk) => this.withRetry("delete records", async () => assertFeishuResponse(await this.client.bitable.v1.appTableRecord.batchDelete({
-        path: { app_token: token, table_id: tableId },
-        data: {
-          records: chunk
-        }
-      }), "delete records")));
+      const responses = await runWithConcurrency(chunks, options.concurrency ?? this.defaultConcurrency, async (chunk) => this.withRetry("delete records", async () => {
+        const request = {
+          path: { app_token: token, table_id: tableId },
+          data: {
+            records: chunk
+          }
+        };
+        return assertFeishuResponse(await this.client.bitable.v1.appTableRecord.batchDelete(this.logRequest("deleteList request", request)), "delete records");
+      }));
       this.logInfo("deleteList completed", {
         tableId,
         chunkCount: chunks.length
@@ -371,25 +388,37 @@ class Bitable {
         mode: buffer.byteLength <= FEISHU_SIMPLE_UPLOAD_LIMIT ? "simple" : "multipart"
       });
       if (buffer.byteLength <= FEISHU_SIMPLE_UPLOAD_LIMIT) {
-        return await this.withRetry("upload file", async () => this.client.drive.v1.media.uploadAll({
+        return await this.withRetry("upload file", async () => {
+          const request = {
+            data: {
+              file_name: fileName,
+              parent_type: options.parentType,
+              parent_node: options.parentNode,
+              size: buffer.byteLength,
+              extra: options.extra,
+              file: buffer
+            }
+          };
+          this.logRequest("uploadFile request", {
+            data: {
+              ...request.data,
+              file: `[Buffer ${buffer.byteLength} bytes]`
+            }
+          });
+          return this.client.drive.v1.media.uploadAll(request);
+        }) ?? {};
+      }
+      const prepare = assertFeishuResponse(await this.withRetry("prepare multipart upload", async () => {
+        const request = {
           data: {
             file_name: fileName,
             parent_type: options.parentType,
             parent_node: options.parentNode,
-            size: buffer.byteLength,
-            extra: options.extra,
-            file: buffer
+            size: buffer.byteLength
           }
-        })) ?? {};
-      }
-      const prepare = assertFeishuResponse(await this.withRetry("prepare multipart upload", async () => this.client.drive.v1.media.uploadPrepare({
-        data: {
-          file_name: fileName,
-          parent_type: options.parentType,
-          parent_node: options.parentNode,
-          size: buffer.byteLength
-        }
-      })), "prepare multipart upload");
+        };
+        return this.client.drive.v1.media.uploadPrepare(this.logRequest("uploadPrepare request", request));
+      }), "prepare multipart upload");
       const uploadId = prepare.data?.upload_id;
       const blockSize = prepare.data?.block_size;
       const blockNum = prepare.data?.block_num;
@@ -402,21 +431,33 @@ class Bitable {
         const start = index * blockSize;
         const end = Math.min(start + blockSize, buffer.byteLength);
         const chunk = buffer.subarray(start, end);
-        await this.withRetry(`upload file chunk ${index + 1}/${blockNum}`, async () => this.client.drive.v1.media.uploadPart({
+        await this.withRetry(`upload file chunk ${index + 1}/${blockNum}`, async () => {
+          const request = {
+            data: {
+              upload_id: uploadId,
+              seq: index,
+              size: chunk.byteLength,
+              file: chunk
+            }
+          };
+          this.logRequest("uploadPart request", {
+            data: {
+              ...request.data,
+              file: `[Buffer ${chunk.byteLength} bytes]`
+            }
+          });
+          return this.client.drive.v1.media.uploadPart(request);
+        });
+      }
+      const finish = assertFeishuResponse(await this.withRetry("finish multipart upload", async () => {
+        const request = {
           data: {
             upload_id: uploadId,
-            seq: index,
-            size: chunk.byteLength,
-            file: chunk
+            block_num: blockNum
           }
-        }));
-      }
-      const finish = assertFeishuResponse(await this.withRetry("finish multipart upload", async () => this.client.drive.v1.media.uploadFinish({
-        data: {
-          upload_id: uploadId,
-          block_num: blockNum
-        }
-      })), "finish multipart upload");
+        };
+        return this.client.drive.v1.media.uploadFinish(this.logRequest("uploadFinish request", request));
+      }), "finish multipart upload");
       return {
         file_token: finish.data?.file_token
       };
@@ -427,14 +468,17 @@ class Bitable {
       fileToken,
       hasExtra: Boolean(extra)
     }, async () => {
-      const response = await this.withRetry("download file", async () => this.client.drive.v1.media.download({
-        path: {
-          file_token: fileToken
-        },
-        params: {
-          extra
-        }
-      }));
+      const response = await this.withRetry("download file", async () => {
+        const request = {
+          path: {
+            file_token: fileToken
+          },
+          params: {
+            extra
+          }
+        };
+        return this.client.drive.v1.media.download(this.logRequest("downloadFile request", request));
+      });
       const buffer = await readableToBuffer(response.getReadableStream());
       this.logInfo("downloadFile completed", {
         fileToken,
@@ -476,7 +520,7 @@ class Bitable {
     return token;
   }
   async executeBatchUpdateRecords(payload) {
-    return this.withRetry("batch update records", async () => assertFeishuResponse(await this.client.bitable.v1.appTableRecord.batchUpdate(payload), "batch update records"));
+    return this.withRetry("batch update records", async () => assertFeishuResponse(await this.client.bitable.v1.appTableRecord.batchUpdate(this.logRequest("batchUpdateRecords request", payload)), "batch update records"));
   }
   async withRetry(label, task) {
     let lastError;
@@ -549,6 +593,12 @@ class Bitable {
       errorMessage: String(error)
     };
   }
+  logRequest(message, payload) {
+    this.logInfo(message, {
+      payload
+    });
+    return payload;
+  }
 }
 // src/index.ts

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "@yuuko1410/feishu-bitable",
-  "version": "0.0.3",
+  "version": "0.0.4",
   "description": "基于 Bun + TypeScript + 飞书官方 SDK 的多维表格操作库",
   "type": "module",
   "main": "./lib/index.cjs",