豆豆友情提示:这是一个非官方 GitHub 代理镜像,主要用于网络测试或访问加速。请勿在此进行登录、注册或处理任何敏感信息。进行这些操作请务必访问官方网站 github.com。 Raw 内容也通过此代理提供。
Skip to content

feat: record client name in telemetry.#1208

Merged
yulunz merged 2 commits intomainfrom
feat/client-name
Mar 21, 2026
Merged

feat: record client name in telemetry.#1208
yulunz merged 2 commits intomainfrom
feat/client-name

Conversation

@yulunz
Copy link
Copy Markdown
Collaborator

@yulunz yulunz commented Mar 19, 2026

The changes in the PR intercepts the name of the MCP client right after server initialization, coverts it into one of the existing enum values, and append it with the telemetry request.

Note that the client name is unset if there is no client connected yet. This means the server_start event won't have a client name.

Tested with gemini-cli.

BEGIN_COMMIT_OVERRIDE
fix(telemetry): record client name
END_COMMIT_OVERRIDE

@yulunz yulunz requested a review from wolfib March 20, 2026 01:13
@yulunz yulunz added this pull request to the merge queue Mar 21, 2026
Merged via the queue into main with commit 9a47b65 Mar 21, 2026
20 checks passed
@yulunz yulunz deleted the feat/client-name branch March 21, 2026 00:15
@Lightning00Blade
Copy link
Copy Markdown
Collaborator

Super nit: this should have been marked as a fix (if we want to make a new release for it) or even chore as it's not an user facing feature per say, but something for us.

You can use the following to update it after the fact - https://github.com/googleapis/release-please?tab=readme-ov-file#how-can-i-fix-release-notes

OrKoN pushed a commit to mvanhorn/chrome-devtools-mcp that referenced this pull request Apr 1, 2026
The changes in the PR intercepts the name of the MCP client right after
server initialization, coverts it into one of the existing enum values,
and append it with the telemetry request.

Note that the client name is unset if there is no client connected yet.
This means the server_start event won't have a client name.

Tested with gemini-cli.

---------

Co-authored-by: Yulun Zeng <yulunz@chromium.org>
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.

3 participants