Skip to content

docs(mail): refresh generated mail skill metadata#1619

Open
VincentChanXX wants to merge 1 commit into
larksuite:mainfrom
VincentChanXX:feat/886af18
Open

docs(mail): refresh generated mail skill metadata#1619
VincentChanXX wants to merge 1 commit into
larksuite:mainfrom
VincentChanXX:feat/886af18

Conversation

@VincentChanXX

@VincentChanXX VincentChanXX commented Jun 26, 2026

Copy link
Copy Markdown

Generated by the harness-coding skill.

  • Task ID: 01KW1M0J005G15J2X98DB4D1HG-4
  • Branch: feat/886af18
  • Target: main

Sprints

ID Title Status Commit
S7 Consume generated mail metadata in larksuite/cli passed 030ed1f

Source specs

  • /home/chenweicheng.x/.local/state/harness-agent/envs/env-077ddb944259/generic-workers/generic-n37-120-184-af8014/coding/traex/01KW1M0J005G15J2X98DB4D1HG/01KW1M0J005G15J2X98DB4D1HG-4/input/tech-design.md

Summary by CodeRabbit

  • Documentation
    • Expanded the mail assistant usage guide with clearer workflows for reading, replying, forwarding, drafts, and timed sending.
    • Added guidance for choosing recipients, sending from approved mailboxes or aliases, and confirming delivery status after sending.
    • Improved instructions for HTML content handling, templates, and shortcuts, with updated command discovery and safety reminders.

Expose generated mail allow and blocked sender resources in the lark-mail skill documentation.

Co-Authored-By: AI
@coderabbitai

coderabbitai Bot commented Jun 26, 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: Pro

Run ID: 3e6470ca-afc5-42e1-80d1-ee7a8243486b

📥 Commits

Reviewing files that changed from the base of the PR and between 39d60cb and 030ed1f.

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

📝 Walkthrough

Walkthrough

The mail skill guide expands the core workflow, recipient search, sending rules, HTML handling, templates, shortcuts, and API references, while adding a Rule concept and stricter discovery and confirmation instructions.

Changes

Lark Mail skill guide

Layer / File(s) Summary
Foundation and workflow
skills/lark-mail/SKILL.md
Expands the skill description, adds the Rule concept, tightens the shared-skill read and -h discovery instructions, and rewrites the basic read/reply/forward/draft workflow.
Recipient search
skills/lark-mail/SKILL.md
Adds the recipient-search procedure, candidate listing, confirmation requirement, field descriptions, and no-match behavior.
Sending lifecycle
skills/lark-mail/SKILL.md
Adds pre-send confirmation, send_status verification, timed-send constraints, send_as mailbox/from selection, and updated recall, share, calendar, and default composition rules.
HTML writing and reading
skills/lark-mail/SKILL.md
Adds HTML style rules, raw EML prohibition, lint behavior, the --no-lint contract, and --html read behavior.
Templates and references
skills/lark-mail/SKILL.md
Adds template merge rules and constraints, updates shortcut entries, and extends the API resource and permission tables.

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

Possibly related PRs

  • larksuite/cli#1527: Also revises skills/lark-mail/SKILL.md around pre-flight help discovery and API/resource guidance.
  • larksuite/cli#228: Also updates mail skill guidance around HTML handling and read output formatting.
  • larksuite/cli#749: Also strengthens confirmation and content-safety rules in the same mail skill guide.

Suggested labels

documentation

Suggested reviewers

  • chanthuang
  • infeng
  • haidaodashushu

Poem

Hop, I stitched the mail guide neat and bright,
With rules for drafts and HTML light.
I sniffed the lint and kept the paths aligned,
Then bounced the templates through my tiny mind.
🐰 A carrot seal on every send tonight!

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description does not follow the required template and is missing Summary, Changes, Test Plan, and Related Issues sections. Rewrite the PR description using the repository template and fill in Summary, Changes, Test Plan, and Related Issues sections.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title is concise and matches the PR’s main change: refreshing generated mail skill metadata.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
✨ 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.

@github-actions github-actions Bot added domain/mail PR touches the mail domain size/L Large or sensitive change across domains or core paths labels Jun 26, 2026
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/L Large or sensitive change across domains or core paths

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant