Skip to content

fix(telemetry): report every region to the one telemetry host - #224

Merged
elkaix merged 1 commit into
mainfrom
fix/telemetry-single-host
Aug 27, 2026
Merged

fix(telemetry): report every region to the one telemetry host#224
elkaix merged 1 commit into
mainfrom
fix/telemetry-single-host

Conversation

@elkaix

@elkaix elkaix commented Aug 27, 2026

Copy link
Copy Markdown
Member

[skip changeset] — no user-perceivable change: telemetry is invisible to users, and the affected path was already failing silently.

Summary by CodeRabbit

  • Changes
    • Telemetry reporting now uses a single endpoint across all regions.
    • Updated telemetry handling to consistently use the .com host.
    • Removed region-based telemetry opt-out behavior.
    • Updated automated checks and test configuration to reflect the unified telemetry destination.

The global region profile pointed at telemetry-logs.pythinker.ai, a host
that does not exist, so anyone whose install marker said 'global' sent
telemetry into a dead name. There is one Pythinker telemetry host; drop
the second endpoint and the install-marker selection that chose it.
@coderabbitai

coderabbitai Bot commented Aug 27, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 76015362-5754-4a9b-b412-9e6d2c5091af

📥 Commits

Reviewing files that changed from the base of the PR and between e6778dc and 67d3dd5.

📒 Files selected for processing (5)
  • apps/pythinker-code/src/utils/region.ts
  • packages/agent-core-v2/src/app/telemetry/cloudAppender.ts
  • packages/agent-core-v2/src/app/telemetry/cloudTransport.ts
  • packages/agent-core-v2/test/app/telemetry/cloudAppender.test.ts
  • packages/agent-gateway/test/setup.ts
💤 Files with no reviewable changes (1)
  • packages/agent-core-v2/src/app/telemetry/cloudAppender.ts

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

Telemetry now uses one shared .com endpoint. CloudTransport no longer reads install markers or accepts marker-related options. Tests reflect the single-host behavior and updated gateway telemetry filtering.

Changes

Telemetry endpoint routing

Layer / File(s) Summary
Unify telemetry endpoint resolution
apps/pythinker-code/src/utils/region.ts, packages/agent-core-v2/src/app/telemetry/cloudTransport.ts
Both region profiles use the shared .com endpoint. CloudTransport removes marker-based endpoint selection and the homeDir and readMarker options.
Update telemetry validation
packages/agent-core-v2/test/app/telemetry/cloudAppender.test.ts, packages/agent-gateway/test/setup.ts
Appender tests expect the single telemetry host and remove marker opt-out cases. Gateway setup no longer blocks the former .ai host.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: 🔵 Low · up to 67d3d

Global telemetry will be sent to the shared .com endpoint instead of the previous .ai endpoint. The change is mergeable with explicit owner awareness that the destination has equivalent security, data-handling, and regional controls, including for queued telemetry during upgrades or rollback.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description explains the changeset decision but omits the required Related Issue, Problem, What changed, and Checklist sections. Update the description to include all required template sections. Add the related issue, problem statement, change summary, and checklist status.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title uses the required fix prefix, stays within 72 characters, uses imperative mood, and accurately describes the telemetry change.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 4…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 4 files.

  • Fix all pre-merge checks with AI

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.

@pkg-pr-new

pkg-pr-new Bot commented Aug 27, 2026

Copy link
Copy Markdown
pnpm dlx https://pkg.pr.new/@pymodel/pythinker-code@67d3dd5
npx https://pkg.pr.new/@pymodel/pythinker-code@67d3dd5

commit: 67d3dd5

@elkaix
elkaix merged commit 455f290 into main Aug 27, 2026
28 of 29 checks passed
@elkaix
elkaix deleted the fix/telemetry-single-host branch August 27, 2026 18:27
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