Skip to content

fix(image): remove UBI branding and deprecated source-level recommendations (TC-4972)#264

Open
a-oren wants to merge 1 commit into
redhat-developer:mainfrom
a-oren:worktree-TC-4972
Open

fix(image): remove UBI branding and deprecated source-level recommendations (TC-4972)#264
a-oren wants to merge 1 commit into
redhat-developer:mainfrom
a-oren:worktree-TC-4972

Conversation

@a-oren

@a-oren a-oren commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Delete UBIIntentionAction and its hardcoded catalog.redhat.com link
  • Remove the deprecated source-level recommendation path (getRecommendation, findMatchingRecommendation) from DockerfileAnnotator — the backend no longer populates it for image analysis since the UBI provider was removed
  • Unify recommendation text to generic "Recommended image:" wording, aligned with the VS Code extension
  • Update HardenedImageIntentionAction text to "Switch to X for enhanced security"
  • Remove Ecosystem Catalog references from plugin.xml and README.md

Implements TC-4972

Test plan

  • ./gradlew build passes (compilation + all tests)
  • No remaining UBI/hardened branding references in codebase
  • getHardenedRecommendations() still works for provider-level recommendations

…ations (TC-4972)

Delete UBIIntentionAction and its hardcoded catalog.redhat.com link.
Remove the deprecated source-level recommendation path (getRecommendation,
findMatchingRecommendation) from DockerfileAnnotator since the backend
no longer populates it for image analysis. Unify recommendation text to
generic "Recommended image:" wording aligned with the VS Code extension.
Update HardenedImageIntentionAction text to "Switch to X for enhanced
security". Remove Ecosystem Catalog references from plugin.xml and
README.md.

Implements TC-4972

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Assisted-by: Claude Code
@sonarqubecloud

Copy link
Copy Markdown

@a-oren

a-oren commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator Author

Verification Report for TC-4972 (commit ae04123)

Check Result Details
Review Feedback N/A No review feedback on PR
Root-Cause Investigation N/A No sub-tasks created
Scope Containment PASS All 4 task files present; 3 extra files are tests/related intention action
Diff Size PASS Net -193 lines proportionate for removal task
Commit Traceability PASS Single commit references TC-4972 in subject and body
Sensitive Patterns PASS No secrets or credentials in added lines
CI Status PASS All 7 CI checks pass (build-linux, build-macos, build-windows, main, verify, SonarCloud x2)
Acceptance Criteria PASS 7 of 7 criteria met (criterion 6 satisfied via getHardenedRecommendations — justified deviation)
Test Quality PASS No repetitive tests; documentation adequate; Eval Quality: N/A
Test Change Classification REDUCTIVE 8 tests removed (35→27) — all covered deleted getRecommendation() method or removed UBI combo scenarios
Verification Commands PASS ./gradlew build succeeds

Overall: PASS

All checks pass. Test changes are reductive but justified — removed tests covered code paths that no longer exist (deleted getRecommendation() method and recommendation parameter). The remaining 27 tests fully cover the active recommendation path (getHardenedRecommendations).


This comment was AI-generated by sdlc-workflow/verify-pr v0.13.2.

@a-oren
a-oren requested a review from ruromero July 21, 2026 12:28
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