Timmy_Foundation/compounding-intelligence
15
0
Fork 0
Code Issues 173 Pull Requests 59 Actions Packages Projects Releases Wiki Activity

feat(review): add Review Quality Scorer #274

Open
Rockachopa wants to merge 1 commits from step35/127-6-9-review-quality-scorer into main
pull from: step35/127-6-9-review-quality-scorer
merge into: Timmy_Foundation:main
Conversation 1 Commits 1 Files Changed 2 +544
Rockachopa commented 2026-04-26 16:26:37 +00:00
Owner
Copy Link

Summary

Adds scripts/review_quality_scorer.py — a standalone quality scorer for PR
reviews. It evaluates review comments across five categories:

  • Style (weight 0.15) — formatting, lint, naming
  • Logic (weight 0.25) — correctness, edge cases, reasoning
  • Security (weight 0.25) — vulnerabilities, auth, secrets
  • Performance(weight 0.15) — bottlenecks, efficiency
  • Tests (weight 0.20) — coverage, quality, missing tests

Each category is scored 0–100 based on keyword presence and review depth.
A weighted composite is produced.

Also adds scripts/test_review_quality_scorer.py with 15 unit tests.

Closes #127

## Summary Adds `scripts/review_quality_scorer.py` — a standalone quality scorer for PR reviews. It evaluates review comments across five categories: - **Style** (weight 0.15) — formatting, lint, naming - **Logic** (weight 0.25) — correctness, edge cases, reasoning - **Security** (weight 0.25) — vulnerabilities, auth, secrets - **Performance**(weight 0.15) — bottlenecks, efficiency - **Tests** (weight 0.20) — coverage, quality, missing tests Each category is scored 0–100 based on keyword presence and review depth. A weighted composite is produced. Also adds `scripts/test_review_quality_scorer.py` with 15 unit tests. Closes #127
Rockachopa added 1 commit 2026-04-26 16:26:38 +00:00
feat(review): add Review Quality Scorer (#127)
Some checks failed
Test / pytest (pull_request) Failing after 8s
Details
c4586eb2be 
Score PR reviews across 5 categories: style, logic, security,
performance, tests. Produces weighted composite and markdown report.

Closes #127
Timmy commented 2026-05-02 18:34:39 +00:00
Owner
Copy Link

🛡️ Goblin Patrol Alert 🛡️

Hey brother — this PR has been idle for 6 days and is unassigned.

The goblin fleet has been notified. A goblin may claim this if it remains stale.

— Timmy Goblin Wizard King

🛡️ **Goblin Patrol Alert** 🛡️ Hey brother — this PR has been idle for **6 days** and is unassigned. The goblin fleet has been notified. A goblin may claim this if it remains stale. — Timmy Goblin Wizard King
Some checks failed
Test / pytest (pull_request) Failing after 8s
Details
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin step35/127-6-9-review-quality-scorer:step35/127-6-9-review-quality-scorer
git checkout step35/127-6-9-review-quality-scorer
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
acceptance-criteria
batch-pipeline
Token masterplan batch pipeline
 bootstrapper
Pre-session context injection
 epic
Epic-level issue
 harvester
Session knowledge extraction
 measurer
Compounding metrics
 milestone:1
Milestone 1: Foundation
 milestone:2
Milestone 2: Integration
 milestone:3
Milestone 3: Measurement
 milestone:4
Milestone 4: Retroactive
 pipeline
Pipeline/integration work
 pipeline
priority:high
priority:medium
retroactive
Processing existing sessions
 throughput-10x
throughput-10x label
 token-masterplan
token-masterplan label
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
Rockachopa Timmy allegro antigravity bezalel claude codex-agent ezra gemini google grok hermes kimi manus perplexity
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Timmy_Foundation/compounding-intelligence#274
Delete Branch "step35/127-6-9-review-quality-scorer"

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?