Open
Conversation
…mpatibility" This reverts commit 32dd153.
This commit introduces several improvements to the Plone MCP server: - **Refactor Vitest mocking for `@modelcontextprotocol/sdk/server/mcp.js`:** - Implemented `vi.hoisted` to correctly mock `McpServer` and `ResourceTemplate` classes, resolving hoisting-related `ReferenceError` issues in unit tests. - This ensures that mock functions are properly initialized and accessible within test blocks, making the unit tests more robust and reliable. - **Fix `ENABLED_TOOLS` environment variable handling:** - Corrected a bug in `src/index.ts` where setting `ENABLED_TOOLS` to an empty string (`""`) would incorrectly register all tools instead of only `plone_configure`. - The logic now correctly interprets an empty `ENABLED_TOOLS` string as a directive to enable only the `plone_configure` tool, aligning with expected behavior. - **Update test setup:** - Removed `jest.setTimeout` from `tests/setup.ts` as it's not relevant for Vitest. - Ensured `vi.clearAllMocks()` is used for proper mock state management in `beforeEach` hooks. - Added `"types": ["vitest/globals"]` to `tsconfig.json` for better TypeScript integration with Vitest's global APIs. These changes enhance the stability and correctness of the MCP server's tool registration mechanism and improve the reliability of its unit tests.
…ocumentation Updated specs/todo.md to accurately reflect Plone REST API endpoint protocols for navigation, version history, and search by location. Removed unsupported parameters and clarified implementation details based on local plone.restapi documentation. Removed 'expand_all_items' from PloneGetNavigationTreeSchema in src/handlers.ts as it is not a supported parameter for the Plone REST API navigation endpoint and was not being used.
This commit introduces a new script, 'scripts/fix-type-imports.js', which automatically
converts TypeScript type import syntax from '{ type SomeType }' to 'type { SomeType }'.
This change aligns with recommended TypeScript practices and improves code consistency.
The script has been applied to all relevant source files in the 'src/' directory.
Move tests
Eea types
Updated to the latest xmcp framework version which introduces changes to: - CLI commands: 'xmcp-dev' → 'dev', 'build-xmcp' → 'build' - Build output structure and transport handling - Development server startup patterns Build artifacts (.xmcp/) changes: - Removed adapter files (adapter-express.js, adapter-nextjs.js) - no longer using Express/Next.js integration mode - Removed stdio.js - now using HTTP transport only - Updated http.js with latest framework code - Cleaned up associated LICENSE.txt files XMCP.md documentation updates: - Updated CLI commands to match new xmcp syntax - Added project structure overview - Added Scripts section explaining available commands - Added Connecting to Your Server section with Cursor/Claude configs - Added Troubleshooting section with xmcp.config.ts examples - Updated tool/prompt examples with async functions and shortcut returns The project now runs as a standalone HTTP MCP server without adapter dependencies.
- Rename specs/ folder to artifacts/ - Move XMCP.md to artifacts/ folder - Remove xmcp-llms-full.txt file
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.