Skip to content

remove experiments toolset - #1667

Merged
SamMorrowDrums merged 2 commits into
mainfrom
tommy/remove-experiments-toolset
Dec 22, 2025
Merged

remove experiments toolset#1667
SamMorrowDrums merged 2 commits into
mainfrom
tommy/remove-experiments-toolset

Conversation

@tommaso-moro

Copy link
Copy Markdown
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)

CopilotAI 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 ownerDecember 22, 2025 10:02

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

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.

FileDescription
pkg/github/tools.goRemoves the ToolsetMetadataExperiments variable definition that was no longer in use
README.mdUpdates 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 mainDec 22, 2025
17 checks passed
@SamMorrowDrums
SamMorrowDrums deleted the tommy/remove-experiments-toolset branch December 22, 2025 10:15
Sign up for freeto 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

@tommaso-moro@SamMorrowDrums