Skip to content

Conversation

@benthomasson
Copy link

This PR is a POC for using Segment and aap-mcp-server together.

This code correctly sends telemetry to segment when the server starts and stops:

Screenshot 2025-11-04 at 11 12 31 AM

@benthomasson benthomasson force-pushed the analytics-claude-code branch from 6983ff3 to 8f5334d Compare November 4, 2025 16:16
Assisted-by: Claude Code
@benthomasson benthomasson force-pushed the analytics-claude-code branch from 8f5334d to 3fc64d6 Compare November 4, 2025 16:17
@benthomasson
Copy link
Author

Now sends UI analytics as well:

Screenshot 2025-11-04 at 11 26 28 AM

@benthomasson
Copy link
Author

The test coverage for the added files is very high:

`--------------------|---------|----------|---------|---------|-------------------------------------------------------

File % Stmts % Branch % Funcs % Lines Uncovered Line #s
All files 23.94 90.47 84.74 23.94
src 25.86 93.57 85.71 25.86
analytics.ts 99.1 96 100 99.1 192-193
...
utils.ts 88.88 90 100 88.88 44-46
-------------------- --------- ---------- --------- --------- -------------------------------------------------------`

@benthomasson
Copy link
Author

This was an exercise in heavily using claude code. It needs thorough review.

@benthomasson
Copy link
Author

@jameswnl @goneri is this POC useful for segment integration?

@goneri
Copy link
Contributor

goneri commented Nov 6, 2025

Hi @benthomasson,

We won't enable the UI in production. The analytics will only cover the /mcp endpoints.

@benthomasson
Copy link
Author

@mabashian can you help test this POC?

@benthomasson
Copy link
Author

@mabashian tested this POC in his environment and was able to send events to segment with it:

Screenshot 2025-11-11 at 11 20 48 AM

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