Repository navigation
chore: migrate agent skills to shared store - #146
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughThe pull request expands review path exclusions, updates Playwright workflow guidance, and adds Convex AI configuration and operating guidance. ChangesAgent guidance and configuration
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 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 `@AGENTS.md`:
- Line 61: Update the development-server instructions in AGENTS.md to replace
`pnpm convex dev` with `npx convex dev`, and explicitly state that `pnpm dev`
and `npx convex dev` must run in separate terminals.
- Around line 161-163: Add the global Convex agent-skill bootstrap commands,
`npx skills add get-convex/agent-skills --all --global` and `npx skills ls -g`,
to the Shared Agent Skills section in AGENTS.md. Clarify that the empty
`aiFiles.skills.agents` configuration does not install these user-scoped skills
through `npx convex ai-files install`.
🪄 Autofix
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: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: eafd66c8-c914-447d-bdb5-330ad71d306e
⛔ Files ignored due to path filters (45)
.agents/skills/convex-create-component/SKILL.mdis excluded by!.agents/**.agents/skills/convex-create-component/agents/openai.yamlis excluded by!.agents/**.agents/skills/convex-create-component/assets/icon.svgis excluded by!**/*.svg,!.agents/**.agents/skills/convex-create-component/references/advanced-patterns.mdis excluded by!.agents/**.agents/skills/convex-create-component/references/hybrid-components.mdis excluded by!.agents/**.agents/skills/convex-create-component/references/local-components.mdis excluded by!.agents/**.agents/skills/convex-create-component/references/packaged-components.mdis excluded by!.agents/**.agents/skills/convex-migration-helper/SKILL.mdis excluded by!.agents/**.agents/skills/convex-migration-helper/agents/openai.yamlis excluded by!.agents/**.agents/skills/convex-migration-helper/assets/icon.svgis excluded by!**/*.svg,!.agents/**.agents/skills/convex-migration-helper/references/migration-patterns.mdis excluded by!.agents/**.agents/skills/convex-migration-helper/references/migrations-component.mdis excluded by!.agents/**.agents/skills/convex-performance-audit/SKILL.mdis excluded by!.agents/**.agents/skills/convex-performance-audit/agents/openai.yamlis excluded by!.agents/**.agents/skills/convex-performance-audit/assets/icon.svgis excluded by!**/*.svg,!.agents/**.agents/skills/convex-performance-audit/references/function-budget.mdis excluded by!.agents/**.agents/skills/convex-performance-audit/references/hot-path-rules.mdis excluded by!.agents/**.agents/skills/convex-performance-audit/references/occ-conflicts.mdis excluded by!.agents/**.agents/skills/convex-performance-audit/references/subscription-cost.mdis excluded by!.agents/**.agents/skills/convex-quickstart/SKILL.mdis excluded by!.agents/**.agents/skills/convex-quickstart/agents/openai.yamlis excluded by!.agents/**.agents/skills/convex-quickstart/assets/icon.svgis excluded by!**/*.svg,!.agents/**.agents/skills/convex-setup-auth/SKILL.mdis excluded by!.agents/**.agents/skills/convex-setup-auth/agents/openai.yamlis excluded by!.agents/**.agents/skills/convex-setup-auth/assets/icon.svgis excluded by!**/*.svg,!.agents/**.agents/skills/convex-setup-auth/references/auth0.mdis excluded by!.agents/**.agents/skills/convex-setup-auth/references/clerk.mdis excluded by!.agents/**.agents/skills/convex-setup-auth/references/convex-auth.mdis excluded by!.agents/**.agents/skills/convex-setup-auth/references/workos-authkit.mdis excluded by!.agents/**.agents/skills/convex/SKILL.mdis excluded by!.agents/**.claude/skills/convexis excluded by!.claude/skills/**.claude/skills/convex-create-componentis excluded by!.claude/skills/**.claude/skills/convex-migration-helperis excluded by!.claude/skills/**.claude/skills/convex-performance-auditis excluded by!.claude/skills/**.claude/skills/convex-quickstartis excluded by!.claude/skills/**.claude/skills/convex-setup-authis excluded by!.claude/skills/**convex/_generated/ai/ai-files.state.jsonis excluded by!**/_generated/**,!convex/_generated/ai/**convex/_generated/ai/guidelines.mdis excluded by!**/_generated/**,!convex/_generated/ai/**skills-lock.jsonis excluded by!skills-lock.jsonskills/convexis excluded by!skills/**skills/convex-create-componentis excluded by!skills/**skills/convex-migration-helperis excluded by!skills/**skills/convex-performance-auditis excluded by!skills/**skills/convex-quickstartis excluded by!skills/**skills/convex-setup-authis excluded by!skills/**
📒 Files selected for processing (3)
.coderabbit.yamlAGENTS.mdconvex.json
Summary
aiFiles.skills.agentsto an empty list so local skills are not recreatedVerification
pnpm typecheckpnpm exec prettier --check .coderabbit.yaml AGENTS.md convex.json convex/_generated/ai/ai-files.state.jsongit diff --check origin/main...HEADnpx convex ai-files status(guidelines and managed AGENTS/CLAUDE sections current; project-local skill warning is expected)Notes
The selected Convex skills are installed at user scope via the shared
npx skillsstore. This PR intentionally contains only the repository-side migration and guardrails.Summary by CodeRabbit