Skip to content

Comments

docs: updates using ai in apps with chat streaming and feedback buttons#2666

Merged
lukegalbraithrussell merged 16 commits intoai-appsfrom
docs-ai-apps
Oct 4, 2025
Merged

docs: updates using ai in apps with chat streaming and feedback buttons#2666
lukegalbraithrussell merged 16 commits intoai-appsfrom
docs-ai-apps

Conversation

@lukegalbraithrussell
Copy link
Contributor

Summary

https://docs-slack-d-ai-apps-me-xahuew.herokuapp.com/tools/bolt-js/concepts/ai-apps/

Requirements (place an x in each [ ])

@codecov
Copy link

codecov bot commented Oct 3, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.40%. Comparing base (dfe0ca2) to head (5f8178f).
⚠️ Report is 17 commits behind head on ai-apps.

Additional details and impacted files
@@             Coverage Diff             @@
##           ai-apps    #2666      +/-   ##
===========================================
+ Coverage    93.37%   93.40%   +0.02%     
===========================================
  Files           37       37              
  Lines         7581     7613      +32     
  Branches       667      668       +1     
===========================================
+ Hits          7079     7111      +32     
  Misses         497      497              
  Partials         5        5              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@zimeg zimeg added docs M-T: Documentation work only semver:minor labels Oct 3, 2025
Comment on lines 549 to 552
const feedback_type = body.actions[0];
if (!('value' in feedback_type)) {
return;
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👁️‍🗨️ note: We have a WIP PR to update this syntax. Will make sure this is included here but it doesn't change the functionalities!

🔗 slack-samples/bolt-js-assistant-template#75

Copy link
Member

@zimeg zimeg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@lukegalbraithrussell LGTM! This all reads fantastic. I like the structure of these pages and think this can publish whenever the time arrives! 📚 ✨

I left comments on small suggestions that might be nice to review but nothing's blocking! I'm not sure when best to merge this is but IIRC we are planning SDK releases on the topic in neartime?

Copy link

@srtaalej srtaalej left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

left one small comment but LGTM!

Copy link
Member

@mwbrooks mwbrooks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Thanks for getting the docs prepared @lukegalbraithrussell!

## Full example : App Agent & Assistant Template

Below is the `app.js` file of the [App Agent & Assistant Template repo](https://github.com/slack-samples/bolt-js-assistant-template/) we've created for you to build off of.
Below is the `app.js` file of the [App Agent & Assistant Template](https://github.com/slack-samples/bolt-js-assistant-template) repo we've created for you to build off of.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note: We're breaking this into multiple files in slack-samples/bolt-js-assistant-template#74, so we may not want to include the source code in the docs and instead link to the repo?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good point; removed and replaced iwth just link

@lukegalbraithrussell lukegalbraithrussell merged commit 567a7b9 into ai-apps Oct 4, 2025
19 checks passed
@lukegalbraithrussell lukegalbraithrussell deleted the docs-ai-apps branch October 4, 2025 01:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs M-T: Documentation work only semver:minor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants