Skip to content

JAVASE-187 Add "Sonar agentic AI" quality profile to the plugin#74

Merged
dorian-burihabwa-sonarsource merged 1 commit intobranch-8.16from
db/JAVASE-187-for-sqcb
Apr 10, 2026
Merged

JAVASE-187 Add "Sonar agentic AI" quality profile to the plugin#74
dorian-burihabwa-sonarsource merged 1 commit intobranch-8.16from
db/JAVASE-187-for-sqcb

Conversation

@dorian-burihabwa-sonarsource
Copy link
Copy Markdown
Contributor

No description provided.

@hashicorp-vault-sonar-prod
Copy link
Copy Markdown
Contributor

hashicorp-vault-sonar-prod Bot commented Apr 1, 2026

JAVASE-187

@sonarqube-next
Copy link
Copy Markdown

sonarqube-next Bot commented Apr 1, 2026

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
0 Dependency risks
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@dorian-burihabwa-sonarsource dorian-burihabwa-sonarsource force-pushed the db/JAVASE-187-for-sqcb branch 2 times, most recently from 887d782 to 6a14b1e Compare April 9, 2026 09:12
@sonarqube-next
Copy link
Copy Markdown

sonarqube-next Bot commented Apr 9, 2026

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
0 Dependency risks
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@dorian-burihabwa-sonarsource dorian-burihabwa-sonarsource marked this pull request as ready for review April 9, 2026 16:02
@sonar-review-alpha
Copy link
Copy Markdown

sonar-review-alpha Bot commented Apr 9, 2026

Summary

This PR introduces a new quality profile called "Sonar agentic AI" to the SonarJava plugin. The profile is defined as a JSON configuration that activates 22 curated SonarJava rules, presumably tailored for detecting issues in code that interacts with or is generated by agentic AI systems. The rules cover areas like resource leaks, infinite loops, null pointer dereferences, and API misuse—categories likely relevant for AI-assisted code patterns.

What reviewers should know

What changed: A single new file (Sonar_agentic_AI_profile.json) was added to the plugin's resource directory defining the profile and its 22 rules.

Key things to verify:

  • Confirm that the rule keys (S2095, S2189, etc.) are all valid, existing SonarJava rules
  • Check whether this profile definition needs to be registered or referenced elsewhere in the plugin (e.g., in a manifest, registry, or main configuration file)
  • Verify the profile name matches any documentation or requirements for the JAVASE-187 task

For context: This appears to be a resource-only change; no code modifications to the plugin itself are included. The profile should be available in SonarQube/SonarCloud once deployed.


  • Generate Walkthrough
  • Generate Diagram

🗣️ Give feedback

@dorian-burihabwa-sonarsource dorian-burihabwa-sonarsource requested a review from a team April 9, 2026 16:05
Copy link
Copy Markdown

@sonar-review-alpha sonar-review-alpha Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Conclusion: The new profile's ruleKeys array is byte-for-byte identical to the existing Sonar_way_profile.json. This is almost certainly unintentional — a distinct profile with a different name but the same rules provides no value and will confuse users choosing between them.

🗣️ Give feedback

"S6376",
"S6377"
]
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The ruleKeys list here is identical to Sonar_way_profile.json (same 22 rules, same order). If this is intentional (e.g. the profile starts as a copy and diverges later), that should be documented. If it is not intentional, the rules need to be updated to reflect what "Sonar agentic AI" is actually meant to cover.

Also worth checking: S6374.json and S3546.json exist in the same resource directory but are absent from both profiles — confirm whether any of those should be included here.

  • Mark as noise

@dorian-burihabwa-sonarsource dorian-burihabwa-sonarsource merged commit 861e5e5 into branch-8.16 Apr 10, 2026
9 checks passed
@dorian-burihabwa-sonarsource dorian-burihabwa-sonarsource deleted the db/JAVASE-187-for-sqcb branch April 10, 2026 08:43
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.

2 participants