Skip to content

Conversation

@joshuatam
Copy link

Motivation and Context

Some old framework is using autoload to include with the typeName directly, e.g. include(ini.php) when the typeName is ini.

If the typeName is the arbitrary type, set autoload in enum_exists to false

How Has This Been Tested?

Yes

Breaking Changes

No

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

Additional context

None

Some old framework is using autoload to include with the typeName directly, e.g. `include(ini.php)` when the typeName is `ini`.

If the typeName is the arbitrary type, set autoload in `enum_exists` to `false`
@chr-hertel
Copy link
Member

Hey @joshuatam, thanks for reaching out and bringing the patch. Can you help me understand the issue here? What's happening without this patch?
E.g. what's the error you're running into - is there a way to reproduce that?

@chr-hertel chr-hertel added the question Further information is requested label Jan 23, 2026
@chr-hertel chr-hertel changed the title Add autoload check for enum types in ReferenceHandler [Server] Add autoload check for enum types in ReferenceHandler Jan 23, 2026
@chr-hertel chr-hertel added Server Issues & PRs related to the Server component bug Something isn't working P3 Nice to haves, rare edge cases labels Jan 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working P3 Nice to haves, rare edge cases question Further information is requested Server Issues & PRs related to the Server component

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants