Skip to content

release: 0.35.1#88

Open
stainless-app[bot] wants to merge 27 commits intomainfrom
release-please--branches--main--changes--next--components--hyperspell
Open

release: 0.35.1#88
stainless-app[bot] wants to merge 27 commits intomainfrom
release-please--branches--main--changes--next--components--hyperspell

Conversation

@stainless-app
Copy link
Copy Markdown
Contributor

@stainless-app stainless-app bot commented Mar 20, 2026

Automated Release PR

0.35.1 (2026-04-01)

Full Changelog: v0.35.0...v0.35.1

Features

Bug Fixes

  • internal: gitignore generated oidc dir (a305f3d)

Chores

  • ci: escape input path in publish-npm workflow (40c8402)
  • ci: skip lint on metadata-only changes (14012e7)
  • internal: fix MCP server TS errors that occur with required client options (b7abcfb)
  • internal: improve local docs search for MCP servers (eee2438)
  • internal: improve local docs search for MCP servers (f218072)
  • internal: support custom-instructions-path flag in MCP servers (c6d4043)
  • internal: support local docs search in MCP servers (4a9b2cf)
  • internal: update gitignore (9270ff5)
  • internal: update multipart form array serialization (97f213b)
  • mcp-server: add support for session id, forward client info (9e3018f)
  • tests: bump steady to v0.19.4 (e614b67)
  • tests: bump steady to v0.19.5 (eadfc98)
  • tests: bump steady to v0.19.6 (4ed8bba)
  • tests: bump steady to v0.19.7 (6aaa435)
  • tests: bump steady to v0.20.1 (bb96b8b)
  • tests: bump steady to v0.20.2 (2a71a6e)

Refactors

  • tests: switch from prism to steady (75ac934)

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

@stainless-app
Copy link
Copy Markdown
Contributor Author

stainless-app bot commented Mar 20, 2026

🧪 Testing

To try out this version of the SDK:

npm install 'https://pkg.stainless.com/s/hyperspell-typescript/2a71a6e88d24933ce432279a88062fa849454586/dist.tar.gz'

Expires at: Fri, 01 May 2026 09:00:09 GMT
Updated at: Wed, 01 Apr 2026 09:00:09 GMT

@canaries-inc
Copy link
Copy Markdown

canaries-inc bot commented Mar 20, 2026

Canary encountered an internal error while analyzing this PR.

@stainless-app stainless-app bot force-pushed the release-please--branches--main--changes--next--components--hyperspell branch from 6972295 to d19a5b9 Compare March 21, 2026 05:08
@entelligence-ai-pr-reviews
Copy link
Copy Markdown



Confidence Score: 5/5 - Safe to Merge

  • No new review comments were generated, indicating the PR appears clean to the automated review.
  • No critical, significant, or high-risk issues were identified in the heuristic analysis.
  • 8 out of 11 changed files were reviewed with zero issues flagged across all severity levels.
  • No existing unresolved comments remain, suggesting previous concerns have been addressed.

@stainless-app stainless-app bot force-pushed the release-please--branches--main--changes--next--components--hyperspell branch from d19a5b9 to 76a7033 Compare March 21, 2026 05:14
@entelligence-ai-pr-reviews
Copy link
Copy Markdown



Confidence Score: 5/5 - Safe to Merge

  • No new review comments were generated, indicating the PR appears clean from an automated review perspective.
  • No critical, significant, or high-risk issues were identified in the heuristic analysis.
  • 8 out of 11 changed files were reviewed with no issues found, providing reasonable coverage confidence.
  • No existing unresolved comments that could indicate lingering concerns.

@stainless-app stainless-app bot force-pushed the release-please--branches--main--changes--next--components--hyperspell branch from 76a7033 to abd5f3a Compare March 21, 2026 21:30
@entelligence-ai-pr-reviews
Copy link
Copy Markdown



Confidence Score: 5/5 - Safe to Merge

  • No new review comments were generated, indicating the PR appears clean from an automated review perspective.
  • No critical, significant, or high-risk issues were identified in the heuristic analysis.
  • 15 out of 18 changed files were reviewed, providing good coverage with no issues found.
  • No existing unresolved comments that would block merging.

