Skip to content

Conversation

@CodeWithKyrian
Copy link
Contributor

This PR removes dead code from the old Chain architecture that was part of the merged libraries, along with obsolete examples.

What was removed:

  • ToolChain, ResourceChain, PromptChain classes
  • CollectionInterface, IdentifierInterface, MetadataInterface for Tool/Resource/Prompt
  • ToolCollectionInterface (unused)
  • Entire examples/09-standalone-cli/ directory (obsolete)
  • Entire examples/10-simple-http-transport/ directory (duplicate of example 02)

Why:
The codebase now uses the new Registry architecture with ReferenceProviderInterface, ReferenceRegistryInterface, and ReferenceHandlerInterface. The old Chain classes and their supporting interfaces are no longer used anywhere in the codebase.

What changed:

  • Fixed STDERR handling in bootstrap.php to prevent errors in web environments (SAPI)
  • Cleaned up CHANGELOG.md to be ready for future entries

- Remove ToolChain, ResourceChain, PromptChain classes
- Remove Collection/Identifier/Metadata interfaces (only used by old Chains)
- Remove ToolCollectionInterface (unused)
- Delete obsolete examples 09-standalone-cli and 10-simple-http-transport
- Fix STDERR handling in bootstrap.php for web environments
- Clean up CHANGELOG.md
Copy link
Member

@chr-hertel chr-hertel left a comment

Choose a reason for hiding this comment

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

Could be three PRs, but makes sense :D
Thanks!

@chr-hertel chr-hertel merged commit 9e16883 into modelcontextprotocol:main Sep 21, 2025
9 checks passed
@chr-hertel chr-hertel changed the title Remove dead code from old Chain architecture and obsolete examples [Server] Remove dead code from old Chain architecture and obsolete examples Sep 21, 2025
@chr-hertel chr-hertel added the Server Issues & PRs related to the Server component label Sep 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Server Issues & PRs related to the Server component

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants