Skip to content

[yauzl] Update to 3.3; keep 2.x as v2#75069

Open
u1f992 wants to merge 1 commit into
DefinitelyTyped:masterfrom
u1f992:yauzl-3
Open

[yauzl] Update to 3.3; keep 2.x as v2#75069
u1f992 wants to merge 1 commit into
DefinitelyTyped:masterfrom
u1f992:yauzl-3

Conversation

@u1f992
Copy link
Copy Markdown
Contributor

@u1f992 u1f992 commented Jun 2, 2026

If changing an existing definition:

  • Provide a URL to documentation or source code which provides context for the suggested changes: https://github.com/thejoshwolfe/yauzl/blob/3.3.2/README.md
  • If this PR brings the type definitions up to date with a new version of the JS library, update the version number in the package.json.

Updates the declarations to track yauzl 3.3.x (was 2.10.x).

Added in 3.x:

  • getFileNameLowLevel, parseExtraFields
  • LocalFileHeader
  • ZipFile.readLocalFileHeader, ZipFile.openReadStreamLowLevel
  • ZipFileOptions.decodeFileData
  • Entry.canDecodeFileData(), Entry.getLastModDate(options), Entry.fileComment, and Entry.fileNameRaw / extraFieldRaw / fileCommentRaw

Marked isCompressed, decompress, decrypt, and dosDateTimeToDate as deprecated.

Version bumped to 3.3.9999. The 2.10.x declarations are kept under v2/ (2.10.9999).

@typescript-bot
Copy link
Copy Markdown
Contributor

typescript-bot commented Jun 2, 2026

@u1f992 Thank you for submitting this PR!

This is a live comment that I will keep updated.

1 package in this PR

Code Reviews

Because this is a widely-used package, a DT maintainer will need to review it before it can be merged.

You can test the changes of this PR in the Playground.

Status

  • ✅ No merge conflicts
  • ✅ Continuous integration tests have passed
  • 🕐 Most recent commit is approved by a DT maintainer

Once every item on this list is checked, I'll ask you for permission to merge and publish the changes.


Diagnostic Information: What the bot saw about this PR
{
  "type": "info",
  "now": "-",
  "pr_number": 75069,
  "author": "u1f992",
  "headCommitOid": "a898b9fe208835a0bd5a836842689639f5f2054e",
  "mergeBaseOid": "f7de65a9e7e2432018aa9e04444e980ea29abe44",
  "lastPushDate": "2026-06-02T14:46:33.000Z",
  "lastActivityDate": "2026-06-02T15:22:56.000Z",
  "hasMergeConflict": false,
  "isFirstContribution": false,
  "tooManyFiles": false,
  "hugeChange": false,
  "tooManyCommits": false,
  "tooManyReviews": false,
  "popularityLevel": "Critical",
  "pkgInfo": [
    {
      "name": "yauzl",
      "kind": "edit",
      "files": [
        {
          "path": "types/yauzl/.npmignore",
          "kind": "package-meta-ok"
        },
        {
          "path": "types/yauzl/index.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/yauzl/package.json",
          "kind": "package-meta-ok"
        },
        {
          "path": "types/yauzl/v2/.npmignore",
          "kind": "package-meta-ok"
        },
        {
          "path": "types/yauzl/v2/index.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/yauzl/v2/package.json",
          "kind": "package-meta-ok"
        },
        {
          "path": "types/yauzl/v2/tsconfig.json",
          "kind": "package-meta-ok"
        },
        {
          "path": "types/yauzl/v2/yauzl-tests.ts",
          "kind": "test"
        },
        {
          "path": "types/yauzl/yauzl-tests.ts",
          "kind": "test"
        }
      ],
      "owners": [
        "ffflorian"
      ],
      "addedOwners": [],
      "deletedOwners": [],
      "popularityLevel": "Critical"
    }
  ],
  "reviews": [
    {
      "type": "approved",
      "reviewer": "ffflorian",
      "date": "2026-06-02T15:22:56.000Z",
      "isMaintainer": false
    }
  ],
  "mainBotCommentID": 4603594147,
  "ciResult": "pass"
}

@typescript-bot
Copy link
Copy Markdown
Contributor

🔔 @ffflorian — please review this PR in the next few days. Be sure to explicitly select Approve or Request Changes in the GitHub UI so I know what's going on.

@typescript-bot typescript-bot moved this from Waiting for Code Reviews to Needs Maintainer Review in Pull Request Status Board Jun 2, 2026
Copy link
Copy Markdown
Collaborator

@ffflorian ffflorian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@typescript-bot typescript-bot added the Owner Approved A listed owner of this package signed off on the pull request. label Jun 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Critical package Owner Approved A listed owner of this package signed off on the pull request.

Projects

Status: Needs Maintainer Review

Development

Successfully merging this pull request may close these issues.

3 participants