कोड पुनरावलोकन आणि त्रुटी शोधणे

नफा:

• श्रेणी आणि तीव्रता टॅगसह प्रारंभिक पुनरावलोकन फिल्टर म्हणून AI वापरण्याची क्षमता
• सकारात्मक/असत्य सत्यापित/लागू करण्यासाठी मानवी मनाने निष्कर्ष फिल्टर करण्याची क्षमता
• व्यवसाय नियम, आर्किटेक्चर आणि सुरक्षा-गंभीर निर्णयांवर मानवी मंजूरी आवश्यकता लागू करण्याची क्षमता

कोड रिव्ह्यू म्हणजे जेव्हा डेव्हलपरने लिहिलेला बदल विलीन होण्यापूर्वी त्याचे पुनरावलोकन इतर कोणीतरी केले. चांगले पुनरावलोकन; हे बग लवकर पकडते, माहिती शेअर करते आणि कोड बेस सुसंगत ठेवते. परंतु पुनरावलोकने थकवणारी असतात, विचलित होण्याची शक्यता असते आणि वेळेच्या दबावाखाली वरवरची बनतात. आर्टिफिशियल इंटेलिजन्स हा येथे दुहेरी सहाय्यक आहे: तो तुम्हाला तुमचा स्वतःचा कोड पूर्व-साफ करण्यास अनुमती देतो जो तुम्ही पुनरावलोकनासाठी सबमिट करता आणि दुसऱ्याच्या PR (पुल रिक्वेस्ट) चे बारीक नजरेने परीक्षण करू शकता.

गंभीर फरक हा आहे: AI गती वाढवते आणि पुनरावलोकन वाढवते, परंतु ते मंजुरीची जबाबदारी घेऊ शकत नाही. "एआय पाहिले, ते स्वच्छ आहे" हे वाक्य समर्थन नाही. अंतिम "विलीनीकरण" निर्णय हा कोड आणि संदर्भ माहित असलेल्या अभियंत्यावर अवलंबून असतो.

पुनरावलोकनात एआय चांगले आणि वाईट काय आहे

यासाठी चांगले: शून्य चेक मिस, रिसोर्स लीक (फाइल/लिंक उघडी राहिली), न पकडलेले अपवाद, स्पष्टपणे चुकीच्या अटी (>= ऐवजी >), नाव बदलणे, वाचनीयता, गहाळ एज केस, साधे सुरक्षा वास (जसे SQL स्ट्रिंग कॉन्कटेनेशन), डुप्लिकेट कोड डिटेक्शन.

कमकुवतता: खोल त्रुटी ज्या तुमच्या व्यवसायाच्या नियमाचे उल्लंघन करतात परंतु त्यांना संदर्भ आणि वेळेची आवश्यकता असते, जसे की सिंटॅक्टिकली योग्य तर्कशास्त्र, आर्किटेक्चरल अनुपालन, वास्तविक कार्यप्रदर्शनातील अडथळे, समवर्ती त्रुटी. AI खोटे सकारात्मक (समस्येसाठी प्रत्यक्षात समस्या नसलेली एखादी गोष्ट चुकणे) आणि खोटे नकारात्मक (खरा बग गहाळ) देखील तयार करतो. म्हणून, त्याचे आउटपुट "सावधगिरीची यादी" आहे, निश्चित निर्णय नाही.

खबरदारी: एआयने "काही हरकत नाही" म्हटल्याने कोड बरोबर असल्याचे सिद्ध होत नाही. खोटे नकारात्मक शांत आहेत; सर्वात धोकादायक चुका त्या आहेत ज्यांचा पुनरावलोकनात कधीही उल्लेख केला जात नाही.

पद्धतशीर पुनरावलोकन पायऱ्या

1. संदर्भ द्या. बदलाचा उद्देश, संबंधित समस्या आणि स्वीकृती निकष, जर असेल तर, प्रॉम्प्टमध्ये जोडा. उद्देशहीन समीक्षा हे उद्दिष्टहीन व्याख्या निर्माण करते.
2. ते श्रेणींमध्ये विभाजित करा. मॉडेलला निष्कर्षांचे "बग/सुरक्षा/कार्यप्रदर्शन/वाचनीयता/शैली" असे वर्गीकरण करण्यास सांगा; त्यामुळे तुम्ही क्रिटिकलला आवाजापासून वेगळे करता.
3. तीव्रता लेबलची विनंती करा. प्रत्येक शोधाला "उच्च/मध्यम/निम्न" रेटिंग द्या आणि "कारणे" आणि "शिफारस केलेली सुधारणा" समाविष्ट करा.
4. ते स्वतःच्या डोळ्यांनी गाळून घ्या. प्रत्येक शोधाचे मूल्यांकन करा: ते खरे आहे का (पडताळणी करा), ते चुकीचे सकारात्मक आहे का (औचित्य लिहा), काही गहाळ आहे का (तुमचे स्वतःचे ज्ञान जोडा).
5. गंभीर मार्ग व्यक्तिचलितपणे सत्यापित करा. AI वर विसंबून न राहता पैसे, ओळख, अधिकृतता आणि डेटा हटवण्याचे मार्ग वाचा आणि कार्यान्वित करा.

तीन मिनी केसेस

केस 1 - मूक शून्य त्रुटी पकडली. एका टीमने 380-लाइन PR चे AI पूर्व-पुनरावलोकन केले. मॉडेलने एक मार्ग ध्वजांकित केला ज्यामध्ये बाह्य सेवा प्रतिसाद शून्य असू शकतो, परंतु कोडमध्ये यासाठी कोणतीही तपासणी केली गेली नाही. मानवी समीक्षकाने हा मार्ग सत्यापित केला आणि एक शून्य चेक जोडला; अशाच त्रुटीमुळे मागील तिमाहीत उत्पादनात 2-तास व्यत्यय आला.

केस 2 - खोटे सकारात्मक निर्मूलन. AI ने लूपमध्ये "संभाव्य कामगिरी समस्या" ध्वजांकित केली. समीक्षकाने हे चुकीचे सकारात्मक म्हणून बंद केले, हे जाणून की लूप केवळ कमाल 5 घटकांसह कार्य करते (ते एनमवर लूप करते). संदर्भ माहीत नसलेल्या मॉडेलने इशारा दिला; संदर्भ माहीत असलेल्या व्यक्तीने योग्य निर्णय घेतला.

केस 3 - AI ने बिझनेस नियम चुकला. मोहिमेच्या नियमानुसार सवलत खाते जास्तीत जास्त 30% असले पाहिजे, कोडने 50% परवानगी दिली आहे. एआयने ही सिंटॅक्टिकली परिपूर्ण तार्किक त्रुटी कधीही लक्षात घेतली नाही; कारण त्याला नियम माहीत नव्हता. स्वीकृती निकष माहित असलेल्या उत्पादन मालकाने पुनरावलोकनात बग पकडला. धडा: व्यवसाय नियम प्रमाणीकरण हे मानवी काम आहे.

चार कॉपी करण्यायोग्य टेम्पलेट्स

उद्देश-देणारं, वर्गीकृत पुनरावलोकन:

भूमिका: सूक्ष्म कोड पुनरावलोकनकर्ता. बदलाचा उद्देश: {{purpose / issue}}या फरकाचे पुनरावलोकन करा. या श्रेणींमध्ये निष्कर्ष प्रदान करा: [बग] [सुरक्षा][कार्यप्रदर्शन] [वाचनीयता] [शैली]. प्रत्येक शोधासाठी: फाइल:पंक्ती, तीव्रता (उच्च/मध्यम/निम्न), कारण, शिफारस केलेले निराकरण. तुम्हाला खात्री नसल्यास "शक्य" चिन्हांकित करा. तुम्हाला व्यवसायाचे नियम माहित नाहीत; मला नियमांची आवश्यकता असलेल्या ठिकाणांबद्दल विचारा.{{diff}}

तुमच्या स्वतःच्या कोडचे पुनरावलोकन करण्याची तयारी करण्यासाठी:

PR उघडण्यापूर्वी या बदलाचे पुनरावलोकन करा. पहा: गहाळ नल/बगचेक, संसाधन लीक, एज केस, गुप्त, न तपासलेली शाखा. प्राधान्य क्रमाने निष्कर्षांची यादी करा; प्रत्येकासाठी 1 ओळ सुधारणा सुचवा.{{code}}

एज केस हंट:

इनपुट्स आणि परिस्थितींची यादी करा जिथे हे कार्य खंडित होऊ शकते: रिक्त, शून्य, खूप मोठे, नकारात्मक, समवर्ती कॉल, नेटवर्क त्रुटी, आंशिक डेटा. प्रत्येक केससाठी, अपेक्षित वर्तन आणि वर्तमान कोड काय करेल ते लिहा.{{function}}

सुरक्षा सुगंध स्कॅनिंग (प्री-स्क्रीनिंग):

या कोडमध्ये सामान्य सुरक्षा वास शोधा: SQL/command concatenation, unvalidated input, immutable embedded secret, insecure deserialize, विशेषाधिकार तपासणीचा अभाव. निष्कर्षांना "विशिष्ट/संभाव्य/ज्ञान" मध्ये विभक्त करा. ही प्राथमिक तपासणी आहे; हा निश्चित नियम नाही.{{code}}

कमकुवत प्रॉम्प्ट / मजबूत प्रॉम्प्ट

कमकुवत: "या PR मध्ये काही चूक आहे का?"

सशक्त: "उद्देश: कार्टच्या एकूणात कूपन सवलत जोडा (सवलत 30% पेक्षा जास्त नसावी — तुम्ही हा नियम स्वतः सत्यापित करू शकत नाही, कोडने वरची मर्यादा लागू केली असेल तर मला सांगा). फरक तपासा; श्रेणी + तीव्रता + सुचवलेल्या सुधारणांनुसार निष्कर्ष द्या, अनिश्चित असल्यास 'शक्य' चिन्हांकित करा. [तफा]

सशक्त आवृत्ती AI चा हेतू, व्यवसाय नियम आणि सीमा स्पष्टपणे सांगते; अशा प्रकारे, उपयुक्त निष्कर्ष येतात आणि मॉडेलसाठी अज्ञात क्षेत्र स्पष्ट राहते.

प्रकार शोधत आहे

AI विश्वसनीयता

माणसाची भूमिका

शून्य/त्रुटी तपासणी गहाळ आहे

उच्च

सत्यापित करा आणि अर्ज करा

वाचनीयता/शैली

उच्च

प्राधान्याने निवडा

साधा सुरक्षा वास

मध्यम

अंतिम करा, वाहनासह स्कॅन करा

व्यवसाय नियमांचे पालन

कमी

तो पूर्णपणे मानवी आहे.

समरूपता/आर्किटेक्चर

कमी

तज्ञांचे पुनरावलोकन आवश्यक आहे

AI पुनरावलोकन हे मानवी पुनरावलोकनासाठी बदली नाही

AI पुनरावलोकनाला “प्रथम फिल्टर” म्हणून स्थान द्या: स्वस्त, वेगवान, अथक प्राथमिक पास. हे फिल्टर मानवी समीक्षकाचे लक्ष बिनमहत्त्वाच्या तपशिलांपासून (एक जागा, एक नाव) मुक्त करते आणि ते अशा ठिकाणी निर्देशित करते ज्यांना खरोखर विचार करणे आवश्यक आहे—व्यवसाय नियम, आर्किटेक्चर, सुरक्षा परिणाम. परंतु विलीनीकरण मंजूरी ही संघातील जबाबदार व्यक्तीची स्वाक्षरी आहे. सुरक्षा-गंभीर बदलांसाठी किमान एका सक्षम अभियंत्याद्वारे स्वतंत्र पुनरावलोकन अनिवार्य आहे.

टीप: AI ने “करण्याच्या” ऐवजी “तपासण्यासारख्या गोष्टी” म्हणून तयार केलेल्या निष्कर्षांची यादी वाचा. एकतर प्रत्येक आयटमची पडताळणी करा आणि लागू करा किंवा तुम्ही ती का पास केली ते एका वाक्यात लिहा; हे ट्रेस पुनरावलोकन ऑडिट करण्यायोग्य बनवते.

सामान्य चुका

• याचा अर्थ "AI दिसला, तो स्वच्छ आहे". खोट्या नकारात्मकतेमुळे हा आत्मविश्वासाचा खोटा अर्थ आहे.
• संदर्भ देत नाही. उद्देश आणि स्वीकृती निकषांशिवाय, मॉडेल केवळ वरवरच्या शैलीतील व्याख्या तयार करते.
• आंधळेपणाने खोटे सकारात्मक लागू करणे. मॉडेलच्या प्रत्येक चेतावणीचे निराकरण केल्याने चालणारा कोड खंडित होऊ शकतो.
• मॉडेलला व्यवसायाच्या नियमाबद्दल विचारणे. मॉडेलला नियम माहित नाही; त्याची पडताळणी करणे माणसावर अवलंबून आहे.
• हिंसाचारात भेदभाव करू नका. त्याच बॅगमध्ये एक गंभीर सुरक्षा शोध आणि नाव सूचना ठेवणे महत्त्वाचे काय आहे हे ओव्हरसॅड करते.

सारांशात

कोड रिव्ह्यूमध्ये एआय हे एक अथक पहिले फिल्टर आहे: ते शून्य/त्रुटी चुकते, एज केसेस आणि साध्या सुरक्षिततेचा वास घेतो; परंतु संदर्भ-आवश्यक दोष जसे की व्यवसायाचे नियम, आर्किटेक्चर आणि समांतरता यावर ते कमकुवत आहे आणि खोटे सकारात्मक आणि चुकीचे नकारात्मक दोन्ही निर्माण करते. श्रेणी आणि तीव्रतेनुसार निष्कर्षांची विनंती करा, मानवी बुद्धिमत्तेसह प्रत्येक फिल्टर करा, गंभीर मार्ग व्यक्तिचलितपणे सत्यापित करा. अनुमोदन नेहमीच जबाबदार अभियंत्याची स्वाक्षरी असते.

अर्ज कार्य

वास्तविक किंवा अलीकडील पीआर/डिफ निवडा. प्रथम, AI ला “उद्दिष्ट-केंद्रित, श्रेणी पुनरावलोकन” टेम्पलेटसह त्याचे पुनरावलोकन करा. निष्कर्ष टेबलमध्ये ठेवा आणि प्रत्येकासाठी ठरवा: खरे (मी सत्यापित केले), चुकीचे सकारात्मक (येथे माझे तर्क आहे), किंवा अंमलात आणायचे. मग स्वत: एक फेरफटका मारा आणि किमान एक गोष्ट (विशेषत: व्यवसाय नियम किंवा एज केस) शोधण्याचा प्रयत्न करा जी AI गहाळ आहे आणि ती लिहा.

चेकलिस्ट

• [ ] मी AI पुनरावलोकन प्रथम फिल्टर म्हणून वापरतो, समर्थन नाही.
• [ ] मी पुनरावलोकन प्रॉम्प्टमध्ये उद्देश आणि स्वीकृती निकष जोडतो.
• [ ] मी श्रेणीनुसार गोंगाटातून निष्कर्ष वेगळे करतो आणि ते मला हवे आहेत.
• [ ] मी जाणीवपूर्वक प्रत्येक शोधाची पुष्टी/असत्य सकारात्मक/लागू करण्यासाठी फिल्टर करतो.
• एक माणूस म्हणून, मी व्यवसायाचे नियम आणि वास्तुशास्त्रीय अनुपालन तपासतो.
• सुरक्षा-गंभीर बदलांसाठी मला पात्र अभियंत्याची मंजुरी आवश्यक आहे.