diff --git a/TODO.md b/TODO.md index e5a4123..f882080 100644 --- a/TODO.md +++ b/TODO.md @@ -309,6 +309,189 @@ Checkstyle отработал `0 errors` (error-уровневые legacy-нар --- +## 🎯 Порядок работ: Задача 1 → Задача 2 → остальное + +> Задано владельцем 2026-10-06. Всё остальное в этом файле (версии 1.19.2–26.3, смена команды, пункты аудита) делается **после** задач 1 и 2, +> потому что и lolib-слой, и структура пакетов затрагивают почти все файлы. + +### Задача 1. Отказаться от `lolib` (`libs/lolib-3.0.0.jar`) + +Почему: исходников и репозитория lolib нет (архив/утерян), jar лежит в `libs/` как бинарник, байткод 21 (мешает п.139 про Java 17), внутри — fat-jar с чужими зависимостями. +Мы тащим его целиком (`implementation(files("libs/lolib-3.0.0.jar"))` + релокация `dev.lolib`), а используем **5 классов в 28 файлах**: + +| Что берём из lolib | Где (файлов) | Замена своим кодом | +|---|---|---| +| `dev.lolib.gui.InventoryGUI` | ~12 (все `menu/*`: `Menu`, `LPMenu`, `MainMenu`, `LobbyMenu`, `SettingsMenu`, `LangMenu`, `SpectatorMenu`, `DuelsMenu`, лидерборд-меню, `PlayerManagementMenu`, `ParkourSettingsMenu`) | собственный `menu/gui/` (инвентарь + `InventoryClickEvent`-листенер, слоты/страницы; per-player состояние — заодно закрывает п.78, п.81) | +| `dev.lolib.scheduler.Scheduler` / `ScheduledTask` | ~16 (`SessionStateManager`, `GhostPlayer/GhostReplay`, `RaceMode`, `SpeedrunMode`, `ParkourSpectator`, `PlayerSpawnSetup`, `InventoryData`, `Schematics`, `StorageSQL`, `LeaderboardStorage`, `World`…) | тонкая обёртка `scheduler/` над Bukkit + Folia-схедулерами (global/region/entity/async) — фундамент для п.21/143; без неё Folia не починить | +| `dev.lolib.worlds.VoidGenerator` | `World.java` | свой `ChunkGenerator` (void) в `world/` — это п.112/145 | +| `dev.lolib.core.LoPlugin` | `LoParkour.java` (+`PluginBootstrap`) | перейти на обычный `JavaPlugin` (что именно давал `LoPlugin` — проверить декомпиляцией: порядок enable/disable, автосервисы) | + +Шаги: +1. **Выяснить фактическую поверхность API:** `javap`/декомпилятор по 5 классам, список методов, реально вызываемых из нашего кода (`grep` по `dev.lolib` — всё остальное внутри jar нам не нужно). +2. **Транзитивные библиотеки:** lolib — fat-jar (gson, HikariCP, caffeine, snakeyaml, slf4j, checkerframework). Сейчас gson/Hikari у нас и так объявлены явно (`FileStatsStorage`, `StorageDisk`, `SQLDataMapper`, `SQLConnectionManager`, `ParkourPlayer`); **caffeine/snakeyaml/slf4j напрямую не импортируются** — убедиться, что без jar сборка и тесты проходят, не добавлять лишнего. +3. Написать замены (scheduler → void-gen → gui → `JavaPlugin`), по одной, с тестами; после каждой — `./gradlew build`. +4. Удалить `libs/lolib-3.0.0.jar`, `implementation(files(...))`, релокацию `dev.lolib` и закомментированный `exclude` в `shadowJar`; убрать `softdepend: VoidGen` из `plugin.yml`. +5. Проверка: jar без `dev/loki/loparkour/lib/lolib/**` (`unzip -l`), плагин стартует на 1.21.x/26.x (см. п.141), размер jar заметно меньше. +6. Папка `libs/` остаётся пустой → удалить, если больше ничего не лежит. + +Критерий готовности: `grep -r "dev.lolib" src build.gradle.kts` пусто, `build` зелёный, меню/таймеры/мир работают. + +> ## ✅ ПРОГРЕСС — Задача 1 ВЫПОЛНЕНА (2026-10-06) +> Поверхность API уточнена декомпиляцией (`javap`): реально использовались `InventoryGUI` (create/title/rows/fill/setItem/open + GUIManager-листенер), +> `Scheduler.get().run/runAsync/runLater/runTimer` + `ScheduledTask.cancel()`, `VoidGenerator.getGenerator()`, `LoPlugin.enable/disable` + `loLogger()` +> (`AutoRegister`/`dependencies`/`load` — no-op, в коде аннотаций нет). Замены (по шагам, `build` зелёный после каждого): +> - **`scheduler/`** — `Scheduler` (фасад, кэш по плагину, `Backend`), `ScheduledTask` (functional interface, только `cancel()` — `isCancelled()` нигде не вызывался), +> `BukkitBackend` (Bukkit-планировщик), `FoliaBackend` (global/async-рефлексия: `RegionizedServer`-детект, fallback с warning — Folia API нет в Spigot/Paper артефактах матрицы). +> Entity/region-варианты (`runAt*`) не реализовывались — их никто не вызывает; добавить при п.143. +> - **`world/core/VoidGenerator`** — `ChunkGenerator` без генерации (`shouldGenerate*` = false; `shouldGenerateBedrock()` не переопределяется — deprecated в 1.19.2/1.20.4, default и так false). +> - **`menu/gui/inventory/`** — `InventoryGui` (тот же флюентный API, что и у lolib: call-site'ы меняли только import) + `GuiManager` +> (UUID→GUI, dispatch по raw slot, отмена кликов, чистка при close/quit). Удалён мёртвый `menu/gui/Menu.java` (legacy-стаб с javadoc на LoLib). +> - **`LoParkour extends JavaPlugin`** — `enable/disable` → `onEnable/onDisable`, `loLogger()` → `getLogger()` (+`log(Level, …)` в `PluginBootstrap`). +> - **Удалено:** `libs/lolib-3.0.0.jar` (+папка `libs/`), `implementation(files(...))`, релокация `dev.lolib`, закомментированный `minimize/exclude`, +> `softdepend: VoidGen`, строка `!libs/*.jar` в `.gitignore`, запись `api_docs/`, упоминания LoLib в комментариях/javadoc/CI. +> - **Проверки:** `grep -r "dev.lolib" src build.gradle.kts` пусто; `unzip -l` — нет ни `dev/lolib`, ни `dev/loki/loparkour/lib/lolib`; +> **jar 9 770 813 → 7 816 441 байт (−20%)**; `build` зелёный: checkstyle **0 errors / 0 warnings**, PMD **0**, **87 тестов / 0 fail**; +> матрица собирается: 1.19.2, 1.20.1, 1.21, 1.21.11 (JDK 21) и 26.1.2, 26.2 (JDK 25). +> - **Новые тесты:** `scheduler/SchedulerTest` (нет Folia-детекта на обычном JVM, кэш по плагину), `menu/gui/inventory/InventoryGuiTest` +> (границы rows, fill не затрагивает поставленное, dispatch+cancel клика), `world/core/VoidGeneratorTest` (синглтон, ничего не генерирует). +> - **Smoke-тест ПРОЙДЕН** (`./gradlew runServer`, Paper 1.21.4): `Loading` → `Enabling LoParkour` → конфиги синхронизированы, +> PAPI-хук зарегистрирован, паркур-мир создан → `Done (27.831s)`, `Disabling` без единого ERROR/Exception/WARN в логе. +> - **Остаток:** Folia-путь `FoliaBackend` не проверялся на Folia-сервере (п.143). + +### Задача 2. Жёсткий рефакторинг под KISS / DRY / SOLID (сразу после задачи 1) + +Правила (становятся **жёсткими гейтами**, не рекомендациями; расширяют `docs/ARCHITECTURE.md` п.5 и п.8): +- **≤ 4 файла в папке** (было «ориентир 3–5»). Больше → делим на подпапки по смыслу. Сейчас: 89 пакетов, **6 превышают 4**: `elytra/generator` (7), `elytra/mode` (6), `command/schematic` (6), `player/core` (5), `generator/jump/placement` (5), `generator/jump/calculation` (5). +- **≤ 200 строк на файл** — гейт уже есть (`FileLength` 200, error); проверить, что супрессий на FileLength не осталось; снизить `MethodLength` 80 → ~40. +- **KISS:** убрать абстракции с одной реализацией, пустые интерфейсы, неиспользуемые слои; метод делает одно (cyclomatic ≤ 10 вместо 20, `NPath` ≤ 50 → ≤ 30, вложенность if ≤ 2). +- **DRY:** убрать дубли (`create()` во всех режимах — п.105, дубликаты в SQL DDL — п.62, `SessionUserManager` ≈ `SessionPlayerManager` — п.76, дубли `canJump` и т.п.). Гейт — PMD CPD (минимум ~60 токенов). +- **SOLID:** SRP (god-классы уже порезаны, закрепить `TooManyMethods` ≤ 10, `TooManyFields`, `GodClass` вернуть), DIP (домен не импортирует `org.bukkit` там, где можно через порт), ISP (узкие интерфейсы), OCP (новые режимы без правок ядра). +- **Правило слоёв** (`api → core → subsystems → platform`) проверять автоматически (см. ниже), а не только в документе. + +Настройка линтеров (усилить «сильно»): +- **Checkstyle:** `FinalLocalVariable`, `MagicNumber`, `ClassFanOutComplexity` (сейчас 20 → 12), `ClassDataAbstractionCoupling` (20 → 10), `ParameterNumber` 7 → 5, `ReturnCount`, `NestedIfDepth` 2 → 1–2, `IllegalCatch`, `IllegalThrows`, `HiddenField`, `VisibilityModifier` (закрывает п.77/107 — публичные mutable-поля), `DesignForExtension`/`FinalClass`, `JavadocMethod` для публичного API; **все warning → error**, `suppressions.xml` обнулить (сейчас 87 строк). +- **PMD 7:** включить категории `bestpractices`, `errorprone`, `design`, `multithreading`, `performance`, `security` целиком (а не выборочные правила); вернуть `GodClass`, добавить `CouplingBetweenObjects`, `LawOfDemeter` (по месту), `AvoidSynchronizedAtMethodLevel`, `DoNotUseThreads`, `AvoidUsingVolatile`; запустить и на `src/test`; `isIgnoreFailures=false` уже стоит — оставить. +- **Новые проверки (предложение, выбрать):** + 1. **ArchUnit** (тест в JUnit): слои, отсутствие циклов между пакетами, «в папке ≤ 4 файла» и «≤ 200 строк» как тест, запрет `BukkitRunnable`/`Bukkit.getScheduler()` вне `scheduler/` (п.109 — Folia-инварианты), запрет `org.bukkit` в `api`/`core`. + 2. **SpotBugs** (+ `find-sec-bugs`) — NPE/гонки/SQL-инъекции, закрывает целый класс пунктов P2 по потокобезопасности. + 3. **Error Prone + NullAway** (compile-time) — null-safety на всём коде, `@Nullable` на границах. + 4. **PMD CPD** — гейт на дубли (DRY). + 5. **JaCoCo** — минимальное покрытие (старт 50% строк, ядро — 70%), отчёт в CI; п.108 (тесты) становится обязательным. + 6. **Spotless** (google-java-format/palantir) — единый формат, `spotlessCheck` в CI; убирает `WhitespaceAround`/`EmptyLineSeparator` шум. + 7. **OWASP dependency-check** или `gradle-versions`/`dependabot` — уязвимости/устаревшие зависимости (HikariCP 5.1.0, mysql-connector 8.3.0, bstats). + 8. **forbidden-apis** — запрет deprecated/запрещённых вызовов (`String.format` без Locale, `Thread.sleep`, `System.out`). + 9. **Gradle `dependency-analysis`** — лишние/недостающие зависимости (поймает остатки lolib). + 10. **Проверка дат/версий:** тест, что `plugin.yml`/`build.gradle.kts`/README/CHANGELOG версии совпадают. +- **CI:** `quality`-job запускает всё выше (`checkstyleMain checkstyleTest pmdMain pmdTest spotbugsMain test jacocoTestCoverageVerification archTest`) и валит PR; на `main` — ещё и матрица версий. + +Порядок: (a) настроить линтеры и **получить красный отчёт** (честный baseline) → (b) зафиксировать список нарушений по пакетам → (c) рефакторить пакет за пакетом (папка ≤ 4, файл ≤ 200) → (d) вычищать супрессии → (e) включить гейты в CI. Нельзя добавлять новые файлы в `suppressions.xml` (уже правило п.7 ARCHITECTURE.md). +Сложность: **L** — трогает ~285 файлов; делать малыми PR по пакетам, `build` зелёный на каждом шаге. + +> ## ✅ ПРОГРЕСС — Задача 2, шаги (a) + (b) ВЫПОЛНЕНЫ, (c) НАЧАТ (2026-10-06) +> +> **(a) Линтеры усилены.** Новый `config/checkstyle/checkstyle.xml` — двухуровневый гейт (ratchet): +> - `severity="error"` (build падает): все прежние error-правила **плюс пороги подтянуты** — `MethodLength` 80→40, +> `CyclomaticComplexity` 20→15, `NestedIfDepth` 3→2, `ParameterNumber` 7→5, `ClassDataAbstractionCoupling` 25→10, +> `ClassFanOutComplexity` 30→12, `MagicNumber` и `FinalLocalVariable` **warning→error** (оба чистые!). +> Добавлены: `ReturnCount` (max 5), `FinalClass`, `HiddenField`, `VisibilityModifier`, `NPathComplexity` (секция вернулась в явном виде). +> - `severity="warning"` (build зелёный, но каждое нарушение видно в отчёте) — **бэклог дизайна**, правило: +> **счётчик обнулился → сразу переводить в error**. Сейчас: `VisibilityModifier` 163, `ReturnCount` 69, `HiddenField` 30, +> `IllegalCatch` 23, `ClassFanOutComplexity` 17, `NPathComplexity` 12, `ParameterNumber` 8, `MethodLength` 5, `CDAC` 2. +> - **`suppressions.xml` ОБНУЛЁН** (45 записей удалено, включая FileLength/NPath/WhitespaceAround/IllegalCatch). +> +> **Красный baseline (честный, до фиксов): 431 нарушение в 129 файлах.** Замеры порогов перед выбором: +> `ReturnCount` max 3→5 (88→69), `NPathComplexity` max 30→50 (36→12) — пороги 3/30 дают мусор на легаси-кодах, +> реальная работа начинается с 5/50. +> +> **(b) Список по пакетам (после фиксов, 329 нарушений / 103 файла / 60 пакетов):** +> +> | Пакет | ↑ | | Файл | ↑ | +> |---|---|---|---|---| +> | `config/options` | 53 | | `config/options/Option.java` | 45 | +> | `config/options/section` | 38 | | `config/options/section/OptionGeneration.java` | 25 | +> | `player/core` | 19 | | `generator/core/model/GeneratorState.java` | 15 | +> | `generator/core/model` | 17 | | `menu/core/Menus.java` | 15 | +> | `config/core` | 15 | | `mode/base/Modes.java` | 15 | +> | `menu/core` | 15 | | `config/core/Config.java` | 14 | +> | `mode/base` | 15 | | `player/core/ParkourPlayer.java` | 13 | +> | `elytra/generator` | 12 | | `config/options/OptionGeneral.java` | 8 | +> +> Остальные 52 пакета — ≤ 7 нарушений; размазаны по `ReturnCount` (52 файла, ≤ 5 каждая) и `VisibilityModifier` (20 файлов). +> +> **(c) Исправлено 102 нарушения (431 → 329):** +> - `FinalClass` **37** — во всех классах с единственным private-конструктором добавлен `final` (безопасно: такой класс всё равно не наследуется). +> - `WhitespaceAround` **22** — 11 пустых блоков `{}` → `{ }` (пустые конструкторы, `new TypeToken<…>(){}`, пустой record-body, `default -> {}`). +> +> **Состояние на конец шага:** `./gradlew build` — **BUILD SUCCESSFUL**, checkstyle **0 error / 329 warning**, PMD **0**, +> тесты **94 / 0 fail (1 skipped)**, jar собирается (матрица версий не перепроверялась после правок — только локальная сборка). +> Изменено 45 файлов (TODO.md, 2 конфига линтеров, 46 исходников). +> +> **Осталось (порядок):** +> 1. Тест-гейт «**в папке ≤ 4 файлов**» (JUnit, без новых зависимостей) + разбивка 6 переполненных пакетов: +> `elytra/generator` 7, `elytra/mode` 6, `command/schematic` 6, `player/core` 5, `generator/jump/placement` 5, `generator/jump/calculation` 5 — +> 10 файлов переносятся в подпапки (механически: package + импорты). +> 2. Пакет за пакетом обнулять бэклог, начиная с `config/options` (91 нарушение в 2 файлах: `Option` — 45 public-полей, +> `OptionGeneration` — 25; это и есть п.77/107 про public mutable state) → после обнуления перевести правило в error. +> 3. `IllegalCatch` 23: 3 сайта намеренные (команда/конфиг с откатом/скетчи — boundary-catch), остальные 20 — выловить конкретные типы; +> **нужно решение владельца** по трём. +> 4. Дальше по плану (a): PMD-категории целиком, CPD, SpotBugs, ArchUnit-слои, JaCoCo, Spotless, quality-job в CI. + +--- + +## 🗺 План: поддержка версий 1.19.2 → 26.3 (Java 21 + Java 25), смена команды + +> Задано владельцем 2026-10-06. Сейчас: `build.gradle.kts` параметризован (`-PmcVersion/-PapiCoordinate/-PjavaRelease/-PapiVersion`), +> CI-матрица собирает 1.19.2 / 1.20.1 / 1.21 / 1.21.11 (Java 21) и 26.1.2 / 26.2 (Java 25, `release: 25`). **Только сборка — запуск на серверах не проверяется.** + +- **139. [P1] Целевой диапазон: Paper/Spigot 1.19.2 … 26.3 и Java 21 + Java 25.** MC 26.x требует Java 25 на сервере, поэтому плагин должен работать на JDK 21 *и* 25. + Ориентир — IP-Reborn: один jar с байткодом Java 21 (`release 21`), собранный на JDK 25, запускается на Java 21 и 25 (`ref/IP-Reborn/README.md`, раздел Building). + → Не выпускать отдельные jar под `release 25` (сломают 1.21.x на Java 21); оставить `release 21` для всех. Проверить, что `lolib-3.0.0.jar` (байткод 21) и shadow-релокации (ASM) читают class-version 65 на JDK 25. + → Если нужен запуск 1.19.2–1.20.4 на Java 17 — отдельное решение: lolib и наш код пересобрать на `release 17` (иначе нижняя граница фактически Java 21). +- **140. [P1] Добавить 26.3 в CI-матрицу и в README.** IP-Reborn проверяет `26.3.build.142-beta`; у нас только 26.1.2/26.2. Обновить бейджи/раздел «Supported servers» (сейчас «Paper 1.19.2+»). +- **141. [P1] Запуск, а не только сборка.** CI собирает jar, но не стартует сервер. → smoke-тест: поднять Paper на каждой версии (`run-paper` уже подключён), проверить, что плагин включается без исключений. Начать с 1.19.2, 1.21.11, 26.3. +- **142. [P1] Убрать API, которых нет в старых/новых версиях.** Известно: `GameRule.*` и `Particle.*` разные (см. п.129: `Particle.FIREWORK` нет в 1.20.4), `Material.STONE_SLAB` (п.100), `BukkitRunnable.runTaskTimerAsynchronously` (п.21), `registerNewObjective(String,String,String)`. В 5.4.0-paper26 IP-Reborn заменял «removed Paper and Adventure API calls» — просмотреть их список как чек-лист. + → Вынести версионно-зависимое в один слой `compat/` (определение версии по `Bukkit.getVersion()` — как `ref/ajParkour/.../utils/VersionSupport.java`; для 26+ учесть смену схемы версий: major ≥ 26). +- **143. [P1] Folia (п.21, issue #1).** `World.create()` уже не вызывает `createWorld` для загруженного мира (коммит «reuse already loaded world…»). Остаётся: `folia-plugin.yml`/`folia-supported: true`, region-scheduler вместо BukkitRunnable, настройки мира из global region. IP-Reborn сознательно *не* поддерживает Folia — честнее убрать бейдж, если не успеваем. +- **144. [P1] Заменить команду `/lp` — конфликт с LuckPerms.** LuckPerms использует `/lp`; у нас `plugin.yml`: `loparkour` с `aliases: [parkour, lp]`. Оба плагина борются за алиас, при загрузке в разном порядке один из них теряет команду. Заодно `parkour` конфликтует с плагином A5H73Y Parkour и с IP. + → Убрать `lp` (и, вероятно, `parkour`) из `aliases`; оставить `loparkour` + короткий алиас без конфликтов (предложение: **`lpk`**; *имя ждёт решения владельца*). + → Править: `src/main/resources/plugin.yml`; все `/lp …` в `README.md` (15), `docs/SCHEMATICS.md` (16), `src/main/resources/docs/SCHEMATICS.md`, `schematics/schematics.yml`, `docs/ARCHITECTURE.md`, локали, тексты сообщений/usage; `ReplayCommandExecutor` (javadoc `/lp replay`). + → Права (`loparkour.*`) не менять. Добавить в CHANGELOG пометку о breaking-изменении алиаса; рассмотреть deprecation-сообщение при вызове старого алиаса, если он останется. + +## 📥 Что взять из `ref/` (IP-Reborn, ajParkour, A5H73Y Parkour) + +> Берём только **идеи**: всё реализуем своим кодом, чужой код в репозиторий не копируем (поэтому атрибуции/NOTICE не нужны). +> Список — кандидаты по структуре и README; каждый пункт перед реализацией проверить по коду. + +### IP-Reborn (`ref/IP-Reborn`) + +- **145. Native void-генератор** (`world/`, в 6.0 убрали зависимость от VoidGen/PaperLib/vilib). У нас внешний `dev.lolib.worlds.VoidGenerator` и `softdepend: VoidGen` — закрывает п.112. +- **146. Паритет режимов** (`ipp/mode/single`, `ipp/mode/multi`, `ipp/mode/lobby`): Practice, Hourglass, SuperJump, TimeTrial, WaveTrial, Speed, TeamSurvival, Lobby/LobbySelection — основа для п.110/111 (invite-лобби). +- **147. `HologramManager`/`TrackedHologram`/`HologramConfig`** (`hologram/`) — заменить хрупкий `HoloHook` (п.87). +- **148. `LegacyDataMigrator` + `MIGRATION.md`** — шаблон безопасной миграции данных старых версий без удаления старых папок; полезно для смены команды/структуры конфигов. +- **149. Жизненный цикл интеграций** (`integration/IntegrationLoader`): снятие PAPI-expansion и хуков при reload/disable; по одному `HandlerList` на каждое кастомное событие — проверить наши `api/event/*`. +- **150. Документация-сайт** (`docs/guide`, `docs/index.md`, zh-перевод) — шаблон для п.114. +- **151. Правила сборки/CI под 26.x** (`pom.xml`, `CHANGES.md` 5.4.0-paper26/6.0.x) — как держат один jar на 1.21.11…26.3. + +### A5H73Y Parkour (`ref/Parkour`, MIT) + +- **152. Versioned upgrade системы** (`upgrade/major/*`) — пошаговые миграции конфигов/данных по версиям; решение для п.26 (ConfigUpdater) и п.62 (SQL-мигратор). +- **153. `PluginBackupUtil`** — автобэкап конфигов/данных перед апгрейдом. +- **154. Реестр команд** (`commands/CommandUsage`, `ParkourAutoTabCompleter`, `utility/parkourCommands.json`): описание команд в данных → одновременно help, tab-complete и права; убрало бы ручной `CommandTabCompleter`. +- **155. Чекпоинты / киты / челленджи / звуки** (`type/checkpoint`, `type/kit`, `type/challenge`, `type/sounds`) — идеи фич для карт; `TaskCooldowns`, `SignUtils` (таблички) — мелкие утилиты. +- **156. Conversation-API для настройки через чат** (`conversation/*`) — альтернатива меню для ввода значений (приз, режим). +- **157. Плагин-обёртки** (`plugin/PluginWrapper`, `EconomyApi`, `PermissionVault`, `BountifulApi`, `PlaceholderApi`) — единый паттерн ленивого подключения soft-зависимостей; решает п.89 (VaultHook кэширует навсегда). +- **158. `parkourPlaceholders.json`** — список плейсхолдеров данными; идея для п.120. + +### ajParkour (`ref/ajParkour`, GPL-3.0) + +- **159. `VersionSupport`** (версия по `Bukkit.getVersion()`, учёт major ≥ 26, action bar через spigot API) — готовая основа для п.142. +- **160. Портал/зона** (`Portal`, `AreaStorage`, `game/PkArea`, `WorldeditSelector`) — вход в паркур через зону/портал в мире, выбор области WorldEdit. +- **161. `TopManager`/`TopEntry`** — компактный top-10 для голограмм/PAPI. +- **162. `Updater`** — проверка новой версии (opt-in, без блокировки main thread). +- **163. `jumps.yml` + `Difficulty`** — выносить параметры прыжков в данные (закрывает мёртвый `jump-validation`/`jump-types`, п.38). + +--- + ## Итог по аудиту - Найдено: **~125 новых пунктов** (P1: 18, P2: 21, P3: 28, P4: 14, тесты: 2, конкуренты: 13, наследие форка: 5, новые режимы: 10, зачистка варнингов: 7). diff --git a/config/checkstyle/checkstyle.xml b/config/checkstyle/checkstyle.xml index 8eba917..fa11a6b 100644 --- a/config/checkstyle/checkstyle.xml +++ b/config/checkstyle/checkstyle.xml @@ -3,6 +3,24 @@ "-//Checkstyle//DTD Checkstyle Configuration 1.3//EN" "https://checkstyle.org/dtds/configuration_1_3.dtd"> + @@ -17,11 +35,8 @@ - - + @@ -37,16 +52,23 @@ - + + - + - + + + + + + + @@ -86,6 +108,18 @@ + + + + + + + + + + + + @@ -102,9 +136,10 @@ + - + @@ -113,11 +148,11 @@ - + - + @@ -126,7 +161,6 @@ - @@ -142,6 +176,7 @@ + @@ -158,7 +193,6 @@ - diff --git a/config/checkstyle/suppressions.xml b/config/checkstyle/suppressions.xml index c3cabca..1ba0eee 100644 --- a/config/checkstyle/suppressions.xml +++ b/config/checkstyle/suppressions.xml @@ -1,88 +1,3 @@ - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - \ No newline at end of file + + diff --git a/src/main/java/dev/loki/loparkour/adaptive/core/model/DifficultyWeights.java b/src/main/java/dev/loki/loparkour/adaptive/core/model/DifficultyWeights.java index 3f33384..4824ee6 100644 --- a/src/main/java/dev/loki/loparkour/adaptive/core/model/DifficultyWeights.java +++ b/src/main/java/dev/loki/loparkour/adaptive/core/model/DifficultyWeights.java @@ -6,7 +6,7 @@ package dev.loki.loparkour.adaptive.core.model; * * Thread-safe: Stateless, all methods are pure functions. */ -public class DifficultyWeights { +public final class DifficultyWeights { private static final double[] LEVEL_THRESHOLDS = {0.3, 0.6, 0.8}; diff --git a/src/main/java/dev/loki/loparkour/adaptive/model/AdaptiveConfig.java b/src/main/java/dev/loki/loparkour/adaptive/model/AdaptiveConfig.java index 3d59e50..bd67285 100644 --- a/src/main/java/dev/loki/loparkour/adaptive/model/AdaptiveConfig.java +++ b/src/main/java/dev/loki/loparkour/adaptive/model/AdaptiveConfig.java @@ -9,7 +9,7 @@ import java.util.concurrent.ConcurrentHashMap; * Cached configuration for the adaptive difficulty system. * Loads settings from config.yml and caches them for performance. */ -public class AdaptiveConfig { +public final class AdaptiveConfig { private AdaptiveConfig() { } diff --git a/src/main/java/dev/loki/loparkour/adaptive/storage/SQLStatsStorage.java b/src/main/java/dev/loki/loparkour/adaptive/storage/SQLStatsStorage.java index ffcef35..74bc3d6 100644 --- a/src/main/java/dev/loki/loparkour/adaptive/storage/SQLStatsStorage.java +++ b/src/main/java/dev/loki/loparkour/adaptive/storage/SQLStatsStorage.java @@ -171,7 +171,7 @@ public class SQLStatsStorage implements StatsRepository { flushBatch(); } - private static class PendingUpdate { + private static final class PendingUpdate { int jumps = 0; int falls = 0; int longestStreak = 0; diff --git a/src/main/java/dev/loki/loparkour/api/core/ParkourAPI.java b/src/main/java/dev/loki/loparkour/api/core/ParkourAPI.java index 46d3a5b..fec21c3 100644 --- a/src/main/java/dev/loki/loparkour/api/core/ParkourAPI.java +++ b/src/main/java/dev/loki/loparkour/api/core/ParkourAPI.java @@ -17,7 +17,7 @@ import org.jetbrains.annotations.Nullable; *
  • For jump structures, see {@link dev.loki.loparkour.schematic.core.ParkourSchematic}
  • * */ -public class ParkourAPI { +public final class ParkourAPI { private ParkourAPI() throws IllegalAccessException { throw new IllegalAccessException("Initializing API class"); diff --git a/src/main/java/dev/loki/loparkour/command/core/CommandTabCompleter.java b/src/main/java/dev/loki/loparkour/command/core/CommandTabCompleter.java index 29435cd..b3a570d 100644 --- a/src/main/java/dev/loki/loparkour/command/core/CommandTabCompleter.java +++ b/src/main/java/dev/loki/loparkour/command/core/CommandTabCompleter.java @@ -13,7 +13,7 @@ import java.util.ArrayList; import java.util.Collections; import java.util.List; -class CommandTabCompleter { +final class CommandTabCompleter { private static final int THREE_ARGS = 3; private static final int FOUR_ARGS = 4; diff --git a/src/main/java/dev/loki/loparkour/command/schematic/SchematicCommandHandler.java b/src/main/java/dev/loki/loparkour/command/schematic/SchematicCommandHandler.java index 8088130..55060a5 100644 --- a/src/main/java/dev/loki/loparkour/command/schematic/SchematicCommandHandler.java +++ b/src/main/java/dev/loki/loparkour/command/schematic/SchematicCommandHandler.java @@ -16,7 +16,7 @@ import java.util.Map; /** * Admin schematic tools: wand, create, list, paste, reload. */ -public class SchematicCommandHandler { +public final class SchematicCommandHandler { private static final int COOLDOWN_MS = 2500; diff --git a/src/main/java/dev/loki/loparkour/config/core/ConfigLoader.java b/src/main/java/dev/loki/loparkour/config/core/ConfigLoader.java index 1b89bdf..e2b33db 100644 --- a/src/main/java/dev/loki/loparkour/config/core/ConfigLoader.java +++ b/src/main/java/dev/loki/loparkour/config/core/ConfigLoader.java @@ -16,7 +16,7 @@ import java.util.logging.Logger; /** * Handles loading, updating, and basic validation of configuration files. */ -public class ConfigLoader { +public final class ConfigLoader { private ConfigLoader() { } diff --git a/src/main/java/dev/loki/loparkour/config/core/merge/ConfigUpdater.java b/src/main/java/dev/loki/loparkour/config/core/merge/ConfigUpdater.java index 50622ff..696f3d3 100644 --- a/src/main/java/dev/loki/loparkour/config/core/merge/ConfigUpdater.java +++ b/src/main/java/dev/loki/loparkour/config/core/merge/ConfigUpdater.java @@ -16,7 +16,7 @@ import java.util.logging.Logger; * preserving user values while adding any new keys from the template. * Uses full YAML paths (e.g. "sql.enabled") to avoid key collisions. */ -public class ConfigUpdater { +public final class ConfigUpdater { private final Logger logger; diff --git a/src/main/java/dev/loki/loparkour/config/locale/LocaleLoader.java b/src/main/java/dev/loki/loparkour/config/locale/LocaleLoader.java index 6b164fc..62855fb 100644 --- a/src/main/java/dev/loki/loparkour/config/locale/LocaleLoader.java +++ b/src/main/java/dev/loki/loparkour/config/locale/LocaleLoader.java @@ -23,7 +23,7 @@ import java.util.stream.Stream; /** * Loads locale files from disk with validation and auto-fix. */ -public class LocaleLoader { +public final class LocaleLoader { private LocaleLoader() { } diff --git a/src/main/java/dev/loki/loparkour/config/locale/Locales.java b/src/main/java/dev/loki/loparkour/config/locale/Locales.java index 6b774cf..f77f6a3 100644 --- a/src/main/java/dev/loki/loparkour/config/locale/Locales.java +++ b/src/main/java/dev/loki/loparkour/config/locale/Locales.java @@ -24,7 +24,7 @@ import java.util.regex.Pattern; * Facade for locale system: loading, caching, and access. * Delegates to LocaleLoader and LocaleCache. */ -public class Locales { +public final class Locales { private Locales() { } diff --git a/src/main/java/dev/loki/loparkour/config/options/Option.java b/src/main/java/dev/loki/loparkour/config/options/Option.java index f58262a..3f9e01d 100644 --- a/src/main/java/dev/loki/loparkour/config/options/Option.java +++ b/src/main/java/dev/loki/loparkour/config/options/Option.java @@ -26,7 +26,7 @@ import java.util.function.BiFunction; * Facade for all config-derived constants. * Delegates to specialized option classes in config.options package. */ -public class Option { +public final class Option { private Option() { } diff --git a/src/main/java/dev/loki/loparkour/config/options/OptionGeneral.java b/src/main/java/dev/loki/loparkour/config/options/OptionGeneral.java index 5c4a2b6..bde6502 100644 --- a/src/main/java/dev/loki/loparkour/config/options/OptionGeneral.java +++ b/src/main/java/dev/loki/loparkour/config/options/OptionGeneral.java @@ -17,7 +17,7 @@ import java.util.Map; /** * General configuration options (heading, border, leads, go-back location, parkour options). */ -public class OptionGeneral { +public final class OptionGeneral { private static final int MAX_LEAD = 128; private static final int MIN_AXIS_VALUES = 4; diff --git a/src/main/java/dev/loki/loparkour/config/options/section/OptionGeneration.java b/src/main/java/dev/loki/loparkour/config/options/section/OptionGeneration.java index 3300cf8..278e349 100644 --- a/src/main/java/dev/loki/loparkour/config/options/section/OptionGeneration.java +++ b/src/main/java/dev/loki/loparkour/config/options/section/OptionGeneration.java @@ -6,7 +6,7 @@ import dev.loki.loparkour.config.core.Config; /** * Generation-related configuration options (validation, memory). */ -public class OptionGeneration { +public final class OptionGeneration { private static final double DEFAULT_MAX_JUMP_DISTANCE = 5.0; private static final double DEFAULT_MAX_HORIZONTAL_DISTANCE = 4.0; diff --git a/src/main/java/dev/loki/loparkour/config/options/section/OptionParticles.java b/src/main/java/dev/loki/loparkour/config/options/section/OptionParticles.java index 397bd7a..7e682e8 100644 --- a/src/main/java/dev/loki/loparkour/config/options/section/OptionParticles.java +++ b/src/main/java/dev/loki/loparkour/config/options/section/OptionParticles.java @@ -10,7 +10,7 @@ import org.bukkit.Sound; /** * Particle and sound configuration options. */ -public class OptionParticles { +public final class OptionParticles { private static final int PARTICLE_SIZE = 10; diff --git a/src/main/java/dev/loki/loparkour/config/options/section/OptionSQL.java b/src/main/java/dev/loki/loparkour/config/options/section/OptionSQL.java index 07f89df..140b132 100644 --- a/src/main/java/dev/loki/loparkour/config/options/section/OptionSQL.java +++ b/src/main/java/dev/loki/loparkour/config/options/section/OptionSQL.java @@ -5,7 +5,7 @@ import dev.loki.loparkour.config.core.Config; /** * SQL database configuration options. */ -public class OptionSQL { +public final class OptionSQL { private OptionSQL() { } diff --git a/src/main/java/dev/loki/loparkour/config/options/section/OptionStyles.java b/src/main/java/dev/loki/loparkour/config/options/section/OptionStyles.java index 53a3ad3..029df20 100644 --- a/src/main/java/dev/loki/loparkour/config/options/section/OptionStyles.java +++ b/src/main/java/dev/loki/loparkour/config/options/section/OptionStyles.java @@ -15,7 +15,7 @@ import java.util.function.BiFunction; /** * Style initialization utilities. */ -public class OptionStyles { +public final class OptionStyles { private OptionStyles() { } diff --git a/src/main/java/dev/loki/loparkour/generator/core/model/GeneratorState.java b/src/main/java/dev/loki/loparkour/generator/core/model/GeneratorState.java index 6917ff6..7260c00 100644 --- a/src/main/java/dev/loki/loparkour/generator/core/model/GeneratorState.java +++ b/src/main/java/dev/loki/loparkour/generator/core/model/GeneratorState.java @@ -51,7 +51,7 @@ public class GeneratorState { public final Map specialChances = new HashMap<>(); public final Map defaultChances = new HashMap<>(); - public GeneratorState() {} + public GeneratorState() { } public void resetScore() { score = 0; diff --git a/src/main/java/dev/loki/loparkour/generator/lifecycle/player/GeneratorCleanup.java b/src/main/java/dev/loki/loparkour/generator/lifecycle/player/GeneratorCleanup.java index 264093b..0d60d98 100644 --- a/src/main/java/dev/loki/loparkour/generator/lifecycle/player/GeneratorCleanup.java +++ b/src/main/java/dev/loki/loparkour/generator/lifecycle/player/GeneratorCleanup.java @@ -169,7 +169,7 @@ public class GeneratorCleanup { case SOUTH -> blockSpawn.add(0, 0, FIRST_BLOCK_OFFSET); case WEST -> blockSpawn.add(-FIRST_BLOCK_OFFSET, 0, 0); case EAST -> blockSpawn.add(FIRST_BLOCK_OFFSET, 0, 0); - default -> {} + default -> { } } } diff --git a/src/main/java/dev/loki/loparkour/hook/floodgate/FloodgateHook.java b/src/main/java/dev/loki/loparkour/hook/floodgate/FloodgateHook.java index e0427ad..dc09025 100644 --- a/src/main/java/dev/loki/loparkour/hook/floodgate/FloodgateHook.java +++ b/src/main/java/dev/loki/loparkour/hook/floodgate/FloodgateHook.java @@ -3,9 +3,9 @@ package dev.loki.loparkour.hook.floodgate; import org.bukkit.entity.Player; import org.geysermc.floodgate.api.FloodgateApi; -public class FloodgateHook { +public final class FloodgateHook { - private FloodgateHook() {} + private FloodgateHook() { } /** * Whether this player is a bedrock player. diff --git a/src/main/java/dev/loki/loparkour/hook/holo/HoloHook.java b/src/main/java/dev/loki/loparkour/hook/holo/HoloHook.java index e69b58b..9d031b6 100644 --- a/src/main/java/dev/loki/loparkour/hook/holo/HoloHook.java +++ b/src/main/java/dev/loki/loparkour/hook/holo/HoloHook.java @@ -7,14 +7,14 @@ import dev.loki.loparkour.leaderboard.model.Score; import dev.loki.loparkour.mode.base.Mode; import me.filoghost.holographicdisplays.api.HolographicDisplaysAPI; -public class HoloHook { +public final class HoloHook { private static final int PLACEHOLDER_REFRESH_TICKS = 100; private static final double EASY_DIFFICULTY_MAX = 0.25; private static final double MEDIUM_DIFFICULTY_MAX = 0.5; private static final double HARD_DIFFICULTY_MAX = 0.75; - private HoloHook() {} + private HoloHook() { } /** * Initializes this hook. diff --git a/src/main/java/dev/loki/loparkour/hook/vault/VaultHook.java b/src/main/java/dev/loki/loparkour/hook/vault/VaultHook.java index 9f7c7da..230c1e0 100644 --- a/src/main/java/dev/loki/loparkour/hook/vault/VaultHook.java +++ b/src/main/java/dev/loki/loparkour/hook/vault/VaultHook.java @@ -6,9 +6,9 @@ import org.bukkit.Bukkit; import org.bukkit.entity.Player; import org.bukkit.plugin.RegisteredServiceProvider; -public class VaultHook { +public final class VaultHook { - private VaultHook() {} + private VaultHook() { } private static Economy economy; diff --git a/src/main/java/dev/loki/loparkour/menu/core/MenuStub.java b/src/main/java/dev/loki/loparkour/menu/core/MenuStub.java index 665740c..1c686a1 100644 --- a/src/main/java/dev/loki/loparkour/menu/core/MenuStub.java +++ b/src/main/java/dev/loki/loparkour/menu/core/MenuStub.java @@ -4,5 +4,5 @@ package dev.loki.loparkour.menu.core; * Placeholder during menu migration; real flows use {@link Menus} / InventoryGui. */ public final class MenuStub { - private MenuStub() {} + private MenuStub() { } } diff --git a/src/main/java/dev/loki/loparkour/menu/core/screen/DynamicMenu.java b/src/main/java/dev/loki/loparkour/menu/core/screen/DynamicMenu.java index db24984..958b5cd 100644 --- a/src/main/java/dev/loki/loparkour/menu/core/screen/DynamicMenu.java +++ b/src/main/java/dev/loki/loparkour/menu/core/screen/DynamicMenu.java @@ -84,5 +84,5 @@ public abstract class DynamicMenu { int id, BiFunction<@NotNull Player, @Nullable ParkourUser, dev.loki.loparkour.menu.gui.MenuItem> item, Predicate predicate - ) {} + ) { } } diff --git a/src/main/java/dev/loki/loparkour/mode/impl/solo/GravityShiftMode.java b/src/main/java/dev/loki/loparkour/mode/impl/solo/GravityShiftMode.java index f9eb54a..6b58e1e 100644 --- a/src/main/java/dev/loki/loparkour/mode/impl/solo/GravityShiftMode.java +++ b/src/main/java/dev/loki/loparkour/mode/impl/solo/GravityShiftMode.java @@ -173,6 +173,6 @@ public class GravityShiftMode implements Mode { return Modes.GRAVITY_SHIFT; } - private record EffectConfig(int amplifier, int durationTicks) {} + private record EffectConfig(int amplifier, int durationTicks) { } } } diff --git a/src/main/java/dev/loki/loparkour/reward/core/Rewards.java b/src/main/java/dev/loki/loparkour/reward/core/Rewards.java index ed45717..6e7ffd4 100644 --- a/src/main/java/dev/loki/loparkour/reward/core/Rewards.java +++ b/src/main/java/dev/loki/loparkour/reward/core/Rewards.java @@ -10,7 +10,7 @@ import java.util.Map; /** * Class that reads the rewards-v2.yml file and puts them in the variables listed below. */ -public class Rewards { +public final class Rewards { private Rewards() { } diff --git a/src/main/java/dev/loki/loparkour/schematic/registry/Schematics.java b/src/main/java/dev/loki/loparkour/schematic/registry/Schematics.java index af51f6c..d561b25 100644 --- a/src/main/java/dev/loki/loparkour/schematic/registry/Schematics.java +++ b/src/main/java/dev/loki/loparkour/schematic/registry/Schematics.java @@ -9,7 +9,7 @@ import java.io.File; /** * Ensures the schematics folder exists; structures are loaded by {@link SchematicManager}. */ -public class Schematics { +public final class Schematics { private Schematics() { } diff --git a/src/main/java/dev/loki/loparkour/session/core/Session.java b/src/main/java/dev/loki/loparkour/session/core/Session.java index b9576cd..b95f957 100644 --- a/src/main/java/dev/loki/loparkour/session/core/Session.java +++ b/src/main/java/dev/loki/loparkour/session/core/Session.java @@ -19,7 +19,7 @@ import java.util.function.Function; * Refactored Session using composition pattern. * Coordinates player management and state management. */ -public class Session { +public final class Session { public ParkourGenerator generator; diff --git a/src/main/java/dev/loki/loparkour/storage/Storage.java b/src/main/java/dev/loki/loparkour/storage/Storage.java index 1e8ff42..6ad4a80 100644 --- a/src/main/java/dev/loki/loparkour/storage/Storage.java +++ b/src/main/java/dev/loki/loparkour/storage/Storage.java @@ -16,7 +16,7 @@ import java.util.UUID; * @author loki * @since 5.0.0 */ -public class Storage { +public final class Storage { private Storage() { } diff --git a/src/main/java/dev/loki/loparkour/storage/disk/StorageDisk.java b/src/main/java/dev/loki/loparkour/storage/disk/StorageDisk.java index 64ba772..cfc873f 100644 --- a/src/main/java/dev/loki/loparkour/storage/disk/StorageDisk.java +++ b/src/main/java/dev/loki/loparkour/storage/disk/StorageDisk.java @@ -20,7 +20,7 @@ import java.util.UUID; * * @since 5.0.0 */ -public class StorageDisk { +public final class StorageDisk { private StorageDisk() { } diff --git a/src/main/java/dev/loki/loparkour/storage/sql/SQLDataMapper.java b/src/main/java/dev/loki/loparkour/storage/sql/SQLDataMapper.java index ad1fd13..9f9e008 100644 --- a/src/main/java/dev/loki/loparkour/storage/sql/SQLDataMapper.java +++ b/src/main/java/dev/loki/loparkour/storage/sql/SQLDataMapper.java @@ -18,7 +18,7 @@ import java.util.UUID; /** * Maps data between Java objects and SQL result sets. */ -public class SQLDataMapper { +public final class SQLDataMapper { private static final int SCORE_PARAM_INDEX = 3; private static final int TIME_PARAM_INDEX = 4; @@ -134,7 +134,7 @@ public class SQLDataMapper { } private static void deserializeSettings(@NotNull ParkourPlayer player, @NotNull String settings) { - final Type type = new TypeToken>() {}.getType(); + final Type type = new TypeToken>() { }.getType(); final Map parsed = LoParkour.getGson().fromJson(settings, type); if (parsed == null) { player.setSettings(new HashMap<>()); @@ -166,5 +166,5 @@ public class SQLDataMapper { String styleName, String settings, long lastSeen - ) {} + ) { } } diff --git a/src/main/java/dev/loki/loparkour/storage/sql/StorageSQL.java b/src/main/java/dev/loki/loparkour/storage/sql/StorageSQL.java index 9691b46..3b692fd 100644 --- a/src/main/java/dev/loki/loparkour/storage/sql/StorageSQL.java +++ b/src/main/java/dev/loki/loparkour/storage/sql/StorageSQL.java @@ -20,7 +20,7 @@ import java.util.logging.Logger; * Facade for SQL storage. Coordinates connection lifecycle and delegates * CRUD to dedicated repository classes. */ -public class StorageSQL { +public final class StorageSQL { private StorageSQL() { } diff --git a/src/main/java/dev/loki/loparkour/storage/sql/query/SQLQueryBuilder.java b/src/main/java/dev/loki/loparkour/storage/sql/query/SQLQueryBuilder.java index 7ee8363..e5273e0 100644 --- a/src/main/java/dev/loki/loparkour/storage/sql/query/SQLQueryBuilder.java +++ b/src/main/java/dev/loki/loparkour/storage/sql/query/SQLQueryBuilder.java @@ -5,7 +5,7 @@ import org.jetbrains.annotations.NotNull; /** * Builds SQL queries for parkour storage operations. */ -public class SQLQueryBuilder { +public final class SQLQueryBuilder { private SQLQueryBuilder() { } diff --git a/src/main/java/dev/loki/loparkour/storage/sql/repo/SQLPlayerRepository.java b/src/main/java/dev/loki/loparkour/storage/sql/repo/SQLPlayerRepository.java index 176d80e..49156d9 100644 --- a/src/main/java/dev/loki/loparkour/storage/sql/repo/SQLPlayerRepository.java +++ b/src/main/java/dev/loki/loparkour/storage/sql/repo/SQLPlayerRepository.java @@ -15,7 +15,7 @@ import java.util.logging.Logger; /** * CRUD for player data. */ -public class SQLPlayerRepository { +public final class SQLPlayerRepository { private static final Logger LOGGER = Logger.getLogger(SQLPlayerRepository.class.getName()); diff --git a/src/main/java/dev/loki/loparkour/storage/sql/repo/SQLScoreRepository.java b/src/main/java/dev/loki/loparkour/storage/sql/repo/SQLScoreRepository.java index a714954..727da8f 100644 --- a/src/main/java/dev/loki/loparkour/storage/sql/repo/SQLScoreRepository.java +++ b/src/main/java/dev/loki/loparkour/storage/sql/repo/SQLScoreRepository.java @@ -17,7 +17,7 @@ import java.util.logging.Logger; /** * CRUD for leaderboard scores. */ -public class SQLScoreRepository { +public final class SQLScoreRepository { private static final Logger LOGGER = Logger.getLogger(SQLScoreRepository.class.getName()); diff --git a/src/main/java/dev/loki/loparkour/storage/sql/schema/SQLSchemaManager.java b/src/main/java/dev/loki/loparkour/storage/sql/schema/SQLSchemaManager.java index 9cc280d..1c30a56 100644 --- a/src/main/java/dev/loki/loparkour/storage/sql/schema/SQLSchemaManager.java +++ b/src/main/java/dev/loki/loparkour/storage/sql/schema/SQLSchemaManager.java @@ -11,7 +11,7 @@ import java.sql.SQLException; /** * Creates the database schema (DDL) for parkour storage. */ -public class SQLSchemaManager { +public final class SQLSchemaManager { private static final String ADAPTIVE_STATS_TABLE = """ CREATE TABLE IF NOT EXISTS loparkour_player_stats ( diff --git a/src/main/java/dev/loki/loparkour/util/item/Materials.java b/src/main/java/dev/loki/loparkour/util/item/Materials.java index eb75124..40ef060 100644 --- a/src/main/java/dev/loki/loparkour/util/item/Materials.java +++ b/src/main/java/dev/loki/loparkour/util/item/Materials.java @@ -15,7 +15,7 @@ import java.util.Locale; */ public final class Materials { - private Materials() {} + private Materials() { } @Nullable public static Material parse(@Nullable String name) { diff --git a/src/main/java/dev/loki/loparkour/util/misc/Probs.java b/src/main/java/dev/loki/loparkour/util/misc/Probs.java index 3592a6c..c77554c 100644 --- a/src/main/java/dev/loki/loparkour/util/misc/Probs.java +++ b/src/main/java/dev/loki/loparkour/util/misc/Probs.java @@ -7,7 +7,7 @@ import java.util.concurrent.ThreadLocalRandom; /** * Probability utilities (replacement for vilib Probs) */ -public class Probs { +public final class Probs { private Probs() { } diff --git a/src/main/java/dev/loki/loparkour/util/misc/SkullSetter.java b/src/main/java/dev/loki/loparkour/util/misc/SkullSetter.java index 6eff611..6c712f8 100644 --- a/src/main/java/dev/loki/loparkour/util/misc/SkullSetter.java +++ b/src/main/java/dev/loki/loparkour/util/misc/SkullSetter.java @@ -6,7 +6,7 @@ import org.bukkit.inventory.meta.SkullMeta; /** * Utility for setting player heads */ -public class SkullSetter { +public final class SkullSetter { private SkullSetter() { } diff --git a/src/main/java/dev/loki/loparkour/util/particle/ParticleUtil.java b/src/main/java/dev/loki/loparkour/util/particle/ParticleUtil.java index e65339d..d1fa111 100644 --- a/src/main/java/dev/loki/loparkour/util/particle/ParticleUtil.java +++ b/src/main/java/dev/loki/loparkour/util/particle/ParticleUtil.java @@ -10,7 +10,7 @@ import org.bukkit.util.BoundingBox; * Utility class for particle effects * Replaces vilib Particles */ -public class ParticleUtil { +public final class ParticleUtil { private ParticleUtil() { } diff --git a/src/main/java/dev/loki/loparkour/util/text/ColorUtil.java b/src/main/java/dev/loki/loparkour/util/text/ColorUtil.java index 871d962..673cc9c 100644 --- a/src/main/java/dev/loki/loparkour/util/text/ColorUtil.java +++ b/src/main/java/dev/loki/loparkour/util/text/ColorUtil.java @@ -10,7 +10,7 @@ import java.util.regex.Pattern; *

    Project standard: {@code &#RRGGBB} and {@code &l} etc. — see {@code .cursor/rules/locale-and-colors.mdc}. * MiniMessage tags are legacy-only; do not use in new locale or Java strings. */ -public class ColorUtil { +public final class ColorUtil { private ColorUtil() { } diff --git a/src/main/java/dev/loki/loparkour/util/world/Locations.java b/src/main/java/dev/loki/loparkour/util/world/Locations.java index cf0483f..ce281d1 100644 --- a/src/main/java/dev/loki/loparkour/util/world/Locations.java +++ b/src/main/java/dev/loki/loparkour/util/world/Locations.java @@ -5,7 +5,7 @@ import org.bukkit.Location; /** * Location utilities (replacement for vilib Locations) */ -public class Locations { +public final class Locations { private Locations() { } diff --git a/src/main/java/dev/loki/loparkour/world/core/Divider.java b/src/main/java/dev/loki/loparkour/world/core/Divider.java index 14b9b6f..9d24f32 100644 --- a/src/main/java/dev/loki/loparkour/world/core/Divider.java +++ b/src/main/java/dev/loki/loparkour/world/core/Divider.java @@ -15,7 +15,7 @@ import java.util.stream.IntStream; * @author loki * @since 5.0.0 */ -public class Divider { +public final class Divider { private Divider() { } diff --git a/src/main/java/dev/loki/loparkour/world/core/World.java b/src/main/java/dev/loki/loparkour/world/core/World.java index 0d9ec68..58a0121 100644 --- a/src/main/java/dev/loki/loparkour/world/core/World.java +++ b/src/main/java/dev/loki/loparkour/world/core/World.java @@ -15,7 +15,7 @@ import java.nio.file.Path; import java.util.Comparator; import java.util.stream.Stream; -public class World { +public final class World { private static final int WORLD_BORDER_SIZE = 10_000_000; private static final int CLEAR_WEATHER_DURATION_TICKS = 1000000;