Skip to content

Conversation

@devin-ai-integration
Copy link
Contributor

Fix smithery.yaml: Add missing runtime: typescript declaration

Summary

Fixes Smithery deployment failure by adding the required runtime: typescript field to smithery.yaml. The previous config only had name: websets which caused validation errors during deployment.

Root cause: Smithery's config schema requires either runtime: typescript or runtime: python to be explicitly declared. The websets-mcp-server was missing this field while exa-mcp-server had it correctly configured.

Changes:

  • Changed smithery.yaml from name: websets to runtime: typescript to match exa-mcp-server's working configuration

Validation:

  • Tested locally with npm install which runs the prepare script → build:stdio completed successfully
  • Build output generated at .smithery/stdio/index.cjs (595.34 KB)

Review & Testing Checklist for Human

  • Deploy to Smithery and verify deployment succeeds without validation errors
  • Confirm the MCP server still works correctly after deployment (test a basic webset operation)

Notes

  • This change aligns websets-mcp-server's config with the working exa-mcp-server configuration
  • The error message was very specific: Invalid literal value, expected "typescript" which directly led to this fix

Link to Devin run: https://app.devin.ai/sessions/a6b83525c3384b38b2b822b32b11ae7c
Requested by: Tanishq Jaiswal (tanishq@exa.ai) / @10ishq

The smithery.yaml was missing the required 'runtime: typescript' field,
causing deployment failures on Smithery with invalid_literal errors.
This field is required by the Smithery config schema and is present
in the working exa-mcp-server configuration.

Fixes the deployment error:
- Invalid literal value, expected "typescript"
- Invalid input for startCommand

Tested locally with npm install and build:stdio - builds successfully.

Co-Authored-By: Tanishq Jaiswal <tanishq.jaiswal97@gmail.com>
@devin-ai-integration
Copy link
Contributor Author

Original prompt from Tanishq
there are two repos for MCP one for apis and one for websets, can you access those?

@devin-ai-integration
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR that start with 'DevinAI' or '@devin'.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@10ishq 10ishq merged commit 311a3fb into master Oct 22, 2025
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.

2 participants