minor: internal language feature cleanup

[MukjepScarlet]

1. use String literal as FormattedCharSequence (skip re-order) instead of Component
2. use Java comparator functions (indy) instead of Kotlin crossinline functions (new class + new instance, by 2.4.20)
3. NOOP suppresses

[liquidbounce-support-bot]

Thanks for the cleanup — the comparator conversions look behaviour-preserving, and a couple of small things stood out while reading the diff:

• ModuleBlockIn.kt and ModuleBlockTrap.kt now use Vec3i.BY_Y / Vec3i.BY_Y_DESC, but the net.ccbluex.liquidbounce.utils.math.BY_Y / BY_Y_DESC imports were left in place. Both are unused now and should be flagged by detekt's UnusedImports.
• In ModuleBlockTrap, comparingDouble { -it.distToCenterSqr(...) } inverts by negation; comparingDouble { it.distToCenterSqr(...) }.reversed() states the intent more directly and matters if two positions ever have equal distance (tie order differs between the two).
• internal constructor on KAsyncCommand narrows the public surface — fine if nothing outside the module instantiates it (add-ons/scripts included), just worth confirming.

No docs impact expected for an internal cleanup like this.

---

🤖 Automated support reply — a human maintainer will review if this doesn't help.