cbor: Scapy-native fields, codec hardening, and cbor2 CI by polybassa · Pull Request #5125 · secdev/scapy

polybassa · 2026-09-01T11:41:21Z

Summary

Rewrite CBOR packet/field APIs toward Scapy ASN.1-style conventions
Harden maps, floats, optionals, and raw-cache fidelity
Add pinned cbor2 differential coverage behind isolated tox/CI

Stack

PR 2 of 3. Depends on #5124 (CRC presets). Reviewable CBOR-only commits sit on top of that foundation; next is BPv7 contrib.

Test plan

./test/run_tests -t test/scapy/layers/cbor.uts -N
tox -e cbor2 (Python 3.12)
flake8 / mypy on touched paths

codecov · 2026-09-01T11:46:20Z

Codecov Report

❌ Patch coverage is 85.83639% with 116 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.95%. Comparing base (15ce846) to head (f16842e).

Files with missing lines	Patch %	Lines
scapy/cbor/cborcodec.py	84.14%	72 Missing ⚠️
scapy/cbor/cbor.py	85.25%	41 Missing ⚠️
scapy/cborpacket.py	95.58%	3 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #5125      +/-   ##
==========================================
- Coverage   81.17%   80.95%   -0.23%     
==========================================
  Files         393      393              
  Lines       98109    99515    +1406     
==========================================
+ Hits        79641    80561     +920     
- Misses      18468    18954     +486

Files with missing lines	Coverage Δ
scapy/cbor/__init__.py	`100.00% <ø> (ø)`
scapy/cbor/cborfields.py	`83.22% <ø> (+5.36%)`	⬆️
scapy/packet.py	`85.02% <100.00%> (+0.21%)`	⬆️
scapy/utils.py	`72.96% <100.00%> (-0.67%)`	⬇️
scapy/cborpacket.py	`96.29% <95.58%> (+6.82%)`	⬆️
scapy/cbor/cbor.py	`85.45% <85.25%> (+1.83%)`	⬆️
scapy/cbor/cborcodec.py	`85.01% <84.14%> (-2.76%)`	⬇️

... and 19 files with indirect coverage changes

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copilot

🔵 Needs a closer look

It makes wide-reaching changes across CBOR core encoding/decoding and packet cache semantics, which warrants careful human validation beyond static review.

Pull request overview

This PR refactors Scapy’s CBOR support toward Scapy-native field conventions (Python-native values, ASN.1-like hooks), strengthens codec correctness (maps, floats, optionals, nesting/indefinite handling, deterministic encoding), and adds an isolated cbor2-based interoperability/differential test lane in tox and CI.

Changes:

Rework CBOR field/container APIs to use Scapy-style any2i/i2m/m2i plus build_result/dissect_result, with improved raw-cache fidelity and parent/child ownership handling.
Harden CBOR codec behavior (indefinite containers, duplicate keys, preferred float widths, deterministic map ordering, max nesting) and introduce CBOR map pair preservation (CBORMapData).
Add pinned cbor2==6.1.4 differential UTScapy campaign plus a dedicated tox env and GitHub Actions job.

File summaries

File	Description
`tox.ini`	Adds a dedicated `cbor2` tox env for pinned differential tests; removes `cbor2` from main deps.
`test/scapy/layers/requirements-cbor2.txt`	Documents/pins optional cbor2 dependency for interop testing.
`test/scapy/layers/generate_cbor2_corpus.py`	Adds helper script to generate reproducible CBOR corpora using cbor2.
`test/scapy/layers/cbor_cbor2_interop.uts`	Adds extensive cbor2-vs-Scapy differential/interoperability coverage.
`test/fields.uts`	Adds a regression test around `ConditionalField.__getattr__`.
`test/configs/windows2.utsc`	Excludes the cbor2 interop campaign from default Windows2 runs and marks keyword.
`test/configs/windows.utsc`	Excludes the cbor2 interop campaign from default Windows runs and marks keyword.
`test/configs/solaris.utsc`	Excludes the cbor2 interop campaign from default Solaris runs and marks keyword.
`test/configs/linux.utsc`	Excludes the cbor2 interop campaign from default Linux runs and marks keyword.
`test/configs/bsd.utsc`	Excludes the cbor2 interop campaign from default BSD runs and marks keyword.
`scapy/cborpacket.py`	Adds CBOR-specific packet cache integration, default-copy deepening, and parent reattachment on copy.
`scapy/cbor/cborfields.py`	Major CBOR field API rewrite (native values, optional/conditional semantics, sequences, deterministic map schema behavior).
`scapy/cbor/cborcodec.py`	Hardens low-level codec: nesting limit, indefinite support, strictness checks, float width preference, deterministic encoding helpers.
`scapy/cbor/cbor.py`	Introduces `CBORMapData`, richer equality/hash semantics, float encoding retention, and native wrapper classes.
`scapy/cbor/__init__.py`	Exposes new CBOR types/fields/results in the public API surface.
`README.md`	Fixes packaging status badge markdown formatting.
`.github/workflows/unittests.yml`	Adds a CI job running `tox -e cbor2` on Python 3.12.

