[PR #5007] [MERGED] perf: lazy load create telemetry #31314

Closed
opened 2026-04-17 22:11:46 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/better-auth/better-auth/pull/5007
Author: @himself65
Created: 9/30/2025
Status: Merged
Merged: 10/1/2025
Merged by: @himself65

Base: canaryHead: himself65/2025/09/30/perf-lazy-load


📝 Commits (1)

  • 1b6451c perf: lazy load create telemetry

📊 Changes

3 files changed (+99 additions, -85 deletions)

View changed files

packages/better-auth/src/telemetry/create-telemetry.ts (+85 -0)
📝 packages/better-auth/src/telemetry/index.ts (+13 -84)
📝 packages/better-auth/src/telemetry/telemetry.test.ts (+1 -1)

📄 Description

Splitting this into different chunks will improve the startup speed and bundle size

Summary by cubic

Lazy-loads telemetry so code is only loaded when createTelemetry is called. This reduces initial bundle size and avoids unnecessary work when telemetry is disabled.

  • Refactors
    • Moved implementation to create-telemetry.ts.
    • index.ts now dynamic-imports create-telemetry to split the bundle.
    • Updated telemetry.test to import create-telemetry directly.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/better-auth/better-auth/pull/5007 **Author:** [@himself65](https://github.com/himself65) **Created:** 9/30/2025 **Status:** ✅ Merged **Merged:** 10/1/2025 **Merged by:** [@himself65](https://github.com/himself65) **Base:** `canary` ← **Head:** `himself65/2025/09/30/perf-lazy-load` --- ### 📝 Commits (1) - [`1b6451c`](https://github.com/better-auth/better-auth/commit/1b6451c14ee14209148676a5a33bdcfa3e26cba6) perf: lazy load create telemetry ### 📊 Changes **3 files changed** (+99 additions, -85 deletions) <details> <summary>View changed files</summary> ➕ `packages/better-auth/src/telemetry/create-telemetry.ts` (+85 -0) 📝 `packages/better-auth/src/telemetry/index.ts` (+13 -84) 📝 `packages/better-auth/src/telemetry/telemetry.test.ts` (+1 -1) </details> ### 📄 Description Splitting this into different chunks will improve the startup speed and bundle size <!-- This is an auto-generated description by cubic. --> ## Summary by cubic Lazy-loads telemetry so code is only loaded when createTelemetry is called. This reduces initial bundle size and avoids unnecessary work when telemetry is disabled. - **Refactors** - Moved implementation to create-telemetry.ts. - index.ts now dynamic-imports create-telemetry to split the bundle. - Updated telemetry.test to import create-telemetry directly. <!-- End of auto-generated description by cubic. --> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-17 22:11:46 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/better-auth#31314