Skip to content

chore: add Vercel Speed Insights integration#2534

Merged
aeneasr merged 2 commits intomasterfrom
chore/add-vercel-speed-insight
Apr 29, 2026
Merged

chore: add Vercel Speed Insights integration#2534
aeneasr merged 2 commits intomasterfrom
chore/add-vercel-speed-insight

Conversation

@jaygiang
Copy link
Copy Markdown
Collaborator

@jaygiang jaygiang commented Apr 28, 2026

Related Issue or Design Document

Checklist

  • I have read the contributing guidelines and signed the CLA.
  • I have referenced an issue containing the design document if my change introduces a new feature.
  • I have read the security policy.
  • I confirm that this pull request does not address a security vulnerability.
    If this pull request addresses a security vulnerability,
    I confirm that I got approval (please contact security@ory.com) from the maintainers to push the changes.
  • I have added tests that prove my fix is effective or that my feature works.
  • I have added the necessary documentation within the code base (if appropriate).

Further comments

Summary by CodeRabbit

  • New Features
    • Integrated performance monitoring and analytics to track site performance metrics, enabling data-driven optimizations for improved user experience.

Copilot AI review requested due to automatic review settings April 28, 2026 18:40
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR integrates Vercel Speed Insights into the Docusaurus-based docs site by adding the SDK, injecting it on the client, and registering the module in the Docusaurus configuration.

Changes:

  • Add @vercel/speed-insights as a dependency.
  • Introduce a Docusaurus clientModules entry that calls injectSpeedInsights() in the browser.
  • Register the new client module in docusaurus.config.ts.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.

File Description
src/clientModules/speedInsights.tsx Adds a client-only module that injects Vercel Speed Insights.
docusaurus.config.ts Registers the Speed Insights client module with Docusaurus.
package.json Adds the @vercel/speed-insights dependency.
package-lock.json Updates lockfile to include the new dependency (and currently includes many unrelated changes).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/clientModules/speedInsights.tsx
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 28, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 52aa358d-ebeb-46dd-a171-0dc50e0b8469

📥 Commits

Reviewing files that changed from the base of the PR and between c8d3c89 and c07b4bf.

⛔ Files ignored due to path filters (1)
  • package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (3)
  • docusaurus.config.ts
  • package.json
  • src/clientModules/speedInsights.tsx

📝 Walkthrough

Walkthrough

Vercel Speed Insights integration is added to a Docusaurus site by introducing the @vercel/speed-insights dependency, registering a client-side module in the Docusaurus configuration, and creating a new module that conditionally initializes speed insights in the browser context.

Changes

Cohort / File(s) Summary
Build Configuration
docusaurus.config.ts, package.json
Added @vercel/speed-insights dependency and registered the client module in Docusaurus configuration via clientModules entry.
Client Module
src/clientModules/speedInsights.tsx
New file that imports and invokes injectSpeedInsights() conditionally in browser (non-SSR) context.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes


A speedy dash, a config's gleam,
Speed Insights join the Docusaurus dream,
No browser hops, no server fright,
Just client-side magic, burning bright! 🐰⚡

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The pull request description is completely empty of substantive content, containing only template placeholders with no explanation of changes, no issue/design document reference, and no checklist items marked. Add a description explaining why Vercel Speed Insights is being integrated and what benefit it provides. Reference any related issue or design document. Check applicable checklist items to confirm compliance with contribution guidelines.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding Vercel Speed Insights integration, which is reflected in the package.json dependency addition, new client module, and Docusaurus configuration update.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/add-vercel-speed-insight

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Member

@aeneasr aeneasr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes please!

@aeneasr aeneasr enabled auto-merge (squash) April 29, 2026 06:17
@aeneasr aeneasr merged commit e71b9c5 into master Apr 29, 2026
18 checks passed
@aeneasr aeneasr deleted the chore/add-vercel-speed-insight branch April 29, 2026 06:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants