Skip to content

Learn rules from the full vLLM-Omni human review audit - #95

Merged
TaffyOfficial merged 1 commit into
mainfrom
agent/full-human-review-audit
Aug 23, 2026
Merged

TaffyOfficial merged 1 commit into
mainfrom
agent/full-human-review-audit

Conversation

@TaffyOfficial

Copy link
Copy Markdown
Contributor

Summary

  • audit all 55 persisted ReviewBot review jobs covering 44 distinct vLLM-Omni PRs and every persisted status
  • classify all 764 human review/review-comment/issue-comment records; deduplicate 217 substantive findings and exclude contested or administrative-only feedback
  • add 15 executable owner rules across CI, Diffusion, Serving, Host Weight Runtime, Model Executor, and MiniCPM-o 4.5; reinforce existing rules instead of duplicating them
  • keep raw audit material temporary and remove it after synthesis

Validation

  • python knowledge/tools/check_knowledge_tree.py
  • python knowledge/tools/check_wiki_lint.py
  • PYTHONPATH=src python -m pytest test/test_thin_mcp_server.py test/test_imupdate_skill.py -q (42 passed)
  • vLLM-Omni release audit: CLEAN

@TaffyOfficial
TaffyOfficial merged commit a0e1e74 into main Aug 23, 2026
3 checks passed
@TaffyOfficial
TaffyOfficial deleted the agent/full-human-review-audit branch August 23, 2026 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants