Repository navigation
fix(deps): take the plugin-gateway snapshot that reads message ids as null - #358
Conversation
|
Note Every file here is a lockfile or a package manifest, so PR Lens left this pull request undrawn. Comment
|
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
There was a problem hiding this comment.
✅ No new issues found.
Reviewed changes — the single commit that moves the bot onto the plugin-gateway snapshot that reads message ids as null.
@wolfstar/plugin-gatewaypinned to the snapshot —projects/bot/package.jsonmoves from^0.16.0to the exact0.16.2-next-20261102131414. The exact pin is right for anextbuild, since a range would not match a prerelease. The integrity hash in the lockfile matches npm, and the peer dependencies are unchanged from 0.16.0.@wolfstar/plugin-cacheraised to^0.8.1— this is the range the snapshot depends on, so the bot and the gateway resolve one copy of the cache. The 0.8.1 integrity hash matches npm.- Lockfile regeneration — the lockfile has the new gateway and cache entries. It also has two unrelated changes from re-resolving:
cac7.0.0 is deduped to 7.1.0, and@smithy/coregets adeprecatednote.
The bot's isWebhookMessage guard in lib/common/guards.ts checks typeof message.webhookId === 'string', so it handles null correctly. The PR description already says the pin is temporary and should go back to ^0.16.2 once the release is approved. Nobody will be reminded of that, so you may want to track it in #357.
claude-sonnet-5-5 | 𝕏

🔗 Linked issue
Related to #357
❓ Type of change
📚 Description
wolfstar-project/plugins#276 makes
Message#webhookId,applicationId,nonceandpositionnullwhen a message has none, as in discord.js. The release that carries it,@wolfstar/plugin-gateway0.16.2, is still staged on npm waiting for approval, so this pins thenextsnapshot published from that merge,0.16.2-next-20261102131414. It goes back to^0.16.2once the release is approved.plugin-cachegoes to^0.8.1, the range the snapshot asks for, so the bot and the gateway share one copy of the cache.plugin-broker0.7.0 pinsplugin-cacheto exactly0.8.0and keeps its own copy until it is bumped too; the broker is off by default.

Confidence Score: 5/5
No merge-blocking issues were identified; the focused message and cache checks passed with the updated dependencies.Reviews (1) · Last reviewed commit: "fix(deps): take the plugin-gateway snaps..." · Reviewed by Greptile