refactor: Clean up codebase and remove Guava dependency

- Remove unused Guava dependency from build configuration
- Simplify Javadoc comments to be more concise throughout
- Remove verbose inline comments that don't add value
- Preserve TODO comments for future implementation tasks

Author: GhostTypes

build.gradle.kts

@@ -37,10 +37,6 @@ dependencies {
     modImplementation(libs.gson)
     include(libs.gson)
 
-    // Guava for caching
-    modImplementation(libs.guava)
-    include(libs.guava)
-
     // Testing
     testImplementation(libs.junitApi)
     testRuntimeOnly(libs.junitEngine)

gradle/libs.versions.toml

@@ -16,7 +16,6 @@ meteor = "1.21.10-SNAPSHOT"
 junit = "5.10.2"
 gson = "2.11.0"
 okhttp = "4.12.0"
-guava = "33.5.0-jre"
 
 [libraries]
 # Fabric base
@@ -30,7 +29,6 @@ meteor-client = { module = "meteordevelopment:meteor-client", version.ref = "met
 # Dependencies for addon functionality
 gson = { module = "com.google.code.gson:gson", version.ref = "gson" }
 okhttp = { module = "com.squareup.okhttp3:okhttp", version.ref = "okhttp" }
-guava = { module = "com.google.guava:guava", version.ref = "guava" }
 
 # Testing
 junitApi = { module = "org.junit.jupiter:junit-jupiter-api", version.ref = "junit" }

src/main/java/com/cope/meteoraddons/MeteorAddonsAddon.java

@@ -11,12 +11,8 @@
 import org.slf4j.LoggerFactory;
 
 /**
- * Meteor Addons Addon - Entry Point
- *
- * Enables browsing, installing, and updating Meteor Client addons directly from within the client.
- * Fetches addon metadata from the meteor-addon-scanner repository.
- *
- * @author Cope
+ * Entry point for Meteor Addons addon.
+ * Enables browsing, installing, and updating Meteor Client addons from within the client.
  */
 public class MeteorAddonsAddon extends MeteorAddon {
     public static final Logger LOG = LoggerFactory.getLogger("Meteor Addons");
@@ -25,20 +21,16 @@ public class MeteorAddonsAddon extends MeteorAddon {
     public void onInitialize() {
         LOG.info("Initializing Meteor Addons Addon");
 
-        // Initialize IconPreloadSystem (handles icon lifecycle)
         IconPreloadSystem iconPreloadSystem = new IconPreloadSystem();
         Systems.add(iconPreloadSystem);
         LOG.info("IconPreloadSystem registered");
 
-        // Initialize AddonManager system (manages addon state, downloads, updates)
         Systems.add(new AddonManager());
         LOG.info("AddonManager system initialized");
 
-        // Register Addons tab in Meteor GUI
         Tabs.add(new AddonsTab());
         LOG.info("Addons tab registered");
 
-        // Start fetching addon metadata on a background thread
         AddonManager.get().init();
         LOG.info("Started fetching addon metadata");
 
@@ -47,7 +39,6 @@ public void onInitialize() {
 
     @Override
     public void onRegisterCategories() {
-        // No custom module categories needed for this addon
     }
 
     @Override

src/main/java/com/cope/meteoraddons/addons/Addon.java

@@ -6,58 +6,17 @@
 import java.util.Optional;
 
 /**
- * Unified interface for all addon types (online, installed, etc).
- * Provides common methods for displaying and managing addons.
+ * Unified interface for addon types (online, installed).
  */
 public interface Addon {
-    /**
-     * Get the display name of the addon.
-     */
     String getName();
-
-    /**
-     * Get the unique identifier of the addon.
-     */
     String getId();
-
-    /**
-     * Get the addon description.
-     */
     Optional<String> getDescription();
-
-    /**
-     * Get the list of addon authors.
-     */
     List<String> getAuthors();
-
-    /**
-     * Get the addon version.
-     */
     String getVersion();
-
-    /**
-     * Get an input stream for the addon icon.
-     * May download remotely or read from local file.
-     */
     Optional<InputStream> getIconStream() throws IOException;
-
-    /**
-     * Get the GitHub repository URL.
-     */
     Optional<String> getGithubUrl();
-
-    /**
-     * Get the Discord server URL.
-     */
     Optional<String> getDiscordUrl();
-
-    /**
-     * Get the homepage URL.
-     */
     Optional<String> getHomepageUrl();
-
-    /**
-     * Check if this addon is currently installed.
-     */
     boolean isInstalled();
 }

src/main/java/com/cope/meteoraddons/addons/InstalledAddon.java

@@ -11,8 +11,7 @@
 import java.util.stream.Collectors;
 
 /**
- * Represents an installed Meteor addon loaded by Fabric.
- * Wraps Fabric's ModContainer to provide addon interface.
+ * Installed Meteor addon wrapper for Fabric's ModContainer.
  */
 public class InstalledAddon implements Addon {
     private final ModContainer modContainer;
@@ -52,7 +51,6 @@ public String getVersion() {
 
     @Override
     public Optional<InputStream> getIconStream() throws IOException {
-        // Try multiple icon paths in order
         String[] iconPaths = {
             metadata.getIconPath(128).orElse(null),
             metadata.getIconPath(64).orElse(null),
@@ -70,7 +68,6 @@ public Optional<InputStream> getIconStream() throws IOException {
                     InputStream stream = java.nio.file.Files.newInputStream(pathOpt.get());
                     return Optional.of(stream);
                 } catch (IOException e) {
-                    // Try next path
                 }
             }
         }
@@ -80,15 +77,13 @@ public Optional<InputStream> getIconStream() throws IOException {
 
     @Override
     public Optional<String> getGithubUrl() {
-        // Try to find GitHub URL in contact info
         return metadata.getContact().get("sources")
             .or(() -> metadata.getContact().get("homepage"))
             .filter(url -> url.contains("github.com"));
     }
 
     @Override
     public Optional<String> getDiscordUrl() {
-        // Try to find Discord URL in contact info
         return metadata.getContact().get("discord")
             .or(() -> metadata.getContact().get("issues"))
             .filter(url -> url.contains("discord"));
@@ -101,7 +96,7 @@ public Optional<String> getHomepageUrl() {
 
     @Override
     public boolean isInstalled() {
-        return true; // Always true for installed addons
+        return true;
     }
 
     public ModContainer getModContainer() {

src/main/java/com/cope/meteoraddons/addons/OnlineAddon.java

@@ -11,8 +11,7 @@
 import java.util.Optional;
 
 /**
- * Wrapper for online addon metadata from the addon scanner.
- * Delegates to AddonMetadata and adds installed status checking.
+ * Online addon wrapper for scanner metadata.
  */
 public class OnlineAddon implements Addon {
     private final AddonMetadata metadata;
@@ -32,7 +31,6 @@ public String getName() {
 
     @Override
     public String getId() {
-        // Use name as ID for online addons (metadata doesn't have a separate ID field)
         return metadata.name.toLowerCase().replace(" ", "-");
     }
 
@@ -49,7 +47,6 @@ public List<String> getAuthors() {
 
     @Override
     public String getVersion() {
-        // Try to extract version from download URL or use mc_version
         if (metadata.mc_version != null) {
             return metadata.mc_version;
         }

src/main/java/com/cope/meteoraddons/gui/screens/BrowseAddonsScreen.java

@@ -21,9 +21,6 @@
 import static meteordevelopment.meteorclient.MeteorClient.mc;
 import static meteordevelopment.meteorclient.utils.Utils.getWindowWidth;
 
-/**
- * Screen for browsing and downloading online addons.
- */
 public class BrowseAddonsScreen extends WindowScreen {
     private static final int CARDS_PER_ROW = 4;
     private boolean isGridView = false;
@@ -36,14 +33,12 @@ public BrowseAddonsScreen(GuiTheme theme) {
     public void initWidgets() {
         AddonManager manager = AddonManager.get();
 
-        // Header
         WHorizontalList header = add(theme.horizontalList()).expandX().widget();
         header.add(theme.label("Available Addons")).expandX();
         header.add(theme.label("MC " + VersionUtil.getCurrentMinecraftVersion()).color(theme.textSecondaryColor()));
 
         add(theme.horizontalSeparator()).expandX();
 
-        // Loading / Error
         if (manager.isLoading()) {
             add(theme.label("Loading addons...")).centerX();
             return;
@@ -65,25 +60,18 @@ public void initWidgets() {
             return;
         }
 
-        // Toolbar
         WHorizontalList toolbar = add(theme.horizontalList()).expandX().widget();
         toolbar.add(theme.label(addons.size() + " addons"));
-        
-        // Spacer using empty expanding label or just layout properties
-        // Meteor doesn't have a spacer widget usually, but expanding a label works or expanding the container.
-        // We'll rely on the previous label expanding? No, WLabel auto-sizes. 
-        // We can add a dummy widget that expands.
         toolbar.add(theme.horizontalList()).expandX();
-        
+
         WButton listBtn = toolbar.add(theme.button(isGridView ? "List" : "[List]")).widget();
         listBtn.action = () -> { isGridView = false; reload(); };
-        
+
         WButton gridBtn = toolbar.add(theme.button(isGridView ? "[Grid]" : "Grid")).widget();
         gridBtn.action = () -> { isGridView = true; reload(); };
 
         add(theme.horizontalSeparator()).expandX();
 
-        // Content
         if (isGridView) {
             initGridView(addons);
         } else {

src/main/java/com/cope/meteoraddons/gui/tabs/AddonsTab.java

@@ -13,8 +13,7 @@
 import static meteordevelopment.meteorclient.MeteorClient.mc;
 
 /**
- * GUI tab for browsing and managing Meteor addons.
- * Provides navigation between Installed and Browse screens.
+ * Main GUI tab for browsing and managing addons.
  */
 public class AddonsTab extends Tab {
     public AddonsTab() {

src/main/java/com/cope/meteoraddons/gui/widgets/WAddonCard.java

@@ -8,9 +8,7 @@
 import meteordevelopment.meteorclient.renderer.Texture;
 
 /**
- * Widget for displaying an addon card in the grid.
- * Simplified design: Icon + Title + View Details button.
- * Shows installed indicator (green checkmark) overlaid on icon for installed addons.
+ * Addon card widget for grid view.
  */
 public class WAddonCard extends WVerticalList {
     private static final double FIXED_WIDTH = 220;
@@ -26,21 +24,17 @@ public WAddonCard(Addon addon, Runnable onOpenDetails) {
     @Override
     protected void onCalculateSize() {
         super.onCalculateSize();
-        // Force fixed width
         width = theme.scale(FIXED_WIDTH);
     }
 
     @Override
     public void init() {
-        // Icon (128x128, scaled down to 64x64 by GPU)
         Texture iconTexture = IconCache.get(addon);
         add(theme.texture(64, 64, 0, iconTexture)).centerX();
 
-        // Title row with optional installed indicator
         WHorizontalList titleRow = add(theme.horizontalList()).centerX().widget();
         titleRow.add(theme.label(addon.getName()));
 
-        // Add installed indicator after title if installed
         if (addon.isInstalled()) {
             Texture installedIcon = IconCache.getInstalledIndicator();
             if (installedIcon != null) {
@@ -49,7 +43,6 @@ public void init() {
             }
         }
 
-        // View details button
         WButton viewButton = add(theme.button("View Details")).expandX().widget();
         viewButton.action = onOpenDetails;
     }