release: 0.5.0#13
Conversation
|
Firetiger deploy monitoring skipped This PR didn't match the auto-monitor filter configured on your GitHub connection:
Reason: This is an automated release PR with only internal bootstrap script changes, not a modification to kernel API endpoints or Temporal workflows. To monitor this PR anyway, reply with |
|
🧪 Testing To try out this version of the SDK: Expires at: Fri, 29 May 2026 19:16:21 GMT |
d3308ac to
f31252b
Compare
| } | ||
| } | ||
| options.defaultHeaders = { ...parsed, ...options.defaultHeaders }; | ||
| } |
There was a problem hiding this comment.
Object spread breaks non-Record defaultHeaders types
Medium Severity
When HYPEMAN_CUSTOM_HEADERS is set, the new code merges env headers into options.defaultHeaders via object spread. However, HeadersLike accepts a Headers instance, an array of header tuples, or a branded NullableHeaders — none of which spread into a plain object correctly. A user-supplied Headers instance is silently dropped (no enumerable own properties); arrays produce numeric-key objects; NullableHeaders exposes its internal values/nulls/brand fields. Downstream buildHeaders/iterateHeaders then treats the result as a record and corrupts the headers.
Reviewed by Cursor Bugbot for commit f31252b. Configure here.
dc844ea to
34cdd30
Compare
34cdd30 to
1e1dd24
Compare
| echo "==> Running prettier --write" | ||
| # format things eslint didn't | ||
| ./node_modules/.bin/prettier --write --cache --cache-strategy metadata . '!**/dist' '!**/*.ts' '!**/*.mts' '!**/*.cts' '!**/*.js' '!**/*.mjs' '!**/*.cjs' | ||
| ./node_modules/.bin/prettier --write --cache --cache-strategy metadata . |
There was a problem hiding this comment.
Prettier now processes dist directory without ignore rule
Medium Severity
The old prettier invocations included '!**/dist' to exclude built output. Now that eslint-plugin-prettier is removed and prettier runs independently, the exclusion patterns were dropped entirely. There is no .prettierignore file in the repository, and prettier only ignores node_modules by default. This means prettier --check . in scripts/lint will fail if dist/ contains files not matching prettier's style, and prettier --write . in scripts/format will unnecessarily reformat build artifacts.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit 1e1dd24. Configure here.
1e1dd24 to
8796aae
Compare
8796aae to
2f48827
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
There are 3 total unresolved issues (including 2 from previous reviews).
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 2f48827. Configure here.
| const parsed: Record<string, string> = {}; | ||
| for (const line of customHeadersEnv.split('\n')) { | ||
| const colon = line.indexOf(':'); | ||
| if (colon >= 0) { |
There was a problem hiding this comment.
Empty header name allowed when line starts with colon
Low Severity
The HYPEMAN_CUSTOM_HEADERS parser uses colon >= 0 which accepts lines starting with :, producing a header with an empty-string name (""). Using colon > 0 would correctly skip such malformed lines, since HTTP header names cannot be empty.
Reviewed by Cursor Bugbot for commit 2f48827. Configure here.


Automated Release PR
0.5.0 (2026-05-01)
Full Changelog: v0.4.0...v0.5.0
Features
Chores
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
Medium Risk
Medium risk: adds env-driven default header injection in the client, which can change outgoing request behavior for users who set
HYPEMAN_CUSTOM_HEADERS; remaining changes are mostly release/versioning and tooling updates.Overview
Bumps the SDK to
0.5.0(manifest/package/version constant) and updates the changelog and OpenAPI stats reference.Adds support for setting default HTTP headers via
HYPEMAN_CUSTOM_HEADERS, merging parsedHeader: valuelines intodefaultHeadersat client construction.Refactors formatting/lint tooling to run Prettier separately from ESLint (drops
eslint-plugin-prettier, updatesscripts/format/scripts/fast-format, and runsprettier --checkinscripts/lint), and improves build postprocessing to rewrite collapsed/** @ts-ignore */in generated.d.tsfiles into standalone// @ts-ignorelines.Reviewed by Cursor Bugbot for commit 2f48827. Bugbot is set up for automated code reviews on this repo. Configure here.