Skip to content

fix(lightspeed): improved MCP ui#3398

Open
ciiay wants to merge 4 commits into
redhat-developer:mainfrom
ciiay:rhdhbugs-3067
Open

fix(lightspeed): improved MCP ui#3398
ciiay wants to merge 4 commits into
redhat-developer:mainfrom
ciiay:rhdhbugs-3067

Conversation

@ciiay

@ciiay ciiay commented Jun 15, 2026

Copy link
Copy Markdown
Member

Hey, I just made a Pull Request!

For RHDHBUGS-3067

✔️ 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)

Screen recording:

rhdhbugs_3067.mp4

Signed-off-by: Yi Cai <yicai@redhat.com>
@rhdh-gh-app

rhdh-gh-app Bot commented Jun 15, 2026

Copy link
Copy Markdown

Changed Packages

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

@codecov

codecov Bot commented Jun 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 54.03%. Comparing base (0496a27) to head (d6dd53d).
⚠️ Report is 12 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3398      +/-   ##
==========================================
+ Coverage   54.01%   54.03%   +0.01%     
==========================================
  Files        2409     2409              
  Lines       87705    87731      +26     
  Branches    24287    24298      +11     
==========================================
+ Hits        47373    47402      +29     
+ Misses      40054    40051       -3     
  Partials      278      278              
Flag Coverage Δ *Carryforward flag
adoption-insights 83.58% <ø> (ø) Carriedforward from 4fed416
ai-integrations 70.03% <ø> (ø) Carriedforward from 4fed416
app-defaults 69.60% <ø> (ø) Carriedforward from 4fed416
augment 46.39% <ø> (ø) Carriedforward from 4fed416
bulk-import 72.69% <ø> (ø) Carriedforward from 4fed416
cost-management 17.48% <ø> (ø) Carriedforward from 4fed416
dcm 60.27% <ø> (ø) Carriedforward from 4fed416
extensions 62.17% <ø> (ø) Carriedforward from 4fed416
global-floating-action-button 74.30% <ø> (ø) Carriedforward from 4fed416
global-header 61.63% <ø> (ø) Carriedforward from 4fed416
homepage 52.67% <ø> (ø) Carriedforward from 4fed416
install-dynamic-plugins 56.23% <ø> (ø) Carriedforward from 4fed416
konflux 91.01% <ø> (ø) Carriedforward from 4fed416
lightspeed 68.66% <ø> (+0.19%) ⬆️
mcp-integrations 85.46% <ø> (ø) Carriedforward from 4fed416
orchestrator 37.33% <ø> (ø) Carriedforward from 4fed416
quickstart 62.09% <ø> (ø) Carriedforward from 4fed416
sandbox 79.56% <ø> (ø) Carriedforward from 4fed416
scorecard 83.93% <ø> (ø) Carriedforward from 4fed416
theme 64.44% <ø> (ø) Carriedforward from 4fed416
translations 8.49% <ø> (ø) Carriedforward from 4fed416
x2a 78.79% <ø> (ø) Carriedforward from 4fed416

*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 0496a27...d6dd53d. 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.

Signed-off-by: Yi Cai <yicai@redhat.com>
Comment thread workspaces/lightspeed/plugins/lightspeed/src/translations/it.ts
Comment thread workspaces/lightspeed/plugins/lightspeed/src/translations/it.ts
Signed-off-by: Yi Cai <yicai@redhat.com>

@HusneShabbir HusneShabbir left a comment

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.

Working as expected. It would be great if we could get the CI clean after addressing Mitesh’s comments—this should help us merge the PR quickly.

Screen.Recording.2026-06-16.at.11.49.42.AM.mov

@ciiay ciiay requested a review from its-mitesh-kumar June 16, 2026 17:20
@sonarqubecloud

Copy link
Copy Markdown

@HusneShabbir HusneShabbir left a comment

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.

Regarding the 3rd criterion in the doc - make sure the rest of the statuses are aligned with the UI, I still see a mismatch where the disabled icon differs from the design, and I’m also unclear about 3 of the 5 tools. When are these expected to be addressed?

Current Implementation:

Image

Figma/Doc:

Image

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