From aca034fdfec0ec29bdb47c5feaa4eafd6e68fac7 Mon Sep 17 00:00:00 2001 From: Jerop Kipruto Date: Mon, 23 Jun 2025 20:29:31 -0400 Subject: Refactor usage statistics to be a top-level setting (#1363) This commit refactors the `usageStatisticsEnabled` setting from a sub-property of the `telemetry` configuration to a top-level setting. This change simplifies the configuration by decoupling usage statistics from the telemetry settings. The documentation has also been updated to reflect this change. --- packages/cli/src/config/config.ts | 3 +-- packages/cli/src/config/settings.ts | 1 + 2 files changed, 2 insertions(+), 2 deletions(-) (limited to 'packages/cli/src') diff --git a/packages/cli/src/config/config.ts b/packages/cli/src/config/config.ts index 4e5b0008..7f957245 100644 --- a/packages/cli/src/config/config.ts +++ b/packages/cli/src/config/config.ts @@ -226,9 +226,8 @@ export async function loadCliConfig( process.env.OTEL_EXPORTER_OTLP_ENDPOINT ?? settings.telemetry?.otlpEndpoint, logPrompts: argv.telemetryLogPrompts ?? settings.telemetry?.logPrompts, - usageStatisticsEnabled: - settings.telemetry?.usageStatisticsEnabled ?? true, }, + usageStatisticsEnabled: settings.usageStatisticsEnabled ?? true, // Git-aware file filtering settings fileFiltering: { respectGitIgnore: settings.fileFiltering?.respectGitIgnore, diff --git a/packages/cli/src/config/settings.ts b/packages/cli/src/config/settings.ts index b149216a..882df403 100644 --- a/packages/cli/src/config/settings.ts +++ b/packages/cli/src/config/settings.ts @@ -49,6 +49,7 @@ export interface Settings { contextFileName?: string | string[]; accessibility?: AccessibilitySettings; telemetry?: TelemetrySettings; + usageStatisticsEnabled?: boolean; preferredEditor?: string; bugCommand?: BugCommandSettings; checkpointing?: CheckpointingSettings; -- cgit v1.2.3