best-practice-extractor

Best Practice Extractor

Extract best practices and successful strategies from conversation history.

Purpose

Analyze conversations to identify:

• Explicit best practice recommendations
• Successful approaches worth repeating
• Patterns worth formalizing
• Lessons learned

Detection Keywords

BEST_PRACTICE_KEYWORDS = [
    "best practice", "should always", "always use", "never use",
    "recommended", "prefer", "important to", "make sure to",
    "don't forget", "remember to", "key is to", "the pattern is",
    "rule of thumb", "guideline", "standard", "convention"
]

Analysis Method

1. Explicit Practice Detection

for msg in messages:
    for kw in BEST_PRACTICE_KEYWORDS:
        if kw in msg.lower():
            # Extract context around keyword
            practices.append({
                'practice': extract_practice(msg, kw),
                'context': msg[:500],
                'session_id': session_id
            })

2. Success Pattern Detection

SUCCESS_INDICATORS = [
    "works great", "perfect", "exactly what", "this is the way",
    "much better", "finally works", "correct approach"
]

for msg in messages:
    if any(ind in msg.lower() for ind in SUCCESS_INDICATORS):
        # Extract what was successful
        successes.append(analyze_success_context(msg))

3. Categorization

CATEGORIES = {
    "code": ["function", "class", "method", "code", "implementation"],
    "workflow": ["workflow", "process", "approach", "steps"],
    "documentation": ["document", "readme", "comment", "docstring"],
    "testing": ["test", "validate", "verify", "check"],
    "architecture": ["pattern", "design", "structure", "organization"]
}

Output Format

{
  "best_practices": [
    {
      "category": "code",
      "practice": "Use @staticmethod for state-free operations in ras-commander",
      "rationale": "Cleaner API, no instantiation needed, consistent with library pattern",
      "implementation": "Add @staticmethod decorator, call directly on class",
      "session_ids": ["abc123"],
      "evidence": ["The static class pattern provides..."]
    }
  ]
}