Skip to content

fix(telemetry): init telemetry in cloud bundle entry points#3346

Merged
la14-1 merged 1 commit intoOpenRouterTeam:mainfrom
AhmedTMM:fix/cloud-bundle-telemetry
Apr 24, 2026
Merged

fix(telemetry): init telemetry in cloud bundle entry points#3346
la14-1 merged 1 commit intoOpenRouterTeam:mainfrom
AhmedTMM:fix/cloud-bundle-telemetry

Conversation

@AhmedTMM
Copy link
Copy Markdown
Collaborator

Summary

  • Cloud bundles (hetzner.js, digitalocean.js, gcp.js, aws.js, sprite.js, local.js, daytona.js) never called initTelemetry(), so _enabled was false and every captureEvent/trackFunnel call in orchestrate.ts was a silent no-op
  • All orchestration funnel events (funnel_cloud_authed through funnel_handoff) were permanently lost — zero real data in PostHog across all time
  • Adds initTelemetry(pkg.version) to all 7 cloud entry points

Test plan

  • Lint clean (biome check)
  • 2104 tests pass (4 pre-existing failures unrelated)
  • After merge + release rebuild, verify funnel_cloud_authed events appear in PostHog

🤖 Generated with Claude Code

@la14-1 la14-1 force-pushed the fix/cloud-bundle-telemetry branch from ab759be to 799dc3f Compare April 24, 2026 01:44
la14-1
la14-1 previously approved these changes Apr 24, 2026
Copy link
Copy Markdown
Member

@la14-1 la14-1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rebased onto current main and bumped version to 1.0.21 (was 1.1.2) to preserve the bridge-release strategy from #3342: keeping the fleet on 1.0.x patches means every user — including those still on 1.0.x — auto-pulls this fix. Shipping a 1.1.x right now would strand anyone who hadn't already picked up 1.0.20.

Change itself is correct and necessary. #3339 made initTelemetry opt-in with _enabled = false by default — but the 7 cloud bundles that runOrchestration executes never called it, silently dropping every funnel_* event from orchestration. 3 lines × 7 entry points fixes it.

Local verification on the rebased branch:

  • bunx @biomejs/biome check src/ — 193 files, no issues
  • bun test — 2108 pass, 0 fail

Shipping after CI confirms.

Cloud bundles (hetzner.js, digitalocean.js, etc.) never called
initTelemetry(), so _enabled was false and every captureEvent/trackFunnel
call in orchestrate.ts was a silent no-op. All orchestration funnel
events (funnel_cloud_authed through funnel_handoff) were lost.

Adds initTelemetry(pkg.version) to all 7 cloud entry points so
funnel events actually reach PostHog.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@la14-1
Copy link
Copy Markdown
Member

la14-1 commented Apr 24, 2026

Rebased against main, version retargeted to 1.0.21 (bridge-release consistency with #3342), and fixed a bundle-build failure: import pkg from "../package.json" resolves to src/package.json (doesn't exist) from a 2-levels-deep file like src/aws/main.ts — all 7 entry points need ../../package.json. Local bun run packages/cli/build-clouds.ts now produces all 7 bundles cleanly.

Also noticed daytona.js is missing from packages/cli/.gitignore while all the other cloud bundles are listed — filed as #3348.

Local verification on the amended commit:

  • bun run packages/cli/build-clouds.ts — 7 built, 0 failed
  • bunx @biomejs/biome check src/ — clean
  • bun test — 2108 pass, 0 fail

Amended into the same commit to keep history clean. Will merge once CI confirms.

Filed from Slack by SPA

Copy link
Copy Markdown
Member

@la14-1 la14-1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Re-approving after the amend that fixed the package.json import path (../../../). All 5 checks green on the amended commit.

@la14-1 la14-1 merged commit 71c61ed into OpenRouterTeam:main Apr 24, 2026
5 checks passed
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.

2 participants