Update undici artifacts to 6.27.0 (license cache + dist)#1040
Merged
Conversation
The Licensed check failed because the cached license record for undici was pinned to 6.24.1 while the installed dependency is 6.27.0, causing "license: mit, allowed: false" / source enumeration errors. Regenerate the cached record with `licensed cache` so it matches the installed version. `licensed status` now reports 0 errors. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the repository’s cached Licensed record for the undici npm dependency to match the currently installed version, unblocking the Licensed / Check licenses workflow.
Changes:
- Regenerated
.licenses/npm/undici.dep.ymlto updateundicifrom6.24.1to6.27.0. - Refreshed the cached license metadata content for that dependency record.
Show a summary per file
| File | Description |
|---|---|
.licenses/npm/undici.dep.yml |
Updates the cached Licensed record to undici@6.27.0 so license checks align with the installed dependency version. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 1/1 changed files
- Comments generated: 0
The committed dist/ bundle was built with undici 6.24.1, but the lockfile resolves undici 6.27.0. The check-dist workflow rebuilds the bundle and detected this drift (uncommitted changes after build). Rebuild dist/setup and dist/cleanup with `npm run build` so the committed bundle matches the installed undici 6.27.0, aligning with the license cache update in this PR. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
brunoborges
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
The Licensed / Check licenses workflow fails on
main(and on PRs such as #1010) with:The cached license record for
undiciwas pinned to6.24.1while the installed dependency is6.27.0, solicensed statusfailed source enumeration.While fixing this, the Check dist/ workflow also surfaced pre-existing drift: the committed
dist/bundle was built withundici 6.24.1, but the lockfile resolvesundici 6.27.0, so a freshnpm run buildproduced uncommitted changes.Fix
licensed cache(v3.9.0) so it matches the installed version. Only.licenses/npm/undici.dep.ymlchanged (version: 6.24.1→6.27.0and refreshed content).licensed statusnow reports53 dependencies checked, 0 errors found.npm run buildto rebuilddist/setup/index.jsanddist/cleanup/index.jsagainst the installedundici 6.27.0(bundled llhttp parser changes only). A clean rebuild now yields 0 diff, socheck-distpasses.Both changes stem from the same
undici 6.24.1→6.27.0bump and are intentionally scoped separately from the Azul Metadata API migration (#1010).