Skip to content

docs(mail): remove get_signatures from skill reference#545

Open
chanthuang wants to merge 1 commit intomainfrom
docs/mail-remove-get-signatures
Open

docs(mail): remove get_signatures from skill reference#545
chanthuang wants to merge 1 commit intomainfrom
docs/mail-remove-get-signatures

Conversation

@chanthuang
Copy link
Copy Markdown
Collaborator

@chanthuang chanthuang commented Apr 17, 2026

Summary

  • get_signatures is not directly exposed as a CLI command — it's wrapped by +signature
  • Remove the raw get_signatures entry from the shortcut reference list and the permissions table in skills/lark-mail/SKILL.md

Summary by CodeRabbit

  • Documentation
    • Removed the get_signatures API method documentation from lark-cli mail.

…signature instead

Change-Id: I3463cbd08d595c1cb9cda4fadc6e2a5ad1c62189
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 17, 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: 85e650aa-0abe-4bcc-968f-766dc8bce2e2

📥 Commits

Reviewing files that changed from the base of the PR and between 4f90fd3 and 56840a3.

📒 Files selected for processing (1)
  • skills/lark-mail/SKILL.md
💤 Files with no reviewable changes (1)
  • skills/lark-mail/SKILL.md

📝 Walkthrough

Walkthrough

The documentation for the Lark Mail CLI was updated to remove the get_signatures method from the user_mailbox.settings API resource and its associated permission scope entry.

Changes

Cohort / File(s) Summary
Lark Mail API Documentation
skills/lark-mail/SKILL.md
Removed user_mailbox.settings.get_signatures API method documentation entry and its corresponding permission scope (mail:user_mailbox:readonly) from the resource table.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A signature erased with care,
From docs so clean, with space to spare,
No method lost, just trimmed with grace,
Simplicity wins the coding race!

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The description is missing required sections (Changes, Test Plan, Related Issues) from the template, though it does provide a summary explaining the rationale for the removal. Add the missing sections from the template: a bulleted Changes list, Test Plan with checkboxes, and Related Issues section to provide complete documentation.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely summarizes the main change: removing get_signatures from the mail skill documentation reference.
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 docs/mail-remove-get-signatures

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/mail PR touches the mail domain size/M Single-domain feat or fix with limited business impact labels Apr 17, 2026
@github-actions
Copy link
Copy Markdown

🚀 PR Preview Install Guide

🧰 CLI update

npm i -g https://pkg.pr.new/larksuite/cli/@larksuite/cli@56840a3f7d319eb3ec16e08c85c5999e9459091e

🧩 Skill update

npx skills add larksuite/cli#docs/mail-remove-get-signatures -y -g

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.

1 participant