@stainless-app stainless-app bot force-pushed the release-please--branches--main--changes--next--components--hyperspell branch from abd5f3a to 651d17f Compare March 22, 2026 08:09
@entelligence-ai-pr-reviews
Copy link
Copy Markdown



Confidence Score: 5/5 - Safe to Merge

  • No critical or significant issues found
  • 15/18 changed files reviewed (83% coverage)

@stainless-app stainless-app bot force-pushed the release-please--branches--main--changes--next--components--hyperspell branch from 651d17f to fc292db Compare March 23, 2026 08:03
@entelligence-ai-pr-reviews
Copy link
Copy Markdown



Confidence Score: 5/5 - Safe to Merge

  • No critical or significant issues found
  • 15/18 changed files reviewed (83% coverage)

@stainless-app stainless-app bot force-pushed the release-please--branches--main--changes--next--components--hyperspell branch from fc292db to 08cc52a Compare March 23, 2026 22:30
@entelligence-ai-pr-reviews
Copy link
Copy Markdown



Confidence Score: 5/5 - Safe to Merge

  • No new review comments were generated, indicating the PR is clean from an automated review perspective.
  • No critical, significant, or high-risk issues were identified in the heuristic analysis.
  • 15 out of 18 changed files were reviewed with zero issues flagged, providing good coverage confidence.
  • No existing unresolved comments carry over from previous reviews that would block merging.

@stainless-app
Copy link
Copy Markdown
Contributor Author

stainless-app bot commented Mar 24, 2026

Release version edited manually

The Pull Request version has been manually set to 0.35.1 and will be used for the release.

If you instead want to use the version number 0.36.0 generated from conventional commits, just remove the label autorelease: custom version from this Pull Request.

@stainless-app stainless-app bot force-pushed the release-please--branches--main--changes--next--components--hyperspell branch from 08cc52a to 15abf4d Compare March 24, 2026 05:10
@entelligence-ai-pr-reviews
Copy link
Copy Markdown



Confidence Score: 5/5 - Safe to Merge

  • No critical or significant issues found
  • 15/19 changed files reviewed (79% coverage)

@stainless-app stainless-app bot force-pushed the release-please--branches--main--changes--next--components--hyperspell branch from 15abf4d to 2dbef30 Compare March 24, 2026 05:14
@entelligence-ai-pr-reviews
Copy link
Copy Markdown



Confidence Score: 5/5 - Safe to Merge

  • No critical or significant issues found
  • 16/20 changed files reviewed (80% coverage)

@stainless-app stainless-app bot force-pushed the release-please--branches--main--changes--next--components--hyperspell branch from 2dbef30 to b86942f Compare March 24, 2026 05:16
@entelligence-ai-pr-reviews
Copy link
Copy Markdown



Confidence Score: 5/5 - Safe to Merge

  • No critical or significant issues found
  • 16/20 changed files reviewed (80% coverage)

@stainless-app stainless-app bot force-pushed the release-please--branches--main--changes--next--components--hyperspell branch from b86942f to dd246c5 Compare March 24, 2026 17:30
@entelligence-ai-pr-reviews
Copy link
Copy Markdown



Confidence Score: 5/5 - Safe to Merge

  • No critical or significant issues found
  • 16/20 changed files reviewed (80% coverage)

@stainless-app stainless-app bot force-pushed the release-please--branches--main--changes--next--components--hyperspell branch 2 times, most recently from 0b83ccb to d4e4451 Compare March 25, 2026 03:29
@entelligence-ai-pr-reviews
Copy link
Copy Markdown



Confidence Score: 5/5 - Safe to Merge

  • No new review comments were generated, indicating the PR is clean with no identified issues.
  • No existing unresolved comments to consider, meaning there are no lingering concerns from previous reviews.
  • 16 out of 21 changed files were reviewed with no issues found, providing good coverage confidence.
  • Heuristic analysis shows zero critical, significant, high-risk, or medium issues detected.

@entelligence-ai-pr-reviews
Copy link
Copy Markdown



