fix:add approval capability to README#224
Merged
liangshuo-1 merged 1 commit intolarksuite:mainfrom Apr 2, 2026
Merged
Conversation
github-actions
Bot
added
the
size/M
📝 WalkthroughWalkthroughUpdated English and Chinese documentation to add a new "Approval" AI Agent capability, adjust counts and table formatting; renamed the Changes
Sequence Diagram(s)(omitted) Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Greptile SummaryThis PR is a documentation-only update that adds the new Confidence Score: 5/5Safe to merge — purely documentation changes with no functional impact. All changes are documentation-only (README files and a JSON display-name field). The only finding is a P2 capitalization nit in the lark-approval skill description, which does not block merge. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A["README.md / README.zh.md"] -->|"Add ✍️ Approval row"| B["Features Table\n(12 domains)"]
A -->|"Add lark-approval row"| C["Agent Skills Table\n(20 skills)"]
D["service_descriptions.json"] -->|"'Approval API' → 'Approval'"| E["Display Title\n(en/zh)"]
Reviews (4): Last reviewed commit: "fix:add approval capability to README" | Re-trigger Greptile |
liangshuo-1
previously approved these changes
Apr 2, 2026
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@README.md`:
- Around line 15-16: The README has inconsistent skill counts: the header lines
"Agent-Native Design" and "Wide Coverage" claim 20 Skills while the Skills table
and Line 9 show 19; fix by making the numbers consistent — either add the
missing 20th skill row to the Skills table (ensure its title and link format
matches existing entries) or change both header lines to say 19, and update any
other occurrences (search for "20 structured [Skills]" and "20 AI Agent
[Skills]") so all references (headers and the Skills table) use the same count.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 365d4468-355d-4258-81cd-3dfc71ea4615
📒 Files selected for processing (3)
README.mdREADME.zh.mdinternal/registry/service_descriptions.json
🚀 PR Preview Install Guide🧰 CLI updatenpm i -g https://pkg.pr.new/larksuite/cli/@larksuite/cli@4c70423dd7a917ab8b42a22081b7966c699566e0🧩 Skill updatenpx skills add zhaoshengmeng626/lark-cli#fix/approval_description_modify -y -g |
zhaoshengmeng626
force-pushed
the
fix/approval_description_modify
branch
from
4c70423 to
a83d07c
Compare
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@README.zh.md`:
- Line 9: The README line claiming "20 个 AI Agent [Skills](./skills/)" is
inconsistent with the Skills table which lists 19 entries; either change that
phrase to "19 个 AI Agent [Skills](./skills/)" or add the missing skill entry to
the Skills table so counts match. Locate the exact string "20 个 AI Agent
[Skills](./skills/)" in README.zh.md and update it, or add the missing skill row
to the Skills table (the skills list/entries) so the total becomes 20 and the
document is consistent.
- Line 16: Update the phrase "12 大业务域、200+ 精选命令、 20 个 AI Agent [Skills]" to
match the actual Skills table count and spacing: change "20" to "19" so it reads
"12 大业务域、200+ 精选命令、19 个 AI Agent [Skills]" and remove the extra space after the
punctuation (、) to ensure consistent spacing across the document; locate and
edit the exact string in README.zh.md.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 9781d296-cf7a-4115-9146-3266d308298d
📒 Files selected for processing (3)
README.mdREADME.zh.mdinternal/registry/service_descriptions.json
✅ Files skipped from review due to trivial changes (1)
- internal/registry/service_descriptions.json
🚧 Files skipped from review as they are similar to previous changes (1)
- README.md
zhaoshengmeng626
force-pushed
the
fix/approval_description_modify
branch
from
a83d07c to
99878a6
Compare
|
|
There was a problem hiding this comment.
♻️ Duplicate comments (1)
README.zh.md (1)
16-16:⚠️ Potential issue | 🟡 MinorRemove the extra space after punctuation in the coverage bullet.
Line 16 has
、 20(extra space after、), which is inconsistent with adjacent Chinese punctuation style.✏️ Proposed fix
-- **覆盖面广** — 12 大业务域、200+ 精选命令、 20 个 AI Agent [Skills](./skills/) +- **覆盖面广** — 12 大业务域、200+ 精选命令、20 个 AI Agent [Skills](./skills/)🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@README.zh.md` at line 16, The bullet in README.zh.md contains an extra space after the punctuation in the text "12 大业务域、200+ 精选命令、 20 个 AI Agent [Skills](./skills/)"; edit that line to remove the space after the Chinese comma so it reads "12 大业务域、200+ 精选命令、20 个 AI Agent [Skills](./skills/)" to match Chinese punctuation spacing style.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Duplicate comments:
In `@README.zh.md`:
- Line 16: The bullet in README.zh.md contains an extra space after the
punctuation in the text "12 大业务域、200+ 精选命令、 20 个 AI Agent [Skills](./skills/)";
edit that line to remove the space after the Chinese comma so it reads "12
大业务域、200+ 精选命令、20 个 AI Agent [Skills](./skills/)" to match Chinese punctuation
spacing style.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 57b11684-8922-4770-9191-e8763c8b88ab
📒 Files selected for processing (3)
README.mdREADME.zh.mdinternal/registry/service_descriptions.json
✅ Files skipped from review due to trivial changes (1)
- internal/registry/service_descriptions.json
zhaoshengmeng626
changed the title
zhaoshengmeng626
force-pushed
the
fix/approval_description_modify
branch
from
99878a6 to
0331966
Compare
zhaoshengmeng626
changed the title
liangshuo-1
approved these changes
Apr 2, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Changes
Summary by CodeRabbit