[JarryShaw]

• Searched for similar pull requests
• Followed the coding style
• make test passes, and a test case covers the change
• Changelog entry: N/A — centralised in docs(changelog): shared 1.5.0 changelog — long-lived, merges last (#610, #616, #617, #618, #620) #657

What is the purpose of your pull request?

• fix — bug fix

---

Description

Closes #2314, per the ruling (take the recommendation, with the ports on the port enum field).

• Fix: PortEnumField.pre_process raises FieldValueError for a port outside the field's width. Before, a directly built TCP or UDP schema masked srcport=70000 to 0x1170 and -1 to 0xffff. UDP, TCP and SCTP already use PortEnumField (refactor(corekit): move stream proxy classes such as _PeekableStream into corekit.io #1516, refactor(corekit): share one PortEnumField across TCP, UDP and SCTP (#1516) #1526).
• Unchanged: NumberField keeps masking, which is its documented contract (fix(corekit): a NumberField whose bit_length is narrower than its octets drops the high bits #1488). The NumberField and Schema docstrings now say that direct construction does not range-check, and name where the checks live.
• Test: test_tcp_udp_schema_port_range_unit.py rejects -1, 65536 and 70000 on both ports of both schemas, and round-trips ports 0 and 65535 byte-exact.
• Fail-without-fix: on origin/main, 12 failed (AssertionError: FieldValueError not raised). On the head, 2 passed with 20 subtests.

[JarryShaw]

Review: GOOD TO GO at ec62c5872 (Sonnet).

• Fix: TCP, UDP and SCTP all declare their ports as PortEnumField, so the range check covers all three. 65536 and -1 raise FieldValueError, while 0 and 65535 pack byte-exact.
• Unchanged: NumberField masking (fix(corekit): a NumberField whose bit_length is narrower than its octets drops the high bits #1488). Parsed ports are always in range, make() already rejected bad ports through _make_port, and AppType members still convert to their port first.
• Fail-without-fix: on origin/main, 12 failed (FieldValueError not raised). On the head, tests/corekit, tests/protocols/transport and the contract tests gave 1110 passed.
• Non-blocking:
  • SCTP is covered only through the shared field, with no SCTP subtest.
  • PortEnumField.pre_process gained a Raises: entry but no versionchanged note.

[JarryShaw]

Rebased ec62c5872 → 841014d29 onto main bfe184bd6. The patch-id is identical, so the GOOD TO GO verdict carries over to the new head.

The old head's Project tests failed on test_no_test_calls_unparse_outside_a_version_gate. That was not this PR. It came from main's #2306 scan test, which called ast.unparse outside a version gate; bfe184bd6 fixes it on main, and CI is re-running on the new head.

[github-actions]

Coverage: 91.56% (unit tier, Python 3.15, 841014d29, Unit Tests run success)

Package	Statements	Missed	Branches	Partial	Cover
pcapkit (top level)	104	4	20	2	95.16%
pcapkit/const	19599	881	2618	867	92.11%
pcapkit/corekit	2591	74	818	37	96.51%
pcapkit/dumpkit	282	3	102	3	98.44%
pcapkit/foundation	3023	23	1044	25	98.82%
pcapkit/interface	138	4	52	3	96.32%
pcapkit/protocols	17437	218	4586	185	98.15%
pcapkit/toolkit	869	36	298	10	95.20%
pcapkit/utilities	504	7	146	5	98.15%
pcapkit/vendor	4432	1988	1018	167	51.87%

Per-file detail: the coverage-html artifact of this run.