merge(cli): annonces inter-agent et messages MCP préservés après changement de projet — #236 (QA verte)

2026-08-22 10:02:52 +02:00
parent 85f659c788 1e172c6e81
commit 963b51a277
9 changed files with 175 additions and 32 deletions
--- a/.ideai/tickets/200/carnet.md
+++ b/.ideai/tickets/200/carnet.md
@ -1,8 +1,8 @@
 ---
 issueRef: "#200"
-version: 16
+version: 19
 updatedBy: {"kind":"agent","agent_id":"a6c6ea12-bfc6-4bdc-8031-324102dfa34d"}
-updatedAt: 1787384954575
+updatedAt: 1787385038497
 ---
 2026-08-14 Main
 - Reprise du ticket après livraison initiale du lot Codex-centric.
@ -19,4 +19,6 @@ updatedAt: 1787384954575
 - Analyse Architect: pas un gap de design; suspicion de chemin secondaire ou de comportement agent qui continue d'utiliser/mentionner `/tmp` explicitement au lieu de suivre l'env temp fourni.
 - Suite proposée: reproduire le cas réel avec le provider/profil concerné, vérifier la valeur réellement vue dans le process agent, puis décider entre fix runtime (si env perdu/contourné) et renfort d'instruction/runtime API (si l'agent hardcode `/tmp`).
 - Git a retenu une branche de fix dédiée `fix/200-managed-tmp-injection-regression` depuis `develop`, HEAD `3344361fc27bace7f68953f7a11623af1d8a7d86`.
- DevBackend: diagnostic retenu = l'env temp était déjà injecté, mais le convention file était généré avant l'allocation du temp, donc aucun signal textuel explicite ne guidait l'agent; correctif implémenté = allocation plus tôt dans le launch path, ajout de `IDEA_MANAGED_TMP`, et injection conditionnelle d'une instruction “Temporaire IdeA géré” avec le chemin concret; tests agent_lifecycle renforcés.
+- DevBackend: diagnostic retenu = l'env temp était déjà injecté, mais le convention file était généré avant l'allocation du temp, donc aucun signal textuel explicite ne guidait l'agent; correctif implémenté = allocation plus tôt dans le launch path, ajout de `IDEA_MANAGED_TMP`, et injection conditionnelle d'une instruction “Temporaire IdeA géré” avec le chemin concret; tests agent_lifecycle renforcés.
+- QA 2026-08-22: VERDICT VERT. Commandes validées: `cargo test -p application --test agent_lifecycle launch_pty_allocates_managed_tmp_env_and_sandbox_from_same_path -- --exact --nocapture`, `cargo test -p application --test agent_lifecycle launch_structured_non_codex_uses_managed_tmp_for_env_without_os_sandbox -- --exact --nocapture`, `cargo test -p application --test agent_lifecycle launch_structured_codex_uses_managed_tmp_for_env_and_writable_roots_without_os_sandbox -- --exact --nocapture`, `cargo test -p application --test agent_lifecycle managed_tmp -- --nocapture`.
+- Git 2026-08-22: commit local `f8cf86c` `fix(infra): temp IdeA géré alloué avant le convention file, exposé via IDEA_MANAGED_TMP et instruction explicite privilégiant le temp géré à /tmp — #200`; merge local `--no-ff` vers `develop` = `85f659c` `merge(infra): temp IdeA géré visible par l'agent avant le spawn — #200 (QA verte)`; aucun push.
--- a/.ideai/tickets/200/issue.md
+++ b/.ideai/tickets/200/issue.md
@ -2,7 +2,7 @@
 id: "0268a6a4-2ceb-4332-a113-29b93a59caf1"
 number: 200
 title: "Remplacer l'usage implicite de /tmp par un temp IdeA géré pour les agents"
-status: "inProgress"
+status: "closed"
 priority: "high"
 sprint: null
 links: []
@ -11,7 +11,7 @@ attachments: []
 createdBy: {"kind":"agent","agent_id":"a6c6ea12-bfc6-4bdc-8031-324102dfa34d"}
 updatedBy: {"kind":"agent","agent_id":"a6c6ea12-bfc6-4bdc-8031-324102dfa34d"}
 createdAt: 1786743649446
