Skip to content

Conversation

jirispilka
Copy link
Collaborator

@jirispilka jirispilka commented Aug 22, 2025

  • Publishing to Smithery uses a personal account due to group account functionality issues.
  • Publication happens through a repository fork at https://github.com/apify-projects/apify-mcp-server since Smithery requires repository write access, which cannot be granted to the main Apify account (https://github.com/apify/apify-mcp-server).
  • The fork requires manual synchronization to stay current, as automatic syncing is not configured.
  • As of August 22nd, Smithery supports external server publishing with the requirement of an /mcp endpoint.

close https://github.com/apify/ai-team/issues/64

@github-actions github-actions bot added the t-ai Issues owned by the AI team. label Aug 22, 2025
Copy link
Contributor

@MQ37 MQ37 left a comment

Choose a reason for hiding this comment

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

This sadly breaks the DockerHub integration 🥲

@github-actions github-actions bot added the tested Temporary label used only programatically for some analytics. label Aug 22, 2025
@jirispilka jirispilka requested a review from MQ37 August 22, 2025 20:46
@jirispilka
Copy link
Collaborator Author

Currently, deployment is failing

image

@@ -7,6 +7,7 @@
"node": ">=18.0.0"
},
"main": "dist/index.js",
"module": "./src/smithery.ts",
Copy link
Contributor

Choose a reason for hiding this comment

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

Info I get from LLM how to use main and module:

- main should point to the CommonJS build (usually in dist/).
- module should point to the ESM build (also in dist/), not to the TypeScript source.
- Both should export the same API.
- Ensure all referenced files are published.

The problem is that we have other exported APIs in the main and module. I think it will break our internal server if Typescript looks at the module and sees only Smithery, not ActorsMcpServer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
t-ai Issues owned by the AI team. tested Temporary label used only programatically for some analytics.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants