ispbills-icli 8.4.0 → 8.4.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ispbills-icli",
-  "version": "8.4.0",
+  "version": "8.4.2",
   "description": "iCli — IspBills AI network engineer in your terminal",
   "keywords": [
     "ispbills",

package/src/client.js

@@ -78,7 +78,9 @@ export function parsePartialPlan(raw) {
  */
 export async function streamChat(cfg, messages, options = {}) {
   const { onEvent = () => {}, signal, context = {} } = options;
-  const body = JSON.stringify({ messages, context });
+  // Always identify ourselves as the icli client so the backend can route
+  // away from type=connect (which requires an interactive browser terminal).
+  const body = JSON.stringify({ messages, context: { ...context, client: 'icli' } });
 
   const doFetch = (token) =>
     fetch(`${cfg.url}/api/v1/ai/chat/stream`, {
@@ -153,6 +155,10 @@ export async function streamChat(cfg, messages, options = {}) {
 
   const reply = finalResult?.reply ?? {};
   const meta = finalResult?._meta ?? {};
-  const text = reply.text || answerText || '';
+  // Prefer whichever is longer: the final parsed reply.text or the text we
+  // streamed live. Guards against any server-side cap silently shortening the
+  // committed answer below what the operator already saw stream by.
+  const finalText = reply.text || '';
+  const text = finalText.length >= answerText.length ? finalText : answerText;
   return { reply, text, meta };
 }

package/src/tui/app.js

@@ -96,9 +96,11 @@ export function App({ cfg, display = {}, onExternal, onExit, initialQuestion })
   }, []);
 
   const runTurn = useCallback(async (question, opts = {}) => {
-    const { depth = 0, archive = null, label = '' } = opts;
-    push({ type: 'user', text: question });
-    log(`\n${glyphs.prompt} ${question}\n`);
+    const { depth = 0, archive = null, label = '', synthetic = false } = opts;
+    if (!synthetic) {
+      push({ type: 'user', text: question });
+      log(`\n${glyphs.prompt} ${question}\n`);
+    }
     convo.current.push({ role: 'user', content: question });
     setBusy(true);
     setHint('');
@@ -136,7 +138,19 @@ export function App({ cfg, display = {}, onExternal, onExit, initialQuestion })
       // so it stays in history instead of vanishing with the live region.
       if (s.status.length) push({ type: 'activity', status: s.status });
       if (type === 'plan') { push({ type: 'plan', reply }); log('[plan] ' + (reply.summary || '')); }
-      else if (type === 'connect') { push({ type: 'connect', reply }); log('[connect] ' + (reply.device?.kind || '')); }
+      else if (type === 'connect') {
+        push({ type: 'connect', reply }); log('[connect] ' + (reply.device?.kind || ''));
+        // The icli has no interactive terminal — send a synthetic session-event
+        // so the backend continues to the plan/answer (rule 4). Depth-guard to
+        // prevent a loop if the backend keeps emitting connect (shouldn't happen).
+        if (depth < 2) {
+          const dev = reply.device || {};
+          const goal = convo.current.filter((m) => m.role === 'user').slice(-1)[0]?.content || '';
+          const sessionEvent = `[session-event] Terminal session opened on ${dev.kind || 'device'} id=${dev.id || '?'} proto=${dev.proto || 'ssh'}. vendor=auto. OPERATOR GOAL: ${goal}`;
+          setLive(null); setBusy(false); abortRef.current = null;
+          return runTurn(sessionEvent, { depth: depth + 1, synthetic: true });
+        }
+      }
       else if (type === 'prompt' || type === 'choice') { push({ type: 'assistant', text: text || reply.question || '' }); log(text || reply.question || ''); }
       else { push({ type: 'assistant', text: text || '(no response)' }); log(text || ''); }
       if (text) convo.current.push({ role: 'assistant', content: text });