లాభాలు:
- వర్గాలు మరియు తీవ్రత ట్యాగ్లతో ప్రారంభ సమీక్ష ఫిల్టర్గా AIని ఉపయోగించగల సామర్థ్యం
- ధృవీకరణ/తప్పుడు పాజిటివ్/వర్తింపజేసేందుకు మానవ మనస్సుతో ఫలితాలను ఫిల్టర్ చేయగల సామర్థ్యం
- వ్యాపార నియమం, నిర్మాణం మరియు భద్రత-క్లిష్టమైన నిర్ణయాలపై మానవ ఆమోదం అవసరాలను అమలు చేయగల సామర్థ్యం
కోడ్ రివ్యూ అంటే డెవలపర్ రాసిన మార్పుని విలీనం చేసే ముందు వేరొకరు రివ్యూ చేస్తారు. మంచి సమీక్ష; ఇది బగ్లను ముందుగానే పట్టుకుంటుంది, సమాచారాన్ని పంచుకుంటుంది మరియు కోడ్ బేస్ను స్థిరంగా ఉంచుతుంది. కానీ సమీక్షలు అలసిపోతాయి, పరధ్యానానికి గురవుతాయి మరియు సమయ ఒత్తిడిలో ఉపరితలంగా మారతాయి. ఆర్టిఫిషియల్ ఇంటెలిజెన్స్ ఇక్కడ రెండు రెట్లు సహాయకం: ఇది మీరు సమీక్ష కోసం సమర్పించే మీ స్వంత కోడ్ను ముందుగా క్లీన్ చేయడానికి మరియు వేరొకరి PR (పుల్ రిక్వెస్ట్)ను పదునైన దృష్టితో పరిశీలించడానికి మిమ్మల్ని అనుమతిస్తుంది.
క్లిష్టమైన వ్యత్యాసం ఇది: AI వేగాన్ని పెంచుతుంది మరియు సమీక్షను మెరుగుపరుస్తుంది, అయితే ఇది ఆమోదం కోసం బాధ్యతను తీసుకోదు. "AI చూసింది, ఇది శుభ్రంగా ఉంది" అనే వాక్యం ఆమోదం కాదు. చివరి "విలీనం" నిర్ణయం కోడ్ మరియు సందర్భం తెలిసిన ఇంజనీర్పై ఆధారపడి ఉంటుంది.
సమీక్షలో AI మంచి మరియు చెడు ఏమిటి
దీని కోసం మంచిది: శూన్య తనిఖీ మిస్లు, రిసోర్స్ లీక్లు (ఫైల్/లింక్ తెరిచి ఉంది), గుర్తించబడని మినహాయింపులు, స్పష్టంగా తప్పు పరిస్థితులు (>= బదులుగా >), పేరు మార్చడం, రీడబిలిటీ, మిస్ ఎడ్జ్ కేస్, సాధారణ భద్రతా వాసనలు (SQL స్ట్రింగ్ కలయిక వంటివి), డూప్లికేట్ కోడ్ డిటెక్షన్.
బలహీనతలు: మీ వ్యాపార నియమాన్ని ఉల్లంఘించే లోతైన లోపాలు, అయితే సందర్భోచితంగా సరైన తర్కం, నిర్మాణ సమ్మతి, వాస్తవ పనితీరు అడ్డంకులు, సమ్మతి లోపాలు వంటి సందర్భం మరియు సమయం అవసరం. AI తప్పుడు పాజిటివ్లు (వాస్తవానికి సమస్య లేనిదాన్ని తప్పుగా భావించడం) మరియు తప్పుడు ప్రతికూలతలను (నిజమైన బగ్ను కోల్పోవడం) కూడా ఉత్పత్తి చేస్తుంది. అందువల్ల, దాని అవుట్పుట్ "జాగ్రత్త జాబితా", ఖచ్చితమైన తీర్పు కాదు.
జాగ్రత్త: AI "సమస్య లేదు" అని చెప్పినందున కోడ్ సరైనదని రుజువు చేయదు. తప్పుడు ప్రతికూలతలు నిశ్శబ్దంగా ఉంటాయి; అత్యంత ప్రమాదకరమైన తప్పులు సమీక్షలో ఎప్పుడూ ప్రస్తావించబడనివి.
క్రమబద్ధమైన సమీక్ష దశలు
- సందర్భం ఇవ్వండి. మార్పు యొక్క ఉద్దేశ్యం, సంబంధిత సమస్య మరియు అంగీకార ప్రమాణాలు ఏవైనా ఉంటే, ప్రాంప్ట్కు జోడించండి. పర్పస్లెస్ రివ్యూ ప్రయోజనం లేని వివరణను ఉత్పత్తి చేస్తుంది.
- దానిని వర్గాలుగా విభజించండి. కనుగొన్న వాటిని "బగ్/సెక్యూరిటీ/పర్ఫార్మెన్స్/రీడబిలిటీ/స్టైల్"గా వర్గీకరించమని మోడల్ని అడగండి; కాబట్టి మీరు శబ్దం నుండి క్లిష్టమైన వాటిని వేరు చేస్తారు.
- తీవ్రత లేబుల్ని అభ్యర్థించండి. ప్రతి అన్వేషణకు "అధిక/మధ్యస్థం/తక్కువ" రేటింగ్ ఇవ్వండి మరియు "కారణం" మరియు "సిఫార్సు చేయబడిన దిద్దుబాటు"ని చేర్చండి.
- మీ స్వంత కళ్ళతో దాన్ని ఫిల్టర్ చేయండి. ప్రతి అన్వేషణను మూల్యాంకనం చేయండి: ఇది వాస్తవమా (ధృవీకరించండి), ఇది తప్పుడు సానుకూలమా (జస్టిఫికేషన్ వ్రాయండి), ఏదైనా తప్పిపోయిందా (మీ స్వంత జ్ఞానాన్ని జోడించండి).
- క్లిష్టమైన మార్గాలను మాన్యువల్గా ధృవీకరించండి. AIపై ఆధారపడకుండా డబ్బు, గుర్తింపు, అధికారం మరియు డేటా తొలగింపుతో కూడిన మార్గాలను మీరే చదవండి మరియు అమలు చేయండి.
మూడు మినీ కేసులు
కేస్ 1 — సైలెంట్ శూన్య లోపం క్యాచ్ చేయబడింది. ఒక బృందం 380-లైన్ PRని AI ప్రీ-రివ్యూ చేసింది. బాహ్య సేవా ప్రతిస్పందన శూన్యంగా ఉండే విధంగా మోడల్ ఫ్లాగ్ చేయబడింది, కానీ కోడ్లో దీని కోసం ఎలాంటి తనిఖీలు చేయలేదు. మానవ సమీక్షకుడు ఈ మార్గాన్ని ధృవీకరించారు మరియు శూన్య తనిఖీని జోడించారు; ఇదే లోపం కారణంగా మునుపటి త్రైమాసికంలో ఉత్పత్తిలో 2 గంటల అంతరాయం ఏర్పడింది.
కేసు 2 - తప్పుడు సానుకూల తొలగింపు. AI లూప్లో "సాధ్యమైన పనితీరు సమస్య"ని ఫ్లాగ్ చేసింది. లూప్ గరిష్టంగా 5 మూలకాలతో మాత్రమే పని చేస్తుందని తెలుసుకుని సమీక్షకుడు దీనిని తప్పుడు పాజిటివ్గా మూసివేశారు (ఇది ఎన్యూమ్పై లూప్ అవుతుంది). సందర్భం తెలియని మోడల్ హెచ్చరించింది; సందర్భం తెలిసిన వ్యక్తి సరైన నిర్ణయం తీసుకున్నాడు.
కేస్ 3 — AI మిస్డ్ బిజినెస్ రూల్ ఎర్రర్. ప్రచార నియమం ప్రకారం తగ్గింపు ఖాతా గరిష్టంగా 30% ఉండాలి, కోడ్ 50% అనుమతించబడుతుంది. ఈ వాక్యనిర్మాణపరంగా ఖచ్చితమైన తార్కిక లోపాన్ని AI ఎప్పుడూ గమనించలేదు; ఎందుకంటే అతనికి నియమం తెలియదు. అంగీకార ప్రమాణాలను తెలిసిన ఉత్పత్తి యజమాని సమీక్షలో బగ్ క్యాచ్ చేయబడింది. పాఠం: వ్యాపార నియమ ధ్రువీకరణ అనేది మానవ పని.
నాలుగు కాపీ చేయగల టెంప్లేట్లు
పర్పస్-ఓరియెంటెడ్, వర్గీకరించబడిన సమీక్ష:
పాత్ర: ఖచ్చితమైన కోడ్ సమీక్షకుడు. మార్పు యొక్క ఉద్దేశ్యం: {{purpose / issue}}ఈ వ్యత్యాసాన్ని సమీక్షించండి. ఈ వర్గాల్లో ఫలితాలను అందించండి: [బగ్] [సెక్యూరిటీ] [పనితీరు] [రీడబిలిటీ] [స్టైల్]. ప్రతి అన్వేషణ కోసం: ఫైల్: అడ్డు వరుస, తీవ్రత(అధిక/మధ్యస్థం/తక్కువ), కారణం, సిఫార్సు చేసిన పరిష్కారం. మీకు ఖచ్చితంగా తెలియకపోతే "సాధ్యం" అని గుర్తు పెట్టండి. మీకు వ్యాపార నియమాలు తెలియవు; నియమాలు అవసరమయ్యే స్థలాల గురించి నన్ను అడగండి.{{diff}}
మీ స్వంత కోడ్ని సమీక్షించడానికి సిద్ధం కావడానికి:
PRని తెరవడానికి ముందు ఈ మార్పును సమీక్షించండి. దీని కోసం వెతకండి: శూన్య/బగ్ చెక్, రిసోర్స్ లీక్, ఎడ్జ్ కేస్, రహస్యం, పరీక్షించని శాఖ. ఫలితాలను ప్రాధాన్యత క్రమంలో జాబితా చేయండి; ప్రతిదానికి 1 లైన్ దిద్దుబాటును సూచించండి.{{code}}
ఎడ్జ్ కేసు వేట:
ఈ ఫంక్షన్ విచ్ఛిన్నమయ్యే ఇన్పుట్లు మరియు పరిస్థితులను జాబితా చేయండి: ఖాళీ, శూన్య, చాలా పెద్ద, ప్రతికూల, ఏకకాలిక కాల్, నెట్వర్క్ లోపం, పాక్షిక డేటా. ప్రతి సందర్భంలో, ఊహించిన ప్రవర్తన మరియు ప్రస్తుత కోడ్ ఏమి చేస్తుందో వ్రాయండి.{{function}}
భద్రతా సువాసన స్కానింగ్ (ప్రీ-స్క్రీనింగ్):
ఈ కోడ్లో సాధారణ భద్రతా వాసనల కోసం వెతకండి: SQL/కమాండ్ కన్కాటేనేషన్, అన్వాలిడేట్ చేయని ఇన్పుట్, ఇమ్యుటబుల్ ఎంబెడెడ్ సీక్రెట్, అసురక్షిత డీరియలైజ్, ప్రివిలేజ్ చెకింగ్ లేకపోవడం. కనుగొన్న వాటిని "నిర్దిష్ట / సంభావ్య / జ్ఞానం"గా విభజించండి. ఇది ప్రాథమిక స్క్రీనింగ్; ఇది ఖచ్చితమైన తీర్పు కాదు.{{code}}
బలహీనమైన ప్రాంప్ట్ / బలమైన ప్రాంప్ట్
బలహీనత: "ఈ PRలో పొరపాటు ఉందా?"
బలమైనది: "ప్రయోజనం: కార్ట్ మొత్తానికి కూపన్ తగ్గింపును జోడించండి (రాయితీ తప్పనిసరిగా 30% కంటే ఎక్కువ ఉండకూడదు - మీరు ఈ నియమాన్ని మీరే ధృవీకరించలేరు, కోడ్ గరిష్ట పరిమితిని విధించినట్లయితే నాకు చెప్పండి). వ్యత్యాసాన్ని పరిశీలించండి; వర్గం + తీవ్రత + సూచించిన దిద్దుబాటు ద్వారా ఫలితాలను ఇవ్వండి, ఖచ్చితంగా తెలియకపోతే 'సాధ్యం' అని గుర్తు పెట్టండి. [diff]"
బలమైన సంస్కరణ AI యొక్క ఉద్దేశం, వ్యాపార నియమం మరియు సరిహద్దును స్పష్టంగా పేర్కొంది; అందువల్ల, ఉపయోగకరమైన ఫలితాలు వస్తాయి మరియు మోడల్కు తెలియని ప్రాంతం స్పష్టంగా ఉంటుంది.
రకాన్ని కనుగొనడం
AI విశ్వసనీయత
మనిషి పాత్ర
శూన్య/ఎర్రర్ చెక్ లేదు
అధిక
ధృవీకరించండి మరియు దరఖాస్తు చేయండి
పఠనీయత/శైలి
అధిక
ప్రాధాన్యత ద్వారా ఎంచుకోండి
సాధారణ భద్రతా వాసన
మధ్యస్థ
ఖరారు చేయండి, వాహనంతో స్కాన్ చేయండి
వ్యాపార నియమాల సమ్మతి
తక్కువ
ఇది పూర్తిగా మానవుడు.
కాన్కరెన్సీ/ఆర్కిటెక్చర్
తక్కువ
నిపుణుల సమీక్ష అవసరం
AI సమీక్ష మానవ సమీక్షకు ప్రత్యామ్నాయం కాదు
AI సమీక్షను "మొదటి ఫిల్టర్"గా ఉంచండి: చౌకైన, వేగవంతమైన, అలసిపోని ప్రిలిమినరీ పాస్. ఈ ఫిల్టర్ మానవ సమీక్షకుల దృష్టిని అప్రధానమైన వివరాల నుండి (స్పేస్, పేరు) విముక్తి చేస్తుంది మరియు వ్యాపార నియమం, నిర్మాణం, భద్రతా ఫలితం వంటి నిజంగా ఆలోచించాల్సిన ప్రదేశాలకు మళ్లిస్తుంది. కానీ విలీన ఆమోదం అనేది జట్టులోని ఒక జవాబుదారీ వ్యక్తి యొక్క సంతకం. భద్రత-క్లిష్టమైన మార్పుల కోసం కనీసం ఒక సమర్థ ఇంజనీర్ ద్వారా స్వతంత్ర సమీక్ష తప్పనిసరి.
చిట్కా: AI "చేయవలసినవి" కాకుండా "తనిఖీ చేయవలసినవి"గా ఉత్పత్తి చేసే అన్వేషణల జాబితాను చదవండి. ప్రతి అంశాన్ని ధృవీకరించండి మరియు వర్తింపజేయండి లేదా మీరు దానిని ఎందుకు ఆమోదించారో ఒక వాక్యంలో వ్రాయండి; ఈ ట్రేస్ సమీక్షను తనిఖీ చేయదగినదిగా చేస్తుంది.
సాధారణ తప్పులు
- దీని అర్థం "AI చూసింది, శుభ్రంగా ఉంది". తప్పుడు ప్రతికూలతల కారణంగా ఇది తప్పుడు విశ్వాసం.
- సందర్భం ఇవ్వడం లేదు. ప్రయోజనం మరియు అంగీకార ప్రమాణాలు లేకుండా, మోడల్ కేవలం ఉపరితల శైలి వివరణలను మాత్రమే ఉత్పత్తి చేస్తుంది.
- గుడ్డిగా తప్పుడు పాజిటివ్లను ప్రయోగిస్తున్నారు. మోడల్ యొక్క ప్రతి హెచ్చరికను పరిష్కరించడం వలన నడుస్తున్న కోడ్ను విచ్ఛిన్నం చేయవచ్చు.
- బిజినెస్ రూల్ గురించి మోడల్ని అడుగుతున్నారు. మోడల్కు నియమం తెలియదు; దాన్ని వెరిఫై చేసుకోవడం మనిషికి ఇష్టం.
- హింస పట్ల వివక్ష చూపవద్దు. కీలకమైన భద్రతా అన్వేషణ మరియు పేరు సూచనను ఒకే బ్యాగ్లో ఉంచడం ముఖ్యమైన వాటిని కప్పివేస్తుంది.
సారాంశంలో
AI అనేది కోడ్ సమీక్షలో అలసిపోని మొదటి ఫిల్టర్: ఇది శూన్య/ఎర్రర్ మిస్లు, ఎడ్జ్ కేస్లను క్యాచ్ చేస్తుంది మరియు సాధారణ భద్రతా వాసనలు బాగా ఉంటాయి; కానీ ఇది వ్యాపార నియమం, నిర్మాణం మరియు సమ్మతి వంటి సందర్భ-అవసరమైన లోపాలపై బలహీనంగా ఉంది మరియు తప్పుడు పాజిటివ్లు మరియు తప్పుడు ప్రతికూలతలు రెండింటినీ ఉత్పత్తి చేస్తుంది. వర్గం మరియు తీవ్రత ఆధారంగా ఫలితాలను అభ్యర్థించండి, ప్రతి ఒక్కటి మానవ మేధస్సుతో ఫిల్టర్ చేయండి, క్లిష్టమైన మార్గాలను మాన్యువల్గా ధృవీకరించండి. ఆమోదం ఎల్లప్పుడూ జవాబుదారీ ఇంజనీర్ యొక్క సంతకం.
అప్లికేషన్ టాస్క్
నిజమైన లేదా ఇటీవలి PR/diffని ఎంచుకోండి. ముందుగా, AI దానిని "ఆబ్జెక్టివ్-ఓరియెంటెడ్, కేటగిరీ రివ్యూ" టెంప్లేట్తో సమీక్షించండి. కనుగొన్న వాటిని పట్టికలో ఉంచండి మరియు ప్రతి ఒక్కటి నిర్ణయించండి: నిజం (నేను ధృవీకరించాను), తప్పుడు పాజిటివ్ (ఇదిగో నా తార్కికం) లేదా అమలు చేయాలి. ఆపై మీరే ఒక పర్యటనలో పాల్గొనండి మరియు AI తప్పిపోయిన కనీసం ఒక విషయాన్ని (ముఖ్యంగా వ్యాపార నియమం లేదా అంచు కేసు) కనుగొని దానిని వ్రాయడానికి ప్రయత్నించండి.
చెక్లిస్ట్
- [ ] నేను AI సమీక్షను మొదటి ఫిల్టర్గా ఉపయోగిస్తాను, ఆమోదం కాదు.
- [ ] నేను సమీక్ష ప్రాంప్ట్కు ప్రయోజనం మరియు అంగీకార ప్రమాణాలను జోడిస్తాను.
- [ ] నేను కనుగొన్న వాటిని శబ్దం నుండి వర్గం వారీగా వేరు చేస్తున్నాను మరియు వాటిని గట్టిగా కోరుకుంటున్నాను.
- [ ] నేను స్పృహతో ప్రతి అన్వేషణను నిర్ధారించడానికి/తప్పు సానుకూలంగా/వర్తింపజేయడానికి ఫిల్టర్ చేస్తాను.
- [ ] ఒక మనిషిగా, నేను వ్యాపార నియమాన్ని మరియు నిర్మాణ సమ్మతిని తనిఖీ చేస్తాను.
- [ ] భద్రత-క్లిష్టమైన మార్పుల కోసం నాకు అర్హత కలిగిన ఇంజనీర్ నుండి ఆమోదం అవసరం.