fix(docs): load Promptwatch analytics via next/script#7683
fix(docs): load Promptwatch analytics via next/script#7683aidankmcalister wants to merge 2 commits intomainfrom
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)
WalkthroughAdded a second third-party script tag to the docs app root layout to load PromptWatch ( Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~4 minutes Principal NotesWhile this change is mechanically simple, I'd recommend pausing to consider a few foundational concerns: Security & Privacy: External script injection creates a trust boundary—ensure the PromptWatch domain and its script are verified as legitimate and maintained. This introduces a third-party dependency on runtime behavior. Performance Impact: The Error Resilience: There's no error handling if the external resource fails to load or is blocked. Consider whether monitoring failures should gracefully degrade or if they warrant observability themselves. Scope Clarity: Confirm whether this monitoring should apply only to the docs app or if it will affect other applications in the monorepo. 🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@apps/docs/src/app/layout.tsx`:
- Around line 44-48: Update the PromptWatch <Script> element in layout.tsx (the
Script that loads https://ingest.promptwatch.com/js/client.min.js) to include
the CookieYes consent attribute data-cookieyes="cookieyes-analytics" so the
script only executes after analytics consent; modify the existing Script props
for that component (keep strategy and data-project-id) and also review the
Tolt.io <Script> (the one loading the tolt.io snippet) to determine if it needs
the same data-cookieyes gating and add it if appropriate.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 824a418a-0c4c-4aed-8537-dfb8e860947c
📒 Files selected for processing (1)
apps/docs/src/app/layout.tsx
|
The latest updates on your projects. Learn more about Argos notifications ↗︎
|
Summary by CodeRabbit