Skip to content

feat: add smithery config #263

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 22 commits into
base: main
Choose a base branch
from
Open

feat: add smithery config #263

wants to merge 22 commits into from

Conversation

nirinchev
Copy link
Collaborator

No description provided.

* main: (40 commits)
  chore: add more details for some api errors (#219)
  fix: use ejson parsing for stdio messages (#218)
  docs: improve getting started experience (#217)
  feat: support flex clusters to atlas tools (#182)
  chore: enforce access list (#214)
  feat: add back the connect tool (#210)
  Update connection string app name if not present (#199)
  chore: update docs with more Service Accounts mentions (#209)
  chore(deps-dev): bump eslint-plugin-prettier from 5.2.6 to 5.4.0 (#205)
  chore(deps-dev): bump @types/node from 22.15.3 to 22.15.9 (#204)
  chore(deps-dev): bump typescript-eslint from 8.31.1 to 8.32.0 (#206)
  chore(deps-dev): bump eslint from 9.25.1 to 9.26.0 (#207)
  chore: add recommended extensions and settings (#200)
  fix: fork checks (#194)
  docs: correct the link for VSCode's MCP usage (#186)
  chore: switch to a matrix for forks (#191)
  chore: skip Atlas Tests and don't track coverage for fork contributions (#188)
  fix: db user test error (#187)
  fix: improve api error messages (#176)
  chore: update quickstart with mcpServers (#185)
  ...
* main: (21 commits)
  fix: updates count tool (#254)
  fix: docker security warnings (#259)
  feat: docker support (#238)
  docs: list alerts docs (#250)
  chore: add hints and update mcp (#249)
  chore: base model SEO change (#248)
  chore(ci): add a PR title check workflow (#247)
  docs: bump node.js version (#246)
  chore: corrects the description of atlas-create-db-user (#240)
  chore: auto-close stale issues (#237)
  chore(deps-dev): bump globals from 16.0.0 to 16.1.0 (#231)
  chore(deps-dev): bump @types/node from 22.15.9 to 22.15.17 (#233)
  chore(deps-dev): bump eslint-config-prettier from 10.1.2 to 10.1.5 (#234)
  feat: Alerts Listing (#230)
  chore(deps-dev): bump @redocly/cli from 1.34.2 to 1.34.3 (#235)
  chore(deps-dev): bump openapi-typescript from 7.6.1 to 7.8.0 (#232)
  chore: release v0.1.1 (#223)
  fix: improve uncaught exception for getAccessToken (#224)
  chore: update issue template (#227)
  chore: switch to `@mongodb-js/device-id` (#196)
  ...
@nirinchev nirinchev marked this pull request as ready for review June 9, 2025 09:18
@Copilot Copilot AI review requested due to automatic review settings June 9, 2025 09:18
@nirinchev nirinchev requested a review from a team as a code owner June 9, 2025 09:18
@nirinchev nirinchev requested a review from gagik June 9, 2025 09:19
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Adds Smithery.ai configuration and related tooling files to enable standardized builds and containerization.

  • Updates README with clearer config option descriptions
  • Introduces .smithery/smithery.yaml for Smithery.ai defaults and CLI command generation
  • Provides a Docker multi-stage build (.smithery/Dockerfile) and .dockerignore

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
README.md Refined configuration table with updated descriptions and formatting
.smithery/smithery.yaml Defined Smithery build config, schema properties, example config, and startCommand function
.smithery/Dockerfile Generated multi-stage Dockerfile for building and running the MCP
.dockerignore Added standard ignore rules for dist, node_modules, and other artifacts
Comments suppressed due to low confidence (2)

.smithery/smithery.yaml:10

  • The property name atlasClientId differs from the README's apiClientId. Consider renaming to apiClientId for consistency across documentation and config.
atlasClientId:

.smithery/smithery.yaml:27

  • The configSchema omits disabledTools and telemetry, which are documented in the README. Add these properties to the schema so users can configure them via Smithery.
exampleConfig:

Comment on lines +7 to +10
COPY ../package.json ../package-lock.json ../tsconfig.json ../tsconfig.build.json ./

# Copy source code
COPY ../src ./src
Copy link
Preview

Copilot AI Jun 9, 2025

Choose a reason for hiding this comment

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

Referencing ../ in COPY may fall outside the Docker build context and fail. Use paths relative to the context (e.g., COPY package.json package-lock.json tsconfig.json tsconfig.build.json ./).

Suggested change
COPY ../package.json ../package-lock.json ../tsconfig.json ../tsconfig.build.json ./
# Copy source code
COPY ../src ./src
COPY package.json package-lock.json tsconfig.json tsconfig.build.json ./
# Copy source code
COPY src ./src

Copilot uses AI. Check for mistakes.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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