Skip to content

fix(core): trim metadata debug output#7366

Merged
Xuanwo merged 2 commits intomainfrom
xuanwo/fix-metadata-debug-output
Apr 9, 2026
Merged

fix(core): trim metadata debug output#7366
Xuanwo merged 2 commits intomainfrom
xuanwo/fix-metadata-debug-output

Conversation

@Xuanwo
Copy link
Copy Markdown
Member

@Xuanwo Xuanwo commented Apr 8, 2026

Which issue does this PR close?

N/A.

Rationale for this change

Metadata currently relies on the derived Debug implementation, which prints every optional field and default boolean state. That makes Entry debug output much noisier than the actual signal, especially during list/debug flows.

What changes are included in this PR?

This change replaces the derived Debug implementation on Metadata with a manual formatter that always prints mode and only prints meaningful fields. Present optional fields are rendered as their inner values instead of Some(...), and unit tests were added to lock in the new output.

Are there any user-facing changes?

Users inspecting Debug output for Metadata and Entry will now see shorter, easier-to-scan output. There is no API behavior change.

AI Usage Statement

Assisted by OpenAI GPT-5.

@Xuanwo Xuanwo marked this pull request as ready for review April 8, 2026 09:46
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. releases-note/fix The PR fixes a bug or has a title that begins with "fix" labels Apr 8, 2026
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Apr 9, 2026
@Xuanwo Xuanwo merged commit 2a12e2d into main Apr 9, 2026
446 of 450 checks passed
@Xuanwo Xuanwo deleted the xuanwo/fix-metadata-debug-output branch April 9, 2026 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer releases-note/fix The PR fixes a bug or has a title that begins with "fix" size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants