@rebon/cli-darwin-arm64 1.8.6 → 1.9.1

package/bin/plugin-host/src/host.mjs

@@ -322,7 +322,17 @@ export class PluginHost {
       });
     }
     this.#cancelled.delete(envelope.call_id);
-    await this.#sendTerminal(envelope, 'success', payload);
+    try {
+      await this.#sendTerminal(envelope, 'success', payload);
+    } catch (error) {
+      if (error instanceof FramingError || !(error instanceof ProtocolError)) throw error;
+      // The answer itself could not be written. The call still ends, as the
+      // failure it is, rather than leaving rebon to wait out its bound.
+      return this.#sendTerminal(envelope, 'error', {
+        code: '[INVALID_ANSWER]',
+        message: `the handler answered a value that is not JSON: ${error.message}`.slice(0, 256),
+      });
+    }
     if (plan.finalize) await plan.finalize();
   }
   #plan(value) {

package/bin/plugin-host/src/methods.mjs

@@ -156,8 +156,11 @@ export function pluginCommandDefinition(input) {
       shape(kind, ['type', 'dialog'], [], 'command kind');
       validateName('dialog', string(kind.dialog, 'kind dialog'));
       break;
+    case 'output':
+      shape(kind, ['type'], [], 'command kind');
+      break;
     default:
-      throw new ProtocolError('[WRONG_SHAPE]', 'command kind must be prompt, explain or panel');
+      throw new ProtocolError('[WRONG_SHAPE]', 'command kind must be prompt, explain, panel or output');
   }
   return Object.freeze({ ...input });
 }

package/bin/plugin-host/src/protocol.mjs

@@ -120,10 +120,44 @@ export function identityOf(envelope) {
   return Object.freeze({ host_epoch: envelope.host_epoch, plugin_id: envelope.plugin_id, scope_id: envelope.scope_id,
     scope_generation: envelope.scope_generation, call_id: envelope.call_id });
 }
+/// A value a plugin produced, as `JSON.stringify` would write it.
+///
+/// What a handler returns or a plugin sends is ordinary JavaScript, and
+/// ordinary JavaScript leaves `undefined` in objects (`{ text, context }` with
+/// no context), carries a `Date`, or builds its result with a class. The wire
+/// takes plain JSON only, and refusing such a value used to mean the call was
+/// never answered at all. This is the normalisation every JS author already
+/// expects: `toJSON` is honoured; an `undefined`, function or symbol property
+/// is left out, and in an array becomes `null`; a class instance is written as
+/// its own enumerable fields. A non-finite number becomes `null`. What still
+/// cannot be written — a cycle, a `BigInt` — is refused with a reason.
+export function toWireJson(value, seen = new Set(), inArray = false) {
+  if (value instanceof RawJsonNumber) return value;
+  if (value !== null && typeof value === 'object' && typeof value.toJSON === 'function') value = value.toJSON();
+  if (value === undefined || typeof value === 'function' || typeof value === 'symbol') return inArray ? null : undefined;
+  if (value === null || typeof value === 'string' || typeof value === 'boolean') return value;
+  if (typeof value === 'number') return Number.isFinite(value) ? value : null;
+  if (typeof value === 'bigint') throw new ProtocolError('invalid_payload', 'a BigInt cannot be written as JSON; convert it to a number or a string');
+  if (seen.has(value)) throw new ProtocolError('invalid_payload', 'payload must not be cyclic');
+  seen.add(value);
+  let copy;
+  if (Array.isArray(value)) copy = value.map((item) => toWireJson(item, seen, true));
+  else {
+    copy = {};
+    for (const key of Object.keys(value)) {
+      const item = toWireJson(value[key], seen, false);
+      if (item !== undefined) copy[key] = item;
+    }
+  }
+  seen.delete(value);
+  return copy;
+}
+const wire = (payload) => (payload === undefined ? null : toWireJson(payload));
+
 export function envelope(identity, message) { return validateEnvelope({ protocol_version: PROTOCOL_VERSION, ...identity, message }); }
-export function terminal(identity, status, payload) { return envelope(identity, { type: 'terminal', status, payload }); }
-export function request(identity, method, payload) { return envelope(identity, { type: 'request', method, payload }); }
-export function notification(identity, method, payload) { return envelope(identity, { type: 'notification', method, payload }); }
-export function chunk(identity, payload) { return envelope(identity, { type: 'chunk', payload }); }
+export function terminal(identity, status, payload) { return envelope(identity, { type: 'terminal', status, payload: wire(payload) }); }
+export function request(identity, method, payload) { return envelope(identity, { type: 'request', method, payload: wire(payload) }); }
+export function notification(identity, method, payload) { return envelope(identity, { type: 'notification', method, payload: wire(payload) }); }
+export function chunk(identity, payload) { return envelope(identity, { type: 'chunk', payload: wire(payload) }); }
 export function cancel(target) { return notification(target, CALL_CANCEL_METHOD, null); }
 export function isPlatformControl(value) { return value.plugin_id === PLATFORM_PLUGIN_ID && value.scope_id === PLATFORM_CONTROL_SCOPE_ID && value.scope_generation === 0; }

package/bin/rebon

@@ -1,4 +1,4 @@
 [diffend] Oversized file quarantined before diffing.
 name: package/bin/rebon
-size: 53673136 bytes
-sha256: b95a4adfc55159cc5a2955194ee5d53c6e6647b9ffda10598d9d86926155c9ab
+size: 55819024 bytes
+sha256: 65dacb976347fb2495540a0d1fb76e44aec3322026076db94529445f4e88516b

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rebon/cli-darwin-arm64",
-  "version": "1.8.6",
+  "version": "1.9.1",
   "description": "Agent cli for coding and more.",
   "license": "Apache-2.0",
   "repository": {