GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Adding metadata_source tag to calls to Python client library for usage logs by lucyking140 · Pull Request #86 · datacommonsorg/agent-toolkit

Conversation

@lucyking140

@lucyking140 lucyking140 commented Sep 20, 2025 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

Passes the "mcp-{version}" flag every time a call is made to fetch from the datacommons-client observation endpoint. This is used to identify queries that originate in the MCP server in Mixer's usage logs.

Updated tests to reflect this new parameter passed into the client.

Also updates the datacommons-client package version to 2.1.2 so that the surface header is included.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
lucyking140 added 2 commits September 20, 2025 14:53
@lucyking140
adding mcp metadata tag
cd74e20
@lucyking140
adding tag to third observation call
763eafa
@lucyking140
lucyking140 had a problem deploying to evals-and-secrets September 20, 2025 19:00 — with GitHub Actions Failure
@lucyking140 lucyking140 changed the title Lucysking/mcp metadata Adding metadata_source tag to calls to Python client library for usage logs Sep 20, 2025
@lucyking140
adding version to mcp flag and updating tests
80a5a78
@lucyking140
lucyking140 had a problem deploying to evals-and-secrets September 22, 2025 19:31 — with GitHub Actions Failure
@lucyking140
linting
1726098
@lucyking140
lucyking140 had a problem deploying to evals-and-secrets September 22, 2025 19:35 — with GitHub Actions Failure
@lucyking140
moving surface_header to be passed once into the DC client
bd68861
@lucyking140
lucyking140 had a problem deploying to evals-and-secrets September 23, 2025 19:11 — with GitHub Actions Failure
@lucyking140
formatting
3ac5fa9
@lucyking140
lucyking140 had a problem deploying to evals-and-secrets September 23, 2025 19:12 — with GitHub Actions Failure
@lucyking140
adding version to tests
b232c7b
@lucyking140
lucyking140 had a problem deploying to evals-and-secrets September 23, 2025 19:16 — with GitHub Actions Error
@lucyking140
removing typo
21797b2
@lucyking140
lucyking140 had a problem deploying to evals-and-secrets September 28, 2025 18:04 — with GitHub Actions Error
@lucyking140
lucyking140 marked this pull request as ready for review October 14, 2025 14:17
keyurva
keyurva approved these changes Oct 14, 2025
View reviewed changes

@keyurva keyurva left a comment

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Lucy!

If possible, can you add the header to the search-indicators call as well in a follow up PR?

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
lucyking140 reacted with thumbs up emoji
All reactions
  • 👍 1 reaction
@lucyking140
lucyking140 had a problem deploying to evals-and-secrets October 15, 2025 18:04 — with GitHub Actions Failure
@lucyking140
Merge branch 'main' of https://github.com/datacommonsorg/agent-toolkit …
a620250 
…into lucysking/mcp-metadata
@lucyking140
lucyking140 had a problem deploying to evals-and-secrets October 15, 2025 18:06 — with GitHub Actions Failure
lucyking140 added 3 commits October 16, 2025 18:08
@lucyking140
updating params
e572809
@lucyking140
Merge branch 'main' of https://github.com/datacommonsorg/agent-toolkit …
d5cb108 
…into lucysking/mcp-metadata
@lucyking140
adding surface header to all client initializations
421646f
@lucyking140
lucyking140 had a problem deploying to evals-and-secrets October 17, 2025 16:57 — with GitHub Actions Failure
@lucyking140
correcting mock with new header
038d991
@lucyking140
lucyking140 had a problem deploying to evals-and-secrets October 17, 2025 17:01 — with GitHub Actions Failure
@lucyking140
adding helper to remove 'rc' from version
9f46b18
@lucyking140
lucyking140 had a problem deploying to evals-and-secrets October 17, 2025 17:58 — with GitHub Actions Failure
@lucyking140
reverting notebook changes
22e065e
@lucyking140
lucyking140 temporarily deployed to evals-and-secrets October 17, 2025 17:59 — with GitHub Actions Inactive
lucyking140 added 2 commits October 17, 2025 14:04
@lucyking140
formatting
57140c6
@lucyking140
formatting
c8d25c6
@lucyking140
lucyking140 had a problem deploying to evals-and-secrets October 17, 2025 18:06 — with GitHub Actions Failure
@lucyking140
lucyking140 temporarily deployed to evals-and-secrets October 17, 2025 18:13 — with GitHub Actions Inactive
@lucyking140
lucyking140 merged commit eeae1b6 into datacommonsorg:main Oct 17, 2025
@lucyking140 lucyking140 mentioned this pull request Oct 17, 2025
Adding surface metadata to search_indicators endpoint #108
Merged

This branch was previously deployed

1 inactive deployment
evals-and-secrets — c8d25c69 Deployed Oct 17, 2025 by lucyking140
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@keyurva keyurva keyurva approved these changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@lucyking140 @keyurva
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.
You are visitor 01402520 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.