Skip to content

closes #9: Add smithery configuration#18

Merged
danixeee merged 4 commits intomainfrom
9-configure-smithery
Sep 29, 2025
Merged

closes #9: Add smithery configuration#18
danixeee merged 4 commits intomainfrom
9-configure-smithery

Conversation

@danixeee
Copy link
Copy Markdown
Contributor

@danixeee danixeee commented Sep 26, 2025

Followed the docs from here: https://smithery.ai/docs/build/deployments/typescript#setup

Seems like smithery.ai has a bug with build, so let's wait until it is resolved so I can deploy and test it.

I have installed @smithery/sdk dev dependency and now the build is working fine. The server is deployed and ready (still not public): https://smithery.ai/server/@plainly-videos/mcp-server

@danixeee danixeee requested a review from ivansenic September 26, 2025 17:15
@danixeee danixeee marked this pull request as ready for review September 26, 2025 17:15
Copy link
Copy Markdown
Contributor

@ivansenic ivansenic left a comment

Choose a reason for hiding this comment

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

Check comments pls..

Comment thread README.md
"plainly": {
"command": "node",
"args": ["<FULL PATH TO dist/index.js>"],
"args": ["<FULL PATH TO dist/cli.js>"],
Copy link
Copy Markdown
Contributor

@ivansenic ivansenic Sep 29, 2025

Choose a reason for hiding this comment

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

needs updating help center once this is published.. do we really needed this? it's a breaking change imo, so if we keep the change, let's release 1.1.0 pls

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

what do you mean? this is for local dev only, npx command is still the same

Base automatically changed from improve-api-errors to main September 29, 2025 14:18
@danixeee danixeee force-pushed the 9-configure-smithery branch from a2fd53a to 9627c86 Compare September 29, 2025 14:21
@danixeee danixeee merged commit 7b724b8 into main Sep 29, 2025
1 check passed
@danixeee danixeee deleted the 9-configure-smithery branch September 29, 2025 14:54
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