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@347aed4042f2c978e8ba608a5b8c060bae35457e🧩 Skill updatenpx skills add suhui928/cli#feat/add_user_profile_meta_api -y -g |
0eb1957 to
2d7dcc2
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1144 +/- ##
=======================================
Coverage 69.64% 69.64%
=======================================
Files 666 666
Lines 64413 64413
=======================================
Hits 44861 44861
Misses 15943 15943
Partials 3609 3609 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
- 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
2d7dcc2 to
347aed4
Compare
* feat(lark-contact): route user_profiles batch_query in skill - 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.
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