refactor: convert TriggerProfile to record and cleanup imports

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: JobsonMarinho

src/main/java/me/byteful/plugin/leveltools/api/trigger/TriggerRegistry.java

@@ -1,13 +1,6 @@
 package me.byteful.plugin.leveltools.api.trigger;
 
-import me.byteful.plugin.leveltools.api.trigger.impl.BlockBreakTrigger;
-import me.byteful.plugin.leveltools.api.trigger.impl.ConsumeTrigger;
-import me.byteful.plugin.leveltools.api.trigger.impl.EntityKillTrigger;
-import me.byteful.plugin.leveltools.api.trigger.impl.FarmingTrigger;
-import me.byteful.plugin.leveltools.api.trigger.impl.FishingTrigger;
-import me.byteful.plugin.leveltools.api.trigger.impl.ArmorDurabilityTrigger;
-import me.byteful.plugin.leveltools.api.trigger.impl.LeftClickTrigger;
-import me.byteful.plugin.leveltools.api.trigger.impl.RightClickTrigger;
+import me.byteful.plugin.leveltools.api.trigger.impl.*;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 
@@ -19,7 +12,7 @@ public final class TriggerRegistry {
 
     public void register(@NotNull Trigger trigger) {
         String id = TriggerIds.normalize(trigger.getTriggerId());
-        if (id.isEmpty()) {
+        if (id == null || id.isEmpty()) {
             throw new IllegalArgumentException("Trigger ID cannot be null or empty");
         }
         triggers.put(id, trigger);

src/main/java/me/byteful/plugin/leveltools/api/trigger/impl/LeftClickTrigger.java

@@ -19,7 +19,7 @@ public String getTriggerId() {
     @Override
     public boolean canHandle(@NotNull TriggerContext context) {
         TriggerProfile profile = context.getTriggerProfile();
-        TriggerSettings settings = profile.getSettings();
+        TriggerSettings settings = profile.settings();
         TriggerSettings.ClickMode clickMode = settings.getClickMode();
 
         if (clickMode == null || clickMode == TriggerSettings.ClickMode.ANY) {

src/main/java/me/byteful/plugin/leveltools/api/trigger/impl/RightClickTrigger.java

@@ -19,7 +19,7 @@ public String getTriggerId() {
     @Override
     public boolean canHandle(@NotNull TriggerContext context) {
         TriggerProfile profile = context.getTriggerProfile();
-        TriggerSettings settings = profile.getSettings();
+        TriggerSettings settings = profile.settings();
         TriggerSettings.ClickMode clickMode = settings.getClickMode();
 
         if (clickMode == null || clickMode == TriggerSettings.ClickMode.ANY) {

src/main/java/me/byteful/plugin/leveltools/listeners/TriggerListener.java

@@ -192,11 +192,11 @@ private void handleTrigger(
 
         double totalModifier = 0.0;
         for (TriggerProfile triggerProfile : triggerProfiles) {
-            if (!triggerId.equals(triggerProfile.getTriggerId())) {
+            if (!triggerId.equals(triggerProfile.triggerId())) {
                 continue;
             }
 
-            if (!triggerProfile.getSlotFilter().matches(slot)) {
+            if (!triggerProfile.slotFilter().matches(slot)) {
                 continue;
             }
 

src/main/java/me/byteful/plugin/leveltools/profile/ProfileManager.java

@@ -134,7 +134,7 @@ public void clear() {
 
     @NotNull
     public RegistrationResult registerTriggerProfile(@NotNull TriggerProfile profile) {
-        String id = profile.getId();
+        String id = profile.id();
         if (triggerProfiles.containsKey(id)) {
             return RegistrationResult.alreadyExists(id, "trigger");
         }

src/main/java/me/byteful/plugin/leveltools/profile/trigger/TriggerProfile.java

@@ -3,32 +3,13 @@
 import me.byteful.plugin.leveltools.api.trigger.TriggerIds;
 import org.jetbrains.annotations.NotNull;
 
-public final class TriggerProfile {
-    private final String id;
-    private final String triggerId;
-    private final XpModifierConfig xpModifier;
-    private final TriggerFilter filter;
-    private final TriggerSlotFilter slotFilter;
-    private final TriggerSettings settings;
-
-    public TriggerProfile(
-            @NotNull String id,
-            @NotNull String triggerId,
-            @NotNull XpModifierConfig xpModifier,
-            @NotNull TriggerFilter filter,
-            @NotNull TriggerSettings settings
-    ) {
+public record TriggerProfile(String id, String triggerId, XpModifierConfig xpModifier, TriggerFilter filter,
+                             TriggerSlotFilter slotFilter, TriggerSettings settings) {
+    public TriggerProfile(@NotNull String id, @NotNull String triggerId, @NotNull XpModifierConfig xpModifier, @NotNull TriggerFilter filter, @NotNull TriggerSettings settings) {
         this(id, triggerId, xpModifier, filter, TriggerSlotFilter.all(), settings);
     }
 
-    public TriggerProfile(
-            @NotNull String id,
-            @NotNull String triggerId,
-            @NotNull XpModifierConfig xpModifier,
-            @NotNull TriggerFilter filter,
-            @NotNull TriggerSlotFilter slotFilter,
-            @NotNull TriggerSettings settings
-    ) {
+    public TriggerProfile(@NotNull String id, @NotNull String triggerId, @NotNull XpModifierConfig xpModifier, @NotNull TriggerFilter filter, @NotNull TriggerSlotFilter slotFilter, @NotNull TriggerSettings settings) {
         this.id = id;
         this.triggerId = TriggerIds.normalize(triggerId);
         this.xpModifier = xpModifier;
@@ -41,33 +22,39 @@ public static Builder builder(@NotNull String id) {
         return new Builder(id);
     }
 
+    @Override
     @NotNull
-    public String getId() {
+    public String id() {
         return id;
     }
 
+    @Override
     @NotNull
-    public String getTriggerId() {
+    public String triggerId() {
         return triggerId;
     }
 
+    @Override
     @NotNull
-    public XpModifierConfig getXpModifier() {
+    public XpModifierConfig xpModifier() {
         return xpModifier;
     }
 
+    @Override
     @NotNull
-    public TriggerFilter getFilter() {
+    public TriggerFilter filter() {
         return filter;
     }
 
+    @Override
     @NotNull
-    public TriggerSlotFilter getSlotFilter() {
+    public TriggerSlotFilter slotFilter() {
         return slotFilter;
     }
 
+    @Override
     @NotNull
-    public TriggerSettings getSettings() {
+    public TriggerSettings settings() {
         return settings;
     }