chore: breaking change - remove wrappers...

[nickboldt]

What does this PR do?

chore: breaking change - remove wrappers from RHDH entirely (RHDHPLAN-256, RHDHPLAN-934)

Signed-off-by: Nick Boldt nboldt@redhat.com

Screenshot/screencast of this PR

N/A

What issues does this PR fix or reference?

N/A (or see commit message above for issue number)

How to test this PR?

N/A

PR Checklist

As the author of this Pull Request I made sure that:

• Code produced is complete
• Code builds without errors
• Tests are covering the bugfix
• Relevant user documentation updated
• Relevant contributing documentation updated

Reviewers

Reviewers, please comment how you tested the PR when approving it.

[github-actions]

The container image build workflow finished with status: failure.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 55.25%. Comparing base (c1acb63) to head (b09a266).
⚠️ Report is 7 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4909      +/-   ##
==========================================
- Coverage   55.82%   55.25%   -0.58%     
==========================================
  Files         121      109      -12     
  Lines        2350     2132     -218     
  Branches      562      536      -26     
==========================================
- Hits         1312     1178     -134     
+ Misses       1032      953      -79     
+ Partials        6        1       -5     

Flag	Coverage Δ
rhdh	55.25% <ø> (-0.58%)	⬇️

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c1acb63...b09a266. Read the comment docs.

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[kadel]

before removing wrappers from image, we need to have a catalog image that is not referencing any wrappers. So we need to update dynamicArtifact: ./dynamic-plugins/dist/ in catalog metadata to their oci versions

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[github-actions]

The container image build workflow finished with status: failure.

issue addressed by Stan's thread.

[openshift-ci]

@nickboldt: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name	Commit	Details	Required	Rerun command
ci/prow/e2e-ocp-helm	b09a266	link	true	/test e2e-ocp-helm

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

[openshift-ci]

PR needs rebase.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

[nickboldt]

On further reflection we have decided to hold on removing wrappers until July. See discussion in https://docs.google.com/document/d/1Kr9qb1jLmngX3G7Qq7dpWVSQMehreIGQuNLL30URkzY/edit?tab=t.0

TL;DR: we want to think through what this will mean for SBOM creation, prod sec exceptions to use the non-advisory pipeline, talk to David about why we can’t just consume npm packages more directly (ie., non-bundled wrappers within oci artifacts), and more.

@jon Koops will start some spike work once he and David are back from PTO in late June; then in early July we’ll all meet back to discuss the plan and if possible we’ll remove wrappers at the same time we explore moving to BS 1.53 (due Jul 14 2026).