imron
Joined 13 years ago
Joined 13 years ago
Even with short interactions though, the effort in reviewing adds up and I ended up building a tool to help with it: https://github.com/imron/crt
I can review in the tool, and it remembers which files I have seen/approved since a given commit, then I can leave comments against source code and have an agent pick up those comments over mcp so it has context when making corrections.
dict[str, Any] is a type, and sometimes it is the right type to specify, but most of the time it's used by someone who wants to shut up the type checker rather than think about the right type.
Move errors from runtime to compile time as much as possible.
"You're trying to kidnap what I've rightfully stolen."
Hah, yeah even when you put it in AGENTS.md or a skill.. constantly having to remind it.. "what does AGENTS.md" say about doing that?".. Thinking.. Thinking.. "Oh, it says I should never do that, I'll remember that next time.."
Next session - same thing.
Grok has its own feel too. It's not as bad as Claude, but one of the things that bugs me is that it is far too terse.
It regularly seems to come up with terms and descriptions for things in its chain of reasoning and then uses these terms in its output assuming you understand what it's talking about.
I find I often have to ask it to re-explain what it means.
The Claudish is dead. Long live the Claudish.