feat: Zulip channel plugin by FtlC-ian · Pull Request #12183 · openclaw/openclaw

FtlC-ian · 2026-02-08T23:49:00Z

Zulip Channel Plugin

Adds a full-featured Zulip channel plugin for OpenClaw.

Features

Event queue polling with long-poll and exponential backoff
Authenticated upload downloads from Zulip's /user_uploads/ paths
Outbound file uploads via /api/v1/user_uploads
Reaction indicators — 👀 on start, ✅ on success, ⚠️ on error (configurable)
Topic directive — agents can switch topics with [[zulip_topic: <topic>]]
HTTP retry with exponential backoff for 429/502/503/504
Full actions API — stream CRUD, user management, reactions, search, pins
Emoji normalization — accepts both :eyes: and eyes formats
Topic → session mapping — each topic gets its own conversation session
Channel docs and unit tests (19 passing)

Config

{
  channels: {
    zulip: {
      enabled: true,
      url: "https://zulip.example.com",
      email: "[email protected]",
      apiKey: "...",
      streams: ["my-stream"],
      reactions: { enabled: true, onStart: "eyes", onSuccess: "check_mark" },
    },
  },
}

Context

This was developed to address the Zulip integration request in #5163. We reviewed both existing PRs (#8365 by @rafaelreis-r and #9643 by @jamie-dit) and incorporated the best ideas from each:

From Zulip: add channel plugin #9643: reaction indicator pattern, check emoji naming, DELETE reactions via query params, topic directive, emoji normalization, docs structure
From Add Zulip channel plugin #8365: reference for basic polling/auth patterns

What this PR adds beyond both:

Full channel actions API (stream CRUD, user management, org settings)
Authenticated upload download + outbound upload support
Comprehensive backoff/retry across all API calls (not just polling)

Testing

✅ Unit tests: 19 passing (upload extraction, emoji normalization, channel config)
✅ Manual testing against live Zulip instance
✅ Security audit (no blocking issues)

Refs: #5163, #8365, #9643

Greptile Overview

Greptile Summary

This PR adds a new @openclaw/zulip channel plugin under extensions/zulip, including config schema, onboarding hooks, outbound send support (including uploads), and an event-queue long-polling monitor that routes stream/topic messages into per-topic sessions. It also updates the channel docs index and marks Zulip as markdown-capable in src/utils/message-channel.ts.

Key integration points:

Channel plugin registration via extensions/zulip/index.ts + openclaw.plugin.json + per-extension package.json.
Runtime message ingestion via extensions/zulip/src/zulip/monitor.ts (event queue registration + long-poll loop, routing/session mapping, optional reactions/typing indicators, media download from /user_uploads/).
Outbound messaging via extensions/zulip/src/zulip/send.ts (target parsing, optional upload to Zulip, markdown table conversion).

Primary merge blockers are config write/read mismatches in the channel setup path (URL written to a different key than the rest of the plugin reads), which will prevent successful configuration from actually working at runtime.

Confidence Score: 3/5

This PR is close, but has a config write/read mismatch that will break real-world setup.
Main functionality is self-contained in the new extension and the overall approach matches existing channel patterns, but the setup path writes the Zulip base URL under the wrong config key, which will cause configured accounts to fail to start. Fixing that and ensuring event polling types are stable would make this safer to merge.
extensions/zulip/src/channel.ts, extensions/zulip/src/zulip/monitor.ts

_{(2/5) Greptile learns from your feedback when you react with thumbs up/down!}

Adds a full Zulip channel plugin with: - Event queue polling with exponential backoff - Authenticated upload downloads from /user_uploads/ - Outbound file uploads via /api/v1/user_uploads - Reaction indicators (eyes → check_mark/warning) - Topic directive [[zulip_topic: <topic>]] - HTTP retry with backoff (429/502/503/504) - Full actions API (stream CRUD, user management, reactions) - Channel docs and unit tests Refs: #5163, openclaw#8365, openclaw#9643

greptile-apps

_{2 files reviewed, 3 comments}

_{Edit Code Review Agent Settings | Greptile}

greptile-apps · 2026-02-08T23:50:33Z

extensions/zulip/src/channel.ts

+      }
+      return null;
+    },


Wrong config key
applyAccountConfig writes the base URL to channels.zulip.url, but the rest of the plugin reads account.baseUrl (resolved from the url field in config). This leaves the default account unconfigured at runtime even though setup succeeded.

Suggested change

}

return null;

},

...(baseUrl ? { url: baseUrl } : {}),

Prompt To Fix With AI

This is a comment left during a code review. Path: extensions/zulip/src/channel.ts Line: 281:283 Comment: **Wrong config key** `applyAccountConfig` writes the base URL to `channels.zulip.url`, but the rest of the plugin reads `account.baseUrl` (resolved from the `url` field in config). This leaves the default account unconfigured at runtime even though setup succeeded. ```suggestion ...(baseUrl ? { url: baseUrl } : {}), ``` How can I resolve this? If you propose a fix, please make it concise.

greptile-apps · 2026-02-08T23:50:34Z

extensions/zulip/src/channel.ts

+          ...next,
+          channels: {
+            ...next.channels,


Wrong config key
Same issue as the default-account path above: the account-scoped config writes channels.zulip.accounts[accountId].url but uses the url field name inconsistently elsewhere (resolved as baseUrl). This makes non-default accounts appear configured but fail at runtime when account.baseUrl is read.

Suggested change

...next,

channels: {

...next.channels,

...(baseUrl ? { url: baseUrl } : {}),

Prompt To Fix With AI

This is a comment left during a code review. Path: extensions/zulip/src/channel.ts Line: 306:308 Comment: **Wrong config key** Same issue as the default-account path above: the account-scoped config writes `channels.zulip.accounts[accountId].url` but uses the `url` field name inconsistently elsewhere (resolved as `baseUrl`). This makes non-default accounts appear configured but fail at runtime when `account.baseUrl` is read. ```suggestion ...(baseUrl ? { url: baseUrl } : {}), ``` How can I resolve this? If you propose a fix, please make it concise.

greptile-apps · 2026-02-08T23:50:35Z

extensions/zulip/src/zulip/monitor.ts

+          onModelSelected,
+        },
+      });
+    } catch (err) {


Event ID type mismatch
lastEventId is assigned from events[events.length - 1].id without normalization; Zulip's event IDs are numbers, and passing a number where the client expects a string can break subsequent polling (e.g., if request building/URLSearchParams assumes string). Consider coercing to String(...) at assignment to keep the type stable across re-register + retry paths.

Prompt To Fix With AI

This is a comment left during a code review. Path: extensions/zulip/src/zulip/monitor.ts Line: 719:722 Comment: **Event ID type mismatch** `lastEventId` is assigned from `events[events.length - 1].id` without normalization; Zulip's event IDs are numbers, and passing a number where the client expects a string can break subsequent polling (e.g., if request building/URLSearchParams assumes string). Consider coercing to `String(...)` at assignment to keep the type stable across re-register + retry paths. How can I resolve this? If you propose a fix, please make it concise.

Addresses Greptile review feedback: ensure lastEventId stays as a number type when assigned from event response data.

FtlC-ian

Thanks for the review! Addressing the feedback:

Config key (lines 283, 308): This is actually correct as-is. The config schema field is url (see config-schema.ts), and accounts.ts reads merged.url to resolve baseUrl. Writing { url: baseUrl } maps the user input to the correct config key.

Event ID type (line 722): Good catch — added Number() coercion in 4e52c40 to keep the type stable.

rafaelreis-r · 2026-02-11T17:44:30Z

Note for Zulip plugin users: Zulip natively renders markdown tables, but OpenClaw's default table mode converts them to code blocks. To get proper table rendering in Zulip, add this to your config:

\\json
{
channels: {
zulip: {
markdown: {
tables: off
}
}
}
}
\\

Without this, tables appear as monospace code instead of formatted tables. Might be worth setting \off\ as the default for the Zulip channel since Zulip supports GFM tables natively.

FtlC-ian · 2026-02-11T18:41:38Z

Good catch! Added ["zulip", "off"] to DEFAULT_TABLE_MODES in c77c9c5 — Zulip now defaults to passing tables through as-is since it handles GFM tables natively. No config needed.

Adds a full Zulip channel plugin with: - Event queue polling with exponential backoff - Authenticated upload downloads from /user_uploads/ - Outbound file uploads via /api/v1/user_uploads - Reaction indicators (eyes → check_mark/warning) - Topic directive [[zulip_topic: <topic>]] - HTTP retry with backoff (429/502/503/504) - Full actions API (stream CRUD, user management, reactions) - Channel docs and unit tests Refs: #5163, openclaw#8365, openclaw#9643

Addresses Greptile review feedback: ensure lastEventId stays as a number type when assigned from event response data.

Renamed 4 duplicate function names to avoid code-analysis failures: - normalizeEmojiName → normalizeZulipEmojiName (uploads.ts) - readStringArrayParam → parseStringArrayParam (actions.ts) - resolveLocalPath → resolveZulipLocalPath (send.ts) - sleep → delay (monitor.ts, client.ts) Formatted accounts.ts and monitor.ts with oxfmt.

…ame function

openclaw-barnacle · 2026-02-12T22:11:28Z

Closing this PR because it has more than 20 labels, which usually means the branch is too noisy. Please recreate the PR from a clean branch.

openclaw-barnacle · 2026-02-12T22:14:42Z

Closing this PR because it has more than 20 labels, which usually means the branch is too noisy. Please recreate the PR from a clean branch.

Adds a full-featured Zulip channel plugin with: Core Features: - Event queue polling with exponential backoff and auto-recovery - Authenticated upload downloads from /user_uploads/ - Outbound file uploads via /api/v1/user_uploads - Reaction indicators (eyes → check_mark/warning) - Topic directive [[zulip_topic: <topic>]] - HTTP retry with backoff (429/502/503/504) - Full actions API (stream CRUD, user management, reactions) - Channel docs and unit tests Key Improvement Over Existing PRs: - Implements CONCURRENT message processing with staggered start times - Other PRs (openclaw#9643, openclaw#14182) process messages sequentially, causing reply delays and "message dumps" when multiple messages arrive - This implementation starts processing each message immediately with a 200ms stagger for natural conversation flow - Replies arrive as each finishes instead of all at once Technical Details: - Fire-and-forget message processing with per-message error handling - Maintains event cursor and queue ID correctly during concurrent processing - Tested live and confirmed significant UX improvement Refs: openclaw#5163, openclaw#8365, openclaw#9643, openclaw#12183

openclaw-barnacle bot added the docs Improvements or additions to documentation label Feb 8, 2026

greptile-apps bot reviewed Feb 8, 2026

View reviewed changes

fix: coerce event ID to number in poll loop

4e52c40

Addresses Greptile review feedback: ensure lastEventId stays as a number type when assigned from event response data.

FtlC-ian commented Feb 9, 2026

View reviewed changes

FtlC-ian added 2 commits February 8, 2026 18:05

chore: update pnpm-lock.yaml for zulip extension

6a84bf3

style: format zulip extension with oxfmt

d4db981

Copilot AI mentioned this pull request Feb 9, 2026

Add OpenClaw repository analysis: branches and PR history (partial) ralf003/openclaw#1

Draft

simpliq-marvin mentioned this pull request Feb 10, 2026

zulip: follow-up hardening on top of #12183 FtlC-ian/openclaw#1

Open

FtlC-ian added 2 commits February 10, 2026 07:41

fix(zulip): harden monitor loop and upload auth safety

6e22189

feat(zulip): add url alias precedence for site/realm

f0e00aa

rafaelreis-r mentioned this pull request Feb 11, 2026

Add Zulip channel plugin #8365

Closed

fix: default Zulip markdown tables to off (native GFM support)

c77c9c5

FtlC-ian added 7 commits February 11, 2026 12:44

fix: coerce event ID to number in poll loop

7e38746

Addresses Greptile review feedback: ensure lastEventId stays as a number type when assigned from event response data.

style: format zulip extension with oxfmt

21adcf3

fix(zulip): harden monitor loop and upload auth safety

fe95eab

feat(zulip): add url alias precedence for site/realm

86c106a

fix: default Zulip markdown tables to off (native GFM support)

0dfdc52

chore: regenerate pnpm-lock.yaml after rebase

a63e9fe

FtlC-ian force-pushed the feat/zulip-plugin branch from c77c9c5 to a63e9fe Compare February 11, 2026 18:45

FtlC-ian added 2 commits February 11, 2026 12:55

merge: keep our CI fixes for zulip extension

a1e0045

openclaw-barnacle bot added channel: bluebubbles Channel integration: bluebubbles channel: discord Channel integration: discord channel: googlechat Channel integration: googlechat channel: imessage Channel integration: imessage channel: line Channel integration: line labels Feb 11, 2026

fix(zulip): update test imports to match renamed normalizeZulipEmojiN…

ee64aea

…ame function

thewilloftheshadow added the trigger-response This label just triggers the auto-response workflow and should be ignored otherwise. label Feb 12, 2026

openclaw-barnacle bot removed the trigger-response This label just triggers the auto-response workflow and should be ignored otherwise. label Feb 12, 2026

openclaw-barnacle bot closed this Feb 12, 2026

thewilloftheshadow added the trigger-response This label just triggers the auto-response workflow and should be ignored otherwise. label Feb 12, 2026

openclaw-barnacle bot removed the trigger-response This label just triggers the auto-response workflow and should be ignored otherwise. label Feb 12, 2026

FtlC-ian mentioned this pull request Feb 12, 2026

feat: Zulip channel plugin with concurrent message processing #15051

Closed

jamie-dit mentioned this pull request Feb 14, 2026

feat: add Zulip channel plugin #15887

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat: Zulip channel plugin#12183

feat: Zulip channel plugin#12183
FtlC-ian wants to merge 17 commits intoopenclaw:mainfrom
FtlC-ian:feat/zulip-plugin

FtlC-ian commented Feb 8, 2026 •

edited by greptile-apps bot

Loading

Uh oh!

greptile-apps bot left a comment

Uh oh!

greptile-apps bot Feb 8, 2026

Uh oh!

greptile-apps bot Feb 8, 2026

Uh oh!

greptile-apps bot Feb 8, 2026

Uh oh!

FtlC-ian left a comment

Uh oh!

rafaelreis-r commented Feb 11, 2026

Uh oh!

FtlC-ian commented Feb 11, 2026

Uh oh!

openclaw-barnacle bot commented Feb 12, 2026

Uh oh!

openclaw-barnacle bot commented Feb 12, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Uh oh!

Conversation

FtlC-ian commented Feb 8, 2026 • edited by greptile-apps bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Zulip Channel Plugin

Features

Config

Context

Testing

Greptile Overview

Greptile Summary

Confidence Score: 3/5

Uh oh!

greptile-apps bot left a comment

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot Feb 8, 2026

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot Feb 8, 2026

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot Feb 8, 2026

Choose a reason for hiding this comment

Uh oh!

FtlC-ian left a comment

Choose a reason for hiding this comment

Uh oh!

rafaelreis-r commented Feb 11, 2026

Uh oh!

FtlC-ian commented Feb 11, 2026

Uh oh!

openclaw-barnacle bot commented Feb 12, 2026

Uh oh!

openclaw-barnacle bot commented Feb 12, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

FtlC-ian commented Feb 8, 2026 •

edited by greptile-apps bot

Loading