@surething/cockpit 1.0.189 → 1.0.191

package/.next-prod/trace-build

@@ -1 +1 @@
-[{"name":"run-webpack","duration":14324238,"timestamp":499618723,"id":17,"parentId":1,"tags":{},"startTime":1776939666272,"traceId":"08253691ab17f82f"},{"name":"run-typescript","duration":14964445,"timestamp":513946659,"id":1516,"parentId":1,"tags":{},"startTime":1776939680600,"traceId":"08253691ab17f82f"},{"name":"static-check","duration":812694,"timestamp":528925508,"id":1519,"parentId":1,"tags":{},"startTime":1776939695579,"traceId":"08253691ab17f82f"},{"name":"static-generation","duration":5983032,"timestamp":530042221,"id":1743,"parentId":1,"tags":{},"startTime":1776939696695,"traceId":"08253691ab17f82f"},{"name":"collect-build-traces","duration":18081020,"timestamp":529738896,"id":1740,"parentId":1,"tags":{},"startTime":1776939696392,"traceId":"08253691ab17f82f"},{"name":"telemetry-flush","duration":52,"timestamp":547823054,"id":1752,"parentId":1,"tags":{},"startTime":1776939714476,"traceId":"08253691ab17f82f"},{"name":"next-build","duration":48350299,"timestamp":499472818,"id":1,"tags":{"buildMode":"default","version":"16.1.6","bundler":"webpack","has-custom-webpack-config":"false","use-build-worker":"true"},"startTime":1776939666126,"traceId":"08253691ab17f82f"}]
+[{"name":"run-webpack","duration":12469889,"timestamp":258042084,"id":17,"parentId":1,"tags":{},"startTime":1776946064942,"traceId":"7edc58480cce636d"},{"name":"run-typescript","duration":14950615,"timestamp":270517130,"id":1516,"parentId":1,"tags":{},"startTime":1776946077417,"traceId":"7edc58480cce636d"},{"name":"static-check","duration":851998,"timestamp":285483513,"id":1519,"parentId":1,"tags":{},"startTime":1776946092384,"traceId":"7edc58480cce636d"},{"name":"static-generation","duration":6156212,"timestamp":286689558,"id":1743,"parentId":1,"tags":{},"startTime":1776946093590,"traceId":"7edc58480cce636d"},{"name":"collect-build-traces","duration":18913013,"timestamp":286338915,"id":1740,"parentId":1,"tags":{},"startTime":1776946093239,"traceId":"7edc58480cce636d"},{"name":"telemetry-flush","duration":58,"timestamp":305255163,"id":1752,"parentId":1,"tags":{},"startTime":1776946112155,"traceId":"7edc58480cce636d"},{"name":"next-build","duration":47383414,"timestamp":257871819,"id":1,"tags":{"buildMode":"default","version":"16.1.6","bundler":"webpack","has-custom-webpack-config":"false","use-build-worker":"true"},"startTime":1776946064772,"traceId":"7edc58480cce636d"}]

package/bin/cock-browser.mjs

@@ -353,6 +353,32 @@ async function run() {
 
 async function formatOutput(action, data) {
   if (data === undefined || data === null) {
+    // evaluate-family silently returning undefined/null is a major source of
+    // confusion for LLM callers — they can't tell "code ran but produced no
+    // value" from "something broke". Write a concrete hint to stderr so the
+    // Bash tool's empty-stdout annotation surfaces it alongside the
+    // "(exit 0 — empty stdout)" note instead of guessing causes.
+    if (action === 'evaluate' || action === 'evaluate_chunk') {
+      process.stderr.write(
+        `(evaluate returned ${data === null ? 'null' : 'undefined'} — nothing to print.\n` +
+        ` Common causes:\n` +
+        `  (a) bare arrow function: \`() => x\` defines but does not invoke — wrap as \`(() => x)()\` or \`(async()=>{...})()\`.\n` +
+        `  (b) a .then(...) callback didn't return the value — add an explicit \`return\` in each step of the chain.\n` +
+        `  (c) accessed a missing property — e.g. \`d.chat.x\` when response has no \`chat\` field, yielding undefined.\n` +
+        ` To force output: wrap the final value with \`JSON.stringify(...)\`, or return a short scalar.\n`
+      );
+    }
+    return;
+  }
+  if (typeof data === 'function') {
+    // Structured clone usually drops functions into undefined before this
+    // point, but if one slips through (e.g. from a custom bridge), surface
+    // it explicitly instead of swallowing.
+    process.stderr.write(
+      `(evaluate returned a function reference, which cannot be serialized.\n` +
+      ` If you defined an arrow function, invoke it: \`(async()=>{...})()\`.\n` +
+      ` To return the function's *result*, call it; to inspect the source, use \`fn.toString()\`.\n`
+    );
     return;
   }
 

package/bin/cock.mjs

@@ -59,18 +59,36 @@ if (!process.env.COCKPIT_PORT) {
 }
 
 // Subcommand routing
+
+// Flush stdout/stderr before exit. process.exit() does NOT wait for
+// async pipe writes to drain — for large outputs (> 16 KiB Node stream
+// highWaterMark) this truncates at exactly 16384 bytes on macOS pipes.
+// Without this, an ollama agent capturing our stdout via execAsync
+// receives a cleanly cut mid-string blob and then misdiagnoses it as
+// "output truncated" (reproduced in sessions 6910d071 & 22727dd4).
+async function flushAndExit(code) {
+  const drain = (stream) => new Promise((resolve) => {
+    if (!stream.writableLength) { resolve(); return; }
+    // Writing an empty string returns false iff the stream is backpressured;
+    // the callback then fires once the kernel has actually accepted the data.
+    stream.write('', 'utf8', () => resolve());
+  });
+  try { await Promise.all([drain(process.stdout), drain(process.stderr)]); } catch { /* ignore */ }
+  process.exit(code);
+}
+
 if (process.argv[2] === 'browser') {
   process.argv.splice(2, 1);
   const mod = await import('./cock-browser.mjs');
   await mod.done;
-  process.exit(0);
+  await flushAndExit(0);
 }
 
 if (process.argv[2] === 'terminal') {
   process.argv.splice(2, 1);
   const mod = await import('./cock-terminal.mjs');
   await mod.done;
-  process.exit(0);
+  await flushAndExit(0);
 }
 
 if (process.argv[2] === 'update') {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@surething/cockpit",
-  "version": "1.0.189",
+  "version": "1.0.191",
   "description": "The Cockpit That Drives AI",
   "author": "Robert",
   "license": "MIT",