Confidence Score: 5/5 - Safe to Merge

  • No new review comments were generated, indicating the PR appears clean from an automated review perspective.
  • No critical, significant, or high-risk issues were identified in the heuristic analysis.
  • Coverage of 16/21 changed files is reasonable, and no unresolved existing comments are blocking the merge.
  • The absence of any flagged issues across all severity levels supports a safe-to-merge assessment.

@stainless-app stainless-app bot force-pushed the release-please--branches--main--changes--next--components--hyperspell branch from d4e4451 to a7b2b11 Compare March 26, 2026 17:30
@stainless-app stainless-app bot force-pushed the release-please--branches--main--changes--next--components--hyperspell branch from eb78e16 to 54b8168 Compare March 29, 2026 11:10
@entelligence-ai-pr-reviews
Copy link
Copy Markdown


Confidence Score: 3/5 - Review Recommended

Likely safe but review recommended — this patch release adds Gmail Actions support, a memory search effort parameter, and offline MCP docs search via MiniSearch, all of which appear well-structured, but a pre-existing unresolved concern about packages/mcp-server/src/instructions.ts remains open: the static import fs from 'fs/promises' at the top of that file will cause a hard failure at module load time in non-Node runtimes such as Deno or Cloudflare Workers, even when the file-reading code path is never reached. This is not introduced by the current PR, but it has not been resolved either, and given that MCP servers are commonly deployed in edge/worker environments, the risk is real enough to warrant attention before shipping.

Key Findings:

  • The static top-level import fs from 'fs/promises' in packages/mcp-server/src/instructions.ts will throw at module initialization in non-Node runtimes (Deno, Cloudflare Workers), making the entire MCP server non-functional in those environments — this pre-existing comment remains unresolved.
  • The additions to union types across actions.ts, auth.ts, connections.ts, integrations.ts, web-crawler.ts, memories.ts, and shared.ts for 'gmail_actions' appear consistent and mechanical, reducing the risk of type-level regressions.
  • The new optional effort parameter on MemorySearchParams is accompanied by test coverage, which is a positive signal for correctness of that feature.
  • 23 of 30 changed files were reviewed, meaning roughly 7 files had no automated review coverage — for a release PR this is an acceptable but non-zero residual risk.
Files requiring special attention
  • packages/mcp-server/src/instructions.ts

@stainless-app stainless-app bot force-pushed the release-please--branches--main--changes--next--components--hyperspell branch from 54b8168 to 655717f Compare March 30, 2026 23:30
Comment on lines 179 to 185
...(codeTool !== undefined && { includeCodeTool: codeTool }),
...(docsTools !== undefined && { includeDocsTools: docsTools }),
codeExecutionMode: defaultOptions.codeExecutionMode,
docsSearchMode: defaultOptions.docsSearchMode,
docsDir: defaultOptions.docsDir,
};
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Correctness: The returned McpOptions object is missing several properties from defaultOptions that are defined in the McpOptions type: stainlessApiKey, codeAllowHttpGets, codeAllowedMethods, codeBlockedMethods, and customInstructionsPath. These will be silently dropped when parseQueryOptions is called, potentially causing features like auth and method filtering to stop working for HTTP connections.

🤖 AI Agent Prompt for Cursor/Windsurf

📋 Copy this prompt to your AI coding assistant (Cursor, Windsurf, etc.) to get help fixing this issue

In packages/mcp-server/src/options.ts, the `parseQueryOptions` function (lines 179-185) returns a `McpOptions` object that is missing several properties from `defaultOptions`: `stainlessApiKey`, `codeAllowHttpGets`, `codeAllowedMethods`, `codeBlockedMethods`, and `customInstructionsPath`. These are defined in the `McpOptions` type but are not forwarded through, so they will be silently dropped. Add these properties to the returned object: `stainlessApiKey: defaultOptions.stainlessApiKey`, `codeAllowHttpGets: defaultOptions.codeAllowHttpGets`, `codeAllowedMethods: defaultOptions.codeAllowedMethods`, `codeBlockedMethods: defaultOptions.codeBlockedMethods`, `customInstructionsPath: defaultOptions.customInstructionsPath`.

@entelligence-ai-pr-reviews
Copy link
Copy Markdown


Confidence Score: 3/5 - Review Recommended

