Skip to content

feat(core): callAsEventStream hook + migrate release to OIDC/Lerna 9#322

Merged
asadali214 merged 2 commits into
masterfrom
feat/core-event-stream-hook
Jul 9, 2026
Merged

feat(core): callAsEventStream hook + migrate release to OIDC/Lerna 9#322
asadali214 merged 2 commits into
masterfrom
feat/core-event-stream-hook

Conversation

@asadali214

Copy link
Copy Markdown
Collaborator

Splits the SSE work (#321) into its core + release-infra half so @apimatic/core can be released independently of @apimatic/sse.

What's here

  • feat(core): new RequestBuilder.callAsEventStream(factory) hook + optional streamReadTimeout constructor param (the SSE package plugs into this; core does not depend on sse).
  • Release infra: Lerna 6→9, OIDC trusted publishing in release.yml (no more NPM_TOKEN), CI Node matrix → 20/22/24, Sonar/dependency-review fixes.

Release plan

Merge this first, then dispatch the Release workflow with patch@apimatic/core 0.10.29 → 0.10.30 via OIDC. The sse package lands in a follow-up PR (stacked on this branch) for a separate minor release.

🤖 Generated with Claude Code

asadali214 and others added 2 commits July 9, 2026 10:04
Add a `callAsEventStream` hook to RequestBuilder that hands the builder
plus the client-wide stream read timeout to a caller-supplied factory
(mirroring how `paginate` delegates to a factory), and carry an optional
`streamReadTimeout` on the builder via a new constructor parameter.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
- bump lerna 6.5.1 -> 9.0.7; drop useWorkspaces (removed in v9); ignore .nx/
- release.yml: id-token:write + contents:write, Node 22, npm>=11.5.1,
  remove NPM_TOKEN/~/.npmrc auth (OIDC trusted publishing)
- main.yml + sonar scan: Node 20+/22 for Lerna 9; modernize action versions
- dependency-review: v4 + allow two dev-only transitive CVEs (js-yaml, tar)

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@sonarqubecloud

sonarqubecloud Bot commented Jul 9, 2026

Copy link
Copy Markdown

@asadali214 asadali214 self-assigned this Jul 9, 2026
@asadali214
asadali214 requested a review from hamzamahmood July 9, 2026 05:09

@hamzamahmood hamzamahmood left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Other changes look good to me. Have we verified the lerna bump to version 9 is non breaking?

Comment thread package.json
@asadali214
asadali214 merged commit f9612b6 into master Jul 9, 2026
7 checks passed
@asadali214
asadali214 deleted the feat/core-event-stream-hook branch July 9, 2026 06:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants