Skip to content

[ENG-4000] fix(s3): remove bogus top-level data envelope from message format#642

Merged
laurenzlong merged 1 commit into
mainfrom
fix/s3-message-format-data-envelope
Jun 30, 2026
Merged

[ENG-4000] fix(s3): remove bogus top-level data envelope from message format#642
laurenzlong merged 1 commit into
mainfrom
fix/s3-message-format-data-envelope

Conversation

@eberle1080

Copy link
Copy Markdown
Contributor

Summary

Fixes a mistake from #631. The S3 destination "Message format" section incorrectly documented read and subscribe action payloads as being wrapped in a top-level data object. There is no wrapper envelope — the object body written to S3 is the message payload itself.

Changes

  • Reworded the "Message format" intro to state the object body is the payload itself, with no wrapper.
  • Unwrapped the read action JSON example so action, projectId, result, etc. are at the top level.
  • Unwrapped the subscribe action JSON example the same way.

Both JSON examples were verified to parse. The JMESPath template data namespace (e.g. data.installationId) is unchanged — it's a handle to the payload root and remains correct.

🤖 Generated with Claude Code

… format

The S3 destination message format docs incorrectly wrapped read and
subscribe action payloads in a top-level `data` object. The object body
is the message payload itself, with no wrapper envelope. Fix the prose
and both JSON examples.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@linear

linear Bot commented Jun 30, 2026

Copy link
Copy Markdown

ENG-4000

@mintlify

mintlify Bot commented Jun 30, 2026

Copy link
Copy Markdown

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
Ampersand 🟢 Ready View Preview Jun 30, 2026, 12:36 AM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@laurenzlong laurenzlong merged commit 330ef9b into main Jun 30, 2026
4 checks passed
@laurenzlong laurenzlong deleted the fix/s3-message-format-data-envelope branch June 30, 2026 00:37
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