Add telemetry opt-out via DO_NOT_TRACK and MCPJAM_TELEMETRY_DISABLED env vars

[ignaciojimenezr]

Closes #1544

[chelojimenez]

✅ Snyk checks have passed. No issues have been found so far.

Status	Scanner	Critical	High	Medium	Low	Total (0)
✅	Open Source Security	0	0	0	0	0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: f59fa04f-dfbf-4130-9940-f998965bdbe0

📥 Commits

Reviewing files that changed from the base of the PR and between a31a834 and 493f1a8.

📒 Files selected for processing (1)

• sdk/README.md

🚧 Files skipped from review as they are similar to previous changes (1)

• sdk/README.md

---

Walkthrough

Two telemetry documentation blocks were added to sdk/README.md describing anonymous usage metrics and environment-variable options to disable telemetry. In sdk/src/telemetry.ts, the exported posthog became conditional: the module now checks DO_NOT_TRACK or MCPJAM_TELEMETRY_DISABLED and exports a noopPosthog shim when disabled, otherwise it exports a real PostHog instance. No other public declarations were changed.

---

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.}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@sdk/src/telemetry.ts`:
- Around line 7-17: The noopPosthog object has a capture signature with no args
which causes the exported posthog to have an incompatible union type; update
noopPosthog so its method signatures match PostHog's API (e.g., change capture
to accept the same EventMessage parameter, and ensure flush/shutdown match
PostHog types), import or reference PostHog/EventMessage types and type-annotate
noopPosthog (or cast noopPosthog as PostHog) so the exported posthog variable
has a consistent PostHog-compatible type rather than PostHog | typeof
noopPosthog.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 2ca44975-0803-4fa9-923e-733b3582e997

📥 Commits

Reviewing files that changed from the base of the PR and between 3817280 and 1493302.

📒 Files selected for processing (2)

• sdk/README.md
• sdk/src/telemetry.ts

[chelojimenez]

:shipi

[mgyarmathy]

Thanks @ignaciojimenezr !

[ignaciojimenezr]

of course!