Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@steipete
Copy link
Contributor

Follow-up to #1369.\n\n- Enforce known short IDs in actions/replies\n- Add regression tests\n- Document new template fields\n- Preserve fetch helper methods in abort wrapper

@cursor
Copy link

cursor bot commented Jan 21, 2026

PR Summary

Strengthens BlueBubbles ID handling and preserves fetch helpers.

  • Require resolveBlueBubblesMessageId(..., { requireKnownShortId: true }) for react/edit/unsend/reply, throwing when numeric short IDs are unknown; update channel/media send paths similarly
  • Tests: add short-ID resolution/error propagation cases and fallback to toolContext.currentChannelId for reactions
  • Docs/config: add {{MessageSidFull}} and {{ReplyToIdFull}} template variables (with {{ReplyToId}}), exposing full provider IDs
  • Infra: wrapFetchWithAbortSignal now returns a wrapper that preserves original fetch helper properties via Object.assign
  • Changelog updated; minor CLI import cleanup

Written by Cursor Bugbot for commit d0e8fae. This will update automatically on new commits. Configure here.

@steipete steipete merged commit 5d73a41 into main Jan 21, 2026
2 checks passed
@steipete steipete deleted the temp/landpr-1369-followup branch January 21, 2026 17:10
@steipete
Copy link
Contributor Author

Landed via temp rebase onto main.

  • Gate: pnpm lint && pnpm build && pnpm test
  • Land commit: d0e8fae
  • Merge commit: 5d73a41

Thanks!

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.

2 participants