Warto pamiętać, że jak jest dużo pól w klasie to warto się zastanowić nad podzieleniem tej klasy na mniejsze (tak radzi wujek Bob).

Oczywiście zawsze jest takie ryzyko, że ktoś będzie przekazywał builder jako parametr w różne miejsca. Ale to już kwestia niezrozumienia po co builder się stosuje i po co stosuje się niemutowalne obiekty, tworzone za pomocą takiego buildera.

Builder taki można napisać jeszcze inaczej:

public final class PersonAlt {
    private final String name;
    private final int age;

    private PersonAlt(String name, int age) {
        this.name = name;
        this.age = age;
    }

    public static Builder builder() {
        return new Builder();
    }

    public static class Builder {
        private String name;
        private int age;

        public Builder name(String name) {
            this.name = name;
            return this;
        }

        public Builder age(int age) {
            this.age = age;
            return this;
        }

        public PersonAlt build() {
            return new PersonAlt(name, age);
        }
    }
    // ... getters ...
}

I teraz masz private final, ale jest to trochę mniej estetyczne bo musisz używać konstruktora obiektu (jak masz dużo pól w klasie, to też i będzie dużo parametrów w konstruktorze). Ale problem z przekazywaniem buildera pozostanie.

Co do public final to nie widziałem, żeby ktoś tak robił. Wszyscy trzymają się raczej konwencji getterów i dostępu przez nie do pól.

Druga sprawa – pola chyba nie muszą być prywatne. Co z public final?