CodeRabbit Review - abc1234

Date: 2025-12-29T14:30:22Z Files Reviewed: 8 Issues Found: 27

Summary

• Critical: 2
• High: 5
• Medium: 12
• Low: 8

Issues

1. SQL Injection Vulnerability (critical)

File: crates/api/src/auth.rs:45 Category: security

Description: User input not sanitized in database query

Suggested Fix: Use parameterized queries with sqlx::query!

### 內存庫集成

#### 自動索引
PlexMCP嵌入監視器會自動對Markdown審查文件進行索引：
1. **文件監視器**：檢測`.coderabbit/reviews/`目錄下的新`.md`文件。
2. **分塊器**：將審查內容拆分為語義部分。
3. **嵌入生成**：使用Ollama（nomic-embed-text）生成嵌入向量。
4. **FAISS存儲**：對向量進行索引，支持相似性搜索。
5. **SQLite FTS5**：對文本進行索引，支持關鍵字搜索。
**域名**：`coderabbit-reviews`

#### 通過MCP工具進行搜索
Claude Code可以使用`find_code_quality_issues()` MCP工具搜索審查歷史。
**返回結果**：
- 相關的過去代碼審查
- 問題描述和解決方案
- 文件位置和行號
- 時間模式（重複問題）

### 使用案例

#### 1. 從過去的錯誤中學習
**查詢**："What SQL injection vulnerabilities have we had?"
**結果**：所有提及SQL注入的過去審查，顯示：
- 問題發生的位置
- 問題的修復方法
- 預防策略

#### 2. 跟蹤重複問題
**查詢**："Show me all unwrap() issues in the last month"
**結果**：對`.unwrap()`使用情況的時間模式分析，幫助識別：
- 存在重複問題的文件
- 需要培訓的開發人員
- 需要重構的區域

#### 3. 安全審計跟蹤
**查詢**："Critical security issues in authentication code"
**結果**：認證相關文件中安全發現的完整歷史：
- 發現的問題
- 問題的修復時間
- 修復問題的提交者

#### 4. 新開發人員入職
**查詢**："Common code quality issues in this codebase"
**結果**：彙總的模式，顯示：
- 項目特定的編碼標準
- 應避免的常見陷阱
- 最佳實踐示例

#### 5. 版本質量檢查
**查詢**："Issues found since last release tag"
**結果**：當前版本週期中引入的所有質量問題：
- 嚴重程度分佈
- 文件影響分析
- 迴歸檢測

### 維護

#### 存儲大小
**增長率**：每次提交約10 - 50 KB（根據變更集大小而異）
**估計年度存儲量**：約10 - 50 MB（假設每年1000次提交）
**建議**：無限期保留（成本低，價值高）

#### 清理（如有需要）
**歸檔舊審查**：
```bash
# Archive reviews older than 1 year
find .coderabbit/reviews -name "*.json" -mtime +365 -exec gzip {} \;
find .coderabbit/reviews -name "*.md" -mtime +365 -exec gzip {} \;

刪除非常舊的審查：

# Delete reviews older than 2 years (CAUTION)
find .coderabbit/reviews -name "*.json" -mtime +730 -delete
find .coderabbit/reviews -name "*.md" -mtime +730 -delete

# Re-index embeddings after deletion
cd .embeddings && npm run reindex

損壞恢復

如果JSON文件損壞：

1. 審查記錄仍存在於git歷史中：

git log --all --full-history -- .coderabbit/reviews/
git show <commit-hash>:.coderabbit/reviews/<filename>

2. 從提交中重新生成：

git checkout <commit-hash>
coderabbit --type committed --output json > recovered.json

Git集成

是否應提交審查記錄？

是 - 審查記錄應提交到git，原因如下：

1. 版本歷史：將審查記錄與特定提交關聯起來。
2. 團隊共享：所有團隊成員可以訪問相同的審查歷史。
3. CI/CD集成：GitHub Actions可以分析趨勢。
4. 備份：審查記錄保存在git歷史中。
5. 可審計性：提供完整的質量跟蹤。

.gitignore考慮事項

不要忽略 .coderabbit/reviews/ - 這些文件應提交。 當前的.gitignore：

# Do NOT ignore CodeRabbit reviews
# !.coderabbit/

# Temporary review files (if any)
.coderabbit/*.tmp
.coderabbit/*.lock

大型倉庫問題

如果審查存儲變得太大（不太可能）： 選項1：對JSON文件使用Git LFS

git lfs track ".coderabbit/reviews/*.json"

選項2：使用單獨的審查倉庫

# Create separate repo
git init coderabbit-reviews
git remote add reviews https://github.com/org/plexmcp-reviews

# Store reviews there instead

故障排除

沒有生成審查記錄

檢查：

# 1. Post-commit hook exists and is executable
ls -la .git/hooks/post-commit
chmod +x .git/hooks/post-commit

# 2. Scripts exist and are executable
ls -la scripts/store-coderabbit-review.sh
chmod +x scripts/store-coderabbit-review.sh

# 3. CodeRabbit authenticated
coderabbit auth status

手動觸發：

bash scripts/store-coderabbit-review.sh

審查記錄無法搜索

檢查：

# 1. Markdown files exist
ls .coderabbit/reviews/*.md

# 2. Watcher is indexing them
cd .embeddings && npm run stats
# Should show coderabbit-reviews domain

# 3. Re-index if needed
npm run reindex

審查記錄為空

原因：CodeRabbit執行失敗（網絡、認證、超時） 修復：

# Check last commit review manually
coderabbit --type committed --output json

# If works, re-generate review
bash scripts/store-coderabbit-review.sh

最佳實踐

1. 立即提交審查記錄

審查記錄與所審查的代碼一起提交時最有價值。 工作流程：

git add .
git commit -m "Implement feature X"
# Post-commit hook runs automatically, stores review

git add .coderabbit/reviews/
git commit -m "Add CodeRabbit review for feature X"
git push

替代方法：在同一提交中包含審查記錄（需要自定義鉤子）

2. 在主要版本發佈前進行審查

在發佈前，搜索未解決的問題：

// Via Claude Code
find_code_quality_issues({
  query: "critical high severity issues",
  severity: "critical"
})

3. 定期進行質量審計

每月審查模式：

# Generate summary report
cd .coderabbit/reviews
grep -h "^### " *.md | sort | uniq -c | sort -rn

4. 將審查記錄鏈接到PR

在PR描述中引用審查文件：

## Code Quality

CodeRabbit review: `.coderabbit/reviews/2025-12-29-143022-abc1234.md`

**Issues found:** 5 (3 high, 2 medium)
**All resolved:** ✅