Using 2 loader mods Neoforge/Fabric

[linkiez]

I tried to install Sinytra Connector to use both neoforge and fabric but the script cant download both.

[itzg]

Posting the intended example would have helped. (Copying over from #3718)

environment:
  TYPE: NEOFORGE
  VERSION: "1.20.4"
  MODRINTH_PROJECTS: |
    connector
    fabric-api
    sodium
    lithium
    iris

Since there's already capability to qualify each MODRINTH_PROJECTS entry

https://docker-minecraft-server.readthedocs.io/en/latest/mods-and-plugins/modrinth/#examples

It seems like a reasonable, general purpose enhancement to allow for specifying an alternate modloader per project entry.

[linkiez]

but i liked you idea of putting a tag in front of mod name

[itzg]

Yes, and sorry to the hurried pr, im new to contributing open source

No problem at all. I appreciate the feature idea and taking the initiative.

General advice: be very cautious about using AI generated PRs. It can quickly get over ambitious and inaccurate. Lean on AI to get you past sticking points, but ultimately the code contributed needs to be your thoughts and solution.

[itzg]

I just now took a look at the image helper code and the loader prefix is already supported:

https://github.com/itzg/mc-image-helper/blob/226b8b781dc21a9bb0f9778caabdd4ae32d3cae1/src/main/java/me/itzg/helpers/modrinth/ModrinthCommand.java#L51-L58

The docs just need to be updated to describe that as an option.

It was added in itzg/mc-image-helper#571 for itzg/mc-image-helper#564

[itzg]

Updating documentation in #3720

[itzg]

And docs have been updated

https://docker-minecraft-server.readthedocs.io/en/latest/mods-and-plugins/modrinth/#usage