-updatedAt: 1787384954575
-version: 16
+updatedAt: 1787385038497
+version: 19
 ---
 Extension du lot temp IdeA géré. Le temp principal launch-scoped existe déjà mais le comportement doit maintenant être généralisé à tous les agents, pas seulement Codex, afin que n'importe quel agent puisse utiliser le répertoire temporaire IdeA via ses variables d'environnement/runtime appropriées. En plus du release au stop de session déjà en place, IdeA doit purger le contenu de ce stockage temporaire à la fermeture de l'application pour éviter l'accumulation résiduelle. Le lot couvre donc: projection cross-agent du temp IdeA principal, cohérence sandbox/runtime selon les profils, et purge à l'extinction d'IdeA.
--- a/.ideai/tickets/236/carnet.md
+++ b/.ideai/tickets/236/carnet.md
@ -0,0 +1,25 @@
+---
+issueRef: "#236"
+version: 12
+updatedBy: {"kind":"agent","agent_id":"a6c6ea12-bfc6-4bdc-8031-324102dfa34d"}
+updatedAt: 1787385711767
+---
+2026-08-21 Main
+- Bug signalé: dans la custom CLI, la conversation s'affiche bien au début puis les `progress`, messages MCP et assimilés cessent de s'afficher au bout d'un moment.
+- Architect: suspect prioritaire frontend, dans `frontend/src/features/agents/CustomAgentChatView.tsx`, autour de `receive`, `foldChunk`, `appendProgress`, `progressUpsertKey`, `progressEntityKey`.
+- Hypothèse principale confirmée par DevFrontend: collisions de clé / upsert trop agressif sur `progress` `tool`/`mcp` sans `progressId`, causant des remplacements au lieu d'un enrichissement correct de l'historique visible.
+- DevFrontend a corrigé la logique legacy: cycles successifs sans `progressId` restent visibles; `completed` finalise puis libère la clé active; doublons provider/IdeA d'une même complétion restent dédupliqués.
+- Fichiers modifiés: `frontend/src/features/agents/CustomAgentChatView.tsx`, `frontend/src/features/agents/CustomAgentChatView.test.tsx`.
+- QA: VERT. Commandes exécutées: `npm test -- src/features/agents/CustomAgentChatView.test.tsx` => 55/55 verts; `npm test -- src/features/agents` => 151/151 verts; `npm run typecheck` => OK.
+- Git a commité le correctif: `aaf628f` `fix(cli): préserve l'affichage des progress legacy sans progressId via clés d'occurrence dédiées — #236`.
+- Git a mergé localement le fix dans `develop` via `--no-ff`: `2c8c71c` `merge(cli): les messages progress/MCP ne disparaissent plus après un certain temps — #236 (QA verte)`.
+- Aucun push effectué. Ticket clôturé.
+
+2026-08-22 Main
+- Réouverture du ticket pour un cas résiduel distinct observé utilisateur: un agent est en cours de travail mais le message MCP / l'annonce inter-agent attendu n'apparaît pas sur la surface des messages.
+- UX: comportement attendu = preview côté demandeur + overlay côté cible visibles pendant le travail, puis purge à idle; après changement de projet, aucune fuite visuelle inter-projet, et réhydratation correcte dans le projet courant.
+- Architect: diagnostic le plus probable = store/provider d'annonces non scopé par `projectId`; l'état app-global survit au switch, `seedBusy` n'hydrate que si absent, et un tour déjà en cours peut donc ne jamais remonter visuellement dans le nouveau projet.
+- Correctif minimal demandé au frontend: `reset + filter + rehydrate` à chaque changement de `projectId`, idéalement avec un store project-scoped.
+- Git: branche locale retenue `fix/236-annonces-store-par-projectid` depuis `develop` @ `85f659c`.
+- DevFrontend: diagnostic confirmé = provider app-global non borné par projet; correctif implémenté = provider explicitement borné par `projectId`, reset de `index + busy` au changement de projet, filtrage des `agentAnnouncement` par `event.projectId`, tests ciblés verts et `npm run typecheck` vert.
+- Risque résiduel signalé par DevFrontend: `agentBusyChanged` ne porte pas encore `projectId`, donc le lifecycle live busy ne peut pas être filtré aussi strictement que les annonces; le bug visé est néanmoins corrigé par reset + réhydratation au switch.
--- a/.ideai/tickets/236/issue.md
+++ b/.ideai/tickets/236/issue.md
@ -0,0 +1,17 @@
+---
+id: "5c03537c-2daf-4802-a6fb-6c8aa92b8aa7"
+number: 236
+title: "Custom CLI: les messages progress/MCP disparaissent après un certain temps"
+status: "inProgress"
+priority: "critical"
+sprint: null
+links: []
+agentRefs: []
+attachments: []
+createdBy: {"kind":"agent","agent_id":"a6c6ea12-bfc6-4bdc-8031-324102dfa34d"}
+updatedBy: {"kind":"agent","agent_id":"a6c6ea12-bfc6-4bdc-8031-324102dfa34d"}
+createdAt: 1787326716055
+updatedAt: 1787385711767
+version: 12
+---
+Bug sur l'affichage des conversations dans la custom CLI: l'historique s'affiche correctement au début, puis au bout d'un moment les messages `progress`, usages MCP et assimilés cessent de s'afficher. Il faut reproduire, diagnostiquer la cause et corriger sans casser l'affichage initial.
--- a/frontend/src/app/App.tsx
+++ b/frontend/src/app/App.tsx
@ -9,7 +9,6 @@ import { useEffect, useState } from "react";
 import type { DomainEvent, HealthReport } from "@/domain";
 import { ProjectsView } from "@/features/projects";
 import { FirstRunWizard } from "@/features/first-run";
