-
Notifications
You must be signed in to change notification settings - Fork 77
feat: segment telemetry #329
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 22 commits
Commits
Show all changes
48 commits
Select commit
Hold shift + click to select a range
5c11a41
feat: implement segment telemetry
MQ37 3e58be6
add reason input schema field for telemetry
MQ37 5eb7f70
fix: reorder telemetry section in the table of contents
MQ37 61480bc
fix: update default userId to an empty string in trackToolCall function
MQ37 1b1215d
fix typo
MQ37 6cd4af3
Merge remote-tracking branch 'origin/master' into segment-telemetry
MQ37 0f1fbda
Merge branch 'master' into segment-telemetry
jirispilka 2f5a4a3
fix: add new variable telemetry-enabled instead of telemetry. Hide de…
jirispilka 92e8024
fix: add constants instead of prod and dev
jirispilka 4016352
fix: add capabilities to tracking
jirispilka 82eefe5
fix: remove unused ENVIROMENT variable
jirispilka d779506
fix: add env variable to enable/disable telemetry
jirispilka 0e6cdfd
fix: add env variable to enable/disable telemetry
jirispilka 72b9ed6
fix: update unit-tests
jirispilka 53bead4
fix: update telemetry properties
jirispilka f46bff9
fix: add status and exec time to telemetry
jirispilka f3fb01c
fix: add tool call number
jirispilka 3f0d09d
fix: remove reason from telemetry
jirispilka a5fd735
fix: change tool status to match Actor run status
jirispilka a551686
fix: change event name
jirispilka 0bf506b
fix: tests
jirispilka 2004c6f
fix: remove TELEMETRY.md
jirispilka 336b246
fix: add user-cache (TTL)
jirispilka 92948ed
Apply suggestions from code review
jirispilka 4c9592f
Apply suggestions from code review
jirispilka 944875a
fix: remove map of clients (one client per deployment)
jirispilka 3b14e56
fix: remove _initializeRequestData from loadToolsFromUrl
jirispilka 4cc26d9
fix: in the ActorsMcpServer use nested telemetry object
jirispilka ab7f161
fix: in the ActorsMcpServer use nested telemetry object
jirispilka 9da99b0
fix: log errors in telemetry
jirispilka d6e7c6d
fix: move ActorsMcpServerOptions and ToolCallCounterStore to types.ts
jirispilka da55972
fix: Improve getAndIncrementToolCallCounter and add test
jirispilka 5ba5751
fix: Improve naming in setupTelemetry function
jirispilka a651486
fix: comment
jirispilka 11e57f0
fix: refactor prepareTelemetryData
jirispilka 571c087
fix: simplify userId cache
jirispilka 421f1a3
fix: make telemetryEnv uppercase -> PROD, DEV
jirispilka 8c8721f
fix: handle telemetry-enabled param in better way (also for null and …
jirispilka 209e41f
fix: simplify - get package version
jirispilka a3c5ece
fix: rename app_name to app
jirispilka 5165cc0
fix: save mcp_client_capabilities
jirispilka dd5291e
fix: set tool-status failed
jirispilka e3a4bc0
fix: Segment requires either userId OR anonymousId, but not both When…
jirispilka 17c1860
Apply suggestions from code review
jirispilka 8c05a44
fix: add units to analyticsClient creation. Replace `sessionToolCallC…
jirispilka 7bfe235
fix: types, makes telemetry.enabled required
jirispilka 0448298
fix: disable telemetry for tests
jirispilka f703e90
fix: remove tool_count_number, it is not required
jirispilka File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.