Skip to content

chore(bulk-import): optimize bulk-import#3822

Open
debsmita1 wants to merge 1 commit into
redhat-developer:mainfrom
debsmita1:optimize-bulk-import
Open

chore(bulk-import): optimize bulk-import#3822
debsmita1 wants to merge 1 commit into
redhat-developer:mainfrom
debsmita1:optimize-bulk-import

Conversation

@debsmita1

@debsmita1 debsmita1 commented Jul 17, 2026

Copy link
Copy Markdown
Member

Hey, I just made a Pull Request!

Resolves:

https://redhat.atlassian.net/browse/RHIDP-15551

  • Cut NFS Module Federation sync size for bulk-import by keeping heavy UI/API graphs off the alpha entry (~468 KB → ~118 KB sync, about −75%).
  • Replaced deprecated NavItemBlueprint with PageBlueprint title + icon, and update the NFS demo sidebar to render auto-discovered navItems.
  • Extracted a thin BulkImportIcon and move getApi out of repository-utils so the alpha/API path no longer pulls sidebar, permissions, or the full icons/utils UI graph.
  • Remaining NFS vs OFS gap (~90 KB) is mostly @backstage/frontend-plugin-api blueprint wiring and its zod schema dependencies (ApiBlueprint / PageBlueprint), not plugin UI. The leftover zod cost is inherent to NFS page/API blueprints.

NFS alpha expose summary

Metric BEFORE AFTER Δ
Sync chunks 5 3 -2
Sync size 467.6 KB 117.8 KB -349.8 KB (-74.8%)
Async chunks 201 204 +3
Async size (listed) 1937.4 KB 2320.7 KB +383.3 KB

✔️ Checklist

  • A changeset describing the change and affected packages. (more info)
  • Added or Updated documentation
  • Tests for new functionality and regression tests for bug fixes
  • Screenshots attached (for UI changes)

@rhdh-gh-app

rhdh-gh-app Bot commented Jul 17, 2026

Copy link
Copy Markdown

Changed Packages

Package Name Package Path Changeset Bump Current Version
app workspaces/bulk-import/packages/app none v0.0.1
@red-hat-developer-hub/backstage-plugin-bulk-import workspaces/bulk-import/plugins/bulk-import patch v7.3.7

@debsmita1
debsmita1 force-pushed the optimize-bulk-import branch from 462aeac to 4b03ff2 Compare July 20, 2026 13:52
@debsmita1
debsmita1 force-pushed the optimize-bulk-import branch from 4b03ff2 to 4168c01 Compare July 20, 2026 13:58
@sonarqubecloud

Copy link
Copy Markdown

@codecov

codecov Bot commented Jul 20, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 83.33333% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 56.53%. Comparing base (5e969f6) to head (4168c01).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3822   +/-   ##
=======================================
  Coverage   56.53%   56.53%           
=======================================
  Files        2357     2359    +2     
  Lines       94934    94937    +3     
  Branches    26612    26610    -2     
=======================================
+ Hits        53671    53675    +4     
+ Misses      40957    40956    -1     
  Partials      306      306           
Flag Coverage Δ *Carryforward flag
adoption-insights 84.54% <ø> (ø) Carriedforward from 5e969f6
ai-integrations 70.06% <ø> (ø) Carriedforward from 5e969f6
app-defaults 69.79% <ø> (ø) Carriedforward from 5e969f6
augment 46.39% <ø> (ø) Carriedforward from 5e969f6
boost 75.27% <ø> (ø) Carriedforward from 5e969f6
bulk-import 72.63% <83.33%> (+0.03%) ⬆️
cost-management 13.55% <ø> (ø) Carriedforward from 5e969f6
dcm 60.72% <ø> (ø) Carriedforward from 5e969f6
extensions 56.18% <ø> (ø) Carriedforward from 5e969f6
global-floating-action-button 71.18% <ø> (ø) Carriedforward from 5e969f6
global-header 62.19% <ø> (ø) Carriedforward from 5e969f6
homepage 50.25% <ø> (ø) Carriedforward from 5e969f6
install-dynamic-plugins 56.77% <ø> (ø) Carriedforward from 5e969f6
intelligent-assistant 69.50% <ø> (ø) Carriedforward from 5e969f6
konflux 91.98% <ø> (ø) Carriedforward from 5e969f6
lightspeed 69.02% <ø> (ø) Carriedforward from 5e969f6
mcp-integrations 83.40% <ø> (ø) Carriedforward from 5e969f6
orchestrator 57.01% <ø> (ø) Carriedforward from 5e969f6
quickstart 65.11% <ø> (ø) Carriedforward from 5e969f6
sandbox 79.56% <ø> (ø) Carriedforward from 5e969f6
scorecard 82.66% <ø> (ø) Carriedforward from 5e969f6
theme 83.85% <ø> (ø) Carriedforward from 5e969f6
translations 5.15% <ø> (ø) Carriedforward from 5e969f6
x2a 79.31% <ø> (ø) Carriedforward from 5e969f6

*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 5e969f6...4168c01. 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.

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.

1 participant