@mmmbuto/nexuscrew 0.9.73 → 0.9.74

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-EAHAmDmX.js"></script>
+  <script type="module" crossorigin src="/assets/index-DRXfJssx.js"></script>
   <link rel="stylesheet" crossorigin href="/assets/index-C_z6r985.css">
 </head>
 <body>

package/frontend/dist/version.json

@@ -1 +1 @@
-{"version":"0.9.73"}
+{"version":"0.9.74"}

package/lib/fleet/builtin.js

@@ -38,7 +38,8 @@ const {
 } = require('./definitions.js');
 const {
   publicCatalog, describeManaged, describeCatalogCredential, defaultShellEngine, defaultAgyEngine, defaultKimiEngine, defaultGrokEngine, defaultVlEngine,
-  CATALOG, credential: resolveProfileCredential, extraModelsFrom, knownMcpServerNames, profileFor,
+  CATALOG, CLIENT_LABELS, CLIENT_ORDER, engineClient, ordinaEngine, isStandardClient,
+  credential: resolveProfileCredential, extraModelsFrom, knownMcpServerNames, profileFor,
   normalizeManagedSpec, codexSurfaceNames,
 } = require('./managed.js');
 const { probeModel, probeCustomEndpoint } = require('./model-probe.js');
@@ -696,10 +697,14 @@ async function createBuiltinFleet(cfg = {}) {
       }
       boot = persistito;
     }
-    boot = backfillShellEngine(defsPath, boot, cfg.log);
+    // L'ordine delle aggiunte segue CLIENT_ORDER (agy, grok, shell...) e poi i
+    // client fuori lista in ordine stabile (kimi, vl): un fleet.json backfillato
+    // nasce gia' nell'ordine che il payload mostra. Ogni backfill resta
+    // idempotente e indipendente: il riordino non cambia cosa viene aggiunto.
     boot = backfillAgyEngine(defsPath, boot, cfg);
-    boot = backfillKimiEngine(defsPath, boot, cfg.log);
     boot = backfillGrokEngine(defsPath, boot, cfg);
+    boot = backfillShellEngine(defsPath, boot, cfg.log);
+    boot = backfillKimiEngine(defsPath, boot, cfg.log);
     boot = backfillVlEngine(defsPath, boot, cfg.log);
     // Ripara (non aggiunge) un desktop.local gia' presente col comando relativo.
     // Il logger va PASSATO: senza, il messaggio di fallimento si ferma a un
