pi-jev-guard 0.1.3 → 0.1.5

package/extensions/index.ts

@@ -307,35 +307,63 @@ export default function (pi: ExtensionAPI) {
     getLastGate: () => lastGate,
   });
 
-  // Seed headless: --print risolve --model prima di session_start, quindi per
-  // il solo caso deepseek/deepseek-flash si pre-registra un overlay statico
-  // (sostituito dal clone live appena il registry è disponibile).
-  // Richiede JEV_AUTO_UPSTREAM: senza, nessun guarded (opt-in).
-  const bootAuto = process.env.JEV_AUTO_UPSTREAM?.trim().toLowerCase();
-  if (bootAuto === "deepseek/deepseek-flash") {
+  // Seed all'avvio: pi ripristina il modello di sessione PRIMA di
+  // session_start, quindi un twin persistito non esisterebbe ancora e
+  // cadrebbe su "Could not restore model ...__jev". Pre-registriamo un
+  // overlay statico dai provider builtin quando la config persiste
+  // automatic (o JEV_AUTO_UPSTREAM è impostato); session_start lo
+  // sostituisce col clone live dal registry.
+  const bootAuto = process.env.JEV_AUTO_UPSTREAM?.trim();
+  const autoOff = bootAuto === "0" || bootAuto?.toLowerCase() === "off";
+  let seedTarget: { provider: string; model: string } | undefined;
+  if (bootAuto && !autoOff) {
+    const parsed = parseUpstreamRef([bootAuto]);
+    if (!("error" in parsed)) seedTarget = parsed;
+  } else if (!bootAuto && config.mode === "automatic") {
+    seedTarget = {
+      provider: config.automatic.upstreamProvider,
+      model: config.automatic.upstreamModel,
+    };
+  }
+  if (seedTarget) {
     try {
       // providers/all è mappato dagli alias di pi; il subpath del singolo
       // provider non lo è e fallirebbe a load (solo installazioni npm).
-      const base = builtinProviders().find((p) => p.id === "deepseek");
-      if (!base) throw new Error("deepseek provider not in builtins");
+      const base = builtinProviders().find((p) => p.id === seedTarget?.provider);
+      if (!base) throw new Error(`provider ${seedTarget.provider} not in builtins`);
+      // Il catalogo builtin può non avere il modello salvato (es. versioni
+      // più vecchie senza deepseek-flash): uniamo la definizione statica.
+      const extra =
+        seedTarget.provider === "deepseek" && seedTarget.model === "deepseek-flash"
+          ? [STATIC_DEEPSEEK_FLASH]
+          : [];
       const seedUpstream: Provider = {
         ...base,
-        getModels: () => [...base.getModels(), { ...STATIC_DEEPSEEK_FLASH }],
+        getModels: () => {
+          const models = base.getModels();
+          const missing = extra.filter(
+            (m) => !models.some((existing) => existing.id === m.id),
+          );
+          return [...models, ...missing];
+        },
       };
+      const hasModel = seedUpstream
+        .getModels()
+        .some((m) => m.id === seedTarget?.model);
+      if (!hasModel) throw new Error(`model ${seedTarget.model} not in builtins`);
       const overlay = buildOverlaidProvider({
         upstream: seedUpstream,
-        twinModelIds: ["deepseek-flash"],
+        twinModelIds: [seedTarget.model],
         getConfig,
         getReview: getGateReview,
         onVerdict: recordGateVerdict,
       });
       pi.registerProvider(overlay);
-      upstreamRefs.set("deepseek", seedUpstream);
-      overlaidProviderId = "deepseek";
-      twinTarget = { provider: "deepseek", model: "deepseek-flash" };
+      upstreamRefs.set(seedTarget.provider, seedUpstream);
+      overlaidProviderId = seedTarget.provider;
+      twinTarget = { provider: seedTarget.provider, model: seedTarget.model };
     } catch {
-      overlaidProviderId = undefined;
-      twinTarget = undefined;
+      // Nessun seed: il twin arriverà da session_start se possibile.
     }
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-jev-guard",
-  "version": "0.1.3",
+  "version": "0.1.5",
   "type": "module",
   "description": "Jev validation guard for pi: on-demand tool + automatic provider gate. Dual backend: TypeSafe direct + OpenRouter.",
   "keywords": [

package/src/commands.ts

@@ -1,5 +1,6 @@
 import { readFileSync, existsSync, statSync, realpathSync, writeFileSync, mkdirSync, chmodSync } from "node:fs";
 import { resolve, dirname } from "node:path";
+import { tmpdir } from "node:os";
 import type { ExtensionAPI, ModelRegistry } from "@earendil-works/pi-coding-agent";
 import type { JevConfig } from "./config.ts";
 import { defaultConfigPath, expandHome, parseUpstreamRef, resolveBackend, saveConfig } from "./config.ts";
@@ -345,9 +346,18 @@ export function registerJevCommand(pi: ExtensionAPI, deps: JevCommandDeps) {
 
       if (sub === "save-key") {
         const cfg = deps.config();
-        const target = cfg.jev.apiKeyFile?.trim()
+        let target = cfg.jev.apiKeyFile?.trim()
           ? expandHome(cfg.jev.apiKeyFile.trim())
           : resolve(dirname(defaultConfigPath()), "jev-api-key.txt");
+        // Mai chiavi in tmp: dir effimera (pulizia automatica = chiave persa).
+        // Può capitare con config stantie caricate in memoria prima di un fix.
+        if (target === tmpdir() || target.startsWith(tmpdir() + "/")) {
+          ctx.ui.notify(
+            `Jev: percorso chiave sotto tmp ignorato (${target}); uso quello canonico.`,
+            "warning",
+          );
+          target = resolve(dirname(defaultConfigPath()), "jev-api-key.txt");
+        }
         let key: string | undefined;
         try {
           key = await ctx.ui.input(