Likely safe but review recommended — this patch release introduces useful extensions to Gmail Actions integration and MCP server capabilities, but has two substantive correctness issues that should be addressed. In packages/mcp-server/src/options.ts, the parseQueryOptions function silently drops several McpOptions properties including stainlessApiKey, codeAllowHttpGets, codeAllowedMethods, codeBlockedMethods, and customInstructionsPath from the returned object, which could cause runtime behavior mismatches when callers depend on those values. A pre-existing concern about the static import fs from 'fs/promises' in packages/mcp-server/src/instructions.ts also remains unresolved, which will cause module-load failures in non-Node runtimes even when the file-reading code path is never invoked.

Key Findings:

  • In parseQueryOptions (options.ts), the reconstructed McpOptions return value omits stainlessApiKey, codeAllowHttpGets, codeAllowedMethods, codeBlockedMethods, and customInstructionsPath — fields that are part of the type definition and populated from defaultOptions. This is a silent data-loss bug that could cause downstream callers to use undefined values where configured options were expected.
  • The static top-level import fs from 'fs/promises' in instructions.ts is a pre-existing unresolved issue: it will throw at module load time in non-Node environments (Cloudflare Workers, Deno) regardless of whether the customInstructionsPath feature is used, limiting the portability of the MCP server package.
  • The Gmail Actions literal additions across actions.ts, auth.ts, connections.ts, integrations.ts, web-crawler.ts, memories.ts, and shared.ts appear to be consistent union type extensions with low risk, and the new effort parameter on MemorySearchParams is additive and optional.
Files requiring special attention
  • packages/mcp-server/src/options.ts
  • packages/mcp-server/src/instructions.ts

@stainless-app stainless-app bot force-pushed the release-please--branches--main--changes--next--components--hyperspell branch from 655717f to 873125d Compare March 31, 2026 04:10
@stainless-app stainless-app bot force-pushed the release-please--branches--main--changes--next--components--hyperspell branch from 873125d to f0c2e07 Compare March 31, 2026 04:11
@entelligence-ai-pr-reviews
Copy link
Copy Markdown


Confidence Score: 3/5 - Review Recommended

Likely safe but review recommended — this PR delivers a solid patch release expanding Gmail Actions support and MCP server enhancements, but two unresolved correctness concerns from a prior review remain open. Specifically, the static import fs from 'fs/promises' in instructions.ts will cause module load failures in non-Node runtimes (e.g., Cloudflare Workers, Deno) even when the file-reading path is never invoked, and the McpOptions object returned in options.ts is missing several properties (stainlessApiKey, codeAllowHttpGets, codeAllowedMethods, etc.) that are defined in the McpOptions type, which could lead to subtle runtime failures or type unsafety downstream. These were flagged in a previous review and remain unaddressed.

Key Findings:

  • In packages/mcp-server/src/instructions.ts, the top-level static import fs from 'fs/promises' will throw at module load time in any non-Node environment, making the entire MCP server package unusable in edge/serverless runtimes without a conditional dynamic import or a runtime guard.
  • In packages/mcp-server/src/options.ts (L179-L185), the returned McpOptions object omits required/expected fields such as stainlessApiKey, codeAllowHttpGets, and codeAllowedMethods that exist on the McpOptions type, meaning callers relying on those fields will silently receive undefined rather than the defaults, potentially causing incorrect behavior.
  • The core SDK changes (adding 'gmail_actions' to union types across actions.ts, auth.ts, connections.ts, integrations.ts, etc., and adding the effort parameter to MemorySearchParams) are additive, non-breaking, and appear well-scoped with corresponding test coverage.
  • The migration from Prism to Steady for mock server tooling and the CI/CD hardening are infrastructure-level changes that don't affect runtime correctness of the SDK itself, but the two unresolved MCP server correctness issues are sufficient reason to recommend a review before merging.
Files requiring special attention
  • packages/mcp-server/src/instructions.ts
  • packages/mcp-server/src/options.ts

@entelligence-ai-pr-reviews
Copy link
Copy Markdown


Confidence Score: 3/5 - Review Recommended

