Skip to content

Guard key insights against null Actions minutes lists - #16

Merged
kgrizz-git merged 1 commit into
mainfrom
fix/sonar-reliability-s6466
Aug 7, 2026
Merged

Guard key insights against null Actions minutes lists#16
kgrizz-git merged 1 commit into
mainfrom
fix/sonar-reliability-s6466

Conversation

@kgrizz-git

Copy link
Copy Markdown
Owner

Summary

  • Add a defensive None fallback for repo_consumers.by_minutes in get_key_insights
  • Prevent an index error when SonarCloud or upstream data returns by_minutes: null
  • Add a regression test covering the null-list case

Why

get_key_insights previously assumed repo_consumers.by_minutes was always a list. When that field is None, indexing the first element can raise an error and break report generation. The new guard keeps the insight calculation resilient while preserving the existing output behavior.

@coderabbitai

coderabbitai Bot commented Aug 7, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 2309be00-ea0a-4a52-84ac-794c3f240977

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

@sonarqubecloud

sonarqubecloud Bot commented Aug 7, 2026

Copy link
Copy Markdown

@kgrizz-git
kgrizz-git merged commit 05c5e0a into main Aug 7, 2026
11 checks passed
@kgrizz-git
kgrizz-git deleted the fix/sonar-reliability-s6466 branch August 7, 2026 23:02
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