Slack support by PCBZ · Pull Request #8 · PCBZ/openclaw-deploy-kit

PCBZ · 2026-04-18T04:39:22Z

No description provided.

…configurations

…iguration

…mple and update bootstrap script

…port

…ctions

Copilot

Pull request overview

Adds Slack (Socket Mode) configuration to the DigitalOcean Terraform deployment, wiring Slack tokens through Terraform into the OpenClaw bootstrap config.

Changes:

Introduces Terraform variables and example tfvars entries for Slack app/bot tokens.
Passes Slack tokens into droplet user_data and conditionally enables the Slack extension in openclaw.json.
Updates docs/examples to mention Slack and adds Slack env var plumbing.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 7 comments.

Show a summary per file

File	Description
`terraform/digitalOcean/variables.tf`	Adds `slack_app_token` / `slack_bot_token` Terraform variables.
`terraform/digitalOcean/terraform.tfvars.example`	Documents optional Slack tokens and Telegram owner ID.
`terraform/digitalOcean/main.tf`	Passes Slack variables into `templatefile()` for bootstrap rendering.
`terraform/digitalOcean/bootstrap.sh`	Installs Slack deps, conditionally loads Slack extension, writes Slack tokens to env/config, plus additional bootstrap behavior changes.
`terraform/digitalOcean/.envrc`	Exports Slack tokens into `TF_VAR_*` for Terraform.
`README.md`	Mentions Slack support and Slack env vars; removes model alias table.
`.env.example`	Adds `SLACK_APP_TOKEN` / `SLACK_BOT_TOKEN` placeholders.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-04-18T05:08:24Z

    "entries": {
-      "telegram": { "enabled": true },
+      "telegram": { "enabled": true }%{if slack_app_token != "" && slack_bot_token != ""},
+      "slack": { "enabled": true }%{endif},


The Slack conditional in plugins.entries leaves a dangling comma when Slack tokens are empty: the comma after %{endif} is outside the conditional, so the rendered JSON becomes invalid (a standalone , line) when Slack is disabled. Move the comma inside the conditional or restructure so the JSON is valid in both branches (e.g., include the comma with the Slack entry or generate entries via jsonencode).

Suggested change

"slack": { "enabled": true }%{endif},

"slack": { "enabled": true },%{endif}

Copilot · 2026-04-18T05:08:24Z


+# ── Telegram (Optional) ─────────────────────────────────────
+# Your Telegram numeric user ID (get from @userinfobot)
+# Grants /model and other privileged commands. Leave empty ("") to disable


The comment for telegram_owner_id says leaving it empty disables Telegram, but the config still enables the Telegram channel unconditionally; an empty owner ID only removes the allowFrom restriction for privileged commands. Please update the comment to reflect the actual behavior (optional privileged access, not Telegram enable/disable).

Suggested change

# Grants /model and other privileged commands. Leave empty ("") to disable

# Grants /model and other privileged commands. Leave empty ("") to skip owner-based restriction

Copilot · 2026-04-18T05:08:24Z

 - Web search via Brave Search (falls back to DuckDuckGo)
 - 8 switchable free LLM models via `/model <alias>`
 - Secrets managed via `.env` — never committed


The README claims "8 switchable free LLM models", but the bootstrap config now defines many more models and the primary model is openrouter/openai/gpt-4o-mini (not marked :free). Please update the README to match the actual configured models/free-tier expectations, or adjust the config to keep the README statement true.

Copilot · 2026-04-18T05:08:25Z

@@ -26,6 +28,8 @@ One-command deployment of an [OpenClaw](https://openclaw.bot) AI agent as a Tele
 - DigitalOcean account + API token
 - OpenRouter API key
 - Telegram bot token (from [@BotFather](https://t.me/BotFather))


Slack tokens are listed under "Prerequisites", but the Terraform variables default to empty strings and the bootstrap only enables Slack when both tokens are set. Consider marking these as optional prerequisites (or move them into an optional section) to avoid implying Slack is required for a Telegram-only deployment.

Suggested change

- Telegram bot token (from [@BotFather](https://t.me/BotFather))

- Telegram bot token (from [@BotFather](https://t.me/BotFather))

### Optional (for Slack support)

Only required if you want to enable Slack in addition to Telegram:

Copilot · 2026-04-18T05:08:25Z

 In Telegram, use `/model <alias>`:



The "Switching Models" section now ends with In Telegram, use /model <alias>: but no longer includes the alias/model list or any guidance, leaving the instructions incomplete. Either restore the table/list (updated to the new aliases) or revise the section so it's actionable.

Suggested change

In Telegram, use `/model <alias>`:

In Telegram, use `/model <alias>` to switch the active model for your chat.

Replace `<alias>` with one of the model aliases configured for your bot deployment. For example:

- `/model llama`

- `/model mistral`

If an alias is not recognized, check the model alias configuration used by your deployment and then retry with a valid alias.

Copilot · 2026-04-18T05:08:25Z

      "model": {
-        "primary": "openrouter/meta-llama/llama-3.3-70b-instruct:free",
+        "primary": "openrouter/openai/gpt-4o-mini",
        "fallbacks": [
+          "openrouter/anthropic/claude-haiku-4.5",
+          "openrouter/deepseek/deepseek-v3.2",
+          "openrouter/meta-llama/llama-3.3-70b-instruct:free",
          "openrouter/auto"
        ]
      },
      "models": {
-        "openrouter/meta-llama/llama-3.3-70b-instruct:free": {"alias": "llama"},
-        "openrouter/cognitivecomputations/dolphin-mistral-24b-venice-edition:free": {"alias": "uncensored"},
-        "openrouter/google/gemma-4-31b-it:free": {"alias": "gemma"},
-        "openrouter/nousresearch/hermes-3-llama-3.1-405b:free": {"alias": "hermes"},
-        "openrouter/nvidia/nemotron-3-super-120b-a12b:free": {"alias": "nemotron"},
-        "openrouter/openai/gpt-oss-120b:free": {"alias": "gpt"},
-        "openrouter/qwen/qwen3-coder:free": {"alias": "coder"},
-        "openrouter/auto": {"alias": "auto"}
+        "openrouter/anthropic/claude-opus-4.6":                                        {"alias": "opus"},
+        "openrouter/anthropic/claude-sonnet-4.6":                                      {"alias": "sonnet"},
+        "openrouter/anthropic/claude-haiku-4.5":                                       {"alias": "haiku"},
+        "openrouter/openai/gpt-5.4":                                                   {"alias": "gpt5"},
+        "openrouter/openai/gpt-4o":                                                    {"alias": "gpt4o"},
+        "openrouter/openai/gpt-4o-mini":                                               {"alias": "mini"},
+        "openrouter/google/gemini-2.5-pro":                                            {"alias": "gemini-pro"},
+        "openrouter/google/gemini-2.5-flash":                                          {"alias": "flash"},
+        "openrouter/deepseek/deepseek-v3.2":                                           {"alias": "deepseek"},
+        "openrouter/deepseek/deepseek-r1":                                             {"alias": "r1"},
+        "openrouter/mistralai/devstral-small":                                         {"alias": "devstral"},
+        "openrouter/meta-llama/llama-3.3-70b-instruct:free":                           {"alias": "llama"},
+        "openrouter/nvidia/nemotron-3-super-120b-a12b:free":                           {"alias": "nemotron"},
+        "openrouter/qwen/qwen3-coder:free":                                            {"alias": "coder"},
+        "openrouter/cognitivecomputations/dolphin-mistral-24b-venice-edition:free":    {"alias": "uncensored"},
+        "openrouter/auto":                                                             {"alias": "auto"}
      },
-      "compaction": {
-        "mode": "safeguard",
-        "reserveTokensFloor": 20000
-      }
+      "compaction": { "mode": "safeguard", "reserveTokensFloor": 4000 }
    }


This PR is titled "Slack support", but bootstrap.sh also changes the default model, model alias set, compaction settings, onboarding behavior, and approval automation timing/logic. Consider splitting these unrelated operational changes into separate PRs (or updating the PR description/title) to keep review and rollback risk manageable.

Copilot · 2026-04-18T05:08:25Z

 write_config() {
 cat > /root/.openclaw/openclaw.json << JSONEOF
 {
  "gateway": {
    "bind": "lan",
-    "auth": {
-      "mode": "token",
-      "token": "${openclaw_gateway_token}"
-    },
+    "auth": { "mode": "token", "token": "${openclaw_gateway_token}" },
    "mode": "local",
-    "remote": {
-      "token": "${openclaw_gateway_token}"
-    }
+    "remote": { "token": "${openclaw_gateway_token}" }


write_config() uses an unquoted heredoc (<< JSONEOF), so bash will still perform $VAR/command substitutions on the rendered JSON at runtime. If any injected token ever contains $, backticks, or $(...), the resulting openclaw.json can be corrupted. Use a quoted heredoc delimiter (e.g., <<'JSONEOF') to prevent shell expansion inside the JSON payload.

PCBZ added 3 commits April 13, 2026 21:51

enhance bootstrap script: validate required secrets and update model …

23eb826

…configurations

add Slack support: include app and bot tokens in environment and conf…

16aaba5

…iguration

add Slack configuration: include app-level and bot tokens in .env.exa…

8d360d7

…mple and update bootstrap script

PCBZ self-assigned this Apr 18, 2026

PCBZ added the enhancement New feature or request label Apr 18, 2026

PCBZ added 3 commits April 17, 2026 21:51

Merge branch 'main' of github.com:PCBZ/OpenClaw_Docker into Slack_Sup…

9565a02

…port

remove scheduled tasks lock file

12e3a59

update README: enhance Slack support details and configuration instru…

28f847e

…ctions

PCBZ requested a review from Copilot April 18, 2026 05:05

Copilot started reviewing on behalf of PCBZ April 18, 2026 05:05 View session

Copilot AI reviewed Apr 18, 2026

View reviewed changes

PCBZ merged commit 3a85e07 into main Apr 19, 2026
9 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Slack support#8

Slack support#8
PCBZ merged 6 commits intomainfrom
Slack_Support

PCBZ commented Apr 18, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Apr 18, 2026

Uh oh!

Copilot AI Apr 18, 2026

Uh oh!

Copilot AI Apr 18, 2026

Uh oh!

Copilot AI Apr 18, 2026

Uh oh!

Copilot AI Apr 18, 2026

Uh oh!

Copilot AI Apr 18, 2026

Uh oh!

Copilot AI Apr 18, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

	"slack": { "enabled": true }%{endif},
	"slack": { "enabled": true },%{endif}

	# Grants /model and other privileged commands. Leave empty ("") to disable
	# Grants /model and other privileged commands. Leave empty ("") to skip owner-based restriction

-In Telegram, use `/model <alias>`:
+In Telegram, use `/model <alias>` to switch the active model for your chat.
+Replace `<alias>` with one of the model aliases configured for your bot deployment. For example:
+- `/model llama`
+- `/model mistral`
+If an alias is not recognized, check the model alias configuration used by your deployment and then retry with a valid alias.

Conversation

PCBZ commented Apr 18, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Apr 18, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 18, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 18, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 18, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 18, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 18, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 18, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants