Skip to content

chore(analytics): drop UI-side Aptabase events#449

Merged
robinebers merged 1 commit intomainfrom
chore/remove-aptabase-events
May 7, 2026
Merged

chore(analytics): drop UI-side Aptabase events#449
robinebers merged 1 commit intomainfrom
chore/remove-aptabase-events

Conversation

@robinebers
Copy link
Copy Markdown
Owner

@robinebers robinebers commented May 7, 2026

Summary

  • Remove 5 custom Aptabase events fired from the React app: provider_refreshed, providers_reordered, setting_changed, update_accepted, and provider_toggled
  • Drop the related track() import + tests; the analytics wrapper module stays in place but has no callers in the JS layer
  • The Rust-side daily-active app_started event remains untouched (sent at most once per UTC day from src-tauri/src/lib.rs)

Test plan

  • bun run test — 1076 passing (one obsolete page_viewed non-tracking test removed)

Made with Cursor


Note

Low Risk
Removes client-side analytics calls and related test assertions without changing core app behavior; low risk aside from losing telemetry signals.

Overview
Drops React/UI-layer Aptabase tracking by removing track() calls for plugin refresh/reorder/toggle, settings changes, and update acceptance (e.g. in App, useProbeRefreshActions, and settings action hooks).

Cleans up tests by removing analytics mocks/assertions and deleting an obsolete startup/navigation tracking test; src/lib/analytics remains covered by a simplified generic event test.

Reviewed by Cursor Bugbot for commit 235fedf. Bugbot is set up for automated code reviews on this repo. Configure here.


Summary by cubic

Removed all UI-side Aptabase analytics to simplify the React app and reduce noise. The only remaining analytics is the Rust-side daily app_started; the JS track() helper remains but has no React callers.

  • Refactors
    • Removed event calls: provider_refreshed, providers_reordered, setting_changed, update_accepted, provider_toggled.
    • Cleaned up related imports, mocks, and assertions in tests; removed the obsolete page_viewed test.
    • Kept src/lib/analytics and track(); still a no-op on web and forwards events when running in Tauri (tests updated).

Written for commit 235fedf. Summary will update on new commits.

Removes the provider_refreshed, providers_reordered, setting_changed,
update_accepted, and provider_toggled custom events. Daily-active
app_started (Rust-side) is the only remaining tracked event.

Co-authored-by: Cursor <cursoragent@cursor.com>
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 12 files

@robinebers robinebers merged commit cb365ef into main May 7, 2026
4 checks passed
@robinebers robinebers deleted the chore/remove-aptabase-events branch May 7, 2026 03:18
fathiraz added a commit to fathiraz/openusage that referenced this pull request May 7, 2026
* main:
  chore: bump version to 0.6.23
  chore(analytics): drop UI-side Aptabase events (robinebers#449)
  fix(claude): remove peak hours indicator integration (robinebers#447)
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.

1 participant