Review details

Files reviewed: 17/18 changed files
Comments generated: 2
Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Capture map-key equivalence, deterministic floats, hash/tag field semantics, and multi-map unknown ownership before changing the implementation. Co-authored-by: Cursor <cursoragent@cursor.com>

Capture map-key equivalence, deterministic floats, hash/tag field semantics, and multi-map unknown ownership before changing the implementation. Co-authored-by: Cursor <cursoragent@cursor.com> AI-Assisted: yes (Composer)

polybassa · 2026-09-11T06:55:23Z

@gpotter2 I've done with a lot of refactorings and a manual review; Could you please have a look?

Capture map-key equivalence, deterministic floats, hash/tag field semantics, and multi-map unknown ownership before changing the implementation. Co-authored-by: Cursor <cursoragent@cursor.com> AI-Assisted: yes (Composer)

polybassa · 2026-09-21T07:56:33Z

@gpotter2 Could you please have a look?

Rewrite the CBOR packet/field layer toward ASN.1-style Scapy APIs, harden maps/floats/optionals, and add pinned cbor2 differential coverage behind an isolated tox/CI job. AI-Assisted: yes (Cursor) Co-authored-by: Cursor <cursoragent@cursor.com>

Rename CBORF_PACKET/SEQUENCE_OF/ARRAY_OF constructor kwargs to pkt_cls so typing.Generic.__new__ does not collide on 3.7, and drop the BPv7-only BundleEidField unit from the CBOR campaign. AI-Assisted: yes (Cursor) Co-authored-by: Cursor <cursoragent@cursor.com>

Capture map-key equivalence, deterministic floats, hash/tag field semantics, and multi-map unknown ownership before changing the implementation. Co-authored-by: Cursor <cursoragent@cursor.com> AI-Assisted: yes (Composer)

Use semantic CBOR key equivalence for duplicate detection and lookup, rebuild deterministic floats from values, drop identity hashing, and treat schema tag numbers as metadata rather than packet fields. Co-authored-by: Cursor <cursoragent@cursor.com> AI-Assisted: yes (Composer)

Arbitrary CBOR values use the lossless object tree; typed floats stay plain Python floats with wire fidelity via the packet raw cache. Co-authored-by: Cursor <cursoragent@cursor.com> AI-Assisted: yes (Composer)

Replace packet-global _cbor_unknown_map_pairs with a dedicated mutable field on each CBORF_MAP so nested and sibling maps keep independent extensions across copy and rebuild. Co-authored-by: Cursor <cursoragent@cursor.com> AI-Assisted: yes (Composer)

Unknown map state now lives in normal fields, so CBOR packets can use Scapy's default build path. Keep only parent reattachment in copy(). Co-authored-by: Cursor <cursoragent@cursor.com> AI-Assisted: yes (Composer)

Reject nonterminal unbounded SEQUENCE_OF, support count_from/max_count, and replace the 1<<30 sentinel with conf.max_list_count. Co-authored-by: Cursor <cursoragent@cursor.com> AI-Assisted: yes (Composer)

…ing packets Default deterministic scanning rejects indefinite containers, drop internal result types from the public surface, and simplify BYTE_STRING_PACKET decode. Co-authored-by: Cursor <cursoragent@cursor.com> AI-Assisted: yes (Composer)

regression.uts leaves conf.debug_dissector True, so the nested constructor test must pin False/True explicitly like BYTE_STRING_PACKET. AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

Drop count_from, shrink cardinality to structural_min, isolate ismutable defaults on CBOR_Packet, and fold encode_value into i2m. AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

Drop structural_min_items and skipped-optional validation so active conditionals and required tails keep ownership of budgeted items. AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

Skip top-level CBOR_ABSENT fingerprints, require direct-terminal SEQUENCE_OF, and drop SEQUENCE suffix lookahead. AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

