chore(web): replace snippet action with CLI guidance#183
chore(web): replace snippet action with CLI guidance#183alan747271363-art wants to merge 2 commits into
Conversation
|
@alan747271363-art is attempting to deploy a commit to the cstrnt's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughCodeSnippetModalButton removes its internal code snippet modal UI and replaces it with a dropdown menu item that opens CLI documentation. Modal-related state and imports are removed, DOCS_URL is added, and a new "Setup via CLI" menu item with event tracking opens the CLI reference guide in a new tab. ChangesReplace Code Snippet Modal with CLI Documentation Link
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 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 `@apps/web/src/components/CodeSnippetModalButton.tsx`:
- Line 47: Add the missing event name to the PlausibleEvents type so the call
trackEvent("Dashboard CLI Instructions Clicked") is statically allowed: edit the
PlausibleEvents declaration (the type used by useTracking) and add a property
"Dashboard CLI Instructions Clicked": never (or an appropriate props type if
needed). Ensure PlausibleEvents is exported/updated so useTracking and the call
in CodeSnippetModalButton.tsx pick up the new event name.
🪄 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: Repository UI
Review profile: CHILL
Plan: Pro
Run ID: 664c3bd3-28e6-45aa-a881-c91b126ff623
📒 Files selected for processing (1)
apps/web/src/components/CodeSnippetModalButton.tsx
Summary
Closes #78.
Validation
pnpm install --frozen-lockfilepnpm exec biome check apps/web/src/components/CodeSnippetModalButton.tsxpnpm --filter web test -- --run --passWithNoTestswas attempted; the suite still fails during test collection due existing module resolution problems (server/queue/Managerand@tryabby/core) outside this UI component change.Summary by CodeRabbit