Skip to content

chore: upgrade all dependencies with vulnerabilities#5183

Merged
timotheeguerin merged 3 commits intomainfrom
copilot/upgrade-dependencies-with-vulnerabilities
Apr 9, 2026
Merged

chore: upgrade all dependencies with vulnerabilities#5183
timotheeguerin merged 3 commits intomainfrom
copilot/upgrade-dependencies-with-vulnerabilities

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 9, 2026

Upgrades direct dependencies in package.json files to fix transitive vulnerability issues, and refreshes the lockfile via conservative rush update.

Changes Made

  • copy-webpack-plugin ^13.0.0^14.0.0 in packages/apps/autorest/package.json and packages/extensions/core/package.json (fixes serialize-javascript RCE vulnerability)
  • node-gyp ^10.0.1^11.0.0 in packages/tools/compare/package.json (fixes tar path traversal vulnerabilities)
  • Regenerated pnpm-lock.yaml using conservative rush update (not --full) to preserve TypeSpec dependency versions and avoid breaking openapi-to-typespec tests

Vulnerabilities Fixed

Package Old Version New Version Vulnerability
serialize-javascript 6.0.2 7.0.5 RCE via RegExp.flags
tar 6.2.1 7.4.3 Path Traversal (multiple CVEs)

Testing

  • ✅ Build passes (all 22 operations successful)
  • ✅ oai2-to-oai3 and core unit tests pass
  • ✅ openapi-to-typespec generate:validate produces clean output
  • ✅ Code review passed
  • ✅ CodeQL security scan passed

…rush changelogs

Agent-Logs-Url: https://github.com/Azure/autorest/sessions/1c294d8f-ce05-4ec6-b67d-019c2b83fdfc

Co-authored-by: timotheeguerin <1031227+timotheeguerin@users.noreply.github.com>
@azure-pipelines
Copy link
Copy Markdown

You may test this build by running autorest --reset and then either:


Add the following CLI flags
Pacakge Flag Description
@autorest/core --version:https://tinyurl.com/2c2et3nx For changes to autorest core.
@autorest/modelerfour --use:https://tinyurl.com/2adj27yv For changes to modelerfour.

Or with all

autorest --version:https://tinyurl.com/2c2et3nx --use:https://tinyurl.com/2adj27yv

or use the following in your autorest configuration:
# For changes to autorest core
version: "https://tinyurl.com/2c2et3nx"

# For changes to modelerfour
use-extension:
  "@autorest/modelerfour": "https://tinyurl.com/2adj27yv"

If this build is good for you, give this comment a thumbs up. (👍) And you should run `autorest --reset` again once you're finished testing to remove it.

@timotheeguerin
Copy link
Copy Markdown
Member

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@timotheeguerin
Copy link
Copy Markdown
Member

@copilot the test are failing in openapi converter

…sions

Agent-Logs-Url: https://github.com/Azure/autorest/sessions/8c4d4647-3836-4fc0-a85f-c7dc07c08002

Co-authored-by: timotheeguerin <1031227+timotheeguerin@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Apr 9, 2026

@copilot the test are failing in openapi converter

The issue was that rush update --full re-resolved all transitive dependencies, bumping @typespec/compiler from 1.5.0 to 1.11.0 and @azure-tools/typespec-azure-resource-manager from 0.61.0 to 0.61.1, which introduced breaking template changes in private-endpoints.tsp. Fixed by using conservative rush update --purge instead to only update what's needed for the direct dependency changes while preserving TypeSpec versions. Commit: 26bbccd

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • dev.azure.com
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node /home/REDACTED/work/_temp/ghcca-node/node/bin/node --enable-source-maps /home/REDACTED/work/_temp/copilot-developer-action-main/dist/index.js (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

@azure-pipelines
Copy link
Copy Markdown

You may test this build by running autorest --reset and then either:


Add the following CLI flags
Pacakge Flag Description
@autorest/core --version:https://tinyurl.com/2bztwegr For changes to autorest core.
@autorest/modelerfour --use:https://tinyurl.com/265zrh64 For changes to modelerfour.

Or with all

autorest --version:https://tinyurl.com/2bztwegr --use:https://tinyurl.com/265zrh64

or use the following in your autorest configuration:
# For changes to autorest core
version: "https://tinyurl.com/2bztwegr"

# For changes to modelerfour
use-extension:
  "@autorest/modelerfour": "https://tinyurl.com/265zrh64"

If this build is good for you, give this comment a thumbs up. (👍) And you should run `autorest --reset` again once you're finished testing to remove it.

@timotheeguerin timotheeguerin marked this pull request as ready for review April 9, 2026 12:49
@timotheeguerin timotheeguerin enabled auto-merge (squash) April 9, 2026 12:49
@azure-pipelines
Copy link
Copy Markdown

You may test this build by running autorest --reset and then either:


Add the following CLI flags
Pacakge Flag Description
@autorest/core --version:https://tinyurl.com/28caxvca For changes to autorest core.
@autorest/modelerfour --use:https://tinyurl.com/2dbufl9g For changes to modelerfour.

Or with all

autorest --version:https://tinyurl.com/28caxvca --use:https://tinyurl.com/2dbufl9g

or use the following in your autorest configuration:
# For changes to autorest core
version: "https://tinyurl.com/28caxvca"

# For changes to modelerfour
use-extension:
  "@autorest/modelerfour": "https://tinyurl.com/2dbufl9g"

If this build is good for you, give this comment a thumbs up. (👍) And you should run `autorest --reset` again once you're finished testing to remove it.

@timotheeguerin timotheeguerin merged commit 13efbce into main Apr 9, 2026
11 checks passed
@timotheeguerin timotheeguerin deleted the copilot/upgrade-dependencies-with-vulnerabilities branch April 9, 2026 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants