Skip to content

live view telemetry#13

Open
Sayan- wants to merge 4 commits into
masterfrom
sayan/kernel-1301-add-more-events
Open

live view telemetry#13
Sayan- wants to merge 4 commits into
masterfrom
sayan/kernel-1301-add-more-events

Conversation

@Sayan-
Copy link
Copy Markdown
Collaborator

@Sayan- Sayan- commented May 22, 2026

Note

Medium Risk
Adds a new built-in plugin that asynchronously sends session connect/disconnect events over HTTP, introducing new outbound network behavior and concurrency/queueing paths (though gated by a config flag and tested).

Overview
Adds a new built-in telemetry plugin that can forward live-view session connect/disconnect events to a configured HTTP endpoint (defaulting to http://127.0.0.1:10001/telemetry/events), including a computed duration_ms on disconnect.

The plugin is opt-in via new flags telemetry.enabled and telemetry.endpoint, uses a bounded in-memory queue with a background worker (dropping events when saturated to avoid blocking session goroutines), and includes tests covering disabled behavior, event emission, and non-blocking behavior on endpoint failures.

Reviewed by Cursor Bugbot for commit c787d1c. Bugbot is set up for automated code reviews on this repo. Configure here.

@firetiger-agent
Copy link
Copy Markdown

Firetiger deploy monitoring skipped

This PR didn't match the auto-monitor filter configured on your GitHub connection:

Any PR that changes the kernel API. Monitor changes to API endpoints (packages/api/cmd/api/) and Temporal workflows (packages/api/lib/temporal) in the kernel repo

Reason: PR title 'live view telemetry' does not indicate changes to kernel API endpoints or Temporal workflows; please provide more details or opt in manually if this affects those areas.

To monitor this PR anyway, reply with @firetiger monitor this.

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit c787d1c. Configure here.

Comment thread server/internal/plugins/telemetry/plugin.go
@Sayan- Sayan- requested review from archandatta and hiroTamada May 22, 2026 22:46
Copy link
Copy Markdown

@hiroTamada hiroTamada left a comment

Choose a reason for hiding this comment

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

LGTM

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