Slack onboarding: improve token help note with manifest option by Takhoffman · Pull Request #30846 · openclaw/openclaw

Takhoffman · 2026-03-01T17:37:35Z

Summary

Salvages the intent of Slack onboarding: improve token help note with manifest option #28110 on a safe feature branch.
Updates Slack onboarding token-help copy to include the manifest flow and current install wording.

Changes

1) Slack API → Create App → From scratch or From manifest (with the JSON below)
3) Install App to workspace to get the xoxb- bot token

Scope

Single file change: src/channels/plugins/onboarding/slack.ts
No logic/runtime behavior changes; copy-only UX improvement.

Credit

Based on Slack onboarding: improve token help note with manifest option #28110 by @yzhong52.

Supersedes

Slack onboarding: improve token help note with manifest option #28110

aisle-research-bot · 2026-03-01T17:37:43Z

🔒 Aisle Security Analysis

✅ We scanned this PR and did not find any security vulnerabilities.

_{Aisle supplements but does not replace security review.}

Analyzed PR: #30846 at commit 11ed2fe

_{Last updated on: 2026-03-01T18:00:45Z}

greptile-apps · 2026-03-01T17:40:39Z

Greptile Summary

This PR is a copy-only UX improvement to the Slack onboarding wizard's token-help note (noteSlackTokenHelp). It updates two string literals: step 1 now advertises the "From manifest" option alongside "From scratch" (the manifest JSON was already rendered below the note), and step 3 is reworded to more plainly describe installing the app to get the bot token.

Key observations:

No logic, runtime, or behavioral changes — purely display text.
The addition of "From manifest" in step 1 is a valuable improvement: it makes the already-present manifest JSON block discoverable upfront.
Step 3 loses the "OAuth & Permissions →" navigation hint that was useful for users taking the "From scratch" path, who still need to configure OAuth scopes before installing. Consider retaining that guidance for the scratch flow.

Confidence Score: 5/5

This PR is safe to merge — it contains only two string literal changes with no logic or runtime impact.
The change is confined to display copy inside a single prompter.note(...) call. There are no algorithmic, API, or configuration changes, making regressions essentially impossible.
No files require special attention beyond the minor style note on step 3 in src/channels/plugins/onboarding/slack.ts.

_{Last reviewed commit: 8b993d1}

greptile-apps

_{1 file reviewed, 1 comment}

_{Edit Code Review Agent Settings | Greptile}

greptile-apps · 2026-03-01T17:40:43Z

src/channels/plugins/onboarding/slack.ts

+      "1) Slack API → Create App → From scratch or From manifest (with the JSON below)",
      "2) Add Socket Mode + enable it to get the app-level token (xapp-...)",
-      "3) OAuth & Permissions → install app to workspace (xoxb- bot token)",
+      "3) Install App to workspace to get the xoxb- bot token",


Lost navigation hint for "From scratch" flow

The old wording "OAuth & Permissions → install app to workspace (xoxb- bot token)" included a navigational path that guided users who chose "From scratch" — they need to first go to OAuth & Permissions to add scopes before they can install the app. The updated line drops that hint.

For the "From manifest" flow this is a non-issue (scopes are pre-configured in the manifest JSON), but "From scratch" users may attempt to install without having added scopes first, leading to a confusing failure.

Consider preserving the guidance, for example:

Suggested change

"3) Install App to workspace to get the xoxb- bot token",

"3) Install App to workspace to get the xoxb- bot token (From scratch: add OAuth scopes under OAuth & Permissions first)",

Prompt To Fix With AI

This is a comment left during a code review. Path: src/channels/plugins/onboarding/slack.ts Line: 104 Comment: **Lost navigation hint for "From scratch" flow** The old wording `"OAuth & Permissions → install app to workspace (xoxb- bot token)"` included a navigational path that guided users who chose "From scratch" — they need to first go to **OAuth & Permissions** to add scopes before they can install the app. The updated line drops that hint. For the "From manifest" flow this is a non-issue (scopes are pre-configured in the manifest JSON), but "From scratch" users may attempt to install without having added scopes first, leading to a confusing failure. Consider preserving the guidance, for example: ```suggestion "3) Install App to workspace to get the xoxb- bot token (From scratch: add OAuth scopes under OAuth & Permissions first)", ``` How can I resolve this? If you propose a fix, please make it concise.

Takhoffman · 2026-03-01T17:57:42Z

PR #30846 - Slack onboarding: improve token help note with manifest option (#30846)

Merged via squash.

Merge commit: a179a0d
Verified: pnpm install --frozen-lockfile, pnpm build, pnpm check, pnpm test:macmini
Changes made:
M CHANGELOG.md
M src/channels/plugins/onboarding/slack.ts
Why these changes were made:
Preserved the original Slack onboarding: improve token help note with manifest option #28110 onboarding wording improvement on a safe feature branch and added required changelog coverage for autoland policy.
Changelog: CHANGELOG.md updated=true required=true opt_out=false

Thanks @yzhong52!

@yzhong52

…law#30846) thanks @yzhong52 Cherry-pick of upstream a179a0d.

@yzhong52