Likely safe but review recommended — this patch release introduces valuable API surface expansions (Gmail Actions support, MCP server enhancements) and appears clean in the new review pass, but two pre-existing unresolved correctness issues remain open that were not addressed before merge. Specifically, the static import fs from 'fs/promises' in packages/mcp-server/src/instructions.ts will cause hard failures at module load time in non-Node runtimes (Deno, Cloudflare Workers), and the McpOptions construction in packages/mcp-server/src/options.ts is missing fields like stainlessApiKey, codeAllowHttpGets, and codeAllowedMethods which could cause type inconsistencies or silent runtime failures downstream.

Key Findings:

  • The static import fs from 'fs/promises' in instructions.ts is a runtime correctness issue: in non-Node environments, this import fails at module load time regardless of whether the file-reading code path is ever reached, making the entire MCP server package unusable in those environments.
  • In options.ts, the returned McpOptions object omits several properties (stainlessApiKey, codeAllowHttpGets, codeAllowedMethods) that are part of the McpOptions type contract — consumers relying on these fields will get undefined silently, which is a correctness/API contract violation.
  • The new-review pass found no additional issues across the 24/31 reviewed changed files, which is a positive signal that the bulk of the additions (Gmail Actions literals across actions.ts, auth.ts, connections.ts, etc., and the effort parameter in MemorySearchParams) appear structurally sound.
  • The two unresolved comments are categorized as correctness issues, not style nits — they represent real runtime failure modes that should be addressed before shipping to consumers depending on this package in non-Node environments.
Files requiring special attention
  • packages/mcp-server/src/instructions.ts
  • packages/mcp-server/src/options.ts

@stainless-app stainless-app bot force-pushed the release-please--branches--main--changes--next--components--hyperspell branch from f0c2e07 to f6d4f44 Compare March 31, 2026 18:31
@entelligence-ai-pr-reviews
Copy link
Copy Markdown


Confidence Score: 3/5 - Review Recommended

Likely safe but review recommended — this patch release adds Gmail Actions support, MCP server enhancements, and mock server migration, but two pre-existing unresolved correctness issues in the MCP server package remain open. Specifically, the static import fs from 'fs/promises' in instructions.ts will cause module-load failures in non-Node runtimes (Deno, Cloudflare Workers), and options.ts returns an McpOptions object missing several type-required properties (stainlessApiKey, codeAllowHttpGets, codeAllowedMethods, etc.), which could cause runtime errors when those fields are accessed. These issues were not introduced by this PR but remain unresolved and affect the MCP server package that is part of this release.

Key Findings:

  • In packages/mcp-server/src/instructions.ts, the top-level import fs from 'fs/promises' is unconditional and will throw at module load time in any non-Node.js runtime environment, making the MCP server non-functional in edge or Deno deployments regardless of which code paths are executed.
  • In packages/mcp-server/src/options.ts, the McpOptions object returned by the options-building function is missing several declared type properties (stainlessApiKey, codeAllowHttpGets, codeAllowedMethods), which represents a type-safety gap that could cause undefined access bugs downstream when callers rely on those fields being present.
  • The new LocalDocsSearch class in local-docs-search.ts using MiniSearch appears to be a new code path that was not covered in the heuristic review (only 24/31 changed files were reviewed), leaving some uncertainty about its correctness.
  • The core type union additions for 'gmail_actions' across actions.ts, auth.ts, connections.ts, integrations.ts, and related files appear straightforward and low-risk as additive changes to discriminated unions.
Files requiring special attention
  • packages/mcp-server/src/instructions.ts
  • packages/mcp-server/src/options.ts
  • packages/mcp-server/src/local-docs-search.ts

@stainless-app stainless-app bot force-pushed the release-please--branches--main--changes--next--components--hyperspell branch from f6d4f44 to 1c7f58a Compare April 1, 2026 08:55
@stainless-app stainless-app bot force-pushed the release-please--branches--main--changes--next--components--hyperspell branch 2 times, most recently from abfdae3 to 5a4091a Compare April 1, 2026 08:58
@stainless-app stainless-app bot force-pushed the release-please--branches--main--changes--next--components--hyperspell branch from 5a4091a to 0820f08 Compare April 1, 2026 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants