Skip to content

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

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

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

Conversation

@onutc
Copy link
Copy Markdown
Member

@onutc onutc commented Apr 1, 2026

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
Copy link
Copy Markdown
Member Author

onutc commented Apr 1, 2026

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