@mmmbuto/nexuscrew 0.9.30 → 0.9.31

package/frontend/dist/index.html

@@ -11,7 +11,7 @@
   <meta name="apple-mobile-web-app-title" content="NexusCrew" />
   <link rel="manifest" href="/manifest.json" />
   <title>NexusCrew</title>
-  <script type="module" crossorigin src="/assets/index-BbcjoZS0.js"></script>
+  <script type="module" crossorigin src="/assets/index-DaAL7P-m.js"></script>
   <link rel="stylesheet" crossorigin href="/assets/index-DqCfZFGi.css">
 </head>
 <body>

package/frontend/dist/version.json

@@ -1 +1 @@
-{"version":"0.9.30"}
+{"version":"0.9.31"}

package/lib/cli/doctor.js

@@ -435,8 +435,37 @@ function checkTmuxServerTermuxPreload(runtimeEnv, execImpl, opts = {}) {
   let raw = '';
   try {
     raw = String(execImpl(opts.tmuxBin || 'tmux', ['show-environment', '-g', 'LD_PRELOAD'], { encoding: 'utf8' }) || '').trim();
-  } catch (_) {
-    return { name: 'tmux server termux-exec', ok: true, warn: true, detail: 'server tmux non attivo; probe rinviato al primo avvio' };
+  } catch (err) {
+    // Con il server ATTIVO `tmux show-environment -g <var>` esce 1 scrivendo
+    // "unknown variable" su stderr: execFileSync lo rigetta come errore, come
+    // fa per un server assente ("no server running"), ma i due esiti non sono
+    // la stessa cosa. L'assenza della variabile merita un WARN col rimedio,
+    // non l'esito «server non attivo».
+    const msg = String((err && (err.stderr || err.message)) || '');
+    if (/unknown variable/i.test(msg)) {
+      return {
+        name: 'tmux server termux-exec', ok: true, warn: true,
+        detail: 'server tmux attivo ma LD_PRELOAD non impostato (tmux: unknown variable); al primo avvio verra\' impostata, oppure: tmux set-environment -g LD_PRELOAD <libtermux-exec-ld-preload.so>',
+      };
+    }
+    // Classificazione dell'errore tmux: solo il «no server running» autentico
+    // dice che il server non c'è. Un «error connecting ... (Permission denied)»
+    // o qualunque altro fallimento lasciano la domanda aperta — il socket
+    // esiste ma questo utente non può leggerlo — e diventano «probe non
+    // verificabile» con lo stderr nel dettaglio, non un falso «server assente».
+    if (/permission denied/i.test(msg)) {
+      return {
+        name: 'tmux server termux-exec', ok: true, warn: true,
+        detail: `probe non verificabile: il socket tmux risponde "${msg}"; il server potrebbe essere attivo ma non leggibile con questo utente`,
+      };
+    }
+    if (/no server running|error connecting/i.test(msg)) {
+      return { name: 'tmux server termux-exec', ok: true, warn: true, detail: 'server tmux non attivo; probe rinviato al primo avvio' };
+    }
+    return {
+      name: 'tmux server termux-exec', ok: true, warn: true,
+      detail: `probe non verificabile: tmux ha risposto "${msg}"; stato del server non determinabile da qui`,
+    };
   }
   const match = raw.match(/^LD_PRELOAD=(.+)$/);
   const trusted = match && trustedTermuxPreload({ ...env, LD_PRELOAD: match[1] }, { platform, home: opts.home });

package/lib/decks/routes.js

@@ -29,7 +29,17 @@ function decksRoutes({ cfg = {}, decksPath }) {
   };
   const find = (st, name) => st.decks.find((d) => d.name === name) || null;
   const checkRevision = (deck, rev) => {
-    if (deck.revision !== rev) throw httpError(409, 'deck modificato da un’altra finestra', { current: deck });
+    if (deck.revision !== rev) {
+      // Traccia il conflitto di revisione con i campi che servono a
+      // capire chi lo ha causato (nessun merge lato server in questo ciclo).
+      console.log(JSON.stringify({
+        event: 'deck-revision-conflict',
+        name: deck.name,
+        expectedRevision: rev,
+        currentRevision: deck.revision,
+      }));
+      throw httpError(409, 'deck modificato da un’altra finestra', { current: deck });
+    }
   };
 
   r.get('/', (_req, res) => {

package/lib/fleet/builtin.js

@@ -37,7 +37,7 @@ const {
 } = require('./definitions.js');
 const {
   publicCatalog, describeManaged, describeCatalogCredential, defaultShellEngine, defaultAgyEngine, defaultKimiEngine, defaultGrokEngine, defaultVlEngine,
-  CATALOG, credential: resolveProfileCredential, extraModelsFrom, knownMcpServerNames,
+  CATALOG, credential: resolveProfileCredential, extraModelsFrom, knownMcpServerNames, profileFor,
 } = require('./managed.js');
 const { probeModel } = require('./model-probe.js');
 const { validEnvKey } = require('./env-key.js');
@@ -874,9 +874,18 @@ async function createBuiltinFleet(cfg = {}) {
     }
     // Se un engine sta USANDO questo modello, toglierlo lo renderebbe invalido
     // al prossimo avvio — e il messaggio arriverebbe a chi non sa cosa e'
-    // cambiato. Meglio rifiutare adesso, dicendo chi lo usa.
+    // cambiato. Meglio rifiutare adesso, dicendo chi lo usa. Un engine risolve
+    // il modello dalla dichiarazione fatta sul SUO id o sul suo profilo
+    // (declaredFor cerca in [engineId, profileId]): il ritiro di
+    // {wantedId, wantedEngine} lo rompe solo se wantedEngine e' una di quelle
+    // chiavi — un altro engine con la propria dichiarazione non c'entra.
     const usato = (defs.engines || [])
-      .filter((e) => e.managed && e.managed.model === wantedId)
+      .filter((e) => {
+        if (!e.managed || e.managed.model !== wantedId) return false;
+        if (e.id === wantedEngine) return true;
+        const profile = profileFor(e.managed.client, e.managed.provider, e.managed.credentialProfile || '');
+        return !!(profile && profile.id === wantedEngine);
+      })
       .map((e) => e.id);
     if (usato.length) {
       throw httpError(409, `modello in uso: ${usato.join(', ')}`);
@@ -1264,7 +1273,7 @@ async function createBuiltinFleet(cfg = {}) {
       engines: defs.engines.map((e) => {
         const out = { ...e, envKeys: Object.keys(e.env || {}).sort() };
         delete out.env;
-        if (e.managed) out.managedInfo = describeManaged(e.managed, { ...cfg, home, extraModels: extraModelsFrom(defs) });
+        if (e.managed) out.managedInfo = describeManaged(e.managed, { ...cfg, home, extraModels: extraModelsFrom(defs), engineId: e.id });
         return out;
       }),
       cells: defs.cells.map((c) => {

package/lib/fleet/definitions.js

@@ -170,7 +170,7 @@ function cellIdFromTmuxSession(session) {
 //
 // Un modello dichiarato NON aggira il controllo: lo estende in modo esplicito.
 // Un id mai dichiarato resta rifiutato.
-const MAX_MODELS = 64;
+const MAX_MODELS = 128;
 const MODEL_ID_MAX = 128;
 
 function parseModel(m) {
@@ -245,7 +245,15 @@ function parseDefinitions(raw, { allowLegacyTmuxNames = true, onReject = null, o
     const models = [];
     const extraModels = new Map();
     if (d.models !== undefined) {
-      if (!Array.isArray(d.models) || d.models.length > MAX_MODELS) return null;
+      if (!Array.isArray(d.models)) return null;
+      if (d.models.length > MAX_MODELS) {
+        // Come per gli engine: il rifiuto NOMINA il limite, altrimenti chi
+        // dichiara si ritrova un 400 generico senza sapere che il cap era
+        // quello. `onIssue` porta campo/valore/regola al messaggio del comando
+        // (400 di una scrittura), `reject` alla lettura del file.
+        if (typeof onIssue === 'function') onIssue({ field: 'models', value: d.models.length, rule: `troppi modelli dichiarati, cap ${MAX_MODELS}` });
+        return reject(`fleet.json rifiutato: ${d.models.length} modelli dichiarati, cap ${MAX_MODELS}; riduci i modelli a ${MAX_MODELS} o meno`);
+      }
       const seen = new Set();
       for (const m of d.models) {
         const model = parseModel(m);
@@ -1081,7 +1089,7 @@ function atomicWrite(p, data) {
 }
 
 const CAPS = {
-  SCHEMA_VERSION, MAX_ENGINES, MAX_CELLS, MAX_ARGS, MAX_ARG_LEN,
+  SCHEMA_VERSION, MAX_ENGINES, MAX_MODELS, MAX_CELLS, MAX_ARGS, MAX_ARG_LEN,
   MAX_ENV_KEYS, MAX_ENV_KEY_LEN, MAX_ENV_VAL_LEN, MAX_LABEL_LEN,
   MAX_COMMAND_LEN, MAX_CWD_LEN, MAX_MODEL_FLAG_LEN, MAX_MODEL_VAL_LEN,
   MAX_PROMPTFLAG_LEN, MAX_PROMPT_LEN, MAX_CELL_COMMAND_LEN, MAX_TMUXSESSION_LEN,

package/lib/fleet/managed.js

@@ -1267,7 +1267,17 @@ function describeManaged(spec, cfg = {}) {
     // (source 'unreadable', authConfigured false) non viene collassata in
     // 'missing' — il discriminante e' CHI ha fallito, non il esito grezzo.
     credentialSource: cred.source,
-    configured, models: [...(profile.models || [])], defaultModel: profile.model || '',
+    configured,
+    // Catalogo builtin del profilo PIU' i modelli dichiarati in
+    // `definitions.models[]` per questo engine (o, legacy, per il profilo).
+    // Senza l'unione un profilo custom — che catalogo non ha — elencava `[]`
+    // anche con due modelli dichiarati: il boot li accettava (declaredFor) ma
+    // la finestra, che legge da qui, non li mostrava.
+    models: [...new Set([
+      ...(profile.models || []),
+      ...declaredModelsFor(extraModels, profile.id, cfg.engineId || null).map((m) => m.id),
+    ])],
+    defaultModel: profile.model || '',
     binary: binary || '', displayName: normalized.displayName || profile.label,
     reason,
   };

package/lib/fleet/runtime.js

@@ -175,7 +175,7 @@ function createBuiltinRuntime(ctx) {
       needsPi ? discoverPiModels({ ...cfg, home }) : {},
     ]);
     const engines = cache.defs.engines.map((e) => {
-      const managed = e.managed ? describeManaged(e.managed, { ...cfg, home, extraModels: extraModelsFrom(cache.defs) }) : null;
+      const managed = e.managed ? describeManaged(e.managed, { ...cfg, home, extraModels: extraModelsFrom(cache.defs), engineId: e.id }) : null;
       return {
         id: e.id, label: e.label, rc: !!e.rc,
         ...(managed ? {
@@ -184,7 +184,7 @@ function createBuiltinRuntime(ctx) {
           models: managed.provider === 'ollama-cloud' ? ollamaModels
             : (managed.client === 'pi'
               ? (e.managed.provider === 'custom'
-                ? [e.managed.model]
+                ? [...new Set([e.managed.model, ...managed.models].filter(Boolean))]
                 : (e.managed.provider === 'native'
                   ? [...new Set(Object.values(piModels).flat())]
                   : (piModels[e.managed.provider] || [])))

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mmmbuto/nexuscrew",
-  "version": "0.9.30",
+  "version": "0.9.31",
   "description": "Faithful browser tmux client — attach to live sessions over a real PTY, localhost-only, mobile-easy",
   "main": "lib/server.js",
   "bin": {