Skip to content

release: 0.5.0#13

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

release: 0.5.0#13
stainless-app[bot] wants to merge 7 commits intomainfrom
release-please--branches--main--changes--next--components--hypeman

Conversation

@stainless-app
Copy link
Copy Markdown
Contributor

@stainless-app stainless-app Bot commented Apr 23, 2026

Automated Release PR

0.5.0 (2026-05-01)

Full Changelog: v0.4.0...v0.5.0

Features

  • support setting headers via env (0578f47)

Chores

  • format: run eslint and prettier separately (ed57ba0)
  • internal: codegen related update (3e8db1a)
  • internal: more robust bootstrap script (b30da47)

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 parsed Header: value lines into defaultHeaders at client construction.

Refactors formatting/lint tooling to run Prettier separately from ESLint (drops eslint-plugin-prettier, updates scripts/format/scripts/fast-format, and runs prettier --check in scripts/lint), and improves build postprocessing to rewrite collapsed /** @ts-ignore */ in generated .d.ts files into standalone // @ts-ignore lines.

Reviewed by Cursor Bugbot for commit 2f48827. Bugbot is set up for automated code reviews on this repo. Configure here.

@firetiger-agent
Copy link
Copy Markdown

Firetiger deploy monitoring skipped

This PR didn't match the auto-monitor filter configured on your GitHub connection:

Any PR that changes the kernel API. Monitor changes to API endpoints (packages/api/cmd/api/) and Temporal workflows (packages/api/lib/temporal) in the kernel repo

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 @firetiger monitor this.

@stainless-app
Copy link
Copy Markdown
Contributor Author

stainless-app Bot commented Apr 23, 2026

🧪 Testing

To try out this version of the SDK:

npm install 'https://pkg.stainless.com/s/hypeman-typescript/ed57ba075db111dd62e7e5c1420f4187edffe868/dist.tar.gz'

Expires at: Fri, 29 May 2026 19:16:21 GMT
Updated at: Wed, 29 Apr 2026 19:16:21 GMT

@stainless-app stainless-app Bot force-pushed the release-please--branches--main--changes--next--components--hypeman branch from d3308ac to f31252b Compare April 28, 2026 04:35
@stainless-app stainless-app Bot changed the title release: 0.4.1 release: 0.5.0 Apr 28, 2026
Comment thread src/client.ts
}
}
options.defaultHeaders = { ...parsed, ...options.defaultHeaders };
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit f31252b. Configure here.

@stainless-app stainless-app Bot force-pushed the release-please--branches--main--changes--next--components--hypeman branch 2 times, most recently from dc844ea to 34cdd30 Compare April 29, 2026 05:45
@stainless-app stainless-app Bot force-pushed the release-please--branches--main--changes--next--components--hypeman branch from 34cdd30 to 1e1dd24 Compare April 29, 2026 19:15
Comment thread scripts/format
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 .
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 1e1dd24. Configure here.

@stainless-app stainless-app Bot force-pushed the release-please--branches--main--changes--next--components--hypeman branch from 1e1dd24 to 8796aae Compare April 30, 2026 06:14
@stainless-app stainless-app Bot force-pushed the release-please--branches--main--changes--next--components--hypeman branch from 8796aae to 2f48827 Compare May 1, 2026 04:53
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

There are 3 total unresolved issues (including 2 from previous reviews).

Fix All in Cursor

❌ 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.

Comment thread src/client.ts
const parsed: Record<string, string> = {};
for (const line of customHeadersEnv.split('\n')) {
const colon = line.indexOf(':');
if (colon >= 0) {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 2f48827. Configure here.

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