Skip to content

chore: remove posthog analytics#26

Merged
WINOFFRG merged 4 commits intomasterfrom
fix/remove-da
Mar 28, 2026
Merged

chore: remove posthog analytics#26
WINOFFRG merged 4 commits intomasterfrom
fix/remove-da

Conversation

@WINOFFRG
Copy link
Copy Markdown
Owner

@WINOFFRG WINOFFRG commented Mar 28, 2026

Summary by CodeRabbit

  • Chores
    • Removed analytics and product insights tracking features
    • Removed the "Improve Product" diagnostics sharing toggle from the interface
    • Removed the PostHog analytics dependency

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 28, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 5b72b560-2b36-475a-8dea-5bf3327863ba

📥 Commits

Reviewing files that changed from the base of the PR and between 2cc436c and 9100f07.

⛔ Files ignored due to path filters (1)
  • bun.lock is excluded by !**/*.lock
📒 Files selected for processing (18)
  • .changeset/metal-poets-build.md
  • package.json
  • src/components/App.tsx
  • src/components/OTTModal.tsx
  • src/entrypoints/background.ts
  • src/entrypoints/content/index.tsx
  • src/hooks/useStore.tsx
  • src/lib/logger.ts
  • src/lib/messaging.ts
  • src/lib/posthog-disabled.ts
  • src/lib/posthog-enabled.ts
  • src/lib/posthog-impl.ts
  • src/lib/posthog-provider-disabled.tsx
  • src/lib/posthog-provider-enabled.tsx
  • src/lib/posthog-provider-impl.tsx
  • src/lib/posthog-provider.tsx
  • src/lib/posthog.ts
  • wxt.config.ts

📝 Walkthrough

Walkthrough

This pull request removes PostHog analytics and product insights functionality from the codebase. Changes include deleting the posthog-js dependency, removing PostHog initialization modules, eliminating the product insights toggle from the UI, and cleaning up related hooks, messaging handlers, logging transports, and build configuration.

Changes

Cohort / File(s) Summary
Dependency & Configuration
.changeset/metal-poets-build.md, package.json, wxt.config.ts
Removed posthog-js dependency, deleted PostHog domain from manifest host permissions and web accessible resources, and marked package update as chore.
PostHog Implementation Modules
src/lib/posthog.ts, src/lib/posthog-impl.ts, src/lib/posthog-enabled.ts, src/lib/posthog-disabled.ts, src/lib/posthog-provider.tsx, src/lib/posthog-provider-impl.tsx, src/lib/posthog-provider-enabled.tsx, src/lib/posthog-provider-disabled.tsx
Removed all PostHog integration and provider modules (171 lines deleted from enabled module, 31 lines from disabled module, and related re-export files completely deleted).
UI Components & State Management
src/components/App.tsx, src/components/OTTModal.tsx, src/hooks/useStore.tsx
Removed PostHog context initialization from App, eliminated product insights toggle UI and related handler from OTTModal, and deleted product insights state, actions, and hooks from store (removed productInsightsEnabled property and related toggle/update methods).
Core Infrastructure
src/entrypoints/background.ts, src/entrypoints/content/index.tsx, src/lib/messaging.ts, src/lib/logger.ts
Removed PostHog initialization from entrypoints, deleted GET_PRODUCT_INSIGHTS_ENABLED message type and handlers, removed PostHogTransport from logger transport chain, and eliminated conditional provider wrapping.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~22 minutes

Possibly related PRs

Poem

🐰 Hops away the telemetry toy,
No more peeking at what brings us joy,
PostHog be gone, your insights retire,
The codebase springs free, lighter and higher!

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/remove-da

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@WINOFFRG WINOFFRG merged commit f9c8344 into master Mar 28, 2026
1 check was pending
@github-actions github-actions Bot mentioned this pull request Mar 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant