Skip to content

build(deps): bump tar from 7.5.16 to 7.5.22 in the npm_and_yarn group across 1 directory - #58

Closed
dependabot[bot] wants to merge 1 commit into
mainfrom
dependabot/npm_and_yarn/npm_and_yarn-e1683985aa
Closed

build(deps): bump tar from 7.5.16 to 7.5.22 in the npm_and_yarn group across 1 directory#58
dependabot[bot] wants to merge 1 commit into
mainfrom
dependabot/npm_and_yarn/npm_and_yarn-e1683985aa

Conversation

@dependabot

@dependabot dependabot Bot commented on behalf of github Jul 27, 2026

Copy link
Copy Markdown

Bumps the npm_and_yarn group with 1 update in the / directory: tar.

Updates tar from 7.5.16 to 7.5.22

Commits

Dependabot compatibility score

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 rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore <dependency name> major version will close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself)
  • @dependabot ignore <dependency name> minor version will close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself)
  • @dependabot ignore <dependency name> will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself)
  • @dependabot unignore <dependency name> will remove all of the ignore conditions of the specified dependency
  • @dependabot unignore <dependency name> <ignore condition> will remove the ignore condition of the specified dependency and ignore conditions
    You can disable automated security fix PRs for this repo from the Security Alerts page.

Bumps the npm_and_yarn group with 1 update in the / directory: [tar](https://github.com/isaacs/node-tar).


Updates `tar` from 7.5.16 to 7.5.22
- [Release notes](https://github.com/isaacs/node-tar/releases)
- [Changelog](https://github.com/isaacs/node-tar/blob/main/CHANGELOG.md)
- [Commits](isaacs/node-tar@v7.5.16...v7.5.22)

---
updated-dependencies:
- dependency-name: tar
  dependency-version: 7.5.22
  dependency-type: indirect
  dependency-group: npm_and_yarn
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot Bot added dependencies Pull requests that update a dependency file javascript Pull requests that update javascript code labels Jul 27, 2026
@usmanabbas7

Copy link
Copy Markdown
Collaborator

Hold this one — the bump is legitimate but the PR does two things the title doesn't say.

1. Only one line of the 432 is the tar bump

$ git diff main HEAD -- yarn.lock | grep -E '^[+-]  resolution:'
-  resolution: "tar@npm:7.5.16"
+  resolution: "tar@npm:7.5.22"

The rest is a lockfile dialect downgrade__metadata.version 10 → 6, cacheKey 10c08, every checksum rewritten from 10c0/169fc208… to bare 169fc208…. Dependabot regenerated the file with an older Yarn than the one that produced main's lockfile.

2. The lockfile is decorative — release CI runs Yarn 1.22.22 against a Berry lockfile

From the last release run (30111489505):

! Corepack is about to download .../yarn-1.22.22.tgz
yarn install v1.22.22
[1/4] Resolving packages...
success Saved lockfile.

success Saved lockfile. on an --immutable install is the tell: --immutable is a Berry flag, Yarn 1's equivalent is --frozen-lockfile, and Yarn 1 ignores the unknown flag. It also can't parse a Berry lockfile, so it floats every ^ range in package.json fresh from the registry — in a job holding GITHUB_TOKEN (push) and PyPI OIDC publishing rights.

Root cause: no packageManager field in package.json, so corepack enable falls back to its default Yarn 1.22.22. .yarnrc.yml (nodeLinker: node-modules) is Berry-only config that Yarn 1 also ignores.

So this PR's tar pin has no effect on what the release job actually installs.

Scope

tar is transitive dev-only tooling — pyproject.toml excludes package.json/yarn.lock/node_modules from the sdist and the wheel target is packages = ["src/convert_sdk"], so there's no published-package exposure. (5 advisories affect 7.5.16 incl. one critical, GHSA-23hp-3jrh-7fpw — worth fixing, not user-facing.) Note the 21 green checks don't cover this: ci.yml is Python-only, no yarn step.

Suggested order

  1. corepack use yarn@4 to write packageManager, and verify yarn install --immutable round-trips main's lockfile unchanged.
  2. Then @dependabot recreate — the tar bump should be a ~4-line diff.
  3. Add a yarn install --immutable job to ci.yml so dialect drift fails a PR instead of passing silently.

Step 1 is worth its own PR regardless — it's a strictly larger hole than the action pinning in #57.

@abbaseya

abbaseya commented Aug 8, 2026

Copy link
Copy Markdown
Collaborator

closed in favor of PR #59

@abbaseya abbaseya closed this Aug 8, 2026
@dependabot @github

dependabot Bot commented on behalf of github Aug 8, 2026

Copy link
Copy Markdown
Author

This pull request was built based on a group rule. Closing it will not ignore any of these versions in future pull requests.

To ignore these dependencies, configure ignore rules in dependabot.yml

@dependabot
dependabot Bot deleted the dependabot/npm_and_yarn/npm_and_yarn-e1683985aa branch August 8, 2026 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file javascript Pull requests that update javascript code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants