I have now started working on mmanon. As discussed in my previous post, before looking at any new features, I decided to give the AI code review a go.
The results surprised me positively, as they were really useful. The AI found some concurrency issues, specifically regarding hash maps that different workers can access. These were not properly locked.
This is something I missed previously, likely due to a lack of experience when I originally wrote the code. To be honest, I probably would have missed this now as well, simply because I was not actively looking for these types of issues. This shows me that the review tool is really useful, especially since it seems to find mistakes that are easy for a human to overlook.
After the discovery, I set the code agent to the task of fixing this issue. It wasn’t a perfect process immediately. At first, there were a couple of issues where the agent called an incorrect function and wanted to perform unnecessary checks. However, these were quickly fixed. In the end, I was able to implement a good PR that has since been merged.
Overall, this means that AI enabled me to find an important issue in mmanon and fix it. Now, the next step is finally implementing the features I planned, that is unless other things turn up that seem more urgent.