stacklok / toolhive-studio / 24718153824

66%

main: 71%

Build Type

Pull #2058

github

Committed by samuv

Commit Message

refactor(telemetry): drop dual-write in sentry IPC handlers

`sentry.is-enabled`, `sentry.opt-in`, and `sentry.opt-out` wrote to
both electron-store and SQLite and read back from electron-store.
Now that `main/src/sentry.ts` reads exclusively from SQLite (via the
shared `getIsTelemetryEnabled` helper), the dual-write is dead weight.

- `sentry.is-enabled` delegates to `getIsTelemetryEnabled()`
- `sentry.opt-in` / `sentry.opt-out` write the new value via
  `writeSetting` only and return the boolean directly
- Drop the `telemetryStore` and `log` imports along with the
  `try/catch` that guarded the SQLite side of the dual-write

`main/src/telemetry-store.ts` is retained for now; it is only
consumed by `main/src/db/reconcile-from-store.ts` for the one-time
electron-store → SQLite migration, matching how `autoUpdateStore`,
`quitConfirmationStore`, and friends are handled.

Pull Request Pull Request #2058: refactor(telemetry): migrate Sentry main-process telemetry gate to SQLite

Coverage Stats

3844 of 6339 branches covered (60.64%)

0 of 11 new or added lines in 3 files covered. (0.0%)

1 existing line in 1 file now uncovered.

5983 of 9020 relevant lines covered (66.33%)

119.38 hits per line