Repository navigation
DYN-10892 Update dependencies for GL 4.3 - #130
Merged
Merged
Conversation
… 24, refresh dependencies (npm update + npm audit fix) and regenerate license output.
DYN-10892: release version for the GL 4.3 dependency refresh. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
edwin-vasquez-ucaldas
requested review from
RobertGlobant20 and
jasonstratton
October 13, 2026 15:28
3 tasks
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.
Purpose
DYN-10892: Annual pre-Global-Launch refresh of NotificationCenter for Dynamo 4.3 (Civil 3D 2028 GL), following last year's #77.
Last cycle, 0.0.33 was published but Dynamo stayed on 0.0.31 (fixed in Dynamo #17382). This release (0.0.34) will also carry the ~70 commits merged since 0.0.33, including axios 1.20.0 and webpack-dev-server 6 / webpack-cli 7 (#120).
Changes
build.ymland in both jobs ofnpm-publish.yml.npm update+npm audit fix(lockfile only, about 18k lines removed through deduplication). EveryresolvedURL still points atregistry.npmjs.org.terser-webpack-plugindeclared as a devDependency (^5.6.1).webpack.config.jsrequires it but never declared it. webpack 5.111 no longer pulls it in.overrides: { "cheerio": "1.0.0-rc.12" }. The unit test uses Enzyme 3.11 (unmaintained), which requirescheerio/lib/utils. cheerio removed that in 1.x final releases, andnpm updatehad moved it to 1.2.0, breakingtests/unit/App.test.js. The override keeps the version that was already in the lockfile. It's test-only.npm run generate_license).Security
The count barely moves because nearly everything left sits behind breaking majors in test tooling: Jest 28 → 30 (30 advisories), Puppeteer 22 → 25 (8) and jest-junit 14 → 17. For a few packages
npm audit fix --forceproposes downgrades, such aswebpack-dev-server@1.14.1andwebpack-cli@6, so it must not be used. Tracked in DYN-11026.npm audit fixonly runs here with--legacy-peer-deps, because@dynamods/notifications-panel@0.0.10declares a React 17 peer dependency (the HIG/Weave issue noted in #77). With the flag, it reports nothing further to fix.Runtime impact
react/react-dom18.2.0 → 18.3.1.sass. They're notices only, not errors.Testing
Run locally on Node 24.15 / npm 11.12 from a clean
npm ci --force:npm run build✅npm run test: 10/10 suites (lint, unit, e2e/Puppeteer) ✅npm run production✅Release
npm version patch --no-git-tag-version. Avoidnpm run version:patch: itspostversionhook auto-commits.v0.0.34to publish to npmsrc/Notifications/Notifications.csprojfrom 0.0.33 (#17382) to 0.0.34🤖 Generated with Claude Code