fix(web-api): widen Slack Lists message field type by AjTheSpidey · Pull Request #2599 · slackapi/node-slack-sdk · GitHub
Skip to content

fix(web-api): widen Slack Lists message field type - #2599

Open
AjTheSpidey wants to merge 3 commits into
slackapi:mainfrom
AjTheSpidey:codex/fix-slack-lists-message-type
Open

AjTheSpidey wants to merge 3 commits into
slackapi:mainfrom
AjTheSpidey:codex/fix-slack-lists-message-type

Conversation

@AjTheSpidey

Copy link
Copy Markdown

Summary

  • add a Slack Lists message metadata type for response values
  • widen SlackListsItemFieldMessage.message to accept request URL arrays, a single returned message object, or returned message object arrays
  • add tsd coverage for the preserved request shape and the response object shapes
  • add a patch changeset for @slack/web-api

Fixes #2598

Validation

  • npm run build --workspace=@slack/logger
  • npm run build --workspace=@slack/types
  • npm run build --workspace=@slack/web-api
  • npm run test:types --workspace=@slack/web-api
  • npx @biomejs/biome check packages/web-api/src/types/request/slackLists.ts packages/web-api/test/types/methods/slacklists.test-d.ts .changeset/slack-lists-message-field.md
  • From packages/web-api: node --import tsx --test src/*.test.ts (154 tests passed)

@AjTheSpidey
AjTheSpidey requested a review from a team as a code owner May 15, 2026 20:05
@changeset-bot

changeset-bot Bot commented May 15, 2026 •

Copy link
Copy Markdown

@salesforce-cla

Copy link
Copy Markdown

Thanks for the contribution! Before we can merge this, we need @AjTheSpidey to sign the Salesforce Inc. Contributor License Agreement.

@hello-ashleyintech

Copy link
Copy Markdown
Contributor

Hi, @AjTheSpidey! Thanks for your contribution! Could you please sign the CLA?

@AjTheSpidey

Copy link
Copy Markdown
Author

…ts-message-type

# Conflicts:
#	packages/web-api/src/types/request/slackLists.ts

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SlackListsItem FieldMessage.message type does not match API response shape

2 participants