Skip to content

fix: pin @nostrify/react to fix CI type errors - #7

Merged
bitkarrot merged 1 commit into
mainfrom
fix/nostrify-dedup
Aug 18, 2026
Merged

bitkarrot merged 1 commit into
mainfrom
fix/nostrify-dedup

Conversation

@bitkarrot

Copy link
Copy Markdown
Owner

Summary

  • Pins @nostrify/react to 0.2.17 to fix TS2322 type mismatch caused by PR fix: update npm dependencies for security #5 npm audit fix
  • The audit bumped @nostrify/react to 0.2.31, which bundled @nostrify/nostrify@0.50.5 internally, conflicting with top-level @nostrify/nostrify@0.47.1
  • Restoring 0.2.17 allows proper deduplication; tsc --noEmit passes with 0 errors

Test plan

  • tsc --noEmit passes
  • npm audit reports 0 vulnerabilities

The npm audit fix had bumped @nostrify/react from 0.2.17 to 0.2.31,
which pulled in a nested @nostrify/nostrify@0.50.5 that conflicted
with the top-level @nostrify/nostrify@0.47.1, causing TS2322 errors.
Pinning back to 0.2.17 restores deduplication and fixes CI.
@bitkarrot
bitkarrot merged commit 6597aa0 into main Aug 18, 2026
2 checks passed
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