Repository navigation
fix(deps): update dependency simple-git to v4 - autoclosed - #85
Closed
renovate[bot] wants to merge 1 commit into
Closed
renovate[bot] wants to merge 1 commit into
renovate[bot] wants to merge 1 commit into
Conversation
|
Important Review skippedBot user detected. To trigger a single review, invoke the ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
renovate
Bot
force-pushed
the
renovate/simple-git-4.x
branch
3 times, most recently
from
October 3, 2026 00:25
c07380c to
774225a
Compare
renovate
Bot
force-pushed
the
renovate/simple-git-4.x
branch
from
October 5, 2026 22:06
774225a to
4597c71
Compare
rishitank
added a commit
that referenced
this pull request
Oct 8, 2026
…fresh - package.json: override tsup's esbuild to ^0.28.1. tsup 8.5.1 asks for ^0.27.0, and GHSA-g7r4-m6w7-qqqr affects esbuild >=0.27.3 <0.28.1, so no 0.27.x fix exists. `npm run build` passes with 0.28.2. - .github/workflows/deps-refresh-once.yml: one-shot runner job running `npm update` within the declared ranges. This clears hono, ip-address, js-yaml, minimatch, nanoid, path-to-regexp, picomatch, postcss, proxy-addr, qs, rollup, sharp, source-map-js and vite. Checked locally on the refreshed lock: typecheck, lint (same 22 warnings as before, 0 errors), all 240 tests and the build pass. Deleted in the next commit. Not fixable here without a major upgrade: simple-git 3 -> 4 (GHSA-858h-whjf-mvg5, GHSA-g4wm-2vf7-vfgr, GHSA-x6jw-m9v5-85vh; Renovate #85/#91) and vitest/@vitest/coverage-v8 3 -> 4.1.11+ (GHSA-82fw-gwwq-j7x9, plus tinypool through vitest; Renovate #74/#82). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_012rYJxu978iUSGFZ1UuqTsi
rishitank
added a commit
that referenced
this pull request
Oct 8, 2026
…; hold TypeScript 7 - simple-git ^3 -> ^4.0.2: GHSA-858h-whjf-mvg5, GHSA-g4wm-2vf7-vfgr, GHSA-x6jw-m9v5-85vh and @simple-git/argv-parser GHSA-v5rq-49vh-5v5c are fixed only in 4.x (Renovate #85 / Dependabot #91 show it is compatible). - vitest and @vitest/coverage-v8 ^3 -> ^4.1.11 together: GHSA-82fw-gwwq-j7x9 (vitest/@vitest/mocker) and tinypool GHSA-5gmw-xhrv-c9v3 / GHSA-85c8-ppgw-ccpr. coverage-v8 peers the exact vitest version, which is why #74 (vitest only) and #82 (coverage-v8 5 only) each fail npm ci. - @eslint/js is imported by eslint.config.js but was never declared; it only resolved because eslint 9 depends on it. ESLint 10 no longer does (#83), so declare it explicitly. - renovate.json: hold TypeScript 7 (typescript-eslint 8.x peers typescript <6.1.0, #84) and group vitest with @vitest/* so they can't split again. - Lockfile refreshed on a runner by the one-shot deps-refresh-once workflow. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_012rYJxu978iUSGFZ1UuqTsi
github-actions Bot
added a commit
that referenced
this pull request
Oct 8, 2026
…simple-git 4 and vitest 4 (#94) * ci: add shared security scan and actionlint workflows - security.yml: osv-scanner and full-history gitleaks via rishitank/.github security.yml@v1 (ecosystem node). Runs alongside the existing CodeQL and npm audit jobs in ci.yml, which stay as they are. - workflows-lint.yml: actionlint via the shared workflow. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_012rYJxu978iUSGFZ1UuqTsi * chore(renovate): extend the house preset instead of config:base config:base is deprecated (renamed config:recommended), and matchPackagePatterns is deprecated in favour of regex entries in matchPackageNames. Switch the base to github>rishitank/.github:node (which already includes config:recommended) and convert the patterns to /regex/ matchPackageNames. Both custom packageRules are kept. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_012rYJxu978iUSGFZ1UuqTsi * chore(deps): override tsup's esbuild; prepare an in-range lockfile refresh - package.json: override tsup's esbuild to ^0.28.1. tsup 8.5.1 asks for ^0.27.0, and GHSA-g7r4-m6w7-qqqr affects esbuild >=0.27.3 <0.28.1, so no 0.27.x fix exists. `npm run build` passes with 0.28.2. - .github/workflows/deps-refresh-once.yml: one-shot runner job running `npm update` within the declared ranges. This clears hono, ip-address, js-yaml, minimatch, nanoid, path-to-regexp, picomatch, postcss, proxy-addr, qs, rollup, sharp, source-map-js and vite. Checked locally on the refreshed lock: typecheck, lint (same 22 warnings as before, 0 errors), all 240 tests and the build pass. Deleted in the next commit. Not fixable here without a major upgrade: simple-git 3 -> 4 (GHSA-858h-whjf-mvg5, GHSA-g4wm-2vf7-vfgr, GHSA-x6jw-m9v5-85vh; Renovate #85/#91) and vitest/@vitest/coverage-v8 3 -> 4.1.11+ (GHSA-82fw-gwwq-j7x9, plus tinypool through vitest; Renovate #74/#82). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_012rYJxu978iUSGFZ1UuqTsi * chore(deps): refresh lockfile to pick up patched versions Ran on a runner by the one-shot deps-refresh-once workflow: npm update --package-lock-only --ignore-scripts --no-audit --no-fund * ci: remove the one-shot deps-refresh-once workflow It refreshed package-lock.json in ed5ae35 (`npm update` within the declared ranges, tsup's esbuild override applied). This commit is what runs CI against the refreshed lockfile. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_012rYJxu978iUSGFZ1UuqTsi * fix(deps): simple-git 4, vitest + coverage-v8 4.1, declare @eslint/js; hold TypeScript 7 - simple-git ^3 -> ^4.0.2: GHSA-858h-whjf-mvg5, GHSA-g4wm-2vf7-vfgr, GHSA-x6jw-m9v5-85vh and @simple-git/argv-parser GHSA-v5rq-49vh-5v5c are fixed only in 4.x (Renovate #85 / Dependabot #91 show it is compatible). - vitest and @vitest/coverage-v8 ^3 -> ^4.1.11 together: GHSA-82fw-gwwq-j7x9 (vitest/@vitest/mocker) and tinypool GHSA-5gmw-xhrv-c9v3 / GHSA-85c8-ppgw-ccpr. coverage-v8 peers the exact vitest version, which is why #74 (vitest only) and #82 (coverage-v8 5 only) each fail npm ci. - @eslint/js is imported by eslint.config.js but was never declared; it only resolved because eslint 9 depends on it. ESLint 10 no longer does (#83), so declare it explicitly. - renovate.json: hold TypeScript 7 (typescript-eslint 8.x peers typescript <6.1.0, #84) and group vitest with @vitest/* so they can't split again. - Lockfile refreshed on a runner by the one-shot deps-refresh-once workflow. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_012rYJxu978iUSGFZ1UuqTsi * chore(deps): refresh lockfile to pick up patched versions Ran on a runner by the one-shot deps-refresh-once workflow: npm install --package-lock-only --ignore-scripts --no-audit --no-fund * ci: remove the one-shot deps-refresh-once workflow Its job is done: a5e9414 committed the lockfile resolved on a runner. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_012rYJxu978iUSGFZ1UuqTsi --------- Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
auto-merge was automatically disabled
October 8, 2026 23:16
Pull request was closed
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.
This PR contains the following updates:
^3.0.0→^4.0.0Release Notes
steveukx/git-js (simple-git)
v4.0.2Compare Source
Patch Changes
68874c2: AddVISUALenvironment variable to set ofallowUnsafeEditorenvironment variables.Thanks to @oss-security-shopify for identifying the vulnerability.
Updated dependencies [
68874c2]v4.0.1Compare Source
Patch Changes
365f52d: Prepare package.json before publishing.v4.0.0Compare Source
Major Changes
98864c6: Major upgrade to v4. In this version:simpleGitexport.simple-git/promise(change to using the mainsimple-gitimport).gitPexport (change to using the mainsimpleGitexport).gitoptions:Thanks to @anir0y, @CFionaBF, @Leeziao, @internetteletubbie, @idisdi, @the-vibe-dev,
@D7EAD, @dellalibera, @gdegrange, @bilguunbicktivism, @cruzryan, @b1ue0ceanRun, @oss-security-shopify
for identifying the vulnerability.
gitchild process.simpleGit.silent()logging is configured through environment variables in thedebugpackagesimpleGit.clearQueue()this has been a noop since v3, switch to using theabortpluginGitResponseErrorthrough a trailing callback function are available only through theerror.gitproperty (previously properties were also spread onto theerroritself with a deprecation notice).Minor Changes
98864c6: Support one-shot stdin viagit.input(data)(stringorBuffer).Thanks to @felipecrs for the feature request and initial implementation.
Patch Changes
d762810: AddallowUnsafeExecdetection torebase -xandrebase --exec.Thanks to @gdegrange for the vulnerability report.
76f308e: Parse binary rename entries without byte counts in diff summaries.Updated dependencies [
c427fba]Updated dependencies [
1bb14df]Updated dependencies [
dfeb116]Updated dependencies [
d762810]Updated dependencies [
d762810]Updated dependencies [
98864c6]Configuration
📅 Schedule: (UTC)
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.