@omnixal/openclaw-nats-plugin 0.1.15 → 0.1.17

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@omnixal/openclaw-nats-plugin",
-  "version": "0.1.15",
+  "version": "0.1.17",
   "description": "NATS JetStream event-driven plugin for OpenClaw",
   "license": "MIT",
   "type": "module",

package/sidecar/src/config.ts

@@ -17,7 +17,7 @@ export const envSchema = {
   },
   gateway: {
     wsUrl: Env.string({ default: 'ws://localhost:18789', env: 'OPENCLAW_WS_URL' }),
-    token: Env.string({ default: '', env: 'OPENCLAW_GATEWAY_TOKEN' }),
+    token: Env.string({ default: '', env: 'OPENCLAW_DEVICE_TOKEN' }),
   },
   consumer: {
     name: Env.string({ default: 'openclaw-main', env: 'NATS_CONSUMER_NAME' }),

package/sidecar/src/gateway/gateway-client.service.ts

@@ -15,6 +15,7 @@ export interface GatewayInjectPayload {
 export class GatewayClientService extends BaseService implements OnModuleInit, OnModuleDestroy {
   private ws: WebSocket | null = null;
   private connected = false;
+  private connectSent = false;
   private reconnectAttempt = 0;
   private reconnectTimer: Timer | null = null;
   private requestId = 0;
@@ -24,33 +25,35 @@ export class GatewayClientService extends BaseService implements OnModuleInit, O
   async onModuleInit(): Promise<void> {
     this.wsUrl = this.config.get('gateway.wsUrl');
     this.token = this.config.get('gateway.token');
-    if (this.wsUrl) {
+    if (this.wsUrl && this.token) {
       this.connect();
+    } else {
+      this.logger.warn('Gateway WebSocket not configured — skipping connection (need wsUrl + deviceToken)');
     }
   }
 
   private connect(): void {
     try {
-      const url = this.token ? `${this.wsUrl}?token=${this.token}` : this.wsUrl;
-      this.ws = new WebSocket(url);
+      this.connectSent = false;
+      this.ws = new WebSocket(this.wsUrl);
 
       this.ws.onopen = () => {
-        this.logger.info('Gateway WebSocket connected');
         this.reconnectAttempt = 0;
-        this.sendConnect();
+        this.logger.info('Gateway WebSocket opened, waiting for connect.challenge');
       };
 
       this.ws.onmessage = (event) => {
-        this.handleMessage(event);
+        this.handleMessage(String(event.data));
       };
 
       this.ws.onclose = () => {
         this.connected = false;
+        this.connectSent = false;
         this.scheduleReconnect();
       };
 
-      this.ws.onerror = (event) => {
-        this.logger.warn('Gateway WebSocket error', event);
+      this.ws.onerror = () => {
+        this.logger.warn('Gateway WebSocket error');
         this.connected = false;
       };
     } catch (err) {
@@ -59,23 +62,77 @@ export class GatewayClientService extends BaseService implements OnModuleInit, O
     }
   }
 
-  private handleMessage(event: { data: unknown }): void {
+  private handleMessage(data: string): void {
+    let frame: any;
     try {
-      const frame = JSON.parse(String(event.data));
-      if (frame.type === 'res' && frame.ok) {
-        this.connected = true;
-      }
+      frame = JSON.parse(data);
     } catch {
-      // ignore parse errors
+      this.logger.warn(`Failed to parse WebSocket message: ${data.slice(0, 200)}`);
+      return;
+    }
+
+    // Server challenge — respond with connect frame
+    if (frame.type === 'event' && frame.event === 'connect.challenge') {
+      this.logger.debug('Received connect.challenge from server');
+      this.sendConnectFrame();
+      return;
+    }
+
+    // Some gateway versions send an event before challenge; treat any pre-connect event as trigger
+    if (!this.connectSent && frame.type === 'event') {
+      this.logger.debug('Received event before connect sent, sending connect frame');
+      this.sendConnectFrame();
+      return;
+    }
+
+    // Successful connect response — must be hello-ok
+    if (frame.type === 'res' && frame.ok === true) {
+      const payload = frame.payload;
+      if (payload?.type === 'hello-ok') {
+        if (!this.connected) {
+          this.connected = true;
+          this.logger.info('OpenClaw handshake complete — connected');
+        }
+        return;
+      }
+      // Regular RPC ok response (e.g. for inject calls)
+      this.logger.debug('Received RPC ok response', { id: frame.id });
+      return;
+    }
+
+    // Error response
+    if (frame.type === 'res' && frame.ok === false) {
+      this.logger.warn('Gateway RPC error', { id: frame.id, error: frame.error });
     }
   }
 
-  private sendConnect(): void {
+  private sendConnectFrame(): void {
+    if (!this.ws || this.ws.readyState !== WebSocket.OPEN || this.connectSent) return;
+    this.connectSent = true;
+    this.logger.info('Sending connect frame');
+
     this.send({
       type: 'req',
-      id: ++this.requestId,
+      id: `connect-${++this.requestId}`,
       method: 'connect',
-      params: {},
+      params: {
+        minProtocol: 3,
+        maxProtocol: 3,
+        client: {
+          id: 'nats-sidecar',
+          version: '1.0.0',
+          platform: 'linux',
+          mode: 'backend',
+        },
+        role: 'operator',
+        scopes: ['operator.read'],
+        caps: [],
+        commands: [],
+        permissions: {},
+        auth: { token: this.token },
+        locale: 'en-US',
+        userAgent: 'nats-sidecar/1.0.0',
+      },
     });
   }
 
@@ -87,7 +144,7 @@ export class GatewayClientService extends BaseService implements OnModuleInit, O
 
   private scheduleReconnect(): void {
     if (this.reconnectTimer) return;
-    const delay = Math.min(1000 * Math.pow(2, this.reconnectAttempt), 30000);
+    const delay = Math.min(1000 * Math.pow(2, this.reconnectAttempt), 30_000);
     this.reconnectAttempt++;
     this.logger.debug(`Reconnecting to Gateway in ${delay}ms (attempt ${this.reconnectAttempt})`);
     this.reconnectTimer = setTimeout(() => {
@@ -102,7 +159,7 @@ export class GatewayClientService extends BaseService implements OnModuleInit, O
     }
     this.send({
       type: 'req',
-      id: ++this.requestId,
+      id: `rpc-${++this.requestId}`,
       method: 'send',
       params: {
         target: payload.target,
@@ -127,5 +184,6 @@ export class GatewayClientService extends BaseService implements OnModuleInit, O
       this.ws = null;
     }
     this.connected = false;
+    this.connectSent = false;
   }
 }