Skip to content

docs: polishing source wording#2493

Merged
mwbrooks merged 1 commit intofeat-ai-apps-thinking-stepsfrom
thinking-steps-docs-polish
Jan 30, 2026
Merged

docs: polishing source wording#2493
mwbrooks merged 1 commit intofeat-ai-apps-thinking-stepsfrom
thinking-steps-docs-polish

Conversation

@lukegalbraithrussell
Copy link
Copy Markdown
Contributor

@lukegalbraithrussell lukegalbraithrussell commented Jan 30, 2026

Summary

Makes it so it all matches the api docs

TODO: update generated reference docs. workspaces update changes some slugs (now full paths, not just subpackage paths), so need to decide if we're okay with all those slug changes appearing here

Requirements (place an x in each [ ])

@lukegalbraithrussell lukegalbraithrussell requested a review from a team as a code owner January 30, 2026 17:53
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Jan 30, 2026

⚠️ No Changeset found

Latest commit: 5018cab

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@lukegalbraithrussell lukegalbraithrussell changed the base branch from main to feat-ai-apps-thinking-steps January 30, 2026 17:53
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 30, 2026 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.04%. Comparing base (582eeec) to head (5018cab).
⚠️ 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    #2493   +/-   ##
============================================================
  Coverage                        93.04%   93.04%           
============================================================
  Files                               40       40           
  Lines                            11266    11266           
  Branches                           719      719           
============================================================
  Hits                             10483    10483           
  Misses                             771      771           
  Partials                            12       12
Flag Coverage Δ
cli-hooks 95.23% <ø> (ø)
cli-test 94.79% <ø> (ø)
oauth 77.39% <ø> (ø)
socket-mode 61.87% <ø> (ø)
web-api 98.01% <ø> (ø)
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.

@mwbrooks mwbrooks added docs M-T: Documentation work only pkg:web-api applies to `@slack/web-api` pkg:types applies to `@slack/types` labels Jan 30, 2026
mwbrooks
mwbrooks approved these changes Jan 30, 2026
View reviewed changes
Copy link
Copy Markdown
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 polishing up the wording @lukegalbraithrussell!

Comment thread packages/types/src/chunk.ts

/**
* An updated title of plans for task and tool calls.
* Used for displaying an updated title of a plan.
Copy link
Copy Markdown
Member

@mwbrooks mwbrooks Jan 30, 2026

Choose a reason for hiding this comment

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

✏️ ✨

@mwbrooks mwbrooks merged commit 6332e51 into feat-ai-apps-thinking-steps Jan 30, 2026
57 checks passed
@mwbrooks mwbrooks deleted the thinking-steps-docs-polish branch January 30, 2026 18:07
@mwbrooks mwbrooks changed the title Docs: polishing source wording docs: polishing source wording Jan 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mwbrooks mwbrooks mwbrooks approved these changes

@zimeg zimeg Awaiting requested review from zimeg

@srtaalej srtaalej Awaiting requested review from srtaalej

Assignees

@lukegalbraithrussell lukegalbraithrussell

Labels

docs M-T: Documentation work only pkg:types applies to `@slack/types` pkg:web-api applies to `@slack/web-api` semver:minor

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@lukegalbraithrussell @mwbrooks