{\ tf1\ansi\ansicpg1252\cocoartf2822 \cocoatextscaling0\cocoaplatform0{\fonttbl\f0\fnil\fcharset0 Menlo-Regular;\f1\fnil\fcharset0 Menlo-Bold;} {\colortbl;\ ed255\green255\blue255;\ ed193\green193\blue193;\ ed24\green24\blue24;\ ed85\green129\blue224; \ ed70\green137\blue204;} {\*\expandedcolortbl;;\cssrgb\c80000\c80000\c80000;\cssrgb\c12157\c12157\c12157;\cssrgb\c40392\c58824\c90196; \cssrgb\c33725\c61176\c83922;} \margl1440\margr1440\vieww11520\viewh8400\viewkind0 \deftab720 \pard\pardeftab720\partightenfactor0
\f0\fs24 \cf2 \cb3 \expnd0\expndtw0\kerning0 \outl0\strokewidth0 \strokec2 Perform a comprehensive code review current staged changes as a senior developer would, focusing on production readiness. Specifically:\cb1 \ \ \pard\pardeftab720\partightenfactor0 \cf4 \cb3 \strokec4 1.\cf2 \strokec2 \f1\b \cf5 \strokec5 Error Handling & Panic Prevention \f0\b0 \cf2 \strokec2 :\cb1 \ \pard\pardeftab720\partightenfactor0 \cf2 \cb3 \cf4 \strokec4 -\cf2 \strokec2 Review all error paths and ensure none can cause panics\cb1 \ \cb3 \cf4 \strokec4 -\cf2 \strokec2 Check for potential nil pointer dereferences\cb1 \ \cb3 \cf4 \strokec4 -\cf2 \strokec2 Verify that all file operations handle errors gracefully\cb1 \ \cb3 \cf4 \strokec4 -\cf2 \strokec2 Look for any unchecked type assertions or array/slice access that could panic\cb1 \ \ \pard\pardeftab720\partightenfactor0 \cf4 \cb3 \strokec4 2.\cf2 \strokec2 \f1\b \cf5 \strokec5 Production Readiness \f0\b0 \cf2 \strokec2 :\cb1 \ \pard\pardeftab720\partightenfactor0 \cf2 \cb3 \cf4 \strokec4 -\cf2 \strokec2 Verify logging levels are appropriate (debug vs info vs warn vs error)\cb1 \ \cb3 \cf4 \strokec4 -\cf2 \strokec2 Check if error messages provide sufficient context for debugging\cb1 \ \cb3 \cf4 \strokec4 -\cf2 \strokec2 Ensure the code handles edge cases\cb1 \ \cb3 \cf4 \strokec4 -\cf2 \strokec2 Consider if there are any race conditions or concurrency issues\cb1 \ \ \pard\pardeftab720\partightenfactor0 \cf4 \cb3 \strokec4 3.\cf2 \strokec2 \f1\b \cf5 \strokec5 Code Quality \f0\b0 \cf2 \strokec2 :\cb1 \ \pard\pardeftab720\partightenfactor0 \cf2 \cb3 \cf4 \strokec4 -\cf2 \strokec2 Look for any code duplication or opportunities for refactoring\cb1 \ \cb3 \cf4 \strokec4 -\cf2 \strokec2 Verify consistent error handling patterns with the rest of the codebase\cb1 \ \cb3 \cf4 \strokec4 -\cf2 \strokec2 Check if variable names are clear and follow Go conventions\cb1 \ \cb3 \cf4 \strokec4 -\cf2 \strokec2 Ensure comments are accurate and helpful\cb1 \ \ \pard\pardeftab720\partightenfactor0 \cf4 \cb3 \strokec4 4.\cf2 \strokec2 \f1\b \cf5 \strokec5 Unit Test \f0\b0 \cf2 \strokec2 :\cb1 \ \pard\pardeftab720\partightenfactor0 \cf2 \cb3 \cf4 \strokec4 -\cf2 \strokec2 Add a comprehensive unit test for the changes made\cb1 \ \ \cb3 Provide specific code fixes for any issues found, and include the complete unit and component test implementation.\cb1 \ }