feat: AI Code Reviewer v2 - Severity-based Actions & Project Context by hapo-nghialuu · Pull Request #7 · haposoft/cafekit

hapo-nghialuu · 2026-01-25T08:47:33Z

AI Code Reviewer v2

New Features

Severity Classification: Issues are now classified as critical, high, medium, low
Smart Actions: critical/high → Request Changes, medium/low → Comment only
Project Context: Injects README.md and ARCHITECTURE.md into AI prompt
Improved Prompt: Much stricter to reduce false positives

Testing

This PR will trigger the AI reviewer on itself. Expect to see the new severity badges in any comments.

github-actions

📝 Một số góp ý để cải thiện code.

Đây là AI review tự động. Vui lòng verify trước khi áp dụng.

github-actions · 2026-01-25T08:55:01Z

@@ -0,0 +1,461 @@
+"""


🟡 MEDIUM

Thiếu docstring cho module, nên thêm mô tả chi tiết về chức năng và cách sử dụng.

github-actions · 2026-01-25T08:55:01Z

+# Configure Logging
+logging.basicConfig(level=logging.INFO, format='%(asctime)s - %(levelname)s - %(message)s')
+
+# Constants


🟡 MEDIUM

Hàm should_review nên sử dụng os.path để xử lý đường dẫn file một cách chính xác hơn.

github-actions · 2026-01-25T08:55:01Z

+BLOCKING_SEVERITIES = ['critical', 'high']
+
+
+def should_review(filename):


🟡 MEDIUM

Hàm get_project_context nên giới hạn kích thước tổng cộng của context để tránh vượt quá giới hạn token của AI.

github-actions · 2026-01-25T08:55:01Z

+                context.append(f"### {filepath}\n{text}")
+        except Exception:
+            # File doesn't exist, skip silently
+            pass


🟡 MEDIUM

Hàm get_file_list nên sử dụng đệ quy hoặc os.walk để duyệt thư mục một cách hiệu quả hơn.

github-actions · 2026-01-25T08:55:01Z

+        return files
+    except Exception as e:
+        logging.warning(f"Could not fetch file list: {e}")
+        return []


🟡 MEDIUM

Hàm get_pr_diff nên xử lý trường hợp file quá lớn để tránh vượt quá giới hạn bộ nhớ.

github-actions · 2026-01-25T08:55:01Z

+        files_data.append(file_info)
+
+    return pr, files_data
+


🟡 MEDIUM

Hàm resolve_thread nên kiểm tra response từ GraphQL để đảm bảo thread được resolve thành công.

github-actions · 2026-01-25T08:55:01Z

+    """Resolve all unresolved bot review threads on the PR."""
+    query = """
+    query($owner: String!, $repo: String!, $prNumber: Int!) {
+      repository(owner: $owner, name: $repo) {


🟡 MEDIUM

Hàm resolve_existing_comments nên giới hạn số lượng threads được resolve để tránh vượt quá giới hạn API.

github-actions · 2026-01-25T08:55:01Z

+        pr_data = data.get('data', {}).get('repository', {}).get('pullRequest')
+        if not pr_data:
+            return
+


🟡 MEDIUM

Hàm analyze_code_with_openrouter nên kiểm tra kích thước của files_data trước khi gửi đến AI để tránh vượt quá giới hạn token.

github-actions · 2026-01-25T08:55:01Z

+        )
+    except Exception as e:
+        logging.error(f"Failed to init OpenAI client: {e}")
+        return []


🟡 MEDIUM

Hàm submit_review nên kiểm tra độ dài của review_comments để tránh vượt quá giới hạn API của GitHub.

github-actions · 2026-01-25T08:55:01Z

+    "filename": "path/to/file.py",
+    "line_number": 42,
+    "severity": "critical|high|medium|low",
+    "comment": "Mô tả vấn đề ngắn gọn bằng tiếng Việt"


🟡 MEDIUM

Hàm main nên thêm xử lý ngoại lệ cho các trường hợp không mong muốn khi đọc file event.

github-actions

📝 Một số góp ý để cải thiện code.

Đây là AI review tự động. Vui lòng verify trước khi áp dụng.

github-actions · 2026-01-25T09:09:09Z

@@ -0,0 +1,475 @@
+"""


🟡 MEDIUM

Thiếu docstring cho module, nên thêm mô tả chi tiết về chức năng và cách sử dụng.

github-actions · 2026-01-25T09:09:09Z

+
+import os
+import json
+import requests


🟢 LOW

Biến IGNORED_EXTENSIONS và IGNORED_DIRS nên được định nghĩa là hằng số với tên viết hoa hoàn toàn.

github-actions · 2026-01-25T09:09:09Z

+GITHUB_GRAPHQL_URL = "https://api.github.com/graphql"
+
+# Severity levels that trigger Request Changes
+BLOCKING_SEVERITIES = ['critical', 'high']


🟡 MEDIUM

Hàm should_review có thể được tối ưu bằng cách sử dụng os.path để xử lý đường dẫn.

github-actions · 2026-01-25T09:09:09Z

+    if any(part in filename.split('/') for part in IGNORED_DIRS):
+        return False
+    return True
+


🟡 MEDIUM

Hàm get_project_context nên xử lý ngoại lệ cụ thể hơn thay vì sử dụng Exception chung chung.

github-actions · 2026-01-25T09:09:09Z

+        files = []
+        while contents:
+            item = contents.pop(0)
+            if item.type == "dir" and item.name not in IGNORED_DIRS:


🟡 MEDIUM

Hàm get_file_list có thể gây ra lỗi nếu repository quá lớn, nên thêm giới hạn số lượng file.

github-actions · 2026-01-25T09:09:09Z

+            if content and hasattr(content, 'decoded_content'):
+                full_content = content.decoded_content.decode('utf-8')
+                # Limit full content size
+                if len(full_content) <= 5000:


🟡 MEDIUM

Hàm get_pr_diff nên kiểm tra file.status cụ thể hơn để tránh bỏ sót các trường hợp cần review.

github-actions · 2026-01-25T09:09:09Z

+    query($owner: String!, $repo: String!, $prNumber: Int!) {
+      repository(owner: $owner, name: $repo) {
+        pullRequest(number: $prNumber) {
+          reviewThreads(last: 100) {


🟡 MEDIUM

Hàm resolve_thread nên kiểm tra phản hồi từ GraphQL để đảm bảo thread được resolve thành công.

github-actions · 2026-01-25T09:09:09Z

+                author = comments[0].get('author')
+                if author and author.get('login') == 'github-actions[bot]':
+                    bot_threads.append(thread.get('id'))
+


🟡 MEDIUM

Hàm analyze_code_with_openrouter nên xử lý ngoại lệ cụ thể hơn và ghi log chi tiết hơn.

github-actions · 2026-01-25T09:09:09Z

+## PROJECT FILES
+{', '.join(file_list[:50])}
+
+## SEVERITY LEVELS (QUAN TRỌNG - PHẢI TRẢ VỀ)


🟡 MEDIUM

Hàm submit_review nên kiểm tra review_comments trước khi gọi pr.create_review để tránh lỗi.

github-actions

📝 Góp ý cải thiện code.

github-actions · 2026-01-25T09:46:34Z

+from github import Github, GithubException
+from openai import OpenAI, AuthenticationError, RateLimitError
+
+# Configure Logging


🟡 MEDIUM

Hàm should_review nên sử dụng os.path để xử lý đường dẫn file một cách nhất quán và tránh lỗi trên các hệ điều hành khác nhau.

github-actions · 2026-01-25T09:46:34Z

+    if any(part in filename.split('/') for part in IGNORED_DIRS):
+        return False
+    return True
+


🟡 MEDIUM

Hàm get_project_context nên giới hạn độ dài của nội dung file một cách nhất quán và tránh cắt cụt nội dung một cách tùy tiện.

github-actions · 2026-01-25T09:46:34Z

+
+    return "\n\n".join(context) if context else "No project documentation found."
+
+


🟡 MEDIUM

Hàm get_file_list có thể gây ra lỗi nếu số lượng file vượt quá giới hạn cho phép của API GitHub.

github-actions · 2026-01-25T09:46:34Z

+    for file in pr.get_files():
+        if not should_review(file.filename):
+            continue
+        if file.status == 'removed':


🟡 MEDIUM

Hàm get_pr_diff nên xử lý trường hợp file bị xóa một cách rõ ràng hơn để tránh lỗi không mong muốn.

github-actions · 2026-01-25T09:46:34Z

+    """
+    headers = {"Authorization": f"Bearer {token}", "Content-Type": "application/json"}
+    payload = {"query": query, "variables": {"owner": repo_owner, "repo": repo_name, "prNumber": pr_number}}
+


🟡 MEDIUM

Hàm get_existing_bot_threads nên kiểm tra và xử lý lỗi từ GraphQL một cách chi tiết hơn.

github-actions · 2026-01-25T09:46:34Z

+    mutation ResolveThread($threadId: ID!) {
+      resolveReviewThread(input: {threadId: $threadId}) {
+        thread { isResolved }
+      }


🟡 MEDIUM

Hàm manage_review_threads nên cải thiện logic so khớp dòng để tránh trường hợp không khớp do thay đổi dòng.

github-actions · 2026-01-25T09:46:34Z

+
+            if new_file == old_file and abs(new_line - old_line) <= 3:
+                # MATCH FOUND!
+                matched_thread_ids.add(old_th['id'])


🟡 MEDIUM

Hàm analyze_code_with_openrouter nên kiểm tra và xử lý lỗi từ API OpenRouter một cách chi tiết hơn.

github-actions · 2026-01-25T09:46:34Z

+- **medium**: Performance issues, bad practices, potential bugs
+- **low**: Minor suggestions, style (chỉ khi rất cần thiết)
+
+## RULES


🟡 MEDIUM

Hàm submit_review nên kiểm tra và xử lý lỗi từ GitHub API một cách chi tiết hơn.

github-actions · 2026-01-25T09:46:34Z

+    # Check for blocking issues
+    has_blocking = any(c.get('severity', '').lower() in BLOCKING_SEVERITIES for c in comments)
+    summary = "🚨 **Phát hiện vấn đề nghiêm trọng.**" if has_blocking else "📝 **Góp ý cải thiện code.**"
+    event = "REQUEST_CHANGES" if has_blocking else "COMMENT"


🟡 MEDIUM

Hàm main nên kiểm tra và xử lý lỗi từ các hàm con một cách chi tiết hơn.

github-actions · 2026-01-25T09:46:34Z

+
+    if not files_data:
+        logging.info("No files to review.")
+        return


🟡 MEDIUM

Script nên có cơ chế logging chi tiết hơn để dễ dàng debug khi có lỗi xảy ra.

hapo-nghialuu added 2 commits January 25, 2026 15:47

feat: AI Code Reviewer v2 - severity-based actions and project context

01228c2

ci: add missing AI Reviewer workflow

103b183

github-actions Bot reviewed Jan 25, 2026

View reviewed changes

fix: address AI review feedback (docstring, exception handling)

7428164

github-actions Bot reviewed Jan 25, 2026

View reviewed changes

feat: implement smart auto-resolve (sync comments instead of wipe)

a36a40e

github-actions Bot reviewed Jan 25, 2026

View reviewed changes

hapo-nghialuu closed this Jan 25, 2026

		BLOCKING_SEVERITIES = ['critical', 'high']


		def should_review(filename):


		return "\n\n".join(context) if context else "No project documentation found."

Conversation

hapo-nghialuu commented Jan 25, 2026

AI Code Reviewer v2

New Features

Testing

Uh oh!

github-actions Bot left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

github-actions Bot left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Jan 25, 2026