From bc99dfba1898123ea824de98a2d5a06e996ed667 Mon Sep 17 00:00:00 2001
From: Agando <roduccomm@gmail.com>
Date: Fri, 23 Oct 2026 20:59:29 +0200
Subject: fix(newos4): outboxResult scoped, markdown mime, old reply path
 retired (v4.6.81)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

delegateTask declared `const outboxResult` inside the AI branch and returned it outside, so a
delegation with the AI module missing threw 'outboxResult is not defined'. The catch turned that
into _handleAgentFailure: the worker stayed idle with no result while spawn_agent reported ok —
the reported symptom exactly. Now function-scoped, and the no-module path returns
'queued — AI module not loaded, nothing executed' rather than claiming 'completed'.

Also: nginx served _docs/*.md as application/octet-stream (no .md type) — now text/markdown +
nosniff; agent-replies.php returns 410 for GET as well as POST, so there is one reply path.

Verified: with the AI module deleted at runtime, delegateTask returns the queued string and
throws nothing.
---
 modules/core/core-bundle.js | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

(limited to 'modules/core/core-bundle.js')

diff --git a/modules/core/core-bundle.js b/modules/core/core-bundle.js
index 73f5ab3..b565365 100644
--- a/modules/core/core-bundle.js
+++ b/modules/core/core-bundle.js
@@ -16721,6 +16721,10 @@ class OrchestratorModule {
             // Single sub-task — fall through to normal path
         }
 
+        // Declared in function scope: the AI branch assigns it, but the return at the end of
+        // this method runs even when that branch never did.
+        let outboxResult = null;
+
         // Find an idle agent and immediately stamp it busy (atomic claim)
         let agent = Object.values(this._agents).find(a => a.status === 'idle');
         if (agent) {
@@ -16809,7 +16813,7 @@ class OrchestratorModule {
                 this.terminal.print(`✅ ${agent.id} completed task`, 'success');
 
                 // Return result for callers that await delegateTask
-                const outboxResult = typeof result === 'string' ? result : (result?.content || 'completed');
+                outboxResult = typeof result === 'string' ? result : (result?.content || 'completed');
 
                 // Check restart policy — if always, set back to idle for next task
                 if (agent.restartPolicy === 'always') {
@@ -16820,6 +16824,7 @@ class OrchestratorModule {
                 }
             } else {
                 this.terminal.print(`ℹ️  AI module not available — task queued for ${agent.id}`, 'info');
+                outboxResult = 'queued — AI module not loaded, nothing executed';
             }
         } catch (e) {
             this._handleAgentFailure(agent.id, e.message);
@@ -16835,7 +16840,7 @@ class OrchestratorModule {
             agent.status = 'idle';
             agent.lastHeartbeat = Date.now();
         }
-        return outboxResult || 'completed';
+        return outboxResult || 'no result recorded';
     }
 
     // ─── Recurring Scheduling ───
-- 
cgit v1.2.3