Skip to content

fix(cli-tools): Value required for --partition in stream subscribe#3263

Merged
teogeb merged 2 commits intomainfrom
cli-tools-partition-flag-in-subcribe
Dec 8, 2025
Merged

fix(cli-tools): Value required for --partition in stream subscribe#3263
teogeb merged 2 commits intomainfrom
cli-tools-partition-flag-in-subcribe

Conversation

@teogeb
Copy link
Copy Markdown
Contributor

@teogeb teogeb commented Dec 8, 2025

Users must provide a partition value when using the --partition flag. If the value is omitted, the command now shows a descriptive error message.

There are no changes to the optionality of the flag: it is still ok to leave it out, in which case the command defaults to partition 0, just like before this PR.

Error

Before this PR we got this error like this when using the flag without a value:

Error: invalid streamPartition value: true
    at ensureValidStreamPartitionIndex (.../network/packages/utils/dist/src/partition.js:10:15)

Now we show standard commander error:

error: option '-p, --partition <partition>' argument missing

@cursor
Copy link
Copy Markdown

cursor Bot commented Dec 8, 2025

You have run out of free Bugbot PR reviews for this billing cycle. This will reset on December 28.

To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@github-actions github-actions Bot added the cli-tools Related to CLI Tools Package label Dec 8, 2025
@teogeb teogeb changed the title feat(cli-tools): Value required for --partition in stream subscribe fix(cli-tools): Value required for --partition in stream subscribe Dec 8, 2025
@github-actions github-actions Bot added the docs label Dec 8, 2025
@teogeb teogeb requested a review from juslesan December 8, 2025 14:04
@teogeb teogeb merged commit 783144d into main Dec 8, 2025
23 checks passed
@teogeb teogeb deleted the cli-tools-partition-flag-in-subcribe branch December 8, 2025 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cli-tools Related to CLI Tools Package docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants