Skip to content

Add NeoForge as a loader status in /hs inv UI #38

@Th0rgal

Description

@Th0rgal

Problem

The /hs inv inventory UI shows Fabric and Forge as dedicated loader status lines (with green/red true/false), but NeoForge is not shown — even though it's fully supported as a ForgeClientType.NEOFORGE in the detection system.

Users have to add a custom generic check to see NeoForge status, and even then it appears as a regular detected mod rather than a loader status.

Root cause: CommandsManager.java lines 171-191 hardcode only "fabric" and "forge" as special loader status lines. There's no equivalent for "neoforge".

Suggested approach

Add NeoForge alongside Fabric and Forge in the loader status section of the inventory lore:

boolean hasNeoForge = hackedPlayer.getGenericChecks().contains("neoforge");

// In the lore section:
lore.add(toLegacy(Component.text("NeoForge: ", NamedTextColor.GOLD)
    .append(hasNeoForge
        ? Component.text("true", NamedTextColor.GREEN)
        : Component.text("false", NamedTextColor.RED))));

Also update the alwaysDisplayedCount logic (line 175-177) and the detectedChecks filter (line 168) to exclude "neoforge" from the regular checks section.

Relevant files

  • hackedserver-spigot/src/main/java/org/hackedserver/spigot/commands/CommandsManager.java (lines 143-217)

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions