Skip to content

fix(slack): restore assistant operation compatibility - #7339

Merged
TheodoreSpeaks merged 2 commits into
stagingfrom
feat/slack-agent-apis
Sep 1, 2026
Merged

fix(slack): restore assistant operation compatibility#7339
TheodoreSpeaks merged 2 commits into
stagingfrom
feat/slack-agent-apis

Conversation

@TheodoreSpeaks

Copy link
Copy Markdown
Collaborator

Summary

  • restore Set Assistant Status and Set Assistant Title in Slack v2
  • preserve legacy and Agent suggested-prompt field layouts from persisted workflow state

Type of Change

  • Bug fix

Testing

  • bunx vitest run blocks/blocks/slack.test.ts
  • bun run apps/sim/scripts/check-canvas-sentences.ts --block=slack_v2
  • bun run lint
  • bun run check:audits

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Sep 1, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated
docs Ready Ready Preview Sep 1, 2026 3:47am UTC

Request Review

@greptile-apps

greptile-apps Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Restores Slack v2 Assistant status and title operations while separating Assistant and Agent suggested-prompt routing.

  • Adds Assistant status, title, and suggested-prompt operations to the Slack v2 catalog.
  • Introduces a distinct Agent suggested-prompts operation with Agent-specific credential, channel, and thread fields.
  • Updates tests, documentation, and deployment integration metadata for the restored operations.

Confidence Score: 5/5

The PR appears safe to merge because no blocking failure remains within the eligible follow-up-review scope.

No blocking failure remains.

Important Files Changed

Filename Overview
apps/sim/blocks/blocks/slack.ts Restores Assistant operations and splits Assistant versus Agent suggested-prompt field and tool routing.
apps/sim/blocks/blocks/slack.test.ts Adds coverage for operation availability, tool selection, parameter mapping, and operation-specific field visibility.
apps/docs/content/docs/integrations/slack.mdx Documents the restored Assistant status and title operations.
packages/deployment-config/src/integrations.json Updates Slack operation metadata and the operation count to reflect the restored and split operations.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
  O[Slack v2 operation] --> A{Operation family}
  A -->|Assistant status/title/prompts| L[Legacy Assistant fields]
  L --> AT[Assistant Slack tool]
  A -->|Agent suggested prompts/status/rename| G[Agent-specific fields]
  G --> GT[Slack Agent tool]
Loading

Reviews (2): Last reviewed commit: "fix(slack): split assistant and agent pr..." | Re-trigger Greptile

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 4 files

Heads up: you’re close to your included review allowance. Set a flex budget so reviews don’t pause.

Fix all with cubic | Re-trigger cubic

Comment thread apps/sim/blocks/blocks/slack.test.ts Outdated
Comment thread apps/sim/blocks/blocks/slack.ts Outdated
@TheodoreSpeaks

Copy link
Copy Markdown
Collaborator Author

@greptile

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