fix(internet): exclude reserved port 0 from internet.port()

[xianjianlf2]

Closes #3937

faker.internet.port() calls this.faker.number.int(65535), whose range is inclusive of 0. Port 0 is listed as Reserved by IANA (RFC 6335 §6) and is never a usable service port, so seeded values fed into a real socket (net.connect({ port: 0 })) trigger OS-assigned ephemeral behavior instead of the seeded value.

This restricts the range to 1..65535:

-    return this.faker.number.int(65535);
+    return this.faker.number.int({ min: 1, max: 65535 });

• Tightened the existing test assertion (toBeGreaterThanOrEqual(0) → 1).
• Regenerated the single affected seeded snapshot (internet > 42 > port); no other snapshots changed.
• pnpm exec vitest run test/modules/internet.spec.ts → 1125 passed, no type errors.

[netlify]

✅ Deploy Preview for fakerjs ready!

Built without sensitive environment variables

Name	Link
🔨 Latest commit	b4873d5
🔍 Latest deploy log	https://app.netlify.com/projects/fakerjs/deploys/6a548ec82117e1000810be00
😎 Deploy Preview	https://deploy-preview-3948.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.

[xDivisionByZerox]

As by comment #3937 (comment) we are currently not sure how to handle this. Since port 0 is not the only reserved port, flat out excluding it without all other reserved ones could lead to more problems than simply returning it for now.

[codecov]

Codecov Report

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

Additional details and impacted files

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

Files with missing lines	Coverage Δ
src/modules/internet/module.ts	100.00% <100.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]

@xianjianlf2 Can you comment your thoughts on the linked issues?

[xianjianlf2]

Replied on the linked issue with my thoughts on bug-vs-feature classification and the reserved-port scope: #3937 (comment)

Short version: I've retracted the inaccurate "must not appear on the wire" wording (updated both descriptions) — the authoritative basis is simply that IANA lists port 0 as Reserved. 0 is a special case (never a usable service port), not a slippery slope toward 1023/1024, so I'd keep this PR as the minimal, non-breaking min: 1 fix and treat configurable ranges/flags as a separate feature. Happy to fold it all into a broader feature PR instead if that's the preferred direction.