Skip to content

fix(render): preserve usergroup and broadcast elements#113

Merged
nwparker merged 1 commit into
stablyai:mainfrom
dwaxe:dwaxe/preserve-notification-elements
Jul 18, 2026
Merged

fix(render): preserve usergroup and broadcast elements#113
nwparker merged 1 commit into
stablyai:mainfrom
dwaxe:dwaxe/preserve-notification-elements

Conversation

@dwaxe

@dwaxe dwaxe commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Messages sent through agent-slack can include user-group mentions and broadcasts such as @here. Slack stores these as rich-text elements, but when the agent later reads the message back, agent-slack omits them from the returned content. The Slack message was delivered correctly, but the agent sees an incomplete version of what it sent. This change preserves those mentions when rendering messages back to the agent.

Co-authored by GPT 5.6-Sol.

@dwaxe
dwaxe force-pushed the dwaxe/preserve-notification-elements branch from 4ed0a04 to b6d139b Compare July 16, 2026 20:14
@dwaxe
dwaxe marked this pull request as ready for review July 16, 2026 20:40
nwparker added a commit that referenced this pull request Jul 18, 2026
Takeover: integrate 5 verified community PRs (#113 #110 #109 #92 #106)
@nwparker
nwparker merged commit ac2b9a5 into stablyai:main Jul 18, 2026
@nwparker

Copy link
Copy Markdown
Contributor

Taken over, verified, and merged into main via #121 (commit 80f0e8d).

Rebased cleanly onto main@452dad2 on branch takeover/pr-113. Verified (composed with the other merged PRs):

  • typecheck clean · 345 tests / 0 fail · lint 0 errors · format clean · build:npm + node --check OK
  • bun test test/render.test.ts15 pass / 0 fail, incl. the new @here/@channel/usergroup round-trip cases

Your commit b6d139b is preserved in history. Thanks @dwaxe! Closing as merged.

@dwaxe
dwaxe deleted the dwaxe/preserve-notification-elements branch July 18, 2026 21:09
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