[7.0] Minecraft extension lacking IDE support

[PaintNinja]

The following code has no IDE support currently:

minecraft {
    mappings channel: 'official', version: '1.21.8'
    
    runs {
        server {
            mainClass = 'net.minecraft.server.Main'

            args 'nogui'
        }
    }
}

After investigating further, I've determined that it's due to a visibility issue - IntelliJ is picking up the internal implementations which are not publicly accessible:

A workaround is to manually cast to the correct type and wrap in a tap, like so:

minecraft {  ((MinecraftExtension.ForProject) it).tap {
    mappings channel: 'official', version: '1.21.8'
    
    runs {
        server {
            mainClass = 'net.minecraft.server.Main'

            args 'nogui'
        }
    }
}}

As for the yellow highlight for the runs block, that's because IntelliJ is a dumbass and thinks Closure<Void> means you expect users to explicitly return null - likely a bug in their Groovy support. I remember advising you to add the return type and forgot about this edge-case, my bad. The fix is to replace Closure<Void> with Closure<?> or have a raw Closure when you don't care about the return type (such as void).

Interestingly, the fg extension does have IDE support. Possibly a difference between how the fg and minecraft extensions are registered?

[Jonathing]

This is actually what I've been working on with other projects such as GradleUtils 3.0 already. The problem is that these types do not implement the HasPublicType interface, which tells the object handler that it should use that type for objects that are handled in the DSL.

[Jonathing]

As I've shown in Discord today, I am doing a slight re-structure of the types in FG7. Hopefully the DSL linter will like them better once they are able to actually decorate the classes, instead of relying on finalized classes. I'm not entirely sure what causes the linter to like one type over the other (ForgeGradleExtension -> MinecraftExtension.ForProject), but that should hopefully give it some more leeway.

[Jonathing]

'Sup. What I said before? Total load of horseshit. What actually happened was that IntelliJ didn't like the fact that the ForProject type was an inner interface. Moving it to the upper level made it work as intended.

I love my life.

[PaintNinja]

lol... nice. New FG7 beta build soon?

[Jonathing]

Is this still an issue? In my testing, this works fine unless the plugin is first applied to settings.gradle.

[PaintNinja]

This has since been fixed, yes. Don't remember the exact build, but definitely sorted in beta 33.