diff options
| author | Agando <roduccomm@gmail.com> | 2026-10-23 20:59:29 +0200 |
|---|---|---|
| committer | Agando <roduccomm@gmail.com> | 2026-10-23 20:59:29 +0200 |
| commit | 4b69d3666a8d842d07df2689200a364b8fdd662e (patch) | |
| tree | 22b1f5b54675fd2d5109179c39d41482235eb879 /modules/core/core-bundle.js | |
| parent | 96dbc9924a9f33c40e88a648370759f688d6283d (diff) | |
fix(newos4): outboxResult scoped, markdown mime, old reply path retired (v4.6.81)
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.
Diffstat (limited to 'modules/core/core-bundle.js')
| -rw-r--r-- | modules/core/core-bundle.js | 9 |
1 files changed, 7 insertions, 2 deletions
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 ─── |