feat: display referenced artifacts in slack response #18058
Merged
Conversation
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
Contributor
Author
This stack of pull requests is managed by Graphite. Learn more about stacking. |
|
Your preview environment pr-18058 has been deployed with errors. |
|
Preview environment logs can be accessed at: https://console.cloud.google.com/logs/query;query=resource.labels.namespace_name%3D%22pr-18058%22;duration=PT30M?project=lightdash-previews |
|
You can ssh into the preview environment by running: |
IrakliJani
approved these changes
Nov 12, 2025
Contributor
Author
Merge activity
|
658b095 to
ee992cc
Compare
Comment on lines
+44
to
+53
| return { | ||
| type: 'button', | ||
| url, | ||
| text: { | ||
| type: 'plain_text', | ||
| text: `📊 ${title}`, | ||
| emoji: true, | ||
| }, | ||
| action_id: 'view_artifact', | ||
| }; |
Contributor
There was a problem hiding this comment.
Slack buttons cannot have both url and action_id properties. When a url is present, the button becomes a link button that opens the URL directly and never triggers the action_id handler. This means handleViewArtifact() will never be called.
Impact: Analytics tracking (mentioned in TODO) will never work, and any future logic added to handleViewArtifact() will never execute.
Fix: Remove either url or action_id:
// Option 1: Keep as URL button (remove action_id)
return {
type: 'button',
url,
text: {
type: 'plain_text',
text: `📊 ${title}`,
emoji: true,
},
};
// Option 2: Make it an action button (remove url, handle navigation in the action handler)
return {
type: 'button',
text: {
type: 'plain_text',
text: `📊 ${title}`,
emoji: true,
},
action_id: 'view_artifact',
value: JSON.stringify({ url }),
};
Suggested change
| return { | |
| type: 'button', | |
| url, | |
| text: { | |
| type: 'plain_text', | |
| text: `📊 ${title}`, | |
| emoji: true, | |
| }, | |
| action_id: 'view_artifact', | |
| }; | |
| return { | |
| type: 'button', | |
| text: { | |
| type: 'plain_text', | |
| text: `📊 ${title}`, | |
| emoji: true, | |
| }, | |
| action_id: 'view_artifact', | |
| value: JSON.stringify({ url }), | |
| }; | |
Spotted by Graphite Agent
Is this helpful? React 👍 or 👎 to let us know.
lightdash-bot
pushed a commit
that referenced
this pull request
Nov 13, 2025
# [0.2165.0](0.2164.5...0.2165.0) (2025-11-13) ### Bug Fixes * update tooltip text and rename "Referenced artifacts" to "Referenced answers" ([#18067](#18067)) ([92852b7](92852b7)) ### Features * add a page to view a single verified answer from a link ([#18057](#18057)) ([c1de74a](c1de74a)) * display referenced artifacts in slack response ([#18058](#18058)) ([c4119a8](c4119a8))
Collaborator
|
🎉 This PR is included in version 0.2165.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
agha4to
pushed a commit
to agha4to/lightdash
that referenced
this pull request
Nov 17, 2025
<!-- Thanks so much for your PR, your contribution is appreciated! ❤️ --> ### Description: Added support for displaying referenced artifacts in Slack responses. When an AI agent references artifacts in its response, these are now shown as clickable buttons in the Slack message, allowing users to easily access the referenced data. The implementation includes: - New function `getReferencedArtifactsBlocks` to generate Slack blocks for referenced artifacts - Added handler for artifact view button clicks in Slack - Updated the message composition logic to fetch and include referenced artifacts ![CleanShot 2025-11-12 at [email protected]](https://app.graphite.com/user-attachments/assets/a0d94ad8-171a-4f7a-acb2-e87c54e61d6d.png)
agha4to
pushed a commit
to agha4to/lightdash
that referenced
this pull request
Nov 17, 2025
# [0.2165.0](lightdash/lightdash@0.2164.5...0.2165.0) (2025-11-13) ### Bug Fixes * update tooltip text and rename "Referenced artifacts" to "Referenced answers" ([lightdash#18067](lightdash#18067)) ([92852b7](lightdash@92852b7)) ### Features * add a page to view a single verified answer from a link ([lightdash#18057](lightdash#18057)) ([c1de74a](lightdash@c1de74a)) * display referenced artifacts in slack response ([lightdash#18058](lightdash#18058)) ([c4119a8](lightdash@c4119a8))
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description:
Added support for displaying referenced artifacts in Slack responses. When an AI agent references artifacts in its response, these are now shown as clickable buttons in the Slack message, allowing users to easily access the referenced data.
The implementation includes:
getReferencedArtifactsBlocksto generate Slack blocks for referenced artifacts