Skip to content

Upgrade to libzim 9.8.1 + other dependencies#215

Merged
benoit74 merged 4 commits into
mainfrom
libzim_9.8.1
Jul 16, 2026
Merged

Upgrade to libzim 9.8.1 + other dependencies#215
benoit74 merged 4 commits into
mainfrom
libzim_9.8.1

Conversation

@benoit74

@benoit74 benoit74 commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Bump libzim to 9.8.1 and update Node.js dependencies (axios, node-gyp, node-addon-api, etc.)
  • Migrate ESLint config from .eslintrc.cjs to flat config (eslint.config.js), required by the ESLint 10 upgrade — same ruleset, no behavior change
  • Fix npm run tsc, which was reporting ~270 errors, almost all from a single missing tsconfig.json "types" entry: TypeScript 6 no longer auto-includes @types/* packages, so Jest globals (describe/it/expect) were invisible. Added "types": ["jest", "node"], plus a few small fixes it then surfaced (missing tqdm type declaration, a non-null assertion in a test, excluding dist/eslint.config.js from the program)
  • Add a tsc type-check step to CI so this doesn't regress silently again

Why bumping module to es2022 is safe

tsconfig.json's module only affects tsc type-checking and ts-jest's in-test transpilation — nothing it emits is shipped. The package's dist/index.js/index.d.ts are hand-written files copied verbatim by npm run prepare, and the native addon is built separately via node-gyp. So this setting is invisible to consumers, whether their own project is CommonJS or ESM.

Why not using Typescript 7

typescript is stuck at version 6 because @typescript-eslint/parser does not (yet?) supports version 7.

@benoit74 benoit74 self-assigned this Jul 16, 2026
@benoit74 benoit74 added this to the 4.5.0 milestone Jul 16, 2026
@benoit74
benoit74 merged commit 0b7c1db into main Jul 16, 2026
28 checks passed
@benoit74
benoit74 deleted the libzim_9.8.1 branch July 16, 2026 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant