chore(wip): état runtime .ideai (flux conversation live)
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
@ -1,11 +1,9 @@
|
||||
---
|
||||
upTo: 8e30c171-2d32-48ab-9971-c6d23a010052
|
||||
upTo: ec87367b-50e1-44fb-b4dd-88c284f02cae
|
||||
objective: IMPLÉMENTATION (code de production uniquement — PAS les nouveaux tests, c'est QA qui les écrit ; mais ne casse aucun test existant). Cadrage validé par l'Architect. Objectif : permettre la délégation
|
||||
---
|
||||
**Objectif :** IMPLÉMENTATION (code de production uniquement — PAS les nouveaux tests, c'est QA qui les écrit ; mais ne casse aucun test existant). Cadrage validé par l'Architect. Objectif : permettre la délégation
|
||||
|
||||
- **Response:** LS8-backend (filet humain niveau 3) implémenté tel que cadré — verdict B. Build OK, tous tests verts, NON commité. ## Signatures domaine constatées (crates/domain/src/session_limit.rs) — RIEN ajouté - `RateLimitSource::{Structured, Pattern, Human}` (enum Copy). - `SessionLimit::new(resets_at_ms: Option<i64>, detected_at_ms: i64, source: RateLimitSource) -> Self` (const, ordre des args = (Option heure, now, source)). - `plan_resume(now_ms: i64, limit: &SessionLimit, conversation_id: Option<String>) -> ResumePlan`. - `ResumePlan::Scheduled { fire_at_ms: i64, conversation_id: Option<String> }` | `ResumePlan::HumanFallback`. Le clamp anti-passé (`fire_at_ms = resets_at_ms.max(now_ms)`) est dans `plan_resume` ⇒ heure saisie déjà passée = reprise immédiate, gratis. ## Fichiers modifiés 1. **crates/application/src/agent/session_limit.rs** - Factorisé la branche `ResumePlan::Scheduled` de `on_rate_limited` en privée `fn arm_scheduled(&self, agent_id, fire_at_ms, node_id, conversation_id, resets_at_ms: Option<i64>)`. `on_rate_limited` l'appelle pour son cas Scheduled → comportement identique (les 9 tests `session_limit_service` existants passent inchangés). - ⚠️ **Écart assumé vs signature suggérée** : j'ai ajouté un 5e param `resets_at_ms: Option<i64>` à `arm_scheduled`. Raison : l'événement `AgentRateLimited` d'origine publie l'**heure de reset brute** (l'Option d'entrée), PAS `fire_at_ms` (qui est clampé). Les deux diffèrent si le reset est passé. Pour garantir le « zéro régression » exigé, j'ai propagé l'heure de reset brute au lieu de republier `Some(fire_at_ms)`. Les deux appelants passent `Some(...)`. - Ajouté la publique `pub fn confirm_human_resume(&self, agent_id: AgentId, node_id: NodeId, conversation_id: Option<String>, resets_at_ms: i64)` : `SessionLimit::new(Some(resets_at_ms), now, RateLimitSource::Human)` → `plan_resume` → sur `Scheduled` appelle `arm_scheduled(..., Some(resets_at_ms))`. Cas `HumanFallback` traité en no-op défensif (inatteignable car `resets_at_ms` toujours `Some`). `execute_resume` et `cancel_resume` INCHANGÉS. 2. **crates/app-tauri/src/commands.rs** — ajout `#[tauri::command] pub async fn set_resume_at(agent_id: String, resets_at_ms: i64, state) -> Result<(), ErrorDto>`, miroir de `cancel_resume`. 3. **crates/app-tauri/src/lib.rs** — `commands::set_resume_at` enregistrée juste après `commands::cancel_resume` dans le `generate_handler!`. ## Lookup agent→node utilisé (existant, rien ajouté) La registry des sessions vivantes expose déjà `node_for_agent`. `set_resume_at` résout : - `node_id = state.structured_sessions.node_for_agent(&id).or_else(|| state.terminal_sessions.node_for_agent(&id))` (structuré d'abord car il porte aussi la conv ; repli terminal/PTY). Aucune cellule vivante ⇒ `ErrorDto::from(AppError::NotFound(...))`. - `conversation_id` best-effort = `state.structured_sessions.session_for_agent(&id).and_then(|s| s.conversation_id())` (via `AgentSession::conversation_id() -> Option<String>`, ports.rs:549). `None` toléré. Méthodes constatées dans crates/application/src/terminal/registry.rs : `TerminalSessions::node_for_agent` (l.175), `StructuredSessions::node_for_agent` (l.375), `session_for_agent`, `meta_for_session`. ## Événements : AUCUN créé. `confirm_human_resume` → `arm_scheduled` réémet la paire existante `AgentRateLimited{Some(t)}` + `AgentResumeScheduled{fire_at_ms}`. Annulation/reprise par les voies inchangées (`AgentResumeCancelled`/`AgentResumed`). ## Compilation / tests - `cargo build -p application -p app-tauri` : Finished, 0 erreur/0 warning. - `cargo test -p domain -p application` : tout vert (domain 39, session_limit_service 9, session_limit_t4 7, + le reste). - `cargo test -p app-tauri` : tout vert (dont wiring 43). ## Pour QA (testabilité) - `confirm_human_resume` testable 100 % sans I/O via les fakes existants (Clock/Scheduler/EventBus) déjà utilisés dans `tests/session_limit_service.rs`. Cas à couvrir : (a) heure future ⇒ `AgentRateLimited{Some}` puis `AgentResumeScheduled{fire_at_ms=resets_at_ms}` + ScheduleId mémorisé ; (b) heure passée ⇒ `fire_at_ms == now` (reprise immédiate) ; (c) dédoublonnage : un `confirm_human_resume` après un armement existant annule le précédent (1 seule entrée armée) ; (d) annulabilité : `cancel_resume` après `confirm_human_resume` publie `AgentResumeCancelled` ; (e) parité auto/humain : même séquence d'événements que `on_rate_limited` cas Scheduled. - `set_resume_at` : `node_for_agent` None ⇒ NOT_FOUND.
|
||||
- **Prompt:** Feature « Agent orchestrateur du projet », branche active `feature/orchestrator-designation`. Implémente le **ticket T1 — Fondation domaine** (pur, aucune I/O). Cadrage Architect ci-dessous, à respecter strictement. ## Modèle On ne persiste que la DÉVIATION au défaut : un champ projet `orchestrator: Option<AgentId>`. `None` = défaut « plus ancien agent » (= `entries.first()`). `Some(id)` = désignation explicite (radio). Ça rend l'état illégal « 2 orchestrateurs » non représentable et donne la rétro-compat gratuitement. ## À implémenter (domaine pur uniquement, T1) 1. **VO `OrchestratorDesignation`** (dans `crates/domain/src/fileguard.rs` ou un petit module dédié, à toi de voir le plus cohérent) : - `none() -> Self` (aucun agent désigné, humain seul) - `of(agent: AgentId) -> Self` - `designated(&self) -> Option<AgentId>` 2. **Politique `fileguard.rs`** — élargis les signatures : - `is_orchestrator(who: ConversationParty, d: &OrchestratorDesignation) -> bool` : `User => true` (humain permanent), `Agent{id} => d.designated() == Some(id)`. (Perd le `const`, OK.) - `may_write_directly(who, res, d: &OrchestratorDesignation) -> bool` : si `res.is_project_context()` → `is_orchestrator(who, d)`, sinon `true`. - **Préserve le test single-writer existant** en lui passant `&OrchestratorDesignation::none()` : l'agent reste refusé, l'humain reste autorisé. 3. **`AgentManifest` (`crates/domain/src/agent.rs`)** : - Champ `orchestrator: Option<AgentId>` avec `#[serde(default, skip_serializing_if = "Option::is_none")]`. - `effective_orchestrator(&self) -> Option<AgentId>` = `self.orchestrator.or_else(|| self.entries.first().map(|e| e.agent_id))` (adapte au vrai nom du champ id de l'entrée). - `orchestrator_designation(&self) -> OrchestratorDesignation` (fold de l'effectif vers le VO). - `designate(&mut self, id) -> Result<(), DomainError>` : sémantique radio (écrase), valide que `id` appartient à `entries`. - `on_agent_deleted(&mut self, removed: AgentId)` : si `orchestrator == Some(removed)` → `None`. - Validation dans le constructeur (`AgentManifest::new` ou équivalent) : `orchestrator == Some(id)` ⇒ `id` doit être présent dans `entries` ; `None` toujours valide. - **Invariant à documenter** : les entrées du manifeste sont en ordre de création (`entries.first()` = plus ancien). Pas de timestamp à ajouter. 4. **`events.rs`** : nouveau `DomainEvent::OrchestratorChanged { project_id, orchestrator: Option<AgentId> }` (adapte aux champs/conventions des variants existants). ## Tests unitaires (dans les modules, T1) - single-writer préservé avec `none()` (agent refusé / humain autorisé) ; - agent désigné autorisé à écrire le contexte projet ; - défaut « plus ancien » quand `orchestrator == None` ; - succession lazy : `on_agent_deleted` du désigné → retombe sur plus ancien ; - `designate` écrase (radio) + rejette un agent absent ; - validation référentielle du constructeur. Respecte l'archi hexagonale (domaine sans I/O), SOLID, le style existant. Compile + teste le crate domaine (`cargo test -p domain` ou le vrai nom du package). Quand c'est vert, réponds via `idea_reply` avec : fichiers touchés, signatures finales exactes (pour que les tickets aval s'y accrochent), et la sortie réelle des tests. Si rouge, donne la sortie d'erreur réelle.
|
||||
- **Response:** ## T1 — Fondation domaine : VERT ✅ `cargo test -p domain` passe intégralement : **lib 175 passed; 0 failed** (dont 11 nouveaux tests), + toutes les suites d'intégration du crate (entities, serde_roundtrip, etc.) restent vertes. Aucune I/O introduite, domaine pur. ### Fichiers touchés (domaine pur uniquement) - `crates/domain/src/fileguard.rs` — VO `OrchestratorDesignation` + élargissement des signatures de politique + tests. - `crates/domain/src/agent.rs` — champ `orchestrator` sur `AgentManifest` + méthodes + tests (module `orchestrator_tests`). - `crates/domain/src/events.rs` — variant `DomainEvent::OrchestratorChanged` + test. - `crates/domain/src/lib.rs` — réexport de `OrchestratorDesignation`. ### Signatures finales exactes (points d'accroche pour les tickets aval) **`fileguard.rs`** (réexportées depuis `domain::`) : ```rust pub struct OrchestratorDesignation(/* private Option<AgentId> */); impl OrchestratorDesignation { pub const fn none() -> Self; pub const fn of(agent: AgentId) -> Self; pub const fn designated(&self) -> Option<AgentId>; } // dérive Debug, Clone, Copy, PartialEq, Eq pub fn is_orchestrator(who: ConversationParty, d: &OrchestratorDesignation) -> bool; pub fn may_write_directly( who: ConversationParty, res: &GuardedResource, d: &OrchestratorDesignation, ) -> bool; ``` Sémantique : `User => true` (humain permanent), `Agent{id} => d.designated() == Some(id)`. `may_write_directly` ⇒ `is_orchestrator` si `res.is_project_context()`, sinon `true`. Les deux perdent `const` (le `match` sur `who` n'est pas const-compatible ici). **`agent.rs` — `AgentManifest`** (nouveau champ `pub orchestrator: Option<AgentId>` avec `#[serde(default, skip_serializing_if = "Option::is_none")]`) : ```rust // constructeur historique inchangé (orchestrator = None, toujours valide) pub fn new(version: u32, entries: Vec<ManifestEntry>) -> Result<Self, DomainError>; // constructeur validant la désignation explicite (intégrité référentielle) pub fn with_orchestrator( version: u32, entries: Vec<ManifestEntry>, orchestrator: Option<AgentId>, ) -> Result<Self, DomainError>; pub fn effective_orchestrator(&self) -> Option<AgentId>; // designation || entries.first() pub fn orchestrator_designation(&self) -> OrchestratorDesignation; // fold de l'effectif vers le VO pub fn designate(&mut self, id: AgentId) -> Result<(), DomainError>; // radio, écrase, valide l'appartenance pub fn on_agent_deleted(&mut self, removed: AgentId); // Some(removed) -> None (succession lazy) ``` Choix d'implémentation à noter : `AgentManifest::new` garde **sa signature** (orchestrator = None) ; la validation référentielle du constructeur demandée est portée par `with_orchestrator` (`new` y délègue avec `None`). Ça évite de casser tous les appelants `AgentManifest::new(version, entries)` existants. La validation : `orchestrator == Some(id)` ⇒ `id ∈ entries`, sinon `DomainError::InconsistentManifest`. Invariant d'ordre documenté sur le doccomment du struct : `entries` en ordre de création, `entries.first()` = plus ancien, pas de timestamp. **`events.rs`** : ```rust DomainEvent::OrchestratorChanged { project_id: ProjectId, orchestrator: Option<AgentId>, } ``` ### Tests couverts (tous verts) - single-writer préservé avec `none()` (agent refusé / humain autorisé) ; - agent désigné autorisé à écrire le contexte projet (+ autre agent refusé, humain toujours OK) — testé côté `fileguard` ET via `AgentManifest::orchestrator_designation()` ; - défaut « plus ancien » quand `orchestrator == None` (+ manifeste vide ⇒ `None`) ; - succession lazy : `on_agent_deleted` du désigné → `None` → retombe sur plus ancien ; no-op si l'agent supprimé n'était pas le désigné ; - `designate` écrase (radio) + rejette un agent absent (et préserve la désignation valide précédente) ; - validation référentielle du constructeur (`with_orchestrator`) ; - `OrchestratorChanged` construit/compare. ### ⚠️ Impact aval à prévoir (hors scope T1, à câbler par les tickets suivants) La couche **infrastructure ne compile plus tel quel** — c'est attendu, le câblage est aval : 1. `crates/infrastructure/src/fileguard/mod.rs:93` appelle encore `may_write_directly(who, &res)` (2 args). L'adaptateur `RwFileGuard` doit recevoir/threader une `OrchestratorDesignation` (probablement injectée depuis le manifeste courant) et passer 3 args. 2. Le nouveau champ `orchestrator` casse les **littéraux de struct exhaustifs** `AgentManifest { version, entries }` (≈ infra `store/context.rs`, application `context_guard.rs`/tests, `app-tauri/state.rs`, plusieurs tests). Fix mécanique : ajouter `orchestrator: None` ou `..Default::default()`. `Default` est dérivé donc `..Default::default()` marche. `cargo test -p domain` étant le périmètre demandé, le crate domaine est vert et autonome ; ces deux points sont la surface de câblage des tickets T2+.
|
||||
- **Prompt:** Suite de la feature orchestrateur, branche `feature/orchestrator-designation`. T1 (domaine) est vert. Implémente maintenant **T2 → T5** (infra + application backend) pour rétablir un workspace qui compile et tester. Suis le cadrage Architect, notamment l'**Alternative A** retenue. ### T2 — Infra : guard = verrou pur + serde rétro-compatible (dépend T1) - **Alternative A (RETENUE)** : `RwFileGuard::acquire_write` (`crates/infrastructure/src/fileguard/mod.rs`) NE fait plus le check d'identité orchestrateur. Il redevient un **verrou pur** (sérialise les writers, comme un rwlock). Donc retire l'appel `may_write_directly` côté guard (la ligne ~93). L'autorisation single-writer remonte dans `ProposeContext` (T3). `GuardError::Forbidden` n'est plus émis par le guard — vérifie ce que ça implique pour le port/les tests du guard (déplace/retire le test « Forbidden » qui n'a plus lieu d'être à ce niveau, documente que le guard est désormais un lock pur). Si la signature du port `FileGuard::acquire_write` portait un paramètre lié à l'identité, garde-la cohérente. - **Serde** : corrige tous les littéraux exhaustifs `AgentManifest { version, entries }` cassés par le nouveau champ (ajoute `orchestrator: None` ou `..Default::default()`). Test round-trip : un `agents.json` legacy SANS le champ `orchestrator` se désérialise → `None` → `effective_orchestrator()` = plus ancien. ### T3 — Application : autorisation propose (cœur MCP) (dépend T1, T2) Dans `ProposeContext` (`crates/application/.../context_guard.rs`), branche globale (target = None) : ``` let manifest = contexts.load_manifest(project).await?; let d = manifest.orchestrator_designation(); if may_write_directly(requester, &GuardedResource::ProjectContext, &d) { let _lease = guard.acquire_write(requester, ProjectContext).await?; // sérialise fs.write(project_context_file, content) -> Written } else { file_proposal(...) -> Proposed { path } // inchangé } ``` Conséquence : quand l'appelant `idea_context_propose` (sans target) EST l'orchestrateur désigné, l'écriture devient DIRECTE ; sinon proposition ; l'humain écrit toujours direct. Tests : agent désigné → write direct ; agent non-désigné → proposition ; humain → direct. ### T4 — Application : défaut + succession (dépend T1) - `DeleteAgent::execute` (`lifecycle.rs`) : après filtrage des entrées, appelle `manifest.on_agent_deleted(removed)` avant `save_manifest` ; émets `DomainEvent::OrchestratorChanged` si `effective_orchestrator()` a changé. - Défaut (Create scratch + template) : pas de logique à ajouter (modèle paresseux), mais ajoute les tests qui figent « 1er agent créé = orchestrateur effectif » et l'émission de `OrchestratorChanged` si l'effectif passe de None→1er. - Tests succession : désigné explicite supprimé → retombe sur plus ancien ; non-désigné supprimé → repointage paresseux. ### T5 — Application : `SetOrchestrator` + ListAgents output (dépend T1) - Nouveau use case `SetOrchestrator { project, agent_id }` : `load_manifest` → `manifest.designate(agent_id)` (radio, écrase) → `save_manifest` → publie `OrchestratorChanged`. `NotFound`/erreur domaine si l'agent n'existe pas. - Étends `ListAgentsOutput` avec `orchestrator: Option<AgentId>` (= `effective_orchestrator()`). - Tests (store mock) : écrasement radio, agent inconnu → erreur, output porte l'effectif. Respecte l'archi hexagonale, SOLID, le style existant, aucun nouveau port (compose FileGuard / AgentContextStore / EventBus existants). Compile et teste les crates infra + application (`cargo test -p infrastructure -p application` ou vrais noms). Quand vert, réponds via `idea_reply` : fichiers touchés, signatures publiques finales de `SetOrchestrator` et du champ ajouté à `ListAgentsOutput` (accroche pour T6), et la sortie réelle des tests. Si rouge, donne l'erreur réelle.
|
||||
- **Prompt:** Tâche ciblée sur la branche courante `feature/orchestrator-designation`. Contexte: - Git a décidé de finir le chantier courant `orchestrator-designation` sur cette branche, sans switch/stash. - Main ne code pas; tu es chargé de la correction backend Rust. - Tests ciblés verts: - `cargo test -p infrastructure input --lib`: 35 passed. - `cargo test -p application --test orchestrator_service`: 45 passed. - Frontend vert: - `cd frontend && npx vitest run`: 41 files, 384 tests passed. - `cd frontend && npx tsc --noEmit`: OK. Problème à corriger: `cargo test --workspace` échoue à la compilation sur `crates/application/src/orchestrator/context_guard.rs`. Sortie réelle: ```text error[E0061]: this function takes 3 arguments but 2 arguments were supplied --> crates/application/src/orchestrator/context_guard.rs:449:17 | 449 | if !may_write_directly(who, &res) { | ^^^^^^^^^^^^^^^^^^----------- argument #3 of type `&OrchestratorDesignation` is missing error[E0560]: struct `ManifestEntry` has no field named `orchestrator` --> crates/application/src/orchestrator/context_guard.rs:616:21 | 616 | orchestrator: None, | ^^^^^^^^^^^^ `ManifestEntry` does not have this field error[E0063]: missing field `orchestrator` in initializer of `AgentManifest` --> crates/application/src/orchestrator/context_guard.rs:613:23 | 613 | manifest: AgentManifest { | ^^^^^^^^^^^^^ missing `orchestrator` ``` Attendu: - Adapter `context_guard.rs` au nouveau modèle `AgentManifest { version, entries, orchestrator }` et à `may_write_directly(resource, designation)` sans contourner l'architecture. - Supprimer l'import inutilisé si nécessaire. - Garder le domaine propriétaire des règles de désignation orchestrateur. - Ne touche pas aux fichiers `.ideai/*` runtime. Validation minimale à exécuter après correction: 1. `cargo test -p application --test orchestrator_service` 2. `cargo test --workspace` si raisonnable. Réponds avec les fichiers modifiés et les commandes/test outputs réels. Si cette tâche arrive via IdeA, utilise impérativement `idea_reply`.
|
||||
@ -23,4 +21,6 @@ objective: IMPLÉMENTATION (code de production uniquement — PAS les nouveaux t
|
||||
- **Prompt:** Implémente le Lot C backend/Tauri `workstate conversation summaries` sur la branche active `feature/workstate-conversation-summaries`. Respecte strictement le cadrage Architect ci-dessous. Objectif produit: - Enrichir `ProjectWorkState` avec `conversations: Vec<ConversationWorkSummary>` top-level, jointes côté frontend via `tickets[].conversationId`. - Résumer uniquement les conversations visibles via les tickets du work-state (conversation ids présents dans `AgentTicketState`). - Read-only, best-effort: aucune nouvelle persistance, aucune mutation/réparation de log/handoff, aucun blocage du live/busy/tickets si preview KO. Sources: - Primaire: `HandoffStore` (`.ideai/conversations/<id>/handoff.md`). - Fallback: `ConversationLog::last(conversation, 3)` uniquement, jamais lecture complète. - `ConversationRegistry` n'est pas source de vérité pour ce lot. Modèle application recommandé: - `ProjectWorkState { agents: Vec<AgentWorkState>, conversations: Vec<ConversationWorkSummary> }`. - `ConversationWorkSummary { conversation_id, status, objective_preview, summary_preview, summary_len, up_to, recent_turns }`. - `ConversationPreviewStatus`: `Ready | Missing | Partial | Unavailable`. - `ConversationTurnWorkPreview { role, source, at_ms, text_preview, text_len }`. Bornes: - `HANDOFF_PREVIEW_MAX_CHARS = 480` - `OBJECTIVE_PREVIEW_MAX_CHARS = 160` - `RECENT_TURNS_MAX = 3` - `TURN_PREVIEW_MAX_CHARS = 220` - Normalisation: trim + collapse whitespace + truncation char-safe. Algorithme: 1. Dédupliquer les `conversation_id` issus des tickets projetés. 2. Tenter `handoffs.load(conversation)`. 3. Si handoff présent et utilisable: `status = Ready`, `summaryPreview`, `summaryLen`, `objectivePreview`, `upTo`. 4. Si absent: `status = Missing`, tenter `log.last(conversation, 3)` et remplir `recentTurns` si possible. 5. Si handoff illisible/erreur mais log lisible: `status = Partial`, `summaryPreview = None`, `recentTurns`. 6. Si handoff et log échouent: `status = Unavailable`, champs preview vides. 7. Ne jamais transformer une erreur preview en `AppError` global. DTO Tauri: - Garder `get_project_work_state`. - Ajouter `ProjectWorkStateDto.conversations` camelCase. - Statuts en `ready/missing/partial/unavailable`. - Types TS seront faits par DevFrontend ensuite, mais les DTO Rust doivent être clairs. Tests attendus: - `crates/application/tests/workstate.rs`: agents/live/busy/tickets conservés si preview échoue; conversation ids dédupliqués; handoff présent ready; handoff absent + log présent missing avec recentTurns borné; handoff erreur + log lisible partial; handoff/log KO unavailable; previews tronquées et whitespace normalisé. - `crates/app-tauri/tests/dto_agents.rs`: sérialisation camelCase `conversations`, statuts, `recentTurns`. - Infrastructure seulement si nécessaire; ne crée pas d'adapter inutile. Commandes minimales: - `cargo fmt --all -- --check` - `cargo test -p application --test workstate` - `cargo test -p app-tauri --test dto_agents` - `cargo check -p app-tauri` - autres pertinentes si tu touches infra/domain. Contraintes: - Ne committe pas. - Ignore dirty runtime éventuel `.ideai/*`. - Retourne fichiers modifiés, signatures/types ajoutés, choix exacts si divergence, commandes exécutées et résultats réels.
|
||||
- **Response:** Lot C backend/Tauri `workstate conversation summaries` **implémenté et vert** sur `feature/workstate-conversation-summaries`. **Rien commité.** ## Fichiers modifiés (6 ; aucun infra/domain — ports HandoffStore/ConversationLog réutilisés) - `crates/application/src/workstate/mod.rs` — modèle + algo best-effort - `crates/application/src/lib.rs` — re-exports - `crates/app-tauri/src/dto.rs` — DTO camelCase - `crates/app-tauri/src/state.rs` — wiring composition root - `crates/application/tests/workstate.rs` — fakes + 9 tests - `crates/app-tauri/tests/dto_agents.rs` — +1 test + 2 littéraux existants mis à jour ## Signatures/types ajoutés **Application :** - `ProjectWorkState.conversations: Vec<ConversationWorkSummary>` (top-level) - `ConversationWorkSummary { conversation_id, status, objective_preview: Option<String>, summary_preview: Option<String>, summary_len: usize, up_to: Option<TurnId>, recent_turns: Vec<ConversationTurnWorkPreview> }` - `enum ConversationPreviewStatus { Ready, Missing, Partial, Unavailable }` - `ConversationTurnWorkPreview { role: TurnRole, source: TicketWorkSource, at_ms: u64, text_preview: String, text_len: usize }` - `trait ConversationLogProvider { fn conversation_log_for(&self, root: &ProjectPath) -> Option<Arc<dyn ConversationLog>> }` (nouveau, jumeau de `HandoffProvider`) - `GetProjectWorkState::with_conversation_sources(handoffs: Arc<dyn HandoffProvider>, logs: Arc<dyn ConversationLogProvider>)` (builder) - Constantes : `HANDOFF_PREVIEW_MAX_CHARS=480`, `OBJECTIVE_PREVIEW_MAX_CHARS=160`, `RECENT_TURNS_MAX=3`, `TURN_PREVIEW_MAX_CHARS=220` **DTO Tauri :** `ProjectWorkStateDto.conversations`, `ConversationWorkSummaryDto`, `ConversationPreviewStatusDto` (`ready/missing/partial/unavailable`), `ConversationTurnWorkPreviewDto`. **state.rs :** `AppConversationLogProvider` (matérialise `FsConversationLog::new(root)`), câblé via `.with_conversation_sources(AppHandoffProvider, AppConversationLogProvider)`. ## Choix exacts / divergences 1. **Réutilisé `HandoffProvider` existant (P7)** comme source primaire + ajouté le jumeau `ConversationLogProvider` pour le repli. Résolus **par project root** (`input.project.root`) car `GetProjectWorkState` est singleton AppState alors que les `Fs*` figent leur racine à la construction — strictement parallèle au pattern P7/P8b. 2. **Builder `with_conversation_sources`** plutôt que d'étendre `new(...)` : sources OFF par défaut, zéro régression sur call sites/tests existants (best-effort absent = aucune summary). 3. **« provider non câblé » = handoff Absent (chemin Missing), jamais une erreur.** `Unavailable` strictement = handoff **erreur** ET log erreur/absent. `Missing` reste Missing même si le log échoue (alors recentTurns vide). Conforme aux étapes 4/5/6 du cadrage. 4. **Ready** ne remplit pas `recent_turns` (le handoff EST le résumé) ; recentTurns peuplé seulement en Missing/Partial. Re-cap défensif à `RECENT_TURNS_MAX` en plus de `log.last(_, 3)`. 5. **Dédup** des `conversation_id` sur tous les tickets projetés (humains inclus), **ordre first-seen** pour refléter l'ordre du panneau. 6. **`TurnRole` domaine réutilisé tel quel dans le DTO** (sérialise déjà camelCase `prompt/response/toolActivity`), cohérent avec le réemploi de `AgentBusyState` ; `source` du tour réutilise `TicketWorkSourceDto` (`kind: human/agent`). 7. **Champs preview optionnels sérialisés en `null`** quand absents (pas omis) — prévisible pour le front, aligné sur le pattern `live` (divergence assumée vs `ConversationDetailsDto` qui omet). 8. **Normalisation factorisée** : helper `preview(text, max)` partagé (trim + collapse whitespace + troncature char-safe), `task_preview` (Lot B) refactoré dessus. ## Commandes exécutées + résultats réels - `cargo fmt --all -- --check` → **clean** (après `cargo fmt --all`). - `cargo test -p application --test workstate` → **21 passed; 0 failed** (12 existants + 9 nouveaux : has_no_conversations, dedups, ready, missing+bounded(3), partial, unavailable, preview_failure_preserves_agents/live/busy/tickets, previews_truncated+normalised, turn_text_truncated). - `cargo test -p app-tauri --test dto_agents` → **21 passed; 0 failed** (+1 `project_work_state_dto_serialises_conversations_camelcase`). - `cargo check -p app-tauri` → **clean** (domain/application/infrastructure/app-tauri compilent). Pas d'adapter infra créé (réemploi de `FsHandoffStore`/`FsConversationLog` existants). Dirty runtime `.ideai/*` ignoré. Reste : types TS (DevFrontend) + commit/merge (Git) + validation e2e.
|
||||
- **Prompt:** QA Lot C est verte mais signale un warning Rust non bloquant: `variant Absent is never constructed` dans `crates/application/tests/workstate.rs:173:5`, probablement un enum/fake de test `HandoffOutcome::Absent` inutilisé. Merci de nettoyer ce warning de façon minimale sans changer le comportement applicatif, puis relance au moins: - `cargo fmt --all -- --check` - `cargo test -p application --test workstate` Ne committe pas. Retourne le diff conceptuel, les fichiers modifiés, et les résultats réels.
|
||||
- **Response:** Warning nettoyé. **Rien commité.** ## Diff conceptuel La variante `HandoffOutcome::Absent` du fake de test n'était jamais construite : l'absence de handoff était déjà modélisée en **ne configurant pas** la conversation (le `load` du fake retombait sur l'arm `_ => Ok(None)`). Correctif minimal, zéro impact applicatif (uniquement le fixture de test) : - Suppression de la variante `Absent` (+ doc-comment expliquant que l'absence = conversation non configurée). - Le match de `FakeHandoffStore::load` passe de `_ => Ok(None)` à `None => Ok(None)` (désormais exhaustif sur `Present`/`Error`/`None`, plus de wildcard). Aucun test ne référençait `Absent` (le test `missing` repose sur une conversation non configurée), donc aucune logique de test ni d'assertion modifiée. ## Fichier modifié - `crates/application/tests/workstate.rs` (uniquement) ## Résultats réels - `cargo fmt --all -- --check` → **clean** (exit 0). - `cargo test -p application --test workstate` → **21 passed; 0 failed**, **aucun warning** (le `variant Absent is never constructed` a disparu).
|
||||
- **Response:** Warning nettoyé. **Rien commité.** ## Diff conceptuel La variante `HandoffOutcome::Absent` du fake de test n'était jamais construite : l'absence de handoff était déjà modélisée en **ne configurant pas** la conversation (le `load` du fake retombait sur l'arm `_ => Ok(None)`). Correctif minimal, zéro impact applicatif (uniquement le fixture de test) : - Suppression de la variante `Absent` (+ doc-comment expliquant que l'absence = conversation non configurée). - Le match de `FakeHandoffStore::load` passe de `_ => Ok(None)` à `None => Ok(None)` (désormais exhaustif sur `Present`/`Error`/`None`, plus de wildcard). Aucun test ne référençait `Absent` (le test `missing` repose sur une conversation non configurée), donc aucune logique de test ni d'assertion modifiée. ## Fichier modifié - `crates/application/tests/workstate.rs` (uniquement) ## Résultats réels - `cargo fmt --all -- --check` → **clean** (exit 0). - `cargo test -p application --test workstate` → **21 passed; 0 failed**, **aucun warning** (le `variant Absent is never constructed` a disparu).
|
||||
- **Prompt:** Implémente le backend/Tauri du Lot D `workstate controlled actions` sur la branche active `feature/workstate-controlled-actions`. Périmètre backend prioritaire: 1. Faire porter Attach par un contrat agent-level fiable, sans spawn: - Use case applicatif `AttachLiveAgent` si nécessaire. - Input: `project`, `agent_id`, `node_id`. - Output: `agent_id`, `node_id`, `session_id`, `kind: LiveSessionKind`. - Comportement: chercher la session live par agent dans PTY puis structured, rebind `node_id` sans créer de session, idempotent si déjà attaché à ce node, NOT_FOUND si agent non live. - Ne touche pas log/handoff/provider sessions. 2. Ajouter `StopLiveAgent` agent-level: - Input: `project`, `agent_id`. - Output: `agent_id`, `session_id`, `kind`. - Comportement: résoudre la session live par agent; si PTY déléguer à la primitive existante de fermeture/kill et retirer registry; si structured shutdown + retirer registry; NOT_FOUND si absent. - Ne supprime pas agent, tickets, conversation summary, handoff. - Publier/laisser publier les events existants d'exit si l'infra le fait déjà; ne pas inventer un bus si absent. 3. Tauri: - Commandes camelCase: - `attach_live_agent({ request: { projectId, agentId, nodeId } }) -> { agentId, nodeId, sessionId, kind }` - `stop_live_agent({ request: { projectId, agentId } }) -> { agentId, sessionId, kind }` - Réutiliser/enrichir DTO `LiveAgent` si cohérent, sinon DTO dédié. - Mapper INVALID ids et NOT_FOUND. Contraintes: - Ne pas lancer d'agent neuf, ne jamais appeler `launch_agent`. - Ne pas créer de cellule/layout côté backend pour Lot D. - Ne pas modifier le frontend. - Ne committe pas. - Ignore dirty runtime `.ideai/*`. Tests attendus: - Application Rust: attach PTY rebind sans changer `session_id`; attach structured; attach absent NOT_FOUND; attach même node idempotent; stop PTY retire session live; stop structured shutdown + retire; stop absent NOT_FOUND. - App-tauri tests DTO/command wiring: request/response camelCase, `kind`, INVALID/NOT_FOUND si conventions existantes le permettent. Commandes minimales: - `cargo fmt --all -- --check` - tests ciblés application pertinents (à choisir selon modules existants) - tests app-tauri pertinents - `cargo check -p app-tauri` Retourne fichiers modifiés, signatures/commandes ajoutées, résultats réels, et blocages/écarts éventuels.
|
||||
- **Response:** Lot D `workstate controlled actions` **implémenté et vert** sur `feature/workstate-controlled-actions`. **Rien commité.** ## Fichiers modifiés / créés - **NOUVEAU** `crates/application/src/workstate/actions.rs` — use cases `AttachLiveAgent` + `StopLiveAgent` - **NOUVEAU** `crates/application/tests/workstate_actions.rs` — 7 tests - `crates/application/src/workstate/mod.rs` — `mod actions;` + re-exports - `crates/application/src/lib.rs` — re-exports crate-root - `crates/app-tauri/src/dto.rs` — DTO request/response - `crates/app-tauri/src/commands.rs` — commande `attach_live_agent` réécrite + `stop_live_agent` - `crates/app-tauri/src/state.rs` — wiring composition root + champs AppState - `crates/app-tauri/src/lib.rs` — enregistrement `stop_live_agent` dans le handler - `crates/app-tauri/tests/dto_agents.rs` — 4 tests DTO ## Signatures ajoutées **Application :** - `AttachLiveAgent::new(live: Arc<LiveSessions>)` ; `execute(AttachLiveAgentInput { project, agent_id, node_id }) -> Result<AttachLiveAgentOutput { agent_id, node_id, session_id, kind: LiveSessionKind }, AppError>` (sync — rebind pur). - `StopLiveAgent::new(live: Arc<LiveSessions>, close: Arc<CloseTerminal>)` ; `async execute(StopLiveAgentInput { project, agent_id }) -> Result<StopLiveAgentOutput { agent_id, session_id, kind }, AppError>`. **Tauri (camelCase) :** - `attach_live_agent({ request: { projectId, agentId, nodeId } }) -> { agentId, nodeId, sessionId, kind }` - `stop_live_agent({ request: { projectId, agentId } }) -> { agentId, sessionId, kind }` - DTO dédiés `AttachLiveAgentResponseDto` / `StopLiveAgentRequestDto` / `StopLiveAgentResponseDto` ; `kind` réutilise `LiveWorkSessionKindDto` (`pty`/`structured`). ## Choix exacts / écarts 1. **Attach = use case dédié** (l'ancienne commande ne gérait que le PTY via `rebind_agent_node` direct) : désormais PTY **puis** structured, idempotent (rebind sur le même node = no-op renvoyant la même session), NOT_FOUND sinon. Zéro spawn, ne touche pas log/handoff/provider sessions. 2. **Stop polymorphe** : PTY → délègue à la primitive existante `CloseTerminal` (remove registry + kill) ; structured → `remove` puis `shutdown()`. Ne supprime ni agent, ni tickets, ni summary, ni handoff. Aucun bus inventé — l'event d'exit est laissé à l'infra existante (pump PTY / backend session). 3. **Ordre PTY-first** pour les deux (conforme au libellé cadrage « PTY puis structured » et à `LiveSessions::session_id_for_agent`) ; non ambigu par l'invariant « 1 session vivante/agent ». 4. **`project` accepté mais non utilisé pour la logique** (registres globaux par agent) — gardé pour la frontière/symétrie, pas de check manifest (le cadrage ne l'exige pas). 5. **⚠️ ÉCART contrat frontend** : `attach_live_agent` renvoyait `LiveAgentListDto` (tableau) ; il renvoie désormais l'objet `{ agentId, nodeId, sessionId, kind }` imposé par le cadrage. Je n'ai **pas** touché le frontend (consigne respectée) ⇒ **mismatch transitoire FE/BE à réconcilier par DevFrontend dans son Lot D**. À flaguer. 6. Pas de cellule/layout créée côté backend (hors périmètre Lot D). ## Résultats réels - `cargo fmt --all -- --check` → **clean** (après `cargo fmt --all`). - `cargo test -p application --test workstate_actions` → **7 passed; 0 failed** (attach PTY rebind sans changer session_id ; attach structured ; attach absent NOT_FOUND ; attach même node idempotent ; stop PTY kill+remove ; stop structured shutdown+remove ; stop absent NOT_FOUND). - `cargo test -p application --test workstate` → **21 passed** (non-régression Lots A/B/C). - `cargo test -p app-tauri --test dto_agents` → **25 passed** (+4 nouveaux : request/response camelCase + `kind` pour attach & stop ; pas de fuite snake_case). - `cargo test -p app-tauri --test list_live_agents_r0b` → **5 passed** (non-régression de la liste live). - `cargo check -p app-tauri` → **clean**. ## Blocages Aucun blocage technique. Seul point d'attention : l'écart de contrat (#5) qui requiert l'adaptation frontend (DevFrontend) + commit/merge (Git) + validation e2e. Dirty runtime `.ideai/*` ignoré.
|
||||
@ -76,3 +76,5 @@
|
||||
{"id":"b9900b46-41ff-4276-a225-991520cf0923","conversation":"d5068bc8-786e-0428-1d14-83b93961a15e","atMs":1781976051405,"source":{"kind":"agent","agentId":"73c853d1-c0fd-463b-ad17-1d24fefa371f"},"role":"response","text":"Lot C backend/Tauri `workstate conversation summaries` **implémenté et vert** sur `feature/workstate-conversation-summaries`. **Rien commité.**\n\n## Fichiers modifiés (6 ; aucun infra/domain — ports HandoffStore/ConversationLog réutilisés)\n- `crates/application/src/workstate/mod.rs` — modèle + algo best-effort\n- `crates/application/src/lib.rs` — re-exports\n- `crates/app-tauri/src/dto.rs` — DTO camelCase\n- `crates/app-tauri/src/state.rs` — wiring composition root\n- `crates/application/tests/workstate.rs` — fakes + 9 tests\n- `crates/app-tauri/tests/dto_agents.rs` — +1 test + 2 littéraux existants mis à jour\n\n## Signatures/types ajoutés\n**Application :**\n- `ProjectWorkState.conversations: Vec<ConversationWorkSummary>` (top-level)\n- `ConversationWorkSummary { conversation_id, status, objective_preview: Option<String>, summary_preview: Option<String>, summary_len: usize, up_to: Option<TurnId>, recent_turns: Vec<ConversationTurnWorkPreview> }`\n- `enum ConversationPreviewStatus { Ready, Missing, Partial, Unavailable }`\n- `ConversationTurnWorkPreview { role: TurnRole, source: TicketWorkSource, at_ms: u64, text_preview: String, text_len: usize }`\n- `trait ConversationLogProvider { fn conversation_log_for(&self, root: &ProjectPath) -> Option<Arc<dyn ConversationLog>> }` (nouveau, jumeau de `HandoffProvider`)\n- `GetProjectWorkState::with_conversation_sources(handoffs: Arc<dyn HandoffProvider>, logs: Arc<dyn ConversationLogProvider>)` (builder)\n- Constantes : `HANDOFF_PREVIEW_MAX_CHARS=480`, `OBJECTIVE_PREVIEW_MAX_CHARS=160`, `RECENT_TURNS_MAX=3`, `TURN_PREVIEW_MAX_CHARS=220`\n\n**DTO Tauri :** `ProjectWorkStateDto.conversations`, `ConversationWorkSummaryDto`, `ConversationPreviewStatusDto` (`ready/missing/partial/unavailable`), `ConversationTurnWorkPreviewDto`. **state.rs :** `AppConversationLogProvider` (matérialise `FsConversationLog::new(root)`), câblé via `.with_conversation_sources(AppHandoffProvider, AppConversationLogProvider)`.\n\n## Choix exacts / divergences\n1. **Réutilisé `HandoffProvider` existant (P7)** comme source primaire + ajouté le jumeau `ConversationLogProvider` pour le repli. Résolus **par project root** (`input.project.root`) car `GetProjectWorkState` est singleton AppState alors que les `Fs*` figent leur racine à la construction — strictement parallèle au pattern P7/P8b.\n2. **Builder `with_conversation_sources`** plutôt que d'étendre `new(...)` : sources OFF par défaut, zéro régression sur call sites/tests existants (best-effort absent = aucune summary).\n3. **« provider non câblé » = handoff Absent (chemin Missing), jamais une erreur.** `Unavailable` strictement = handoff **erreur** ET log erreur/absent. `Missing` reste Missing même si le log échoue (alors recentTurns vide). Conforme aux étapes 4/5/6 du cadrage.\n4. **Ready** ne remplit pas `recent_turns` (le handoff EST le résumé) ; recentTurns peuplé seulement en Missing/Partial. Re-cap défensif à `RECENT_TURNS_MAX` en plus de `log.last(_, 3)`.\n5. **Dédup** des `conversation_id` sur tous les tickets projetés (humains inclus), **ordre first-seen** pour refléter l'ordre du panneau.\n6. **`TurnRole` domaine réutilisé tel quel dans le DTO** (sérialise déjà camelCase `prompt/response/toolActivity`), cohérent avec le réemploi de `AgentBusyState` ; `source` du tour réutilise `TicketWorkSourceDto` (`kind: human/agent`).\n7. **Champs preview optionnels sérialisés en `null`** quand absents (pas omis) — prévisible pour le front, aligné sur le pattern `live` (divergence assumée vs `ConversationDetailsDto` qui omet).\n8. **Normalisation factorisée** : helper `preview(text, max)` partagé (trim + collapse whitespace + troncature char-safe), `task_preview` (Lot B) refactoré dessus.\n\n## Commandes exécutées + résultats réels\n- `cargo fmt --all -- --check` → **clean** (après `cargo fmt --all`).\n- `cargo test -p application --test workstate` → **21 passed; 0 failed** (12 existants + 9 nouveaux : has_no_conversations, dedups, ready, missing+bounded(3), partial, unavailable, preview_failure_preserves_agents/live/busy/tickets, previews_truncated+normalised, turn_text_truncated).\n- `cargo test -p app-tauri --test dto_agents` → **21 passed; 0 failed** (+1 `project_work_state_dto_serialises_conversations_camelcase`).\n- `cargo check -p app-tauri` → **clean** (domain/application/infrastructure/app-tauri compilent).\n\nPas d'adapter infra créé (réemploi de `FsHandoffStore`/`FsConversationLog` existants). Dirty runtime `.ideai/*` ignoré. Reste : types TS (DevFrontend) + commit/merge (Git) + validation e2e."}
|
||||
{"id":"0834676a-dc01-4cdf-97a4-37cff81bf3d2","conversation":"d5068bc8-786e-0428-1d14-83b93961a15e","atMs":1781976363926,"source":{"kind":"agent","agentId":"a6ced819-b893-4213-b003-9e9dc79b9641"},"role":"prompt","text":"QA Lot C est verte mais signale un warning Rust non bloquant: `variant Absent is never constructed` dans `crates/application/tests/workstate.rs:173:5`, probablement un enum/fake de test `HandoffOutcome::Absent` inutilisé.\n\nMerci de nettoyer ce warning de façon minimale sans changer le comportement applicatif, puis relance au moins:\n- `cargo fmt --all -- --check`\n- `cargo test -p application --test workstate`\n\nNe committe pas. Retourne le diff conceptuel, les fichiers modifiés, et les résultats réels."}
|
||||
{"id":"8e30c171-2d32-48ab-9971-c6d23a010052","conversation":"d5068bc8-786e-0428-1d14-83b93961a15e","atMs":1781976412762,"source":{"kind":"agent","agentId":"73c853d1-c0fd-463b-ad17-1d24fefa371f"},"role":"response","text":"Warning nettoyé. **Rien commité.**\n\n## Diff conceptuel\nLa variante `HandoffOutcome::Absent` du fake de test n'était jamais construite : l'absence de handoff était déjà modélisée en **ne configurant pas** la conversation (le `load` du fake retombait sur l'arm `_ => Ok(None)`). Correctif minimal, zéro impact applicatif (uniquement le fixture de test) :\n- Suppression de la variante `Absent` (+ doc-comment expliquant que l'absence = conversation non configurée).\n- Le match de `FakeHandoffStore::load` passe de `_ => Ok(None)` à `None => Ok(None)` (désormais exhaustif sur `Present`/`Error`/`None`, plus de wildcard).\nAucun test ne référençait `Absent` (le test `missing` repose sur une conversation non configurée), donc aucune logique de test ni d'assertion modifiée.\n\n## Fichier modifié\n- `crates/application/tests/workstate.rs` (uniquement)\n\n## Résultats réels\n- `cargo fmt --all -- --check` → **clean** (exit 0).\n- `cargo test -p application --test workstate` → **21 passed; 0 failed**, **aucun warning** (le `variant Absent is never constructed` a disparu)."}
|
||||
{"id":"6092e15f-2659-4986-960a-0a1b3bc705ff","conversation":"d5068bc8-786e-0428-1d14-83b93961a15e","atMs":1781976761247,"source":{"kind":"agent","agentId":"a6ced819-b893-4213-b003-9e9dc79b9641"},"role":"prompt","text":"Implémente le backend/Tauri du Lot D `workstate controlled actions` sur la branche active `feature/workstate-controlled-actions`.\n\nPérimètre backend prioritaire:\n1. Faire porter Attach par un contrat agent-level fiable, sans spawn:\n - Use case applicatif `AttachLiveAgent` si nécessaire.\n - Input: `project`, `agent_id`, `node_id`.\n - Output: `agent_id`, `node_id`, `session_id`, `kind: LiveSessionKind`.\n - Comportement: chercher la session live par agent dans PTY puis structured, rebind `node_id` sans créer de session, idempotent si déjà attaché à ce node, NOT_FOUND si agent non live.\n - Ne touche pas log/handoff/provider sessions.\n\n2. Ajouter `StopLiveAgent` agent-level:\n - Input: `project`, `agent_id`.\n - Output: `agent_id`, `session_id`, `kind`.\n - Comportement: résoudre la session live par agent; si PTY déléguer à la primitive existante de fermeture/kill et retirer registry; si structured shutdown + retirer registry; NOT_FOUND si absent.\n - Ne supprime pas agent, tickets, conversation summary, handoff.\n - Publier/laisser publier les events existants d'exit si l'infra le fait déjà; ne pas inventer un bus si absent.\n\n3. Tauri:\n - Commandes camelCase:\n - `attach_live_agent({ request: { projectId, agentId, nodeId } }) -> { agentId, nodeId, sessionId, kind }`\n - `stop_live_agent({ request: { projectId, agentId } }) -> { agentId, sessionId, kind }`\n - Réutiliser/enrichir DTO `LiveAgent` si cohérent, sinon DTO dédié.\n - Mapper INVALID ids et NOT_FOUND.\n\nContraintes:\n- Ne pas lancer d'agent neuf, ne jamais appeler `launch_agent`.\n- Ne pas créer de cellule/layout côté backend pour Lot D.\n- Ne pas modifier le frontend.\n- Ne committe pas.\n- Ignore dirty runtime `.ideai/*`.\n\nTests attendus:\n- Application Rust: attach PTY rebind sans changer `session_id`; attach structured; attach absent NOT_FOUND; attach même node idempotent; stop PTY retire session live; stop structured shutdown + retire; stop absent NOT_FOUND.\n- App-tauri tests DTO/command wiring: request/response camelCase, `kind`, INVALID/NOT_FOUND si conventions existantes le permettent.\n\nCommandes minimales:\n- `cargo fmt --all -- --check`\n- tests ciblés application pertinents (à choisir selon modules existants)\n- tests app-tauri pertinents\n- `cargo check -p app-tauri`\n\nRetourne fichiers modifiés, signatures/commandes ajoutées, résultats réels, et blocages/écarts éventuels."}
|
||||
{"id":"ec87367b-50e1-44fb-b4dd-88c284f02cae","conversation":"d5068bc8-786e-0428-1d14-83b93961a15e","atMs":1781977281733,"source":{"kind":"agent","agentId":"73c853d1-c0fd-463b-ad17-1d24fefa371f"},"role":"response","text":"Lot D `workstate controlled actions` **implémenté et vert** sur `feature/workstate-controlled-actions`. **Rien commité.**\n\n## Fichiers modifiés / créés\n- **NOUVEAU** `crates/application/src/workstate/actions.rs` — use cases `AttachLiveAgent` + `StopLiveAgent`\n- **NOUVEAU** `crates/application/tests/workstate_actions.rs` — 7 tests\n- `crates/application/src/workstate/mod.rs` — `mod actions;` + re-exports\n- `crates/application/src/lib.rs` — re-exports crate-root\n- `crates/app-tauri/src/dto.rs` — DTO request/response\n- `crates/app-tauri/src/commands.rs` — commande `attach_live_agent` réécrite + `stop_live_agent`\n- `crates/app-tauri/src/state.rs` — wiring composition root + champs AppState\n- `crates/app-tauri/src/lib.rs` — enregistrement `stop_live_agent` dans le handler\n- `crates/app-tauri/tests/dto_agents.rs` — 4 tests DTO\n\n## Signatures ajoutées\n**Application :**\n- `AttachLiveAgent::new(live: Arc<LiveSessions>)` ; `execute(AttachLiveAgentInput { project, agent_id, node_id }) -> Result<AttachLiveAgentOutput { agent_id, node_id, session_id, kind: LiveSessionKind }, AppError>` (sync — rebind pur).\n- `StopLiveAgent::new(live: Arc<LiveSessions>, close: Arc<CloseTerminal>)` ; `async execute(StopLiveAgentInput { project, agent_id }) -> Result<StopLiveAgentOutput { agent_id, session_id, kind }, AppError>`.\n\n**Tauri (camelCase) :**\n- `attach_live_agent({ request: { projectId, agentId, nodeId } }) -> { agentId, nodeId, sessionId, kind }`\n- `stop_live_agent({ request: { projectId, agentId } }) -> { agentId, sessionId, kind }`\n- DTO dédiés `AttachLiveAgentResponseDto` / `StopLiveAgentRequestDto` / `StopLiveAgentResponseDto` ; `kind` réutilise `LiveWorkSessionKindDto` (`pty`/`structured`).\n\n## Choix exacts / écarts\n1. **Attach = use case dédié** (l'ancienne commande ne gérait que le PTY via `rebind_agent_node` direct) : désormais PTY **puis** structured, idempotent (rebind sur le même node = no-op renvoyant la même session), NOT_FOUND sinon. Zéro spawn, ne touche pas log/handoff/provider sessions.\n2. **Stop polymorphe** : PTY → délègue à la primitive existante `CloseTerminal` (remove registry + kill) ; structured → `remove` puis `shutdown()`. Ne supprime ni agent, ni tickets, ni summary, ni handoff. Aucun bus inventé — l'event d'exit est laissé à l'infra existante (pump PTY / backend session).\n3. **Ordre PTY-first** pour les deux (conforme au libellé cadrage « PTY puis structured » et à `LiveSessions::session_id_for_agent`) ; non ambigu par l'invariant « 1 session vivante/agent ».\n4. **`project` accepté mais non utilisé pour la logique** (registres globaux par agent) — gardé pour la frontière/symétrie, pas de check manifest (le cadrage ne l'exige pas).\n5. **⚠️ ÉCART contrat frontend** : `attach_live_agent` renvoyait `LiveAgentListDto` (tableau) ; il renvoie désormais l'objet `{ agentId, nodeId, sessionId, kind }` imposé par le cadrage. Je n'ai **pas** touché le frontend (consigne respectée) ⇒ **mismatch transitoire FE/BE à réconcilier par DevFrontend dans son Lot D**. À flaguer.\n6. Pas de cellule/layout créée côté backend (hors périmètre Lot D).\n\n## Résultats réels\n- `cargo fmt --all -- --check` → **clean** (après `cargo fmt --all`).\n- `cargo test -p application --test workstate_actions` → **7 passed; 0 failed** (attach PTY rebind sans changer session_id ; attach structured ; attach absent NOT_FOUND ; attach même node idempotent ; stop PTY kill+remove ; stop structured shutdown+remove ; stop absent NOT_FOUND).\n- `cargo test -p application --test workstate` → **21 passed** (non-régression Lots A/B/C).\n- `cargo test -p app-tauri --test dto_agents` → **25 passed** (+4 nouveaux : request/response camelCase + `kind` pour attach & stop ; pas de fuite snake_case).\n- `cargo test -p app-tauri --test list_live_agents_r0b` → **5 passed** (non-régression de la liste live).\n- `cargo check -p app-tauri` → **clean**.\n\n## Blocages\nAucun blocage technique. Seul point d'attention : l'écart de contrat (#5) qui requiert l'adaptation frontend (DevFrontend) + commit/merge (Git) + validation e2e. Dirty runtime `.ideai/*` ignoré."}
|
||||
|
||||
Reference in New Issue
Block a user