Skip to content

Conversation

@srtaalej
Copy link
Contributor

Summary

This PR adds the task_display_mode option to the start of chat streams.

const stream = new ChatStreamer(client, client.logger,
  {
    channel: 'C1234',
    thread_ts: '1234.56',
    chunks: [
      {
        type: 'markdown_text',
        text: 'Hello world',
      },
    ],
    recipient_team_id: 'T1234',
    recipient_user_id: 'U1234',
    task_display_mode: 'timeline'
  },
);

Requirements (place an x in each [ ])

@srtaalej srtaalej self-assigned this Jan 26, 2026
@srtaalej srtaalej requested a review from a team as a code owner January 26, 2026 23:39
@srtaalej srtaalej added pkg:types applies to `@slack/types` semver:minor pkg:web-api applies to `@slack/web-api` and removed pkg:types applies to `@slack/types` labels Jan 26, 2026
@codecov
Copy link

codecov bot commented Jan 26, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.07%. Comparing base (16f377d) to head (0cd39dd).
⚠️ Report is 1 commits behind head on feat-ai-apps-thinking-steps.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@                     Coverage Diff                      @@
##           feat-ai-apps-thinking-steps    #2479   +/-   ##
============================================================
  Coverage                        93.07%   93.07%           
============================================================
  Files                               40       40           
  Lines                            11213    11213           
  Branches                           708      708           
============================================================
  Hits                             10437    10437           
  Misses                             764      764           
  Partials                            12       12           
Flag Coverage Δ
cli-hooks 95.23% <ø> (ø)
cli-test 94.79% <ø> (ø)
webhook 96.66% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

@srtaalej LGTM! Thanks for a fast fix to stacked issues prior 🙏 ✨

@zimeg zimeg added the enhancement M-T: A feature request for new functionality label Jan 27, 2026
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.

⚠️ There are failing tests due to a linter issue. I've left a suggestion that should fix it (confirmed locally).

✅ Thanks for adding the task_display_mode support so quickly! 🙌🏻 We're starting to feel feature complete!

🧪 I've ran the test locally (see my suggestion to fix the error) and tested manually with a sample app, which works great! 👏🏻

Comment on lines +259 to +260
* @description Specifies how tasks are displayed in the message. A "timeline" displays individual tasks
* interleaved with text and "plan" displays all tasks together.
Copy link
Member

Choose a reason for hiding this comment

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

praise: Really like this description!

Co-authored-by: Michael Brooks <mbrooks@slack-corp.com>
@srtaalej srtaalej merged commit f2ac218 into feat-ai-apps-thinking-steps Jan 27, 2026
39 of 57 checks passed
@srtaalej srtaalej deleted the ale-task-display-mode-2 branch January 27, 2026 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement M-T: A feature request for new functionality pkg:web-api applies to `@slack/web-api` semver:minor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants