Skip to content

Upgrade harper to 5.2.1 - #23

Merged
BboyAkers merged 2 commits into
mainfrom
harper-5.2.1
Aug 20, 2026
Merged

Upgrade harper to 5.2.1#23
BboyAkers merged 2 commits into
mainfrom
harper-5.2.1

Conversation

@BboyAkers

Copy link
Copy Markdown
Member

Bumps the harper dependency to ^5.2.1 (latest on npm) and regenerates the lockfile.

The lockfile is regenerated in full rather than updated in place: an in-place npm install --package-lock-only on macOS silently drops harper's optional native dependencies (bufferutil, utf-8-validate, segfault-handler), which makes npm ci fail on Linux CI. A full regeneration keeps them in the tree.

Verified locally: harper resolves to 5.2.1 and npm ci succeeds.

Co-Authored-By: Claude Opus 5 (1M context) noreply@anthropic.com

Bump the harper dependency to ^5.2.1 and regenerate the lockfile.
Regenerated in full so the optional native deps (bufferutil,
utf-8-validate, segfault-handler) stay in the tree for Linux CI.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request updates the harper development dependency in package.json from version ^5.0.10 to ^5.2.1. There are no review comments, and I have no feedback to provide.

The previous lockfile was generated with npm 11, which does not
auto-install the peer dependencies of an optional dependency. harper
5.2.1 pulls alasql, which optionally depends on react-native-fs, whose
peers (react-native, react) npm 12 installs and npm 11 does not. CI runs
npm 12 on Node 24/26, so npm ci failed there with those packages
"missing from lock file" while Node 22 (npm 11) passed.

Regenerated with npm 12 so the lockfile carries the full tree.
lockfileVersion stays 3; npm ci verified under both npm 11 and npm 12.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@BboyAkers
BboyAkers merged commit bcf4a6b into main Aug 20, 2026
4 checks passed
@BboyAkers
BboyAkers deleted the harper-5.2.1 branch August 20, 2026 17:20
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