Skip to content

fix: 多维表格 batch_create/batch_update records 上限改为 1000#208

Merged
foxzool merged 1 commit into
mainfrom
fix/bitable-records-limit
Jun 16, 2026
Merged

fix: 多维表格 batch_create/batch_update records 上限改为 1000#208
foxzool merged 1 commit into
mainfrom
fix/bitable-records-limit

Conversation

@foxzool

@foxzool foxzool commented Jun 16, 2026

Copy link
Copy Markdown
Owner

问题

PR #207 (issue #203) 将 batch_create/batch_updaterecords 上限设为 100,存在两个问题:

  1. 上限过严:飞书官方上限为 1000 条(错误码 1254104 RecordExceedLimit),设为 100 会拒绝 101-1000 条的合法请求
  2. 死代码:原 if records.len() > 500 业务规则检查被新的 100 限制覆盖,永不触发

修复

  • MAX_LEN 从 100 改为 1000(匹配飞书官方上限)
  • 删除冗余的 500 业务规则检查块(已被宏覆盖)
  • 错误消息更新为「单次最多 1000 条」

来源

PR #206/#207 code review 发现(Minor #1)。

验证

  • cargo build -p openlark-docs 通过

PR #207 误将 records 上限设为 100,与飞书官方上限 1000 冲突,
且原 500 业务规则成为死代码。改为 1000 并删除冗余检查。

code review (PR #206/#207) 发现的问题。
@foxzool foxzool added bug Something isn't working priority:p1 高优先级 scope:api typed API 覆盖率、API 缺口与模型实现 labels Jun 16, 2026
@foxzool foxzool merged commit 25e3997 into main Jun 16, 2026
119 checks passed
@foxzool foxzool deleted the fix/bitable-records-limit branch June 16, 2026 04:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working priority:p1 高优先级 scope:api typed API 覆盖率、API 缺口与模型实现

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant