Skip to content

Upgrade to copy-webpack-plugin v14#5417

Merged
tdonohue merged 1 commit intoDSpace:mainfrom
tdonohue:update_copy-webpack-plugin
Apr 7, 2026
Merged

Upgrade to copy-webpack-plugin v14#5417
tdonohue merged 1 commit intoDSpace:mainfrom
tdonohue:update_copy-webpack-plugin

Conversation

@tdonohue
Copy link
Copy Markdown
Member

@tdonohue tdonohue commented Apr 7, 2026

References

Description

Upgrades us to the latest version of copy-webpack-plugin (version 14) which does not use tar anymore. The version of copy-webpack-plugin we currently use is five years old. So, upgrading also removes a large number of other older dependencies.

This upgrade was achieved by running:

npm install copy-webpack-plugin@latest -include=dev

This should probably also be backported to DSpace 9.x. However, it cannot be backported to DSpace 8.x because the latest version of copy-webpack-plugin requires Node 20+.

Instructions for Reviewers

  • Verify build passes and all tests succeed
  • Verify production and development builds still work properly
    • NOTE: I've already verified production build via npm run build:prod && npm run serve:ssr. No change in behavior.

@tdonohue tdonohue added this to the 10.0 milestone Apr 7, 2026
@tdonohue tdonohue added dependencies Pull requests that update a dependency file high priority 1 APPROVAL pull request only requires a single approval to merge port to dspace-9_x This PR needs to be ported to `dspace-9_x` branch for next bug-fix release security Security related fix labels Apr 7, 2026
Copy link
Copy Markdown
Contributor

@nwoodward nwoodward left a comment

Choose a reason for hiding this comment

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

Everything is working fine for me with npm run start:dev and npm run serve:ssr. I also tested IIIF using Mirador with npm run start:mirador:prod and didn't notice any new issues with the viewer. I'm +1.

@github-project-automation github-project-automation bot moved this to 👍 Reviewer Approved in DSpace 10.0 Release Apr 7, 2026
@tdonohue tdonohue merged commit c69bd3a into DSpace:main Apr 7, 2026
16 checks passed
@github-project-automation github-project-automation bot moved this from 👍 Reviewer Approved to ✅ Done in DSpace 10.0 Release Apr 7, 2026
@dspace-bot
Copy link
Copy Markdown
Contributor

Backport failed for dspace-9_x, because it was unable to cherry-pick the commit(s).

Please cherry-pick the changes locally and resolve any conflicts.

git fetch origin dspace-9_x
git worktree add -d .worktree/backport-5417-to-dspace-9_x origin/dspace-9_x
cd .worktree/backport-5417-to-dspace-9_x
git switch --create backport-5417-to-dspace-9_x
git cherry-pick -x 8f0d3caf4c9b77fdc985739b0f5e10ab4ec9669d

@tdonohue tdonohue deleted the update_copy-webpack-plugin branch April 7, 2026 22:03
@tdonohue tdonohue removed the port to dspace-9_x This PR needs to be ported to `dspace-9_x` branch for next bug-fix release label Apr 7, 2026
@tdonohue
Copy link
Copy Markdown
Member Author

tdonohue commented Apr 7, 2026

Manually ported to 9.x in #5427

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1 APPROVAL pull request only requires a single approval to merge dependencies Pull requests that update a dependency file high priority security Security related fix

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

3 participants