Chore/clean unused code: Add apiKey guard to PostHog initialization

[mattheworiordan]

See relaed Chore/clean unused code and resources .

This skips PostHog initialization if no apiKey is provided. This prevents errors when PostHog is not configured in certain environments (this guard exists for Mixpanel)

Summary by CodeRabbit

• Bug Fixes
  • Fixed initialization errors that occurred when analytics configuration was incomplete, improving overall application stability.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

A guard clause was added to the initPosthog function to return early when the apiKey parameter is falsy, preventing PostHog initialization with invalid credentials.

Changes

Cohort / File(s)	Summary
PostHog Guard Clause src/core/insights/posthog.ts	Added early-return guard condition to skip PostHog initialization when apiKey is falsy (null, undefined, or empty string).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A tiny guard stands at the gate,
"No empty keys shall pass today!"
With just four lines, we seal our fate,
And guard PostHog the rabbit way. ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The PR description is incomplete. It lacks required sections from the template including 'How do you manually test this?', 'Merge/Deploy Checklist', and 'Frontend Checklist' sections.	Complete the PR description by filling out the missing template sections, particularly the testing instructions and applicable checklists for this change.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: adding an apiKey guard to PostHog initialization to prevent errors when the key is not provided.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[aralovelace]

I agree with this change since since apikey is required it will never be null.