Skip to content

Conversation

js07
Copy link
Collaborator

@js07 js07 commented Sep 4, 2025

WHY

js07 added 4 commits July 31, 2025 20:28
Show configuration error when channel is not found and as_user is false

Use bot token instead of user token when posting message unless as_user
is true
Copy link
Contributor

coderabbitai bot commented Sep 4, 2025

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch slack-v2-test-components

Tip

👮 Agentic pre-merge checks are now available in preview!

Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.

  • Built-in checks – Quickly apply ready-made checks to enforce title conventions, require pull request descriptions that follow templates, validate linked issues for compliance, and more.
  • Custom agentic checks – Define your own rules using CodeRabbit’s advanced agentic capabilities to enforce organization-specific policies and workflows. For example, you can instruct CodeRabbit’s agent to verify that API documentation is updated whenever API schema files are modified in a PR. Note: Upto 5 custom checks are currently allowed during the preview period. Pricing for this feature will be announced in a few weeks.

Please see the documentation for more information.

Example:

reviews:
  pre_merge_checks:
    custom_checks:
      - name: "Undocumented Breaking Changes"
        mode: "warning"
        instructions: |
          Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).

Please share your feedback with us on this Discord post.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

vercel bot commented Sep 5, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Preview Comments Updated (UTC)
pipedream-docs-redirect-do-not-edit Ignored Ignored Sep 26, 2025 4:12am

js07 added 22 commits September 7, 2025 19:37
in v2, user tokens will not have the `files:read` scope
and remove historical messages for new-keyword-mention source since the
search:read scope is no longer requested.
ignore error when attempting to add app to
DM or group DM
Actions:
- get-file
- list-files
- list-replies
- send-message
- send-message-advanced
- send-message-to-channel

Triggers:
- new-keyword-mention
- new-message-in-channels
- new-user-mention

Use bot token for Get File, List Files, and List Replies
since the user token will no longer have `files:read` nor
`*:history` scopes.
for Send Message actions, Get File, List Files, and List Replies
rejected by approval team
- Reply to a Message
- Send Block Kit Message
- Send Large Message

and fix channel membership check in Send Message actions
Send Message and Send Large Message actions

Checking membership is only needed if sending message as the bot

Add channel membership checks to
Delete Message and Update Message actions
reverted new-direct-message version bump
and make it optional

Disable it in the Send Message to User or Group action
since the app can't be added to
a DM or group DM
and always use user token to get conversation history
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant