Skip to content

feat(x2a): Improve the module page details layout - #5050

Merged
yray-pixel merged 3 commits into
redhat-developer:mainfrom
yray-pixel:feat/module-details-layout
Oct 4, 2026
Merged

yray-pixel merged 3 commits into
redhat-developer:mainfrom
yray-pixel:feat/module-details-layout

Conversation

@yray-pixel

Copy link
Copy Markdown
Contributor

Make the module details page simpler and easier to read:

  • Merge 'Artifacts to review' into 'Module Details' section
  • Moved the artifact description into a tooltip.
  • Present phase metadata in a compact layout, including copyable truncated job names, IDs, and commit IDs.
  • Group phase guidance and re-run actions in a bordered instruction panel.
  • Show retry total elapsed time consistently for regular phases and adversarial reviews.
  • Update the analysis action wording to Re-run analysis .
  • Show error details only when an error exists, with an explicit label.

@rhdh-gh-app

rhdh-gh-app Bot commented Sep 30, 2026

Copy link
Copy Markdown

Changed Packages

Package Name Package Path Changeset Bump Current Version
@red-hat-developer-hub/backstage-plugin-x2a workspaces/x2a/plugins/x2a patch v2.0.2

@codecov

codecov Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 89.88764% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 63.84%. Comparing base (d69098f) to head (bf997c2).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5050      +/-   ##
==========================================
- Coverage   64.05%   63.84%   -0.21%     
==========================================
  Files        2717     2709       -8     
  Lines      107661   106689     -972     
  Branches    30323    30075     -248     
==========================================
- Hits        68959    68117     -842     
- Misses      36858    38003    +1145     
+ Partials     1844      569    -1275     
Flag Coverage Δ *Carryforward flag
adoption-insights 84.77% <ø> (ø) Carriedforward from 43310b1
ai-integrations 87.10% <ø> (-0.14%) ⬇️ Carriedforward from 43310b1
app-defaults 68.90% <ø> (ø) Carriedforward from 43310b1
augment 46.67% <ø> (ø) Carriedforward from 43310b1
boost 93.37% <ø> (ø) Carriedforward from 43310b1
bulk-import 73.12% <ø> (ø) Carriedforward from 43310b1
cost-management 13.56% <ø> (ø) Carriedforward from 43310b1
dcm 74.40% <ø> (ø) Carriedforward from 43310b1
e2e-adoption-insights 60.00% <ø> (ø) Carriedforward from 43310b1
e2e-extensions 62.31% <ø> (ø) Carriedforward from 43310b1
e2e-global-header 52.40% <ø> (ø) Carriedforward from 43310b1
e2e-homepage 61.11% <ø> (ø) Carriedforward from 43310b1
e2e-intelligent-assistant 45.49% <ø> (ø) Carriedforward from 43310b1
e2e-orchestrator 49.49% <ø> (ø) Carriedforward from 43310b1
e2e-orchestrator-plugin 49.48% <ø> (ø) Carriedforward from 43310b1
e2e-quickstart 54.83% <ø> (ø) Carriedforward from 43310b1
e2e-scorecard 49.77% <ø> (ø) Carriedforward from 43310b1
e2e-theme 16.43% <ø> (ø) Carriedforward from 43310b1
extensions 58.30% <ø> (ø) Carriedforward from 43310b1
global-floating-action-button 71.18% <ø> (ø) Carriedforward from 43310b1
global-header 69.10% <ø> (ø) Carriedforward from 43310b1
homepage 55.05% <ø> (-0.11%) ⬇️ Carriedforward from 43310b1
install-dynamic-plugins 84.02% <ø> (ø) Carriedforward from 43310b1
intelligent-assistant 78.54% <ø> (ø) Carriedforward from 43310b1
konflux 91.98% <ø> (ø) Carriedforward from 43310b1
lightspeed 69.02% <ø> (ø) Carriedforward from 43310b1
mcp-integrations 84.46% <ø> (ø) Carriedforward from 43310b1
orchestrator 77.69% <ø> (ø) Carriedforward from 43310b1
quickstart 65.83% <ø> (ø) Carriedforward from 43310b1
sandbox 79.56% <ø> (ø) Carriedforward from 43310b1
scorecard 88.97% <ø> (-0.05%) ⬇️ Carriedforward from 43310b1
theme 87.44% <ø> (ø) Carriedforward from 43310b1
translations 7.91% <ø> (ø) Carriedforward from 43310b1
x2a 78.80% <89.88%> (+0.32%) ⬆️

*This pull request uses carry forward flags. Click here to find out more.


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d69098f...bf997c2. 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.

Comment thread workspaces/x2a/plugins/x2a/src/components/PhaseDetails.tsx Outdated
Comment thread workspaces/x2a/plugins/x2a/src/components/ModulePage/AdversarialReviewSection.tsx Outdated
Comment thread workspaces/x2a/plugins/x2a/src/components/PhaseDetails.tsx Outdated
@yray-pixel
yray-pixel marked this pull request as ready for review October 4, 2026 07:28
@yray-pixel
yray-pixel requested a review from a team as a code owner October 4, 2026 07:28
@sonarqubecloud

sonarqubecloud Bot commented Oct 4, 2026

Copy link
Copy Markdown

@yray-pixel
yray-pixel enabled auto-merge (squash) October 4, 2026 07:36
@yray-pixel
yray-pixel merged commit d8a5f82 into redhat-developer:main Oct 4, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants