fidelizare-integrate 0.6.0 → 0.6.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fidelizare-integrate",
-  "version": "0.6.0",
+  "version": "0.6.2",
   "description": "Asistent de integrare Fidelizare pentru softuri de gestiune si case de marcat. Scaneaza codul, gaseste locul potrivit si propune integrarea API-ului, in siguranta.",
   "type": "module",
   "bin": {

package/src/cli.js

@@ -109,8 +109,14 @@ async function main() {
 
   // No fallback. Clear error + support ticket.
   rule();
-  out('  ' + paint('✗ Integrarea nu a putut rula.', C.redBright) + '  ' + paint('(' + r.reason + ')', C.dim) + '\n');
-  if (r.reason === 'no_key') note('Lipseste cheia. Lipeste cheia ta API Fidelizare cand ti-o cere CLI-ul.', C.amber);
+  if (r.reason === 'no_integration') {
+    out('  ' + paint('✗ Nu am gasit un loc sigur de integrare.', C.redBright) + '\n');
+    if (r.summary && r.summary !== 'oprit') note(r.summary.split('\n')[0].slice(0, 120), C.dim);
+    note('Nimic nu a fost scris. Cheia NU a fost salvata.', C.amber);
+  } else {
+    out('  ' + paint('✗ Integrarea nu a putut rula.', C.redBright) + '  ' + paint('(' + r.reason + ')', C.dim) + '\n');
+    if (r.reason === 'no_key') note('Lipseste cheia. Lipeste cheia ta API Fidelizare cand ti-o cere CLI-ul.', C.amber);
+  }
   supportNote();
   rl.close();
 }

package/src/core/agent-loop.js

@@ -35,12 +35,28 @@ Pasi:
 3. Creeaza un helper (create_file) care apeleaza API-ul conform documentatiei, cu
    cheia din variabila de mediu FIDELIZARE_API_KEY (NICIODATA in cod), in try/catch
    ce nu blocheaza vanzarea.
-4. Insereaza apelul (insert_code) exact pe acea ramura, plus import-ul necesar.
+4. OBLIGATORIU editeaza FISIERUL EXISTENT care gestioneaza vanzarea: cu insert_code
+   adaugi (a) import-ul/require helper-ului sus in fisier si (b) apelul helper-ului
+   EXACT pe ramura unde se detecteaza cardul de fidelitate. Integrarea nu e completa
+   daca ai creat doar fisiere noi; codul care ruleaza la scanare TREBUIE sa cheme
+   helper-ul. Reciteste fisierul dupa fiecare insert_code (numerele de linie se
+   schimba) inainte de urmatoarea inserare.
 5. Scrie un test cu mock (create_file) care verifica: apelul se face cu fidelity_card
    = codul scanat; o eroare de retea NU arunca si NU blocheaza fluxul. Ruleaza-l cu
    run_test pana trece.
 6. Verifica sintaxa (check_syntax) si apeleaza finish cu un rezumat scurt.
 
+Vorbeste cu utilizatorul cand ai nevoie (tool ask_user): cand nu gasesti fisierul
+sau ramura de integrare, cand exista mai multe locuri posibile, sau cand ai o
+nelamurire (o decizie da/nu, ce fisier, ce optiune). Inainte de prima editare a unui
+fisier existent, confirma scurt cu ask_user: "Integrez in <fisier> la <ramura>, e
+corect?". NU ghici cai si NU te opri in tacere: daca nu stii, intreaba.
+
+Nu apela finish inainte de a fi inserat apelul helper-ului in fisierul existent de
+vanzare cu insert_code. Daca ai folosit doar create_file, NU ai terminat. Daca,
+dupa ce ai intrebat, chiar nu exista un loc de integrare, apeleaza finish si explica
+clar ca nu s-a putut integra (nu inventa modificari).
+
 Reguli: modificari minime, fara refactor; pastreaza stilul si limbajul proiectului;
 nu scrie chei in cod; nu inventa cai de fisier.`;
 
@@ -53,6 +69,7 @@ const TOOLS = [
   { type: 'function', function: { name: 'web_fetch', description: 'Descarca un document de pe web (https), ex. documentatia API.', parameters: { type: 'object', properties: { url: { type: 'string' } }, required: ['url'] } } },
   { type: 'function', function: { name: 'run_test', description: 'Ruleaza o comanda de test/compilare in proiect (gated, doar runnere cunoscute).', parameters: { type: 'object', properties: { command: { type: 'string' } }, required: ['command'] } } },
   { type: 'function', function: { name: 'check_syntax', description: 'Verifica sintaxa unui fisier (doar Node).', parameters: { type: 'object', properties: { path: { type: 'string' } }, required: ['path'] } } },
+  { type: 'function', function: { name: 'ask_user', description: 'Intreaba utilizatorul cand nu esti sigur: ce fisier/ramura de integrare, o decizie da/nu, sau o alegere dintr-o lista de optiuni. Foloseste in loc sa ghicesti sau sa te opresti.', parameters: { type: 'object', properties: { question: { type: 'string' }, options: { type: 'array', items: { type: 'string' } } }, required: ['question'] } } },
   { type: 'function', function: { name: 'finish', description: 'Termina integrarea.', parameters: { type: 'object', properties: { summary: { type: 'string' } }, required: ['summary'] } } },
 ];
 
@@ -139,6 +156,23 @@ export async function runAgentLoop({ stack, rootDir, files, askRaw, auto, model,
       try { execFileSync('node', ['--check', a], { stdio: 'pipe' }); return 'OK sintaxa valida'; }
       catch (e) { return 'EROARE sintaxa: ' + (e.stderr?.toString() || e.message).slice(0, 300); }
     },
+    ask_user: async ({ question, options }) => {
+      const q = String(question || '').trim();
+      const opts = Array.isArray(options) ? options.filter(Boolean).map(String) : [];
+      out('\n  ' + paint('? ', C.amber) + paint(q, C.white) + '\n');
+      opts.forEach((o, i) => out('     ' + paint(i + 1 + ')', C.dim) + ' ' + paint(o, C.ink) + '\n'));
+      if (auto) {
+        const def = opts.length ? opts[0] : 'da';
+        out('     ' + paint('› ' + def + ' (auto)', C.green) + '\n');
+        return def;
+      }
+      const a = (await askRaw(opts.length ? '  Alege (numar sau text):' : '  Raspuns (da / nu / text):')).trim();
+      if (opts.length && /^\d+$/.test(a)) {
+        const n = parseInt(a, 10);
+        if (n >= 1 && n <= opts.length) return opts[n - 1];
+      }
+      return a || (opts.length ? opts[0] : 'nu');
+    },
     finish: ({ summary }) => { finished = summary || 'gata'; return 'gata'; },
   };
 
@@ -147,8 +181,9 @@ export async function runAgentLoop({ stack, rootDir, files, askRaw, auto, model,
     { role: 'user', content: `Stack: ${stack}. Documentatie API: ${guide}. Proiect cu ${files.length} fisiere sursa. Incepe cu web_fetch pe documentatie, apoi orienteaza-te in cod, integreaza si scrie + ruleaza un test cu mock.` },
   ];
 
-  const NAMES = { list_files: 'List', read_file: 'Read', grep: 'Grep', create_file: 'Write', insert_code: 'Edit', web_fetch: 'Fetch', run_test: 'Test', check_syntax: 'Check', finish: 'Done' };
+  const NAMES = { list_files: 'List', read_file: 'Read', grep: 'Grep', create_file: 'Write', insert_code: 'Edit', web_fetch: 'Fetch', run_test: 'Test', check_syntax: 'Check', ask_user: 'Ask', finish: 'Done' };
   const brief = (name, a) => {
+    if (name === 'ask_user') return '';
     if (name === 'grep') return a.pattern ? '"' + a.pattern + '"' : '';
     if (name === 'web_fetch') return a.url || '';
     if (name === 'run_test') return a.command ? '`' + a.command + '`' : '';
@@ -191,5 +226,12 @@ export async function runAgentLoop({ stack, rootDir, files, askRaw, auto, model,
     }
   }
 
-  return { ok: true, changes: [...new Set(changes)], summary: finished || 'oprit', model: usedModel };
+  // Honest outcome: a finish (or a stall) with zero applied edits is NOT a
+  // success. Report it as no_integration so the CLI shows an error + support
+  // link instead of a green "done" + writing the key to .env.
+  const applied = [...new Set(changes)];
+  if (applied.length === 0) {
+    return { ok: false, reason: 'no_integration', summary: finished || 'oprit', model: usedModel };
+  }
+  return { ok: true, changes: applied, summary: finished || 'gata', model: usedModel };
 }