fix: replace subprocess spawn with direct session injection for Zulip extension #22

2026-06-24T20:24:42Z

abiba-bot commented

2026-06-24 20:24:42 +00:00

Problem

The Zulip extension spawned a brand-new pi -p --mode json subprocess for every incoming Zulip DM. This required a full cold-start: loading extensions, initializing the provider, and connecting the LLM. This caused ETIMEDOUT and ENOBUFS errors.

Additionally, the zulip-js import used a named { createClient } import, but the library only exports a default function.

Solution

Replaced the subprocess architecture with direct session injection:

pi.sendUserMessage() injects Zulip DMs directly into the current pi session
agent_end event handler captures the assistant response and relays it back
Pending reply queue handles ordered delivery

Changes

Removed spawnSync subprocess entirely
Fixed zulip-js import (CJS default export, not named)
Added types.d.ts for zulip-js type declarations
Uses yaml package for config parsing (matching main branch conventions)
Removed all subprocess timeout/buffer error paths

## Problem The Zulip extension spawned a brand-new `pi -p --mode json` subprocess for every incoming Zulip DM. This required a full cold-start: loading extensions, initializing the provider, and connecting the LLM. This caused ETIMEDOUT and ENOBUFS errors. Additionally, the `zulip-js` import used a named `{ createClient }` import, but the library only exports a default function. ## Solution Replaced the subprocess architecture with direct session injection: 1. `pi.sendUserMessage()` injects Zulip DMs directly into the current pi session 2. `agent_end` event handler captures the assistant response and relays it back 3. Pending reply queue handles ordered delivery ## Changes - Removed spawnSync subprocess entirely - Fixed zulip-js import (CJS default export, not named) - Added types.d.ts for zulip-js type declarations - Uses `yaml` package for config parsing (matching main branch conventions) - Removed all subprocess timeout/buffer error paths

abiba-bot added 1 commit 2026-06-24 20:24:42 +00:00

fix(zulip): replace subprocess spawn with direct session injection

CI / validate (pull_request) Failing after 3s

Details

ab3cb50eb4

- Removed spawnSync subprocess (pi -p --mode json) for Zulip DM processing
- Messages now inject directly into the current pi session via
  pi.sendUserMessage(), reusing the already-hot LLM — zero cold-start
- agent_end event handler captures assistant response and relays it to Zulip
- Added pending reply queue for ordered delivery of multiple Zulip messages
- Fixed zulip-js import (CJS default export, not named createClient)
- Added types.d.ts for zulip-js with default export declaration
- Updated package.json with yaml dependency, @earendil-works/pi-coding-agent
- Config supports both nested YAML and flat env-var formats
- Removed all subprocess timeout/buffer error paths (no longer needed)

jerome merged commit 0079d11d94 into main

2026-06-24 20:25:56 +00:00

abiba-bot referenced this pull request

2026-06-24 20:47:56 +00:00

feat(hermes): full Zulip adapter with subprocess agent routing + streaming #24

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: SyslogSolution/zulip-platform-plugins#22