Feat/approval support addsign

[Paulazaaza-dev]

Summary

optimize approval skills

Changes

• modify description
• remove redundancy content
• add reverse boundary paragraph

Related Issues

• None

Summary by CodeRabbit

• Documentation
  • Expanded Lark approval skill documentation with detailed coverage and limitations information
  • Added clarification on approval task handling defaults
  • Restructured out-of-scope section with explicit item listings
  • Streamlined documentation structure

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 03dfed7d-1dfe-409e-93e9-34310ee47398

📥 Commits

Reviewing files that changed from the base of the PR and between f3949f0 and a82fa24.

📒 Files selected for processing (1)

• skills/lark-approval/SKILL.md

---

📝 Walkthrough

Walkthrough

Updated the skills/lark-approval/SKILL.md documentation file to expand the skill description with specific coverage and limitations for Feishu approval operations. Added an identity section documenting default --as user handling, and restructured the out-of-scope section to explicitly list unsupported capabilities (approval definition, form design, and instance initiation).

Changes

Lark Approval Skill Documentation

Layer / File(s)	Summary
Skill scope and identity documentation skills/lark-approval/SKILL.md	Expanded the skill description to clarify which approval queries and actions are supported versus delegated. Added an "身份" section noting approval operations default to --as user. Replaced the previous "权限表" with a structured "不在本 skill 范围" section explicitly listing out-of-scope items: approval definition/form design, initiating new approval instances, and non-approval todo tasks.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• larksuite/cli#867: Both PRs modify skills/lark-approval/SKILL.md to document 飞书审批 task capabilities and scope mappings, with overlapping coverage of approval operations.
• larksuite/cli#217: Both PRs modify skills/lark-approval/SKILL.md to define and clarify the Feishu approval skill's API coverage and scope.
• larksuite/cli#224: Both PRs introduce or describe the lark-approval approval capability through documentation updates.

Suggested labels

size/M, domain/ccm

Suggested reviewers

• liangshuo-1
• MaxHuang22

Poem

🐰 A skill once vague, now crystal clear,
With boundaries drawn for all to hear—
"What we do" and "what we don't"—
Identity noted: --as user, we wont!
Documentation dances, precise and bright,
The approval way, now in the light! ✨

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	The title 'Feat/approval support addsign' does not align with the actual changes, which are documentation updates to clarify scope and remove redundancy from SKILL.md, not feature additions.	Change title to reflect the actual changes, such as 'docs: clarify lark-approval skill scope and limitations' or 'refactor: reorganize approval skill documentation'.
Description check	⚠️ Warning	The description is missing required sections including 'Test Plan' and lacks specific details about what was modified; it only provides vague bullet points without context.	Add 'Test Plan' section, provide specific details about which parts of SKILL.md were modified, and include concrete examples of the changes made.

✅ Passed checks (3 passed)

Check name	Status	Explanation
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.

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

✨ 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

• X
• Mastodon
• Reddit
• LinkedIn

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