Skip to content

fix: resolve npm test dependencies and add pnpm lockfile#62

Merged
Naomi-Gift merged 3 commits into
HubDApp:mainfrom
Prz-droid:fix/npm-test-dependencies
Jun 2, 2026
Merged

fix: resolve npm test dependencies and add pnpm lockfile#62
Naomi-Gift merged 3 commits into
HubDApp:mainfrom
Prz-droid:fix/npm-test-dependencies

Conversation

@Prz-droid
Copy link
Copy Markdown
Contributor

close #53

- Regenerated pnpm lockfile with proper native dependency resolution
- Added @rolldown/binding-win32-x64-msvc for Vitest build tools
- npm test now runs successfully (32 tests passing)
- Updated README with Node.js version requirements (18+ LTS)
- Added pnpm installation guidance and native dependency handling docs
- All 4 test files passing with Vitest v4.1.8
@drips-wave
Copy link
Copy Markdown

drips-wave Bot commented Jun 2, 2026

@Prz-droid Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

Prz-droid added 2 commits June 2, 2026 10:20
- Remove invalid required_error parameter from Zod string schema
- Fix implicit any types in error handling
- Add proper ZodError interface for type safety
- All lint, test, and build checks now pass
- Create detailed SETUP.md with quick start and troubleshooting
- Add GitHub Actions CI workflow for multi-platform testing
- Test on Node 18.x and 20.x across Ubuntu, macOS, Windows
- Include dependency audit and lockfile verification
- Verify SETUP.md documentation completeness
@Naomi-Gift Naomi-Gift merged commit cf4a8fb into HubDApp:main Jun 2, 2026
1 check failed
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.

Fix local test runner dependency setup so npm test starts cleanly

3 participants