Skip to content

Remove toolkits#784

Merged
EricGustin merged 3 commits intomainfrom
ericgustin/remove-toolkit-refs
Feb 26, 2026
Merged

Remove toolkits#784
EricGustin merged 3 commits intomainfrom
ericgustin/remove-toolkit-refs

Conversation

@EricGustin
Copy link
Member

@EricGustin EricGustin commented Feb 26, 2026

These are being moved to monorepo. This repository already has plenty of examples at examples/mcp_servers/

Sister PR: https://github.com/ArcadeAI/monorepo/pull/449


Note

Medium Risk
Primarily repo/CI/build wiring changes with large deletions; risk is mainly broken workflows or missing packaging/build steps due to removed toolkit-specific logic.

Overview
Removes the toolkits/ packages (code, tests, evals, and per-toolkit configs) as they’re being moved to a separate monorepo.

Simplifies automation accordingly: the setup-uv-env composite action no longer has toolkit/contrib/lib modes or working-directory handling and always runs a single uv sync, release-on-version-change stops passing toolkit flags and no longer skips toolkit tests, and the root Makefile drops toolkit install/check/test/build/dist/clean targets plus collapses GHCR publishing to a single flow.

Written by Cursor Bugbot for commit ca9e4aa. This will update automatically on new commits. Configure here.

@codecov
Copy link

codecov bot commented Feb 26, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@EricGustin EricGustin marked this pull request as ready for review February 26, 2026 06:38
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix prepared a fix for the issue found in the latest run.

  • ✅ Fixed: docker-base target is now identical to docker
    • Removed the redundant docker-base target which was identical to the docker target and not referenced anywhere else in the codebase.

Create PR

Or push these changes by commenting:

@cursor push d973df12c2
Preview (d973df12c2)
diff --git a/Makefile b/Makefile
--- a/Makefile
+++ b/Makefile
@@ -83,14 +83,6 @@
 	@cd docker && make docker-build
 	@cd docker && make docker-run
 
-.PHONY: docker-base
-docker-base: ## Build and run the Docker container
-	@echo "🚀 Building lib packages..."
-	@make full-dist
-	@echo "🚀 Building Docker image"
-	@cd docker && make docker-build
-	@cd docker && make docker-run
-
 .PHONY: publish-ghcr
 publish-ghcr: ## Publish to the GHCR
 	@cd docker && make publish-ghcr
This Bugbot Autofix run was free. To enable autofix for future PRs, go to the Cursor dashboard.

@EricGustin
Copy link
Member Author

@cursor push d973df1

@EricGustin EricGustin requested review from byrro and jottakka February 26, 2026 07:25
@jottakka
Copy link
Contributor

I like this one!

@EricGustin EricGustin merged commit 830480d into main Feb 26, 2026
36 of 37 checks passed
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.

4 participants