[JarryShaw]

• Searched for similar pull requests

• Followed the coding style (make pylint, make mypy, make isort) — not run

• make test passes, and a test case covers the change — the 11 test_hip_* modules, 7 related tests/protocols modules, tests/project, and the protocols/internet leg (570 tests, OK)

• Changelog entry — N/A — centralised in docs(changelog): shared 1.5.0 changelog — long-lived, merges last (#610, #616, #617, #618, #620) #657

• fix — corrects a defect

---

Closes #1417
Refs #1288

Names reserved at HIP.ver (4, 3) and at RouteDstParameter.flags / RouteViaParameter.flags (2, 14), per #654, carried through the reader, the data model and make() (default 0). The new make() keywords are version_reserved and flags_reserved.

Rebuilt through HIP.from_data(p.info).data: ver 2f→21 before, 2f after; ROUTE_DST/ROUTE_VIA flags ffff→c000 before, ffff after.

Breaking: data.internet.hip.HIP and data.internet.hip.Flags gain a required reserved field. test_hip_unit.py built the old namespace dicts and Flags without it, so I updated it.

New tests/protocols/internet/test_hip_reserved_bits_unit.py: 4 tests, 19 subtests, all passing; all 4 tests fail with the fix reverted.

Rebase note: #1416 has not merged. When rebasing over it, remove the 3 HIP entries from KNOWN_GAPS in tests/protocols/schema/test_bitfield_coverage_unit.py.

[JarryShaw]

Verdict: GOOD TO GO at 222c01161 (Sonnet cross-review; author ran on Opus).

• Repro: on main, version 2f→21 and ROUTE_DST/ROUTE_VIA flags ffff→c000. All are byte-exact here; make()→parse is exact for version_reserved=0b101 and flags_reserved=0x1234. The new module fails 20 on main.
• RFCs (fetched): 7401 §5.1 gives the header octet as Version(4) | RES.(3) | 1. RFC 6028 Flags define only SYMMETRIC (bit 0) and MUST_FOLLOW (bit 1), so reserved(2,14) is right, MSB-first.
• Blast radius: nothing outside this PR constructs Data_HIP or Flags or hand-builds these dicts. Every other HIP-touching module passes. The test_hip_unit.py edits only add reserved; no assertion was dropped.
• Fixtures: 23 captures, 1614 frames. 49 frames differ, and only by the new reserved keys (all 0). All 49 HIP layers rebuild byte-exact.
• Leg: protocols/internet 570, OK.
• test(schema): every BitField namespace covers all of its bits #1416: with its test applied on top, the only failure is these 3 HIP KNOWN_GAPS entries going stale. Whichever merges second deletes them.
• Nit (reviewer-reported, not blocking): the new test's docstring cites RFC 6028 §5; the reviewer places the Flags definition in §4.1.

[github-actions]

Coverage: 89.43% (unit tier, Python 3.14, 222c01161, Unit Tests run success)

Package	Statements	Missed	Branches	Partial	Cover
pcapkit (top level)	104	4	20	4	93.55%
pcapkit/const	18791	968	2352	869	90.90%
pcapkit/corekit	2038	89	634	25	94.76%
pcapkit/dumpkit	234	2	84	2	98.74%
pcapkit/foundation	2637	98	928	48	94.78%
pcapkit/interface	112	7	40	5	92.11%
pcapkit/protocols	16506	219	4420	183	98.04%
pcapkit/toolkit	539	74	168	3	84.58%
pcapkit/utilities	429	4	122	4	98.55%
pcapkit/vendor	4409	2342	1006	157	43.25%

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