Skip to content

SONARJAVA-7095 Use a common model for Spring context gathering - #6282

Open
aurelien-coet-sonarsource wants to merge 2 commits into
masterfrom
ac/SONARJAVA-7095
Open

aurelien-coet-sonarsource wants to merge 2 commits into
masterfrom
ac/SONARJAVA-7095

Conversation

@aurelien-coet-sonarsource

@aurelien-coet-sonarsource aurelien-coet-sonarsource commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

Summary by Gitar

  • Spring context gathering:
    • Added SpringContextGatheringModel to centralize per-file Spring context data collection
    • Updated BeanDefinitionGatherer and ComponentScanPackageGatherer to use the shared gathering model

This will update automatically on new commits.

@aurelien-coet-sonarsource aurelien-coet-sonarsource changed the title Use a serializable model for Spring context gathering SONARJAVA-7095 Use a serializable model for Spring context gathering Oct 1, 2026
@hashicorp-vault-sonar-prod

Copy link
Copy Markdown
Contributor

SONARJAVA-7095

@aurelien-coet-sonarsource aurelien-coet-sonarsource changed the title SONARJAVA-7095 Use a serializable model for Spring context gathering SONARJAVA-7095 Use a common model for Spring context gathering Oct 1, 2026
@aurelien-coet-sonarsource
aurelien-coet-sonarsource added this pull request to stack #6284 October 1, 2026 11:38
@aurelien-coet-sonarsource
aurelien-coet-sonarsource force-pushed the ac/SONARJAVA-7095 branch 2 times, most recently from 3b81fee to e92203f Compare October 1, 2026 12:50
@aurelien-coet-sonarsource
aurelien-coet-sonarsource marked this pull request as ready for review October 1, 2026 13:24
@aurelien-coet-sonarsource
aurelien-coet-sonarsource force-pushed the ac/SONARJAVA-7095 branch 2 times, most recently from 1bc2362 to c27d8e2 Compare October 2, 2026 11:27
@gitar-bot

gitar-bot Bot commented Oct 2, 2026

Copy link
Copy Markdown
Code Review ✅ Approved

🟡 Medium risk · Centralizes Spring bean and package collection across main and test scans

Refactors Spring context gathering to use a centralized, serializable per-file model. No issues found.

Review coverage

🧪 Functional validation 0 of 2 objectives covered

📋 Rules No rules evaluated

Cross-repo coverage 5 repositories selected

🤖 Auto-approval Not enabled · Set up

Implementation Status ◻️ 0 of 2 objectives covered
◻️ SONARJAVA-7095 - 0 of 2 objectives covered

This PR uses a common model for Spring context gathering, while on-disk serialization and de-serialization are not part of this diff.

Other objectives on this issue, possibly covered elsewhere:

  • ◻️ Implement on-disk serialization for the Spring context model
  • ◻️ Implement de-serialization for the Spring context model
Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Counting what did not apply, without listing it.

Comment with these commands to change the behavior for this request:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@sonarqube-next

sonarqube-next Bot commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

This branch has not been deployed

No deployments
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