-import { AnnouncementsProvider } from "@/features/announcements";
 import { AppExitConfirmDialog } from "@/features/appExit";
 import { PluginRuntimeProvider } from "@/features/plugins";
 import { Panel, Spinner, Toolbar } from "@/shared";
@ -65,7 +64,6 @@ export function App() {
  }, [profile]);

  return (
-    <AnnouncementsProvider>
    <PluginRuntimeProvider>
    <div className="flex h-full flex-col bg-canvas text-content">
      {/* ── Header ── */}
@ -119,7 +117,6 @@ export function App() {
    </div>
    <AppExitConfirmDialog />
    </PluginRuntimeProvider>
-    </AnnouncementsProvider>
  );
 }

--- a/frontend/src/features/announcements/AnnouncementsProvider.test.tsx
+++ b/frontend/src/features/announcements/AnnouncementsProvider.test.tsx
@ -24,17 +24,19 @@ import { TargetAnnouncementsOverlay } from "./TargetAnnouncementsOverlay";

 function setup(
  node: React.ReactNode,
-  opts: { workState?: MockWorkStateGateway } = {},
+  opts: { projectId?: string | null; workState?: MockWorkStateGateway } = {},
 ) {
  const system = new MockSystemGateway();
  const workState = opts.workState;
  const gateways = { system, workState } as unknown as Gateways;
-  render(
+  const result = render(
    <DIProvider gateways={gateways}>
-      <AnnouncementsProvider>{node}</AnnouncementsProvider>
+      <AnnouncementsProvider projectId={opts.projectId ?? "p1"}>
+        {node}
+      </AnnouncementsProvider>
    </DIProvider>,
  );
-  return { system };
+  return { system, ...result };
 }

 type AnnouncementEvent = Extract<DomainEvent, { type: "agentAnnouncement" }>;
@ -84,6 +86,27 @@ describe("F3 — overlay lifecycle is the target's busy state", () => {
    expect(screen.getByText("en cours")).not.toBeNull();
  });

+  it("ignores streamed announcements from another project", async () => {
+    const { system } = setup(
+      <TargetAnnouncementsOverlay projectId="p1" agentId="agent-T" />,
+    );
+    await flush();
+
+    act(() => {
+      system.emit(busy("agent-T", true));
+      system.emit(
+        announcement({
+          projectId: "p2",
+          target: "agent-T",
+          text: "other-project",
+        }),
+      );
+    });
+
+    expect(overlay()).not.toBeNull();
+    expect(screen.queryByText("other-project")).toBeNull();
+  });
+
  it("mounts on busy:true even before any announcement (banner only)", async () => {
    const { system } = setup(
      <TargetAnnouncementsOverlay projectId="p1" agentId="agent-T" />,
@ -171,6 +194,64 @@ describe("F3 — overlay lifecycle is the target's busy state", () => {
    expect(overlay()).not.toBeNull();
  });

+  it("resets and rehydrates when switching project", async () => {
+    const workState = new MockWorkStateGateway();
+    workState._setProjectWorkState("p1", {
+      agents: [
+        {
+          agentId: "agent-T",
+          name: "Target",
+          profileId: "prof",
+          busy: { state: "idle" },
+          tickets: [],
+        },
+      ],
+      conversations: [],
+    });
+    workState._setProjectWorkState("p2", {
+      agents: [
+        {
+          agentId: "agent-T",
+          name: "Target",
+          profileId: "prof",
+          busy: { state: "busy", ticket: "t-p2", sinceMs: 456 },
+          tickets: [],
+        },
+      ],
+      conversations: [],
+    });
+
+    const system = new MockSystemGateway();
+    const gateways = { system, workState } as unknown as Gateways;
+    const { rerender } = render(
+      <DIProvider gateways={gateways}>
+        <AnnouncementsProvider projectId="p1">
+          <TargetAnnouncementsOverlay projectId="p1" agentId="agent-T" />
+        </AnnouncementsProvider>
+      </DIProvider>,
+    );
+    await flush();
+
+    act(() => {
+      system.emit(busy("agent-T", true));
+      system.emit(announcement({ projectId: "p1", text: "old-p1" }));
+    });
+    expect(screen.getByText("old-p1")).not.toBeNull();
+
+    rerender(
+      <DIProvider gateways={gateways}>
+        <AnnouncementsProvider projectId="p2">
+          <TargetAnnouncementsOverlay projectId="p2" agentId="agent-T" />
+        </AnnouncementsProvider>
+      </DIProvider>,
+    );
+    await flush();
+
+    expect(overlay()).not.toBeNull();
+    expect(screen.queryByText("old-p1")).toBeNull();
+    expect(screen.getByText("Travaille sur une demande en cours.")).not.toBeNull();
+  });
+
  it("stays down when the read-model reports the agent idle", async () => {
    const workState = new MockWorkStateGateway();
    workState._setProjectWorkState("p1", {
--- a/frontend/src/features/announcements/AnnouncementsProvider.tsx
+++ b/frontend/src/features/announcements/AnnouncementsProvider.tsx
@ -1,5 +1,5 @@
 /**
- * `AnnouncementsProvider` — the live, app-wide store for inter-agent announcements
+ * `AnnouncementsProvider` — the live, project-scoped store for inter-agent announcements
 * (ticket #4, F1 wiring). Subscribes **once** to the domain-event stream via the
 * {@link SystemGateway} and folds two orthogonal signals:
 *
@ -22,8 +22,9 @@
 * busy is the sole lifecycle authority, which is exactly what we want.)
 *
 * Consumers read via {@link useTargetAnnouncements} (F3) and
- * {@link useRequesterAnnouncements} (F2). The store is never persisted — it is
- * rebuilt live from the event stream and the read-model snapshot.
+ * {@link useRequesterAnnouncements} (F2). The store is never persisted and is
+ * reset whenever the provider's project changes; it is rebuilt live from the
+ * event stream and the read-model snapshot for that project.
 */

 import {
@ -76,7 +77,13 @@ const DEFAULT_STORE: AnnouncementsStore = {

 const AnnouncementsContext = createContext<AnnouncementsStore | null>(null);

-export function AnnouncementsProvider({ children }: { children: ReactNode }) {
+export function AnnouncementsProvider({
+  projectId,
+  children,
+}: {
+  projectId: string | null;
+  children: ReactNode;
+}) {
  const { system } = useGateways();
  const [index, setIndex] = useState<AnnouncementIndex>(emptyAnnouncementIndex);
  const [busy, setBusy] = useState<BusyMap>({});
@ -87,15 +94,21 @@ export function AnnouncementsProvider({ children }: { children: ReactNode }) {
    setBusy((prev) => (agentId in prev ? prev : { ...prev, [agentId]: value }));
  }, []);

+  useEffect(() => {
+    setIndex(emptyAnnouncementIndex());
+    setBusy({});
+  }, [projectId]);
+
  useEffect(() => {
    // `system` may be absent in unit tests injecting a partial gateway set.
-    if (!system) return;
+    if (!system || !projectId) return;
    let unsubscribe: (() => void) | undefined;
    let cancelled = false;

    void system
      .onDomainEvent((event) => {
        if (event.type === "agentAnnouncement") {
+          if (event.projectId !== projectId) return;
          // Content of the overlay/preview.
          setIndex((prev) =>
            appendAnnouncement(prev, {
@ -127,7 +140,7 @@ export function AnnouncementsProvider({ children }: { children: ReactNode }) {
      cancelled = true;
      unsubscribe?.();
    };
-  }, [system]);
+  }, [system, projectId]);

  const store = useMemo<AnnouncementsStore>(
    () => ({ index, busy, seedBusy }),
--- a/frontend/src/features/layout/LayoutGrid.cellControlsLayering.test.tsx
+++ b/frontend/src/features/layout/LayoutGrid.cellControlsLayering.test.tsx
@ -123,7 +123,7 @@ function renderGrid(
 ) {
  return render(
    <DIProvider gateways={gateways}>
-      <AnnouncementsProvider>
+      <AnnouncementsProvider projectId="p1">
        <LayoutGrid projectId="p1" cwd="/home/me/proj" {...props} />
      </AnnouncementsProvider>
    </DIProvider>,
--- a/frontend/src/features/projects/ProjectsView.tsx
+++ b/frontend/src/features/projects/ProjectsView.tsx
@ -52,6 +52,7 @@ import {
  type SettingsSection,
 } from "@/features/settings";
 import { GitGraphView } from "@/features/git";
+import { AnnouncementsProvider } from "@/features/announcements";
 import { usePluginMenus, usePluginRuntime } from "@/features/plugins";
 import type { WhenContext } from "@/plugins/runtime";
 import {
@ -735,7 +736,7 @@ export function ProjectsView() {
        <p className="text-sm text-muted">Open a project to use this panel.</p>
      );
    }
-    return (
+    const body = (
      <ViewPanelBody
        panel={panel as ViewPanelId}
        projectId={active.id}
@ -743,6 +744,11 @@ export function ProjectsView() {
        onOpenConversation={openConversation}
      />
    );
+    return panel === "agents" ? (
+      <AnnouncementsProvider projectId={active.id}>{body}</AnnouncementsProvider>
+    ) : (
+      body
+    );
  }

  // Projects manager renders inline in the welcome area only when it is not
@ -924,17 +930,19 @@ export function ProjectsView() {
                projectId={active.id}
              />
            ) : (
-              <LayoutGrid
-                key={`${active.id}-${activeLayout?.id ?? "default"}`}
-                projectId={active.id}
-                cwd={active.root}
-                layoutId={activeLayout?.id}
-                onOpenConversation={openConversation}
-                onOpenPluginsSettings={() => {
-                  setSettingsSection("plugins");
-                  dismissFloating();
-                }}
-              />
+              <AnnouncementsProvider projectId={active.id}>
+                <LayoutGrid
+                  key={`${active.id}-${activeLayout?.id ?? "default"}`}
+                  projectId={active.id}
+                  cwd={active.root}
+                  layoutId={activeLayout?.id}
+                  onOpenConversation={openConversation}
+                  onOpenPluginsSettings={() => {
+                    setSettingsSection("plugins");
+                    dismissFloating();
+                  }}
+                />
+              </AnnouncementsProvider>
            )}
          </>
        ) : viewerConversation ? (