Skip to content

Document sender allow and block list commands - #2640

Open
oOvalm wants to merge 1 commit into
larksuite:mainfrom
oOvalm:feat/797b2bd
Open

Document sender allow and block list commands#2640
oOvalm wants to merge 1 commit into
larksuite:mainfrom
oOvalm:feat/797b2bd

Conversation

@oOvalm

@oOvalm oOvalm commented Sep 7, 2026

Copy link
Copy Markdown

Adds user-facing documentation for listing, querying, setting, and deleting sender allow and block list entries. Documents read and modify permission requirements, command arguments, response semantics, and common errors.

Summary by CodeRabbit

  • Documentation
    • Added guidance for managing user-level trusted and blocked sender lists.
    • Documented how to list, search, update, and delete sender entries.
    • Added details on permissions, pagination, conflict handling, retries, confirmation, and error verification.
    • Added quick-reference links for these sender-list operations.

Explain the generated resource names and provide a complete setup, verification, and cleanup sequence.
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@github-actions github-actions Bot added domain/mail PR touches the mail domain size/M Single-domain feat or fix with limited business impact labels Sep 7, 2026
@coderabbitai

coderabbitai Bot commented Sep 7, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Team

Run ID: 814bcebe-f166-40d4-91c7-7ed34ed09573

📥 Commits

Reviewing files that changed from the base of the PR and between c2afcce and 372f973.

📒 Files selected for processing (2)
  • skills/lark-mail/SKILL.md
  • skills/lark-mail/references/lark-mail-user-sender-lists.md

Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review.


📝 Walkthrough

Walkthrough

Changes

新增用户级发件人白名单和黑名单的概念说明、速查入口及完整 CLI 操作指南。文档覆盖资源映射、分页查询、状态变更、删除、权限 scope、失败项校验和错误处理。

用户级发件人名单

Layer / File(s) Summary
发件人名单概念与操作规范
skills/lark-mail/SKILL.md, skills/lark-mail/references/lark-mail-user-sender-lists.md
核心文档新增 user_mailbox.allow_sendersuser_mailbox.blocked_senders 说明及速查入口。参考文档定义 allow/block 操作、分页与精确匹配、批量写入和删除、验证闭环、所需 scope 及结构化错误处理。

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to 372f9

This adds documentation and quick-reference links for user sender allowlist and blocklist operations. No actionable merge-blocking risk is identified.

Suggested reviewers: liangshuo-1

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is relevant but does not follow the required template. It omits the Summary, Changes, Test Plan, and Related Issues sections. Rewrite the description using all required template sections. List the main changes, document the test results or verification status, and state related issues or use “None”.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: documentation for sender allow and block list commands.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
  • Fix all pre-merge checks with AI
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain/mail PR touches the mail 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