@W-20742343 Product telemetry#209
Merged
Merged
Conversation
jarhun88
commented
Feb 5, 2026
jarhun88
commented
Feb 5, 2026
anyoung-tableau
approved these changes
Feb 9, 2026
allisonbierschenk
pushed a commit
to allisonbierschenk/tableau-mcp
that referenced
this pull request
Apr 14, 2026
* adding product telemetry wiring * pod and hostname * remove example * linting * making changes to telemetry forwarder * simplifying the telemetry * added podname * passing sessionID to all tools * added PRODUCT_TELEMETRY_ENABLED env variable * pod to podname * fixing sessionID error * remove log * patch bump * fix test tsc compilation * fix lint * move podname out of properties override * lint fix * updated telemetry to trigger from response * refactor all env variables to only be set in config.ts, and pass productTelemetryBase to tool.ts * additional info for PRODUCT_TELEMETRY_ENABLED env var * set sessionID type to be string | undefined and remove _ from productTelemtry dir name * remove public modifier and fix test reference to productTelemetry dir * remove unnecessary request params * making prod telemetry call use async await * move singleton to telemetryforwarder.ts * remove env var PRODUCT_TELEMETRY_ENABLED from telemetryForwarder.ts * exportedForTesting DirectTelemetryForwarder * remove unnecessary export * string literal for eventType param * string literal * fix getHttpStatus error type being unknown * change sitename to siteluid * remove parameters * refactor errors to handle constrainedError status codes * remove logs * remove export * update tests * podname using env variable from config * address nits * create tests for getSiteLuidFromAccessToken * version bump
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
IMPORTANT: Please do not create a Pull Request without creating an issue first.
Any change needs to be discussed before proceeding. Failure to do so may result in the rejection of
the pull request.
Pull Request Template
Description
product telemetry to get UIP data that we can observe on how the server is used. For now we are aiming to track the following fields
here is the product telemetry tableau system that were utilizing https://docs.google.com/document/d/11zY6e2uXLlwetLo7NtrebmDa61TVa5WkYPIi7ngg63Q/edit?tab=t.0#heading=h.mskhvfu51hln
here is the associated juno schema http://juno.tsi.lan/manage/analytics.tableau.tableau_mcp.tableau_mcp.tool_call
created a new env variable to disable the product telemetry. (default is enabled)
passing through sessionID through all tools
Motivation and Context
Type of Change
How Has This Been Tested?
Related Issues
Checklist
npm run version. For example,use
npm run version:patchfor a patch version bump.environment variable or changing its default value.
Contributor Agreement
By submitting this pull request, I confirm that:
its Contribution Checklist.