github-starred/better-auth
2
0
Fork 0
mirror of https://github.com/better-auth/better-auth.git synced 2026-05-22 22:32:01 -05:00
Code Issues 2.5k Packages Projects Releases 112 Wiki Activity

[PR #5152] [MERGED] refactor: move telemetry to single package #14056

New Issue
Closed
opened 2026-04-13 09:17:32 -05:00 by GiteaMirror · 0 comments
GiteaMirror commented 2026-04-13 09:17:32 -05:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/better-auth/better-auth/pull/5152
Author: @himself65
Created: 10/7/2025
Status: ✅ Merged
Merged: 10/7/2025
Merged by: @himself65

Base: canary ← Head: himself65/2025/10/07/telemetry

📝 Commits (2)

  • b4a9cfe refactor: move telemetry to single package
  • ee80b6a fixup! refactor: move telemetry to single package

📊 Changes

21 files changed (+150 additions, -22 deletions)

View changed files

📝 packages/better-auth/package.json (+1 -0)
📝 packages/better-auth/src/index.ts (+5 -3)
📝 packages/better-auth/src/init.ts (+1 -2)
➕ packages/telemetry/build.config.ts (+10 -0)
➕ packages/telemetry/package.json (+41 -0)
📝 packages/telemetry/src/detectors/detect-auth-config.ts (+2 -4)
📝 packages/telemetry/src/detectors/detect-database.ts (+1 -1)
📝 packages/telemetry/src/detectors/detect-framework.ts (+1 -1)
📝 packages/telemetry/src/detectors/detect-project-info.ts (+0 -0)
📝 packages/telemetry/src/detectors/detect-runtime.ts (+0 -0)
📝 packages/telemetry/src/detectors/detect-system-info.ts (+1 -1)
📝 packages/telemetry/src/index.ts (+4 -1)
📝 packages/telemetry/src/project-id.ts (+3 -3)
📝 packages/telemetry/src/telemetry.test.ts (+0 -0)
📝 packages/telemetry/src/types.ts (+26 -0)
➕ packages/telemetry/src/utils/hash.ts (+9 -0)
➕ packages/telemetry/src/utils/id.ts (+5 -0)
➕ packages/telemetry/src/utils/import-util.ts (+3 -0)
📝 packages/telemetry/src/utils/package-json.ts (+0 -0)
➕ packages/telemetry/tsconfig.json (+9 -0)

...and 1 more files

📄 Description

Summary by cubic

Moved all telemetry code into a new package (@better-auth/telemetry) and updated better-auth to depend on and re-export it. This simplifies usage, reduces coupling, and avoids circular dependencies.

  • Refactors

    • Added @better-auth/telemetry with unbuild config and dual CJS/ESM exports.
    • Moved detectors, project-id, types, and tests; added hash, id, import, and package-json utils.
    • Introduced minimal BetterAuthOptions in telemetry to prevent circular imports.
    • better-auth now imports from @better-auth/telemetry and re-exports createTelemetry, getTelemetryAuthConfig, and TelemetryEvent.
    • Updated pnpm-lock; no functional changes expected.

  • Migration

    • If you imported internal telemetry modules from better-auth, switch to @better-auth/telemetry.
    • No changes needed if using telemetry via better-auth’s top-level exports.

🔄 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/5152 **Author:** [@himself65](https://github.com/himself65) **Created:** 10/7/2025 **Status:** ✅ Merged **Merged:** 10/7/2025 **Merged by:** [@himself65](https://github.com/himself65) **Base:** `canary` ← **Head:** `himself65/2025/10/07/telemetry` --- ### 📝 Commits (2) - [`b4a9cfe`](https://github.com/better-auth/better-auth/commit/b4a9cfec538eefa931106b732d5e88f8f16a9236) refactor: move telemetry to single package - [`ee80b6a`](https://github.com/better-auth/better-auth/commit/ee80b6a201540dba94f0a9b0b61a0db0ac9eb0ed) fixup! refactor: move telemetry to single package ### 📊 Changes **21 files changed** (+150 additions, -22 deletions) <details> <summary>View changed files</summary> 📝 `packages/better-auth/package.json` (+1 -0) 📝 `packages/better-auth/src/index.ts` (+5 -3) 📝 `packages/better-auth/src/init.ts` (+1 -2) ➕ `packages/telemetry/build.config.ts` (+10 -0) ➕ `packages/telemetry/package.json` (+41 -0) 📝 `packages/telemetry/src/detectors/detect-auth-config.ts` (+2 -4) 📝 `packages/telemetry/src/detectors/detect-database.ts` (+1 -1) 📝 `packages/telemetry/src/detectors/detect-framework.ts` (+1 -1) 📝 `packages/telemetry/src/detectors/detect-project-info.ts` (+0 -0) 📝 `packages/telemetry/src/detectors/detect-runtime.ts` (+0 -0) 📝 `packages/telemetry/src/detectors/detect-system-info.ts` (+1 -1) 📝 `packages/telemetry/src/index.ts` (+4 -1) 📝 `packages/telemetry/src/project-id.ts` (+3 -3) 📝 `packages/telemetry/src/telemetry.test.ts` (+0 -0) 📝 `packages/telemetry/src/types.ts` (+26 -0) ➕ `packages/telemetry/src/utils/hash.ts` (+9 -0) ➕ `packages/telemetry/src/utils/id.ts` (+5 -0) ➕ `packages/telemetry/src/utils/import-util.ts` (+3 -0) 📝 `packages/telemetry/src/utils/package-json.ts` (+0 -0) ➕ `packages/telemetry/tsconfig.json` (+9 -0) _...and 1 more files_ </details> ### 📄 Description <!-- This is an auto-generated description by cubic. --> ## Summary by cubic Moved all telemetry code into a new package (@better-auth/telemetry) and updated better-auth to depend on and re-export it. This simplifies usage, reduces coupling, and avoids circular dependencies. - **Refactors** - Added @better-auth/telemetry with unbuild config and dual CJS/ESM exports. - Moved detectors, project-id, types, and tests; added hash, id, import, and package-json utils. - Introduced minimal BetterAuthOptions in telemetry to prevent circular imports. - better-auth now imports from @better-auth/telemetry and re-exports createTelemetry, getTelemetryAuthConfig, and TelemetryEvent. - Updated pnpm-lock; no functional changes expected. - **Migration** - If you imported internal telemetry modules from better-auth, switch to @better-auth/telemetry. - No changes needed if using telemetry via better-auth’s top-level exports. <!-- 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-13 09:17:32 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
adapter
astro
awaiting external contributor
blocked
breaking
breaking change
bug
c-devops
core
credentials
database
dependencies
devops
devtools
docs
documentation
duplicate
elysia
enhancement
enterprise
expo
express
fastify
good first issue
help wanted
hono
identity
infra
integration
invalid
javascript
locked
maintenance
need-more-information
needs: info
needs: repro
nextjs
nuxt
oauth
organization
P0
payments
perf
platform
plugin
pull-request
Mirrored from GitHub Pull Request
 question
ready
regression
remix
security
social-provider
solid
stale
svelte
tanstack-start
tracking
version-bump
vue
wontfix
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
GiteaMirror ninjasurge
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/better-auth#14056
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?