[JarryShaw]

• Searched for similar pull requests
• Followed the coding style
• make test passes, and a test case covers the change (prose only; narrow selections below)
• 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?

• docs — documentation only

---

Description

Part of #1103 round 5, repository-files slice, on main at 4c3d30163. Prose and comments only.

Checks

• AST: with docstrings stripped, the AST is identical on the 12 changed .py files.
• actionlint 1.7.12: the branch reports the same 4 findings as main and adds none.

Stale or false claims fixed

• benchmark.yml and Makefile: the speed tables are in docs/source/engines.rst and are hand-updated, so the "no longer goes stale" claim is gone.
• bump_version.py: a human approval sits between the bump and the release (ci(release): one approval for the whole release, not four separate environment gates #887). 1.5.0b9's release was approved three days after its bump (fix(ci): Create Release tags main's tip at approval time, not the commit it was created for #1634).
• CONTRIBUTING.md:
  • bandit and vermin are clean since ci(lint): clear the advisory linters' long-standing noise so vermin and bandit can be promoted #2145;
  • test_isort_clean.py checks isort;
  • the vermin floor claim is updated.
• README.md: CHANGELOG.md holds the newest entry only.
• util:
  • _actions.py no longer mentions 3.7;
  • the caller lists in discussion_report.py and leg_summary.py are complete;
  • run_unittest_leg.py no longer cites a table that was removed.
• examples/benchmark/README.md: the matrix includes 3.15.
• experimental.yml: win-64 is published.

Rule fixes

• Timed context removed: "now that", "yet".
• Counts removed: "three nested limits", "two util/ scripts", and others.
• The generators' file:line anchors now name ARP.read and PCAPNG.run.

Tests: tests/project/test_docs_* plus the conventions test, 94 passed. The 77 tests/project modules that read the changed files, 1565 passed.

[JarryShaw]

GOOD TO GO at 158224902 (Sonnet review).

• Workflows: yaml.safe_load gives the same result on main and HEAD for all 16 workflows. Every hunk in the 8 changed ones is a # comment. actionlint shows only the false positive that lint.yml documents. The branch adds no findings; I compared it with main, which reports the same 4.
• engines.rst: the speed tables are copied by hand. Nothing writes to the file, and benchmark.yml runs weekly.
• bump_version.py:
  • Only github-release has a required reviewer (ci(release): one approval for the whole release, not four separate environment gates #887).
  • fix(ci): Create Release tags main's tip at approval time, not the commit it was created for #1634 records that 1.5.0b9's release run was created 2026-10-10 and approved 2026-10-13.
• Lint and isort claims:
  • bandit and vermin are clean since ci(lint): clear the advisory linters' long-standing noise so vermin and bandit can be promoted #2145.
  • vermin.ini targets 3.8, matching requires-python >=3.8.
  • test_isort_clean.py exists.
• CHANGELOG.md: it holds the newest entry only.
• Caller lists: the leg_summary.py and discussion_report.py lists are complete.
• Timeout: run_unittest_leg.py's default matches the ordering step's 20-minute timeout.
• Wording: the count and timed-context edits keep their meaning.
• Tests: the AST guard is OK on all 12 .py files. 88 tests/project modules that read the changed files: 1752 passed.
• Not a defect: _actions.py still keeps to 3.7 syntax, which test_actions_helper_2047 pins. Whether to raise that floor is one of the rulings asked for on docs: #719-style prose sweep after the #1090–#1102 defect wave #1103.

[github-actions]

Coverage: 91.55% (unit tier, Python 3.15, 158224902, Unit Tests run success)

Package	Statements	Missed	Branches	Partial	Cover
pcapkit (top level)	104	4	20	2	95.16%
pcapkit/const	19599	880	2618	866	92.12%
pcapkit/corekit	2589	75	816	37	96.48%
pcapkit/dumpkit	282	3	102	3	98.44%
pcapkit/foundation	2971	23	1036	25	98.80%
pcapkit/interface	138	4	52	3	96.32%
pcapkit/protocols	17418	219	4574	185	98.14%
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.

[JarryShaw]

Ready to merge at 158224902.

• Review: Sonnet GOOD TO GO. The YAML of all 16 workflows parses identically to main; this PR changes comments only.
• CI: all 61 CheckRuns passed, none failed or pending. Required checks passed is SUCCESS, and the KTLO legs show no warnings.
• Ordering: I merged this PR into main and ran tests/project plus the root modules under the plain unittest ordering leg: 2288 tests, 0 failures, 0 errors.
• Conflicts: none with the other round-5 PRs.