docs(connector-slack): document the full Slack scope + event set#160
docs(connector-slack): document the full Slack scope + event set#160harry-harish wants to merge 1 commit into
Conversation
The scope table only listed assistant:write + chat:write; the shipped surfaces need more. Grounded each entry against the adapter's actual API calls / event registrations: - app_mentions:read (#155 @peek-in-channel), channels/groups/im:history (thread auto-continue + DMs), files:write (#156 share_session upload), canvases:write (#159 journey canvas), files:read (journey files.info permalink — without it the journey degrades to Block Kit), commands (/peek). - Added Event subscriptions + Socket Mode sections and a reinstall note. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Signed-off-by: Harish Kumar <22562634+harry-harish@users.noreply.github.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe Slack connector README now documents Socket Mode setup, capability-specific bot scopes, required bot events, app reinstall requirements after scope changes, and the interaction between ChangesSlack connector setup
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
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. Comment |
What
The
@peekdev/connector-slackREADME's scope table only listedassistant:write+chat:write— it predated the channel/share/journey surfaces. This documents the complete, code-grounded Slack app setup so a maintainer configuring the app doesn't hit silent no-ops.Grounded against the adapter's actual API calls + event registrations
chat:write—chat.postMessage+assistant.threads.setStatusassistant:write— assistant registration / status / title / promptsapp_mentions:read—@peekin a channel (PR feat(connector-slack): @peek in a channel — app_mention + threaded auto-continue #155)channels:history/groups:history/im:history— message events for thread auto-continue + DMsfiles:write—share_sessionbundle upload (files.uploadV2, PR feat(peek): share a session to a Slack channel — consent-gatedshare_session#156)canvases:write— session-journey canvas (conversations.canvases.create, PR feat(peek): rebuild the session journey in Slack (canvas + Block Kit fallback) #159)files:read— the journey'sfiles.infopermalink resolution; without it the journey silently degrades to the Block Kit fallback (this one wasn't obvious and wasn't documented anywhere)commands— the/peekslash commandAlso added Event subscriptions + Socket Mode sections and a reinstall note.
Scope
Docs only.
@peekdev/connector-slackisprivate:true(never published) → no changeset. No code change.🤖 Generated with Claude Code
Summary by CodeRabbit
chat:writeandassistant:writepermissions.