fix: drop preview_images from data_visualization blocks on retrieval#144
Merged
Merged
Conversation
…tion Slack renders data_visualization charts server-side and attaches `preview_images` on retrieval, but rejects that field on send with `unknown property 'preview_images'`. Add it to RETRIEVAL_ONLY_KEYS so round-tripped charts stay send-valid. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014aydKTubf32kAFUbnCNLRb
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Slack attaches
preview_imagestodata_visualizationblocks when retrieving them, but the API rejects these fields when sending the blocks back. This change addspreview_imagesto the retrieval-only keys that are stripped during sanitization fordata_visualizationblocks.Why
When retrieving blocks containing
data_visualizationelements, Slack includes server-rendered chart previews in apreview_imagesfield. However, when these blocks are sent back to Slack (e.g., via block updates), the API rejectspreview_imagesas an unknown property. The sanitizer needs to remove this field specifically fromdata_visualizationblocks while preserving it on other block types that may legitimately use it.Test plan
preview_imagesis dropped fromdata_visualizationblockspreview_imagesis preserved on other block types (e.g.,section)https://claude.ai/code/session_014aydKTubf32kAFUbnCNLRb