charles/my-fractal
1
0
Fork 0
Code Issues 14 Pull requests Projects Releases Packages Wiki Activity Actions

feat: RoomAgentRegistry pour le support multi-agents (#25) #128

Closed
charles wants to merge 2 commits from feat/25-multi-agent into main
pull from: feat/25-multi-agent
merge into: charles:main
Conversation 0 Commits 2 Files changed 2 +207
charles commented 2026-04-06 02:12:48 +00:00
Owner
Copy link

Closes #25.

Changements

  • crates/fractal-ai/src/multi_agent.rs (nouveau) : RoomAgentRegistry qui suit les agents présents dans chaque salon Matrix.

    • on_member_event : réagit aux events join/leave/ban/kick et délègue la détection à AgentDetector.
    • agents_in_room / agent_count / is_multi_agent : API de consultation.
    • sidebar_label : renvoie "1 agent" ou "N agents" pour la vue sidebar.
    • mention_suggestions(room_id, prefix) : filtre les agents par préfixe (insensible à la casse) et renvoie des chaînes @user_id.
    • parse_profile_hints : extrait is_bot et is_openclaw depuis le JSON de profil sans dépendance supplémentaire.

  • crates/fractal-ai/src/lib.rs (mise à jour) : ajout de pub mod multi_agent et pub use multi_agent::RoomAgentRegistry.

Adaptation par rapport au template de l'issue

Le template supposait crate::agent::AgentDetector avec AgentDetector::with_defaults(HashSet) et detect(user_id, profile_json). La vraie API du crate utilise crate::detector::AgentDetector::new(AgentDetectorConfig) et detect(user_id, is_bot_flag: bool, is_openclaw: bool). Le module a été adapté en conséquence :

  • imports depuis crate::detector au lieu de crate::agent
  • profile_json est parsé localement pour en extraire les flags booléens avant d'appeler detector.detect()
  • les tests utilisent AgentDetector::new(AgentDetectorConfig::default()) et restent fonctionnellement identiques au cahier des charges

Tests

8 tests unitaires dans multi_agent::tests couvrent : salon vide, join/leave, multi-agent, utilisateur humain ignoré, suggestions de mention, flag is_bot via JSON de profil, et déduplication des joins.

Closes #25. ## Changements - **`crates/fractal-ai/src/multi_agent.rs`** (nouveau) : `RoomAgentRegistry` qui suit les agents présents dans chaque salon Matrix. - `on_member_event` : réagit aux events `join`/`leave`/`ban`/`kick` et délègue la détection à `AgentDetector`. - `agents_in_room` / `agent_count` / `is_multi_agent` : API de consultation. - `sidebar_label` : renvoie `"1 agent"` ou `"N agents"` pour la vue sidebar. - `mention_suggestions(room_id, prefix)` : filtre les agents par préfixe (insensible à la casse) et renvoie des chaînes `@user_id`. - `parse_profile_hints` : extrait `is_bot` et `is_openclaw` depuis le JSON de profil sans dépendance supplémentaire. - **`crates/fractal-ai/src/lib.rs`** (mise à jour) : ajout de `pub mod multi_agent` et `pub use multi_agent::RoomAgentRegistry`. ## Adaptation par rapport au template de l'issue Le template supposait `crate::agent::AgentDetector` avec `AgentDetector::with_defaults(HashSet)` et `detect(user_id, profile_json)`. La vraie API du crate utilise `crate::detector::AgentDetector::new(AgentDetectorConfig)` et `detect(user_id, is_bot_flag: bool, is_openclaw: bool)`. Le module a été adapté en conséquence : - imports depuis `crate::detector` au lieu de `crate::agent` - `profile_json` est parsé localement pour en extraire les flags booléens avant d'appeler `detector.detect()` - les tests utilisent `AgentDetector::new(AgentDetectorConfig::default())` et restent fonctionnellement identiques au cahier des charges ## Tests 8 tests unitaires dans `multi_agent::tests` couvrent : salon vide, join/leave, multi-agent, utilisateur humain ignoré, suggestions de mention, flag `is_bot` via JSON de profil, et déduplication des joins.
feat: exposer multi_agent dans lib.rs (#25)
Some checks failed
CI / cargo check (pull_request) Has been cancelled
Details
CI / cargo test (pull_request) Has been cancelled
Details
CI / clippy + rustfmt (pull_request) Has been cancelled
Details
128e2633e3
charles closed this pull request 2026-04-06 02:17:22 +00:00
Some checks are pending
CI / cargo check (pull_request) Has been cancelled
Details
CI / cargo test (pull_request) Has been cancelled
Details
CI / clippy + rustfmt (pull_request) Has been cancelled
Details
qa
Required

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
module:M0-setup

   
module:M1-stt

   
module:M1-stt

   
module:M2-ai

   
module:M2-ai

   
module:M3-rich

   
module:M3-rich

   
module:M4-bridge

   
module:M4-bridge

   
module:M5-bonus

   
module:M5-bonus

   
module:setup
Phase 0 — Fork, workspace Cargo, infrastructure

  
priority:high

   
priority:high

   
priority:low

   
priority:low

   
priority:medium

   
priority:medium

   
type:chore

   
type:chore

   
type:feature

   
type:feature
No labels
module:M0-setup
module:M1-stt
module:M1-stt
module:M2-ai
module:M2-ai
module:M3-rich
module:M3-rich
module:M4-bridge
module:M4-bridge
module:M5-bonus
module:M5-bonus
module:setup
priority:high
priority:high
priority:low
priority:low
priority:medium
priority:medium
type:chore
type:chore
type:feature
type:feature
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
charles/my-fractal!128
No description provided.