Exercism has a really good summary of what great code reviews look like: https://tinyletter.com/exercism/letters/exercism-behind-the-scenes-unhelpful-insulting-and-rude-feedback-antipatterns
miniblog.
Related Posts
I've realised that AGENTS.md / CLAUDE.md solve two distinct problems:
* It tells the LLM the conventions of the project, avoiding follow-up prompts.
* It precomputes a summary of the project, saving a bunch of searches at the start of a session.
> try wrapping the expression in `summary::FileContent::Text`
This rust-analyzer feature amazed me. I'd changed an API from taking a &str to a Text(String) and it offered a quick fix to correct the call sites!
Excellent summary on the problems that tree-sitter solves, how it differs from LSP, and why it's such a great fit for editors: https://www.masteringemacs.org/article/tree-sitter-complications-of-parsing-languages
I'm regularly impressed by how many parsers are available and how accurate they are.