Skip to content

Conversation

@tommaso-moro
Copy link
Contributor

Summary

I am removing the remaining code and docs referencing the experiments toolset

Why

We used to have an experiments toolset that was removed as part of the Go SDK migration, this PR removes the last few references to it.

What changed

  • Remove it from docs
  • Remove it from toolset metadata

MCP impact

  • No tool or API changes
  • Tool schema or behavior changed
  • New tool added

Prompts tested (tool changes only)

  • n/a

Security / limits

  • No security or limits impact
  • Auth / permissions considered
  • Data exposure, filtering, or token/size limits considered

Lint & tests

  • Linted locally with ./script/lint
  • Tested locally with ./script/test

Docs

  • Not needed
  • Updated (README / docs / examples)

Copilot AI review requested due to automatic review settings December 22, 2025 10:02
@tommaso-moro tommaso-moro requested a review from a team as a code owner December 22, 2025 10:02
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR completes the removal of the deprecated experiments toolset by cleaning up its last remaining references in the codebase.

  • Removes the ToolsetMetadataExperiments variable definition from the toolset metadata declarations
  • Updates Docker usage documentation to remove experiments from the example toolset list

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
pkg/github/tools.go Removes the ToolsetMetadataExperiments variable definition that was no longer in use
README.md Updates the Docker example command to remove experiments from the toolsets environment variable

The changes look good and are complete. I've verified that:

  • No remaining references to "experiments" or "ToolsetMetadataExperiments" exist in the codebase
  • The removal is clean and doesn't leave any orphaned references
  • The changes align with the stated purpose of removing the last remaining references to the deprecated toolset

@SamMorrowDrums SamMorrowDrums merged commit b5be18c into main Dec 22, 2025
17 checks passed
@SamMorrowDrums SamMorrowDrums deleted the tommy/remove-experiments-toolset branch December 22, 2025 10:15
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.

3 participants