review-firmware

用途：审查嵌入式 C/C++ 固件代码变更，关注最佳实践。调用时传入文件或提交（$ARGUMENTS）；未指定则审查最近一次变更，可先用 git diff --stat HEAD~1 查看改动范围。\n\n主要能力（按检查清单）：\n内存安全：缓冲区溢出与数组越界、空指针检查、内存泄漏（malloc 未 free）、函数栈使用。\n资源管理：资源是否释放、GPIO 初始化/反初始化、信号量/互斥量死锁风险、中断处理是否精简。\nESP-IDF 实践：任务栈大小、FreeRTOS 队列/信号量用法、临界区阻塞调用、WiFi/网络错误处理。\n代码质量：命名是否清晰、魔数应改为 #define 或 const、重复代码、错误码传递。\n安全：硬编码凭据（应放 Kconfig/NVS）、输入校验、使用 snprintf 而非 sprintf 等不安全字符串函数、证书校验。\n\n输出：对发现的问题给出具体行号和修改建议。\n\n限制：资料未提供安装命令、运行环境、支持的芯片或框架版本、是否收费等信息；该技能面向嵌入式固件与 ESP-IDF/FreeRTOS 场景，通用性有限。