Skip to content

fix(slack-gateway): preserve ACP reply whitespace#194

Closed
onutc wants to merge 2 commits into
mainfrom
fix-slack-gateway-chunk-whitespace
Closed

fix(slack-gateway): preserve ACP reply whitespace#194
onutc wants to merge 2 commits into
mainfrom
fix-slack-gateway-chunk-whitespace

Conversation

@onutc

@onutc onutc commented Apr 1, 2026

Copy link
Copy Markdown

Summary

  • preserve ACP chunk whitespace and newlines in the Slack gateway reply path
  • move ACP text extraction/joining into a shared lossless helper used by the API and Slack gateway
  • add regression coverage for chunk-boundary whitespace, final Slack payload integrity, and document the invariant

Validation

  • go test ./... (workdir: acptext)
  • go test ./... (workdir: integrations/slack-gateway)
  • go test ./... (workdir: api)
  • git diff --check
  • npx -y @simpledoc/simpledoc check

@onutc

onutc commented Apr 1, 2026

Copy link
Copy Markdown
Author

Superseded by clean PR #195. Closing this one because #194 includes unrelated changes from its original branch ancestry.

@onutc onutc closed this Apr 1, 2026
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.

1 participant