Fail SEMANTIC_TAG(SEQUENCE_OF) at construction, let optional own presence matching, and drop the single-use _mark_absent helper. AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

Use CBOR_KeyKind norms and CBOR_FingerprintKind cache tags, drop redundant int() enum casts, half floats via struct >e, and make EnumElement deepcopy a singleton. AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

Reject nested REMAINDER_OF, require usable CBOR_root for packet classes, snapshot generic CBOR_Object fingerprints, rename SEQUENCE/SEQUENCE_OF to ITEMS/REMAINDER_OF, and slim max_items. AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

Recurse remainder rejection only through unframed ITEMS, apply conf.max_list_count to MAP, and tighten Packet default-copy coverage. AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

Align remainder validation and MAP limit tests with the re-review's preferred form, including ITEMS(ARRAY(...), tail) coverage. AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

Cover conf.max_list_count exactly-at-limit for unknown MAP keys and ITEMS wrapping ARRAY_INDEFINITE(REMAINDER_OF). AI-Assisted: yes (Cursor Grok 4.6) Co-authored-by: Cursor <cursoragent@cursor.com>

Count preserved unknown MAP members in is_empty, encode nested ARRAY_OF fields, and reject remainder/kwargs misplacements. AI-Assisted: yes (Cursor Grok 4.6) Co-authored-by: Cursor <cursoragent@cursor.com>

Cover CBORF_ITEMS TypeError for unexpected keywords alongside the existing ARRAY encode_indefinite case. AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

Stop conditional suffix reservation from using undecoded discriminators, initialize zero-budget remainders and nested optionals, fingerprint float bit patterns, and reject nested packet-valued homogeneous collections. AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

Inline unframed ITEMS into array schemas for budgeting, reserve conditionals only after required predecessors are decoded, and trim preferred-float helpers. AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

Replace the positional between-required heuristic with reservation based on already-decoded field names so unrelated required fields do not suppress a known active conditional. AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

Put external_cbor2 on the interop campaign, assert preferred float wire bytes, fix indefinite budget and ITEMS payload checks, and drop the non-determinism scanner helper. AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

Proxy getfieldval and subclass methods on the reservation view, propagate real predicate errors, cap indefinite min_items work instead of requiring quadratic growth, and trim interop helpers. AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

Drop the fake reservation view. Predicates run on the packet; unread fields do not reserve from defaults. Map dissection uses field wrappers, and UTS covers greedy optional-before-flag. AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

Drop unread-field interception. Suffix reservation uses min_items on the real packet. Default-on trailing conditionals over-reserve until the discriminator is dissected. AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

Only optionals inspect the suffix. Conditionals reserve when named discriminators are already in pkt.fields, so default-on flags round-trip instead of rejecting their own encodings. AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

Unread default-on conditionals no longer starve a preceding optional during suffix budgeting. AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

A skipped conditional is decided but never stored in pkt.fields, so later depends_on treated it as unread. AI-Assisted: yes (Cursor Grok 4.6) Co-authored-by: Cursor <cursoragent@cursor.com>

depends_on does not delay ConditionalField evaluation. Unknown names include the conditional field in the error. AI-Assisted: yes (Cursor Grok 4.6) Co-authored-by: Cursor <cursoragent@cursor.com>

AI-Assisted: yes (Cursor Grok 4.6) Co-authored-by: Cursor <cursoragent@cursor.com>

AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

Same-semantic child assignment clears the child cache; the parent must see that and rebuild overlong encodings. AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

polybassa · 2026-09-30T09:42:20Z

@@ -0,0 +1,3 @@
+# Optional interoperability-test dependency. Not required by Scapy itself.
+# cbor2 6.1.4 requires Python 3.10 or newer.


move into tox

polybassa · 2026-09-30T09:42:34Z

@@ -0,0 +1,151 @@
+#!/usr/bin/env python3


as deterministic UTS file

gpotter2 · 2026-09-30T09:41:44Z

       cryptography
       coverage[toml]
       python-can
-       cbor2


You can keep this if you want

gpotter2 · 2026-09-30T09:42:23Z

+[testenv:cbor2]
+description = "CBOR differential tests against pinned cbor2 6.1.4 (Python >= 3.10)"
+basepython = python3.12
+deps =
+       cbor2==6.1.4
+       coverage[toml]
+commands =
+  {envpython} {env:DISABLE_COVERAGE:-m coverage run} -m scapy.tools.UTscapy \
+      -t test/scapy/layers/cbor_cbor2_interop.uts -N {posargs}
+
+


