-
Notifications
You must be signed in to change notification settings - Fork 41
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
base: main
Are you sure you want to change the base?
Conversation
* 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) ...
There was a problem hiding this 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'sapiClientId
. Consider renaming toapiClientId
for consistency across documentation and config.
atlasClientId:
.smithery/smithery.yaml:27
- The configSchema omits
disabledTools
andtelemetry
, which are documented in the README. Add these properties to the schema so users can configure them via Smithery.
exampleConfig:
COPY ../package.json ../package-lock.json ../tsconfig.json ../tsconfig.build.json ./ | ||
|
||
# Copy source code | ||
COPY ../src ./src |
There was a problem hiding this comment.
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 ./
).
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>
No description provided.