@@ -1204,7 +1209,10 @@ async function createBuiltinFleet(cfg = {}) {
     // rifiutato se la dichiarazione non c'e' ancora — fail-closed corretto, ma
     // che al ripristino si legge come «il backup era rotto».
     const models = Array.isArray(opts.models) ? opts.models : [];
-    if (models.length > 64) throw httpError(400, 'troppi modelli nel backup');
+    // Il tetto dei modelli accettati e' lo stesso della validazione del
+    // fleet.json (CAPS.MAX_MODELS): un hardcode separato qui era gia'
+    // divergente (64 contro 128) e rifiutava backup validi.
+    if (models.length > CAPS.MAX_MODELS) throw httpError(400, `troppi modelli nel backup (max ${CAPS.MAX_MODELS})`);
     for (const m of models) {
       if (!m || typeof m !== 'object' || Array.isArray(m)) throw httpError(400, 'definizione modello non valida');
       if (typeof m.id !== 'string' || !m.id.trim() || typeof m.engine !== 'string' || !m.engine.trim()) {
@@ -1378,9 +1386,20 @@ async function createBuiltinFleet(cfg = {}) {
     const realHome = resolveCwd(home, home);
     return {
       schemaVersion: defs.schemaVersion,
-      engines: defs.engines.map((e) => {
+      // UNICO punto di ordinamento del payload: API, UI e tool MCP leggono
+      // tutti da qui. ordinaEngine riordina in lettura: il fleet.json resta
+      // com'e', mai riscritto.
+      engines: ordinaEngine(defs.engines).map((e) => {
         const out = { ...e, envKeys: Object.keys(e.env || {}).sort() };
         delete out.env;
+        // `standard`: calcolato dal client (CLIENT_ORDER), mai persistito nel
+        // fleet.json. La UI ci divide i selettori in due gruppi senza
+        // duplicare la lista dei client standard.
+        out.standard = isStandardClient(engineClient(e));
+        // Le etichette escono in minuscolo: vale per le label generate dai
+        // client e anche per quelle scritte a mano nel fleet.json, che nel
+        // file restano esattamente come le ha scritte l'operatore.
+        if (typeof out.label === 'string' && out.label) out.label = out.label.toLowerCase();
         if (e.managed) out.managedInfo = describeManaged(e.managed, {
           ...cfg, home, extraModels: extraModelsFrom(defs), engineId: e.id,
           endpointProbe: cfg.endpointProbe || defaultEndpointProbe,
@@ -1541,7 +1560,9 @@ async function createBuiltinFleet(cfg = {}) {
         rc: { type: 'boolean', required: false, default: false },
         managed: {
           type: 'object', requiredFor: 'managed',
-          client: { type: 'enum', values: ['claude', 'codex', 'codex-vl', 'grok', 'vl', 'pi', 'agy', 'kimi', 'shell'] },
+          // Enum allineato a CLIENT_ORDER (standard prima, nell'ordine voluto)
+          // e poi i client noti fuori lista, in ordine stabile.
+          client: { type: 'enum', values: [...CLIENT_ORDER, ...Object.keys(CLIENT_LABELS).filter((c) => !CLIENT_ORDER.includes(c))] },
           provider: { type: 'catalog', source: 'managedCatalog' },
           credentialProfile: { type: 'string', required: false, max: 32 },
           model: { type: 'string', required: false, max: CAPS.MAX_MODEL_VAL_LEN },

package/lib/fleet/managed.js

@@ -446,7 +446,40 @@ const CREDENTIAL_SOURCE_VALUES = Object.freeze([
   'providers-shell', 'keys-file', 'secure-file', 'legacy',
   'compatibility', 'missing', 'unreadable',
 ]);
-const CLIENT_LABELS = Object.freeze({ claude: 'Claude Code', codex: 'Codex', 'codex-vl': 'Codex-VL', grok: 'Grok', vl: 'VL', pi: 'Pi', agy: 'Agy', kimi: 'Kimi Code CLI', shell: 'Shell' });
+// Etichette dei client in minuscolo (decisione operatore 2026-10-17): il nome
+// mostrato e' il client stesso. La normalizzazione vale in uscita (payload,
+// UI, tool MCP); il testo scritto a mano nel fleet.json resta nel file cosi'
+// com'e' e viene mostrato in minuscolo dal punto di uscita del payload.
+const CLIENT_LABELS = Object.freeze({ claude: 'claude', codex: 'codex', 'codex-vl': 'codex-vl', grok: 'grok', vl: 'vl', pi: 'pi', opencode: 'opencode', agy: 'agy', kimi: 'kimi', shell: 'shell' });
+// Client "standard" nell'ordine voluto dall'operatore (2026-10-17). E' LA
+// lista: il campo calcolato `standard` del payload delle definizioni, il rango
+// di ordinaEngine(), l'ordine del seed e l'enum client dello schema derivano
+// tutti da qui. I client fuori lista (vl, kimi, custom...) sono supportati:
+// chiudono l'elenco in ordine stabile, mai buttati via.
+const CLIENT_ORDER = Object.freeze(['claude', 'codex-vl', 'codex', 'pi', 'opencode', 'agy', 'grok', 'shell']);
+// Il client di un engine: managed.client quando c'e' (engine gestiti), primo
+// segmento dell'id altrimenti (claude.hosted -> claude): cosi' i derivati
+// restano attaccati al loro client anche quando nascono come id.
+function engineClient(engine) {
+  if (engine && engine.managed && typeof engine.managed.client === 'string' && engine.managed.client) return engine.managed.client;
+  if (engine && typeof engine.id === 'string' && engine.id) return engine.id.split('.')[0];
+  return '';
+}
+// Ordine di SOLA LETTURA: nessun fleet.json viene riscritto. Rango del client
+// secondo CLIENT_ORDER, a parita' di rango resta l'ordine di inserimento
+// (Array#sort e' stabile da ES2019); i client fuori lista chiudono l'array.
+function ordinaEngine(engines) {
+  if (!Array.isArray(engines)) return engines;
+  const rank = new Map(CLIENT_ORDER.map((client, index) => [client, index]));
+  return [...engines].sort((a, b) => {
+    const ra = rank.get(engineClient(a));
+    const rb = rank.get(engineClient(b));
+    return (ra === undefined ? CLIENT_ORDER.length : ra) - (rb === undefined ? CLIENT_ORDER.length : rb);
+  });
+}
+function isStandardClient(client) {
+  return typeof client === 'string' && CLIENT_ORDER.includes(client);
+}
 const PROVIDER_ID_RE = /^[a-z][a-z0-9_-]{0,31}$/;
 
 function validBaseUrl(value) {
@@ -588,6 +621,23 @@ const CATALOG = Object.freeze([
   { id: 'pi.ollama', client: 'pi', provider: 'ollama', label: 'Ollama local', auth: 'none', protocol: 'openai-completions', piProvider: 'ollama', requiresModel: true, core: true, piExtension: { baseUrl: 'http://127.0.0.1:11434/v1', apiKey: 'ollama' } },
   { id: 'pi.custom', client: 'pi', provider: 'custom', label: 'Custom provider', auth: 'dynamic', protocol: 'openai-responses', protocols: ['openai-responses', 'anthropic-messages', 'openai-completions', 'google-generative-ai'], custom: true, core: true },
 
+  // OpenCode (client nativo, 0.9.75): TUI headless-friendly con gateway proprio
+  // («provider» integrato `opencode`) e login locale (`opencode auth login`).
+  // MISURATO su opencode 1.18.33 (zero credenziali salvate):
+  // i modelli *-free del gateway rispondono ANONYMOUS, quindi la cella parte
+  // anche prima del login; il set completo arriva con l'account. Per questo
+  // auth e' 'login' (delegato, mai chiavi gestite da NexusCrew) e NON c'e'
+  // l'array `models`: l'elenco dei modelli del gateway ruota lato opencode e
+  // un elenco statico qui marcirebbe in silenzio; il modello resta libero
+  // (niente strictModels) e la forma e' `<model>` (prefisso gateway aggiunto
+  // dal launcher) oppure `provider/model` esplicito per chi sa cosa chiede.
+  // I provider A CHIAVE del catalogo (zai, ollama-cloud, openrouter...) NON
+  // entrano in questo client oggi: opencode.jsonc accetta `apiKey` SOLO come
+  // stringa letterale (il riferimento {env:...} e' rifiutato, misurato), e
+  // scrivere una chiave su disco viola la regola del repo (mai segreti nei
+  // file generati). Residuo dichiarato nel referto del lotto.
+  { id: 'opencode.native', client: 'opencode', provider: 'native', label: 'OpenCode account (gateway + local login)', auth: 'login', protocol: 'opencode_native', core: true, notice: 'opencode-native' },
+
   // Agy: client gestito primario (Linux/macOS non-Termux). Auth delegata al
   // login locale del client (nessuno store letto/copiato). Nessun remote-control
   // NexusCrew; permission default 'standard' (unsafe aggiunge --dangerously-skip-
@@ -744,10 +794,13 @@ function normalizeManagedSpec(value, { extraModels = null, engineId = null, onIs
 function defaultDefinitions() {
   return {
     schemaVersion: 1,
-    engines: CATALOG.filter((p) => p.default).map((p) => ({
+    // Il seed segue CLIENT_ORDER: una flotta nuova nasce gia' nell'ordine che
+    // il payload mostra (ordinaEngine), senza dover contare sull'ordine con
+    // cui le voci default sono scritte nel CATALOG.
+    engines: ordinaEngine(CATALOG.filter((p) => p.default).map((p) => ({
       id: p.id, label: CLIENT_LABELS[p.client], rc: !!p.rc,
       managed: { client: p.client, provider: p.provider, model: p.model || '', permissionPolicy: p.client === 'claude' ? 'unsafe' : 'standard' },
-    })),
+    }))),
     cells: [],
   };
 }
@@ -2422,6 +2475,9 @@ function resolveManagedEngine(engine, cell, cfg = {}) {
     // autonomous, nessuna domanda) NON e' mappato: il contratto NexusCrew
     // distingue solo standard/unsafe e il default resta interattivo.
     if (spec.client === 'kimi') args.push('--yolo');
+    // OpenCode: unsafe -> --auto (flag reale di `opencode --help` 1.18.33:
+    // «auto-approve permissions that are not explicitly denied (dangerous!)»).
+    if (spec.client === 'opencode') args.push('--auto');
   }
   let shellOneShot = false;
   if (spec.client === 'shell') {
@@ -2772,6 +2828,26 @@ function resolveManagedEngine(engine, cell, cfg = {}) {
     // dal push generico qui sotto come ultimo argomento. Senza prompt parte il
     // TUI interattivo nella cwd della cella.
     if (model) args.push('--model', model);
+  } else if (spec.client === 'opencode') {
+    // OpenCode TUI (0.9.75, misurato su 1.18.33): auth delegata al login/locale
+    // del client (gateway `opencode`, free tier anche senza credenziali) —
+    // niente env provider, niente credenziali su argv, NE' config scritta da
+    // NexusCrew: MCP e provider custom restano nell'opencode.jsonc dell'utente
+    // (o nel file per-cella che l'operatore passa con OPENCODE_CONFIG, che
+    // qui non si compone). `--standalone` NON esiste in 1.18.33: ogni
+    // invocazione gira col proprio server e muore con essa (misurato con ps
+    // dopo la chiusura), quindi il TUI parte nudo — quel flag sarebbe servito
+    // per un server di background condiviso per utente che questa versione non
+    // avvia. Il modello viaggia come `-m provider/model`: un id senza `/`
+    // e' del gateway integrato (`opencode/<model>`), uno gia' prefissato
+    // passa tale e quale (l'operatore che ha config sue sa cosa chiede).
+    // Il prompt di cella va su `--prompt` (precompila la casella di input,
+    // pattern di agy --prompt-interactive): il posizionale e' il PATH di
+    // progetto, mai testo. Niente send-keys: il classifier di readiness
+    // (prompt-delivery) copre kimi/claude/vl e un marker nuovo va misurato
+    // sul TUI vivo prima di essere fidato — residuo dichiarato.
+    if (model) args.push('-m', model.includes('/') ? model : `opencode/${model}`);
+    if (cell?.prompt) args.push('--prompt');
   } else if (spec.client === 'vl') {
     // VL/Vivling runtime TUI: auth is handled by the runtime in every variant
     // (config.toml / VL_API_KEY by name via envPassthrough), never
@@ -3014,7 +3090,7 @@ module.exports = {
   ALIBABA_CODEX_MODELS, ALIBABA_TOKEN_PLAN_CONTEXT, ALIBABA_PI_MODELS,
   OPENCODE_GO_MESSAGES_MODELS, OPENCODE_GO_RESPONSES_MODELS, OPENCODE_GO_CHAT_MODELS, OPENCODE_GO_LIMITS,
   OPENCODE_GO_ANTHROPIC_ROOT, OPENCODE_GO_API_BASE,
-  CLIENT_LABELS, normalizeManagedSpec, profileFor,
+  CLIENT_LABELS, CLIENT_ORDER, engineClient, ordinaEngine, isStandardClient, normalizeManagedSpec, profileFor,
   defaultDefinitions, defaultShellEngine, defaultAgyEngine, defaultKimiEngine, defaultGrokEngine, defaultVlEngine, describeManaged, describeCatalogCredential, discoverOllamaModels, resolveManagedEngine, needsExplicitNode,
   discoverPiModels, EXTERNAL_DISCOVERY_TIMEOUT_MS, parseEnvFile, parseProviderShellFile, findBinary, publicCatalog, writePiProviderExtension, customCatalogFor,
   codexSurfaceNames, writeCellCodexProfile,

package/package.json

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