chore: upgrade @pkcprotocol/pkc-js to 0.0.99 - #146
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Team Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe package manifest updates ChangesPKC dependency update
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This updates the PKC library to 0.0.99 with a synchronized lockfile. The supplied build and test results pass, with no identified merge-blocking compatibility or runtime risk. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Warning Some tools did not complete. Review the errors below. 🔧 ESLint
package.jsonOops! Something went wrong! :( ESLint: 8.27.0 Error: ESLint configuration in --config » eslint-config-oclif is invalid:
Referenced from: /.eslintrc Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
Upgrades
@pkcprotocol/pkc-jsfrom 0.0.89 to 0.0.99 (latest).Notable upstream changes:
pkc.destroy()(libp2p-js block fetches outlive pkc.destroy() and log after teardown, failing vitest with exit 1 pkcprotocol/pkc-js#345)destroy()waits for websockets to close (fix(rpc): make client and server destroy() wait for their websockets to close pkcprotocol/pkc-js#326); kubo/helia updates driven by gossip pushes instead of a 1s poll (perf(community): drive kubo/helia updates from gossip pushes instead of a 1s poll (#308 #307) pkcprotocol/pkc-js#311)repo.gccannot race clients still fetching them (fix(community): defer unpinning superseded cids so repo.gc cannot race clients still fetching them pkcprotocol/pkc-js#306)No CLI code changes were needed.
Testing
npm run build && npm run build:testpassnpm run test:cli: 44 files, 342 tests passed, 1 skippedCloses #145
Summary by CodeRabbit