Skip to content

Publish MCP server as npm package and update dashboard middleware#3

Merged
Adhish-Krishna merged 2 commits intomasterfrom
development
Feb 26, 2026
Merged

Publish MCP server as npm package and update dashboard middleware#3
Adhish-Krishna merged 2 commits intomasterfrom
development

Conversation

@Adhish-Krishna
Copy link
Member

  • published @simplens/mcp package to npm for the users to use simplens mcp server in their local machine itself with streamable-http or stdio transports.
  • update the dashboard middleware to check for authorization header for api related requests if no session is present. Because this request may be made my the mcp server. So that the mcp server can actually access the api routes specific to the dashboard

…th stdio and streamable-http mode using the npm package
…ests for /api requests to allow requests from mcp server
@Adhish-Krishna Adhish-Krishna merged commit 1665ec3 into master Feb 26, 2026
3 checks passed
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: f75cd89568

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +10 to +12
"files": [
"dist",
"README.md"

Choose a reason for hiding this comment

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

P1 Badge Build dist before publishing npm package

The package now whitelists only dist and README.md, but there is still no prepack/prepare step to run tsc, so publishing from a clean checkout (or CI) produces a tarball without dist/index.js; in that state, both main and bin point to a missing file and npx @simplens/mcp cannot start. This is a release blocker for any publish flow that does not manually build first.

Useful? React with 👍 / 👎.

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.

1 participant