chore: enforce the declaration ordering convention in ci - #1887

Thanks for contributing! A quick note: this project is in the process of moving from LGPL-2.1 to Apache 2.0, see #1835.

By continuing with this pull request you agree that this new contribution is offered under Apache 2.0.

This notice does not affect your previous contributions. If you would like to consent to relicensing those as well, please post the consent statement on #1835; silence is never treated as consent for past work.

If offering this contribution under Apache 2.0 does not work for you, no hard feelings; reply with the exact phrase I do not agree to the Apache 2.0 license or simply close this pull request.

Merging this PR will not alter performance

✅ 24 untouched benchmarks

_{Comparing enforce_ordering_ci (dd9091d) with master (d44c3f8)}

Codecov Report

❌ Patch coverage is 99.71347% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 99.87%. Comparing base (d44c3f8) to head (dd9091d).
⚠️ Report is 1 commits behind head on master.

@@           Coverage Diff           @@
##           master    #1887   +/-   ##
=======================================
  Coverage   99.87%   99.87%           
=======================================
  Files          33       33           
  Lines        3910     3910           
  Branches      568      568           
=======================================
  Hits         3905     3905           
  Misses          3        3           
  Partials        2        2           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

The one line codecov flags (record_manager.py:68, the failed to remove listener log) predates this PR and is currently unreachable: listeners is a set, so removing an unregistered listener raises KeyError while the handler catches ValueError, a leftover from the list days. #1783 addresses that behavior; covering the line belongs there rather than in this declaration move.