refactor: extract shared NumberOrRange type

[Shinigami92]

related to #1443

Preview:

• https://deploy-preview-3939.fakerjs.dev/api/string.html#alpha

[netlify]

✅ Deploy Preview for fakerjs ready!

Name	Link
🔨 Latest commit	92a370d
🔍 Latest deploy log	https://app.netlify.com/projects/fakerjs/deploys/6a4e77edc408ba00087da0ec
😎 Deploy Preview	https://deploy-preview-3939.fakerjs.dev
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.
🤖 Make changes	Run an agent on this branch

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.84%. Comparing base (c2cd91d) to head (92a370d).

Additional details and impacted files

@@           Coverage Diff           @@
##             next    #3939   +/-   ##
=======================================
  Coverage   98.84%   98.84%           
=======================================
  Files         923      923           
  Lines        3216     3216           
  Branches      583      566   -17     
=======================================
  Hits         3179     3179           
  Misses         33       33           
  Partials        4        4           

Files with missing lines	Coverage Δ
src/modules/airline/module.ts	100.00% <ø> (ø)
src/modules/date/module.ts	100.00% <ø> (ø)
src/modules/helpers/module.ts	94.84% <100.00%> (ø)
src/modules/lorem/module.ts	100.00% <100.00%> (ø)
src/modules/string/module.ts	100.00% <100.00%> (ø)
src/modules/system/module.ts	100.00% <ø> (ø)
src/modules/word/filter-word-list-by-length.ts	100.00% <ø> (ø)
src/modules/word/module.ts	96.00% <ø> (ø)

🚀 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.

[ST-DDT]

I added a preview link to the description.
If possible/relevant, please add a docs link for easy verification yourself.

[ST-DDT]

Note: this is no longer detected as option like if used as standalone parametern.

https://fakerjs.dev/api/string.html#sample

https://deploy-preview-3939.fakerjs.dev/api/string.html#sample

Should we remove jsdoc @param length.min/max in that case?

[Shinigami92]

Note: this is no longer detected as option like if used as standalone parametern.

fakerjs.dev/api/string.html#sample

[deploy-preview-3939.fakerjs.dev/api/string.html#sample](https://deploy-preview-3939.fakerjs.dev/api/string.html#sample) Should we remove jsdoc `@param length.min/max` in that case?

I not really sure yet, while working on the doc script implementation, I was already thinking why there is only newUnionType and newSimpleType, and I felt like I miss somehow a newObjectType 🤔 And maybe this here is the thing uncovered it 🤔

[ST-DDT]

If you want to restore the previous behavior, you may need to look at this:

faker/scripts/apidocs/processing/parameter.ts

Line 173 in 58394eb

if (!isOptionsLikeType(type)) {

Though I'm not sure the parameter is an object type in the first place.

Stepping away from the previous behavior for this case, may be an option as well.

[Shinigami92]

If you want to restore the previous behavior, you may need to look at this:

faker/scripts/apidocs/processing/parameter.ts

Line 173 in 58394eb

if (!isOptionsLikeType(type)) {

Though I'm not sure the parameter is an object type in the first place.

Stepping away from the previous behavior for this case, may be an option as well.

Thanks for the context, I won't have time today to work on that, earliest tomorrow evening.