-
Notifications
You must be signed in to change notification settings - Fork 0
release: 0.10.1 #32
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?
release: 0.10.1 #32
Conversation
|
🧪 Testing To try out this version of the SDK, run: Expires at: Sun, 08 Feb 2026 05:54:08 GMT |
3d1972d to
852d01e
Compare
Release version edited manuallyThe Pull Request version has been manually set to If you instead want to use the version number |
9986f8a to
b493336
Compare
1488508 to
ee90f42
Compare
b55ea91 to
806118a
Compare
806118a to
be50c7d
Compare
… validation issues
be50c7d to
d088711
Compare
d088711 to
4702c78
Compare
4702c78 to
bb838ab
Compare
bb838ab to
e91729d
Compare
e91729d to
6939dde
Compare
6939dde to
f0a44fc
Compare
f0a44fc to
9bde25d
Compare
|
Note: this release PR has been open for more than 60 days. Until this PR is merged, we will temporarily stop pushing codegen updates due to changes to Stainless' codegen. |
7 similar comments
|
Note: this release PR has been open for more than 60 days. Until this PR is merged, we will temporarily stop pushing codegen updates due to changes to Stainless' codegen. |
|
Note: this release PR has been open for more than 60 days. Until this PR is merged, we will temporarily stop pushing codegen updates due to changes to Stainless' codegen. |
|
Note: this release PR has been open for more than 60 days. Until this PR is merged, we will temporarily stop pushing codegen updates due to changes to Stainless' codegen. |
|
Note: this release PR has been open for more than 60 days. Until this PR is merged, we will temporarily stop pushing codegen updates due to changes to Stainless' codegen. |
|
Note: this release PR has been open for more than 60 days. Until this PR is merged, we will temporarily stop pushing codegen updates due to changes to Stainless' codegen. |
|
Note: this release PR has been open for more than 60 days. Until this PR is merged, we will temporarily stop pushing codegen updates due to changes to Stainless' codegen. |
|
Note: this release PR has been open for more than 60 days. Until this PR is merged, we will temporarily stop pushing codegen updates due to changes to Stainless' codegen. |
fd06ac9 to
ab89b55
Compare
|
Note: this release PR has been open for more than 60 days. Until this PR is merged, we will temporarily stop pushing codegen updates due to changes to Stainless' codegen. |
4 similar comments
|
Note: this release PR has been open for more than 60 days. Until this PR is merged, we will temporarily stop pushing codegen updates due to changes to Stainless' codegen. |
|
Note: this release PR has been open for more than 60 days. Until this PR is merged, we will temporarily stop pushing codegen updates due to changes to Stainless' codegen. |
|
Note: this release PR has been open for more than 60 days. Until this PR is merged, we will temporarily stop pushing codegen updates due to changes to Stainless' codegen. |
|
Note: this release PR has been open for more than 60 days. Until this PR is merged, we will temporarily stop pushing codegen updates due to changes to Stainless' codegen. |
ab89b55 to
5b1a0ae
Compare
|
Note: this release PR has been open for more than 60 days. Until this PR is merged, we will temporarily stop pushing codegen updates due to changes to Stainless' codegen. |
5b1a0ae to
6b22f05
Compare
|
Note: this release PR has been open for more than 60 days. Until this PR is merged, we will temporarily stop pushing codegen updates due to changes to Stainless' codegen. |
6b22f05 to
ff01518
Compare
|
Note: this release PR has been open for more than 60 days. Until this PR is merged, we will temporarily stop pushing codegen updates due to changes to Stainless' codegen. |
ff01518 to
d633f38
Compare
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.
This PR is being reviewed by Cursor Bugbot
Details
Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
| jsonrpc: '2.0', | ||
| error: { | ||
| code: -32000, | ||
| message: 'Unauthorized', |
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.
HTTP transport ignores mcpOptions configuration
Medium Severity
The newServer function only destructures clientOptions, req, and res from its parameter object, but the post function passes an object containing both clientOptions and mcpOptions (via spread operator on line 48). Since mcpOptions is not destructured in newServer, it's never passed to initMcpServer. This means user configuration like includeDocsTools: false passed through streamableHTTPApp will be silently ignored, and the server will always use default tool selection behavior.
Additional Locations (1)
| { | ||
| "dxt_version": "0.2", | ||
| "name": "@nbai/sdk-mcp", | ||
| "version": "0.10.0", |
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.
Manifest version doesn't match package version
Low Severity
The manifest.json file specifies version 0.10.0 while package.json specifies version 0.10.1. This version mismatch could cause confusion or issues when the MCP bundle is distributed, as the manifest metadata won't reflect the actual package version being released.
| switch (options.transport) { | ||
| case 'stdio': | ||
| await launchStdioServer(options, selectedTools); | ||
| await launchStdioServer(); |
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.
Stdio transport ignores CLI mcpOptions completely
Medium Severity
The launchStdioServer function is called without passing the parsed CLI options, so it takes no parameters. Inside launchStdioServer, initMcpServer({ server }) is called without mcpOptions. This means user CLI options like --no-tools=docs are parsed and logged correctly, but the actual server initialization ignores them and uses defaults. The log message shows the correct filtered tools, but the running server exposes all tools.
Additional Locations (1)
| 'formats', | ||
| 'tool-name-length', | ||
| ] as const; | ||
|
|
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.
CLI accepts code tool options that have no effect
Low Severity
The CLI options --tools and --no-tools accept 'code' as a valid choice, but the code only processes the 'docs' option via shouldIncludeToolType('docs'). The McpOptions type only has includeDocsTools, and selectTools unconditionally adds codeTool() on line 114. Users passing --no-tools=code would see the option accepted without error, but the code tool would still be included.
Additional Locations (1)
| gh release upload ${{ github.event.release.tag_name }} \ | ||
| packages/mcp-server/nbai_sdk_api.mcpb | ||
| env: | ||
| GH_TOKEN: ${{ secrets.GITHUB_TOKEN }} |
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.
Missing condition causes workflow failure on manual dispatch
Medium Severity
The "Upload MCP Server DXT GitHub release asset" step runs unconditionally but uses github.event.release.tag_name which is only populated during release events. When the workflow is triggered via workflow_dispatch (manual trigger), this value will be empty, causing the gh release upload command to fail with an error about missing tag name. The step needs an if: github.event_name == 'release' condition.
| } | ||
|
|
||
| const { is_error, result, log_lines, err_lines } = (await res.json()) as WorkerOutput; | ||
| const hasLogs = log_lines.length > 0 || err_lines.length > 0; |
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.
Missing null check on API response arrays
Low Severity
After destructuring the API response, log_lines.length and err_lines.length are accessed directly on line 79. If the remote API returns null or undefined for these fields instead of empty arrays, this will throw a TypeError: Cannot read property 'length' of null. The code assumes the WorkerOutput type is strictly enforced by the API, but no defensive checks exist.
|
Note: this release PR has been open for more than 60 days. Until this PR is merged, we will temporarily stop pushing codegen updates due to changes to Stainless' codegen. |
1 similar comment
|
Note: this release PR has been open for more than 60 days. Until this PR is merged, we will temporarily stop pushing codegen updates due to changes to Stainless' codegen. |
Automated Release PR
0.10.1 (2026-01-09)
Full Changelog: v0.10.0...v0.10.1
Features
streamableHTTPApp(5d124ad)Bug Fixes
Performance Improvements
Chores
compilerOptions.baseUrlfrom tsconfig.json (f4f38c1)docs_searchtool at other URLs (8ee83d5)This pull request is managed by Stainless's GitHub App.
The semver version number is based on included commit messages. Alternatively, you can manually set the version number in the title of this pull request.
For a better experience, it is recommended to use either rebase-merge or squash-merge when merging this pull request.
🔗 Stainless website
📚 Read the docs
🙋 Reach out for help or questions
Note
Release 0.10.1
executecode-mode tool andsearch_docstool; default tool selection revamped; support logging level changes; parse query-string options; expose client options instreamableHTTPApp; support unix socket and header-based auth; README adds install links..mcpb) with manifest and scripts; upload bundle in CI and GitHub Releases; make packages public viapublishConfig..eslintcache,*.mcpb.Written by Cursor Bugbot for commit d633f38. This will update automatically on new commits. Configure here.