Does it make sense to change this to an `ImmutableList`?