Skip to content

fix: send no analytics, traces, or crash reports off the machine - #57

Merged
pseudoseed merged 1 commit into
mainfrom
t3/no-external-analytics
Nov 5, 2026
Merged

pseudoseed merged 1 commit into
mainfrom
t3/no-external-analytics

Conversation

@pseudoseed

Copy link
Copy Markdown
Owner

The fork shipped upstream's outside sinks: server usage events to PostHog through ingest.t3.codes (on by default, key baked in), T3 Connect relay traces to Axiom from server, web, and mobile, and mobile crash reports and launch timings through expo-observe.

Each is disabled at one point, so callers and settings stay as upstream has them and no build config or baked key can turn one back on:

  • AnalyticsService.layer is a no-op.
  • relayTracing.layer never builds an OTLP exporter.
  • Mobile Observe endpoint is https://observe.invalid and dispatching is configured off; the "Send crash reports" toggle is removed.

UI copy and docs now say nothing is sent. docs/internals/product-analytics.md lists the 3 choke points for future upstream syncs. User-configured OTLP export to your own receiver is unchanged.

Tests: the web runtime and mobile tracing tests now assert no request is made even with trace credentials set. vp run check:changed passes.

Model: Claude Opus 5.5, harness: Claude Code in PseudoCode.

🤖 Generated with Claude Code

The fork shipped upstream's outside sinks: server usage events to PostHog
through ingest.t3.codes, relay traces to Axiom from server, web, and mobile,
and mobile crash reports through expo-observe. Each is now disabled at one
point in code, so no build config or baked key can turn it back on.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XL 📱 Native Change Changes the native fingerprint; merging blocks production OTAs until a new store build ships. labels Nov 5, 2026
@github-actions

github-actions Bot commented Nov 5, 2026

Copy link
Copy Markdown

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 5.0 KiB 5.0 KiB −29 B (−0.6%) 6.8 KiB ✅
Codex Thread snapshot wire 3.8 KiB 3.8 KiB 0 B (0.0%) 4.9 KiB ✅
Codex Live turn WebSocket wire 1.2 KiB 1.2 KiB −29 B (−2.3%) 2.0 KiB ✅
Codex Live turn WebSocket decoded 21.6 KiB 21.5 KiB −41 B (−0.2%) 24.4 KiB ✅
Codex Live turn messages 2 1 −1 (−50.0%) 8 ✅
Claude Total thread wire 5.0 KiB 5.0 KiB +29 B (+0.6%) 6.8 KiB ✅
Claude Thread snapshot wire 3.8 KiB 3.8 KiB 0 B (0.0%) 4.9 KiB ✅
Claude Live turn WebSocket wire 1.2 KiB 1.3 KiB +29 B (+2.3%) 2.0 KiB ✅
Claude Live turn WebSocket decoded 21.9 KiB 21.9 KiB +41 B (+0.2%) 24.4 KiB ✅
Claude Live turn messages 1 2 +1 (+100.0%) 8 ✅

Baseline: 5e1d484 · PR result: 0834dcf · Source CI: failure

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 108.5 KiB
  • Claude decoded thread snapshot: 108.8 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

@pseudoseed
pseudoseed merged commit 1ae397d into main Nov 5, 2026
23 of 25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📱 Native Change Changes the native fingerprint; merging blocks production OTAs until a new store build ships. size:XL vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant