Bump tar, @capacitor/cli and electron-builder - #11
Conversation
Bumps [tar](https://github.com/isaacs/node-tar) to 7.5.16 and updates ancestor dependencies [tar](https://github.com/isaacs/node-tar), [@capacitor/cli](https://github.com/ionic-team/capacitor) and [electron-builder](https://github.com/electron-userland/electron-builder/tree/HEAD/packages/electron-builder). These dependencies need to be updated together. Updates `tar` from 6.2.1 to 7.5.16 - [Release notes](https://github.com/isaacs/node-tar/releases) - [Changelog](https://github.com/isaacs/node-tar/blob/main/CHANGELOG.md) - [Commits](isaacs/node-tar@v6.2.1...v7.5.16) Updates `@capacitor/cli` from 6.2.1 to 8.4.0 - [Release notes](https://github.com/ionic-team/capacitor/releases) - [Changelog](https://github.com/ionic-team/capacitor/blob/main/CHANGELOG.md) - [Commits](ionic-team/capacitor@6.2.1...8.4.0) Updates `electron-builder` from 24.13.3 to 26.15.3 - [Release notes](https://github.com/electron-userland/electron-builder/releases) - [Changelog](https://github.com/electron-userland/electron-builder/blob/master/packages/electron-builder/CHANGELOG.md) - [Commits](https://github.com/electron-userland/electron-builder/commits/electron-builder@26.15.3/packages/electron-builder) --- updated-dependencies: - dependency-name: tar dependency-version: 7.5.16 dependency-type: indirect - dependency-name: "@capacitor/cli" dependency-version: 8.4.0 dependency-type: direct:development - dependency-name: electron-builder dependency-version: 26.15.3 dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com>
Atlas Commons PR review
Overall: fail
View PR #11 · |
Atlas Commons code review (LLM)SummaryDependabot PR #11 groups a tar 6 → 7.5.16 security bump (via lockfile) with direct upgrades of The electron-builder and tar updates look directionally correct for security and build tooling. The Capacitor CLI 8 bump is the main concern: runtime packages ( Potential issues
"devDependencies": {
"@capacitor/cli": "^8.4.0"
},
"dependencies": {
"@capacitor/android": "^6.2.1",
"@capacitor/core": "^6.2.1"
}
Suggestions
Overall assessmentThe tar and electron-builder updates address real security/tooling debt, and removing the release workflow’s However, bumping Advisory review only — not a merge approval. View PR #11 · |
|
/atlas fix |
|
Applying review fixes on this PR branch now — may take several minutes. |
…n-builder in release CI Revert @capacitor/cli to ^6.2.1 so it matches @capacitor/core and @capacitor/android on v6. Keeping electron-builder at ^26.15.3 preserves tar 7.5.16 on the desktop packaging path while avoiding the fragile CLI 8 / runtime 6 skew flagged in review. Remove release.yml steps that reinstalled electron-builder@^24 after npm ci, which would undo the package.json bump on Linux and Windows release builds. Signed-off-by: Cursor Agent <cursoragent@cursor.com> Co-authored-by: Amateur-God <Amateur-God@users.noreply.github.com>
Atlas Commons auto-fixSummaryPer the maintainer instruction, I undid the prior revert and restored the original Dependabot scope: I kept the release.yml fix from the earlier commit (removed the steps that reinstalled From the review, I also added CI coverage for Local validation passed: Commits
Files touched
Notes
Git
Automated fix attempt — review the diff and CI before merging. Cursor run |
|
/atlas fix undo revert and go back to the original PR |
|
Applying review fixes with your additional instructions — may take several minutes. |
Undo the @capacitor/cli revert so the PR matches Dependabot's original tar/@capacitor/cli/electron-builder bump. Keep the release.yml fix that stops reinstalling electron-builder@^24 after npm ci. Add npm run pack and npx cap sync android to CI, align Android Java to 21 for Capacitor CLI 8, and update release/android workflows accordingly. Signed-off-by: Cursor Agent <cursoragent@cursor.com> Co-authored-by: Amateur-God <Amateur-God@users.noreply.github.com>
|
/atlas llm |
|
Running the LLM code review now. |
Bumps tar to 7.5.16 and updates ancestor dependencies tar, @capacitor/cli and electron-builder. These dependencies need to be updated together.
Updates
tarfrom 6.2.1 to 7.5.16Changelog
Sourced from tar's changelog.
... (truncated)
Commits
cf213387.5.1621a8220do not apply PAX header fields to meta entries52632cfupdate project deps302f51ffix inconsequential typo in PENDINGLINKS symbol name55dbb99remove some uses of mutate-fs87cc3097.5.157aef486fix: regression in pending links detection6244eb37.5.149704d8cstricter protection against hardlinks preempting their targets700734fupdate workflows and depsMaintainer changes
This version was pushed to npm by isaacs, a new releaser for tar since your current version.
Install script changes
This version adds
preparescript that runs during installation. Review the package contents before updating.Updates
@capacitor/clifrom 6.2.1 to 8.4.0Release notes
Sourced from @capacitor/cli's releases.
... (truncated)
Changelog
Sourced from @capacitor/cli's changelog.
... (truncated)
Commits
41fd9deRelease 8.4.036b729dchore: Decrease timeout for CI jobs from 60 to 30 minutes (#8476)4c6c321fix(android): show only the requested system bar (#8480)d4ad7fffix(SystemBars): respectinsetsHandlingdisable (#8481)f1077efchore: fix typo in declarations.ts (#8488)1d031a4fix(cli): revert live reload config on failure (#8485)b4b297afeat(cli): capture ios_package_manager in telemetry (#8482)258867bfeat(cli): add experimental packageOptions (#8471)e456de0fix(SystemBars): makesafe-area-inset-xavailable on API <= 34 (#8424)93c72defeat: add method getDouble to plugin config (#7638)Maintainer changes
This version was pushed to npm by capacitor-plugin-bot, a new releaser for
@capacitor/clisince your current version.Updates
electron-builderfrom 24.13.3 to 26.15.3Release notes
Sourced from electron-builder's releases.
... (truncated)
Changelog
Sourced from electron-builder's changelog.
... (truncated)
Commits
512a57echore(deploy): Release v26.15.3 (#9858)a6117b3chore(deploy): Release v26.15.2 (#9848)d57f094chore(deploy): Release v26.15.1 (#9842)bed3a9cchore(deploy): Release v26.15.0 (electron-updater@6.8.9) (#9825)198c10cchore: replace app-builder-binnode-dep-treeandrebuild-node-modules(#9...e236392chore(deploy): Release v26.14.0 (#9812)72d298cchore(deploy): Release v26.13.1 (#9805)ffd11c7chore(deploy): Release v26.13.0 (electron-updater@6.8.8) (#9793)2c8c71achore(deploy): Release v26.12.1 (electron-updater@6.8.7) (#9782)d6a5aeefix: harden generated-file output, argument construction, and download valida...Maintainer changes
This version was pushed to npm by GitHub Actions, a new releaser for electron-builder 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.