Add CI improvements: Dependabot, code coverage, and code quality

.github/dependabot.yml

@@ -0,0 +1,11 @@
+version: 2
+updates:
+  - package-ecosystem: "maven"
+    directory: "/"
+    schedule:
+      interval: "weekly"
+
+  - package-ecosystem: "github-actions"
+    directory: "/"
+    schedule:
+      interval: "weekly"

.github/workflows/ci.yml

@@ -11,13 +11,13 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v6
 
       - name: Set up JDK 21
-        uses: actions/setup-java@v4
+        uses: actions/setup-java@v5
         with:
           java-version: '21'
-          distribution: 'temurin'
+          distribution: 'corretto'
           cache: maven
 
       - name: Build and verify

.github/workflows/code-coverage.yml

@@ -0,0 +1,30 @@
+name: Code Coverage
+
+on:
+  push:
+    branches: [main]
+  pull_request:
+    branches: [main]
+
+jobs:
+  coverage:
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v6
+
+      - name: Set up JDK 21
+        uses: actions/setup-java@v5
+        with:
+          java-version: '21'
+          distribution: 'corretto'
+          cache: maven
+
+      - name: Run tests with coverage
+        run: mvn test jacoco:report -B
+
+      - name: Upload coverage report
+        uses: actions/upload-artifact@v6
+        with:
+          name: coverage-report
+          path: target/site/jacoco/

.github/workflows/code-quality.yml

@@ -0,0 +1,24 @@
+name: Code Quality
+
+on:
+  push:
+    branches: [main]
+  pull_request:
+    branches: [main]
+
+jobs:
+  formatting:
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v6
+
+      - name: Set up JDK 21
+        uses: actions/setup-java@v5
+        with:
+          java-version: '21'
+          distribution: 'corretto'
+          cache: maven
+
+      - name: Check formatting
+        run: mvn spotless:check -B

CLAUDE.md

@@ -0,0 +1,5 @@
+# Atlas Project Instructions
+
+## Code Style
+- Always use explicit imports — never use wildcard imports (e.g., `import com.example.Foo`, not `import com.example.*`)
+- No line should exceed 140 characters in length

pom.xml

@@ -14,6 +14,7 @@
         <kotlin.code.style>official</kotlin.code.style>
         <kotlin.version>2.1.10</kotlin.version>
         <java.version>21</java.version>
+        <jacoco.version>0.8.14</jacoco.version>
     </properties>
 
     <repositories>
@@ -99,6 +100,68 @@
                     </execution>
                 </executions>
             </plugin>
