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	f53525b
🔍 Latest deploy log	https://app.netlify.com/projects/fakerjs/deploys/6a5e492526486300098298ad
😎 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.91%. Comparing base (974490f) to head (f53525b).

Additional details and impacted files

@@           Coverage Diff           @@
##             next    #3939   +/-   ##
=======================================
  Coverage   98.91%   98.91%           
=======================================
  Files         924      924           
  Lines        3224     3224           
  Branches      585      585           
=======================================
  Hits         3189     3189           
  Misses         31       31           
  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.