Skip to content

fix: 统一 record 批量写入限制为 200 条,移除'建议'改为强制要求#499

Merged
zhouyue-bytedance merged 1 commit intomainfrom
fix/lark-base-batch-limit-consistency
Apr 16, 2026
Merged

fix: 统一 record 批量写入限制为 200 条,移除'建议'改为强制要求#499
zhouyue-bytedance merged 1 commit intomainfrom
fix/lark-base-batch-limit-consistency

Conversation

@zhouyue-bytedance
Copy link
Copy Markdown
Collaborator

@zhouyue-bytedance zhouyue-bytedance commented Apr 16, 2026

  • 第106行:批量单次改为 200 条(与 lark-base-record-batch-create.md 对齐)
  • 第290行:批量写入改为 200 条(与 API 限制对齐)
  • 第291行:连续写入改为'必须串行'(强制要求,非建议)
  • 第314行:错误代码 1254104 改为 200 条限制

修复 SKILL.md 与 reference 文档的数字不一致问题

Summary by CodeRabbit

  • Documentation
    • Reduced recommended max batch write size from 500 to 200 items.
    • Clarified batch-update behavior: a single patch is applied uniformly across multiple records (same-value updates).
    • Stipulated that continuous writes to the same table must be performed serially.
    • Updated guidance on handling batch constraint violations and recommended splitting into smaller calls.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 16, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 2744bd34-8fcd-48f0-938a-a07a24c67b0e

📥 Commits

Reviewing files that changed from the base of the PR and between 333275e and 50c57f2.

📒 Files selected for processing (1)
  • skills/lark-base/SKILL.md

📝 Walkthrough

Walkthrough

Documentation updated in skills/lark-base/SKILL.md to clarify +record-batch-update as a single-patch-applied-across-multiple-records operation, lower recommended max batch size from 500 to 200, require serial continuous writes to the same table, and change error 1254104 message accordingly.

Changes

Cohort / File(s) Summary
Batch Operation Docs
skills/lark-base/SKILL.md
Clarify +record-batch-update semantics (single patch for multiple records); reduce max batch size recommendation from 500 → 200; require serialization for continuous same-table writes; update error 1254104 message from "batch超500条" to "batch超200条".

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested reviewers

  • zgz2048

Poem

🐰 I nibbled notes and found the change,
Two hundred hops across the range,
Writes in line, one-after-one,
Errors fixed and doc updated — done! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is largely incomplete, missing the required Template sections (Summary, Changes formatted as bullet list, Test Plan, and Related Issues). Only raw implementation details are provided. Rewrite the description following the repository template: add a Summary section explaining motivation, a formatted Changes section with bullet points, a Test Plan section with checkboxes, and a Related Issues section.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The PR title clearly and specifically describes the main changes: unifying record batch write limits to 200 and removing optional guidance in favor of mandatory requirements.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/lark-base-batch-limit-consistency

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions Bot added domain/base PR touches the base domain size/M Single-domain feat or fix with limited business impact labels Apr 16, 2026
- 第106行:批量单次改为 200 条(与 lark-base-record-batch-create.md 对齐)
- 第290行:批量写入改为 200 条(与 API 限制对齐)
- 第291行:连续写入改为'必须串行'(强制要求,非建议)
- 第314行:错误代码 1254104 改为 200 条限制

修复 SKILL.md 与 reference 文档的数字不一致问题
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 16, 2026

🚀 PR Preview Install Guide

🧰 CLI update

npm i -g https://pkg.pr.new/larksuite/cli/@larksuite/cli@50c57f2c8d683c03198d69d06b02ce50b8d62134

🧩 Skill update

npx skills add larksuite/cli#fix/lark-base-batch-limit-consistency -y -g

@zhouyue-bytedance zhouyue-bytedance force-pushed the fix/lark-base-batch-limit-consistency branch from 333275e to 50c57f2 Compare April 16, 2026 06:40
@zhouyue-bytedance zhouyue-bytedance merged commit e10bf8e into main Apr 16, 2026
11 checks passed
@zhouyue-bytedance zhouyue-bytedance deleted the fix/lark-base-batch-limit-consistency branch April 16, 2026 07:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain/base PR touches the base domain size/M Single-domain feat or fix with limited business impact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants