feat: Add LLM-enhanced PR analysis for intelligent cross-branch coordination #1
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Implement LLMAnalyzer class with configurable model support - Add code change analysis for breaking change detection - Include commit message quality evaluation - Business impact assessment capabilities - Coordination planning for cross-team dependencies - Graceful fallback when LLM unavailable This enables smart/deep intelligence levels for git analysis tools.
- Add intelligence_level parameter (fast/smart/deep) - Integrate LLM breaking change detection for smart/deep modes - Enhanced coordination requirements analysis - Context-aware business impact assessment - Merge LLM recommendations into primary output - Preserve fast mode as default for backwards compatibility Enables much deeper analysis of cross-branch impacts beyond simple file pattern matching.
- Add intelligence_level support for smart/deep analysis - LLM commit message quality evaluation - Enhanced business impact assessment for PR decisions - Coordination planning integration - Merge enhanced recommendations into validation output - Maintain fast mode baseline for daily use Provides comprehensive PR safety validation with business context.
- Add git_analyzer and pr_manager to tool registry - Configure appropriate permission levels (REQUIRE_APPROVAL) - Map tool actions in executor - Enable both tools for agent use - Follow clippy's tool integration patterns Makes the new LLM-enhanced PR analysis available throughout clippy.
- Interactive demonstration of git_analyzer and pr_manager - Showcases exact multi-branch scenario (subfeatureB impacts) - Support for fast/smart/deep intelligence levels - Examples for subfeatureA, subfeatureF, mobile F'1 coordination - Usage examples for both fast and LLM-enhanced modes Provides running examples of the new cross-branch analysis capabilities.
- README_PR_FEATURES: Overview of original problem and solution - README_LLM_ENHANCEMENTS: Sweet spot LLM integration guide - docs/PR_ANALYSIS_FEATURES: Detailed technical documentation - Usage examples and configuration guides - Performance trade-offs and best practices Complete documentation for the new cross-branch PR analysis system.
- Remove fast mode and establish smart mode as the default intelligence level across git_analyzer and pr_manager tools - Update documentation to reflect LLM analysis as standard functionality rather than optional enhancement - Modify demo script to always use LLM-enhanced analysis, removing fast/llm mode selection - Improve error handling in LLM analysis to raise exceptions when LLM is unavailable - Update performance tables and feature descriptions to focus on smart vs deep analysis modes - Ensure all analysis results include LLM insights for better coordination recommendations This change streamlines the user experience by providing intelligent analysis by default, eliminating the need for users to explicitly opt-in to LLM features while maintaining the deep analysis option for comprehensive evaluations.
cellwebb
force-pushed
the
main
branch
2 times, most recently
from
5c25bdb to
fd4f610
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🤖 LLM-Enhanced PR Analysis for Intelligent Cross-Branch Coordination
🎯 Problem Solved
This implements the exact scenario requested:
Question: How to analyze commits in
subfeatureBas they might affect:F(parent feature) ✅subfeatureA(sibling) ✅M(main monolith) ✅F'1(mobile feature) ✅🚀 Solution: Sweet Spot Intelligence Levels
📊 Two New Tools Added:
git_analyzer- Advanced cross-branch impact analysispr_manager- Comprehensive PR workflow management🧠 Intelligence Levels:
🎣 Key Features
📈 git_analyzer Capabilities:
🛡️ pr_manager Capabilities:
📋 Demos & Documentation
examples/pr_analysis_demo.pyREADME_LLM_ENHANCEMENTS.mddocs/PR_ANALYSIS_FEATURES.md🎪 Usage Examples
🏆 Real-World Impact
Before:
After (Smart Mode):
🎉 Benefits Achieved
✅ Fast baseline preserved - Daily workflow unchanged
✅ Optional intelligence - Use LLM when you need it
✅ Configurable cost - Choose your intelligence level
✅ Real business value - Actually understands coordination needs
✅ Graceful fallback - Works even if LLM unavailable
🧪 Tests
The tools include comprehensive error handling and graceful fallbacks:
📚 Documentation
This provides the perfect hybrid system for intelligent PR management - fast when you need speed, smart when you need insights! 🤖📎✨