agentic-api 2.0.646 → 2.0.684

package/dist/src/rules/git/repo.pr.js

@@ -40,20 +40,22 @@ exports.gitGetPRMetadata = gitGetPRMetadata;
 exports.gitGetAllPR = gitGetAllPR;
 exports.gitGetClosedPRs = gitGetClosedPRs;
 exports.gitLoadPR = gitLoadPR;
-exports.gitPRReplaceFile = gitPRReplaceFile;
 exports.gitPRUpdateComments = gitPRUpdateComments;
 exports.gitClosePR = gitClosePR;
 exports.gitClosePRRobust = gitClosePRRobust;
-exports.gitGetNextPRNumber = gitGetNextPRNumber;
 exports.gitNewValidationRequest = gitNewValidationRequest;
 exports.gitNewPR = gitNewPR;
 const errors_1 = require("../errors");
 const path_1 = require("path");
 const fs = __importStar(require("fs/promises"));
 const repo_tools_1 = require("./repo.tools");
+const repo_1 = require("./repo");
 /**
  * Synchronise une branche PR avec son mergeBase pour corriger les références orphelines
  *
+ * @deprecated Utiliser le workflow applicatif (RulesWorkflow.syncPullRequest) et
+ * les opérations robustes orientées Git Notes. API conservée pour compatibilité.
+ *
  * FIXME: Cette fonction a un problème de gestion des métadonnées manquantes.
  *
  * PROBLÈME IDENTIFIÉ:
@@ -191,7 +193,7 @@ async function gitSyncPR(git, branch, user) {
  * @param branch Nom de la branche PR
  * @param validationToken Token utilisé pour marquer la fermeture (optionnel, utilise la config par défaut)
  * @returns true si le PR est fermé (dernier commit contient le token de validation)
- * DEPRECATED: use gitIsPRClosedRobust instead
+ * @deprecated Utiliser gitIsPRClosedRobust.
  */
 async function gitIsPRClosed(git, branch) {
     return gitIsPRClosedRobust(git, branch);
@@ -199,6 +201,8 @@ async function gitIsPRClosed(git, branch) {
 /**
  * Vérifie si une branche PR est fermée en utilisant Git Notes (avec fallback)
  * Version robuste qui remplace gitIsPRClosed
+ * @deprecated Préférer gitLoadPR() pour récupérer l'état PR complet.
+ *
  * @param git Instance Git
  * @param branch Nom de la branche PR
  * @param validationToken Token utilisé pour marquer la fermeture (optionnel, utilise la config par défaut)
@@ -287,6 +291,8 @@ async function gitGetAllPR(git, options = {}) {
 }
 /**
  * Finds closed PRs by scanning the history of the draft branch for merge commits with PR metadata.
+ * @deprecated Utiliser gitGetAllPR(git, { closed: true }) pour unifier le listing PR.
+ *
  * @param git SimpleGit instance
  * @param gitConfig The git configuration
  * @returns A list of PRInfo objects for closed PRs.
@@ -342,49 +348,6 @@ async function gitLoadPR(git, branch) {
         throw new errors_1.GitOperationError(`Failed to retrieve PR: ${error}`, 'pr_load');
     }
 }
-/**
- * Remplace explicitement un nom de fichier dans les métadonnées d'une PR.
- * Utilisé pour refléter immédiatement les renames sans dépendre de git diff.
- */
-async function gitPRReplaceFile(git, branch, options = {}, config) {
-    const { remove, add } = options;
-    if (!remove && !add) {
-        return;
-    }
-    const gitConf = (0, repo_tools_1.gitLoad)(config);
-    const metadata = await gitGetPRMetadata(git, branch, config);
-    if (!metadata) {
-        return;
-    }
-    const currentFiles = Array.isArray(metadata.files) ? metadata.files : [];
-    const seen = new Set();
-    const nextFiles = [];
-    for (const file of currentFiles) {
-        if (!file) {
-            continue;
-        }
-        if (remove && file === remove) {
-            continue;
-        }
-        if (seen.has(file)) {
-            continue;
-        }
-        seen.add(file);
-        nextFiles.push(file);
-    }
-    if (add && !seen.has(add)) {
-        nextFiles.push(add);
-        seen.add(add);
-    }
-    const changed = currentFiles.length !== nextFiles.length ||
-        currentFiles.some((file, idx) => file !== nextFiles[idx]);
-    if (!changed) {
-        return;
-    }
-    metadata.files = nextFiles;
-    const headCommitHash = (await git.revparse([branch])).trim();
-    await (0, repo_tools_1.gitWriteNote)(git, headCommitHash, metadata, gitConf.gitNotes.namespace);
-}
 async function gitPRUpdateComments(git, branch, details, config) {
     const gitConf = (0, repo_tools_1.gitLoad)(config);
     const metadata = await gitGetPRMetadata(git, branch, config);
@@ -405,7 +368,8 @@ async function gitPRUpdateComments(git, branch, details, config) {
     return gitLoadPR(git, branch);
 }
 /**
- * DEPRECATED: use gitClosePRRobust instead
+ * @deprecated Utiliser gitClosePRRobust.
+ *
  * Ferme un PR en ajoutant un commit avec le token de validation
  * @param git Instance Git
  * @param branch Branche PR à fermer
@@ -483,6 +447,7 @@ async function gitClosePRRobust(git, branch, closedBy, message, config) {
         // Note: "ours" dans ce contexte signifie "garder la version de la branche qu'on merge" (la PR)
         // car nous sommes sur rule-editor et on merge la PR
         let mergeResult;
+        let mergeCommitHash;
         try {
             mergeResult = await git
                 .env({
@@ -498,6 +463,7 @@ async function gitClosePRRobust(git, branch, closedBy, message, config) {
                 '-m', `Merge branch '${branch}'`,
                 branch
             ]);
+            mergeCommitHash = (await git.revparse(['HEAD'])).trim();
         }
         catch (mergeError) {
             if (gitConf.verbose) {
@@ -537,15 +503,22 @@ async function gitClosePRRobust(git, branch, closedBy, message, config) {
                     }
                 }
                 // Amend le commit de merge avec les bons contenus
-                await git.commit('--amend', ['--no-edit'], {
+                const amendResult = await git.commit('--amend', ['--no-edit'], {
                     '--author': `${closedBy.name} <${closedBy.email}>`
                 });
+                mergeCommitHash = amendResult.commit || (await git.revparse(['HEAD'])).trim();
                 mergeResult = { summary: { changes: prFiles.length } };
             }
             catch (fallbackError) {
                 throw new errors_1.PRClosureError(`Failed to merge PR ${branch} with all strategies: ${mergeError}\nFallback error: ${fallbackError}`, branch, 'merge_failed');
             }
         }
+        // 6. Attacher aussi la note au commit de merge sur la branche cible.
+        // Cela garantit la traçabilité même si la branche PR est supprimée.
+        if (!mergeCommitHash) {
+            mergeCommitHash = (await git.revparse(['HEAD'])).trim();
+        }
+        await (0, repo_tools_1.gitWriteNote)(git, mergeCommitHash, finalMetadata, gitConf.gitNotes.namespace);
         const result = {
             metadata: finalMetadata,
             failed: false,
@@ -583,37 +556,6 @@ async function gitClosePRRobust(git, branch, closedBy, message, config) {
         (0, repo_tools_1.unlock)('checkout');
     }
 }
-/**
- * Trouve le prochain numéro de PR disponible (protégé contre la concurrence)
- * @param git Instance Git
- * @param validationPrefix Préfixe des branches de validation
- * @returns Prochain numéro de PR
- */
-async function gitGetNextPRNumber(git, validationPrefix) {
-    await (0, repo_tools_1.lock)('pr-number');
-    try {
-        const allBranches = await (0, repo_tools_1.gitGetAllBranches)(git);
-        const prBranches = allBranches.filter((branch) => branch.startsWith(validationPrefix));
-        if (prBranches.length === 0) {
-            return 1;
-        }
-        const numbers = prBranches
-            .map((branch) => {
-            const numberPart = branch.substring(validationPrefix.length);
-            // Gérer les cas où le nom de branche contient plus que le numéro après le préfixe
-            const actualNumber = parseInt(numberPart.split('-')[0], 10);
-            return actualNumber;
-        })
-            .filter((num) => !isNaN(num));
-        return numbers.length > 0 ? Math.max(...numbers) + 1 : 1;
-    }
-    catch (error) {
-        throw new errors_1.GitOperationError(`Failed to get next PR number: ${error}`, 'pr_number', { validationPrefix, originalError: error });
-    }
-    finally {
-        (0, repo_tools_1.unlock)('pr-number');
-    }
-}
 /**
  * Crée une nouvelle branche de validation (pour un PR vers rule-editor)
  * à partir d'une branche source (typiquement rule-editor).
@@ -639,6 +581,20 @@ async function gitNewValidationRequest(git, files, description, author, options
             throw new errors_1.PRCreationError('Author is required', undefined, files);
         }
         try {
+            // Vérification atomique côté API: empêcher plusieurs PR ouvertes sur les mêmes fichiers.
+            const openPRs = await gitGetAllPR(git, {
+                closed: false,
+                validationPrefix: validationBranchPrefix
+            });
+            const conflictingPRs = openPRs.filter((pr) => pr.files.some((prFile) => files.includes(prFile)));
+            if (conflictingPRs.length > 0) {
+                const conflictingFiles = files.filter((file) => conflictingPRs.some((pr) => pr.files.includes(file)));
+                if (gitConfig.verbose) {
+                    console.log('🌶️ DEBUG: gitNewValidationRequest -- conflicting PRs:', conflictingPRs.map((pr) => pr.branch));
+                }
+                const firstConflict = conflictingPRs[0].branch;
+                throw new errors_1.PRCreationError(`Le fichier est déjà dans la validation en cours #${firstConflict.replace(validationBranchPrefix, '')}`, firstConflict, conflictingFiles);
+            }
             // ✅ CLEANUP: Nettoyer tout état de merge corrompu avant de créer la branche
             // Cela évite l'erreur "you need to resolve your current index first"
             try {
@@ -654,7 +610,7 @@ async function gitNewValidationRequest(git, files, description, author, options
                     console.warn('Cleanup before branch creation failed (non-critical):', cleanupError);
                 }
             }
-            const nextID = await gitGetNextPRNumber(git, validationBranchPrefix);
+            const nextID = await (0, repo_1.gitAllocateNextPRNumber)(git, validationBranchPrefix, gitConfig);
             const newBranchName = `${validationBranchPrefix}${nextID}`;
             await git.checkoutBranch(newBranchName, sourceBranch);
             const initialCommitMessage = `#${nextID} ${description}`;
@@ -710,7 +666,7 @@ async function gitNewValidationRequest(git, files, description, author, options
     }
 }
 /**
- * DEPRECATED
+ * @deprecated Utiliser gitNewValidationRequest().
  */
 async function gitNewPR(git, files, description, author, options = {}) {
     return gitNewValidationRequest(git, files, description, author, options);

package/dist/src/rules/types.d.ts

@@ -122,7 +122,7 @@ export interface FrontMatter {
     id?: number;
     /** Titre descriptif de la règle */
     title: string;
-    /** Ancien nom de fichier (pour renommage) */
+    /** FIXME(oldfile): champ legacy transitoire pour notification UI rename */
     oldfile?: string;
     /** Auteur original de la règle (format git: "Name <email>") */
     author?: string;

package/dist/src/stategraph/stategraph.d.ts

@@ -38,7 +38,7 @@ export declare class AgentStateGraph implements IAgentStateGraph {
     set(agentName: string, content: string): void;
     /**
      * Ajoute une étape au CONTEXT TRAIL et met à jour le message system
-     * Injecte automatiquement le trail dans le system message via updateSystemMessage()
+     * Injecte automatiquement le trail dans le system message via updateSystemContextTrail()
      *
      * Utilisée par:
      * - readCompletionsStream (responses.ts) : après chaque tool call exécuté
@@ -64,7 +64,7 @@ export declare class AgentStateGraph implements IAgentStateGraph {
      * Met à jour le message system avec le nouveau trail via regexp
      * @param agentName Nom de l'agent
      */
-    private updateSystemMessage;
+    private updateSystemContextTrail;
     /**
      * Additionne l'usage des tokens pour un agent
      * @param agentName Nom de l'agent

package/dist/src/stategraph/stategraph.js

@@ -86,12 +86,12 @@ class AgentStateGraph {
             discussion.messages[0].agent = systemMessage.agent;
             discussion.messages[0].timestamp = new Date();
         }
-        this.updateSystemMessage(discussion);
+        this.updateSystemContextTrail(discussion);
         discussion.updatedAt = new Date();
     }
     /**
      * Ajoute une étape au CONTEXT TRAIL et met à jour le message system
-     * Injecte automatiquement le trail dans le system message via updateSystemMessage()
+     * Injecte automatiquement le trail dans le system message via updateSystemContextTrail()
      *
      * Utilisée par:
      * - readCompletionsStream (responses.ts) : après chaque tool call exécuté
@@ -105,7 +105,7 @@ class AgentStateGraph {
         if (!discussion)
             return;
         discussion.trailSteps.push(step);
-        this.updateSystemMessage(discussion);
+        this.updateSystemContextTrail(discussion);
     }
     /**
      * Retourne tous les steps du CONTEXT TRAIL pour une discussion
@@ -127,15 +127,14 @@ class AgentStateGraph {
             return '(No actions yet)';
         }
         return discussion.trailSteps.map(s => {
-            const idPart = s.id ? ` [id: ${s.id}]` : '';
-            return `- ${s.tool}: ${s.context} "${s.reason}"${idPart}`;
+            return `- ${s.tool}: ${s.reason}`;
         }).join('\n');
     }
     /**
      * Met à jour le message system avec le nouveau trail via regexp
      * @param agentName Nom de l'agent
      */
-    updateSystemMessage(discussion) {
+    updateSystemContextTrail(discussion) {
         const systemMessage = discussion.messages.find((m) => m.role === 'system');
         if (!systemMessage)
             return;

package/dist/src/stategraph/types.d.ts

@@ -41,14 +41,10 @@ export interface TokenUsage {
  * Utilisé pour maintenir une mémoire de travail des actions effectuées par les outils
  */
 export interface StepTrail {
-    /** Nom de l'outil (ex: "transferAgents", "database") */
+    /** Nom de l'outil (ex: "lookupMfilesIntervenant", "lookupKnowledge") */
     tool: string;
-    /** Contexte de l'action (ex: "orientation → guess-word") */
-    context: string;
-    /** Justification/raison de l'action */
+    /** Raison de l'appel (context du résultat ou justification de l'appel) */
     reason: string;
-    /** ID optionnel pour détection de boucles */
-    id?: string;
 }
 /**
  * Discussion d'agent - thread de messages pour un agent spécifique

package/dist/src/types.d.ts

@@ -42,6 +42,7 @@ export type ToolContractOutput<T> = {
         can_retry: boolean;
         error_code?: string;
         transfer_message?: string;
+        pipeline?: string[];
     };
 };
 export interface AgentModelMapping {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agentic-api",
-  "version": "2.0.646",
+  "version": "2.0.684",
   "description": "API pour l'orchestration d'agents intelligents avec séquences et escalades automatiques",
   "main": "dist/src/index.js",
   "types": "dist/src/index.d.ts",