[JarryShaw]

• Searched for similar pull requests
• Followed the coding style (make pylint, make mypy, make isort)
• make test passes, and a test case covers the change
• Changelog entry: N/A — centralised in chore(docs): centralise changelog entries for unreleased versions #2483

What is the purpose of your pull request?

• refactor — changes neither behaviour nor performance

---

Description of your pull request and other information

The owner's option 1(a) on #2580. The Internet checksum helpers move into pcapkit.corekit.checksum.internet: ChecksumStatus, internet_checksum, compute_checksum, verify_checksum, partial_checksum, fragment_payload and the private _rebuilding. Protocol code no longer imports them from protocols-level modules. pseudo_header and verify_upper sit beside IP_Pseudo in pcapkit.protocols.misc.checksum.ip_pseudo.

Not breaking: the old module was never released and _rebuilding is private, so there is no re-export. test_checksum_import_boundary_2580_unit keeps corekit.checksum free of protocols and foundation imports; the review's mutation run showed it fails on each injected violation. utilities-or-corekit.rst records the ruling.

The worker's questions were settled on their recommendations: keep the strict boundary test, leave the lazy imports lazy, and do not aggregate into corekit or pcapkit.all.

Closes #2580

[JarryShaw]

GOOD TO GO at c73728dd9. A Sonnet review covered the move, and I checked its nit fold myself.

• Nothing moved changes: every moved definition is AST-identical apart from imports, and no reference to the old paths is left in pcapkit/, the docs, the examples or the tests.
• The old module is in no release tag, so it gets no re-export or versionremoved.
• The boundary test catches each of the 5 injected violations, and the docs warning set is unchanged against main.
• 113 modules pass, and the corekit leg (947 tests) is OK. The one protocols/internet error, in test_mh_timestamp_info_1886_unit, fails the same way on main.

[JarryShaw]

Resolve conflicts

[JarryShaw]

On it. The conflicts come from #2544 and #2583 merging into main. A rebase of c73728dd9 onto main is in progress. It keeps both sides and retargets any new main code that imports the checksum names from their old paths. I'll push it with a lease and post the rebase note here, giving old to new heads, the patch-id check and the review status.

[JarryShaw]

Rebased onto main (5145a10d8): c73728dd9 → e54460d28, still one commit.

• Conflict: only docs/source/pcapkit/protocols/misc/checksum.rst. docs(pcapkit): give every undocumented shipped module a page #2584 re-underlined the Internet Checksum section that this PR moves to corekit/checksum/internet.rst, so the section stays deleted. docs(pcapkit): give every undocumented shipped module a page #2584's new intro cited the old misc.checksum.checksum path; it now points to pcapkit.corekit.checksum.internet.
• Patch-id (--stable, outside that file): 899a58ca95ac on both heads, so the rest of the patch is identical. Nothing new on main imports a moved name from its old path.
• Tests: 103 modules pass (the checksum modules, the boundary test, refactor(application)!: dispatch HTTP to HTTPv1/HTTPv2 under the RFC names (#2524) #2544's HTTP/TCP modules, the member-order guard and the docs page checks), and the corekit leg ran 947 tests, all OK.
• CI on the old head: it had not finished (0 failed, 14 in progress), so there was no KTLO tally. Review: GOOD TO GO still stands. The resolution only retargets one cross-reference, and I checked it myself.

[github-actions]

Coverage: 94.02% (unit tier, Python 3.15, e54460d28, Unit Tests run success)

Package	Statements	Missed	Branches	Partial	Cover
pcapkit (top level)	104	4	20	2	95.16%
pcapkit/const	20093	144	2712	125	98.78%
pcapkit/corekit	2664	74	836	37	96.60%
pcapkit/dumpkit	281	3	100	3	98.43%
pcapkit/foundation	3360	27	1186	32	98.70%
pcapkit/interface	138	4	52	3	96.32%
pcapkit/protocols	20291	221	5352	205	98.32%
pcapkit/toolkit	961	28	342	10	96.32%
pcapkit/utilities	506	7	144	5	98.15%
pcapkit/vendor	5009	2222	1148	182	52.31%

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