PR #71550: feat(slack): expose channels.slack.logLevel so operators can silence socket-mode pong-timeout noise (#71531)

• Repository: openclaw/openclaw
• Author: hclsys
• State: open | merged: False
• Link: https://github.com/openclaw/openclaw/pull/71550

Description (problem / solution / changelog)

Closes #71531.

Bug

The bundled @slack/socket-mode 2.0.6 emits an unconditional WARN every ~30s when the periodic pong is late:

A pong wasn't received from the server before the timeout of 5000ms!

The library auto-reconnects cleanly, so the WARN is noise, not a failure signal — but it floods gateway.err.log and operators end up filtering it externally to recover signal-to-noise. The existing pingPongLoggingEnabled flag is already the default false in 2.0.6 but does NOT gate the pong-timeout WARN — it fires unconditionally in SlackWebSocket.js.

The only effective knob is bolt's App({ logLevel }) option, which gates ALL sub-library output. openclaw was not passing it, so bolt defaulted to INFO.

Fix

Add an optional channels.slack.logLevel knob (DEBUG | INFO | WARN | ERROR) and forward it to bolt's App constructor when set. When omitted, behavior is unchanged (bolt's INFO default).

channels:
  slack:
    logLevel: ERROR   # silences socket-mode pong-timeout WARNs

Diff

• src/config/zod-schema.providers-core.ts — add logLevel to SlackConfigSchema
• extensions/slack/src/monitor/provider-support.ts — accept + forward logLevel in createSlackBoltApp
• extensions/slack/src/monitor/provider.ts — thread slackCfg.logLevel through
• extensions/slack/src/monitor/provider.interop.test.ts — 2 new tests:
  • forwards logLevel to bolt's App when provided
  • omits logLevel when not set so bolt's INFO default is preserved

48 LOC across 4 files. Lint clean (pnpm oxlint — 0 warnings, 0 errors).

Out of scope

• Per-account override (channels.slack.accounts.<id>.logLevel) — can be added later if needed; the channel-level knob covers the reported use case.
• Per-sub-library filtering — bolt doesn't expose this granularity directly.
• Custom logger injection — same.

🤖 generated with assistance from Claude Code Co-authored-by: HCL [email protected]

Changed files

• extensions/slack/src/monitor/provider-support.ts (modified, +14/-0)
• extensions/slack/src/monitor/provider.interop.test.ts (modified, +35/-0)
• extensions/slack/src/monitor/provider.ts (modified, +5/-0)
• src/config/types.slack.ts (modified, +8/-0)
• src/config/zod-schema.providers-core.ts (modified, +6/-0)

PR #73162: fix(slack): remove socket reconnect attempt cap so gateway stays connected indefinitely

• Repository: openclaw/openclaw
• Author: suboss87
• State: open | merged: False
• Link: https://github.com/openclaw/openclaw/pull/73162

Description (problem / solution / changelog)

What and why

After 12 consecutive failed Slack socket reconnects the provider threw an error and exited permanently. The gateway never restarts providers after they exit, so all subsequent Slack messages were silently dropped while the bot still appeared online in Slack (bots always show green in Slack's UI regardless of socket state).

This is the root cause of #72808. Users who hit a network blip or Slack server restart at the wrong time would lose Slack entirely until they manually restarted OpenClaw.

Fix

Set maxAttempts: 0 in SLACK_SOCKET_RECONNECT_POLICY. The existing maxAttempts > 0 && guard in provider.ts already treats 0 as unlimited — this convention was clearly intentional given the || "∞" fallback already present in the log format strings.

Non-recoverable auth errors (account_inactive, invalid_auth, token_revoked, etc.) already bail immediately via isNonRecoverableSlackAuthError before the maxAttempts check runs, so removing the cap does not create an infinite retry loop for permanent credential failures.

With maxAttempts: 0 the reconnect log now reads retry N/∞ in Xs, which is the correct behavior for a long-running gateway service.

Testing

• 813 Slack extension tests pass unchanged (no test asserted on maxAttempts behavior)
• Extension prod + test typecheck lanes green

Fixes #72808

Changed files

• extensions/slack/src/monitor/reconnect-policy.ts (modified, +1/-1)