Skip to content

chore(main): release 8.11.1#1542

Merged
nicknisi merged 2 commits intomainfrom
release-please--branches--main--components--node
Apr 5, 2026
Merged

chore(main): release 8.11.1#1542
nicknisi merged 2 commits intomainfrom
release-please--branches--main--components--node

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions github-actions bot commented Apr 4, 2026

🤖 I have created a release beep boop

8.11.1 (2026-04-04)

Bug Fixes

  • replace node:events with eventemitter3 in runtime client (#1540) (e293be1)

This PR was generated with Release Please. See documentation.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Apr 4, 2026

Greptile Summary

This is an automated Release Please PR bumping @workos-inc/node from v8.11.0 to v8.11.1. It packages the bug fix from #1540 which replaced the Node.js built-in node:events EventEmitter with eventemitter3 inside FeatureFlagsRuntimeClient, enabling compatibility with non-Node.js runtimes such as Cloudflare Workers and edge environments.

  • package.json: Version bumped to 8.11.1; eventemitter3 v5.0.4 added as a runtime dependency (previously only the worker entry-point was affected by missing node:events support)
  • package-lock.json: Lock file updated with eventemitter3 v5.0.4 integrity hash
  • CHANGELOG.md: Release entry added in conventional changelog format
  • .release-please-manifest.json: Manifest version updated to 8.11.1

The underlying code change itself (in src/feature-flags/runtime-client.ts) correctly overrides emit to restore Node.js error-throwing semantics for unhandled 'error' events, since eventemitter3 silently drops them — this is a good defensive measure.

Confidence Score: 5/5

This PR is safe to merge — it is a clean automated release PR with no custom logic changes.

All four changed files are standard Release Please artifacts (manifest, changelog, package.json version bump, lock file). The actual runtime code change was landed in the prior commit (e293be1) and this PR simply publishes it as a patch release. No logic issues, security concerns, or custom-rule violations were found.

No files require special attention.

Important Files Changed

Filename Overview
.release-please-manifest.json Version bumped to 8.11.1 — standard release manifest update
CHANGELOG.md New v8.11.1 section added documenting the eventemitter3 bug fix
package.json Version bumped to 8.11.1; eventemitter3 v5.0.4 correctly added as a runtime dependency
package-lock.json eventemitter3 v5.0.4 lock entry added with resolved URL and integrity hash

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[FeatureFlagsRuntimeClient] -->|extends| B[EventEmitter - eventemitter3]
    B -. previously used .-> C[EventEmitter - node:events]
    A -->|override emit| D{event === 'error' AND no listeners?}
    D -->|yes| E[throw Error — restore Node.js behavior]
    D -->|no| F[super.emit — eventemitter3 default]
Loading

Reviews (1): Last reviewed commit: "chore(main): release 8.11.1" | Re-trigger Greptile

@nicknisi nicknisi merged commit 6ff6197 into main Apr 5, 2026
8 checks passed
@nicknisi nicknisi deleted the release-please--branches--main--components--node branch April 5, 2026 01:10
@github-actions
Copy link
Copy Markdown
Contributor Author

github-actions bot commented Apr 5, 2026

🤖 Created releases:

🌻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

1 participant