Bump shell-quote and concurrently#597
Conversation
Bumps [shell-quote](https://github.com/ljharb/shell-quote) to 1.8.4 and updates ancestor dependency [concurrently](https://github.com/open-cli-tools/concurrently). These dependencies need to be updated together. Updates `shell-quote` from 1.8.3 to 1.8.4 - [Changelog](https://github.com/ljharb/shell-quote/blob/main/CHANGELOG.md) - [Commits](ljharb/shell-quote@v1.8.3...v1.8.4) Updates `concurrently` from 9.2.1 to 10.0.3 - [Release notes](https://github.com/open-cli-tools/concurrently/releases) - [Commits](open-cli-tools/concurrently@v9.2.1...v10.0.3) --- updated-dependencies: - dependency-name: shell-quote dependency-version: 1.8.4 dependency-type: indirect - dependency-name: concurrently dependency-version: 10.0.3 dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com>
There was a problem hiding this comment.
🤖 Codex review: Security review looks good to me.
I checked the PR diff plus the npm-published artifact diffs for shell-quote, concurrently, and the changed transitive yargs/chalk/string-width stack. The shell-quote change is defensive validation for object-token quoting. I did not find new install lifecycle hooks, network access, credential/env harvesting, dynamic code execution, or suspicious filesystem behavior.
The notable concurrently changes are expected for v10: ESM-only packaging, Node >=22, shell selection via --shell/npm_config_script_shell, and CLI-only quote normalization. This repo already requires Node >=24.14.1, and the lockfile integrity values for the direct upgraded packages match npm registry metadata. Approving from a security-review perspective.
Bumps shell-quote to 1.8.4 and updates ancestor dependency concurrently. These dependencies need to be updated together.
Updates
shell-quotefrom 1.8.3 to 1.8.4Changelog
Sourced from shell-quote's changelog.
Commits
ff166e2v1.8.44378a6e[Fix]quote: validate object-token shapes22ebec0[Dev Deps] update@ljharb/eslint-config,auto-changelog,eslint, `npmig...9f3caa3[Tests] increase coverage3344a04[readme] replace runkit CI badge with shields.io check-runs badge699c511[Dev Deps] update@ljharb/eslint-configUpdates
concurrentlyfrom 9.2.1 to 10.0.3Release notes
Sourced from concurrently's releases.
... (truncated)
Commits
435f61b10.0.35ea69c6ci: use node 24 in the release workflow18e128110.0.2e70686f10.0.1a95bcebRename flow-controller{.d -> }.tsced4245ci: configure trusted publisher flowcf2eaa210.0.01b9bae4deps: upgrade yargs to v18 (#593)b05ee75Bump min Node.js version to v22ae60bc4Scope quote normalization to CLI input (#585)Maintainer changes
This version was pushed to npm by GitHub Actions, a new releaser for concurrently since your current version.
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)You can disable automated security fix PRs for this repo from the Security Alerts page.