Skip to content

Conversation

@jonaslagoni
Copy link
Contributor

@jonaslagoni jonaslagoni commented Dec 16, 2025

Note

Prioritizes environment variables for telemetry config and adds env overrides for endpoint, tracking ID, and API secret, with updated docs and comprehensive tests.

  • Telemetry Config (src/telemetry/config.ts):
    • Prioritizes environment variables over project and global config.
    • Applies env overrides for CODEGEN_TELEMETRY_ENDPOINT, CODEGEN_TELEMETRY_ID, CODEGEN_TELEMETRY_API_SECRET; keep disable via CODEGEN_TELEMETRY_DISABLED/DO_NOT_TRACK.
    • Extends disabled config to include apiSecret.
  • Tests (test/telemetry/config.spec.ts):
    • Adds cases for individual and combined env var overrides and their precedence over project/global config.
    • Verifies disable precedence via env vars and non-throwing error paths.
  • Docs (docs/telemetry.md):
    • Documents env var priority and configuration order.
    • Provides env examples for custom endpoint, tracking ID, and API secret.

Written by Cursor Bugbot for commit 17ba837. This will update automatically on new commits. Configure here.

@netlify
Copy link

netlify bot commented Dec 16, 2025

Deploy Preview for the-codegen-project canceled.

Name Link
🔨 Latest commit 17ba837
🔍 Latest deploy log https://app.netlify.com/projects/the-codegen-project/deploys/6941ef63b8734f0008c60e80

@jonaslagoni jonaslagoni merged commit 3f6ed79 into main Dec 16, 2025
20 checks passed
@jonaslagoni jonaslagoni deleted the reenable_telemetry_env branch December 16, 2025 23:50
@jonaslagoni
Copy link
Contributor Author

🎉 This PR is included in version 0.60.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants