Skip to content

feat(protoenc): decode UDT type anchors - #263

Draft
benbellick wants to merge 1 commit into
refactor/preserve-testcase-udt-namesfrom
refactor/decode-udt-type-anchors
Draft

feat(protoenc): decode UDT type anchors#263
benbellick wants to merge 1 commit into
refactor/preserve-testcase-udt-namesfrom
refactor/decode-udt-type-anchors

Conversation

@benbellick

Copy link
Copy Markdown
Member

UDT serialization now assigns anchors from semantic extension type IDs, but the decode path also needs registry context to recover those IDs from protobuf anchors.

This adds the registry-aware type decode path and uses it from expression and literal decoding paths that already have an extension registry available.


Note: This PR was developed with AI assistance. All changes have been reviewed, and I take full responsibility for this contribution.

@codecov

codecov Bot commented Jun 9, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 35.52632% with 49 lines in your changes missing coverage. Please review.
✅ Project coverage is 70.08%. Comparing base (04903bb) to head (745b829).

Files with missing lines Patch % Lines
expr/literals.go 21.42% 31 Missing and 2 partials ⚠️
expr/expression.go 55.17% 7 Missing and 6 partials ⚠️
expr/functions.go 40.00% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@                           Coverage Diff                            @@
##           refactor/preserve-testcase-udt-names     #263      +/-   ##
========================================================================
- Coverage                                 70.30%   70.08%   -0.22%     
========================================================================
  Files                                        48       48              
  Lines                                     11044    11099      +55     
========================================================================
+ Hits                                       7764     7779      +15     
- Misses                                     2910     2941      +31     
- Partials                                    370      379       +9     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@benbellick
benbellick force-pushed the refactor/preserve-testcase-udt-names branch from ac6af5c to 04903bb Compare June 9, 2026 20:54
@benbellick
benbellick force-pushed the refactor/decode-udt-type-anchors branch from 0be6a45 to 745b829 Compare June 9, 2026 20:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant