Skip to content

Fix Slack Connect image reads - #1705

Merged
RhysSullivan merged 1 commit into
mainfrom
fix/slack-connect-files
Aug 20, 2026
Merged

Fix Slack Connect image reads#1705
RhysSullivan merged 1 commit into
mainfrom
fix/slack-connect-files

Conversation

@RhysSullivan

Copy link
Copy Markdown
Collaborator

Summary

  • recover Slack Connect images when Slack MCP reports file_not_found
  • resolve and download them through Slack Web API using the existing OAuth grant
  • limit the fallback to Slack-hosted image reads with URL and size guards

Verification

  • bun run format:check
  • bun run lint
  • bun run typecheck
  • bun run test
  • live read of an externally uploaded Slack Connect PNG

@github-actions

github-actions Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Cloudflare preview

Torn down — the PR is closed.

@pkg-pr-new

pkg-pr-new Bot commented Aug 20, 2026

Copy link
Copy Markdown

Open in StackBlitz

@executor-js/cli

npm i https://pkg.pr.new/@executor-js/cli@1705

@executor-js/config

npm i https://pkg.pr.new/@executor-js/config@1705

@executor-js/execution

npm i https://pkg.pr.new/@executor-js/execution@1705

@executor-js/sdk

npm i https://pkg.pr.new/@executor-js/sdk@1705

@executor-js/codemode-core

npm i https://pkg.pr.new/@executor-js/codemode-core@1705

@executor-js/runtime-quickjs

npm i https://pkg.pr.new/@executor-js/runtime-quickjs@1705

@executor-js/plugin-file-secrets

npm i https://pkg.pr.new/@executor-js/plugin-file-secrets@1705

@executor-js/plugin-graphql

npm i https://pkg.pr.new/@executor-js/plugin-graphql@1705

@executor-js/plugin-keychain

npm i https://pkg.pr.new/@executor-js/plugin-keychain@1705

@executor-js/plugin-mcp

npm i https://pkg.pr.new/@executor-js/plugin-mcp@1705

@executor-js/plugin-onepassword

npm i https://pkg.pr.new/@executor-js/plugin-onepassword@1705

@executor-js/plugin-openapi

npm i https://pkg.pr.new/@executor-js/plugin-openapi@1705

executor

npm i https://pkg.pr.new/executor@1705

commit: 639b783

@RhysSullivan
RhysSullivan marked this pull request as ready for review August 20, 2026 16:10
@RhysSullivan
RhysSullivan merged commit e9920c3 into main Aug 20, 2026
79 of 80 checks passed
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