AccessTransformer can narrow access in dev

[FiniteReality]

I was copy+pasting a few lines and accidentally had this in my AT file:

protected net.minecraft.client.renderer.entity.EntityRenderDispatcher options

This file is public by default, since it's used in DebugRenderer:

if (camera.isInitialized() && minecraft.getEntityRenderDispatcher().options != null) {

However, AccessTransformer seems to have made this protected, which causes Neoform to fail:

/home/dev/Development/Gander/rendering/build/neoForm/neoFormJoined1.20.6-20240429.153634/steps/unzipSources/unpacked/net/minecraft/client/renderer/debug/DebugRenderer.java:249: error: options has protected access in EntityRenderDispatcher
        if (camera.isInitialized() && minecraft.getEntityRenderDispatcher().options != null) {
                                                                           ^

[lukebemish]

NG should use JST for ATs now, as I understand it -- meaning this is likely a flaw on the logic on the JST end, which does not seem to take into account the existing modifier: https://github.com/neoforged/JavaSourceTransformer/blob/main/accesstransformers/src/main/java/net/neoforged/jst/accesstransformers/ApplyATsVisitor.java#L124-L139

[Matyrobbrt]

This is indeed a JST issue, as it currently doesn't exactly have error reporting.