Skip to content

[fix] hide report metadata outside verbose mode#12

Merged
b4prog merged 1 commit into
mainfrom
simplify-output
Jun 29, 2026
Merged

[fix] hide report metadata outside verbose mode#12
b4prog merged 1 commit into
mainfrom
simplify-output

Conversation

@b4prog

@b4prog b4prog commented Jun 29, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • Bug Fixes

    • Reduced extra summary details in standard report output, so non-verbose reports now show only the most relevant information.
    • Ensured detailed metrics and analyzed extensions appear only in verbose output.
    • Updated duplicate-report snapshots to match the streamlined output.
  • Chores

    • Bumped the package version to 0.7.5.

@coderabbitai

coderabbitai Bot commented Jun 29, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 5a79f049-6a05-4808-9c7e-2a3f904b073d

📥 Commits

Reviewing files that changed from the base of the PR and between 163b78c and 477b37d.

⛔ Files ignored due to path filters (1)
  • Cargo.lock is excluded by !**/*.lock
📒 Files selected for processing (4)
  • Cargo.toml
  • src/lib.rs
  • src/report/complexity_renderer.rs
  • src/report/duplicate_renderer.rs
💤 Files with no reviewable changes (1)
  • src/lib.rs
📜 Recent review details
🔇 Additional comments (3)
src/report/duplicate_renderer.rs (1)

48-50: LGTM!

Also applies to: 61-68, 198-199, 250-252

src/report/complexity_renderer.rs (1)

36-36: LGTM!

Also applies to: 71-88, 185-188, 237-239

Cargo.toml (1)

3-3: LGTM!


📝 Walkthrough

Walkthrough

"Analyzed extensions", max cognitive complexity, and max cyclomatic complexity lines are moved from the default output to verbose-only output in both duplicate_renderer and complexity_renderer. Each renderer gains a render_verbose_summary helper. Tests and an integration snapshot are updated accordingly, and the crate version is bumped to 0.7.5.

Changes

Verbose-only rendering of analyzed extensions and complexity

Layer / File(s) Summary
duplicate_renderer: gate Analyzed extensions to verbose
src/report/duplicate_renderer.rs
render_report_summary conditionally calls a new render_verbose_summary helper; that helper emits the sorted "Analyzed extensions" line. Tests updated to assert absence in quiet mode and presence in verbose mode.
complexity_renderer: gate extensions and max-complexity to verbose
src/report/complexity_renderer.rs
render_complexity_report routes verbose summary through a new render_verbose_summary helper writing analyzed extensions and max cognitive/cyclomatic complexity. renders_empty_report and renders_verbose_files_and_timings tests updated.
Snapshot test update and version bump
src/lib.rs, Cargo.toml
Integration snapshot removes the dynamic "Analyzed extensions" lines from its expected array; crate version bumped to 0.7.5.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • b4prog/CodeM8#3: Also modifies duplicate-report rendering to gate sections (Code/Locations) to verbose mode in the same renderer.
  • b4prog/CodeM8#9: Directly modifies duplicate_renderer.rs to update verbose vs. non-verbose output for analyzed metrics.
  • b4prog/CodeM8#10: Introduces the complexity rendering subsystem in complexity_renderer.rs that this PR modifies.
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the main change: moving report metadata behind verbose mode.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch simplify-output

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@b4prog b4prog merged commit 3a32e51 into main Jun 29, 2026
3 checks passed
@b4prog b4prog deleted the simplify-output branch June 29, 2026 11:10
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