It stops reporting into upstream's accounts
Explore this workspace.
Technical background
Every build of this fork carried upstream Tabby's own credentials as literals, so its crashes went to Eugeny's Sentry project, its launches to his Mixpanel, and a published artifact would have authenticated against his Keygen distribution account — all under a version string that means nothing there. The updater and the release-notes tab read his releases too, so this build would offer to update itself to a different program. Analytics is removed outright rather than repointed, and the two switches that fed it go with it: a toggle that no longer sends anything is worse than no toggle, because it reads as a choice the user is making.
The problem
This fork shipped upstream Tabby's own credentials as literals, and had done since the first commit. Its crashes went to Eugeny's Sentry project, its launches to his Mixpanel, and a published artifact would have authenticated against his Keygen distribution account — all under a version string that means nothing in any of them. The updater and the release-notes tab read his releases, so this build would have offered to update itself to a different program.
How it works
The Sentry DSN comes from the environment now, and an absent one skips init entirely rather than initialising a client aimed somewhere else. Mixpanel is gone. Keygen is gone: distribution is GitHub releases only, and the packagecloud step that pushed into eugeny/tabby went with it. The update and release-notes feeds, Website, Source code, Report a problem and FUNDING.yml all point at this project.
Details
- The analytics switches were removed with the analytics. A toggle that no longer sends anything is worse than no toggle, because it reads as a choice the user is making.
- The Discord and Weblate links are upstream's community rather than this project's, and are removed rather than repointed at nothing.
sentry-upload.mjsguards itself on the same credentials, so the three workflow steps that call it became no-ops instead of failures.
What this does not claim
config.service.ts'shttps://api.tabby.shliteral is deliberately not changed. It is compared inside aconfig.version < 7migration, so changing it changes which users get their sync token cleared. It is a version check, not an endpoint.- There is no Sentry project to point at yet, so this build reports no crashes anywhere. That is a deliberate silence, not an oversight.