fix: replace plimit with own promises limiter implementation to plug mem leak by jerome-benoit · Pull Request #386 · tinylibs/tinybench

@jerome-benoit

+ avoid memory leak
+ trigger the GC less often

closes tinylibs#365

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

Copilot AI review requested due to automatic review settings

November 2, 2025 21:20

@jerome-benoit

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>
Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>
Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

@jerome-benoit jerome-benoit changed the title fix: replace plimit with own promises limiter implementation fix: replace plimit with own promises limiter implementation to plug mem leak

Nov 2, 2025
Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>
Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>
Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>
Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

Uzlopak

Uzlopak

Uzlopak

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>
Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>
- Standardize naming to pattern: [prefix-technique] [core-functionality] [(async/sync)]
- Convert verbose descriptions to concise technical format
- Update 32 test descriptions across 8 files for consistency
- Improve code readability and maintainability
- All tests continue to pass (104 passing, 1 skipped)

43081j

43081j

@Uzlopak

@Uzlopak

@Uzlopak

jerome-benoit

@Uzlopak

@Uzlopak

with less time it should be less flaky

@Uzlopak

@Uzlopak

@Uzlopak

jerome-benoit

jerome-benoit

jerome-benoit

jerome-benoit

jerome-benoit

jerome-benoit

@Uzlopak

@Uzlopak

@Uzlopak Uzlopak marked this pull request as ready for review

November 9, 2025 20:32