Why the `TryGetValue` call here? We're already asserting that `views` knows the `filepath` key.

Wouldn't `viewSettings = views[filepath]` be enough?

Moreover, this would remove the braces and reduce the nesting :wink: