Skip to content

feat(tests): added jetstream/jetstreamManager context property - #342

Merged
aricart merged 1 commit into
mainfrom
watcher-prefix
Dec 16, 2025
Merged

aricart merged 1 commit into
mainfrom
watcher-prefix

Conversation

@aricart

@aricart aricart commented Oct 9, 2025 •

Copy link
Copy Markdown
Member

watcherPrefix which is used for the creation of watchers and ordered push consumers internally. It is used as the prefix for the subject that will deliver to the push consumer that services a watcher.

jetstream(nc, {watcherPrefix: "hello"}); or await jetstreamManager(nc, {watcherPrefix: "hello"});

@aricart
aricart requested review from Jarema and scottf October 9, 2025 15:42

@scottf scottf left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

…r contexts

Signed-off-by: Alberto Ricart <alberto@synadia.com>
constructor(nc: NatsConnection, opts?: JetStreamOptions) {
this.nc = nc as NatsConnectionImpl;
opts = opts || {} as JetStreamOptions;
opts.watcherPrefix = opts.watcherPrefix || this.nc.options.inboxPrefix;

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.

Slight leaky, could be in the types that provide object store buckets and key value store buckets?

@aricart
aricart merged commit 1c246c8 into main Dec 16, 2025
10 checks passed
@aricart
aricart deleted the watcher-prefix branch December 16, 2025 15:28
zeekay pushed a commit to hanzomsg/nats.js that referenced this pull request Sep 10, 2026
…r contexts (nats-io#342)

Signed-off-by: Alberto Ricart <alberto@synadia.com>
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.

3 participants