fix(slack): wrap session key in backticks to prevent emoji shortcode parsing by pushkarsingh32 · Pull Request #30266 · openclaw/openclaw

pushkarsingh32 · 2026-03-01T03:39:42Z

Summary

Wraps the session key in the usage footer with backticks (inline code) so Slack doesn't auto-parse colon-delimited segments (:main:, :slack:, :direct:) as emoji shortcodes

Before: Usage: 3 in / 1.8k out · session agent:main:slack:direct:u045c0gtsmb
(:main:, :slack:, :direct: rendered as grey emoji placeholder boxes)

After: Usage: 3 in / 1.8k out · session `agent:main:slack:direct:u045c0gtsmb`
(session key rendered as inline code — no emoji parsing)

Changes

src/auto-reply/reply/agent-runner.ts: Wrap sessionKey in backticks when appending to usage footer
src/auto-reply/reply/agent-runner.misc.runreplyagent.test.ts: Update test assertion to match new format

Test plan

Existing test suite passes (pnpm test — 23/23 tests pass in the affected file)
Manual verification: send a message via Slack DM with responseUsage: "full" and confirm the session key renders as inline code without emoji shortcode parsing

Fixes #30258

greptile-apps · 2026-03-01T03:43:11Z

Greptile Summary

Wraps the session key in backticks when appending to the usage footer (when responseUsage="full") to prevent Slack from parsing colon-delimited segments like :main:, :slack:, :direct: as emoji shortcodes.

Modified agent-runner.ts to wrap sessionKey in backticks in the formatted usage string
Updated test assertion in agent-runner.misc.runreplyagent.test.ts to match the new format with backticks
This is a targeted fix that only affects the display of session keys to end users in Slack, not internal logging or processing
The change is minimal and well-tested

Confidence Score: 5/5

This PR is safe to merge with minimal risk
The change is minimal (wrapping a single variable in backticks), well-targeted to fix a specific Slack rendering issue, and includes a corresponding test update. No logical errors or edge cases identified.
No files require special attention

_{Last reviewed commit: 4cc21dc}

…parsing Session keys like `agent:main:slack:direct:u045c0gtsmb` contain colon-delimited segments that Slack auto-parses as emoji shortcodes. Wrapping the session key in backticks (inline code) prevents Slack from interpreting `:main:`, `:slack:`, `:direct:` as emoji. Fixes openclaw#30258

Takhoffman · 2026-03-01T15:24:00Z

PR #30266 - fix(slack): wrap session key in backticks to prevent emoji shortcode parsing (#30266)

Merged via squash.

Merge commit: 4ba0a4d
Verified: pnpm build, pnpm check, pnpm test:macmini
Changes made:
M\tCHANGELOG.md
M\textensions/slack/src/channel.test.ts
Why these changes were made:
Added the required changelog entry and updated the Slack plugin config test to match the current async snapshot API ( argument + awaited return), so the PR remains minimal and passes strict gates on latest main.
Changelog: CHANGELOG.md updated=true required=true opt_out=false

Thanks @pushkarsingh32!

Takhoffman · 2026-03-01T15:24:07Z

Correction on merge summary details:

Changes made: CHANGELOG.md, extensions/slack/src/channel.test.ts
Why: add required changelog entry and update Slack config test to current async snapshot API (cfg argument + awaited return) so gates pass on latest main.

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d6aabc4591

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

chatgpt-codex-connector · 2026-03-01T15:27:23Z

src/auto-reply/reply/agent-runner.ts

      });
      if (formatted && responseUsageMode === "full" && sessionKey) {
-        formatted = `${formatted} · session ${sessionKey}`;
+        formatted = `${formatted} · session \`${sessionKey}\``;


Escape backticks before wrapping session key in code span

This interpolation wraps sessionKey in single backticks without escaping the key itself, so any backtick inside the key prematurely closes the code span and produces malformed markdown in the footer. Session keys are built from peer IDs that are lowercased but not punctuation-sanitized (src/routing/session-key.ts:129-142), and some channels can supply IDs/nicks containing ` (for example via extensions/irc/src/inbound.ts:280-288), so with responseUsage=full the Slack formatting fix can regress to broken rendering and re-enable shortcode parsing on the trailing segment.

Useful? React with 👍 / 👎.

@pushkarsingh32

…parsing (openclaw#30266) thanks @pushkarsingh32 Cherry-pick of upstream 4ba0a4d.

@pushkarsingh32