Repository navigation
rust(feat): expose measurement received time in MCP get_data - #830
Draft
rowen-sift wants to merge 2 commits into
Draft
rowen-sift wants to merge 2 commits into
rowen-sift wants to merge 2 commits into
Conversation
MCP get_data can pass include_received_at through to the data API and write sift_received_at as a unix-nanos column when the service returns it. upload_dataset skips those receipt columns. Co-authored-by: Rowen Wu <rowenwu@users.noreply.github.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This branch has not been deployed
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
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.
Supersedes #817, which closed when its branch was renamed.
Description
MCP
get_datanow acceptsinclude_received_at(aliasincludeReceivedAt). When it is true, the tool sends that flag onGetDataRequestand, when the service returnssift_received_atin extras, writes a receipt-time column next to the channel.timestamp_unix_nanosstays the measurement generation time.<channel> {channel_id="...", received_at="true", ...}. Cells are null when that sample has no receipt time, or when the channel has no sample on the row.upload_datasetskipsreceived_at="true"columns so they are not ingested as channels.Verification
cargo test -p sift_mcp --lib: 577 passed.get_dataparameter deserialization accepts bothinclude_received_atandincludeReceivedAt, and defaults to false.🤖 Generated with Claude Code