🗑️ Deprecate `Random::next(n)` and `Random::nextInt()` by dubzzz · Pull Request #6687 · dubzzz/fast-check

dubzzz · 2026-03-06T22:05:24Z

Mark Random::next(n) and Random::nextInt() (no-argument overload only) as @deprecated following the bump to pure-rand v8.

Related to #6679.

Generated with Claude Code

Co-authored-by: Nicolas DUBIEN <[email protected]> Co-Authored-By: Claude Sonnet 4.6 <[email protected]>

changeset-bot · 2026-03-06T22:05:30Z

🦋 Changeset detected

Latest commit: 9231112

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
fast-check	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

github-actions · 2026-03-06T22:05:47Z

👋 It looks like the PR template was not used for this pull request.

Please edit the PR description to use the template, or close this PR and open a new one that uses it.
Not using the template may result in your change not being reviewed.

The template includes:

A Description section
A Checklist section with items to verify before submitting

Thank you for your contribution! 🙏

dubzzz · 2026-03-06T22:06:08Z

  /**
   * Generate an integer having `bits` random bits
   * @param bits - Number of bits to generate
+   * @deprecated


@claude add details on the alternative, suggest calling nextInt instead with the right parameters

Claude finished @dubzzz's task in 49s —— View job

Tasks

Read Random.ts to understand current deprecation messages

Update @deprecated tag for next(bits) at line 39 with alternative suggestion

Update @deprecated tag for nextInt() (no-arg) at line 55 with alternative suggestion

Commit and push changes

Updated the @deprecated messages in packages/fast-check/src/random/generator/Random.ts:

next(bits): @deprecated Prefer \nextInt(0, (1 << bits) - 1)` instead`

nextInt() (no-arg): @deprecated Prefer \nextInt(0x80000000 | 0, 0x7fffffff | 0)` for a full 32-bit signed integer range, or `nextInt(min, max)` with explicit bounds`

dubzzz · 2026-03-06T22:06:21Z


  /**
   * Generate a random integer (32 bits)
+   * @deprecated


@claude add details on the alternative, suggest calling nextInt with the right parameters

Claude finished @dubzzz's task in 1m 24s —— View job

Tasks

Gather context

Read Random.ts to understand the deprecated methods

Update deprecation notices with alternative nextInt calls

Commit and push changes

Summary

Updated the @deprecated JSDoc tags in packages/fast-check/src/random/generator/Random.ts to include explicit alternatives:

next(bits) (line 39): @deprecated Prefer {@link nextInt} with explicit bounds: \nextInt(0, (1 << bits) - 1)``

nextInt() no-arg overload (line 55): @deprecated Prefer {@link nextInt} with explicit bounds: \nextInt(-2147483648, 2147483647)``

Using {@link nextInt} creates a clickable link in generated API docs, and the code examples show users exactly what parameters to pass.
|

dubzzz

Review

Update @fast-check/worker to use pure-rand v8, mirroring the upgrade done in fast-check. Updates import paths from the v7-style `pure-rand/generator/XorShift` to the v8-style `pure-rand/generator/xorshift128plus`. Co-authored-by: Nicolas DUBIEN <[email protected]> Co-authored-by: Claude Sonnet 4.6 <[email protected]>

Co-authored-by: Nicolas DUBIEN <[email protected]> Co-Authored-By: Claude Sonnet 4.6 <[email protected]>

dubzzz · 2026-03-06T22:08:24Z

@claude make sure that claude code will only open PRs following our template for PRs and use the emoji naming convention (check our rules on copilot)

pkg-pr-new · 2026-03-06T22:08:28Z

@fast-check/examples

@fast-check/ava

npm i https://pkg.pr.new/@fast-check/ava@6687

fast-check

npm i https://pkg.pr.new/fast-check@6687

@fast-check/jest

npm i https://pkg.pr.new/@fast-check/jest@6687

@fast-check/packaged

npm i https://pkg.pr.new/@fast-check/packaged@6687

@fast-check/poisoning

npm i https://pkg.pr.new/@fast-check/poisoning@6687

@fast-check/vitest

npm i https://pkg.pr.new/@fast-check/vitest@6687

@fast-check/worker

npm i https://pkg.pr.new/@fast-check/worker@6687

commit: 84bd527

Co-authored-by: Nicolas DUBIEN <[email protected]> Co-Authored-By: Claude Sonnet 4.6 <[email protected]>

claude · 2026-03-06T22:09:31Z

Claude finished @dubzzz's task in 1m 0s —— View job

Tasks

Gather context
Read CLAUDE.md and copilot rules
Find PR template
Create CLAUDE.md with PR template and emoji naming instructions
Commit and push changes

Summary

Created CLAUDE.md at the repository root with instructions for Claude Code to follow when opening PRs:

PR title naming convention — gitmoji format per .github/copilot-instructions.md:
- emoji Description for fast-check package changes
- emoji(package-name) Description for other packages
- 50-character max on the description part
PR body — always use the template from .github/PULL_REQUEST_TEMPLATE.md, keeping all sections (Description + Checklist) intact

Claude Code automatically reads CLAUDE.md on every invocation, so these rules will be applied when it opens future PRs.
|

Instructs Claude Code to follow the gitmoji naming convention and the PR template when opening pull requests. Co-authored-by: Nicolas DUBIEN <[email protected]>

dubzzz · 2026-03-06T22:11:49Z

@claude create a dedicated PR for the worker related changes, drop them from this PR

dubzzz · 2026-03-06T22:12:03Z

@claude create a dedicated PR for the pr guidelines related changes, drop them from this PR

claude · 2026-03-06T22:12:03Z

Claude finished @dubzzz's task in 1m 13s —— View job

Tasks

Gather context
Identify worker-related changes in the current PR
Create a new branch for worker changes
Cherry-pick worker commit and push new branch
Create a new PR for worker changes
Revert worker changes from current PR branch

Summary

Done! Here's what was done:

New PR created: #6689 — ⚡️(worker) Bump pure-rand to v8 — contains the worker-related changes cherry-picked from this branch.
This PR — the worker commit (⚡️(worker) Bump pure-rand to v8) has been reverted. The remaining changes are now only:
- CLAUDE.md — PR naming and template guidelines
- packages/fast-check/src/random/generator/Random.ts — deprecation of Random::next(n) and Random::nextInt()
  · New worker PR #6689

claude · 2026-03-06T22:12:16Z

Claude finished @dubzzz's task in 1m 20s —— View job

Tasks

Gather context
Read CLAUDE.md content
Create new branch for CLAUDE.md changes
Commit and push CLAUDE.md to new branch
Create dedicated PR for CLAUDE.md
Remove CLAUDE.md from current PR branch
Push changes to current branch

Summary

Created #6690 with the CLAUDE.md PR guidelines changes
Removed CLAUDE.md from this PR

This PR now only contains the deprecation changes to packages/fast-check/src/random/generator/Random.ts.
| New PR #6690

This reverts commit da69699.

Co-authored-by: Nicolas DUBIEN <[email protected]> Co-Authored-By: Claude Sonnet 4.6 <[email protected]>

Deprecate the Random::next(n) and Random::nextInt() methods.

codecov · 2026-03-06T22:24:39Z

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.54%. Comparing base (7238a83) to head (9231112).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #6687   +/-   ##
=======================================
  Coverage   94.54%   94.54%           
=======================================
  Files         210      210           
  Lines        5685     5685           
  Branches     1500     1500           
=======================================
  Hits         5375     5375           
  Misses        297      297           
  Partials       13       13

Flag	Coverage Δ
tests	`94.54% <ø> (ø)`

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

## Description Add a `CLAUDE.md` file at the repository root so Claude Code follows our PR naming convention (gitmoji format) and always uses the PR template when opening pull requests. Related to #6687. ## Checklist — _Don't delete this checklist and make sure you do the following before opening the PR_ - [x] I have a full understanding of every line in this PR — whether the code was hand-written, AI-generated, copied from external sources or produced by any other tool - [x] I flagged the impact of my change (minor / patch / major) either by running `pnpm run bump` or by following the instructions from the changeset bot - [x] I kept this PR focused on a single concern and did not bundle unrelated changes - [x] I followed the [gitmoji](https://gitmoji.dev/) specification for the name of the PR, including the package scope (e.g. `🐛(vitest) Something...`) when the change targets a package other than `fast-check` - [x] I added relevant tests and they would have failed without my PR (when applicable) --------- Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com> Co-authored-by: Nicolas DUBIEN <[email protected]> Co-authored-by: Claude Sonnet 4.6 <[email protected]> Co-authored-by: Nicolas DUBIEN <[email protected]> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

## Description `Random` no longer exposes the no-argument overload of `nextInt()` nor the `next(bits)` method — both deprecated since 4.6.0 (#6687) — and drops `nextBoolean()` as well, which was not used anywhere in the codebase, the documentation, or the website. **Breaking change**: any caller of these methods must switch to `nextInt(min, max)` with explicit bounds — `nextInt(-2147483648, 2147483647)` replaces `nextInt()`, `nextInt(0, (1 << bits) - 1)` replaces `next(bits)`, and `nextInt(0, 1) === 1` replaces `nextBoolean()`. Flagged as major via a changeset. Generated values are not impacted: `nextDouble` now draws its two integers directly through `uniformInt` with the exact bounds previously used by `next(26)`/`next(27)`, so random sequences stay bit-for-bit identical. The motivation is to keep a single way to draw integers from `Random` and finish the cleanup started with the deprecations of #6687. Every other method was checked for usage before deciding its fate: `clone`, `nextInt(min, max)`, `nextBigInt` and `getState` are all used by production code (arbitraries, worker package), and `nextDouble` is used by e2e tests and documented integrations (e.g. the faker randomizer pattern from the blog) — they all stay. The PR is focused on this single concern. On the tests side, specs relying on the dropped methods (`Random.spec.ts`, `Runner.spec.ts`, `func.spec.ts`, stubs and the worker property spec) were switched to `nextInt` with explicit bounds — they would fail to compile against the new signature otherwise — and the `fakeRandom()` helper no longer exposes mocks for the removed methods while now properly overriding `nextDouble`. ## Checklist — _Don't delete this checklist and make sure you do the following before opening the PR_ - [ ] I have a full understanding of every line in this PR — whether the code was hand-written, AI-generated, copied from external sources or produced by any other tool - [ ] I flagged the impact of my change (minor / patch / major) either by running `pnpm run bump` or by following the instructions from the changeset bot - [ ] I kept this PR focused on a single concern and did not bundle unrelated changes - [ ] I followed the [gitmoji](https://gitmoji.dev/) specification for the name of the PR, including the package scope (e.g. `🐛(vitest) Something...`) when the change targets a package other than `fast-check` - [ ] I added relevant tests and they would have failed without my PR (when applicable)   🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_01WL9nDkBv45rMBMvqTHQZFm --------- Co-authored-by: Claude <[email protected]>

deprecate Random::next(n) and Random::nextInt() (no-arg)

f87eb9b

Co-authored-by: Nicolas DUBIEN <[email protected]> Co-Authored-By: Claude Sonnet 4.6 <[email protected]>

dubzzz commented Mar 6, 2026

View reviewed changes

github-actions Bot and others added 2 commits March 6, 2026 22:07

improve @deprecated messages with alternative suggestions

02455e1

Co-authored-by: Nicolas DUBIEN <[email protected]> Co-Authored-By: Claude Sonnet 4.6 <[email protected]>

improve deprecation messages with alternative nextInt calls

208d538

Co-authored-by: Nicolas DUBIEN <[email protected]> Co-Authored-By: Claude Sonnet 4.6 <[email protected]>

🔧 Add CLAUDE.md with PR naming and template guidelines

97660b9

Instructs Claude Code to follow the gitmoji naming convention and the PR template when opening pull requests. Co-authored-by: Nicolas DUBIEN <[email protected]>

Revert "⚡️(worker) Bump pure-rand to v8"

53321c5

This reverts commit da69699.

claude Bot mentioned this pull request Mar 6, 2026

🔧 Refine GH Action triggering CLAUDE #6690

Merged

5 tasks

github-actions Bot and others added 2 commits March 6, 2026 22:13

🔥 Remove CLAUDE.md from this PR (moved to dedicated PR)

3804d16

Co-authored-by: Nicolas DUBIEN <[email protected]> Co-Authored-By: Claude Sonnet 4.6 <[email protected]>

Deprecate Random::next and Random::nextInt methods

9231112

Deprecate the Random::next(n) and Random::nextInt() methods.

dubzzz enabled auto-merge (squash) March 6, 2026 22:16

dubzzz merged commit c62a288 into main Mar 6, 2026
45 checks passed

dubzzz deleted the claude/pr-6679-20260306-2203 branch March 6, 2026 22:24

ghost-of-high-heart Bot mentioned this pull request Apr 3, 2026

Update fast-check to 4.6.0 wereHamster/computation#506

Merged

dubzzz mentioned this pull request Jul 24, 2026

💥 Drop methods from Random #7179

Merged

5 tasks

Uh oh!

Conversation

dubzzz commented Mar 6, 2026

Uh oh!

changeset-bot Bot commented Mar 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🦋 Changeset detected

Uh oh!

github-actions Bot commented Mar 6, 2026

Uh oh!

dubzzz Mar 6, 2026

Choose a reason for hiding this comment

Uh oh!

claude Bot Mar 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Tasks

Uh oh!

dubzzz Mar 6, 2026

Choose a reason for hiding this comment

Uh oh!

claude Bot Mar 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Tasks

Summary

Uh oh!

dubzzz left a comment

Choose a reason for hiding this comment

Uh oh!

dubzzz commented Mar 6, 2026

Uh oh!

pkg-pr-new Bot commented Mar 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

claude Bot commented Mar 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Tasks

Summary

Uh oh!

dubzzz commented Mar 6, 2026

Uh oh!

dubzzz commented Mar 6, 2026

Uh oh!

claude Bot commented Mar 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Tasks

Summary

Uh oh!

claude Bot commented Mar 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Tasks

Summary

Uh oh!

Uh oh!

codecov Bot commented Mar 6, 2026

Codecov Report

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

changeset-bot Bot commented Mar 6, 2026 •

edited

Loading

claude Bot Mar 6, 2026 •

edited

Loading

claude Bot Mar 6, 2026 •

edited

Loading

pkg-pr-new Bot commented Mar 6, 2026 •

edited

Loading

claude Bot commented Mar 6, 2026 •

edited

Loading

claude Bot commented Mar 6, 2026 •

edited

Loading

claude Bot commented Mar 6, 2026 •

edited

Loading