Definitely violates the obviousness principle. I only spent about 2 minutes checking this out, but didn't notice that `BaseAgentMode` inherits from `dict`.

I liked the way you had it with the command map before.