MokoConsulting/MokoGitea-APP
1
0
Fork 0
Code Issues 158 Pull Requests 5 Releases 96 Wiki Activity

feat(ai): PR review service (services/ai/review.go) #365

New Issue
Open
opened 2026-05-31 16:28:34 +00:00 by jmiller · 0 comments
jmiller commented 2026-05-31 16:28:34 +00:00
Owner
Copy Link
Copy Source

Summary

Built-in PR review service that triggers on PR open/push and posts smart comments from mokogitea-claude[bot].

Parent Issue

#359

Behavior

  1. PR opened or new commits pushed
  2. If repo has AI enabled and auto_review=true:
    • Fetch the diff
    • Read .mokogitea/CLAUDE.md for project rules
    • Send to Claude with review prompt
    • Post summary comment
    • Post inline review comments with suggestions
    • Apply risk label if configured

Comment Types

  • Summary: what the PR does, key changes, risk areas
  • Inline reviews: bugs, security, performance, style (using Gitea suggestion blocks)
  • Risk labels: review: low-risk / review: needs-attention / review: security-concern

Strictness Levels

  • minimal: only critical bugs and security
  • standard: bugs, security, performance, major style
  • thorough: everything including suggestions and readability

Implementation

  • services/ai/review.go
  • services/ai/prompt.go
  • services/ai/comment.go
  • Hooks into notify service (same pattern as Actions)
  • Debounce: wait 30s after last push
  • Skip draft PRs unless requested

Dependencies

  • AI settings schema
  • #362 (mokogitea-claude[bot])
## Summary Built-in PR review service that triggers on PR open/push and posts smart comments from mokogitea-claude[bot]. ## Parent Issue #359 ## Behavior 1. PR opened or new commits pushed 2. If repo has AI enabled and auto_review=true: - Fetch the diff - Read .mokogitea/CLAUDE.md for project rules - Send to Claude with review prompt - Post summary comment - Post inline review comments with suggestions - Apply risk label if configured ## Comment Types - **Summary**: what the PR does, key changes, risk areas - **Inline reviews**: bugs, security, performance, style (using Gitea suggestion blocks) - **Risk labels**: review: low-risk / review: needs-attention / review: security-concern ## Strictness Levels - minimal: only critical bugs and security - standard: bugs, security, performance, major style - thorough: everything including suggestions and readability ## Implementation - services/ai/review.go - services/ai/prompt.go - services/ai/comment.go - Hooks into notify service (same pattern as Actions) - Debounce: wait 30s after last push - Skip draft PRs unless requested ## Dependencies - AI settings schema - #362 (mokogitea-claude[bot])
Sign in to join this conversation.
Labels
Clear labels
breaking-change
Breaking API or behavior change
 ci-cd
CI/CD pipeline changes
 config
Configuration changes
 dependencies
Dependency updates
 deploy-failure
Deployment failed
 docker
Docker/container changes
 documentation
Documentation changes
 good first issue
Good for newcomers
 health-check
Repo health check result
 help wanted
Extra attention needed
 mokostandards
Related to MokoStandards framework
 push-failure
Git push operation failed
 security
Security vulnerability or hardening
 size/l
200-500 lines changed
 size/m
50-200 lines changed
 size/s
10-50 lines changed
 size/xl
500+ lines changed
 size/xs
< 10 lines changed
 standards-drift
Deviates from MokoStandards
 standards-update
MokoStandards compliance update
 sync-failure
Sync or mirror failed
 tech-debt
Technical debt and TODO/FIXME items
 upstream
upstream
Inherited from upstream Gitea
 work-in-progress
Draft or incomplete work
No labels
Priority Medium
Type Feature
Status —
Priority —
Type —
Milestone
No items
No Milestone
Assignees
Clear assignees
jmiller (Jonathan Miller) moko-deploy (Moko Deploy Bot)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: MokoConsulting/MokoGitea-APP#365
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?