Not a merge-blocker, and I know this was already written this way, but I hate to see names of built-in types like "str" used as variable names, even if technically they're not reserved words.