Skip to content

chore(#5036): update @backstage/cli and add cli-defaults - #5037

Open
fullsend-ai-coder[bot] wants to merge 3 commits into
mainfrom
agent/5036-update-backstage-cli
Open

fullsend-ai-coder[bot] wants to merge 3 commits into
mainfrom
agent/5036-update-backstage-cli

Conversation

@fullsend-ai-coder

Copy link
Copy Markdown
Contributor

Summary

  • Updated @backstage/cli in root package.json from ^0.27.0 to ^0.36.5
  • Added @backstage/cli-defaults: "^0.1.5" to the bulk-import and translations workspace package.json files — these are UI team-owned workspaces (per CODEOWNERS) that were missing the dependency

All workspaces on backstage 1.54.6 already had @backstage/cli ^0.36.5, so no workspace CLI version updates were needed.

✔️ Checklist

  • A changeset describing the change and affected packages. N/A — this is a devDependency version alignment with no user-facing behavior change
  • Added or Updated documentation
  • Tests for new functionality and regression tests for bug fixes — N/A, devDependency-only change
  • Screenshots attached (for UI changes)

Closes #5036

Post-script verification

  • Branch is not main/master (agent/5036-update-backstage-cli)
  • Secret scan passed (gitleaks — bd0634257e7222bca2688907fcc57147112d9733..HEAD)
  • PR body secret scan passed (gitleaks — no-git)

Update @backstage/cli in root package.json from ^0.27.0 to ^0.36.5.

Add @backstage/cli-defaults ^0.1.5 to the bulk-import and translations
workspace package.json files. These are UI team-owned workspaces that
were missing the cli-defaults devDependency. All other UI team
workspaces already have it.

All workspaces on backstage 1.54.6 already have @backstage/cli ^0.36.5,
so no workspace CLI version updates were needed.

Closes #5036
@christoph-jerolimov

Copy link
Copy Markdown
Member

/fs-fix run yarn install in the root and in the touched workspaces to update yarn.lock file

@fullsend-ai-coder

fullsend-ai-coder Bot commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor Author

🤖 Finished Fix · ❌ Failure (post-script /home/runner/work/rhdh-plugins/rhdh-plugins/.fullsend/.fullsend-cache/resources/sha256/f0b37e364cfe852baffb406950fba12d481781d1ec6d4ec0d4d22f581fa19dd8/scripts/post-fix.sh failed: exit sta…) · Started 3:05 PM UTC · Completed 3:56 PM UTC

Commit: 32bf5a3 · View workflow run →

Runtime: claude · Model: opus → claude-opus-4-6 · Effort: high · Cost: $6.19

@fullsend-ai-coder

Copy link
Copy Markdown
Contributor Author

⚠️ Post-fix script failed — Setup error (exit code 1)

The fix agent completed, but the post-fix script failed before finishing.

Workflow run: https://github.com/redhat-developer/rhdh-plugins/actions/runs/36587244079

Details:
Extracted repo not found at repo
Please check the workflow logs for full details and retry with /fs-fix if appropriate.

@christoph-jerolimov

Copy link
Copy Markdown
Member

/fs-fix run yarn install in the root and in the touched workspaces to update yarn.lock file

@fullsend-ai-coder

fullsend-ai-coder Bot commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor Author

🤖 Finished Fix · ✅ Success · Started 4:13 PM UTC · Completed 4:40 PM UTC

Commit: 32bf5a3 · View workflow run →

Runtime: claude · Model: opus → claude-opus-4-6 · Effort: high · Cost: $3.47

…paces

Run yarn install in bulk-import and translations workspaces to add
@backstage/cli-defaults to their yarn.lock files.

Note: root yarn.lock was not updated. yarn install at the root fails
because @backstage/cli 0.36.5 introduces a transitive dependency on
got 11.8.2 via a Yarn patch protocol, but the required patch file
(.yarn/patches/got-npm-11.8.2-c1eb105458.patch) does not exist in
this repository. This needs human investigation.

Note: pre-commit hooks were not run due to remaining time budget.

Addresses #5037
@fullsend-ai-coder

Copy link
Copy Markdown
Contributor Author

🔧 Fix agent — iteration 1 (human-triggered)

Updated yarn.lock files in bulk-import and translations workspaces to include @backstage/cli-defaults dependency. Root yarn.lock could not be updated due to a missing Yarn patch file for got@11.8.2 — this requires human investigation.

Fixed (2):

  1. Run yarn install in touched workspaces to update yarn.lock (workspaces/bulk-import/yarn.lock): Ran yarn install in bulk-import workspace to add @backstage/cli-defaults entry to yarn.lock
  2. Run yarn install in touched workspaces to update yarn.lock (workspaces/translations/yarn.lock): Ran yarn install in translations workspace to add @backstage/cli-defaults entry to yarn.lock

Disagreed (1):

  1. Run yarn install in the root to update yarn.lock: Root yarn install fails because @backstage/cli@^0.36.5 introduces a transitive dependency on got@11.8.2 via Yarn patch protocol, but the required patch file (.yarn/patches/got-npm-11.8.2-c1eb105458.patch) does not exist in this repository. This patch file originates from the Backstage monorepo and is not shipped with any published npm package. A human needs to either: (1) copy the patch file from the backstage/backstage repo, (2) add a resolution to avoid the patched got version, or (3) investigate which transitive dependency pulls in the patched got reference.

Tests: passed

Decision points
  • Root yarn.lock update blocked by missing got patch file (alternatives: Copy patch file from backstage/backstage repo, Add a yarn resolution to avoid the patched got version, Skip root yarn.lock update and document the issue; rationale: The sandbox cannot fetch from GitHub to retrieve the patch file. The got patch is a transitive dependency introduced by @backstage/cli@0.36.5 that expects a .yarn/patches/ file that does not exist in this repo. Committed the workspace lockfile updates and documented the root issue for human follow-up.)

Next steps:

  • /fs-review — request a re-review of the changes
  • /fs-fix <your instruction> — run another fix pass with specific guidance
  • Push commits directly — review re-runs automatically on push
    Updated by fullsend fix agent

@christoph-jerolimov

Copy link
Copy Markdown
Member

/fs-fix run yarn install in the root to update yarn.lock file

@fullsend-ai-coder

fullsend-ai-coder Bot commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor Author

🤖 Finished Fix · ✅ Success · Started 5:09 PM UTC · Completed 5:33 PM UTC

Commit: ad8e7a6 · View workflow run →

Runtime: claude · Model: opus → claude-opus-4-6 · Effort: high · Cost: $5.47

Run yarn install at the root to update yarn.lock after
@backstage/cli upgrade to ^0.36.5. Add the got@11.8.2
compatibility patch required by Yarn Berry's built-in
dependency resolution (sourced from yarnpkg/berry).

Addresses #5037

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@fullsend-ai-coder

Copy link
Copy Markdown
Contributor Author

🔧 Fix agent — iteration 2 (human-triggered)

Updated root yarn.lock by running yarn install after @backstage/cli version bump. Added required got@11.8.2 compatibility patch from the yarnpkg/berry repository to resolve Yarn Berry dependency resolution failure.

Fixed (1):

  1. Run yarn install in the root to update yarn.lock file (yarn.lock): Ran yarn install at the repo root to update yarn.lock after the @backstage/cli upgrade from ^0.27.0 to ^0.36.5. This required first adding the got@11.8.2 compatibility patch (.yarn/patches/got-npm-11.8.2-c1eb105458.patch) that Yarn Berry's built-in dependency resolution expects. The patch was sourced from the yarnpkg/berry repository and fixes a promise settlement race condition in the got HTTP library.

Tests: passed

Decision points
  • Add got@11.8.2 compatibility patch required by Yarn Berry (alternatives: Skip the patch and find an alternative resolution, Add the patch from the yarnpkg/berry repository; rationale: Yarn Berry's built-in dependency resolution requires this patch for got@11.8.2. The patch is part of Yarn's compatibility database and was sourced from the official yarnpkg/berry repository. Without it, yarn install fails during the resolution step.)

Next steps:

  • /fs-review — request a re-review of the changes
  • /fs-fix <your instruction> — run another fix pass with specific guidance
  • Push commits directly — review re-runs automatically on push
    Updated by fullsend fix agent

@sonarqubecloud

Copy link
Copy Markdown

@codecov

codecov Bot commented Sep 29, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.66%. Comparing base (bd06342) to head (056baf8).
⚠️ Report is 4 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5037   +/-   ##
=======================================
  Coverage   63.66%   63.66%           
=======================================
  Files        2708     2708           
  Lines      106621   106621           
  Branches    30060    30047   -13     
=======================================
  Hits        67882    67882           
  Misses      36911    36911           
  Partials     1828     1828           
Flag Coverage Δ *Carryforward flag
adoption-insights 84.77% <ø> (ø) Carriedforward from bd06342
ai-integrations 87.10% <ø> (ø) Carriedforward from bd06342
app-defaults 68.90% <ø> (ø) Carriedforward from bd06342
augment 46.67% <ø> (ø) Carriedforward from bd06342
boost 92.39% <ø> (ø) Carriedforward from bd06342
bulk-import 73.12% <ø> (ø)
cost-management 13.56% <ø> (ø) Carriedforward from bd06342
dcm 74.29% <ø> (ø) Carriedforward from bd06342
e2e-adoption-insights 60.00% <ø> (ø) Carriedforward from bd06342
e2e-extensions 62.31% <ø> (ø) Carriedforward from bd06342
e2e-global-header 52.40% <ø> (ø) Carriedforward from bd06342
e2e-homepage 61.11% <ø> (ø) Carriedforward from bd06342
e2e-intelligent-assistant 45.51% <ø> (ø) Carriedforward from bd06342
e2e-orchestrator 49.49% <ø> (ø) Carriedforward from bd06342
e2e-orchestrator-plugin 49.48% <ø> (ø) Carriedforward from bd06342
e2e-quickstart 54.83% <ø> (ø) Carriedforward from bd06342
e2e-scorecard 49.77% <ø> (ø) Carriedforward from bd06342
e2e-theme 16.36% <ø> (ø) Carriedforward from bd06342
extensions 58.30% <ø> (ø) Carriedforward from bd06342
global-floating-action-button 71.18% <ø> (ø) Carriedforward from bd06342
global-header 69.10% <ø> (ø) Carriedforward from bd06342
homepage 55.05% <ø> (ø) Carriedforward from bd06342
install-dynamic-plugins 73.52% <ø> (ø) Carriedforward from bd06342
intelligent-assistant 78.53% <ø> (ø) Carriedforward from bd06342
konflux 91.98% <ø> (ø) Carriedforward from bd06342
lightspeed 69.02% <ø> (ø) Carriedforward from bd06342
mcp-integrations 84.46% <ø> (ø) Carriedforward from bd06342
orchestrator 77.69% <ø> (ø) Carriedforward from bd06342
quickstart 65.83% <ø> (ø) Carriedforward from bd06342
sandbox 79.56% <ø> (ø) Carriedforward from bd06342
scorecard 88.98% <ø> (ø) Carriedforward from bd06342
theme 87.70% <ø> (ø) Carriedforward from bd06342
translations 7.91% <ø> (ø) Carriedforward from bd06342
x2a 78.48% <ø> (ø) Carriedforward from bd06342

*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 bd06342...056baf8. 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.

Update @backstage/cli

1 participant