Skip to content

Conversation

@srtaalej
Copy link
Contributor

Summary

This PR enables chunks processing in the chat stream helper

    const streamer = new ChatStreamer(client, client.logger, {
      channel: CHANNEL_ID,
      thread_ts: threadTs,
      recipient_team_id: 'TEAM_ID,
      recipient_user_id: 'USER_ID',
    }, { buffer_size: 1 });

    await streamer.append({
      chunks: [
        {
          type: 'markdown_text',
          text: '**Hello!** I am starting to process your request...\n\n'
        }]
    });

    await streamer.stop({
      chunks: [
        {
          type: 'markdown_text',
          text: '\n\n---\n\n✅ **All tasks completed successfully!**\n',
        },
        {
          type: 'markdown_text',
          text: 'Thank you for using the streaming API with chunks.',
        },
      ],
    });

Requirements (place an x in each [ ])

@codecov
Copy link

codecov bot commented Jan 16, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.09%. Comparing base (b44cee1) to head (eb8b462).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@                       Coverage Diff                       @@
##           feat-ai-apps-thinking-steps    #2470      +/-   ##
===============================================================
- Coverage                        93.09%   93.09%   -0.01%     
===============================================================
  Files                               40       40              
  Lines                            11240    11233       -7     
  Branches                           713      712       -1     
===============================================================
- Hits                             10464    10457       -7     
  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.

@srtaalej srtaalej self-assigned this Jan 16, 2026
@srtaalej srtaalej marked this pull request as ready for review January 20, 2026 17:04
@srtaalej srtaalej requested a review from a team as a code owner January 20, 2026 17:04
@srtaalej srtaalej added the enhancement M-T: A feature request for new functionality label Jan 20, 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.

🙌🏻 Exciting, thanks for opening this PR @srtaalej!

It looks like you've also added the new task_update check along with the plan_update 😄

🧪 @zimeg and I will take a look at the PR soon. In the meantime, it looks like there are some failed tests that will need to be resolved.

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 semver:minor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants