If we're going to make the file types to ignore an option (and I'm okay with that, although I think it may be unnecessary, since we should assume that nobody ever wants to view these files in text mode) there's no reason this shouldn't be an array instead of a CSV. We have JSON, we should use it instead of having to split() this string every time. I had another comment, but I can't remember what it was right now.