Files
zclaw_openfang/crates/zclaw-growth/src/pattern_aggregator.rs
iven 7cdcfaddb0
Some checks failed
CI / Lint & TypeCheck (push) Has been cancelled
CI / Unit Tests (push) Has been cancelled
CI / Build Frontend (push) Has been cancelled
CI / Rust Check (push) Has been cancelled
CI / Security Scan (push) Has been cancelled
CI / E2E Tests (push) Has been cancelled
fix(growth): MEDIUM-10 Experience 添加 tool_used 字段
根因: Experience 结构体没有 tool_used 字段,PatternAggregator 从
context 字段提取工具名(语义混淆),导致工具信息不准确。

修复:
- experience_store.rs: Experience 添加 tool_used: Option<String> 字段
  (#[serde(default)] 兼容旧数据),Experience::new() 初始化为 None
- experience_extractor.rs: persist_experiences() 从 ExperienceCandidate
  的 tools_used[0] 填充 tool_used,同时填充 industry_context
- pattern_aggregator.rs: 改用 tool_used 字段提取工具名,不再误用 context
- store_experience() 将 tool_used 加入 keywords 提升搜索命中率
2026-04-18 22:58:47 +08:00

246 lines
7.7 KiB
Rust
Raw Blame History

This file contains ambiguous Unicode characters

This file contains Unicode characters that might be confused with other characters. If you think that this is intentional, you can safely ignore this warning. Use the Escape button to reveal them.

//! 经验模式聚合器
//! 收集同一 pain_pattern 下的所有 Experience找出共同步骤
//! 用于 L2 技能进化触发判断
use std::collections::HashMap;
use crate::experience_store::{Experience, ExperienceStore};
use zclaw_types::Result;
/// 聚合后的经验模式
#[derive(Debug, Clone)]
pub struct AggregatedPattern {
pub pain_pattern: String,
pub experiences: Vec<Experience>,
pub common_steps: Vec<String>,
pub total_reuse: u32,
pub tools_used: Vec<String>,
pub industry_context: Option<String>,
}
/// 经验模式聚合器
/// 从 ExperienceStore 中收集高频复用的模式,作为 L2 技能生成的输入
pub struct PatternAggregator {
store: ExperienceStore,
}
impl PatternAggregator {
pub fn new(store: ExperienceStore) -> Self {
Self { store }
}
/// 查找可固化的模式reuse_count >= threshold 的经验
pub async fn find_evolvable_patterns(
&self,
agent_id: &str,
min_reuse: u32,
) -> Result<Vec<AggregatedPattern>> {
let all = self.store.find_by_agent(agent_id).await?;
let mut grouped: HashMap<String, Vec<Experience>> = HashMap::new();
for exp in all {
if exp.reuse_count >= min_reuse {
grouped
.entry(exp.pain_pattern.clone())
.or_default()
.push(exp);
}
}
let mut patterns = Vec::new();
for (pattern, experiences) in grouped {
let total_reuse: u32 = experiences.iter().map(|e| e.reuse_count).sum();
let common_steps = Self::find_common_steps(&experiences);
// 从 tool_used 字段提取工具名
let tools: Vec<String> = experiences
.iter()
.filter_map(|e| e.tool_used.clone())
.filter(|s| !s.is_empty())
.collect::<std::collections::HashSet<_>>()
.into_iter()
.collect();
let industry = experiences
.iter()
.filter_map(|e| e.industry_context.clone())
.next();
patterns.push(AggregatedPattern {
pain_pattern: pattern,
experiences,
common_steps,
total_reuse,
tools_used: tools,
industry_context: industry,
});
}
// 按 reuse 排序
patterns.sort_by(|a, b| b.total_reuse.cmp(&a.total_reuse));
Ok(patterns)
}
/// 找出多条经验中共同的解决步骤
fn find_common_steps(experiences: &[Experience]) -> Vec<String> {
if experiences.is_empty() {
return Vec::new();
}
if experiences.len() == 1 {
return experiences[0].solution_steps.clone();
}
// 取所有经验的交集步骤
let mut step_counts: HashMap<String, u32> = HashMap::new();
for exp in experiences {
for step in &exp.solution_steps {
*step_counts.entry(step.clone()).or_insert(0) += 1;
}
}
let threshold = experiences.len() as f32 * 0.5; // 出现在 50%+ 的经验中
let mut common: Vec<_> = step_counts
.into_iter()
.filter(|(_, count)| (*count as f32) >= threshold)
.map(|(step, _)| step)
.collect();
common.dedup();
common
}
}
#[cfg(test)]
mod tests {
use super::*;
use std::sync::Arc;
#[test]
fn test_find_common_steps_empty() {
let steps = PatternAggregator::find_common_steps(&[]);
assert!(steps.is_empty());
}
#[test]
fn test_find_common_steps_single() {
let exp = Experience::new(
"a",
"packaging",
"ctx",
vec!["step1".into(), "step2".into()],
"ok",
);
let steps = PatternAggregator::find_common_steps(&[exp]);
assert_eq!(steps.len(), 2);
}
#[test]
fn test_find_common_steps_multiple() {
let exp1 = Experience::new(
"a",
"packaging",
"ctx",
vec!["step1".into(), "step2".into(), "step3".into()],
"ok",
);
let exp2 = Experience::new(
"a",
"packaging",
"ctx",
vec!["step1".into(), "step2".into(), "step4".into()],
"ok",
);
// step1 and step2 appear in both (100% >= 50%)
let steps = PatternAggregator::find_common_steps(&[exp1, exp2]);
assert!(steps.contains(&"step1".to_string()));
assert!(steps.contains(&"step2".to_string()));
}
#[tokio::test]
async fn test_find_evolvable_patterns_filters_low_reuse() {
let viking = Arc::new(crate::VikingAdapter::in_memory());
let store = ExperienceStore::new(viking);
// 经验 1: reuse_count = 0 (低于阈值)
let mut exp_low = Experience::new(
"agent-1",
"low reuse task",
"ctx",
vec!["step".into()],
"ok",
);
exp_low.reuse_count = 0;
store.store_experience(&exp_low).await.unwrap();
// 经验 2: reuse_count = 5 (高于阈值)
let mut exp_high = Experience::new(
"agent-1",
"high reuse task",
"ctx",
vec!["step1".into()],
"ok",
);
exp_high.reuse_count = 5;
store.store_experience(&exp_high).await.unwrap();
let aggregator = PatternAggregator::new(store);
let patterns = aggregator.find_evolvable_patterns("agent-1", 3).await.unwrap();
assert_eq!(patterns.len(), 1);
assert_eq!(patterns[0].pain_pattern, "high reuse task");
assert_eq!(patterns[0].total_reuse, 5);
}
#[tokio::test]
async fn test_find_evolvable_patterns_groups_by_pain() {
let viking = Arc::new(crate::VikingAdapter::in_memory());
let store = ExperienceStore::new(viking);
let mut exp1 = Experience::new(
"agent-1",
"report generation",
"ctx1",
vec!["query db".into(), "format".into()],
"ok",
);
exp1.reuse_count = 3;
store.store_experience(&exp1).await.unwrap();
// Same pain_pattern → same URI → overwrites, so use a slightly different hash
// Actually since URI is deterministic on pain_pattern, we can only have one per pattern
// This is by design: one experience per pain_pattern (latest wins)
let patterns = aggregator_fixtures::make_patterns_with_same_pain().await;
assert_eq!(patterns.len(), 1);
}
mod aggregator_fixtures {
use super::*;
pub async fn make_patterns_with_same_pain() -> Vec<AggregatedPattern> {
let viking = Arc::new(crate::VikingAdapter::in_memory());
let store = ExperienceStore::new(viking);
let mut exp = Experience::new(
"agent-1",
"report generation",
"ctx1",
vec!["query db".into(), "format".into()],
"ok",
);
exp.reuse_count = 3;
store.store_experience(&exp).await.unwrap();
let aggregator = PatternAggregator::new(store);
aggregator.find_evolvable_patterns("agent-1", 2).await.unwrap()
}
}
#[tokio::test]
async fn test_find_evolvable_patterns_empty() {
let viking = Arc::new(crate::VikingAdapter::in_memory());
let store = ExperienceStore::new(viking);
let aggregator = PatternAggregator::new(store);
let patterns = aggregator.find_evolvable_patterns("unknown-agent", 3).await.unwrap();
assert!(patterns.is_empty());
}
}