chore(web): replace snippet action with CLI guidance#183
chore(web): replace snippet action with CLI guidance#183alan747271363-art wants to merge 1 commit 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. |
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🎯 2 (Simple) | ⏱️ ~7 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)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint skipped: no ESLint configuration detected in root package.json. To enable, add 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 |
![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 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
| Generate Code Snippet | ||
| <DropdownMenuItem | ||
| onClick={() => { | ||
| trackEvent("Dashboard CLI Instructions Clicked"); |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Verify whether the new event is declared in tracking contracts/types and how useTracking is typed.
set -euo pipefail
echo "== Locate useTracking implementation =="
fd -i 'tracking.ts' apps/web/src lib src || true
rg -n -C3 '\buseTracking\b|\btrackEvent\b' apps/web/src lib src
echo
echo "== Locate event type maps / schemas =="
rg -n -C3 'type\s+ServerEvents|type\s+.*Events|Dashboard CLI Instructions Clicked|Dashboard Code Clicked' apps/web/src packages
echo
echo "== Inspect plausible event declarations =="
fd -i 'plausible-events.ts' apps/web/src packages || true
rg -n -C3 'flag_removal_pr_created|Dashboard CLI Instructions Clicked' apps/web/src packagesRepository: tryabby/abby
Length of output: 22877
🏁 Script executed:
#!/bin/bash
set -euo pipefail
echo "== Locate PlausibleEvents typing file =="
fd -i 'plausible-events.ts' apps/web/src
echo
echo "== Search for the new event in the plausible event types =="
rg -n "Dashboard CLI Instructions Clicked" apps/web/src/types apps/web/src/lib
echo
echo "== Inspect plausible-events.ts (event list area) =="
plf="$(fd -i 'plausible-events.ts' apps/web/src | head -n 1)"
wc -l "$plf"
sed -n '1,220p' "$plf"Repository: tryabby/abby
Length of output: 197
🏁 Script executed:
#!/bin/bash
set -euo pipefail
echo "== Inspect plausible-events.ts =="
plf="apps/web/src/types/plausible-events.ts"
wc -l "$plf"
sed -n '1,220p' "$plf"
echo
echo "== Inspect useTracking (apps/web/src/lib/tracking.ts) =="
trk="apps/web/src/lib/tracking.ts"
wc -l "$trk"
sed -n '1,220p' "$trk"
echo
echo "== Search for Dashboard CLI Instructions Clicked in the repo =="
rg -n "Dashboard CLI Instructions Clicked" .Repository: tryabby/abby
Length of output: 1985
Add "Dashboard CLI Instructions Clicked" to the tracked-events typing
useTracking is typed by PlausibleEvents (apps/web/src/lib/tracking.ts → apps/web/src/types/plausible-events.ts), but "Dashboard CLI Instructions Clicked" is not present in PlausibleEvents—so this event name should be added there (with never if it has no props).
🤖 Prompt for 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.
In `@apps/web/src/components/CodeSnippetModalButton.tsx` at 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.
1 participant
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