[JarryShaw]

Please follow the guide below

• You will be asked some questions, please read them carefully and answer honestly

• Put an x into all the boxes [ ] relevant to your pull request (like that [x])

• Use Preview tab to see how your pull request will actually look like

• 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. Narrow runs only, listed below.

• Added a changelog entry under docs/source/changelog/ and regenerated CHANGELOG.md, if the change is user-visible: 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?

Tick the commit type your subject line carries.

• fix — corrects a defect
• feat — adds a feature
• perf — changes performance, not behaviour
• refactor — changes neither behaviour nor performance
• test — tests only
• docs — documentation only
• ci — workflows or build tooling
• release — bumps the version or rolls up a distribution
• chore — anything else

---

Description

Part of #1103, the whole-repository prose sweep, for the linktrans slice. It follows the #719 rules: prose only, accuracy first, no churn and no timed context.

Version directives, all 1.5.0 and naming the issue:

• versionchanged on ARP, InARP, L2TP, VLAN, TCP and UDP.
• versionadded on L2TPv2, C_Tag, S_Tag and SCTP.

Accuracy:

• Header tables use the data-model keys: ethernet.*, tcp.options,
  l2tp.version, and the tcp.flags.reserved row on the TCP page.
• _read_mode_poc names the POC-permitted option; SACK typo fixed.
• Timed wording dropped: "currently" (7), "as before" (2).

Rendered output:

• Dead cross-references now resolve: Protocol.* for ProtocolBase.*,
  Protocol.info, apptype.apptype.AppType, VLAN's C_Tag/S_Tag links.
• Stray quote removed from the FastOpenCookie docstring.
• TCP, SCTP, Link and Transport pages document the referenced
  helpers, schemas and hooks they were missing.

Checks:

• An AST guard holds on every changed .py: code is identical once docstrings are blanked.
• tests/project and every test that reads the edited paths pass.
• A merged Sphinx build of all nine slices adds no new warnings.
• A Sonnet cross-review returned GOOD TO GO.

Not covered here; a round-2 PR follows: the autodoc lists of tcp.rst and sctp.rst (checked by script only), and pcapkit/protocols/data/link/** and data/transport/**.

Refs #1103

[JarryShaw]

Verdict: GOOD TO GO at c8f49a3ba. A Sonnet cross-review checked it (author: Opus).

• Scope and AST guard: 20 files, all inside the slice. All 16 changed .py files are identical once docstrings are blanked, and a negative control confirmed the check catches a real change.
• Directives: all 10 are correct. The 6 versionchanged sit on names that shipped in 1.4.1.post2; the 4 versionadded sit on names absent from it.
• Behaviour claims: the ARP and InARP claims were checked by running both trees.
• Accuracy: more than 30 prose edits were checked, with 0 wrong claims, 0 churn and 0 timed wording.
• Render: the merged build has no new warnings, and unresolved cross-references on the 14 pages went down.
• Non-blocking: three cosmetic unresolved links (Self twice and Option.registry) are left for round 2.

[github-actions]

Coverage: 89.59% (unit tier, Python 3.15, c8f49a3ba, Unit Tests run success)

Package	Statements	Missed	Branches	Partial	Cover
pcapkit (top level)	104	3	20	3	95.16%
pcapkit/const	18862	954	2368	868	91.06%
pcapkit/corekit	2461	69	780	35	96.54%
pcapkit/dumpkit	276	3	100	3	98.40%
pcapkit/foundation	2915	164	1016	34	93.59%
pcapkit/interface	138	7	52	5	93.68%
pcapkit/protocols	17184	224	4574	189	98.08%
pcapkit/toolkit	872	177	298	8	77.18%
pcapkit/utilities	512	2	158	4	99.10%
pcapkit/vendor	4417	2342	1010	157	43.38%

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

[JarryShaw]

Verdict: GOOD TO GO at 97ed26c03. A Sonnet re-check (author: Opus) after SCTP's versionadded lost its placeholder-module text, per the ruling recorded on #1103. Against c8f49a3ba the range-diff shows only those 3 deleted lines. The AST guard finds 0 of 16 files differ, docutils gives 0 warnings, and the run was 1001 passed, 1 skipped.