GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — feat: report agent connection methods and app identities by EhabY · Pull Request #30467 · coder/coder

Conversation

@EhabY

@EhabY EhabY commented Oct 7, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

Summary

Agents now report how a connection was made and which app made it, completing the agent side of the Completing Scalable IDE Session Tracking RFC. Follows merged #30466, which added the matching connection log fields.

Closes CLIENT-1037.

What changes

Agent API v2.13

Connection gains two fields and the nested Method enum. Field numbers for existing fields are unchanged. The METHOD_ prefix follows the protobuf enum naming guidance: enum values share the Connection scope with Type.

Field Number Notes
app_name 9 The client app, such as cursor. Empty when unknown.
connection_method 10 METHOD_SSH, METHOD_RECONNECTING_PTY, or METHOD_PORT_FORWARDING (reserved; not sent yet).
type 3 Deprecated, superseded by connection_method and app_name. Agents no longer send it; coderd still reads it from agents before v2.13.

API v2.12 shipped in v2.38.0; this PR requests v2.13 through ConnectRPC213WithRole. Released servers predating the new fields reject that RPC version before receiving reports, so dual-send provides no compatibility benefit. Advance the minor if it ships without this handler.

Agent

  • SSH and reconnecting PTY handlers set the method themselves, so a client-supplied app name can't change it.
  • SSH sessions report the app from CODER_SSH_SESSION_TYPE. Sessions without one still count as SSH but report no app.
  • Disconnect reports repeat the connect report's method, app, and client session ID.

Coderd

  • If a report has no method, it comes from an agent before v2.13, so coderd maps its type as before: VS Code and JetBrains become SSH with that app.
  • If a report has a method, coderd uses the method and the normalized app name, and ignores type.
  • Unknown methods are rejected, so they are never logged as SSH. The RPC connection stays open.

Compatibility

Agent Coderd Result
New New Method and app logged as reported.
Old New Method and app recovered from type, as before.
New Old Not supported, as with every minor bump: coderd before v2.13 rejects the agent's API version.

Stack

Follows #30466 (merged). Replaces #29675. Out of scope: JetBrains per-product names and a raw port-forward producer.

Review guide and line breakdown

Counts are relative to main.

Category Added Deleted Where to look
Source +84 −40 Reporter, SSH and PTY handlers, agent.proto, sdk2db ingestion, v2.13 client plumbing
Generated +708 −620 agent/proto/agent.pb.go
Tests +154 −52 Ingestion mapping, reporter, real agent reporting through coderd, agent test-client shim
Total +946 −712

Generated with Coder Agents on behalf of @EhabY.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
This was referenced Oct 7, 2026
feat: separate connection log methods from app names #30466
Merged
feat: report the app that made a connection #29675
Closed
feat: separate connection log methods from app identities #29674
Closed
@github-actions github-actions Bot assigned EhabY Oct 7, 2026
@EhabY
EhabY added this pull request to stack #30469 October 7, 2026 14:01
@EhabY
EhabY force-pushed the feat/agent-connection-methods branch from 98e6a68 to 2ee229b Compare October 7, 2026 17:33
This was referenced Oct 8, 2026
fix: deduplicate template insights family metrics #30497
Merged
feat!: label the VS Code family and report built-in app usage per app #29906
Merged
@EhabY
EhabY force-pushed the feat/agent-connection-methods branch 2 times, most recently from 0334811 to 3b02590 Compare October 8, 2026 14:36
@EhabY
EhabY force-pushed the feat/agent-connection-methods branch 3 times, most recently from 79e5cf4 to 527a142 Compare October 9, 2026 12:27
@EhabY
EhabY force-pushed the feat/agent-connection-methods branch from 527a142 to 8e4a0ba Compare October 12, 2026 12:42
Base automatically changed from feat/connection-log-methods to main October 12, 2026 12:56
@EhabY
EhabY force-pushed the feat/agent-connection-methods branch 3 times, most recently from 13db794 to 5319db6 Compare October 12, 2026 13:19
@EhabY
EhabY marked this pull request as ready for review October 12, 2026 13:19
@EhabY
EhabY requested review from johnstcn and spikecurtis as code owners October 12, 2026 13:19
@EhabY
EhabY force-pushed the feat/agent-connection-methods branch 3 times, most recently from d2199df to 9d1fb02 Compare October 12, 2026 21:01
@linear-code

linear-code Bot commented Oct 13, 2026

Copy link
Copy Markdown

CLIENT-1037

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@EhabY EhabY mentioned this pull request Oct 13, 2026
Separate connection log methods from app identities #27412
Closed
6 tasks done
@EhabY EhabY mentioned this pull request Oct 13, 2026
Normalize workspace agent session counts into a child table #27410
Closed
1 of 3 tasks
@EhabY
EhabY requested a review from code-asher October 13, 2026 16:19
@EhabY
EhabY force-pushed the feat/agent-connection-methods branch from 9d1fb02 to 98590bc Compare October 13, 2026 16:48
@EhabY EhabY mentioned this pull request Oct 13, 2026
fix(net/portmapper): release mappings and UPnP connections on Close coder/tailscale#135
Merged
code-asher
code-asher approved these changes Oct 13, 2026
View reviewed changes
Comment thread coderd/workspaceagentsrpc_test.go Resolved

Uh oh!

There was an error while loading. Please reload this page.

@coder coder Bot mentioned this pull request Oct 13, 2026
docs: add guide for tracing workspace connections #30707
Closed
@EhabY
feat: report agent connection methods and app identities
e61109c
@EhabY
EhabY force-pushed the feat/agent-connection-methods branch from 98590bc to e61109c Compare October 14, 2026 10:25
@EhabY
EhabY merged commit f573ec3 into main Oct 14, 2026
29 checks passed

Uh oh!

There was an error while loading. Please reload this page.

@EhabY
EhabY deleted the feat/agent-connection-methods branch October 14, 2026 10:54
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 14, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@code-asher code-asher code-asher approved these changes
@spikecurtis spikecurtis Awaiting requested review from spikecurtis spikecurtis is a code owner
@johnstcn johnstcn Awaiting requested review from johnstcn

Uh oh!

There was an error while loading. Please reload this page.

Assignees

@EhabY EhabY

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@EhabY @code-asher
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.