dev #29

Merged
Latte merged 10 commits from dev into main 2025-12-29 13:15:47 +00:00

10 Commits

Author SHA1 Message Date
e067687517 Merge branch 'main' into dev
All checks were successful
Enterprise AI Code Review / ai-review (pull_request) Successful in 9s
2025-12-29 13:15:40 +00:00
15b1de695b Merge pull request 'fix: Prevent bot self-trigger infinite loops in all workflows' (#28) from hotfix/prevent-bot-self-trigger into dev
All checks were successful
Enterprise AI Code Review / ai-review (pull_request) Successful in 36s
Reviewed-on: #28
2025-12-29 13:13:25 +00:00
3bde95b42d Merge branch 'dev' into hotfix/prevent-bot-self-trigger
All checks were successful
Enterprise AI Code Review / ai-review (pull_request) Successful in 31s
2025-12-29 13:13:17 +00:00
a8bfdf43d9 Merge pull request 'docs: Add Milestone 2 verification and deployment status report' (#27) from docs/milestone-2-verification into dev
Reviewed-on: #27
2025-12-29 13:12:55 +00:00
8afad737ba fix: Prevent bot self-trigger infinite loops in all workflows
All checks were successful
Enterprise AI Code Review / ai-review (pull_request) Successful in 34s
CRITICAL FIX: Bot was triggering itself causing 10+ duplicate runs

Problem:
- When bot posts comments containing @codebot (e.g., help responses, PR reviews)
- Workflows trigger on those bot comments
- Bot responds again with @codebot mention
- Infinite loop → 10+ duplicate workflow runs → excessive API costs

Solution:
- Added github.event.comment.user.login != 'Bartender' to all workflow conditions
- Prevents bot from reacting to its own comments
- Bot username 'Bartender' is now hardcoded in workflows

Changes:
- .gitea/workflows/ai-comment-reply.yml: Added bot username check
- .gitea/workflows/ai-chat.yml: Added bot username check
- .gitea/workflows/ai-issue-triage.yml: Added bot username check
- CLAUDE.md: Documented bot self-trigger prevention and username update instructions
- README.md: Added Step 3 to bot customization with critical warning

Impact:
- Eliminates infinite loop scenarios
- Prevents excessive API costs from duplicate runs
- Workflows only trigger on human user comments

Note: If bot username changes from 'Bartender', all three workflow files must be updated.
2025-12-29 13:12:19 +00:00
3cf60bc36e docs: Add Milestone 2 verification and deployment status report
All checks were successful
Enterprise AI Code Review / ai-review (pull_request) Successful in 30s
- Complete documentation verification for all 3 Milestone 2 features
- PR Summary Generator: Auto-generate comprehensive PR summaries
- PR Changelog Generator: Keep a Changelog format entries
- Code Diff Explainer: Plain-language translations for non-technical stakeholders
- Verified all documentation (README.md, CLAUDE.md, config.yml)
- Confirmed workflow routing prevents duplicate runs
- 28 new tests added (54 total) covering all features
- All features ready for production deployment

Status: 100% complete, production-ready
2025-12-29 13:06:22 +00:00
c712fe670b Merge pull request 'feat: Add @codebot explain-diff command for plain-language PR explanations' (#26) from feature/code-diff-explainer into dev
Reviewed-on: #26
2025-12-29 12:53:32 +00:00
37f3eb45d0 feat: Add @codebot explain-diff command for plain-language PR explanations
All checks were successful
Enterprise AI Code Review / ai-review (pull_request) Successful in 39s
Implements code diff explainer that translates technical changes into
plain language for non-technical stakeholders (PMs, designers, new team members).

Features:
- Plain-language explanations without jargon
- File-by-file breakdown with 'what' and 'why' context
- Architecture impact analysis
- Breaking change detection
- Perfect for onboarding and cross-functional reviews

Implementation:
- Added explain_diff.md prompt template with plain-language guidelines
- Implemented _handle_explain_diff_command() in PRAgent
- Added _format_diff_explanation() for readable markdown
- Updated PRAgent.can_handle() to route explain-diff commands
- Added 'explain-diff' to config.yml commands list

Workflow Safety (prevents duplicate runs):
- Added '@codebot explain-diff' to ai-comment-reply.yml conditions
- Excluded from ai-chat.yml to prevent duplication
- Only triggers on PR comments (not issues)
- Manual command only (no automatic triggering)

Testing:
- 9 comprehensive tests in TestDiffExplanation class
- Tests command detection, formatting, plain-language output
- Verifies prompt formatting and empty section handling

Documentation:
- Updated README.md with explain-diff command and examples
- Added detailed implementation guide in CLAUDE.md
- Included plain-language rules and use cases

Related: Milestone 2 high-priority feature - code diff explainer
2025-12-29 12:44:54 +00:00
1d468e360e Merge pull request 'feat: Add @codebot changelog command for Keep a Changelog format generation' (#25) from feature/pr-changelog-generator into dev
Reviewed-on: #25
2025-12-29 11:53:57 +00:00
15beb0fb5b feat: Add @codebot changelog command for Keep a Changelog format generation
All checks were successful
Enterprise AI Code Review / ai-review (pull_request) Successful in 41s
Implements PR changelog generator that analyzes diffs and generates
Keep a Changelog format entries ready for CHANGELOG.md.

Features:
- Generates structured changelog entries (Added/Changed/Fixed/etc.)
- Automatically detects breaking changes
- Includes technical details (files, LOC, components)
- User-focused language filtering out noise
- Ready to copy-paste into CHANGELOG.md

Implementation:
- Added changelog.md prompt template with Keep a Changelog format
- Implemented _handle_changelog_command() in PRAgent
- Added _format_changelog() for markdown formatting
- Updated PRAgent.can_handle() to route changelog commands
- Added 'changelog' to config.yml commands list

Workflow Safety (prevents duplicate runs):
- Added '@codebot changelog' to ai-comment-reply.yml conditions
- Excluded from ai-chat.yml to prevent duplication
- Only triggers on PR comments (not issues)
- Manual command only (no automatic triggering)

Testing:
- 9 comprehensive tests in TestChangelogGeneration class
- Tests command detection, formatting, config validation
- Verifies prompt formatting and Keep a Changelog structure

Documentation:
- Updated README.md with changelog command and examples
- Added detailed implementation guide in CLAUDE.md
- Included example output and use cases

Related: Milestone 2 feature - PR changelog generation for release notes
2025-12-29 10:52:48 +00:00