feat: add contact skill domain guidance#1144
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughWalkthroughThis PR updates lark-contact documentation: expands the skill description with prerequisites and examples, and adds a domain guide mapping contact intents to the correct ChangesContact Skill Documentation Update
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@skill-template/domains/contact.md`:
- Around line 7-12: Links in skill-template/domains/contact.md are using
incorrect relative paths (e.g., `references/lark-contact-search-user.md`,
`../lark-im/SKILL.md`) and thus break; update each Markdown link to use paths
relative to the skill-template/domains/ directory so they resolve correctly (for
example, change `references/...` to `../references/...` or the correct sibling
path), and verify all other instances like
`references/lark-contact-get-user.md`, `../lark-calendar/SKILL.md`, and the
`+search-user`/`+get-user` reference lines are adjusted the same way.
🪄 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: d7d1b5b8-2652-4179-97c8-a27ec2f76a74
📒 Files selected for processing (2)
skill-template/domains/contact.mdskills/lark-contact/SKILL.md
🚀 PR Preview Install Guide🧰 CLI updatenpm i -g https://pkg.pr.new/larksuite/cli/@larksuite/cli@0eb19570336d5200df3196ece7dab7bc681d9ded🧩 Skill updatenpx skills add suhui928/cli#feat/add_user_profile_meta_api -y -g |
- Add user_profiles batch_query row to the routing table. - Add a worked example next to the search-user one, with `lark-cli schema` first (best practice: don't guess `--data` / `--params`). - Trim description: drop the duplicated trigger clause, add personal_status / signature to the capability list so routing picks this skill up for those queries. Change-Id: Ib32c56f6a389b3c248409228f4c7461453d1dcd3
0eb1957 to
2d7dcc2
Compare
Summary
Add contact domain guidance for AI agents and update the contact skill documentation so agents can
better understand when and how to use contact-related CLI commands.
Changes
skill-template/domains/contact.mdskills/lark-contact/SKILL.mdwith the generated contact skill contentTest Plan
make unit-testgo vet ./...gofmt -l .go mod tidygo run github.com/golangci/golangci-lint/v2/cmd/golangci-lint@v2.1.6 run --new-from-rev=origin/mainRelated Issues
N/A
Summary by CodeRabbit