Skip to content

fix(opencode-plugin-v2): restore the redis optional-peer entries dropped from the lockfile - #15727

Merged
diegosouzapw merged 2 commits into
release/v3.8.52from
fix/plugin-v2-lockfile-redis
Oct 7, 2026
Merged

diegosouzapw merged 2 commits into
release/v3.8.52from
fix/plugin-v2-lockfile-redis

Conversation

@diegosouzapw

Copy link
Copy Markdown
Owner

npm ci in @omniroute/opencode-plugin-v2 fails on the release tip with Missing: redis@6.3.0 (and @redis/bloom, client, json, search, time-series) from lock file, which turns the Test v2 (Node 22/24) jobs red. Cause: the Dependabot bump #15648 regenerated this package's lockfile and dropped 114 lines, including the redis / @redis/* entries the tree still resolves (redis is an optional peer, range >=5.0.0 <7.0.0).

Fix: regenerated the lock with npm install --package-lock-only — 97 additions, 0 removals, package.json untouched.

Validation: npm ci --dry-run is clean on npm 12.0.2, the pinned 11.15.0 and npm 10; a real npm ci installs; lockfile policy (check-lockfile) and check-deps pass.

Note: after the lock is fixed, one plugin-v2 test still fails on the tip (snapshot-stale-entries → "retired field still loads the valid entry"), caused by the toolsOnly: true default from #14554, not by this change; handled separately.

…ped from the lockfile

The Dependabot bump in #15648 regenerated the package lock and removed the
redis / @redis/* entries that the dependency tree still resolves, so npm ci
failed with 'Missing: redis@6.3.0 from lock file' and the Test v2 jobs went
red. Regenerated with npm install --package-lock-only: 97 additions, no
removals, package.json untouched; npm ci --dry-run is clean on npm 10, 11.15.0
and 12.
@diegosouzapw
diegosouzapw merged commit f376a12 into release/v3.8.52 Oct 7, 2026
13 of 17 checks passed
@diegosouzapw
diegosouzapw deleted the fix/plugin-v2-lockfile-redis branch October 7, 2026 17:47
@github-actions

github-actions Bot commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

CI Coverage Report

  • Coverage job: skipped
  • PR test policy: success

Coverage artifact was not available for this run.

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