freddie 0.0.129 → 0.0.131

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "freddie",
-  "version": "0.0.129",
+  "version": "0.0.131",
   "type": "module",
   "description": "Open JS agent harness built on pi-mono, floosie, xstate, and anentrypoint-design",
   "bin": {
@@ -25,7 +25,7 @@
     "@libsql/client": "^0.5.0",
     "@mariozechner/pi-ai": "^0.70.6",
     "@mariozechner/pi-tui": "^0.70.6",
-    "acptoapi": "^1.0.144",
+    "acptoapi": "^1.0.146",
     "anentrypoint-design": "latest",
     "commander": "^14.0.0",
     "dotenv": "^16.6.1",

package/plugins/bash/handler.js

@@ -26,7 +26,21 @@ export const _tool = ({
             const t = setTimeout(() => { try { child.kill('SIGKILL') } catch {} resolve({ exitCode: -1, stdout, stderr: stderr + '\n[timeout]', timedOut: true }) }, timeout_ms)
             child.stdout?.on('data', d => stdout += d.toString())
             child.stderr?.on('data', d => stderr += d.toString())
-            child.on('close', code => { clearTimeout(t); resolve({ exitCode: code, stdout, stderr }) })
+            child.on('close', code => {
+                clearTimeout(t)
+                const result = { exitCode: code, stdout, stderr }
+                // Windows cmd.exe can silently swallow ALL output (exit 0, empty
+                // stdout+stderr) for certain nested-quote one-liners (witnessed:
+                // node -e with double-quoted body containing single-quoted JS
+                // strings plus an embedded literal \n) -- indistinguishable from
+                // "ran fine and printed nothing" without this signal, and models
+                // have been observed misdiagnosing it as "bash tool unavailable"
+                // and silently substituting an unverified reimplementation.
+                if (process.platform === 'win32' && code === 0 && !stdout && !stderr && command.length > 40) {
+                    result.note = 'exitCode 0 with no output at all on a non-trivial command is unusual on Windows -- cmd.exe can silently swallow output for commands with nested quotes (e.g. node -e one-liners mixing double/single quotes and \\n). If you expected output, try writing the script to a file and running it instead of an inline -e/-c one-liner.'
+                }
+                resolve(result)
+            })
             child.on('error', e => { clearTimeout(t); resolve({ exitCode: -1, stdout, stderr: stderr + '\n' + e.message }) })
         })
     },

package/src/agent/llm_resolver.js

@@ -74,6 +74,29 @@ function adapt(result) {
 // Mirror lib/named-chains.js BUILTIN — acptoapi resolves unknown names.
 const NAMED_CHAIN_NAMES = new Set(['fast', 'cheap', 'smart', 'reasoning', 'free', 'local', 'auto'])
 
+// A hand-typed agent.model_preference used to be joined in its literal config
+// order, permanently overriding acptoapi's own swe_bench-score-aware ranking
+// (buildAutoChain sorts by score + tool-capability tiering) with no
+// reconciliation at all — witnessed picking a 62-score model ahead of a
+// 79.6-score one purely because it was configured first. Re-rank the user's
+// chosen providers using acptoapi's OWN scored pool (buildAutoChain is a
+// sanctioned top-level export; this never reaches into unexported internals
+// like lib/swe-bench-scores) so provider CHOICE stays user-controlled while
+// ORDER reflects real capability. Unscored links keep their relative config
+// order at the tail, matching acptoapi's own null-score handling.
+function orderByScore(links) {
+    let pool
+    try { pool = typeof sdk.buildAutoChain === 'function' ? sdk.buildAutoChain(undefined, { hasTools: true }) : [] } catch { pool = [] }
+    if (!Array.isArray(pool) || !pool.length) return links
+    const rank = new Map(pool.map((l, i) => [l.model, i]))
+    return [...links].sort((a, b) => {
+        const ra = rank.has(a) ? rank.get(a) : Infinity
+        const rb = rank.has(b) ? rank.get(b) : Infinity
+        if (ra !== rb) return ra - rb
+        return links.indexOf(a) - links.indexOf(b)
+    })
+}
+
 async function buildModel({ provider, model, inputModel }) {
     if (provider) return `${provider}/${model || DEFAULTS[provider] || ''}`.replace(/\/$/, '')
     if (model) return model
@@ -88,7 +111,7 @@ async function buildModel({ provider, model, inputModel }) {
     const pref = getConfigValue('agent.model_preference', [])
     if (Array.isArray(pref) && pref.length) {
         const links = pref.map(p => `${p.provider}/${p.model || DEFAULTS[p.provider] || ''}`.replace(/\/$/, '')).filter(s => s.includes('/'))
-        if (links.length) return links.join(', ')
+        if (links.length) return orderByScore(links).join(', ')
     }
     const auto = typeof sdk.buildAutoChain === 'function' ? sdk.buildAutoChain(undefined) : []
     const keyed = Array.isArray(auto) ? auto.filter(l => { const p = l.model.split('/')[0]; const env = PROVIDER_KEYS[p]; return env && process.env[env] }) : []