Skip to content

feat(x2a): add action to resync project migration#3196

Open
mareklibra wants to merge 2 commits into
redhat-developer:mainfrom
mareklibra:FLPATH-3214.recalculateModuleList
Open

feat(x2a): add action to resync project migration#3196
mareklibra wants to merge 2 commits into
redhat-developer:mainfrom
mareklibra:FLPATH-3214.recalculateModuleList

Conversation

@mareklibra
Copy link
Copy Markdown
Member

Fixes: FLPATH-3214

The user can newly update the project migration plan in an external flow and then let the X2A resync the module list changes.


resync.mp4

@rhdh-gh-app
Copy link
Copy Markdown

rhdh-gh-app Bot commented May 21, 2026

Important

This PR includes changes that affect public-facing API. Please ensure you are adding/updating documentation for new features or behavior.

Changed Packages

Package Name Package Path Changeset Bump Current Version
@red-hat-developer-hub/backstage-plugin-x2a-backend workspaces/x2a/plugins/x2a-backend minor v1.4.2
@red-hat-developer-hub/backstage-plugin-x2a-common workspaces/x2a/plugins/x2a-common minor v1.2.1
@red-hat-developer-hub/backstage-plugin-x2a-mcp-extras workspaces/x2a/plugins/x2a-mcp-extras minor v0.2.1
@red-hat-developer-hub/backstage-plugin-x2a-node workspaces/x2a/plugins/x2a-node minor v0.2.2
@red-hat-developer-hub/backstage-plugin-x2a workspaces/x2a/plugins/x2a minor v1.2.2

@codecov
Copy link
Copy Markdown

codecov Bot commented May 21, 2026

Codecov Report

❌ Patch coverage is 75.95628% with 44 lines in your changes missing coverage. Please review.
✅ Project coverage is 61.03%. Comparing base (815580b) to head (84cea22).
⚠️ Report is 8 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3196      +/-   ##
==========================================
+ Coverage   60.97%   61.03%   +0.05%     
==========================================
  Files        2098     2101       +3     
  Lines       65140    65310     +170     
  Branches    17029    17075      +46     
==========================================
+ Hits        39721    39863     +142     
- Misses      25180    25199      +19     
- Partials      239      248       +9     
Flag Coverage Δ *Carryforward flag
adoption-insights 83.58% <ø> (ø) Carriedforward from 8767d35
ai-integrations 70.03% <ø> (ø) Carriedforward from 8767d35
app-defaults 69.60% <ø> (ø) Carriedforward from 8767d35
augment 69.36% <ø> (ø) Carriedforward from 8767d35
bulk-import 72.86% <ø> (ø) Carriedforward from 8767d35
cost-management 16.49% <ø> (ø) Carriedforward from 8767d35
dcm 32.85% <ø> (ø) Carriedforward from 8767d35
extensions 61.79% <ø> (ø) Carriedforward from 8767d35
global-floating-action-button 74.30% <ø> (ø) Carriedforward from 8767d35
global-header 61.68% <ø> (ø) Carriedforward from 8767d35
homepage 50.95% <ø> (ø) Carriedforward from 8767d35
konflux 91.01% <ø> (ø) Carriedforward from 8767d35
lightspeed 68.33% <ø> (ø) Carriedforward from 8767d35
mcp-integrations 81.59% <ø> (ø) Carriedforward from 8767d35
orchestrator 36.36% <ø> (ø) Carriedforward from 8767d35
quickstart 62.88% <ø> (ø) Carriedforward from 8767d35
sandbox 79.42% <ø> (ø) Carriedforward from 8767d35
scorecard 83.72% <ø> (ø) Carriedforward from 8767d35
theme 64.54% <ø> (ø) Carriedforward from 8767d35
translations 8.49% <ø> (ø) Carriedforward from 8767d35
x2a 78.47% <75.95%> (+0.18%) ⬆️

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


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 815580b...84cea22. 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.

@mareklibra mareklibra force-pushed the FLPATH-3214.recalculateModuleList branch 2 times, most recently from 056afe2 to a32aae0 Compare May 21, 2026 07:41
Signed-off-by: Marek Libra <marek.libra@gmail.com>
@mareklibra mareklibra force-pushed the FLPATH-3214.recalculateModuleList branch from a32aae0 to 8767d35 Compare May 21, 2026 07:54
@mareklibra mareklibra marked this pull request as ready for review May 21, 2026 08:20
technology?: ReturnType<typeof normalizeSourceTechnology>;
}> = [];

for (const mm of metadataModules) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This fucntion is almost impossible to follow, should we have an abstraction here? It's 100 lines more of code, and It does so many things, that I think that It should be break to avoid scenarios where this is going to be difficult to debug.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

simplified

@sonarqubecloud
Copy link
Copy Markdown

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.

2 participants