पाठ 24 / 25

केस स्टडी: स्वचालित Pull Request Review

इस कोर्स के patterns से PR review workflow डिज़ाइन करें और हर चुनाव समझाएँ।

डिज़ाइन के चुनाव

Trigger: pull request खुलता है। Router बदले हुए paths जाँचता है। एक parallel step में security reviewer और test-gap reviewer read-only subagents के रूप में चलते हैं। Skill टीम की review checklist देता है। Orchestrator निष्कर्षों को एक comment में मिलाता है। Hook या CI check किसी भी write access को रोकता है, और merge का निर्णय इंसान लेता है।

एक पूरा workflow

असली workflow एक pattern, skill, subagent, hooks और eval को एक भरोसेमंद loop में जोड़ता है।

चार हिस्से साथ काम करते हुए।
चित्र 8.1 — Pattern, skill, hooks और eval एक साथ।

Flow की रूपरेखा

इसे text रूप में diagram की तरह पढ़ें। हर box पहले के किसी section की अवधारणा से जुड़ता है।

PR opened
  -> route by changed paths                (routing)
  -> [security reviewer | test-gap reviewer]  (parallel subagents, read-only)
  -> merge findings with checklist skill     (orchestrator + skill)
  -> post ONE comment                        (limited write tool)
  -> human decides to merge                  (checkpoint)
  -> eval set re-run when prompts change     (evals)

त्वरित जाँच: अंतिम merge निर्णय इंसान पर क्यों छोड़ा गया है?

  • Models कोड नहीं पढ़ सकते
  • यह उच्च-प्रभाव वाला काम है, इसलिए checkpoint सुरक्षित है
  • Git bots को मना करता है
  • अंदाज़ा लगाना सस्ता है
Answer

यह उच्च-प्रभाव वाला काम है, इसलिए checkpoint सुरक्षित है — उच्च-प्रभाव और मुश्किल से पलटने योग्य कामों में इंसानी मंज़ूरी चाहिए।