fix(ts-sdk): honor options.priority and options.scope in EventClient.bulkPush

[sushaan-k]

Hi! Picking up #3931.

EventClient.bulkPush(type, inputs, options) silently dropped options.priority and options.scope. The per-event mapping only read those fields from each input, and never fell back to the shared options argument. additionalMetadata already falls back to options.additionalMetadata, and EventClient.push reads all three fields directly from options, so the bulk path was the odd one out.

Fix mirrors the additionalMetadata pattern:

priority: input.priority ?? options.priority,
scope: input.scope ?? options.scope,

Per-event overrides still win, and the shared options apply to every event in the batch that doesn't set its own.

Testing

Added a unit test that covers both paths: an event with no per-event override picks up the shared option, an event with a per-event override keeps its own.

cd sdks/typescript
pnpm exec jest src/clients/event/event-client.test.ts   # 6/6 pass
pnpm lint:check                                          # clean
pnpm exec tsc                                            # clean

Closes #3931

[vercel]

@sushaan-k is attempting to deploy a commit to the Hatchet Team on Vercel.

A member of the Team first needs to authorize it.

[Copilot]

Pull request overview

This PR fixes an inconsistency in the TypeScript SDK’s EventClient.bulkPush by ensuring options.priority and options.scope are applied as batch-wide defaults for events that don’t specify per-event overrides (matching push behavior and how additionalMetadata already works).

Changes:

• Apply options.priority / options.scope as fallbacks in bulkPush’s per-event request mapping (input.* ?? options.*).
• Add a unit test validating both the shared-default behavior and per-event override precedence.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
sdks/typescript/src/clients/event/event-client.ts	Ensures bulk-pushed events inherit priority/scope from shared options when not set per event.
sdks/typescript/src/clients/event/event-client.test.ts	Adds coverage confirming defaults are applied and per-event values take precedence.

[BloggerBust]

This fix looks good! It makes bulkPush consistent with push for priority and scope, while still allowing event values to override the shared options. The added test covers the important shared-default and event-override cases, and I also ran the tests and typechecker locally. Thanks!

[BloggerBust]

@sushaan-k one release housekeeping request before merge: could you bump the TypeScript SDK package version in sdks/typescript/package.json to 1.23.0 and add a changelog entry for the bulkPush priority/scope fix in sdks/typescript/CHANGELOG.md?

PR #3933 can add its own separate changelog entry under the same version.