+            <plugin>
+                <groupId>com.diffplug.spotless</groupId>
+                <artifactId>spotless-maven-plugin</artifactId>
+                <version>3.2.1</version>
+                <configuration>
+                    <kotlin>
+                        <ktlint>
+                            <version>1.0.1</version>
+                        </ktlint>
+                    </kotlin>
+                    <java>
+                        <googleJavaFormat>
+                            <version>1.19.1</version>
+                        </googleJavaFormat>
+                    </java>
+                </configuration>
+            </plugin>
+            <plugin>
+                <groupId>org.jacoco</groupId>
+                <artifactId>jacoco-maven-plugin</artifactId>
+                <version>${jacoco.version}</version>
+                <configuration>
+                    <excludes>
+                        <exclude>net/sf/jsqlparser/**</exclude>
+                    </excludes>
+                </configuration>
+                <executions>
+                    <execution>
+                        <id>prepare-agent</id>
+                        <goals>
+                            <goal>prepare-agent</goal>
+                        </goals>
+                    </execution>
+                    <execution>
+                        <id>report</id>
+                        <phase>test</phase>
+                        <goals>
+                            <goal>report</goal>
+                        </goals>
+                    </execution>
+                    <execution>
+                        <id>jacoco-check</id>
+                        <goals>
+                            <goal>check</goal>
+                        </goals>
+                        <configuration>
+                            <rules>
+                                <rule>
+                                    <element>PACKAGE</element>
+                                    <limits>
+                                        <limit>
+                                            <counter>LINE</counter>
+                                            <value>COVEREDRATIO</value>
+                                            <minimum>0.0</minimum>
+                                        </limit>
+                                    </limits>
+                                </rule>
+                            </rules>
+                        </configuration>
+                    </execution>
+                </executions>
+            </plugin>
         </plugins>
     </build>
 

src/main/kotlin/com/coderjoe/atlas/Atlas.kt

@@ -2,12 +2,12 @@ package com.coderjoe.atlas
 
 import com.coderjoe.atlas.core.AtlasBlockListener
 import com.coderjoe.atlas.core.BlockSystem
-import com.coderjoe.atlas.guide.GuideBook
-import com.coderjoe.atlas.guide.GuideBookListener
 import com.coderjoe.atlas.fluid.FluidBlockDialog
 import com.coderjoe.atlas.fluid.FluidBlockFactory
 import com.coderjoe.atlas.fluid.FluidBlockPersistence
 import com.coderjoe.atlas.fluid.FluidBlockRegistry
+import com.coderjoe.atlas.guide.GuideBook
+import com.coderjoe.atlas.guide.GuideBookListener
 import com.coderjoe.atlas.power.PowerBlockDialog
 import com.coderjoe.atlas.power.PowerBlockFactory
 import com.coderjoe.atlas.power.PowerBlockPersistence
@@ -57,51 +57,63 @@ class Atlas : JavaPlugin() {
         initTransportSystem()
 
         // Register unified listener
-        val powerSystem = BlockSystem<com.coderjoe.atlas.power.PowerBlock>(
-            name = "power",
-            registry = powerBlockRegistry,
-            factory = PowerBlockFactory,
-            descriptors = powerDescriptors(),
-            showDialog = { player, block ->
-                PowerBlockDialog.showPowerDialog(player, block as com.coderjoe.atlas.power.PowerBlock, powerBlockRegistry)
-            }
-        )
-
-        val fluidSystem = BlockSystem<com.coderjoe.atlas.fluid.FluidBlock>(
-            name = "fluid",
-            registry = fluidBlockRegistry,
-            factory = FluidBlockFactory,
-            descriptors = fluidDescriptors(),
-            showDialog = { player, block ->
-                FluidBlockDialog.showFluidDialog(player, block as com.coderjoe.atlas.fluid.FluidBlock, fluidBlockRegistry)
-            }
-        )
-
-        val transportSystem = BlockSystem<com.coderjoe.atlas.transport.TransportBlock>(
-            name = "transport",
-            registry = transportBlockRegistry,
-            factory = TransportBlockFactory,
-            descriptors = transportDescriptors(),
-            showDialog = { player, block ->
-                TransportBlockDialog.showTransportDialog(player, block as com.coderjoe.atlas.transport.TransportBlock, transportBlockRegistry)
-            }
-        )
+        val powerSystem =
+            BlockSystem<com.coderjoe.atlas.power.PowerBlock>(
+                name = "power",
+                registry = powerBlockRegistry,
+                factory = PowerBlockFactory,
+                descriptors = powerDescriptors(),
+                showDialog = { player, block ->
+                    PowerBlockDialog.showPowerDialog(player, block as com.coderjoe.atlas.power.PowerBlock, powerBlockRegistry)
+                },
+            )
+
+        val fluidSystem =
+            BlockSystem<com.coderjoe.atlas.fluid.FluidBlock>(
+                name = "fluid",
+                registry = fluidBlockRegistry,
+                factory = FluidBlockFactory,
+                descriptors = fluidDescriptors(),
+                showDialog = { player, block ->
+                    FluidBlockDialog.showFluidDialog(player, block as com.coderjoe.atlas.fluid.FluidBlock, fluidBlockRegistry)
+                },
+            )
+
+        val transportSystem =
+            BlockSystem<com.coderjoe.atlas.transport.TransportBlock>(
+                name = "transport",
+                registry = transportBlockRegistry,
+                factory = TransportBlockFactory,
+                descriptors = transportDescriptors(),
+                showDialog = { player, block ->
+                    TransportBlockDialog.showTransportDialog(
+                        player,
+                        block as com.coderjoe.atlas.transport.TransportBlock,
+                        transportBlockRegistry,
+                    )
+                },
+            )
 
         server.pluginManager.registerEvents(
             AtlasBlockListener(this, listOf(powerSystem, fluidSystem, transportSystem)),
-            this
+            this,
         )
 
         val guideBookListener = GuideBookListener(this)
         server.pluginManager.registerEvents(guideBookListener, this)
         server.addRecipe(GuideBook.createRecipe(this))
 
         // Auto-save every 5 minutes (6000 ticks)
-        autoSaveTask = server.scheduler.runTaskTimer(this, Runnable {
-            powerBlockPersistence.save(powerBlockRegistry)
-            fluidBlockPersistence.save(fluidBlockRegistry)
-            transportBlockPersistence.save(transportBlockRegistry)
-        }, 6000L, 6000L)
+        autoSaveTask =
+            server.scheduler.runTaskTimer(
+                this,
+                Runnable {
+                    powerBlockPersistence.save(powerBlockRegistry)
+                    fluidBlockPersistence.save(fluidBlockRegistry)
+                    transportBlockPersistence.save(transportBlockRegistry)
+                },
+                6000L, 6000L,
+            )
 
         logger.atlasInfo("Atlas plugin enabled!")
     }
@@ -169,7 +181,7 @@ class Atlas : JavaPlugin() {
 
     private fun transportDescriptors(): Map<String, com.coderjoe.atlas.core.BlockDescriptor> {
         return listOf(
-            com.coderjoe.atlas.transport.block.ConveyorBelt.descriptor
+            com.coderjoe.atlas.transport.block.ConveyorBelt.descriptor,
         ).associateBy { it.baseBlockId }
     }
 
@@ -184,7 +196,7 @@ class Atlas : JavaPlugin() {
             com.coderjoe.atlas.power.block.MultiPowerCable.descriptor,
             com.coderjoe.atlas.power.block.CobblestoneFactory.descriptor,
             com.coderjoe.atlas.power.block.ObsidianFactory.descriptor,
-            com.coderjoe.atlas.power.block.PowerMerger.descriptor
+            com.coderjoe.atlas.power.block.PowerMerger.descriptor,
         ).associateBy { it.baseBlockId }
     }
 
@@ -193,7 +205,7 @@ class Atlas : JavaPlugin() {
             com.coderjoe.atlas.fluid.block.FluidPump.descriptor,
             com.coderjoe.atlas.fluid.block.FluidPipe.descriptor,
             com.coderjoe.atlas.fluid.block.FluidContainer.descriptor,
-            com.coderjoe.atlas.fluid.block.FluidMerger.descriptor
+            com.coderjoe.atlas.fluid.block.FluidMerger.descriptor,
         ).associateBy { it.baseBlockId }
     }
 }

src/main/kotlin/com/coderjoe/atlas/NexoIntegration.kt

@@ -57,7 +57,10 @@ class NexoIntegration(private val plugin: JavaPlugin) {
         }
     }
 
-    private fun discoverResources(prefix: String, suffix: String): List<String> {
+    private fun discoverResources(
+        prefix: String,
+        suffix: String,
+    ): List<String> {
         val url = javaClass.classLoader.getResource(prefix) ?: return emptyList()
 
         return when (url.protocol) {

src/main/kotlin/com/coderjoe/atlas/PlayerJoinListener.kt

@@ -5,9 +5,8 @@ import org.bukkit.event.Listener
 import org.bukkit.event.player.PlayerJoinEvent
 
 class PlayerJoinListener(
-    private val resourcePackManager: ResourcePackManager
+    private val resourcePackManager: ResourcePackManager,
 ) : Listener {
-
     @EventHandler
     fun onPlayerJoin(event: PlayerJoinEvent) {
         val player = event.player

src/main/kotlin/com/coderjoe/atlas/ResourcePackManager.kt

@@ -3,14 +3,12 @@ package com.coderjoe.atlas
 import net.kyori.adventure.resource.ResourcePackInfo
 import net.kyori.adventure.resource.ResourcePackRequest
 import net.kyori.adventure.text.Component
-import net.kyori.adventure.text.format.NamedTextColor
 import org.bukkit.entity.Player
 import org.bukkit.plugin.java.JavaPlugin
 import java.net.URI
 import java.util.UUID
 
 class ResourcePackManager(private val plugin: JavaPlugin) {
-
     private var packInfo: ResourcePackInfo? = null
     private var isRequired: Boolean = false
     private var promptMessage: Component? = null
@@ -39,11 +37,12 @@ class ResourcePackManager(private val plugin: JavaPlugin) {
         val packId = UUID.nameUUIDFromBytes(url.toByteArray())
 
         try {
-            packInfo = ResourcePackInfo.resourcePackInfo()
-                .uri(URI.create(url))
-                .hash(hash)
-                .id(packId)
-                .build()
+            packInfo =
+                ResourcePackInfo.resourcePackInfo()
+                    .uri(URI.create(url))
+                    .hash(hash)
+                    .id(packId)
+                    .build()
 
             if (prompt.isNotBlank()) {
                 promptMessage = Component.text(prompt)
@@ -58,14 +57,15 @@ class ResourcePackManager(private val plugin: JavaPlugin) {
     fun sendToPlayer(player: Player) {
         val info = packInfo ?: return
 
-        val request = ResourcePackRequest.resourcePackRequest()
-            .packs(info)
-            .required(isRequired)
-            .replace(true)
-            .apply {
-                promptMessage?.let { prompt(it) }
-            }
-            .build()
+        val request =
+            ResourcePackRequest.resourcePackRequest()
+                .packs(info)
+                .required(isRequired)
+                .replace(true)
+                .apply {
+                    promptMessage?.let { prompt(it) }
+                }
+                .build()
 
         player.sendResourcePacks(request)