This you can remove and just include it in the normal test case

gpotter2 · 2026-09-30T09:42:41Z

@@ -0,0 +1,3 @@
+# Optional interoperability-test dependency. Not required by Scapy itself.


remove this file

gpotter2 · 2026-09-30T09:45:02Z

@@ -0,0 +1,151 @@
+#!/usr/bin/env python3
+"""Generate a reproducible CBOR corpus with cbor2 6.1.4.


You can include this in the UTS directly

This was referenced Sep 1, 2026

New BPv7 with working UTS and patched cbor modules to support it #5075

Closed

bpv7: Bundle Protocol v7 contrib with RFC validation #5126

Open

polybassa force-pushed the split/cbor-scapy-native branch from 5f9b34d to 414ac55 Compare September 1, 2026 11:57

polybassa requested review from gpotter2 and a lite review from Copilot September 1, 2026 12:04

Copilot started reviewing on behalf of polybassa September 1, 2026 12:04 View session

Copilot AI reviewed Sep 1, 2026

View reviewed changes

Comment thread tox.ini

Comment thread test/fields.uts Outdated

polybassa force-pushed the split/cbor-scapy-native branch from 414ac55 to 4a552ff Compare September 9, 2026 05:01

polybassa force-pushed the split/cbor-scapy-native branch from 4a552ff to f9e6104 Compare September 9, 2026 06:32

polybassa force-pushed the split/cbor-scapy-native branch from b433d7e to 06a065b Compare September 10, 2026 06:30

polybassa force-pushed the split/cbor-scapy-native branch 2 times, most recently from b7fc549 to 1f059ae Compare September 16, 2026 15:34

polybassa force-pushed the split/cbor-scapy-native branch from 1f059ae to fce7bb4 Compare September 16, 2026 15:37

Nils Weiss and others added 9 commits September 29, 2026 20:54

cbor: store CBORF_ANY as CBOR_Object and drop islist workaround

e814722

Arbitrary CBOR values use the lossless object tree; typed floats stay plain Python floats with wire fidelity via the packet raw cache. Co-authored-by: Cursor <cursoragent@cursor.com> AI-Assisted: yes (Composer)

cbor: restore standard Packet.do_build and slim copy()

f677cf4

Unknown map state now lives in normal fields, so CBOR packets can use Scapy's default build path. Keep only parent reattachment in copy(). Co-authored-by: Cursor <cursoragent@cursor.com> AI-Assisted: yes (Composer)

cbor: require explicit SEQUENCE_OF framing and Scapy list limits

6b72b80

Reject nonterminal unbounded SEQUENCE_OF, support count_from/max_count, and replace the 1<<30 sentinel with conf.max_list_count. Co-authored-by: Cursor <cursoragent@cursor.com> AI-Assisted: yes (Composer)

Nils Weiss and others added 26 commits September 29, 2026 20:54

cbor: isolate nested TypeError test from debug_dissector pollution

c8323fd

regression.uts leaves conf.debug_dissector True, so the nested constructor test must pin False/True explicitly like BYTE_STRING_PACKET. AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

cbor: terminal SEQUENCE_OF and trim review abstractions

d0ebf36

Drop count_from, shrink cardinality to structural_min, isolate ismutable defaults on CBOR_Packet, and fold encode_value into i2m. AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

cbor: reserve optionals with live min_items

2795c42

Drop structural_min_items and skipped-optional validation so active conditionals and required tails keep ownership of budgeted items. AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

cbor: fix absent cache and greedy unframed sequences

293f7ec

Skip top-level CBOR_ABSENT fingerprints, require direct-terminal SEQUENCE_OF, and drop SEQUENCE suffix lookahead. AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

cbor: reject tagged SEQUENCE_OF; slim optional dissect

f41ab8b

Fail SEMANTIC_TAG(SEQUENCE_OF) at construction, let optional own presence matching, and drop the single-use _mark_absent helper. AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

cbor: allow nested framed REMAINDER_OF; bound MAP dissect

288f903

Recurse remainder rejection only through unframed ITEMS, apply conf.max_list_count to MAP, and tighten Packet default-copy coverage. AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

cbor: prefer ITEMS isinstance for remainder recursion

44e9d05

Align remainder validation and MAP limit tests with the re-review's preferred form, including ITEMS(ARRAY(...), tail) coverage. AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

cbor: add MAP at-limit tests and nested indefinite remainder

c934464

Cover conf.max_list_count exactly-at-limit for unknown MAP keys and ITEMS wrapping ARRAY_INDEFINITE(REMAINDER_OF). AI-Assisted: yes (Cursor Grok 4.6) Co-authored-by: Cursor <cursoragent@cursor.com>

cbor: keep unknown maps through optional tags

9458654

Count preserved unknown MAP members in is_empty, encode nested ARRAY_OF fields, and reject remainder/kwargs misplacements. AI-Assisted: yes (Cursor Grok 4.6) Co-authored-by: Cursor <cursoragent@cursor.com>

cbor: reject unused ITEMS kwargs in UTS

ad300a8

Cover CBORF_ITEMS TypeError for unexpected keywords alongside the existing ARRAY encode_indefinite case. AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

cbor: default depends_on does not reserve array suffix items

bbfb4f3

Unread default-on conditionals no longer starve a preceding optional during suffix budgeting. AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

cbor: reserve array suffix from resolved fields, not pkt.fields

4fd9f6e

A skipped conditional is decided but never stored in pkt.fields, so later depends_on treated it as unread. AI-Assisted: yes (Cursor Grok 4.6) Co-authored-by: Cursor <cursoragent@cursor.com>

cbor: document depends_on as optional-tail reservation only

8167203

depends_on does not delay ConditionalField evaluation. Unknown names include the conditional field in the error. AI-Assisted: yes (Cursor Grok 4.6) Co-authored-by: Cursor <cursoragent@cursor.com>

cbor: drop unused ITEMS dissect helper and type-check depends_on

1d5f95e

AI-Assisted: yes (Cursor Grok 4.6) Co-authored-by: Cursor <cursoragent@cursor.com>

cbor: type depends_on as Sequence[str] to match runtime

f6a7991

AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

cbor: include nested child raw_packet_cache in parent fingerprint

f16842e

Same-semantic child assignment clears the child cache; the parent must see that and rebuild overlong encodings. AI-Assisted: yes (Composer) Co-authored-by: Cursor <cursoragent@cursor.com>

polybassa force-pushed the split/cbor-scapy-native branch from fce7bb4 to f16842e Compare September 29, 2026 18:54

polybassa commented Sep 30, 2026

View reviewed changes

gpotter2 reviewed Sep 30, 2026

View reviewed changes

Navigation Menu

Uh oh!

cbor: Scapy-native fields, codec hardening, and cbor2 CI - #5125

cbor: Scapy-native fields, codec hardening, and cbor2 CI#5125
polybassa wants to merge 81 commits into
secdev:masterfrom
polybassa:split/cbor-scapy-native

polybassa commented Sep 1, 2026

Uh oh!

codecov Bot commented Sep 1, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

polybassa commented Sep 11, 2026

Uh oh!

polybassa commented Sep 21, 2026

Uh oh!

polybassa Sep 30, 2026

Uh oh!

polybassa Sep 30, 2026

Uh oh!

gpotter2 Sep 30, 2026

Uh oh!

gpotter2 Sep 30, 2026

Uh oh!

gpotter2 Sep 30, 2026

Uh oh!

gpotter2 Sep 30, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

		@@ -0,0 +1,3 @@
		# Optional interoperability-test dependency. Not required by Scapy itself.
		# cbor2 6.1.4 requires Python 3.10 or newer.

		@@ -0,0 +1,151 @@
		#!/usr/bin/env python3
		"""Generate a reproducible CBOR corpus with cbor2 6.1.4.

Uh oh!

Conversation

polybassa commented Sep 1, 2026

Summary

Stack

Test plan

Uh oh!

codecov Bot commented Sep 1, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🔵 Needs a closer look

Uh oh!

Uh oh!

Uh oh!

polybassa commented Sep 11, 2026

Uh oh!

polybassa commented Sep 21, 2026

Uh oh!

polybassa Sep 30, 2026

Choose a reason for hiding this comment

Uh oh!

polybassa Sep 30, 2026

Choose a reason for hiding this comment

Uh oh!

gpotter2 Sep 30, 2026

Choose a reason for hiding this comment

Uh oh!

gpotter2 Sep 30, 2026

Choose a reason for hiding this comment

Uh oh!

gpotter2 Sep 30, 2026

Choose a reason for hiding this comment

Uh oh!

gpotter2 Sep 30, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

codecov Bot commented Sep 1, 2026 •

edited

Loading