oft-agt 0.5.16 → 0.5.18

package/cli.mjs

@@ -51,6 +51,7 @@ async function credentials(config) {
 }
 async function stopBot() {
   const current = child;
+  child = null;
   if (!current || current.exitCode !== null || current.signalCode) return;
   await new Promise(resolve => { const timer = setTimeout(() => current.kill('SIGKILL'), 5000); current.once('exit', () => { clearTimeout(timer); resolve(); }); current.kill('SIGTERM'); });
 }
@@ -89,6 +90,7 @@ async function deploy(config, message, notify) {
   await run(process.execPath, ['bot.mjs', '--deploy'], { cwd: target, env, stdio: 'ignore' });
   const previous = child;
   const previousId = active;
+  child = null;
   if (previous && previous.exitCode === null && !previous.signalCode) {
     await new Promise(resolve => { const timeout = setTimeout(() => { previous.kill('SIGKILL'); }, 5000); previous.once('exit', () => { clearTimeout(timeout); resolve(); }); previous.kill('SIGTERM'); });
   }
@@ -109,7 +111,8 @@ async function deploy(config, message, notify) {
     }
     throw error;
   }
-  child.once('exit', () => { if (active === message.id) notify('failed'); });
+  const deployedChild=child;
+  deployedChild.once('exit', () => { if (child === deployedChild && active === message.id) notify('runtime-failed'); });
   active = message.id;
   await save(path.join(home, 'active.json'), { id: message.id });
   notify('running');
@@ -203,7 +206,7 @@ async function start(config) {
           if (message.type === 'deploy') {
             console.log(`[${new Date().toISOString()}] Nouveau déploiement reçu.`);
             if (await exists(path.join(home, 'paused.json'))) await unlink(path.join(home, 'paused.json'));
-            try { await deploy(config, message, async state => { send({ type: 'deployment-status', id: message.id, state, runtimeVersion:state==='running'?await runtimeVersion():null }); console.log(({ received: 'Déploiement validé.', installing: 'Préparation du bot.', running: 'Déploiement terminé. Bot démarré.', failed: 'Échec du déploiement.' })[state] || 'État du déploiement modifié.'); }); }
+            try { await deploy(config, message, async state => { if(state==='runtime-failed'){send({type:'runtime-status',state:'failed'});return;} send({ type: 'deployment-status', id: message.id, state, runtimeVersion:state==='running'?await runtimeVersion():null }); console.log(({ received: 'Déploiement validé.', installing: 'Préparation du bot.', running: 'Déploiement terminé. Bot démarré.', failed: 'Échec du déploiement.' })[state] || 'État du déploiement modifié.'); }); }
             catch { send({ type: 'deployment-status', id: message.id, state: 'failed' }); console.error('Déploiement refusé ou échoué.'); }
           }
         }).catch(() => console.error('Message refusé.'));
