Skip to content

fix(dashboard): sync pnpm lockfile overrides#8637

Merged
Soulter merged 1 commit into
AstrBotDevs:masterfrom
EterUltimate:codex/fix-dashboard-lockfile-overrides-20260607
Jun 7, 2026
Merged

fix(dashboard): sync pnpm lockfile overrides#8637
Soulter merged 1 commit into
AstrBotDevs:masterfrom
EterUltimate:codex/fix-dashboard-lockfile-overrides-20260607

Conversation

@EterUltimate
Copy link
Copy Markdown
Contributor

@EterUltimate EterUltimate commented Jun 7, 2026

Summary

  • Regenerate dashboard/pnpm-lock.yaml so it records the pnpm.overrides declared in dashboard/package.json.
  • Align the transitive lodash-es lockfile entries with the declared lodash-es@4.17.23 override.

Evidence from PR scan

  • Dashboard CI on master commit 6ae103a failed in run 27066032353.
  • The same workflow also blocked unrelated Python-only PR [codex] fix event bus task retention #8636 at run 27079164635, job 79921828789.
  • Failed job log shows ERR_PNPM_LOCKFILE_CONFIG_MISMATCH during pnpm install --frozen-lockfile: current overrides config did not match the lockfile.
  • dashboard/package.json declares pnpm.overrides for immutable and lodash-es, while the previous lockfile lacked top-level overrides.

Checks

pm exec --yes pnpm@10.28.2 -- install --lockfile-only --ignore-scriptsn- pm exec --yes pnpm@10.28.2 -- install --frozen-lockfile --ignore-scriptsn- git diff --check`n

Notes

  • Local full pnpm install --frozen-lockfile progressed past the original lockfile mismatch, then this Windows sandbox blocked the �sbuild postinstall child process with spawn EPERM; CI should validate lifecycle scripts on Ubuntu.

Summary by Sourcery

Regenerate the dashboard pnpm lockfile to sync with the pnpm.overrides configuration and resolve lockfile override mismatches in CI.

Bug Fixes:

  • Fix pnpm lockfile configuration mismatch errors during dashboard CI by aligning the lockfile with declared overrides.

Chores:

  • Update dashboard/pnpm-lock.yaml to record pnpm.overrides for dependencies such as lodash-es and keep transitive entries consistent with the specified versions.

@dosubot dosubot Bot added size:XS This PR changes 0-9 lines, ignoring generated files. area:webui The bug / feature is about webui(dashboard) of astrbot. labels Jun 7, 2026
Copy link
Copy Markdown
Contributor

@sourcery-ai sourcery-ai Bot left a comment

Choose a reason for hiding this comment

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

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the dashboard/pnpm-lock.yaml file to add dependency overrides for immutable (4.3.8) and lodash-es (4.17.23). This upgrade replaces the older lodash-es@4.17.21 version across various package dependencies. There are no review comments to address, and I have no additional feedback to provide.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Jun 7, 2026
@Soulter Soulter merged commit 0db7fc9 into AstrBotDevs:master Jun 7, 2026
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:webui The bug / feature is about webui(dashboard) of astrbot. lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants