fix(slack): guard Socket Mode listeners access during startup#28702

Merged

Takhoffman merged 2 commits intoopenclaw:mainfrom

Glucksberg:fix/issue-21715

Mar 1, 2026

Contributor

Glucksberg commented Feb 27, 2026

Summary

Wraps app.options() registration in try-catch to prevent crash-loop
Falls back to static_select menus when external arg menus unavailable
Regression test verifying graceful degradation

Fixes #21715

🤖 Generated with Claude Code

openclaw-barnacle bot added channel: slack size: S experienced-contributor labels

Contributor

greptile-apps bot commented Feb 27, 2026

Greptile Summary

Wraps Socket Mode app.options() registration in try-catch to prevent crash-loops when Bolt receiver isn't initialized during startup, with graceful fallback to static_select menus when external_select registration fails.

Changed supportsExternalArgMenus from const to let to allow mutation after error
Added try-catch around registerArgOptions() call that sets flag to false and logs on failure
Comprehensive test verifies registration succeeds and menus fall back to static_select when options() throws
Follows defensive programming pattern with appropriate graceful degradation

Confidence Score: 5/5

Safe to merge with no risk
Clean defensive fix with comprehensive test coverage and proper graceful degradation. Changes are minimal, well-scoped, and follow established error handling patterns.
No files require special attention

_{Last reviewed commit: 6d45e8d}

Takhoffman force-pushed the fix/issue-21715 branch from ab5a11d to efb486a Compare

March 1, 2026 15:25

aisle-research-bot bot commented Mar 1, 2026 •

edited

Loading

🔒 Aisle Security Analysis

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

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

Analyzed PR: #28702 at commit a72cc99

Glucksberg and others added 2 commits

March 1, 2026 09:26


          fix(slack): guard Socket Mode listeners access during startup

1a1d5b9


          Slack: fix snapshot test contract and add changelog for slash fallback

a72cc99

Takhoffman force-pushed the fix/issue-21715 branch from efb486a to a72cc99 Compare

March 1, 2026 15:29

Takhoffman merged commit 3aad6c8 into openclaw:main

9 checks passed

Contributor

Takhoffman commented Mar 1, 2026

PR #28702 - fix(slack): guard Socket Mode listeners access during startup (#28702)

Merged via squash.

Merge commit: 3aad6c8
Verified: pnpm install --frozen-lockfile, pnpm build, pnpm check, pnpm test:macmini
Changes made:
M CHANGELOG.md
M extensions/slack/src/channel.test.ts
Why these changes were made:
Added required changelog coverage for the slash fallback fix and updated Slack channel tests to match current snapshot API (cfg required + async return), which unblocked strict verification gates on this branch.
Changelog: CHANGELOG.md updated=true required=true opt_out=false

Thanks @Glucksberg!

zooqueen added a commit to hanzoai/bot that referenced this pull request


          fix(slack): guard Socket Mode listeners access during startup (opencl…

52e40a8

…aw#28702) thanks @Glucksberg

Cherry-pick of upstream 3aad6c8.

ansh pushed a commit to vibecode/openclaw that referenced this pull request


          fix(slack): guard Socket Mode listeners access during startup (opencl…

e8bc88a

…aw#28702) thanks @Glucksberg

Verified:
- pnpm install --frozen-lockfile
- pnpm build
- pnpm check
- pnpm test:macmini

Co-authored-by: Glucksberg <80581902+Glucksberg@users.noreply.github.com>
Co-authored-by: Tak Hoffman <781889+Takhoffman@users.noreply.github.com>

steipete pushed a commit to Sid-Qin/openclaw that referenced this pull request


          fix(slack): guard Socket Mode listeners access during startup (opencl…

9c71123

…aw#28702) thanks @Glucksberg

Verified:
- pnpm install --frozen-lockfile
- pnpm build
- pnpm check
- pnpm test:macmini

Co-authored-by: Glucksberg <80581902+Glucksberg@users.noreply.github.com>
Co-authored-by: Tak Hoffman <781889+Takhoffman@users.noreply.github.com>

safzanpirani pushed a commit to safzanpirani/clawdbot that referenced this pull request


          fix(slack): guard Socket Mode listeners access during startup (opencl…

b3cd4b3

…aw#28702) thanks @Glucksberg

Verified:
- pnpm install --frozen-lockfile
- pnpm build
- pnpm check
- pnpm test:macmini

Co-authored-by: Glucksberg <80581902+Glucksberg@users.noreply.github.com>
Co-authored-by: Tak Hoffman <781889+Takhoffman@users.noreply.github.com>

steipete pushed a commit to Sid-Qin/openclaw that referenced this pull request


          fix(slack): guard Socket Mode listeners access during startup (opencl…

9ef0d5d

…aw#28702) thanks @Glucksberg

Verified:
- pnpm install --frozen-lockfile
- pnpm build
- pnpm check
- pnpm test:macmini

Co-authored-by: Glucksberg <80581902+Glucksberg@users.noreply.github.com>
Co-authored-by: Tak Hoffman <781889+Takhoffman@users.noreply.github.com>

robertchang-ga pushed a commit to robertchang-ga/openclaw that referenced this pull request


          fix(slack): guard Socket Mode listeners access during startup (opencl…

0fac55c

…aw#28702) thanks @Glucksberg

Verified:
- pnpm install --frozen-lockfile
- pnpm build
- pnpm check
- pnpm test:macmini

Co-authored-by: Glucksberg <80581902+Glucksberg@users.noreply.github.com>
Co-authored-by: Tak Hoffman <781889+Takhoffman@users.noreply.github.com>

hanqizheng pushed a commit to hanqizheng/openclaw that referenced this pull request


          fix(slack): guard Socket Mode listeners access during startup (opencl…

d441dd1

…aw#28702) thanks @Glucksberg

Verified:
- pnpm install --frozen-lockfile
- pnpm build
- pnpm check
- pnpm test:macmini

Co-authored-by: Glucksberg <80581902+Glucksberg@users.noreply.github.com>
Co-authored-by: Tak Hoffman <781889+Takhoffman@users.noreply.github.com>

execute008 pushed a commit to execute008/openclaw that referenced this pull request


          fix(slack): guard Socket Mode listeners access during startup (opencl…

d13eba0

…aw#28702) thanks @Glucksberg

Verified:
- pnpm install --frozen-lockfile
- pnpm build
- pnpm check
- pnpm test:macmini

Co-authored-by: Glucksberg <80581902+Glucksberg@users.noreply.github.com>
Co-authored-by: Tak Hoffman <781889+Takhoffman@users.noreply.github.com>

dorgonman pushed a commit to kanohorizonia/openclaw that referenced this pull request


          fix(slack): guard Socket Mode listeners access during startup (opencl…

ff7304f

…aw#28702) thanks @Glucksberg

Verified:
- pnpm install --frozen-lockfile
- pnpm build
- pnpm check
- pnpm test:macmini

Co-authored-by: Glucksberg <80581902+Glucksberg@users.noreply.github.com>
Co-authored-by: Tak Hoffman <781889+Takhoffman@users.noreply.github.com>

sachinkundu pushed a commit to sachinkundu/openclaw that referenced this pull request


          fix(slack): guard Socket Mode listeners access during startup (opencl…

7c58baa

…aw#28702) thanks @Glucksberg

Verified:
- pnpm install --frozen-lockfile
- pnpm build
- pnpm check
- pnpm test:macmini

Co-authored-by: Glucksberg <80581902+Glucksberg@users.noreply.github.com>
Co-authored-by: Tak Hoffman <781889+Takhoffman@users.noreply.github.com>

zooqueen pushed a commit to hanzoai/bot that referenced this pull request


          fix(slack): guard Socket Mode listeners access during startup (opencl…

cde7406

…aw#28702) thanks @Glucksberg

Verified:
- pnpm install --frozen-lockfile
- pnpm build
- pnpm check
- pnpm test:macmini

Co-authored-by: Glucksberg <80581902+Glucksberg@users.noreply.github.com>
Co-authored-by: Tak Hoffman <781889+Takhoffman@users.noreply.github.com>

zooqueen added a commit to hanzoai/bot that referenced this pull request


          fix(slack): guard Socket Mode listeners access during startup (opencl…

d35eca8

…aw#28702) thanks @Glucksberg

Cherry-pick of upstream 3aad6c8.

Mateljan1 pushed a commit to Mateljan1/openclaw that referenced this pull request


          fix(slack): guard Socket Mode listeners access during startup (opencl…

438780c

…aw#28702) thanks @Glucksberg

Verified:
- pnpm install --frozen-lockfile
- pnpm build
- pnpm check
- pnpm test:macmini

Co-authored-by: Glucksberg <80581902+Glucksberg@users.noreply.github.com>
Co-authored-by: Tak Hoffman <781889+Takhoffman@users.noreply.github.com>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

channel: slack size: S