@@ -233,7 +236,7 @@ function printHelp() {
   status                Afficher l’état local
   logs                  Lire les 100 dernières lignes du journal
   token [--stdin]       Régénérer l’authentification agent ; --stdin lit 64 caractères hexadécimaux
-  remove --yes          Révoquer la liaison et supprimer les données locales de cette instance
+  remove | delete --yes Supprimer une instance arrêtée de la liste et révoquer sa liaison
   list                  Afficher les instances installées
   help                  Afficher cette aide
 Options : --bot=IDENTIFIANT_COURT (voir list) pour choisir un bot ; start --foreground pour diagnostiquer.
@@ -242,7 +245,7 @@ Documentation : https://doc-builder.oftone.fr/agent`);
 }
 async function main() {
   const [requestedCommand, ...args] = process.argv.slice(2);
-  const command = requestedCommand || 'login';
+  const command = requestedCommand==='delete'?'remove':requestedCommand || 'login';
   const source = path.dirname(fileURLToPath(import.meta.url));
   if (['help', '--help', '-h'].includes(command)) { printHelp(); return; }
   if (command === 'list') { for (const instance of await listInstances(baseHome)) console.log(`${instance.agentId.slice(0, 8)} · ${(await localStatus(instance.home)).running ? 'actif' : 'arrêté'}`); return; }
@@ -327,7 +330,8 @@ async function main() {
   }
   if (command === 'remove') {
     if (!args.includes('--yes')) { console.log('Supprime l’agent sélectionné et ses données locales, y compris son token Discord. Relancez avec remove --yes pour confirmer.'); return; }
-    await stopBackground(home); const config = JSON.parse(await readFile(configPath, 'utf8'));
+    const state=await localStatus(home);if(state.running||state.starting||state.legacy)throw Error('REMOVE_RUNNING');
+    const config = JSON.parse(await readFile(configPath, 'utf8'));
     let revoked = false;
     try { const response = await fetch(`${config.relay}/v1/agents/${config.object}/remove`, { method: 'POST', headers: { Authorization: `Bearer ${config.credential}` }, signal: AbortSignal.timeout(15000) }); revoked = response.ok; } catch {}
     await removeLocalInstance(home, baseHome, config);
@@ -335,5 +339,5 @@ async function main() {
   }
   throw new Error('COMMAND_UNKNOWN');
 }
-main().catch(error => { const messages = { LOGIN_BUSY: 'Une connexion Discord est déjà en cours sur cette machine.', LOGIN_EXPIRED: 'La demande a expiré. Relancez npx oft-agt login.', LOGIN_START_FAILED: 'Impossible de démarrer la connexion. Réessayez dans quelques minutes.', LOGIN_FAILED: 'Connexion refusée. Relancez npx oft-agt login.', AUTH_ROTATION_BUSY: 'Une modification de l’authentification est déjà en cours pour cette instance. Réessayez après sa fin.', COMMAND_UNKNOWN: 'Commande inconnue. Consultez oftone-agent help.', LEGACY_FOREGROUND: 'Un ancien agent fonctionne au premier plan. Arrêtez-le avec Ctrl+C une seule fois, puis utilisez start.', DAEMON_START_FAILED: 'L’agent n’a pas démarré. Consultez logs.', DAEMON_START_TIMEOUT: 'Démarrage non confirmé. Consultez status et logs avant de réessayer.', DAEMON_STOP_TIMEOUT: 'L’arrêt n’est pas encore confirmé. Consultez status et logs.', AUTH_ROTATION_FAILED: 'Changement d’authentification non confirmé. Réessayez token ou start pour reprendre sans perdre la liaison.', AUTH_ROTATION_INVALID: 'Token d’authentification invalide : 64 caractères hexadécimaux attendus.', PAIRING_INVALID: 'Commande de liaison invalide. Copiez à nouveau la commande complète depuis le dashboard.', PAIRING_REJECTED: 'Commande de liaison expirée, déjà utilisée ou révoquée. Générez-en une nouvelle depuis le bot concerné.', INSTANCE_INVALID: 'Identifiant d’instance invalide. Copiez la commande depuis les paramètres du bot concerné.', INSTANCE_NOT_FOUND: 'Cette liaison n’est pas installée sur cette machine. Utilisez la commande locale de cette instance ou configurez une nouvelle liaison.', INSTANCE_REQUIRED: 'Précisez l’instance avec la commande affichée dans les paramètres du bot ou la commande locale affichée à l’installation.', INSTANCE_EXISTS: 'Cette instance existe déjà. Utilisez sa commande locale de démarrage.', AGENT_ACTIVE: 'Cet agent est déjà actif. Arrêtez uniquement le processus de cette instance avant de le relancer.' }; console.error(messages[error.code || error.message] || 'Échec de l’agent. Vérifiez la liaison et la configuration locale. Aucun secret affiché.'); process.exitCode = 1; });
+main().catch(error => { const messages = { REMOVE_RUNNING:'Arrêtez ce bot avec stop avant de le supprimer de la liste.', LOGIN_BUSY: 'Une connexion Discord est déjà en cours sur cette machine.', LOGIN_EXPIRED: 'La demande a expiré. Relancez npx oft-agt login.', LOGIN_START_FAILED: 'Impossible de démarrer la connexion. Réessayez dans quelques minutes.', LOGIN_FAILED: 'Connexion refusée. Relancez npx oft-agt login.', AUTH_ROTATION_BUSY: 'Une modification de l’authentification est déjà en cours pour cette instance. Réessayez après sa fin.', COMMAND_UNKNOWN: 'Commande inconnue. Consultez oftone-agent help.', LEGACY_FOREGROUND: 'Un ancien agent fonctionne au premier plan. Arrêtez-le avec Ctrl+C une seule fois, puis utilisez start.', DAEMON_START_FAILED: 'L’agent n’a pas démarré. Consultez logs.', DAEMON_START_TIMEOUT: 'Démarrage non confirmé. Consultez status et logs avant de réessayer.', DAEMON_STOP_TIMEOUT: 'L’arrêt n’est pas encore confirmé. Consultez status et logs.', AUTH_ROTATION_FAILED: 'Changement d’authentification non confirmé. Réessayez token ou start pour reprendre sans perdre la liaison.', AUTH_ROTATION_INVALID: 'Token d’authentification invalide : 64 caractères hexadécimaux attendus.', PAIRING_INVALID: 'Commande de liaison invalide. Copiez à nouveau la commande complète depuis le dashboard.', PAIRING_REJECTED: 'Commande de liaison expirée, déjà utilisée ou révoquée. Générez-en une nouvelle depuis le bot concerné.', INSTANCE_INVALID: 'Identifiant d’instance invalide. Copiez la commande depuis les paramètres du bot concerné.', INSTANCE_NOT_FOUND: 'Cette liaison n’est pas installée sur cette machine. Utilisez la commande locale de cette instance ou configurez une nouvelle liaison.', INSTANCE_REQUIRED: 'Précisez l’instance avec la commande affichée dans les paramètres du bot ou la commande locale affichée à l’installation.', INSTANCE_EXISTS: 'Cette instance existe déjà. Utilisez sa commande locale de démarrage.', AGENT_ACTIVE: 'Cet agent est déjà actif. Arrêtez uniquement le processus de cette instance avant de le relancer.' }; console.error(messages[error.code || error.message] || 'Échec de l’agent. Vérifiez la liaison et la configuration locale. Aucun secret affiché.'); process.exitCode = 1; });
 

package/compile-plan.mjs

@@ -46,9 +46,9 @@ export function compile(input) {
     if (!plain(command)) fail(`Commande ${commandIndex + 1} : configuration absente.`);
 
     validateEventGraph(command);
-    const name = text(command.name ?? entry.name, 'Nom de commande', 32, true);
+    const name = text(command.name ?? entry.name, 'Nom', command.trigger?.kind==='event'?100:32, true);
 
-    if (!/^[a-z0-9_-]{1,32}$/.test(name)) fail(`/${name} : utilisez des minuscules, chiffres, tirets ou underscores.`);
+    if (command.trigger?.kind!=='event' && !/^[a-z0-9_-]{1,32}$/.test(name)) fail(`/${name} : utilisez des minuscules, chiffres, tirets ou underscores.`);
 
     if (names.has(name)) fail(`Commande dupliquée : /${name}.`);
 
@@ -134,7 +134,7 @@ export function compile(input) {
 
       if (next.has(edge.source)) fail(`/${name}/${edge.source} : plusieurs sorties ne sont pas encore prises en charge. Reliez les actions en séquence.`);
 
-      if (incoming.has(edge.target)) fail(`/${name}/${edge.target} : plusieurs entrées ne sont pas prises en charge.`);
+
 
       next.set(edge.source, edge.target);
 
@@ -153,7 +153,9 @@ export function compile(input) {
     }
     const visited = new Set(options.map(option => option.id));
 
-    function compilePath(rootId, inheritedMessages = new Set(), inheritedWait = 0, depth = 0) {
+    let expanded = 0;
+    function compilePath(rootId, inheritedMessages = new Set(), inheritedWait = 0, depth = 0, ancestry = new Set()) {
+    const path = new Set(ancestry);
 
     if (depth > 32) fail(`/${name} : conditions imbriquées trop profondément (32 maximum).`);
 
@@ -167,7 +169,9 @@ export function compile(input) {
 
     while (current) {
 
-      if (visited.has(current)) fail(`/${name} : boucle détectée.`);
+      if (path.has(current)) fail(`/${name} : boucle détectée.`);
+      path.add(current);
+      if (++expanded > 10000) fail(`/${name} : parcours trop complexe (10 000 étapes maximum).`);
 
       visited.add(current);
 
@@ -185,7 +189,7 @@ export function compile(input) {
 
         const rules = conditionConfigs.get(current).rules;
 
-        calls.push({ id: current, fn: 'condition', args: { kind: block.type, rules }, branches: rules.map(rule => ({ id: branches.get(rule.id), ruleId: rule.id, calls: compilePath(branches.get(rule.id), emittedMessages, waitBudget, depth + 1) })), otherwise: { id: branches.get('else'), calls: compilePath(branches.get('else'), emittedMessages, waitBudget, depth + 1) } });
+        calls.push({ id: current, fn: 'condition', args: { kind: block.type, rules }, branches: rules.map(rule => ({ id: branches.get(rule.id), ruleId: rule.id, calls: compilePath(branches.get(rule.id), emittedMessages, waitBudget, depth + 1, path) })), otherwise: { id: branches.get('else'), calls: compilePath(branches.get('else'), emittedMessages, waitBudget, depth + 1, path) } });
 
       } else if (isVariableBlock(block.type)) {
         fn = block.type; args = validateVariableBlock(block.type, config, variableDefinitions);
@@ -246,8 +250,8 @@ export function compile(input) {
         if(children.length) args.buttons=children.map(child=>({id:child.id,config:validateButton(child.config)}));
         for(const child of children){
           const config=validateButton(child.config);
-          const childCalls=compilePath(child.id,emittedMessages,0,depth+1);
-          buttonBranches.push({id:child.id,parentId:current,config,calls:childCalls});
+          const childCalls=compilePath(child.id,emittedMessages,0,depth+1,path);
+          if (!buttonBranches.some(branch => branch.id === child.id)) buttonBranches.push({id:child.id,parentId:current,config,calls:childCalls});
         }
 
       } else if (block.type === 'apiRequest') { fn='apiRequest'; args=validateApiRequest(config); if([...blocks.values()].some(b=>b.id!==block.id&&b.type==='apiRequest'&&b.config?.name===args.name))fail('Chaque requête API doit avoir un nom unique.');

package/npm-shrinkwrap.json

@@ -1,12 +1,12 @@
 {
   "name": "oft-agt",
-  "version": "0.5.16",
+  "version": "0.5.18",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "oft-agt",
-      "version": "0.5.16",
+      "version": "0.5.18",
       "license": "UNLICENSED",
       "dependencies": {
         "discord.js": "^14.14.0",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "oft-agt",
-  "version": "0.5.16",
+  "version": "0.5.18",
   "private": false,
   "type": "module",
   "engines": {

package/runtime/version.mjs

@@ -1 +1 @@
-export const RUNTIME_VERSION = '0.5.16';
+export const RUNTIME_VERSION = '0.5.18';

package/usage.mjs

@@ -12,7 +12,7 @@ export async function createUsageCollector(home, now = () => Date.now()) {
   let writing = Promise.resolve();
   return {
     record(event) {
-      if (event?.type !== 'usage' || !/^[a-z0-9_-]{1,32}$/.test(event.command || '') || typeof event.success !== 'boolean' || !Number.isFinite(event.durationMs) || event.durationMs < 0 || event.durationMs > 900000) return;
+      if (event?.type !== 'usage' || (typeof event.command!=='string'||!event.command.trim()||event.command.length>100||/[\u0000-\u001f\u007f]/.test(event.command)) || typeof event.success !== 'boolean' || !Number.isFinite(event.durationMs) || event.durationMs < 0 || event.durationMs > 900000) return;
       const current = Math.floor(now() / HOUR) * HOUR;
       const hour = event.hour ?? current;
       if (!Number.isSafeInteger(hour) || hour % HOUR || hour > current || hour < current - 719 * HOUR) return;

package/version.mjs

@@ -1 +1 @@
-export const AGENT_VERSION = '0.5.16';
+export const AGENT_VERSION = '0.5.18';