Skip to content

fix: hide empty nested objects with -e flag#1186

Merged
pamburus merged 1 commit into
masterfrom
feature/empty-fields
Dec 14, 2025
Merged

fix: hide empty nested objects with -e flag#1186
pamburus merged 1 commit into
masterfrom
feature/empty-fields

Conversation

@pamburus
Copy link
Copy Markdown
Owner

@pamburus pamburus commented Dec 14, 2025

Apply hide_empty_fields recursively to nested object fields and hide objects that become empty after filtering. Use single-pass approach with buffer rollback to avoid double-parsing.

  • Hide empty nested fields when flattening (default behavior)
  • Hide empty objects and objects with all empty fields
  • Show ellipsis (...) indicator when fields are hidden
  • Efficient single-pass implementation with buffer rollback

@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 14, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.12%. Comparing base (8c83331) to head (3e251bb).
⚠️ Report is 78 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1186      +/-   ##
==========================================
+ Coverage   84.10%   84.12%   +0.02%     
==========================================
  Files          47       47              
  Lines        9453     9465      +12     
==========================================
+ Hits         7950     7962      +12     
  Misses       1503     1503              

☔ View full report in Codecov by Sentry.
📢 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.

@pamburus pamburus marked this pull request as ready for review December 14, 2025 22:03
@pamburus pamburus merged commit 04abba1 into master Dec 14, 2025
12 checks passed
@pamburus pamburus deleted the feature/empty-fields branch December 14, 2025 22:03
@pamburus pamburus added the bug Something isn't working label Jan 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant