Release v4.3.0

Remove specific dependency on Sponge API 8 in the mod info file (#1221 )
Fix class cast exception in NodeTypeKey#getTypeName (#1230 )
2018-09-23 15:04:15 +01:00 · 2018-09-23 15:02:02 +01:00 · 2018-09-23 14:38:27 +01:00 · 2018-09-23 14:30:09 +01:00 · 2018-09-23 11:28:48 +01:00 · 2018-09-21 21:53:06 +01:00
564 changed files with 11713 additions and 11564 deletions
@@ -1,40 +1,12 @@
-# Created by https://www.gitignore.io/
-
 ### Intellij ###
-# Covers JetBrains IDEs: IntelliJ, RubyMine, PhpStorm, AppCode, PyCharm, CLion, Android Studio and Webstorm
-# Reference: https://intellij-support.jetbrains.com/hc/en-us/articles/206544839
-
-# User-specific stuff:
 .idea/
 *.iws
 /out/
 *.iml
 .idea_modules/
-
-# JIRA plugin
 atlassian-ide-plugin.xml

-# Crashlytics plugin (for Android Studio and IntelliJ)
-com_crashlytics_export_strings.xml
-crashlytics.properties
-crashlytics-build.properties
-fabric.properties
-
-
-### Maven ###
-target/
-pom.xml.tag
-pom.xml.releaseBackup
-pom.xml.versionsBackup
-pom.xml.next
-release.properties
-dependency-reduced-pom.xml
-buildNumber.properties
-.mvn/timing.properties
-
-
 ### Eclipse ###
-
 .metadata
 bin/
 tmp/
@@ -46,73 +18,27 @@ local.properties
 .settings/
 .loadpath
 .recommenders
-
-# Eclipse Core
-.project
-
-# External tool builders
 .externalToolBuilders/
-
-# Locally stored "Eclipse launch configurations"
 *.launch
-
-# PyDev specific (Python IDE for Eclipse)
-*.pydevproject
-
-# CDT-specific (C/C++ Development Tooling)
-.cproject
-
-# JDT-specific (Eclipse Java Development Tools)
-.classpath
-
-# Java annotation processor (APT)
 .factorypath
-
-# PDT-specific (PHP Development Tools)
-.buildpath
-
-# sbteclipse plugin
-.target
-
-# Tern plugin
-.tern-project
-
-# TeXlipse plugin
-.texlipse
-
-# STS (Spring Tool Suite)
-.springBeans
-
-# Code Recommenders
 .recommenders/
-
+.apt_generated/
+.project
+.classpath

 ### Linux ###
 *~
-
-# temporary files which can be created if a process still has a handle open of a deleted file
 .fuse_hidden*
-
-# KDE directory preferences
 .directory
-
-# Linux trash folder which might appear on any partition or disk
 .Trash-*
-
-# .nfs files are created when an open file is removed but is still being accessed
 .nfs*

-
 ### macOS ###
-*.DS_Store
+.DS_Store
 .AppleDouble
 .LSOverride
-
-# Icon must end with two \r
 Icon
-# Thumbnails
 ._*
-# Files that might appear in the root of a volume
 .DocumentRevisions-V100
 .fseventsd
 .Spotlight-V100
@@ -120,45 +46,34 @@ Icon
 .Trashes
 .VolumeIcon.icns
 .com.apple.timemachine.donotpresent
-# Directories potentially created on remote AFP share
 .AppleDB
 .AppleDesktop
 Network Trash Folder
 Temporary Items
 .apdisk

+### NetBeans ###
+nbproject/private/
+build/
+nbbuild/
+dist/
+nbdist/
+.nb-gradle/

 ### Windows ###
-# Windows image file caches
+# Windows thumbnail cache files
 Thumbs.db
 ehthumbs.db
-
-# Folder config file
-Desktop.ini
-
-# Recycle Bin used on file shares
+ehthumbs_vista.db
+*.stackdump
+[Dd]esktop.ini
 $RECYCLE.BIN/
-
-# Windows Installer files
-*.cab
-*.msi
-*.msm
-*.msp
-
-# Windows shortcuts
 *.lnk

-
-### Java ###
-*.class
-
-# Mobile Tools for Java (J2ME)
-.mtj.tmp/
-
-# Package Files #
-*.jar
-*.war
-*.ear
-
-# virtual machine crash logs, see http://www.java.com/en/download/help/error_hotspot.xml
-hs_err_pid*
+### Gradle ###
+.gradle
+/build/
+/out/
+gradle-app.setting
+!gradle-wrapper.jar
+.gradletasknamecache
@@ -1,4 +1,4 @@
-![alt text](https://i.imgur.com/7TwJZ5e.png "Banner")
+![alt text](https://i.imgur.com/ToguFkQ.png "Banner")
 # LuckPerms [![Build Status](https://ci.lucko.me/job/LuckPerms/badge/icon)](https://ci.lucko.me/job/LuckPerms/) [![Javadocs](https://javadoc.io/badge/me.lucko.luckperms/luckperms-api.svg)](https://javadoc.io/doc/me.lucko.luckperms/luckperms-api) [![Maven Central](https://maven-badges.herokuapp.com/maven-central/me.lucko.luckperms/luckperms-api/badge.svg)](https://maven-badges.herokuapp.com/maven-central/me.lucko.luckperms/luckperms-api)

 LuckPerms is an advanced permissions implementation aiming to be a fast, reliable and flexible alternative to existing permission plugins. The project's main goals are centered around high performance and a wide feature set, filling the gaps of functionality and building upon existing features found in other plugins.
@@ -14,21 +14,20 @@ The latest **downloads** can be found on the [project's homepage](https://luckpe
 More information about the project & how to use the plugin can be found in the [wiki](https://github.com/lucko/LuckPerms/wiki).

 ## Building
-LuckPerms uses Maven to handle dependencies.
+LuckPerms uses Gradle to handle dependencies & building.

 #### Requirements
 * Java 8 JDK or newer
-* Maven 3.3.x or newer (the older versions will not work.)
 * Git

 #### Compiling from source
 ```sh
 git clone https://github.com/lucko/LuckPerms.git
 cd LuckPerms/
-mvn clean package
+./gradlew build
 ```

-You can find the output jars in the `target` directories.
+You can find the output jars in the `build/libs` directories.

 ## Contributing
 #### Pull Requests
@@ -37,14 +36,11 @@ If you make any changes or improvements to the plugin which you think would be b
 LuckPerms loosely follows the [Google Java Style Guide](https://google.github.io/styleguide/javaguide.html). Generally, try to copy the style of code found in the class you're editing. 

 #### Project Layout
-The project is split up into 5 seperate modules.
+The project is split up into a few separate modules.

 * **API** - The public, semantically versioned API used by other plugins wishing to integrate with and retrieve data from LuckPerms. This module (for the most part) does not contain any implementation itself, and is provided by the plugin.
 * **Common** - The common module contains most of the code which implements the respective LuckPerms plugins. This abstract module reduces duplicated code throughout the project.
-* **Bukkit, BungeeCord, Sponge & Nukkit** - Each use the common module to implement plugins on the respective server platforms.
+* **Bukkit, BungeeCord, Sponge, Nukkit & Velocity** - Each use the common module to implement plugins on the respective server platforms.

 ## License
 LuckPerms is licensed under the permissive MIT license. Please see [`LICENSE.txt`](https://github.com/lucko/LuckPerms/blob/master/LICENSE.txt) for more info.
-
-## Thanks
-Thanks to ej-technologies for granting LuckPerms an open source licence to their [Java Profiling Software](http://www.ej-technologies.com/products/jprofiler/overview.html "Java Profiler").
@@ -0,0 +1,93 @@
+project.version = '4.3'
+
+dependencies {
+    compileOnly 'com.google.guava:guava:19.0'
+    compileOnly 'org.checkerframework:checker-qual:2.5.5'
+}
+
+// disabled by default
+if (false && project.hasProperty('sonatypeUsername') && project.hasProperty('sonatypePassword')) {
+    apply plugin: 'signing'
+
+    javadoc {
+        options.encoding = 'UTF-8'
+        options.charSet = 'UTF-8'
+        options.links(
+                'https://google.github.io/guava/releases/21.0/api/docs/',
+                'https://docs.oracle.com/javase/8/docs/api/'
+        )
+        options.addStringOption('Xdoclint:none', '-quiet')
+    }
+
+    task javadocJar(type: Jar, dependsOn: javadoc) {
+        classifier 'javadoc'
+        from javadoc.destinationDir
+    }
+
+    task sourcesJar(type: Jar) {
+        classifier 'sources'
+        from sourceSets.main.allSource
+    }
+
+    artifacts {
+        archives javadocJar
+        archives sourcesJar
+    }
+
+    signing {
+        useGpgCmd()
+        sign configurations.archives
+    }
+
+    uploadArchives {
+        repositories {
+            mavenDeployer {
+                beforeDeployment { MavenDeployment deployment -> signing.signPom(deployment) }
+
+                repository(url: 'https://oss.sonatype.org/service/local/staging/deploy/maven2/') {
+                    authentication(userName: sonatypeUsername, password: sonatypePassword)
+                }
+
+                pom {
+                    project {
+                        name 'LuckPerms API'
+                        description 'An advanced permissions plugin for Bukkit/Spigot, BungeeCord, Sponge, Nukkit and Velocity.'
+                        url 'https://github.com/lucko/LuckPerms'
+
+                        licenses {
+                            license {
+                                name 'MIT'
+                                url 'https://opensource.org/licenses/MIT'
+                            }
+                        }
+
+                        developers {
+                            developer {
+                                name 'Luck'
+                                email 'git@lucko.me'
+                                url 'https://github.com/lucko'
+                            }
+                        }
+
+                        scm {
+                            connection 'scm:git:https://github.com/lucko/LuckPerms.git'
+                            developerConnection 'scm:git:git@github.com:lucko/LuckPerms.git'
+                            url 'https://github.com/lucko/LuckPerms'
+
+                        }
+
+                        ciManagement {
+                            system 'Jenkins'
+                            url 'https://ci.lucko.me/job/LuckPerms'
+                        }
+
+                        issueManagement {
+                            system 'GitHub'
+                            url 'https://github.com/lucko/LuckPerms/issues'
+                        }
+                    }
+                }
+            }
+        }
+    }
+}
@@ -1,96 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<project xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-         xmlns="http://maven.apache.org/POM/4.0.0"
-         xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
-    <parent>
-        <artifactId>luckperms</artifactId>
-        <groupId>me.lucko.luckperms</groupId>
-        <version>4.2-SNAPSHOT</version>
-    </parent>
-    <modelVersion>4.0.0</modelVersion>
-
-    <artifactId>luckperms-api</artifactId>
-
-    <build>
-        <finalName>LuckPerms-API-${release.version}</finalName>
-        <resources>
-            <resource>
-                <directory>../</directory>
-                <filtering>false</filtering>
-                <includes>
-                    <include>LICENSE.txt</include>
-                </includes>
-            </resource>
-        </resources>
-        <plugins>
-            <plugin>
-                <groupId>org.apache.maven.plugins</groupId>
-                <artifactId>maven-compiler-plugin</artifactId>
-                <version>${compiler.version}</version>
-                <configuration>
-                    <source>1.8</source>
-                    <target>1.8</target>
-                </configuration>
-            </plugin>
-        </plugins>
-    </build>
-
-    <profiles>
-        <profile>
-            <id>deployment</id>
-            <build>
-                <plugins>
-                    <plugin>
-                        <groupId>org.apache.maven.plugins</groupId>
-                        <artifactId>maven-source-plugin</artifactId>
-                        <version>3.0.1</version>
-                        <executions>
-                            <execution>
-                                <id>attach-sources</id>
-                                <goals>
-                                    <goal>jar</goal>
-                                </goals>
-                            </execution>
-                        </executions>
-                    </plugin>
-                    <plugin>
-                        <groupId>org.apache.maven.plugins</groupId>
-                        <artifactId>maven-javadoc-plugin</artifactId>
-                        <version>3.0.0</version>
-                        <executions>
-                            <execution>
-                                <id>attach-javadocs</id>
-                                <goals>
-                                    <goal>jar</goal>
-                                </goals>
-                            </execution>
-                        </executions>
-                        <configuration>
-                            <links>
-                                <link>https://google.github.io/guava/releases/21.0/api/docs/</link>
-                            </links>
-                        </configuration>
-                    </plugin>
-                </plugins>
-            </build>
-        </profile>
-    </profiles>
-
-    <dependencies>
-        <!-- guava -->
-        <dependency>
-            <groupId>com.google.guava</groupId>
-            <artifactId>guava</artifactId>
-            <version>19.0</version>
-            <scope>provided</scope>
-        </dependency>
-        <!-- jsr305 -->
-        <dependency>
-            <groupId>com.google.code.findbugs</groupId>
-            <artifactId>jsr305</artifactId>
-            <version>3.0.2</version>
-            <scope>provided</scope>
-        </dependency>
-    </dependencies>
-
-</project>
@@ -27,9 +27,9 @@ package me.lucko.luckperms;

 import me.lucko.luckperms.api.LuckPermsApi;

-import java.util.Optional;
+import org.checkerframework.checker.nullness.qual.NonNull;

-import javax.annotation.Nonnull;
+import java.util.Optional;

 /**
 * Provides static access to the {@link LuckPermsApi}.
@@ -49,8 +49,7 @@ public final class LuckPerms {
     * @return an api instance
     * @throws IllegalStateException if the api is not loaded
     */
-    @Nonnull
-    public static LuckPermsApi getApi() {
+    public static @NonNull LuckPermsApi getApi() {
        if (instance == null) {
            throw new IllegalStateException("API is not loaded.");
        }
@@ -66,8 +65,7 @@ public final class LuckPerms {
     *
     * @return an optional api instance
     */
-    @Nonnull
-    public static Optional<LuckPermsApi> getApiSafe() {
+    public static @NonNull Optional<LuckPermsApi> getApiSafe() {
        return Optional.ofNullable(instance);
    }

@@ -25,9 +25,9 @@

 package me.lucko.luckperms.api;

-import java.util.concurrent.CompletableFuture;
+import org.checkerframework.checker.nullness.qual.NonNull;

-import javax.annotation.Nonnull;
+import java.util.concurrent.CompletableFuture;

 /**
 * Represents the object responsible for handling action logging.
@@ -41,8 +41,7 @@ public interface ActionLogger {
     *
     * @return a new builder
     */
-    @Nonnull
-    LogEntry.Builder newEntryBuilder();
+    LogEntry.@NonNull Builder newEntryBuilder();

    /**
     * Gets a {@link Log} instance from the plugin storage.
@@ -50,8 +49,7 @@ public interface ActionLogger {
     * @return a log instance
     * @see Storage#getLog()
     */
-    @Nonnull
-    CompletableFuture<Log> getLog();
+    @NonNull CompletableFuture<Log> getLog();

    /**
     * Submits a log entry to the plugin to be handled.
@@ -69,8 +67,7 @@ public interface ActionLogger {
     * @param entry the entry to submit
     * @return a future which will complete when the action is done
     */
-    @Nonnull
-    CompletableFuture<Void> submit(@Nonnull LogEntry entry);
+    @NonNull CompletableFuture<Void> submit(@NonNull LogEntry entry);

    /**
     * Submits a log entry to the plugins storage handler.
@@ -80,8 +77,7 @@ public interface ActionLogger {
     * @param entry the entry to submit
     * @return a future which will complete when the action is done
     */
-    @Nonnull
-    CompletableFuture<Void> submitToStorage(@Nonnull LogEntry entry);
+    @NonNull CompletableFuture<Void> submitToStorage(@NonNull LogEntry entry);

    /**
     * Submits a log entry to the plugins log broadcasting handler.
@@ -92,7 +88,6 @@ public interface ActionLogger {
     * @param entry the entry to submit
     * @return a future which will complete when the action is done
     */
-    @Nonnull
-    CompletableFuture<Void> broadcastAction(@Nonnull LogEntry entry);
+    @NonNull CompletableFuture<Void> broadcastAction(@NonNull LogEntry entry);

 }
@@ -25,12 +25,12 @@

 package me.lucko.luckperms.api;

+import org.checkerframework.checker.nullness.qual.NonNull;
+
 import java.util.Map;
 import java.util.Objects;
 import java.util.Optional;

-import javax.annotation.Nonnull;
-
 /**
 * Represents a type of chat meta
 *
@@ -43,13 +43,12 @@ public enum ChatMetaType {
     */
    PREFIX("prefix") {
        @Override
-        public boolean matches(@Nonnull Node node) {
+        public boolean matches(@NonNull Node node) {
            return Objects.requireNonNull(node, "node").isPrefix();
        }

-        @Nonnull
        @Override
-        public Map.Entry<Integer, String> getEntry(@Nonnull Node node) {
+        public Map.@NonNull Entry<Integer, String> getEntry(@NonNull Node node) {
            return Objects.requireNonNull(node, "node").getPrefix();
        }
    },
@@ -59,13 +58,12 @@ public enum ChatMetaType {
     */
    SUFFIX("suffix") {
        @Override
-        public boolean matches(@Nonnull Node node) {
+        public boolean matches(@NonNull Node node) {
            return Objects.requireNonNull(node, "node").isSuffix();
        }

-        @Nonnull
        @Override
-        public Map.Entry<Integer, String> getEntry(@Nonnull Node node) {
+        public Map.@NonNull Entry<Integer, String> getEntry(@NonNull Node node) {
            return Objects.requireNonNull(node, "node").getSuffix();
        }
    };
@@ -82,7 +80,7 @@ public enum ChatMetaType {
     * @param node the node to test
     * @return true if the node has the same type
     */
-    public abstract boolean matches(@Nonnull Node node);
+    public abstract boolean matches(@NonNull Node node);

    /**
     * Returns if the passed node should be ignored when searching for meta of this type
@@ -90,7 +88,7 @@ public enum ChatMetaType {
     * @param node the node to test
     * @return true if the node does not share the same type
     */
-    public boolean shouldIgnore(@Nonnull Node node) {
+    public boolean shouldIgnore(@NonNull Node node) {
        return !matches(node);
    }

@@ -101,8 +99,7 @@ public enum ChatMetaType {
     * @return the entry
     * @throws IllegalStateException if the node does not share the same type
     */
-    @Nonnull
-    public abstract Map.Entry<Integer, String> getEntry(@Nonnull Node node);
+    public abstract Map.@NonNull Entry<Integer, String> getEntry(@NonNull Node node);

    @Override
    public String toString() {
@@ -116,8 +113,7 @@ public enum ChatMetaType {
     * @return the parsed chat meta type
     * @since 3.4
     */
-    @Nonnull
-    public static Optional<ChatMetaType> ofNode(@Nonnull Node node) {
+    public static @NonNull Optional<ChatMetaType> ofNode(@NonNull Node node) {
        if (node.isPrefix()) {
            return Optional.of(PREFIX);
        } else if (node.isSuffix()) {
@@ -30,13 +30,12 @@ import com.google.common.collect.ImmutableSet;
 import me.lucko.luckperms.api.context.ContextSet;
 import me.lucko.luckperms.api.context.ImmutableContextSet;

+import org.checkerframework.checker.nullness.qual.NonNull;
+
 import java.util.EnumSet;
 import java.util.Objects;
 import java.util.Set;

-import javax.annotation.Nonnull;
-import javax.annotation.concurrent.Immutable;
-
 /**
 * Encapsulates the {@link ContextSet contexts} and {@link LookupSetting settings} for
 * a permission or meta lookup.
@@ -45,7 +44,6 @@ import javax.annotation.concurrent.Immutable;
 *
 * @since 2.11
 */
-@Immutable
 public class Contexts {

    /**
@@ -82,8 +80,7 @@ public class Contexts {
     *
     * @return a context that will satisfy all contextual requirements.
     */
-    @Nonnull
-    public static Contexts allowAll() {
+    public static @NonNull Contexts allowAll() {
        return FullySatisfiedContexts.getInstance();
    }

@@ -96,8 +93,7 @@ public class Contexts {
     * @return the global contexts
     * @since 3.3
     */
-    @Nonnull
-    public static Contexts global() {
+    public static @NonNull Contexts global() {
        return GLOBAL;
    }

@@ -113,8 +109,7 @@ public class Contexts {
     * @param isOp the value of {@link LookupSetting#IS_OP}
     * @return a new instance
     */
-    @Nonnull
-    public static Contexts of(@Nonnull ContextSet contextSet, boolean includeNodesSetWithoutServer, boolean includeNodesSetWithoutWorld, boolean resolveInheritance, boolean applyParentsWithoutServer, boolean applyParentsWithoutWorld, boolean isOp) {
+    public static @NonNull Contexts of(@NonNull ContextSet contextSet, boolean includeNodesSetWithoutServer, boolean includeNodesSetWithoutWorld, boolean resolveInheritance, boolean applyParentsWithoutServer, boolean applyParentsWithoutWorld, boolean isOp) {
        Objects.requireNonNull(contextSet, "contextSet");
        EnumSet<LookupSetting> settings = formSettings(
                includeNodesSetWithoutServer,
@@ -130,6 +125,14 @@ public class Contexts {
        return new Contexts(contextSet.makeImmutable(), ImmutableSet.copyOf(settings));
    }

+    private static EnumSet<LookupSetting> asEnumSet(Set<LookupSetting> settings) {
+        if (settings instanceof EnumSet<?>) {
+            return ((EnumSet<LookupSetting>) settings);
+        } else {
+            return EnumSet.copyOf(settings);
+        }
+    }
+
    /**
     * Creates a new {@link Contexts} instance.
     *
@@ -137,11 +140,11 @@ public class Contexts {
     * @param settings the settings
     * @return a new instance
     */
-    public static Contexts of(@Nonnull ContextSet contextSet, @Nonnull Set<LookupSetting> settings) {
+    public static Contexts of(@NonNull ContextSet contextSet, @NonNull Set<LookupSetting> settings) {
        Objects.requireNonNull(contextSet, "contextSet");
        Objects.requireNonNull(settings, "settings");

-        EnumSet<LookupSetting> settingsCopy = EnumSet.copyOf(settings);
+        EnumSet<LookupSetting> settingsCopy = asEnumSet(settings);
        if (contextSet.isEmpty() && DEFAULT_SETTINGS.equals(settingsCopy)) {
            return GLOBAL;
        }
@@ -175,7 +178,7 @@ public class Contexts {
     * @deprecated in favour of {@link #of(ContextSet, boolean, boolean, boolean, boolean, boolean, boolean)}
     */
    @Deprecated
-    public Contexts(@Nonnull ContextSet contextSet, boolean includeNodesSetWithoutServer, boolean includeNodesSetWithoutWorld, boolean resolveInheritance, boolean applyParentsWithoutServer, boolean applyParentsWithoutWorld, boolean isOp) {
+    public Contexts(@NonNull ContextSet contextSet, boolean includeNodesSetWithoutServer, boolean includeNodesSetWithoutWorld, boolean resolveInheritance, boolean applyParentsWithoutServer, boolean applyParentsWithoutWorld, boolean isOp) {
        this.contextSet = Objects.requireNonNull(contextSet, "contextSet").makeImmutable();
        this.settings = ImmutableSet.copyOf(formSettings(
                includeNodesSetWithoutServer,
@@ -188,7 +191,7 @@ public class Contexts {
        this.hashCode = calculateHashCode();
    }

-    protected Contexts(@Nonnull ImmutableContextSet contextSet, @Nonnull ImmutableSet<LookupSetting> settings) {
+    protected Contexts(@NonNull ImmutableContextSet contextSet, @NonNull ImmutableSet<LookupSetting> settings) {
        this.contextSet = contextSet;
        this.settings = settings;
        this.hashCode = calculateHashCode();
@@ -200,8 +203,7 @@ public class Contexts {
     * @return an immutable context from this instance
     * @since 2.13
     */
-    @Nonnull
-    public ContextSet getContexts() {
+    public @NonNull ContextSet getContexts() {
        return this.contextSet;
    }

@@ -211,8 +213,7 @@ public class Contexts {
     * @return the settings
     * @since 4.2
     */
-    @Nonnull
-    public Set<LookupSetting> getSettings() {
+    public @NonNull Set<LookupSetting> getSettings() {
        return this.settings;
    }

@@ -223,7 +224,7 @@ public class Contexts {
     * @return the value
     * @since 4.2
     */
-    public boolean hasSetting(@Nonnull LookupSetting setting) {
+    public boolean hasSetting(@NonNull LookupSetting setting) {
        return this.settings.contains(setting);
    }

@@ -299,9 +300,8 @@ public class Contexts {
        return hasSetting(LookupSetting.APPLY_PARENTS_SET_WITHOUT_WORLD);
    }

-    @Nonnull
    @Override
-    public String toString() {
+    public @NonNull String toString() {
        return "Contexts(contextSet=" + this.contextSet + ", settings=" + this.settings + ")";
    }

@@ -25,9 +25,9 @@

 package me.lucko.luckperms.api;

-import java.util.Optional;
+import org.checkerframework.checker.nullness.qual.NonNull;

-import javax.annotation.Nonnull;
+import java.util.Optional;

 /**
 * Encapsulates the result of {@link User}s demotion along a {@link Track}.
@@ -41,8 +41,7 @@ public interface DemotionResult extends MutateResult {
     *
     * @return the status
     */
-    @Nonnull
-    Status getStatus();
+    @NonNull Status getStatus();

    @Override
    default boolean wasSuccess() {
@@ -60,8 +59,7 @@ public interface DemotionResult extends MutateResult {
     *
     * @return the group the user was demoted from.
     */
-    @Nonnull
-    Optional<String> getGroupFrom();
+    @NonNull Optional<String> getGroupFrom();

    /**
     * Gets the name of the group the user was demoted from, if applicable.
@@ -71,8 +69,7 @@ public interface DemotionResult extends MutateResult {
     *
     * @return the group the user was demoted to.
     */
-    @Nonnull
-    Optional<String> getGroupTo();
+    @NonNull Optional<String> getGroupTo();

    /**
     * The result status
@@ -25,10 +25,10 @@

 package me.lucko.luckperms.api;

-import java.util.UUID;
+import org.checkerframework.checker.nullness.qual.NonNull;
+import org.checkerframework.checker.nullness.qual.Nullable;

-import javax.annotation.Nonnull;
-import javax.annotation.Nullable;
+import java.util.UUID;

 /**
 * Represents an entity on the server.
@@ -49,24 +49,21 @@ public interface Entity {
     *
     * @return the uuid of the object, if available
     */
-    @Nullable
-    UUID getUniqueId();
+    @Nullable UUID getUniqueId();

    /**
     * Gets the name of the object
     *
     * @return the object name
     */
-    @Nonnull
-    String getName();
+    @NonNull String getName();

    /**
     * Gets the entities type.
     *
     * @return the type
     */
-    @Nonnull
-    Type getType();
+    @NonNull Type getType();

    /**
     * The different types of {@link Entity}
@@ -31,7 +31,7 @@ import me.lucko.luckperms.api.caching.CachedData;
 import me.lucko.luckperms.api.caching.MetaContexts;
 import me.lucko.luckperms.api.context.ImmutableContextSet;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * A special instance of {@link Contexts}, which when passed to:
@@ -54,8 +54,7 @@ public final class FullySatisfiedContexts extends Contexts {
    // singleton
    private static final FullySatisfiedContexts INSTANCE = new FullySatisfiedContexts();

-    @Nonnull
-    public static Contexts getInstance() {
+    public static @NonNull Contexts getInstance() {
        return INSTANCE;
    }

@@ -63,9 +62,8 @@ public final class FullySatisfiedContexts extends Contexts {
        super(ImmutableContextSet.empty(), ImmutableSet.copyOf(Contexts.global().getSettings()));
    }

-    @Nonnull
    @Override
-    public String toString() {
+    public @NonNull String toString() {
        return "FullySatisfiedContexts()";
    }

@@ -26,11 +26,13 @@
 package me.lucko.luckperms.api;

 import me.lucko.luckperms.api.caching.GroupData;
+import me.lucko.luckperms.api.context.ContextSet;
+
+import org.checkerframework.checker.nullness.qual.NonNull;
+import org.checkerframework.checker.nullness.qual.Nullable;

 import java.util.OptionalInt;

-import javax.annotation.Nonnull;
-
 /**
 * An inheritable holder of permission data.
 */
@@ -41,8 +43,32 @@ public interface Group extends PermissionHolder {
     *
     * @return the name of the group
     */
-    @Nonnull
-    String getName();
+    @NonNull String getName();
+
+    /**
+     * Gets the groups "display name", if it has one that differs from it's actual name.
+     *
+     * <p>The lookup is made using the current servers active context.</p>
+     *
+     * <p>Will return <code>null</code> if the groups display name is equal to it's
+     * {@link #getName() actual name}.</p>
+     *
+     * @return the display name
+     * @since 4.3
+     */
+    @Nullable String getDisplayName();
+
+    /**
+     * Gets the groups "display name", if it has one that differs from it's actual name.
+     *
+     * <p>Will return <code>null</code> if the groups display name is equal to it's
+     * {@link #getName() actual name}.</p>
+     *
+     * @param contextSet the contexts to lookup in
+     * @return the display name
+     * @since 4.3
+     */
+    @Nullable String getDisplayName(@NonNull ContextSet contextSet);

    /**
     * Gets the weight of this group, if present.
@@ -50,8 +76,7 @@ public interface Group extends PermissionHolder {
     * @return the group weight
     * @since 2.17
     */
-    @Nonnull
-    OptionalInt getWeight();
+    @NonNull OptionalInt getWeight();

    /**
     * Gets the groups's {@link GroupData} cache.
@@ -59,8 +84,7 @@ public interface Group extends PermissionHolder {
     * @return the groups cached data.
     * @since 4.0
     */
-    @Nonnull
    @Override
-    GroupData getCachedData();
+    @NonNull GroupData getCachedData();

 }
@@ -27,19 +27,17 @@ package me.lucko.luckperms.api;

 import me.lucko.luckperms.api.context.ContextSet;

+import org.checkerframework.checker.nullness.qual.NonNull;
+
 import java.util.Optional;
 import java.util.OptionalLong;

-import javax.annotation.Nonnull;
-import javax.annotation.concurrent.Immutable;
-
 /**
 * A relationship between a {@link PermissionHolder} and a permission.
 *
 * @param <T> the identifier type of the holder
 * @since 2.17
 */
-@Immutable
 public interface HeldPermission<T> {

    /**
@@ -47,16 +45,14 @@ public interface HeldPermission<T> {
     *
     * @return the holder
     */
-    @Nonnull
-    T getHolder();
+    @NonNull T getHolder();

    /**
     * Gets the permission being held
     *
     * @return the permission
     */
-    @Nonnull
-    String getPermission();
+    @NonNull String getPermission();

    /**
     * Gets the value of the permission
@@ -70,16 +66,14 @@ public interface HeldPermission<T> {
     *
     * @return the server
     */
-    @Nonnull
-    Optional<String> getServer();
+    @NonNull Optional<String> getServer();

    /**
     * Gets the world where the permission is held
     *
     * @return the world
     */
-    @Nonnull
-    Optional<String> getWorld();
+    @NonNull Optional<String> getWorld();

    /**
     * Gets the time in unix time when the permission will expire
@@ -100,7 +94,6 @@ public interface HeldPermission<T> {
     *
     * @return a Node copy of this permission
     */
-    @Nonnull
-    Node asNode();
+    @NonNull Node asNode();

 }
@@ -25,9 +25,9 @@

 package me.lucko.luckperms.api;

-import java.util.Map;
+import org.checkerframework.checker.nullness.qual.NonNull;

-import javax.annotation.Nonnull;
+import java.util.Map;

 /**
 * Wrapper around parts of the LuckPerms configuration file
@@ -39,8 +39,7 @@ public interface LPConfiguration {
     *
     * @return the name of this server
     */
-    @Nonnull
-    String getServer();
+    @NonNull String getServer();

    /**
     * Gets if the users on this server will have their global permissions applied
@@ -78,8 +77,7 @@ public interface LPConfiguration {
     *
     * @return the storage method string from the configuration
     */
-    @Nonnull
-    String getStorageMethod();
+    @NonNull String getStorageMethod();

    /**
     * Gets true if split storage is enabled
@@ -96,11 +94,9 @@ public interface LPConfiguration {
     * method. For example: key = user, value = json
     * @since 2.7
     */
-    @Nonnull
-    Map<String, String> getSplitStorageOptions();
+    @NonNull Map<String, String> getSplitStorageOptions();

-    @Nonnull
-    Unsafe unsafe();
+    @NonNull Unsafe unsafe();

    interface Unsafe {

@@ -115,8 +111,7 @@ public interface LPConfiguration {
         * @return the corresponding object, if one is present
         * @throws IllegalArgumentException if the key isn't known
         */
-        @Nonnull
-        Object getObject(String key);
+        @NonNull Object getObject(String key);
    }

 }
@@ -25,32 +25,36 @@

 package me.lucko.luckperms.api;

-import javax.annotation.Nonnull;
-import javax.annotation.concurrent.Immutable;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
- * A node with a traceable origin
+ * An extension of {@link Node}, providing information about
+ * where the node originated from.
 *
 * @since 2.11
 */
-@Immutable
 public interface LocalizedNode extends Node {

    /**
-     * Gets the delegate node
+     * Gets the delegate node.
+     *
+     * <p>Result is never another {@link LocalizedNode} instance.</p>
     *
     * @return the node this instance is representing
     */
-    @Nonnull
-    Node getNode();
+    @NonNull Node getNode();

    /**
-     * Gets the location where the {@link Node} is inherited from
+     * Gets the location where the {@link Node} is inherited from.
     *
-     * @return where the node was inherited from. Will not return null.
-     * @see PermissionHolder#getObjectName()
+     * <p>The resultant string is the {@link PermissionHolder#getObjectName() object name} of the
+     * permission holder the node was inherited from.</p>
+     *
+     * <p>If the node was not inherited, the {@link PermissionHolder#getObjectName() object name}
+     * of the permission holder itself (the one that defined the node) will be returned.</p>
+     *
+     * @return where the node was inherited from.
     */
-    @Nonnull
-    String getLocation();
+    @NonNull String getLocation();

 }
@@ -25,13 +25,11 @@

 package me.lucko.luckperms.api;

-import java.util.SortedMap;
+import org.checkerframework.checker.nullness.qual.NonNull;
+
 import java.util.SortedSet;
 import java.util.UUID;

-import javax.annotation.Nonnull;
-import javax.annotation.concurrent.Immutable;
-
 /**
 * Represents the internal LuckPerms log.
 *
@@ -42,73 +40,22 @@ import javax.annotation.concurrent.Immutable;
 *
 * <p>All methods are thread safe, and return immutable and thread safe collections.</p>
 */
-@Immutable
 public interface Log {

    /**
-     * Get the content of the log
+     * Gets the {@link LogEntry}s that make up this log.
     *
-     * @return a {@link SortedSet} of all of the {@link LogEntry} objects in this {@link Log}
+     * @return the content
     */
-    @Nonnull
-    SortedSet<LogEntry> getContent();
+    @NonNull SortedSet<LogEntry> getContent();

    /**
-     * Get the content of the log
-     *
-     * @return all content in this log
-     */
-    @Nonnull
-    SortedSet<LogEntry> getRecent();
-
-    /**
-     * Gets the recent content separated by page
-     *
-     * @param pageNo the page number
-     * @return the page content
-     * @throws IllegalArgumentException if the pageNo is less than 1
-     * @throws IllegalStateException if the log doesn't contain enough entries to populate the page. See {@link #getRecentMaxPages()}
-     */
-    @Nonnull
-    SortedMap<Integer, LogEntry> getRecent(int pageNo);
-
-    /**
-     * Gets the max page number for the recent pages.
-     *
-     * @return the max page number allowed in the {@link #getRecent(int)} method
-     */
-    int getRecentMaxPages();
-
-
-    /**
-     * Gets the recent content for a given actor
+     * Gets the entries in the log performed by the given actor.
     *
     * @param actor the uuid of the actor to filter by
-     * @return all content in this log where is actor = uuid
+     * @return the content for the given actor
     */
-    @Nonnull
-    SortedSet<LogEntry> getRecent(@Nonnull UUID actor);
-
-    /**
-     * Gets the recent content for a given actor, separated by page
-     *
-     * @param pageNo the page number
-     * @param actor  the uuid of the actor to filter by
-     * @return the page content
-     * @throws IllegalArgumentException if the pageNo is less than 1
-     * @throws IllegalStateException if the log doesn't contain enough entries to populate the page. See {@link #getRecentMaxPages(UUID)}
-     */
-    @Nonnull
-    SortedMap<Integer, LogEntry> getRecent(int pageNo, @Nonnull UUID actor);
-
-    /**
-     * Gets the max page number for the recent pages.
-     *
-     * @param actor the actor to filter by
-     * @return the max page number allowed in the {@link #getRecent(int, UUID)} method
-     */
-    int getRecentMaxPages(@Nonnull UUID actor);
-
+    @NonNull SortedSet<LogEntry> getContent(@NonNull UUID actor);

    /**
     * Gets the log content for a given user
@@ -116,29 +63,7 @@ public interface Log {
     * @param uuid the uuid to filter by
     * @return all content in this log where the user = uuid
     */
-    @Nonnull
-    SortedSet<LogEntry> getUserHistory(@Nonnull UUID uuid);
-
-    /**
-     * Gets the log content for a given user, separated by page
-     *
-     * @param pageNo the page number
-     * @param uuid   the uuid of the acted user to filter by
-     * @return the page content
-     * @throws IllegalArgumentException if the pageNo is less than 1
-     * @throws IllegalStateException if the log doesn't contain enough entries to populate the page. See {@link #getUserHistoryMaxPages(UUID)}
-     */
-    @Nonnull
-    SortedMap<Integer, LogEntry> getUserHistory(int pageNo, @Nonnull UUID uuid);
-
-    /**
-     * Gets the max page number for the user history pages.
-     *
-     * @param uuid the uuid to filter by
-     * @return the max page number allowed in the {@link #getUserHistory(int, UUID)} method
-     */
-    int getUserHistoryMaxPages(@Nonnull UUID uuid);
-
+    @NonNull SortedSet<LogEntry> getUserHistory(@NonNull UUID uuid);

    /**
     * Gets the log content for a given group
@@ -146,29 +71,7 @@ public interface Log {
     * @param name the name to filter by
     * @return all content in this log where the group = name
     */
-    @Nonnull
-    SortedSet<LogEntry> getGroupHistory(@Nonnull String name);
-
-    /**
-     * Gets the log content for a given group, separated by page
-     *
-     * @param pageNo the page number
-     * @param name   the name of the acted group to filter by
-     * @return the page content
-     * @throws IllegalArgumentException if the pageNo is less than 1
-     * @throws IllegalStateException if the log doesn't contain enough entries to populate the page. See {@link #getGroupHistoryMaxPages(String)}
-     */
-    @Nonnull
-    SortedMap<Integer, LogEntry> getGroupHistory(int pageNo, @Nonnull String name);
-
-    /**
-     * Gets the max page number for the group history pages.
-     *
-     * @param name the name to filter by
-     * @return the max page number allowed in the {@link #getGroupHistory(int, String)} method
-     */
-    int getGroupHistoryMaxPages(@Nonnull String name);
-
+    @NonNull SortedSet<LogEntry> getGroupHistory(@NonNull String name);

    /**
     * Gets the log content for a given track
@@ -176,57 +79,6 @@ public interface Log {
     * @param name the name to filter by
     * @return all content in this log where the track = name
     */
-    @Nonnull
-    SortedSet<LogEntry> getTrackHistory(@Nonnull String name);
-
-    /**
-     * Gets the log content for a given track, separated by page
-     *
-     * @param pageNo the page number
-     * @param name   the name of the acted track to filter by
-     * @return the page content
-     * @throws IllegalArgumentException if the pageNo is less than 1
-     * @throws IllegalStateException if the log doesn't contain enough entries to populate the page. See {@link #getTrackHistoryMaxPages(String)}
-     */
-    @Nonnull
-    SortedMap<Integer, LogEntry> getTrackHistory(int pageNo, @Nonnull String name);
-
-    /**
-     * Gets the max page number for the track history pages.
-     *
-     * @param name the name to filter by
-     * @return the max page number allowed in the {@link #getTrackHistory(int, String)} method
-     */
-    int getTrackHistoryMaxPages(@Nonnull String name);
-
-
-    /**
-     * Gets the log content for a given search query
-     *
-     * @param query the query to filter by
-     * @return all content in this log where the content matches query
-     */
-    @Nonnull
-    SortedSet<LogEntry> getSearch(@Nonnull String query);
-
-    /**
-     * Gets the log content for a given search query, separated by page
-     *
-     * @param pageNo the page number
-     * @param query  the query to filter by
-     * @return the page content
-     * @throws IllegalArgumentException if the pageNo is less than 1
-     * @throws IllegalStateException if the log doesn't contain enough entries to populate the page. See {@link #getSearchMaxPages(String)}}
-     */
-    @Nonnull
-    SortedMap<Integer, LogEntry> getSearch(int pageNo, @Nonnull String query);
-
-    /**
-     * Gets the max page number for the search query pages.
-     *
-     * @param query the query to filter by
-     * @return the max page number allowed in the {@link #getSearch(int, String)} method
-     */
-    int getSearchMaxPages(@Nonnull String query);
+    @NonNull SortedSet<LogEntry> getTrackHistory(@NonNull String name);

 }
@@ -25,19 +25,17 @@

 package me.lucko.luckperms.api;

+import org.checkerframework.checker.nullness.qual.NonNull;
+import org.checkerframework.checker.nullness.qual.Nullable;
+
 import java.util.Optional;
 import java.util.UUID;

-import javax.annotation.Nonnull;
-import javax.annotation.Nullable;
-import javax.annotation.concurrent.Immutable;
-
 /**
 * Represents a logged action.
 *
 * @see ActionLogger#newEntryBuilder() for creating an instance
 */
-@Immutable
 public interface LogEntry extends Comparable<LogEntry> {

    /**
@@ -54,24 +52,21 @@ public interface LogEntry extends Comparable<LogEntry> {
     *
     * @return the actor id
     */
-    @Nonnull
-    UUID getActor();
+    @NonNull UUID getActor();

    /**
     * Gets the name describing the actor.
     *
     * @return the name of the actor
     */
-    @Nonnull
-    String getActorName();
+    @NonNull String getActorName();

    /**
     * Gets the type of action.
     *
     * @return the action type
     */
-    @Nonnull
-    Type getType();
+    @NonNull Type getType();

    /**
     * Gets the uuid of the object which was acted upon.
@@ -80,16 +75,14 @@ public interface LogEntry extends Comparable<LogEntry> {
     *
     * @return the uuid of acted object
     */
-    @Nonnull
-    Optional<UUID> getActed();
+    @NonNull Optional<UUID> getActed();

    /**
     * Gets the name describing the object which was acted upon
     *
     * @return the name of the acted object
     */
-    @Nonnull
-    String getActedName();
+    @NonNull String getActedName();

    /**
     * Returns a string describing the action which took place.
@@ -99,8 +92,7 @@ public interface LogEntry extends Comparable<LogEntry> {
     *
     * @return the action
     */
-    @Nonnull
-    String getAction();
+    @NonNull String getAction();

    /**
     * Represents the type of a {@link LogEntry}.
@@ -108,22 +100,9 @@ public interface LogEntry extends Comparable<LogEntry> {
     * @since 3.3
     */
    enum Type {
-        USER('U'),
-        GROUP('G'),
-        TRACK('T');
+        USER('U'), GROUP('G'), TRACK('T');

-        private final char code;
-
-        Type(char code) {
-            this.code = code;
-        }
-
-        public char getCode() {
-            return this.code;
-        }
-
-        @Nonnull
-        public static Type valueOf(char code) {
+        public static @NonNull Type valueOf(char code) {
            switch (code) {
                case 'U':
                case 'u':
@@ -138,6 +117,16 @@ public interface LogEntry extends Comparable<LogEntry> {
                    throw new IllegalArgumentException("Unknown code: " + code);
            }
        }
+
+        private final char code;
+
+        Type(char code) {
+            this.code = code;
+        }
+
+        public char getCode() {
+            return this.code;
+        }
    }

    /**
@@ -152,8 +141,7 @@ public interface LogEntry extends Comparable<LogEntry> {
         * @return the builder
         * @see LogEntry#getTimestamp()
         */
-        @Nonnull
-        Builder setTimestamp(long timestamp);
+        @NonNull Builder setTimestamp(long timestamp);

        /**
         * Sets the actor of the entry.
@@ -162,8 +150,7 @@ public interface LogEntry extends Comparable<LogEntry> {
         * @return the builder
         * @see LogEntry#getActor()
         */
-        @Nonnull
-        Builder setActor(@Nonnull UUID actor);
+        @NonNull Builder setActor(@NonNull UUID actor);

        /**
         * Sets the actor name of the entry.
@@ -172,8 +159,7 @@ public interface LogEntry extends Comparable<LogEntry> {
         * @return the builder
         * @see LogEntry#getActorName()
         */
-        @Nonnull
-        Builder setActorName(@Nonnull String actorName);
+        @NonNull Builder setActorName(@NonNull String actorName);

        /**
         * Sets the type of the entry.
@@ -182,8 +168,7 @@ public interface LogEntry extends Comparable<LogEntry> {
         * @return the builder
         * @see LogEntry#getType()
         */
-        @Nonnull
-        Builder setType(@Nonnull Type type);
+        @NonNull Builder setType(@NonNull Type type);

        /**
         * Sets the acted object for the entry.
@@ -192,8 +177,7 @@ public interface LogEntry extends Comparable<LogEntry> {
         * @return the builder
         * @see LogEntry#getActed()
         */
-        @Nonnull
-        Builder setActed(@Nullable UUID acted);
+        @NonNull Builder setActed(@Nullable UUID acted);

        /**
         * Sets the acted name for the entry.
@@ -202,8 +186,7 @@ public interface LogEntry extends Comparable<LogEntry> {
         * @return the builder
         * @see LogEntry#getActedName()
         */
-        @Nonnull
-        Builder setActedName(@Nonnull String actedName);
+        @NonNull Builder setActedName(@NonNull String actedName);

        /**
         * Sets the action of the entry.
@@ -212,16 +195,14 @@ public interface LogEntry extends Comparable<LogEntry> {
         * @return the builder
         * @see LogEntry#getAction()
         */
-        @Nonnull
-        Builder setAction(@Nonnull String action);
+        @NonNull Builder setAction(@NonNull String action);

        /**
         * Creates a {@link LogEntry} instance from the builder.
         *
         * @return a new log entry instance
         */
-        @Nonnull
-        LogEntry build();
+        @NonNull LogEntry build();

    }

@@ -37,14 +37,14 @@ import me.lucko.luckperms.api.messenger.MessengerProvider;
 import me.lucko.luckperms.api.metastacking.MetaStackFactory;
 import me.lucko.luckperms.api.platform.PlatformInfo;

+import org.checkerframework.checker.nullness.qual.NonNull;
+import org.checkerframework.checker.nullness.qual.Nullable;
+
 import java.util.Optional;
 import java.util.Set;
 import java.util.UUID;
 import java.util.concurrent.CompletableFuture;

-import javax.annotation.Nonnull;
-import javax.annotation.Nullable;
-
 /**
 * The LuckPerms API.
 *
@@ -62,8 +62,7 @@ public interface LuckPermsApi {
     * @return the platform info
     * @since 4.0
     */
-    @Nonnull
-    PlatformInfo getPlatformInfo();
+    @NonNull PlatformInfo getPlatformInfo();

    /**
     * Gets the {@link UserManager}, responsible for managing
@@ -78,8 +77,7 @@ public interface LuckPermsApi {
     * @return the user manager
     * @since 4.0
     */
-    @Nonnull
-    UserManager getUserManager();
+    @NonNull UserManager getUserManager();

    /**
     * Gets the {@link GroupManager}, responsible for managing
@@ -94,8 +92,7 @@ public interface LuckPermsApi {
     * @return the group manager
     * @since 4.0
     */
-    @Nonnull
-    GroupManager getGroupManager();
+    @NonNull GroupManager getGroupManager();

    /**
     * Gets the {@link TrackManager}, responsible for managing
@@ -110,8 +107,7 @@ public interface LuckPermsApi {
     * @return the track manager
     * @since 4.0
     */
-    @Nonnull
-    TrackManager getTrackManager();
+    @NonNull TrackManager getTrackManager();

    /**
     * Schedules the execution of an update task, and returns an encapsulation
@@ -125,8 +121,7 @@ public interface LuckPermsApi {
     * @return a future
     * @since 4.0
     */
-    @Nonnull
-    CompletableFuture<Void> runUpdateTask();
+    @NonNull CompletableFuture<Void> runUpdateTask();

    /**
     * Gets the {@link EventBus}, used for subscribing to internal LuckPerms
@@ -135,16 +130,14 @@ public interface LuckPermsApi {
     * @return the event bus
     * @since 3.0
     */
-    @Nonnull
-    EventBus getEventBus();
+    @NonNull EventBus getEventBus();

    /**
     * Gets a representation of the plugins configuration
     *
     * @return the configuration
     */
-    @Nonnull
-    LPConfiguration getConfiguration();
+    @NonNull LPConfiguration getConfiguration();

    /**
     * Gets an object representing the plugins primary {@link Storage} backend.
@@ -155,8 +148,7 @@ public interface LuckPermsApi {
     * @return a storage instance
     * @since 2.14
     */
-    @Nonnull
-    Storage getStorage();
+    @NonNull Storage getStorage();

    /**
     * Gets the {@link MessagingService}, if present.
@@ -170,8 +162,7 @@ public interface LuckPermsApi {
     *
     * @return the messaging service instance, if present.
     */
-    @Nonnull
-    Optional<MessagingService> getMessagingService();
+    @NonNull Optional<MessagingService> getMessagingService();

    /**
     * Registers a {@link MessengerProvider} for use by the platform.
@@ -182,7 +173,7 @@ public interface LuckPermsApi {
     * @param messengerProvider the messenger provider.
     * @since 4.1
     */
-    void registerMessengerProvider(@Nonnull MessengerProvider messengerProvider);
+    void registerMessengerProvider(@NonNull MessengerProvider messengerProvider);

    /**
     * Gets the {@link ActionLogger}.
@@ -193,7 +184,7 @@ public interface LuckPermsApi {
     * @return the action logger
     * @since 4.1
     */
-    ActionLogger getActionLogger();
+    @NonNull ActionLogger getActionLogger();

    /**
     * Gets a {@link UuidCache}.
@@ -203,11 +194,10 @@ public interface LuckPermsApi {
     *
     * @return the uuid cache
     * @deprecated this feature is now handled internally - and the API returns a
-     *             No-op implementation of this class.
+     * No-op implementation of this class.
     */
-    @Nonnull
    @Deprecated
-    UuidCache getUuidCache();
+    @NonNull UuidCache getUuidCache();

    /**
     * Gets the {@link ContextManager}.
@@ -218,7 +208,7 @@ public interface LuckPermsApi {
     * @return the context manager
     * @since 4.0
     */
-    ContextManager getContextManager();
+    @NonNull ContextManager getContextManager();

    /**
     * Gets the {@link NodeFactory}.
@@ -227,8 +217,7 @@ public interface LuckPermsApi {
     *
     * @return the node factory
     */
-    @Nonnull
-    NodeFactory getNodeFactory();
+    @NonNull NodeFactory getNodeFactory();

    /**
     * Gets the {@link MetaStackFactory}.
@@ -240,8 +229,7 @@ public interface LuckPermsApi {
     * @return the meta stack factory
     * @since 3.2
     */
-    @Nonnull
-    MetaStackFactory getMetaStackFactory();
+    @NonNull MetaStackFactory getMetaStackFactory();



@@ -263,8 +251,7 @@ public interface LuckPermsApi {
     * @return a {@link User} object, if one matching the uuid is loaded, or null if not
     * @throws NullPointerException if the uuid is null
     */
-    @Nullable
-    default User getUser(@Nonnull UUID uuid) {
+    default @Nullable User getUser(@NonNull UUID uuid) {
        return getUserManager().getUser(uuid);
    }

@@ -275,8 +262,7 @@ public interface LuckPermsApi {
     * @return an optional {@link User} object
     * @throws NullPointerException if the uuid is null
     */
-    @Nonnull
-    default Optional<User> getUserSafe(@Nonnull UUID uuid) {
+    default @NonNull Optional<User> getUserSafe(@NonNull UUID uuid) {
        return getUserManager().getUserOpt(uuid);
    }

@@ -287,8 +273,7 @@ public interface LuckPermsApi {
     * @return a {@link User} object, if one matching the uuid is loaded, or null if not
     * @throws NullPointerException if the name is null
     */
-    @Nullable
-    default User getUser(@Nonnull String name) {
+    default @Nullable User getUser(@NonNull String name) {
        return getUserManager().getUser(name);
    }

@@ -299,8 +284,7 @@ public interface LuckPermsApi {
     * @return an optional {@link User} object
     * @throws NullPointerException if the name is null
     */
-    @Nonnull
-    default Optional<User> getUserSafe(@Nonnull String name) {
+    default @NonNull Optional<User> getUserSafe(@NonNull String name) {
        return getUserManager().getUserOpt(name);
    }

@@ -309,8 +293,7 @@ public interface LuckPermsApi {
     *
     * @return a {@link Set} of {@link User} objects
     */
-    @Nonnull
-    default Set<User> getUsers() {
+    default @NonNull Set<User> getUsers() {
        return getUserManager().getLoadedUsers();
    }

@@ -321,7 +304,7 @@ public interface LuckPermsApi {
     * @return true if the user is loaded
     * @throws NullPointerException if the uuid is null
     */
-    default boolean isUserLoaded(@Nonnull UUID uuid) {
+    default boolean isUserLoaded(@NonNull UUID uuid) {
        return getUserManager().isLoaded(uuid);
    }

@@ -331,7 +314,7 @@ public interface LuckPermsApi {
     * @param user the user to unload
     * @throws NullPointerException if the user is null
     */
-    default void cleanupUser(@Nonnull User user) {
+    default void cleanupUser(@NonNull User user) {
        getUserManager().cleanupUser(user);
    }

@@ -342,8 +325,7 @@ public interface LuckPermsApi {
     * @return a {@link Group} object, if one matching the name exists, or null if not
     * @throws NullPointerException if the name is null
     */
-    @Nullable
-    default Group getGroup(@Nonnull String name) {
+    default @Nullable Group getGroup(@NonNull String name) {
        return getGroupManager().getGroup(name);
    }

@@ -356,8 +338,7 @@ public interface LuckPermsApi {
     * @return an optional {@link Group} object
     * @throws NullPointerException if the name is null
     */
-    @Nonnull
-    default Optional<Group> getGroupSafe(@Nonnull String name) {
+    default @NonNull Optional<Group> getGroupSafe(@NonNull String name) {
        return getGroupManager().getGroupOpt(name);
    }

@@ -366,8 +347,7 @@ public interface LuckPermsApi {
     *
     * @return a {@link Set} of {@link Group} objects
     */
-    @Nonnull
-    default Set<Group> getGroups() {
+    default @NonNull Set<Group> getGroups() {
        return getGroupManager().getLoadedGroups();
    }

@@ -378,7 +358,7 @@ public interface LuckPermsApi {
     * @return true if the group is loaded
     * @throws NullPointerException if the name is null
     */
-    default boolean isGroupLoaded(@Nonnull String name) {
+    default boolean isGroupLoaded(@NonNull String name) {
        return getGroupManager().isLoaded(name);
    }

@@ -390,8 +370,7 @@ public interface LuckPermsApi {
     * if not
     * @throws NullPointerException if the name is null
     */
-    @Nullable
-    default Track getTrack(@Nonnull String name) {
+    default @Nullable Track getTrack(@NonNull String name) {
        return getTrackManager().getTrack(name);
    }

@@ -404,8 +383,7 @@ public interface LuckPermsApi {
     * @return an optional {@link Track} object
     * @throws NullPointerException if the name is null
     */
-    @Nonnull
-    default Optional<Track> getTrackSafe(@Nonnull String name) {
+    default @NonNull Optional<Track> getTrackSafe(@NonNull String name) {
        return getTrackManager().getTrackOpt(name);
    }

@@ -414,8 +392,7 @@ public interface LuckPermsApi {
     *
     * @return a {@link Set} of {@link Track} objects
     */
-    @Nonnull
-    default Set<Track> getTracks() {
+    default @NonNull Set<Track> getTracks() {
        return getTrackManager().getLoadedTracks();
    }

@@ -426,7 +403,7 @@ public interface LuckPermsApi {
     * @return true if the track is loaded
     * @throws NullPointerException if the name is null
     */
-    default boolean isTrackLoaded(@Nonnull String name) {
+    default boolean isTrackLoaded(@NonNull String name) {
        return getTrackManager().isLoaded(name);
    }

@@ -436,8 +413,7 @@ public interface LuckPermsApi {
     * @return a new builder
     * @since 4.0
     */
-    @Nonnull
-    default LogEntry.Builder newLogEntryBuilder() {
+    default LogEntry.@NonNull Builder newLogEntryBuilder() {
        return getActionLogger().newEntryBuilder();
    }

@@ -450,8 +426,7 @@ public interface LuckPermsApi {
     * @throws NullPointerException     if the permission is null
     * @since 2.6
     */
-    @Nonnull
-    default Node.Builder buildNode(@Nonnull String permission) throws IllegalArgumentException {
+    default Node.@NonNull Builder buildNode(@NonNull String permission) throws IllegalArgumentException {
        return getNodeFactory().newBuilder(permission);
    }

@@ -461,7 +436,7 @@ public interface LuckPermsApi {
     * @param calculator the context calculator to register. The type MUST be the player class of the platform.
     * @throws ClassCastException if the type is not the player class of the platform.
     */
-    default void registerContextCalculator(@Nonnull ContextCalculator<?> calculator) {
+    default void registerContextCalculator(@NonNull ContextCalculator<?> calculator) {
        getContextManager().registerCalculator(calculator);
    }

@@ -473,8 +448,7 @@ public interface LuckPermsApi {
     * @param user the user to get contexts for
     * @return an optional containing contexts. Will return empty if the user is not online.
     */
-    @Nonnull
-    default Optional<Contexts> getContextForUser(@Nonnull User user) {
+    default @NonNull Optional<Contexts> getContextForUser(@NonNull User user) {
        return getContextManager().lookupApplicableContexts(user);
    }

@@ -485,8 +459,7 @@ public interface LuckPermsApi {
     * @return a set of contexts.
     * @since 2.17
     */
-    @Nonnull
-    default ContextSet getContextForPlayer(@Nonnull Object player) {
+    default @NonNull ContextSet getContextForPlayer(@NonNull Object player) {
        return getContextManager().getApplicableContext(player);
    }

@@ -497,8 +470,7 @@ public interface LuckPermsApi {
     * @return a set of contexts.
     * @since 3.3
     */
-    @Nonnull
-    default Contexts getContextsForPlayer(@Nonnull Object player) {
+    default @NonNull Contexts getContextsForPlayer(@NonNull Object player) {
        return getContextManager().getApplicableContexts(player);
    }

@@ -25,7 +25,7 @@

 package me.lucko.luckperms.api;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * A means to push changes to other servers using the platforms networking
@@ -71,6 +71,6 @@ public interface MessagingService {
     * @param user the user to push the update for
     * @since 4.1
     */
-    void pushUserUpdate(@Nonnull User user);
+    void pushUserUpdate(@NonNull User user);

 }
@@ -37,6 +37,9 @@ import me.lucko.luckperms.api.nodetype.types.PrefixType;
 import me.lucko.luckperms.api.nodetype.types.SuffixType;
 import me.lucko.luckperms.api.nodetype.types.WeightType;

+import org.checkerframework.checker.nullness.qual.NonNull;
+import org.checkerframework.checker.nullness.qual.Nullable;
+
 import java.util.Date;
 import java.util.List;
 import java.util.Map;
@@ -46,10 +49,6 @@ import java.util.Set;
 import java.util.concurrent.TimeUnit;
 import java.util.stream.Stream;

-import javax.annotation.Nonnull;
-import javax.annotation.Nullable;
-import javax.annotation.concurrent.Immutable;
-
 /**
 * Represents a LuckPerms "node".
 *
@@ -71,13 +70,13 @@ import javax.annotation.concurrent.Immutable;
 * <p>Nodes have the following attributes:</p>
 * <p></p>
 * <ul>
- *     <li>{@link #getPermission() permission} - the actual permission string</li>
- *     <li>{@link #getValue() value} - the value of the node (false for negated)</li>
- *     <li>{@link #isOverride() override} - if the node is marked as having special priority over other nodes</li>
- *     <li>{@link #getServer() server} - the specific server where this node should apply</li>
- *     <li>{@link #getWorld() world} - the specific world where this node should apply</li>
- *     <li>{@link #getContexts() context} - the additional contexts required for this node to apply </li>
- *     <li>{@link #getExpiry() expiry} - the time when this node should expire</li>
+ * <li>{@link #getPermission() permission} - the actual permission string</li>
+ * <li>{@link #getValue() value} - the value of the node (false for negated)</li>
+ * <li>{@link #isOverride() override} - if the node is marked as having special priority over other nodes</li>
+ * <li>{@link #getServer() server} - the specific server where this node should apply</li>
+ * <li>{@link #getWorld() world} - the specific world where this node should apply</li>
+ * <li>{@link #getContexts() context} - the additional contexts required for this node to apply </li>
+ * <li>{@link #getExpiry() expiry} - the time when this node should expire</li>
 * </ul>
 *
 * <p>The 'permission' property of a {@link Node} is also used in some cases to represent state
@@ -90,13 +89,13 @@ import javax.annotation.concurrent.Immutable;
 * <p>The current types are:</p>
 * <p></p>
 * <ul>
- *     <li>normal - just a regular permission</li>
- *     <li>{@link InheritanceType} - an "inheritance node" marks that the holder should inherit data from another group</li>
- *     <li>{@link PrefixType} - represents an assigned prefix</li>
- *     <li>{@link SuffixType} - represents an assigned suffix</li>
- *     <li>{@link MetaType} - represents an assigned meta option</li>
- *     <li>{@link WeightType} - marks the weight of the object holding this node</li>
- *     <li>{@link DisplayNameType} - marks the display name of the object holding this node</li>
+ * <li>normal - just a regular permission</li>
+ * <li>{@link InheritanceType} - an "inheritance node" marks that the holder should inherit data from another group</li>
+ * <li>{@link PrefixType} - represents an assigned prefix</li>
+ * <li>{@link SuffixType} - represents an assigned suffix</li>
+ * <li>{@link MetaType} - represents an assigned meta option</li>
+ * <li>{@link WeightType} - marks the weight of the object holding this node</li>
+ * <li>{@link DisplayNameType} - marks the display name of the object holding this node</li>
 * </ul>
 *
 * <p>The core node state must be immutable in all implementations.</p>
@@ -104,7 +103,6 @@ import javax.annotation.concurrent.Immutable;
 * @see NodeFactory for obtaining and constructing instances.
 * @since 2.6
 */
-@Immutable
 public interface Node {

    /**
@@ -115,8 +113,7 @@ public interface Node {
     *
     * @return the actual permission node
     */
-    @Nonnull
-    String getPermission();
+    @NonNull String getPermission();

    /**
     * Gets the value of the node.
@@ -132,8 +129,7 @@ public interface Node {
     *
     * @return the value of this node as a Tristate
     */
-    @Nonnull
-    default Tristate getTristate() {
+    default @NonNull Tristate getTristate() {
        return Tristate.fromBoolean(getValue());
    }

@@ -163,16 +159,14 @@ public interface Node {
     *
     * @return an {@link Optional} containing the server, if one is defined
     */
-    @Nonnull
-    Optional<String> getServer();
+    @NonNull Optional<String> getServer();

    /**
     * Gets the world this node applies on, if the node is world specific.
     *
     * @return an {@link Optional} containing the world, if one is defined
     */
-    @Nonnull
-    Optional<String> getWorld();
+    @NonNull Optional<String> getWorld();

    /**
     * Gets if this node is server specific.
@@ -211,7 +205,7 @@ public interface Node {
     * @return true if the node should apply
     * @since 2.13
     */
-    boolean shouldApplyWithContext(@Nonnull ContextSet contextSet);
+    boolean shouldApplyWithContext(@NonNull ContextSet contextSet);

    /**
     * Resolves any shorthand parts of this node and returns the full list of
@@ -221,8 +215,7 @@ public interface Node {
     *
     * @return a list of full nodes
     */
-    @Nonnull
-    List<String> resolveShorthand();
+    @NonNull List<String> resolveShorthand();

    /**
     * Gets if this node is assigned temporarily.
@@ -254,8 +247,7 @@ public interface Node {
     * @return the {@link Date} when this node will expire
     * @throws IllegalStateException if the node is not temporary
     */
-    @Nonnull
-    Date getExpiry() throws IllegalStateException;
+    @NonNull Date getExpiry() throws IllegalStateException;

    /**
     * Gets the number of seconds until this permission will expire.
@@ -282,20 +274,18 @@ public interface Node {
     * @return the extra contexts required for this node to apply
     * @since 2.13
     */
-    @Nonnull
-    ContextSet getContexts();
+    @NonNull ContextSet getContexts();

    /**
     * The same as {@link #getContexts()}, but also includes context pairs for
     * "server" and "world" keys if present.
     *
     * @return the full contexts required for this node to apply
-     * @since 3.1
     * @see Contexts#SERVER_KEY
     * @see Contexts#WORLD_KEY
+     * @since 3.1
     */
-    @Nonnull
-    ContextSet getFullContexts();
+    @NonNull ContextSet getFullContexts();

    /**
     * Gets if this node is a wildcard permission.
@@ -347,10 +337,7 @@ public interface Node {
     * @since 4.2
     */
    default <T extends NodeType> T typeData(NodeTypeKey<T> key) throws IllegalStateException {
-        return getTypeData(key)
-                .orElseThrow(() ->
-                        new IllegalStateException("Node '" + getPermission() + "' does not have the '" + key.getTypeName() + "' type.")
-                );
+        return getTypeData(key).orElseThrow(() -> new IllegalStateException("Node '" + getPermission() + "' does not have the '" + key.getTypeName() + "' type."));
    }

    /**
@@ -369,8 +356,7 @@ public interface Node {
     * @return the name of the group
     * @throws IllegalStateException if this node doesn't have {@link InheritanceType} data
     */
-    @Nonnull
-    default String getGroupName() throws IllegalStateException {
+    default @NonNull String getGroupName() throws IllegalStateException {
        return typeData(InheritanceType.KEY).getGroupName();
    }

@@ -390,8 +376,7 @@ public interface Node {
     * @return the meta entry
     * @throws IllegalStateException if this node doesn't have {@link MetaType} data
     */
-    @Nonnull
-    default Map.Entry<String, String> getMeta() throws IllegalStateException {
+    default Map.@NonNull Entry<String, String> getMeta() throws IllegalStateException {
        return typeData(MetaType.KEY);
    }

@@ -411,8 +396,7 @@ public interface Node {
     * @return the meta entry
     * @throws IllegalStateException if this node doesn't have {@link PrefixType} data
     */
-    @Nonnull
-    default Map.Entry<Integer, String> getPrefix() throws IllegalStateException {
+    default Map.@NonNull Entry<Integer, String> getPrefix() throws IllegalStateException {
        return typeData(PrefixType.KEY).getAsEntry();
    }

@@ -432,8 +416,7 @@ public interface Node {
     * @return the meta entry
     * @throws IllegalStateException if this node doesn't have {@link SuffixType} data
     */
-    @Nonnull
-    default Map.Entry<Integer, String> getSuffix() throws IllegalStateException {
+    default Map.@NonNull Entry<Integer, String> getSuffix() throws IllegalStateException {
        return typeData(SuffixType.KEY).getAsEntry();
    }

@@ -451,7 +434,7 @@ public interface Node {
     * Gets if this Node is equal to another node as defined by the given
     * {@link StandardNodeEquality} predicate.
     *
-     * @param other the other node
+     * @param other             the other node
     * @param equalityPredicate the predicate
     * @return true if this node is considered equal
     * @since 4.1
@@ -462,7 +445,7 @@ public interface Node {
     * Gets if this Node is equal to another node as defined by the given
     * {@link NodeEqualityPredicate}.
     *
-     * @param other the other node
+     * @param other             the other node
     * @param equalityPredicate the predicate
     * @return true if this node is considered equal
     * @since 4.1
@@ -477,11 +460,11 @@ public interface Node {
     *
     * @param other the other node
     * @return true if the two nodes are almost equal
-     * @deprecated in favour of {@link #equals(Node, NodeEqualityPredicate)}
     * @see StandardNodeEquality#IGNORE_VALUE
+     * @deprecated in favour of {@link #equals(Node, NodeEqualityPredicate)}
     */
    @Deprecated
-    default boolean equalsIgnoringValue(@Nonnull Node other) {
+    default boolean equalsIgnoringValue(@NonNull Node other) {
        return equals(other, StandardNodeEquality.IGNORE_VALUE);
    }

@@ -491,11 +474,11 @@ public interface Node {
     *
     * @param other the other node
     * @return true if the two nodes are almost equal
-     * @deprecated in favour of {@link #equals(Node, NodeEqualityPredicate)}
     * @see StandardNodeEquality#IGNORE_EXPIRY_TIME_AND_VALUE
+     * @deprecated in favour of {@link #equals(Node, NodeEqualityPredicate)}
     */
    @Deprecated
-    default boolean almostEquals(@Nonnull Node other) {
+    default boolean almostEquals(@NonNull Node other) {
        return equals(other, StandardNodeEquality.IGNORE_EXPIRY_TIME_AND_VALUE);
    }

@@ -505,12 +488,12 @@ public interface Node {
     *
     * @param other the other node
     * @return true if the two nodes are almost equal
+     * @see StandardNodeEquality#IGNORE_VALUE_OR_IF_TEMPORARY
     * @since 2.8
     * @deprecated in favour of {@link #equals(Node, NodeEqualityPredicate)}
-     * @see StandardNodeEquality#IGNORE_VALUE_OR_IF_TEMPORARY
     */
    @Deprecated
-    default boolean equalsIgnoringValueOrTemp(@Nonnull Node other) {
+    default boolean equalsIgnoringValueOrTemp(@NonNull Node other) {
        return equals(other, StandardNodeEquality.IGNORE_VALUE_OR_IF_TEMPORARY);
    }

@@ -539,7 +522,7 @@ public interface Node {
         * @return the builder
         * @since 4.2
         */
-        Builder copyFrom(@Nonnull Node node);
+        Builder copyFrom(@NonNull Node node);

        /**
         * Sets the value of negated for the node.
@@ -548,8 +531,7 @@ public interface Node {
         * @return the builder
         * @see Node#isNegated()
         */
-        @Nonnull
-        Builder setNegated(boolean negated);
+        @NonNull Builder setNegated(boolean negated);

        /**
         * Sets the value of the node.
@@ -558,8 +540,7 @@ public interface Node {
         * @return the builder
         * @see Node#getValue()
         */
-        @Nonnull
-        Builder setValue(boolean value);
+        @NonNull Builder setValue(boolean value);

        /**
         * Sets the override property for the node.
@@ -571,8 +552,7 @@ public interface Node {
         * @return the builder
         * @see Node#isOverride()
         */
-        @Nonnull
-        Builder setOverride(boolean override);
+        @NonNull Builder setOverride(boolean override);

        /**
         * Sets the time when the node should expire.
@@ -584,8 +564,7 @@ public interface Node {
         * @return the builder
         * @see Node#getExpiryUnixTime()
         */
-        @Nonnull
-        Builder setExpiry(long expiryUnixTimestamp);
+        @NonNull Builder setExpiry(long expiryUnixTimestamp);

        /**
         * Sets the time when the node should expire.
@@ -594,12 +573,11 @@ public interface Node {
         * system time.</p>
         *
         * @param duration how long the node should be added for
-         * @param unit the unit <code>duration</code> is measured in
+         * @param unit     the unit <code>duration</code> is measured in
         * @return the builder
         * @since 4.2
         */
-        @Nonnull
-        default Builder setExpiry(long duration, TimeUnit unit) {
+        default @NonNull Builder setExpiry(long duration, TimeUnit unit) {
            Preconditions.checkArgument(duration > 0, "duration must be positive");
            long seconds = Objects.requireNonNull(unit, "unit").toSeconds(duration);
            long timeNow = System.currentTimeMillis() / 1000L;
@@ -612,8 +590,7 @@ public interface Node {
         * @return the builder
         * @since 4.2
         */
-        @Nonnull
-        Builder clearExpiry();
+        @NonNull Builder clearExpiry();

        /**
         * Sets the world value for the node.
@@ -622,8 +599,7 @@ public interface Node {
         * @return the builder
         * @see Node#getWorld()
         */
-        @Nonnull
-        Builder setWorld(@Nullable String world);
+        @NonNull Builder setWorld(@Nullable String world);

        /**
         * Sets the server value for the node.
@@ -632,20 +608,18 @@ public interface Node {
         * @return the builder
         * @see Node#getServer()
         */
-        @Nonnull
-        Builder setServer(@Nullable String server);
+        @NonNull Builder setServer(@Nullable String server);

        /**
         * Appends an extra context onto the node.
         *
-         * @param key the context key
+         * @param key   the context key
         * @param value the context value
         * @return the builder
         * @see ContextSet
         * @see Node#getContexts()
         */
-        @Nonnull
-        Builder withExtraContext(@Nonnull String key, @Nonnull String value);
+        @NonNull Builder withExtraContext(@NonNull String key, @NonNull String value);

        /**
         * Appends extra contexts onto the node.
@@ -655,8 +629,7 @@ public interface Node {
         * @see ContextSet
         * @see Node#getContexts()
         */
-        @Nonnull
-        Builder withExtraContext(@Nonnull Map<String, String> map);
+        @NonNull Builder withExtraContext(@NonNull Map<String, String> map);

        /**
         * Appends extra contexts onto the node.
@@ -666,8 +639,7 @@ public interface Node {
         * @see ContextSet
         * @see Node#getContexts()
         */
-        @Nonnull
-        Builder withExtraContext(@Nonnull Set<Map.Entry<String, String>> context);
+        @NonNull Builder withExtraContext(@NonNull Set<Map.Entry<String, String>> context);

        /**
         * Appends an extra context onto the node.
@@ -677,8 +649,7 @@ public interface Node {
         * @see ContextSet
         * @see Node#getContexts()
         */
-        @Nonnull
-        Builder withExtraContext(@Nonnull Map.Entry<String, String> entry);
+        @NonNull Builder withExtraContext(Map.@NonNull Entry<String, String> entry);

        /**
         * Appends extra contexts onto the node.
@@ -688,8 +659,7 @@ public interface Node {
         * @see ContextSet
         * @see Node#getContexts()
         */
-        @Nonnull
-        Builder withExtraContext(@Nonnull ContextSet contextSet);
+        @NonNull Builder withExtraContext(@NonNull ContextSet contextSet);

        /**
         * Sets the extra contexts for the node.
@@ -700,16 +670,14 @@ public interface Node {
         * @see Node#getContexts()
         * @since 4.2
         */
-        @Nonnull
-        Builder setExtraContext(@Nonnull ContextSet contextSet);
+        @NonNull Builder setExtraContext(@NonNull ContextSet contextSet);

        /**
         * Creates a {@link Node} instance from the builder.
         *
         * @return a new node instance
         */
-        @Nonnull
-        Node build();
+        @NonNull Node build();
    }

 }
@@ -25,7 +25,7 @@

 package me.lucko.luckperms.api;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * A rule for determining if two nodes are equal.
@@ -33,8 +33,7 @@ import javax.annotation.Nonnull;
 * <p>Generally, individual instances of this interface should fulfil the same
 * requirements as the {@link Object#equals(Object)} contract.</p>
 *
- * <p>Some standard implementations are provided by
- * {@link StandardNodeEquality}.</p>
+ * <p>Some standard implementations are provided by {@link StandardNodeEquality}.</p>
 *
 * @since 4.1
 */
@@ -48,6 +47,6 @@ public interface NodeEqualityPredicate {
     * @param o2 the second node
     * @return true if equal
     */
-    boolean areEqual(@Nonnull Node o1, @Nonnull Node o2);
+    boolean areEqual(@NonNull Node o1, @NonNull Node o2);

 }
@@ -25,7 +25,7 @@

 package me.lucko.luckperms.api;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * Assists with constructing {@link Node} instances.
@@ -41,8 +41,7 @@ public interface NodeFactory {
     * @return a node builder instance
     * @throws NullPointerException if the permission is null
     */
-    @Nonnull
-    Node.Builder newBuilder(@Nonnull String permission);
+    Node.@NonNull Builder newBuilder(@NonNull String permission);

    /**
     * Creates a node builder instance from an existing node
@@ -51,8 +50,7 @@ public interface NodeFactory {
     * @return a node builder instance
     * @throws NullPointerException if the other node is null
     */
-    @Nonnull
-    Node.Builder newBuilderFromExisting(@Nonnull Node other);
+    Node.@NonNull Builder newBuilderFromExisting(@NonNull Node other);


    /**
@@ -64,19 +62,17 @@ public interface NodeFactory {
     * @throws IllegalStateException if the group instance was not obtained from LuckPerms.
     * @since 3.1
     */
-    @Nonnull
-    Node.Builder makeGroupNode(@Nonnull Group group);
+    Node.@NonNull Builder makeGroupNode(@NonNull Group group);

    /**
     * Creates a node builder from a group
     *
     * @param groupName the name of the group
     * @return a node builder instance
-     * @throws NullPointerException  if the groupName is null
+     * @throws NullPointerException if the groupName is null
     * @since 4.0
     */
-    @Nonnull
-    Node.Builder makeGroupNode(@Nonnull String groupName);
+    Node.@NonNull Builder makeGroupNode(@NonNull String groupName);

    /**
     * Creates a node builder from a key value pair
@@ -86,21 +82,19 @@ public interface NodeFactory {
     * @return a node builder instance
     * @throws NullPointerException if the key or value is null
     */
-    @Nonnull
-    Node.Builder makeMetaNode(@Nonnull String key, @Nonnull String value);
+    Node.@NonNull Builder makeMetaNode(@NonNull String key, @NonNull String value);

    /**
     * Creates a node builder for the given chat meta type
     *
-     * @param type the type
+     * @param type     the type
     * @param priority the priority
-     * @param value the value for the prefix/suffix
+     * @param value    the value for the prefix/suffix
     * @return a node builder instance
     * @throws NullPointerException if the type or value is null
     * @since 3.2
     */
-    @Nonnull
-    Node.Builder makeChatMetaNode(@Nonnull ChatMetaType type, int priority, @Nonnull String value);
+    Node.@NonNull Builder makeChatMetaNode(@NonNull ChatMetaType type, int priority, @NonNull String value);

    /**
     * Creates a node builder from a prefix string and priority
@@ -110,8 +104,7 @@ public interface NodeFactory {
     * @return a node builder instance
     * @throws NullPointerException if the prefix is null
     */
-    @Nonnull
-    Node.Builder makePrefixNode(int priority, @Nonnull String prefix);
+    Node.@NonNull Builder makePrefixNode(int priority, @NonNull String prefix);

    /**
     * Creates a node builder from a prefix string and priority
@@ -121,7 +114,6 @@ public interface NodeFactory {
     * @return a node builder instance
     * @throws NullPointerException if the suffix is null
     */
-    @Nonnull
-    Node.Builder makeSuffixNode(int priority, @Nonnull String suffix);
+    Node.@NonNull Builder makeSuffixNode(int priority, @NonNull String suffix);

 }
@@ -31,46 +31,61 @@ import com.google.common.collect.Multimap;
 import me.lucko.luckperms.api.caching.CachedData;
 import me.lucko.luckperms.api.context.ContextSet;
 import me.lucko.luckperms.api.context.ImmutableContextSet;
+import me.lucko.luckperms.api.manager.GroupManager;
+import me.lucko.luckperms.api.manager.UserManager;
+import me.lucko.luckperms.api.nodetype.types.MetaType;
+import me.lucko.luckperms.api.nodetype.types.PrefixType;
+import me.lucko.luckperms.api.nodetype.types.SuffixType;
+
+import org.checkerframework.checker.nullness.qual.NonNull;

 import java.util.List;
 import java.util.Map;
 import java.util.Set;
 import java.util.SortedSet;
+import java.util.UUID;
 import java.util.concurrent.CompletableFuture;
 import java.util.function.Predicate;

-import javax.annotation.Nonnull;
-
 /**
- * An object which holds permissions.
- *
- * <p>Any changes made to permission holding objects will be lost unless the
- * instance is saved back to the {@link Storage}.</p>
+ * Generic superinterface for an object which holds permissions.
 */
 public interface PermissionHolder {

    /**
-     * Gets the objects name.
+     * Gets the objects generic name.
     *
-     * <p>{@link User#getUuid()}, {@link User#getName()} or {@link Group#getName()} should normally be used instead of
-     * this method.</p>
+     * <p>The result of this method is guaranteed to be a unique identifier for distinct instances
+     * of the same type of object.</p>
+     *
+     * <p>For {@link User}s, this method returns a {@link UUID#toString() string} representation of
+     * the users {@link User#getUuid() unique id}.</p>
+     *
+     * <p>For {@link Group}s, this method returns the {@link Group#getName() group name}.</p>
+     *
+     * <p>The {@link User#getUuid()}, {@link User#getName()} and {@link Group#getName()} methods
+     * define a "tighter" specification for obtaining object identifiers.</p>
     *
     * @return the identifier for this object. Either a uuid string or name.
     */
-    @Nonnull
-    String getObjectName();
+    @NonNull String getObjectName();

    /**
     * Gets a friendly name for this holder, to be displayed in command output, etc.
     *
-     * <p>This will <strong>always</strong> return a value, eventually falling back to {@link #getObjectName()} if no
-     * other "friendlier" identifiers are present.</p>
+     * <p>This will <strong>always</strong> return a value, eventually falling back to
+     * {@link #getObjectName()} if no other "friendlier" identifiers are present.</p>
+     *
+     * <p>For {@link User}s, this method will attempt to return the {@link User#getName() username},
+     * before falling back to {@link #getObjectName()}.</p>
+     *
+     * <p>For {@link Group}s, this method will attempt to return the groups display name, before
+     * falling back to {@link #getObjectName()}.</p>
     *
     * @return a friendly identifier for this holder
     * @since 3.2
     */
-    @Nonnull
-    String getFriendlyName();
+    @NonNull String getFriendlyName();

    /**
     * Gets the holders {@link CachedData} cache.
@@ -78,8 +93,7 @@ public interface PermissionHolder {
     * @return the holders cached data.
     * @since 3.2
     */
-    @Nonnull
-    CachedData getCachedData();
+    @NonNull CachedData getCachedData();

    /**
     * Refreshes and applies any changes to the cached holder data.
@@ -90,8 +104,7 @@ public interface PermissionHolder {
     * @return the task future
     * @since 4.0
     */
-    @Nonnull
-    CompletableFuture<Void> refreshCachedData();
+    @NonNull CompletableFuture<Void> refreshCachedData();

    /**
     * Gets the backing multimap containing every permission this holder has.
@@ -102,8 +115,7 @@ public interface PermissionHolder {
     * @return the holders own permissions
     * @since 3.3
     */
-    @Nonnull
-    ImmutableSetMultimap<ImmutableContextSet, Node> getNodes();
+    @NonNull ImmutableSetMultimap<ImmutableContextSet, Node> getNodes();

    /**
     * Gets the backing multimap containing every transient permission this holder has.
@@ -115,8 +127,7 @@ public interface PermissionHolder {
     * @return the holders own permissions
     * @since 3.3
     */
-    @Nonnull
-    ImmutableSetMultimap<ImmutableContextSet, Node> getTransientNodes();
+    @NonNull ImmutableSetMultimap<ImmutableContextSet, Node> getTransientNodes();

    /**
     * Gets a flattened/squashed view of the holders permissions.
@@ -128,60 +139,90 @@ public interface PermissionHolder {
     *
     * <p>Use {@link #getPermissions()} for a view without duplicates.</p>
     *
+     * <p>This method <b>does not</b> resolve inheritance rules.</p>
+     *
     * @return a list of the holders own nodes.
     * @since 3.3
     */
-    @Nonnull
-    List<Node> getOwnNodes();
+    @NonNull List<Node> getOwnNodes();

    /**
     * Gets a sorted set of all held permissions.
     *
+     * <p>Effectively a sorted version of {@link #getOwnNodes()}, without duplicates. Use the
+     * aforementioned method if you don't require either of these attributes.</p>
+     *
+     * <p>This method <b>does not</b> resolve inheritance rules.</p>
+     *
+     * <p>Although this method is named getPermissions, it will actually return all types of node.</p>
+     *
     * @return an immutable set of permissions in priority order
     * @since 2.6
     */
-    @Nonnull
-    SortedSet<? extends Node> getPermissions();
+    @NonNull SortedSet<? extends Node> getPermissions();

    /**
-     * Similar to {@link #getPermissions()}, except without transient permissions.
+     * Similar to {@link #getPermissions()}, except only including permissions from the enduring
+     * node map. (See {@link #getNodes()})
     *
-     * <p>Unlike transient permissions, enduring permissions will be saved to storage, and exist after the session.</p>
+     * <p>Unlike transient permissions, enduring permissions will be saved to storage, and exist
+     * after the session.</p>
+     *
+     * <p>This method <b>does not</b> resolve inheritance rules.</p>
+     *
+     * <p>Although this method is named getEnduringPermissions, it will actually return all types
+     * of node.</p>
     *
     * @return a set of nodes
     * @since 2.6
     */
-    @Nonnull
-    Set<? extends Node> getEnduringPermissions();
+    @NonNull Set<? extends Node> getEnduringPermissions();

    /**
-     * Similar to {@link #getPermissions()}, except without enduring permissions.
+     * Similar to {@link #getPermissions()}, except only including permissions from the enduring
+     * node map. (See {@link #getTransientNodes()})
     *
     * <p>Transient permissions only exist for the duration of the session.</p>
     *
+     * <p>This method <b>does not</b> resolve inheritance rules.</p>
+     *
+     * <p>Although this method is named getTransientPermissions, it will actually return all types
+     * of node.</p>
+     *
     * @return a set of nodes
     * @since 2.6
     */
-    @Nonnull
-    Set<? extends Node> getTransientPermissions();
+    @NonNull Set<? extends Node> getTransientPermissions();

    /**
-     * Processes the nodes and returns the non-temporary ones.
+     * A filtered view of this holders nodes, only including permanent entries.
+     *
+     * <p>Data is sourced from {@link #getOwnNodes()}, filtered, and then collected to a set.</p>
+     *
+     * <p>This method <b>does not</b> resolve inheritance rules.</p>
+     *
+     * <p>Although this method is named getPermanentPermissionNodes, it will actually return all types
+     * of node.</p>
     *
     * @return a set of permanent nodes
     * @since 2.6
     */
-    @Nonnull
-    Set<Node> getPermanentPermissionNodes();
+    @NonNull Set<Node> getPermanentPermissionNodes();

    /**
-     * Processes the nodes and returns the temporary ones.
+     * A filtered view of this holders nodes, only including temporary entries.
+     *
+     * <p>Data is sourced from {@link #getOwnNodes()}, filtered, and then collected to a set.</p>
+     *
+     * <p>This method <b>does not</b> resolve inheritance rules.</p>
+     *
+     * <p>Although this method is named getTemporaryPermissionNodes, it will actually return all types
+     * of node.</p>
     *
     * @return a set of temporary nodes
     * @since 2.6
     */
-    @Nonnull
-    Set<Node> getTemporaryPermissionNodes();
+    @NonNull Set<Node> getTemporaryPermissionNodes();

    /**
     * Recursively resolves this holders permissions.
@@ -191,12 +232,15 @@ public interface PermissionHolder {
     *
     * <p>This means the list will contain duplicates.</p>
     *
+     * <p>Inheritance is performed according to the platforms rules, and the order will vary
+     * depending on the accumulation order. By default, the holders own nodes are first in the list,
+     * with the entries from the end of the inheritance tree appearing last.</p>
+     *
     * @param contexts the contexts for the lookup
     * @return a list of nodes
     * @since 3.3
     */
-    @Nonnull
-    List<LocalizedNode> resolveInheritances(Contexts contexts);
+    @NonNull List<LocalizedNode> resolveInheritances(@NonNull Contexts contexts);

    /**
     * Recursively resolves this holders permissions.
@@ -209,142 +253,157 @@ public interface PermissionHolder {
     * <p>Unlike {@link #resolveInheritances(Contexts)}, this method does not
     * filter by context, at all.</p>
     *
+     * <p>Inheritance is performed according to the platforms rules, and the order will vary
+     * depending on the accumulation order. By default, the holders own nodes are first in the list,
+     * with the entries from the end of the inheritance tree appearing last.</p>
+     *
     * @return a list of nodes
     * @since 3.3
     */
-    @Nonnull
-    List<LocalizedNode> resolveInheritances();
+    @NonNull List<LocalizedNode> resolveInheritances();

    /**
     * Gets a mutable sorted set of the nodes that this object has and inherits, filtered by context
     *
-     * <p>Unlike {@link #getAllNodesFiltered(Contexts)}, this method will not filter individual nodes. The context is only
-     * used to determine which groups should apply.</p>
+     * <p>Unlike {@link #getAllNodesFiltered(Contexts)}, this method will not filter individual
+     * nodes by context. The context is only used to determine which groups should apply.</p>
     *
-     * <p>Nodes are sorted into priority order.</p>
+     * <p>Nodes are sorted into priority order. The order of inheritance is only important during
+     * the process of flattening inherited entries.</p>
     *
     * @param contexts the context for the lookup
-     * @return a mutable sorted set of permissions
+     * @return an immutable sorted set of permissions
     * @throws NullPointerException if the context is null
     * @since 2.11
     */
-    @Nonnull
-    SortedSet<LocalizedNode> getAllNodes(@Nonnull Contexts contexts);
+    @NonNull SortedSet<LocalizedNode> getAllNodes(@NonNull Contexts contexts);

    /**
     * Gets a mutable sorted set of the nodes that this object has and inherits.
     *
     * <p>Unlike {@link #getAllNodes(Contexts)}, this method does not filter by context, at all.</p>
     *
-     * <p>Nodes are sorted into priority order.</p>
+     * <p>Nodes are sorted into priority order. The order of inheritance is only important during
+     * the process of flattening inherited entries.</p>
     *
-     * @return a mutable sorted set of permissions
+     * @return an immutable sorted set of permissions
     * @throws NullPointerException if the context is null
     * @since 3.3
     */
-    @Nonnull
-    SortedSet<LocalizedNode> getAllNodes();
+    @NonNull SortedSet<LocalizedNode> getAllNodes();

    /**
     * Gets a mutable set of the nodes that this object has and inherits, filtered by context.
     *
-     * <p>Unlike {@link #getAllNodes(Contexts)}, this method WILL filter individual nodes, and only return ones that fully
-     * meet the context provided.</p>
+     * <p>Unlike {@link #getAllNodes(Contexts)}, this method WILL filter individual nodes,
+     * and only return ones that fully meet the context provided.</p>
     *
     * @param contexts the context for the lookup
     * @return a mutable set of permissions
     * @throws NullPointerException if the context is null
     * @since 2.11
     */
-    @Nonnull
-    Set<LocalizedNode> getAllNodesFiltered(@Nonnull Contexts contexts);
+    @NonNull Set<LocalizedNode> getAllNodesFiltered(@NonNull Contexts contexts);

    /**
-     * Converts the output of {@link #getAllNodesFiltered(Contexts)}, and expands shorthand permissions.
+     * Converts the output of {@link #getAllNodesFiltered(Contexts)} into string and boolean form,
+     * and expands shorthand permissions.
     *
-     * @param contexts the context for the lookup
-     * @param lowerCase if the keys should be made lowercase whilst being exported
+     * @param contexts           the context for the lookup
+     * @param convertToLowercase if the keys should be made lowercase whilst being exported
     * @return a mutable map of permissions
     */
-    @Nonnull
-    Map<String, Boolean> exportNodes(@Nonnull Contexts contexts, boolean lowerCase);
+    @NonNull Map<String, Boolean> exportNodes(@NonNull Contexts contexts, boolean convertToLowercase);

    /**
-     * Removes temporary permissions that have expired
+     * Removes any temporary permissions that have expired.
+     *
+     * <p>This method is called periodically by the platform, so it is only necessary to run
+     * if you want to guarentee that the current data is totally up-to-date.</p>
     */
    void auditTemporaryPermissions();

    /**
-     * Checks to see if the object has a certain permission
+     * Checks to see if the object has a certain permission.
     *
-     * @param node the node to check for
+     * <p>Although this method is named hasPermission, it can be used for all node types.</p>
+     *
+     * @param node              the node to check for
     * @param equalityPredicate how to determine if a node matches
     * @return a Tristate for the holders permission status for the node
     * @throws NullPointerException if the node is null
     * @since 4.1
     */
-    @Nonnull
-    Tristate hasPermission(@Nonnull Node node, @Nonnull NodeEqualityPredicate equalityPredicate);
+    @NonNull Tristate hasPermission(@NonNull Node node, @NonNull NodeEqualityPredicate equalityPredicate);

    /**
-     * Checks to see if the object has a certain permission
+     * Checks to see if the object has a certain permission.
     *
-     * @param node the node to check for
+     * <p>Although this method is named hasTransientPermission, it can be used for all node types.</p>
+     *
+     * @param node              the node to check for
     * @param equalityPredicate how to determine if a node matches
     * @return a Tristate for the holders permission status for the node
     * @throws NullPointerException if the node is null
     * @since 4.1
     */
-    @Nonnull
-    Tristate hasTransientPermission(@Nonnull Node node, @Nonnull NodeEqualityPredicate equalityPredicate);
+    @NonNull Tristate hasTransientPermission(@NonNull Node node, @NonNull NodeEqualityPredicate equalityPredicate);

    /**
-     * Checks to see if the object inherits a certain permission
+     * Checks to see if the object inherits a certain permission.
     *
-     * @param node the node to check for
+     * <p>Although this method is named inheritsPermission, it can be used for all node types.</p>
+     *
+     * @param node              the node to check for
     * @param equalityPredicate how to determine if a node matches
     * @return a Tristate for the holders inheritance status for the node
     * @throws NullPointerException if the node is null
     * @since 4.1
     */
-    @Nonnull
-    Tristate inheritsPermission(@Nonnull Node node, @Nonnull NodeEqualityPredicate equalityPredicate);
+    @NonNull Tristate inheritsPermission(@NonNull Node node, @NonNull NodeEqualityPredicate equalityPredicate);

    /**
-     * Checks to see if the object has a certain permission
+     * Checks to see if the object has a certain permission.
+     *
+     * <p>Although this method is named hasPermission, it can be used for all node types.</p>
     *
     * @param node the node to check for
     * @return a Tristate for the holders permission status for the node
     * @throws NullPointerException if the node is null
     * @since 2.6
     */
-    @Nonnull
-    Tristate hasPermission(@Nonnull Node node);
+    @NonNull Tristate hasPermission(@NonNull Node node);

    /**
-     * Checks to see if the object has a certain permission
+     * Checks to see if the object has a certain permission.
+     *
+     * <p>Although this method is named hasTransientPermission, it can be used for all node types.</p>
     *
     * @param node the node to check for
     * @return a Tristate for the holders permission status for the node
     * @throws NullPointerException if the node is null
     * @since 2.6
     */
-    @Nonnull
-    Tristate hasTransientPermission(@Nonnull Node node);
+    @NonNull Tristate hasTransientPermission(@NonNull Node node);

    /**
-     * Checks to see if the object inherits a certain permission
+     * Checks to see if the object inherits a certain permission.
+     *
+     * <p>Although this method is named inheritsPermission, it can be used for all node types.</p>
     *
     * @param node the node to check for
     * @return a Tristate for the holders inheritance status for the node
     * @throws NullPointerException if the node is null
     * @since 2.6
     */
-    @Nonnull
-    Tristate inheritsPermission(@Nonnull Node node);
+    @NonNull Tristate inheritsPermission(@NonNull Node node);

    /**
-     * Check to see if this holder inherits another group directly
+     * Check to see if this holder inherits another group in the global context.
+     *
+     * <p>"Global context" simply means an empty context set.</p>
+     *
+     * <p>This method only checks for direct inheritance - one hop up the inheritance tree.</p>
     *
     * @param group The group to check membership of
     * @return true if the group inherits the other group
@@ -352,141 +411,315 @@ public interface PermissionHolder {
     * @throws IllegalStateException if the group instance was not obtained from LuckPerms.
     * @since 4.0
     */
-    boolean inheritsGroup(@Nonnull Group group);
+    boolean inheritsGroup(@NonNull Group group);

    /**
-     * Check to see if this holder inherits another group directly
+     * Check to see if this holder inherits another group.
     *
-     * @param group The group to check membership of
+     * <p>This method only checks for direct inheritance - one hop up the inheritance tree.</p>
+     *
+     * @param group      The group to check membership of
     * @param contextSet the context set to filter by
     * @return true if the group inherits the other group
     * @throws NullPointerException  if the group is null
     * @throws IllegalStateException if the group instance was not obtained from LuckPerms.
     * @since 4.0
     */
-    boolean inheritsGroup(@Nonnull Group group, @Nonnull ContextSet contextSet);
+    boolean inheritsGroup(@NonNull Group group, @NonNull ContextSet contextSet);

    /**
-     * Sets a permission for the object
+     * Sets a permission node for the permission holder.
+     *
+     * <p>Although this method is named setPermission, it can be used for all node types.</p>
+     *
+     * <p>The effect of this mutate operation will not persist in storage unless changes are
+     * explicitly saved. If changes are not saved, the effect will only be observed until the next
+     * time the holders permission data is (re)loaded. Changes to {@link User}s should be saved
+     * using {@link UserManager#saveUser(User)}, and changes to {@link Group}s should be saved
+     * using {@link GroupManager#saveGroup(Group)}.</p>
+     *
+     * <p>Before making changes to a user or group, it may be a good idea to load a fresh copy of
+     * the backing data from the storage if you haven't done so already, to avoid overwriting changes
+     * made already. This can be done via {@link UserManager#loadUser(UUID)} or
+     * {@link GroupManager#loadGroup(String)} respectively.</p>
     *
     * @param node The node to be set
     * @return the result of the operation
     * @throws NullPointerException if the node is null
     * @since 4.0
     */
-    @Nonnull
-    DataMutateResult setPermission(@Nonnull Node node);
+    @NonNull DataMutateResult setPermission(@NonNull Node node);

    /**
-     * Sets a transient permission for the object
+     * Sets a permission node for the permission holder.
+     *
+     * <p>Although this method is named setPermission, it can be used for all node types.</p>
+     *
+     * <p>The effect of this mutate operation will not persist in storage unless changes are
+     * explicitly saved. If changes are not saved, the effect will only be observed until the next
+     * time the holders permission data is (re)loaded. Changes to {@link User}s should be saved
+     * using {@link UserManager#saveUser(User)}, and changes to {@link Group}s should be saved
+     * using {@link GroupManager#saveGroup(Group)}.</p>
+     *
+     * <p>Before making changes to a user or group, it may be a good idea to load a fresh copy of
+     * the backing data from the storage if you haven't done so already, to avoid overwriting changes
+     * made already. This can be done via {@link UserManager#loadUser(UUID)} or
+     * {@link GroupManager#loadGroup(String)} respectively.</p>
+     *
+     * @param node                    The node to be set
+     * @param temporaryMergeBehaviour The behaviour used to merge temporary permission entries
+     * @return the result of the operation
+     * @throws NullPointerException if the node is null
+     * @since 4.3
+     */
+    @NonNull TemporaryDataMutateResult setPermission(@NonNull Node node, @NonNull TemporaryMergeBehaviour temporaryMergeBehaviour);
+
+    /**
+     * Sets a transient permission for the permission holder.
     *
     * <p>A transient node is a permission that does not persist.
-     * Whenever a user logs out of the server, or the server restarts, this permission will disappear.
-     * It is never saved to the datastore, and therefore will not apply on other servers.</p>
+     * Whenever a user logs out of the server, or the server restarts, this permission will
+     * disappear. It is never saved to the datastore, and therefore will not apply on other
+     * servers.</p>
     *
-     * <p>This is useful if you want to temporarily set a permission for a user while they're online, but don't
-     * want it to persist, and have to worry about removing it when they log out.</p>
+     * <p>This is useful if you want to temporarily set a permission for a user while they're
+     * online, but don't want it to persist, and have to worry about removing it when they log
+     * out.</p>
     *
-     * <p>For unsetting a transient permission, see {@link #unsetTransientPermission(Node)}</p>
+     * <p>For unsetting a transient permission, see {@link #unsetTransientPermission(Node)}.</p>
+     *
+     * <p>Although this method is named setTransientPermission, it can be used for all node types.</p>
     *
     * @param node The node to be set
     * @return the result of the operation
     * @throws NullPointerException if the node is null
     * @since 4.0
     */
-    @Nonnull
-    DataMutateResult setTransientPermission(@Nonnull Node node);
+    @NonNull DataMutateResult setTransientPermission(@NonNull Node node);

    /**
-     * Unsets a permission for the object
+     * Sets a transient permission for the permission holder.
+     *
+     * <p>A transient node is a permission that does not persist.
+     * Whenever a user logs out of the server, or the server restarts, this permission will
+     * disappear. It is never saved to the datastore, and therefore will not apply on other
+     * servers.</p>
+     *
+     * <p>This is useful if you want to temporarily set a permission for a user while they're
+     * online, but don't want it to persist, and have to worry about removing it when they log
+     * out.</p>
+     *
+     * <p>For unsetting a transient permission, see {@link #unsetTransientPermission(Node)}.</p>
+     *
+     * <p>Although this method is named setTransientPermission, it can be used for all node types.</p>
+     *
+     * @param node                    The node to be se
+     * @param temporaryMergeBehaviour The behaviour used to merge temporary permission entries
+     * @return the result of the operation
+     * @throws NullPointerException if the node is null
+     * @since 4.3
+     */
+    @NonNull TemporaryDataMutateResult setTransientPermission(@NonNull Node node, @NonNull TemporaryMergeBehaviour temporaryMergeBehaviour);
+
+    /**
+     * Unsets a permission for the permission holder.
+     *
+     * <p>Although this method is named unsetPermission, it can be used for all node types.</p>
+     *
+     * <p>The effect of this mutate operation will not persist in storage unless changes are
+     * explicitly saved. If changes are not saved, the effect will only be observed until the next
+     * time the holders permission data is (re)loaded. Changes to {@link User}s should be saved
+     * using {@link UserManager#saveUser(User)}, and changes to {@link Group}s should be saved
+     * using {@link GroupManager#saveGroup(Group)}.</p>
+     *
+     * <p>Before making changes to a user or group, it may be a good idea to load a fresh copy of
+     * the backing data from the storage if you haven't done so already, to avoid overwriting changes
+     * made already. This can be done via {@link UserManager#loadUser(UUID)} or
+     * {@link GroupManager#loadGroup(String)} respectively.</p>
     *
     * @param node The node to be unset
     * @return the result of the operation
     * @throws NullPointerException if the node is null
     * @since 4.0
     */
-    @Nonnull
-    DataMutateResult unsetPermission(@Nonnull Node node);
+    @NonNull DataMutateResult unsetPermission(@NonNull Node node);

    /**
-     * Unsets a transient permission for the object
+     * Unsets a transient permission for the permission holder.
+     *
+     * <p>Although this method is named unsetTransientPermission, it can be used for all node types.</p>
     *
     * @param node The node to be unset
     * @return the result of the operation
     * @throws NullPointerException if the node is null
     * @since 4.0
     */
-    @Nonnull
-    DataMutateResult unsetTransientPermission(@Nonnull Node node);
+    @NonNull DataMutateResult unsetTransientPermission(@NonNull Node node);

    /**
-     * Clears any nodes from the holder which pass the predicate
+     * Clears any nodes from the holder which pass the predicate.
+     *
+     * <p>This method only targets enduring data.</p>
+     *
+     * <p>The effect of this mutate operation will not persist in storage unless changes are
+     * explicitly saved. If changes are not saved, the effect will only be observed until the next
+     * time the holders permission data is (re)loaded. Changes to {@link User}s should be saved
+     * using {@link UserManager#saveUser(User)}, and changes to {@link Group}s should be saved
+     * using {@link GroupManager#saveGroup(Group)}.</p>
+     *
+     * <p>Before making changes to a user or group, it may be a good idea to load a fresh copy of
+     * the backing data from the storage if you haven't done so already, to avoid overwriting changes
+     * made already. This can be done via {@link UserManager#loadUser(UUID)} or
+     * {@link GroupManager#loadGroup(String)} respectively.</p>
     *
     * @param test the predicate to test for nodes which should be removed
     * @since 3.2
     */
-    void clearMatching(@Nonnull Predicate<Node> test);
+    void clearMatching(@NonNull Predicate<Node> test);

    /**
-     * Clears any transient nodes from the holder which pass the predicate
+     * Clears any transient nodes from the holder which pass the predicate.
+     *
+     * <p>The effect of this mutate operation will not persist in storage unless changes are
+     * explicitly saved. If changes are not saved, the effect will only be observed until the next
+     * time the holders permission data is (re)loaded. Changes to {@link User}s should be saved
+     * using {@link UserManager#saveUser(User)}, and changes to {@link Group}s should be saved
+     * using {@link GroupManager#saveGroup(Group)}.</p>
+     *
+     * <p>Before making changes to a user or group, it may be a good idea to load a fresh copy of
+     * the backing data from the storage if you haven't done so already, to avoid overwriting changes
+     * made already. This can be done via {@link UserManager#loadUser(UUID)} or
+     * {@link GroupManager#loadGroup(String)} respectively.</p>
     *
     * @param test the predicate to test for nodes which should be removed
     * @since 3.2
     */
-    void clearMatchingTransient(@Nonnull Predicate<Node> test);
+    void clearMatchingTransient(@NonNull Predicate<Node> test);

    /**
-     * Clears all nodes held by the object
+     * Clears all nodes held by the permission holder.
+     *
+     * <p>The effect of this mutate operation will not persist in storage unless changes are
+     * explicitly saved. If changes are not saved, the effect will only be observed until the next
+     * time the holders permission data is (re)loaded. Changes to {@link User}s should be saved
+     * using {@link UserManager#saveUser(User)}, and changes to {@link Group}s should be saved
+     * using {@link GroupManager#saveGroup(Group)}.</p>
+     *
+     * <p>Before making changes to a user or group, it may be a good idea to load a fresh copy of
+     * the backing data from the storage if you haven't done so already, to avoid overwriting changes
+     * made already. This can be done via {@link UserManager#loadUser(UUID)} or
+     * {@link GroupManager#loadGroup(String)} respectively.</p>
     *
     * @since 2.17
     */
    void clearNodes();

    /**
-     * Clears all nodes held by the object in a specific context
+     * Clears all nodes held by the permission holder in a specific context.
+     *
+     * <p>The effect of this mutate operation will not persist in storage unless changes are
+     * explicitly saved. If changes are not saved, the effect will only be observed until the next
+     * time the holders permission data is (re)loaded. Changes to {@link User}s should be saved
+     * using {@link UserManager#saveUser(User)}, and changes to {@link Group}s should be saved
+     * using {@link GroupManager#saveGroup(Group)}.</p>
+     *
+     * <p>Before making changes to a user or group, it may be a good idea to load a fresh copy of
+     * the backing data from the storage if you haven't done so already, to avoid overwriting changes
+     * made already. This can be done via {@link UserManager#loadUser(UUID)} or
+     * {@link GroupManager#loadGroup(String)} respectively.</p>
     *
     * @param contextSet the contexts to filter by
     * @since 3.2
     */
-    void clearNodes(@Nonnull ContextSet contextSet);
+    void clearNodes(@NonNull ContextSet contextSet);

    /**
-     * Clears all parent groups
+     * Clears all parent groups.
+     *
+     * <p>The effect of this mutate operation will not persist in storage unless changes are
+     * explicitly saved. If changes are not saved, the effect will only be observed until the next
+     * time the holders permission data is (re)loaded. Changes to {@link User}s should be saved
+     * using {@link UserManager#saveUser(User)}, and changes to {@link Group}s should be saved
+     * using {@link GroupManager#saveGroup(Group)}.</p>
+     *
+     * <p>Before making changes to a user or group, it may be a good idea to load a fresh copy of
+     * the backing data from the storage if you haven't done so already, to avoid overwriting changes
+     * made already. This can be done via {@link UserManager#loadUser(UUID)} or
+     * {@link GroupManager#loadGroup(String)} respectively.</p>
     *
     * @since 2.17
     */
    void clearParents();

    /**
-     * Clears all parent groups in a specific context
+     * Clears all parent groups in a specific context.
+     *
+     * <p>The effect of this mutate operation will not persist in storage unless changes are
+     * explicitly saved. If changes are not saved, the effect will only be observed until the next
+     * time the holders permission data is (re)loaded. Changes to {@link User}s should be saved
+     * using {@link UserManager#saveUser(User)}, and changes to {@link Group}s should be saved
+     * using {@link GroupManager#saveGroup(Group)}.</p>
+     *
+     * <p>Before making changes to a user or group, it may be a good idea to load a fresh copy of
+     * the backing data from the storage if you haven't done so already, to avoid overwriting changes
+     * made already. This can be done via {@link UserManager#loadUser(UUID)} or
+     * {@link GroupManager#loadGroup(String)} respectively.</p>
     *
     * @param contextSet the contexts to filter by
     * @since 3.2
     */
-    void clearParents(@Nonnull ContextSet contextSet);
+    void clearParents(@NonNull ContextSet contextSet);

    /**
-     * Clears all meta held by the object
+     * Clears all meta held by the permission holder.
+     *
+     * <p>Meta nodes in this case, are any nodes which have a {@link MetaType}, {@link PrefixType}
+     * or {@link SuffixType} type.</p>
+     *
+     * <p>The effect of this mutate operation will not persist in storage unless changes are
+     * explicitly saved. If changes are not saved, the effect will only be observed until the next
+     * time the holders permission data is (re)loaded. Changes to {@link User}s should be saved
+     * using {@link UserManager#saveUser(User)}, and changes to {@link Group}s should be saved
+     * using {@link GroupManager#saveGroup(Group)}.</p>
+     *
+     * <p>Before making changes to a user or group, it may be a good idea to load a fresh copy of
+     * the backing data from the storage if you haven't done so already, to avoid overwriting changes
+     * made already. This can be done via {@link UserManager#loadUser(UUID)} or
+     * {@link GroupManager#loadGroup(String)} respectively.</p>
     *
     * @since 2.17
     */
    void clearMeta();

    /**
-     * Clears all meta held by the object in a specific context
+     * Clears all meta held by the permission holder in a specific context.
+     *
+     * <p>Meta nodes in this case, are any nodes which have a {@link MetaType}, {@link PrefixType}
+     * or {@link SuffixType} type.</p>
+     *
+     * <p>The effect of this mutate operation will not persist in storage unless changes are
+     * explicitly saved. If changes are not saved, the effect will only be observed until the next
+     * time the holders permission data is (re)loaded. Changes to {@link User}s should be saved
+     * using {@link UserManager#saveUser(User)}, and changes to {@link Group}s should be saved
+     * using {@link GroupManager#saveGroup(Group)}.</p>
+     *
+     * <p>Before making changes to a user or group, it may be a good idea to load a fresh copy of
+     * the backing data from the storage if you haven't done so already, to avoid overwriting changes
+     * made already. This can be done via {@link UserManager#loadUser(UUID)} or
+     * {@link GroupManager#loadGroup(String)} respectively.</p>
     *
     * @param contextSet the contexts to filter by
     * @since 3.2
     */
-    void clearMeta(@Nonnull ContextSet contextSet);
+    void clearMeta(@NonNull ContextSet contextSet);

    /**
-     * Clears all transient permissions the holder has.
+     * Clears all transient nodes the permission holder has.
     */
    void clearTransientNodes();

    /**
-     * Sets a permission for the object
+     * Sets a permission for the permission holder.
     *
     * @param node The node to be set
     * @return the result of the operation
@@ -494,14 +727,13 @@ public interface PermissionHolder {
     * @since 3.1
     * @deprecated now forwards to {@link #setPermission(Node)}.
     */
-    @Nonnull
    @Deprecated
-    default DataMutateResult setPermissionUnchecked(@Nonnull Node node) {
+    default @NonNull DataMutateResult setPermissionUnchecked(@NonNull Node node) {
        return setPermission(node);
    }

    /**
-     * Sets a transient permission for the object
+     * Sets a transient permission for the permission holder.
     *
     * @param node The node to be set
     * @return the result of the operation
@@ -509,39 +741,36 @@ public interface PermissionHolder {
     * @since 3.1
     * @deprecated now forwards to {@link #setTransientPermission(Node)}
     */
-    @Nonnull
    @Deprecated
-    default DataMutateResult setTransientPermissionUnchecked(@Nonnull Node node) {
+    default @NonNull DataMutateResult setTransientPermissionUnchecked(@NonNull Node node) {
        return setTransientPermission(node);
    }

    /**
-     * Unsets a permission for the object
+     * Unsets a permission for the permission holder.
     *
     * @param node The node to be unset
-     * @throws NullPointerException if the node is null
     * @return the result of the operation
+     * @throws NullPointerException if the node is null
     * @since 3.1
     * @deprecated now forwards to {@link #unsetPermission(Node)}
     */
-    @Nonnull
    @Deprecated
-    default DataMutateResult unsetPermissionUnchecked(@Nonnull Node node) {
+    default @NonNull DataMutateResult unsetPermissionUnchecked(@NonNull Node node) {
        return unsetPermission(node);
    }

    /**
-     * Unsets a transient permission for the object
+     * Unsets a transient permission for the permission holder.
     *
     * @param node The node to be unset
-     * @throws NullPointerException if the node is null
     * @return the result of the operation
+     * @throws NullPointerException if the node is null
     * @since 3.1
     * @deprecated now forwards to {@link #unsetTransientPermission(Node)}
     */
-    @Nonnull
    @Deprecated
-    default DataMutateResult unsetTransientPermissionUnchecked(@Nonnull Node node) {
+    default @NonNull DataMutateResult unsetTransientPermissionUnchecked(@NonNull Node node) {
        return unsetTransientPermission(node);
    }

@@ -25,15 +25,20 @@

 package me.lucko.luckperms.api;

+import me.lucko.luckperms.api.manager.UserManager;
+
+import org.checkerframework.checker.nullness.qual.NonNull;
+import org.checkerframework.checker.nullness.qual.Nullable;
+
 import java.util.Set;
 import java.util.UUID;

-import javax.annotation.Nonnull;
-import javax.annotation.Nullable;
-
 /**
 * Encapsulates the result of an operation to save uuid data about a player.
 *
+ * <p>The corresponding method can be found at
+ * {@link UserManager#savePlayerData(UUID, String)}.</p>
+ *
 * @since 4.2
 */
 public interface PlayerSaveResult {
@@ -43,8 +48,7 @@ public interface PlayerSaveResult {
     *
     * @return the status
     */
-    @Nonnull
-    Set<Status> getStatus();
+    @NonNull Set<Status> getStatus();

    /**
     * Gets if the result includes a certain status code.
@@ -52,7 +56,7 @@ public interface PlayerSaveResult {
     * @param status the status to check for
     * @return if the result includes the status
     */
-    boolean includes(@Nonnull Status status);
+    boolean includes(@NonNull Status status);

    /**
     * Gets the old username involved in the result
@@ -60,8 +64,7 @@ public interface PlayerSaveResult {
     * @return the old username
     * @see Status#USERNAME_UPDATED
     */
-    @Nullable
-    String getOldUsername();
+    @Nullable String getOldUsername();

    /**
     * Gets the other uuids involved in the result
@@ -69,8 +72,7 @@ public interface PlayerSaveResult {
     * @return the other uuids
     * @see Status#OTHER_UUIDS_PRESENT_FOR_USERNAME
     */
-    @Nullable
-    Set<UUID> getOtherUuids();
+    @Nullable Set<UUID> getOtherUuids();

    /**
     * The various states the result can take
@@ -25,9 +25,9 @@

 package me.lucko.luckperms.api;

-import java.util.Optional;
+import org.checkerframework.checker.nullness.qual.NonNull;

-import javax.annotation.Nonnull;
+import java.util.Optional;

 /**
 * Encapsulates the result of {@link User}s promotion along a {@link Track}.
@@ -41,8 +41,7 @@ public interface PromotionResult extends MutateResult {
     *
     * @return the status
     */
-    @Nonnull
-    Status getStatus();
+    @NonNull Status getStatus();

    @Override
    default boolean wasSuccess() {
@@ -57,8 +56,7 @@ public interface PromotionResult extends MutateResult {
     *
     * @return the group the user was promoted from.
     */
-    @Nonnull
-    Optional<String> getGroupFrom();
+    @NonNull Optional<String> getGroupFrom();

    /**
     * Gets the name of the group the user was promoted from, if applicable.
@@ -71,8 +69,7 @@ public interface PromotionResult extends MutateResult {
     *
     * @return the group the user was promoted to.
     */
-    @Nonnull
-    Optional<String> getGroupTo();
+    @NonNull Optional<String> getGroupTo();

    /**
     * The result status
@@ -25,7 +25,7 @@

 package me.lucko.luckperms.api;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * Standard {@link NodeEqualityPredicate}s.
@@ -72,7 +72,7 @@ public enum StandardNodeEquality implements NodeEqualityPredicate {
    IGNORE_VALUE_OR_IF_TEMPORARY;

    @Override
-    public boolean areEqual(@Nonnull Node o1, @Nonnull Node o2) {
+    public boolean areEqual(@NonNull Node o1, @NonNull Node o2) {
        return o1.standardEquals(o2, this);
    }
 }
@@ -29,15 +29,15 @@ import me.lucko.luckperms.api.manager.GroupManager;
 import me.lucko.luckperms.api.manager.TrackManager;
 import me.lucko.luckperms.api.manager.UserManager;

+import org.checkerframework.checker.nullness.qual.NonNull;
+import org.checkerframework.checker.nullness.qual.Nullable;
+
 import java.util.List;
 import java.util.Set;
 import java.util.UUID;
 import java.util.concurrent.CompletableFuture;
 import java.util.concurrent.Executor;

-import javax.annotation.Nonnull;
-import javax.annotation.Nullable;
-
 /**
 * A means of loading and saving permission data to/from the backend.
 *
@@ -50,8 +50,7 @@ public interface Storage {
     *
     * @return the name of the implementation
     */
-    @Nonnull
-    String getName();
+    @NonNull String getName();

    /**
     * Gets whether the storage instance is allowing logins on the platform.
@@ -70,9 +69,8 @@ public interface Storage {
     * @throws NullPointerException if entry is null
     * @deprecated in favour of {@link ActionLogger#submit(LogEntry)}.
     */
-    @Nonnull
    @Deprecated
-    CompletableFuture<Boolean> logAction(@Nonnull LogEntry entry);
+    @NonNull CompletableFuture<Boolean> logAction(@NonNull LogEntry entry);

    /**
     * Loads and returns the entire log from storage
@@ -80,9 +78,8 @@ public interface Storage {
     * @return a log instance, could be null if loading failed
     * @deprecated in favour of {@link ActionLogger#getLog()}
     */
-    @Nonnull
    @Deprecated
-    CompletableFuture<Log> getLog();
+    @NonNull CompletableFuture<Log> getLog();

    /**
     * Loads a user's data from the main storage into the plugins local storage.
@@ -93,9 +90,8 @@ public interface Storage {
     * @throws NullPointerException if uuid is null
     * @deprecated in favour of {@link UserManager#loadUser(UUID, String)}
     */
-    @Nonnull
    @Deprecated
-    CompletableFuture<Boolean> loadUser(@Nonnull UUID uuid, @Nullable String username);
+    @NonNull CompletableFuture<Boolean> loadUser(@NonNull UUID uuid, @Nullable String username);

    /**
     * Loads a user's data from the main storage into the plugins local storage.
@@ -105,9 +101,8 @@ public interface Storage {
     * @throws NullPointerException if uuid is null
     * @deprecated in favour of {@link UserManager#loadUser(UUID)}
     */
-    @Nonnull
    @Deprecated
-    default CompletableFuture<Boolean> loadUser(@Nonnull UUID uuid) {
+    default @NonNull CompletableFuture<Boolean> loadUser(@NonNull UUID uuid) {
        return loadUser(uuid, null);
    }

@@ -122,9 +117,8 @@ public interface Storage {
     * @throws IllegalStateException if the user instance was not obtained from LuckPerms.
     * @deprecated in favour of {@link UserManager#saveUser(User)}
     */
-    @Nonnull
    @Deprecated
-    CompletableFuture<Boolean> saveUser(@Nonnull User user);
+    @NonNull CompletableFuture<Boolean> saveUser(@NonNull User user);

    /**
     * Gets a set all "unique" user UUIDs.
@@ -134,9 +128,8 @@ public interface Storage {
     * @return a set of uuids, or null if the operation failed.
     * @deprecated in favour of {@link UserManager#getUniqueUsers()}
     */
-    @Nonnull
    @Deprecated
-    CompletableFuture<Set<UUID>> getUniqueUsers();
+    @NonNull CompletableFuture<Set<UUID>> getUniqueUsers();

    /**
     * Searches for a list of users with a given permission.
@@ -147,9 +140,8 @@ public interface Storage {
     * @since 2.17
     * @deprecated in favour of {@link UserManager#getWithPermission(String)}
     */
-    @Nonnull
    @Deprecated
-    CompletableFuture<List<HeldPermission<UUID>>> getUsersWithPermission(@Nonnull String permission);
+    @NonNull CompletableFuture<List<HeldPermission<UUID>>> getUsersWithPermission(@NonNull String permission);

    /**
     * Creates and loads a group into the plugins local storage
@@ -160,9 +152,8 @@ public interface Storage {
     * @throws IllegalArgumentException if the name is invalid
     * @deprecated in favour of {@link GroupManager#createAndLoadGroup(String)}
     */
-    @Nonnull
    @Deprecated
-    CompletableFuture<Boolean> createAndLoadGroup(@Nonnull String name);
+    @NonNull CompletableFuture<Boolean> createAndLoadGroup(@NonNull String name);

    /**
     * Loads a group into the plugins local storage.
@@ -173,9 +164,8 @@ public interface Storage {
     * @throws IllegalArgumentException if the name is invalid
     * @deprecated in favour of {@link GroupManager#loadGroup(String)}
     */
-    @Nonnull
    @Deprecated
-    CompletableFuture<Boolean> loadGroup(@Nonnull String name);
+    @NonNull CompletableFuture<Boolean> loadGroup(@NonNull String name);

    /**
     * Loads all groups from the storage into memory
@@ -183,9 +173,8 @@ public interface Storage {
     * @return true if the operation completed successfully.
     * @deprecated in favour of {@link GroupManager#loadAllGroups()}
     */
-    @Nonnull
    @Deprecated
-    CompletableFuture<Boolean> loadAllGroups();
+    @NonNull CompletableFuture<Boolean> loadAllGroups();

    /**
     * Saves a group back to storage.
@@ -198,9 +187,8 @@ public interface Storage {
     * @throws IllegalStateException if the group instance was not obtained from LuckPerms.
     * @deprecated in favour of {@link GroupManager#saveGroup(Group)}
     */
-    @Nonnull
    @Deprecated
-    CompletableFuture<Boolean> saveGroup(@Nonnull Group group);
+    @NonNull CompletableFuture<Boolean> saveGroup(@NonNull Group group);

    /**
     * Permanently deletes a group from storage.
@@ -211,9 +199,8 @@ public interface Storage {
     * @throws IllegalStateException if the group instance was not obtained from LuckPerms.
     * @deprecated in favour of {@link GroupManager#deleteGroup(Group)}
     */
-    @Nonnull
    @Deprecated
-    CompletableFuture<Boolean> deleteGroup(@Nonnull Group group);
+    @NonNull CompletableFuture<Boolean> deleteGroup(@NonNull Group group);

    /**
     * Searches for a list of groups with a given permission.
@@ -224,9 +211,8 @@ public interface Storage {
     * @since 2.17
     * @deprecated in favour of {@link GroupManager#getWithPermission(String)}
     */
-    @Nonnull
    @Deprecated
-    CompletableFuture<List<HeldPermission<String>>> getGroupsWithPermission(@Nonnull String permission);
+    @NonNull CompletableFuture<List<HeldPermission<String>>> getGroupsWithPermission(@NonNull String permission);

    /**
     * Creates and loads a track into the plugins local storage
@@ -237,9 +223,8 @@ public interface Storage {
     * @throws IllegalArgumentException if the name is invalid
     * @deprecated in favour of {@link TrackManager#createAndLoadTrack(String)}
     */
-    @Nonnull
    @Deprecated
-    CompletableFuture<Boolean> createAndLoadTrack(@Nonnull String name);
+    @NonNull CompletableFuture<Boolean> createAndLoadTrack(@NonNull String name);

    /**
     * Loads a track into the plugins local storage.
@@ -250,9 +235,8 @@ public interface Storage {
     * @throws IllegalArgumentException if the name is invalid
     * @deprecated in favour of {@link TrackManager#loadTrack(String)}
     */
-    @Nonnull
    @Deprecated
-    CompletableFuture<Boolean> loadTrack(@Nonnull String name);
+    @NonNull CompletableFuture<Boolean> loadTrack(@NonNull String name);

    /**
     * Loads all tracks from the storage into memory
@@ -260,9 +244,8 @@ public interface Storage {
     * @return true if the operation completed successfully.
     * @deprecated in favour of {@link TrackManager#loadAllTracks()}
     */
-    @Nonnull
    @Deprecated
-    CompletableFuture<Boolean> loadAllTracks();
+    @NonNull CompletableFuture<Boolean> loadAllTracks();

    /**
     * Saves a track back to storage. You should call this after you make any changes to a track.
@@ -273,9 +256,8 @@ public interface Storage {
     * @throws IllegalStateException if the track instance was not obtained from LuckPerms.
     * @deprecated in favour of {@link TrackManager#saveTrack(Track)}
     */
-    @Nonnull
    @Deprecated
-    CompletableFuture<Boolean> saveTrack(@Nonnull Track track);
+    @NonNull CompletableFuture<Boolean> saveTrack(@NonNull Track track);

    /**
     * Permanently deletes a track from storage
@@ -286,9 +268,8 @@ public interface Storage {
     * @throws IllegalStateException if the track instance was not obtained from LuckPerms.
     * @deprecated in favour of {@link TrackManager#deleteTrack(Track)}
     */
-    @Nonnull
    @Deprecated
-    CompletableFuture<Boolean> deleteTrack(@Nonnull Track track);
+    @NonNull CompletableFuture<Boolean> deleteTrack(@NonNull Track track);

    /**
     * Saves UUID caching data to the global cache
@@ -300,8 +281,7 @@ public interface Storage {
     * @throws IllegalArgumentException if the username is invalid
     * @deprecated in favour of {@link UserManager#savePlayerData(UUID, String)}
     */
-    @Nonnull
-    CompletableFuture<Boolean> saveUUIDData(@Nonnull String username, @Nonnull UUID uuid);
+    @NonNull CompletableFuture<Boolean> saveUUIDData(@NonNull String username, @NonNull UUID uuid);

    /**
     * Gets a UUID from a username
@@ -312,8 +292,7 @@ public interface Storage {
     * @throws IllegalArgumentException if the username is invalid
     * @deprecated in favour of {@link UserManager#lookupUuid(String)}
     */
-    @Nonnull
-    CompletableFuture<UUID> getUUID(@Nonnull String username);
+    @NonNull CompletableFuture<UUID> getUUID(@NonNull String username);

    /**
     * Gets a username from a UUID
@@ -324,9 +303,7 @@ public interface Storage {
     * @since 2.17
     * @deprecated in favour of {@link UserManager#lookupUsername(UUID)}
     */
-    @Nonnull
-    @Deprecated
-    CompletableFuture<String> getName(@Nonnull UUID uuid);
+    @NonNull @Deprecated CompletableFuture<String> getName(@NonNull UUID uuid);

    /**
     * Returns an executor which will run all passed runnables on the
@@ -338,9 +315,7 @@ public interface Storage {
     * @return an executor instance
     * @deprecated as plugins should create their own executors
     */
-    @Nonnull
-    @Deprecated
-    Executor getSyncExecutor();
+    @NonNull @Deprecated Executor getSyncExecutor();

    /**
     * Returns an executor which will run all passed runnables asynchronously
@@ -352,8 +327,6 @@ public interface Storage {
     * @return an executor instance
     * @deprecated as plugins should create their own executors
     */
-    @Nonnull
-    @Deprecated
-    Executor getAsyncExecutor();
+    @NonNull @Deprecated Executor getAsyncExecutor();

 }
@@ -23,26 +23,33 @@
 *  SOFTWARE.
 */

-package me.lucko.luckperms.common.model;
+package me.lucko.luckperms.api;
+
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
- * Controls how temporary permissions/parents/meta should be set
+ * Extension of {@link DataMutateResult} for temporary set operations.
+ *
+ * @since 4.3
 */
-public enum TemporaryModifier {
+public interface TemporaryDataMutateResult {

    /**
-     * Durations will be added to the existing expiry time of a permission
+     * Gets the underlying result.
+     *
+     * @return the result
     */
-    ACCUMULATE,
+    @NonNull DataMutateResult getResult();

    /**
-     * Durations will be replaced if the new duration is later than the current expiration
+     * Gets the node that resulted from any {@link TemporaryMergeBehaviour}
+     * processing.
+     *
+     * <p>If no processing took place, the same instance will be returned by
+     * this method.</p>
+     *
+     * @return the resultant node
     */
-    REPLACE,
-
-    /**
-     * The command will just fail if you try to add another permission with the same expiry
-     */
-    DENY
+    @NonNull Node getMergedNode();

 }
@@ -0,0 +1,58 @@
+/*
+ * This file is part of LuckPerms, licensed under the MIT License.
+ *
+ *  Copyright (c) lucko (Luck) <luck@lucko.me>
+ *  Copyright (c) contributors
+ *
+ *  Permission is hereby granted, free of charge, to any person obtaining a copy
+ *  of this software and associated documentation files (the "Software"), to deal
+ *  in the Software without restriction, including without limitation the rights
+ *  to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+ *  copies of the Software, and to permit persons to whom the Software is
+ *  furnished to do so, subject to the following conditions:
+ *
+ *  The above copyright notice and this permission notice shall be included in all
+ *  copies or substantial portions of the Software.
+ *
+ *  THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ *  IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ *  FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+ *  AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ *  LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+ *  OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+ *  SOFTWARE.
+ */
+
+package me.lucko.luckperms.api;
+
+/**
+ * Controls how the implementation should behave when new temporary nodes are set
+ * that would otherwise conflict with existing entries.
+ *
+ * <p>The default behaviour of {@link PermissionHolder#setPermission(Node)} is
+ * to return a result of {@link DataMutateResult#ALREADY_HAS} when an equivalent
+ * node is found. This can be replicated using {@link #FAIL_WITH_ALREADY_HAS}.</p>
+ *
+ * <p>However, the {@link PermissionHolder#setPermission(Node, TemporaryMergeBehaviour)}
+ * method allows this behaviour to be customized for temporary permissions.</p>
+ *
+ * @since 4.3
+ */
+public enum TemporaryMergeBehaviour {
+
+    /**
+     * Expiry durations will be added to the existing expiry time of a permission.
+     */
+    ADD_NEW_DURATION_TO_EXISTING,
+
+    /**
+     * Expiry durations will be replaced if the new duration is longer than the current one.
+     */
+    REPLACE_EXISTING_IF_DURATION_LONGER,
+
+    /**
+     * The operation will fail if an existing temporary node is present.
+     */
+    FAIL_WITH_ALREADY_HAS
+
+}
@@ -27,10 +27,10 @@ package me.lucko.luckperms.api;

 import me.lucko.luckperms.api.context.ContextSet;

-import java.util.List;
+import org.checkerframework.checker.nullness.qual.NonNull;
+import org.checkerframework.checker.nullness.qual.Nullable;

-import javax.annotation.Nonnull;
-import javax.annotation.Nullable;
+import java.util.List;

 /**
 * An ordered chain of {@link Group}s.
@@ -42,8 +42,7 @@ public interface Track {
     *
     * @return the name of this track
     */
-    @Nonnull
-    String getName();
+    @NonNull String getName();

    /**
     * Gets a list of the groups on this track
@@ -54,8 +53,7 @@ public interface Track {
     *
     * @return an ordered {@link List} of the groups on this track
     */
-    @Nonnull
-    List<String> getGroups();
+    @NonNull List<String> getGroups();

    /**
     * Gets the number of groups on this track
@@ -74,8 +72,7 @@ public interface Track {
     * @throws NullPointerException  if the group is null
     * @throws IllegalStateException if the group instance was not obtained from LuckPerms.
     */
-    @Nullable
-    String getNext(@Nonnull Group current);
+    @Nullable String getNext(@NonNull Group current);

    /**
     * Gets the previous group on the track, before the one provided
@@ -87,41 +84,37 @@ public interface Track {
     * @throws NullPointerException  if the group is null
     * @throws IllegalStateException if the group instance was not obtained from LuckPerms.
     */
-    @Nullable
-    String getPrevious(@Nonnull Group current);
+    @Nullable String getPrevious(@NonNull Group current);

    /**
     * Promotes the given user along this track.
     *
-     * @param user the user to promote
+     * @param user       the user to promote
     * @param contextSet the contexts to promote the user in
     * @return the result of the action
     * @since 4.2
     */
-    @Nonnull
-    PromotionResult promote(@Nonnull User user, @Nonnull ContextSet contextSet);
+    @NonNull PromotionResult promote(@NonNull User user, @NonNull ContextSet contextSet);

    /**
     * Demotes the given user along this track.
     *
-     * @param user the user to demote
+     * @param user       the user to demote
     * @param contextSet the contexts to demote the user in
     * @return the result of the action
     * @since 4.2
     */
-    @Nonnull
-    DemotionResult demote(@Nonnull User user, @Nonnull ContextSet contextSet);
+    @NonNull DemotionResult demote(@NonNull User user, @NonNull ContextSet contextSet);

    /**
     * Appends a group to the end of this track
     *
     * @param group the group to append
     * @return the result of the operation
-     * @throws NullPointerException      if the group is null
-     * @throws IllegalStateException     if the group instance was not obtained from LuckPerms.
+     * @throws NullPointerException  if the group is null
+     * @throws IllegalStateException if the group instance was not obtained from LuckPerms.
     */
-    @Nonnull
-    DataMutateResult appendGroup(@Nonnull Group group);
+    @NonNull DataMutateResult appendGroup(@NonNull Group group);

    /**
     * Inserts a group at a certain position on this track
@@ -133,8 +126,7 @@ public interface Track {
     * @throws NullPointerException      if the group is null
     * @throws IllegalStateException     if the group instance was not obtained from LuckPerms.
     */
-    @Nonnull
-    DataMutateResult insertGroup(@Nonnull Group group, int position) throws IndexOutOfBoundsException;
+    @NonNull DataMutateResult insertGroup(@NonNull Group group, int position) throws IndexOutOfBoundsException;

    /**
     * Removes a group from this track
@@ -144,8 +136,7 @@ public interface Track {
     * @throws NullPointerException  if the group is null
     * @throws IllegalStateException if the group instance was not obtained from LuckPerms.
     */
-    @Nonnull
-    DataMutateResult removeGroup(@Nonnull Group group);
+    @NonNull DataMutateResult removeGroup(@NonNull Group group);

    /**
     * Removes a group from this track
@@ -154,8 +145,7 @@ public interface Track {
     * @return the result of the operation
     * @throws NullPointerException if the group is null
     */
-    @Nonnull
-    DataMutateResult removeGroup(@Nonnull String group);
+    @NonNull DataMutateResult removeGroup(@NonNull String group);

    /**
     * Checks if a group features on this track
@@ -165,7 +155,7 @@ public interface Track {
     * @throws NullPointerException  if the group is null
     * @throws IllegalStateException if the group instance was not obtained from LuckPerms.
     */
-    boolean containsGroup(@Nonnull Group group);
+    boolean containsGroup(@NonNull Group group);

    /**
     * Checks if a group features on this track
@@ -174,7 +164,7 @@ public interface Track {
     * @return true if the group is on this track
     * @throws NullPointerException if the group is null
     */
-    boolean containsGroup(@Nonnull String group);
+    boolean containsGroup(@NonNull String group);

    /**
     * Clear all of the groups from this track
@@ -25,7 +25,7 @@

 package me.lucko.luckperms.api;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * Represents three different states of a setting.
@@ -61,8 +61,7 @@ public enum Tristate {
     * @param val the boolean value
     * @return {@link #TRUE} or {@link #FALSE}, if the value is <code>true</code> or <code>false</code>, respectively.
     */
-    @Nonnull
-    public static Tristate fromBoolean(boolean val) {
+    public static @NonNull Tristate fromBoolean(boolean val) {
        return val ? TRUE : FALSE;
    }

@@ -77,8 +76,7 @@ public enum Tristate {
     *         is <code>null</code>, <code>true</code> or <code>false</code>, respectively.
     * @since 4.1
     */
-    @Nonnull
-    public static Tristate fromNullableBoolean(Boolean val) {
+    public static @NonNull Tristate fromNullableBoolean(Boolean val) {
        return val == null ? UNDEFINED : val ? TRUE : FALSE;
    }

@@ -27,10 +27,10 @@ package me.lucko.luckperms.api;

 import me.lucko.luckperms.api.caching.UserData;

-import java.util.UUID;
+import org.checkerframework.checker.nullness.qual.NonNull;
+import org.checkerframework.checker.nullness.qual.Nullable;

-import javax.annotation.Nonnull;
-import javax.annotation.Nullable;
+import java.util.UUID;

 /**
 * A player which holds permission data.
@@ -42,40 +42,40 @@ public interface User extends PermissionHolder {
     *
     * @return the users Mojang assigned unique id
     */
-    @Nonnull
-    UUID getUuid();
+    @NonNull UUID getUuid();

    /**
     * Gets the users username
     *
-     * <p>Returns null if no username is associated with this user.</p>
+     * <p>Returns null if no username is known for the user.</p>
     *
     * @return the users username
     */
-    @Nullable
-    String getName();
+    @Nullable String getName();

    /**
     * Gets the users current primary group.
     *
-     * <p>The result of this method depends on which method is configured for primary group calculation. It may not
-     * be the same as any value set through {@link #setPrimaryGroup(String)}.</p>
+     * <p>The result of this method depends on which method is configured for primary group
+     * calculation. It may not be the same as any value set through
+     * {@link #setPrimaryGroup(String)}.</p>
     *
     * @return the users primary group
     */
-    @Nonnull
-    String getPrimaryGroup();
+    @NonNull String getPrimaryGroup();

    /**
-     * Sets a users primary group. This will only take effect if platform is using stored primary groups.
+     * Sets a users primary group.
+     *
+     * <p>This modifies the "stored value" for the users primary group, which may or may not
+     * actually take effect, depending on how the platform is calculating primary groups.</p>
     *
     * @param group the new primary group
     * @return if the change was applied successfully
-     * @throws IllegalStateException     if the user is not a member of that group
-     * @throws NullPointerException      if the group is null
+     * @throws IllegalStateException if the user is not a member of that group
+     * @throws NullPointerException  if the group is null
     */
-    @Nonnull
-    DataMutateResult setPrimaryGroup(@Nonnull String group);
+    @NonNull DataMutateResult setPrimaryGroup(@NonNull String group);

    /**
     * Gets the user's {@link UserData} cache.
@@ -83,17 +83,15 @@ public interface User extends PermissionHolder {
     * @return the users cached data.
     * @since 3.2
     */
-    @Nonnull
    @Override
-    UserData getCachedData();
+    @NonNull UserData getCachedData();

    /**
     * Refresh and re-assign the users permissions.
     *
-     * <p>This request is not buffered, and the refresh call will be ran directly. This should be called on an
-     * asynchronous thread.</p>
-     *
-     * @deprecated in favour of {@link #refreshCachedData()}.
+     * @deprecated Calling this method is no longer necessary. Permissions data is now refreshed on
+     * demand, as changes are made. Consider use of {@link #refreshCachedData()}
+     * instead. This method is now implemented as a no-op.
     */
    @Deprecated
    void refreshPermissions();
@@ -101,11 +99,8 @@ public interface User extends PermissionHolder {
    /**
     * Pre-calculates some values in the user's data cache.
     *
-     * <p>Is it <b>not</b> necessary to call this method before
-     * using {@link #getCachedData()}.</p>
-     *
     * @since 2.17
-     * @deprecated because use of this method is no longer necessary.
+     * @deprecated Use of this method is no longer necessary. It is implemented as a no-op.
     */
    @Deprecated
    void setupDataCache();
@@ -25,16 +25,16 @@

 package me.lucko.luckperms.api;

-import java.util.UUID;
+import org.checkerframework.checker.nullness.qual.NonNull;

-import javax.annotation.Nonnull;
+import java.util.UUID;

 /**
 * A UUID cache for online users, between external Mojang UUIDs, and internal
 * LuckPerms UUIDs.
 *
 * @deprecated this feature is now handled internally - and the API returns a
- *             No-op implementation of this class.
+ * No-op implementation of this class.
 */
@Deprecated
 public interface UuidCache {
@@ -50,9 +50,8 @@ public interface UuidCache {
     *                   <code>ProxiedPlayer#getUniqueId</code>
     * @return the corresponding internal UUID
     */
-    @Nonnull
    @Deprecated
-    UUID getUUID(@Nonnull UUID mojangUuid);
+    @NonNull UUID getUUID(@NonNull UUID mojangUuid);

    /**
     * Gets a users "external", server assigned unique id, from the internal
@@ -62,8 +61,7 @@ public interface UuidCache {
     *                     <code>User#getUuid</code>
     * @return the corresponding external UUID
     */
-    @Nonnull
    @Deprecated
-    UUID getExternalUUID(@Nonnull UUID internalUuid);
+    @NonNull UUID getExternalUUID(@NonNull UUID internalUuid);

 }
@@ -28,11 +28,12 @@ package me.lucko.luckperms.api.caching;
 import me.lucko.luckperms.api.Contexts;
 import me.lucko.luckperms.api.PermissionHolder;

+import org.checkerframework.checker.nullness.qual.NonNull;
+
+import java.util.Objects;
 import java.util.Set;
 import java.util.concurrent.CompletableFuture;

-import javax.annotation.Nonnull;
-
 /**
 * Holds cached permission and meta lookup data for a {@link PermissionHolder}.
 *
@@ -56,8 +57,7 @@ public interface CachedData {
     * @return a permission data instance
     * @throws NullPointerException if contexts is null
     */
-    @Nonnull
-    PermissionData getPermissionData(@Nonnull Contexts contexts);
+    @NonNull PermissionData getPermissionData(@NonNull Contexts contexts);

    /**
     * Gets MetaData from the cache, given a specified context.
@@ -67,8 +67,7 @@ public interface CachedData {
     * @throws NullPointerException if contexts is null
     * @since 3.2
     */
-    @Nonnull
-    MetaData getMetaData(@Nonnull MetaContexts contexts);
+    @NonNull MetaData getMetaData(@NonNull MetaContexts contexts);

    /**
     * Gets MetaData from the cache, given a specified context.
@@ -77,8 +76,7 @@ public interface CachedData {
     * @return a meta data instance
     * @throws NullPointerException if contexts is null
     */
-    @Nonnull
-    MetaData getMetaData(@Nonnull Contexts contexts);
+    @NonNull MetaData getMetaData(@NonNull Contexts contexts);

    /**
     * Calculates permission data, bypassing the cache.
@@ -95,8 +93,7 @@ public interface CachedData {
     * @return a permission data instance
     * @throws NullPointerException if contexts is null
     */
-    @Nonnull
-    PermissionData calculatePermissions(@Nonnull Contexts contexts);
+    @NonNull PermissionData calculatePermissions(@NonNull Contexts contexts);

    /**
     * Calculates meta data, bypassing the cache.
@@ -114,8 +111,7 @@ public interface CachedData {
     * @throws NullPointerException if contexts is null
     * @since 3.2
     */
-    @Nonnull
-    MetaData calculateMeta(@Nonnull MetaContexts contexts);
+    @NonNull MetaData calculateMeta(@NonNull MetaContexts contexts);

    /**
     * Calculates meta data, bypassing the cache.
@@ -132,8 +128,7 @@ public interface CachedData {
     * @return a meta data instance
     * @throws NullPointerException if contexts is null
     */
-    @Nonnull
-    MetaData calculateMeta(@Nonnull Contexts contexts);
+    @NonNull MetaData calculateMeta(@NonNull Contexts contexts);

    /**
     * (Re)calculates permission data for a given context.
@@ -151,7 +146,7 @@ public interface CachedData {
     * @param contexts the contexts to recalculate in.
     * @throws NullPointerException if contexts is null
     */
-    void recalculatePermissions(@Nonnull Contexts contexts);
+    void recalculatePermissions(@NonNull Contexts contexts);

    /**
     * (Re)calculates meta data for a given context.
@@ -170,7 +165,7 @@ public interface CachedData {
     * @throws NullPointerException if contexts is null
     * @since 3.2
     */
-    void recalculateMeta(@Nonnull MetaContexts contexts);
+    void recalculateMeta(@NonNull MetaContexts contexts);

    /**
     * (Re)calculates meta data for a given context.
@@ -188,7 +183,7 @@ public interface CachedData {
     * @param contexts the contexts to recalculate in.
     * @throws NullPointerException if contexts is null
     */
-    void recalculateMeta(@Nonnull Contexts contexts);
+    void recalculateMeta(@NonNull Contexts contexts);

    /**
     * (Re)loads permission data for a given context.
@@ -211,12 +206,11 @@ public interface CachedData {
     * until the recalculation has been performed.</p>
     *
     * @param contexts the contexts to reload in.
-     * @throws NullPointerException if contexts is null
     * @return a future
+     * @throws NullPointerException if contexts is null
     * @since 4.0
     */
-    @Nonnull
-    CompletableFuture<? extends PermissionData> reloadPermissions(@Nonnull Contexts contexts);
+    @NonNull CompletableFuture<? extends PermissionData> reloadPermissions(@NonNull Contexts contexts);

    /**
     * (Re)loads meta data for a given context.
@@ -239,12 +233,11 @@ public interface CachedData {
     * until the recalculation has been performed.</p>
     *
     * @param contexts the contexts to reload in.
-     * @throws NullPointerException if contexts is null
     * @return a future
+     * @throws NullPointerException if contexts is null
     * @since 4.0
     */
-    @Nonnull
-    CompletableFuture<? extends MetaData> reloadMeta(@Nonnull MetaContexts contexts);
+    @NonNull CompletableFuture<? extends MetaData> reloadMeta(@NonNull MetaContexts contexts);

    /**
     * (Re)loads meta data for a given context.
@@ -267,12 +260,11 @@ public interface CachedData {
     * until the recalculation has been performed.</p>
     *
     * @param contexts the contexts to reload in.
-     * @throws NullPointerException if contexts is null
     * @return a future
+     * @throws NullPointerException if contexts is null
     * @since 4.0
     */
-    @Nonnull
-    CompletableFuture<? extends MetaData> reloadMeta(@Nonnull Contexts contexts);
+    @NonNull CompletableFuture<? extends MetaData> reloadMeta(@NonNull Contexts contexts);

    /**
     * Recalculates permission data for all known contexts.
@@ -317,8 +309,7 @@ public interface CachedData {
     * @return a future
     * @since 4.0
     */
-    @Nonnull
-    CompletableFuture<Void> reloadPermissions();
+    @NonNull CompletableFuture<Void> reloadPermissions();

    /**
     * Reloads meta data for all known contexts.
@@ -341,8 +332,7 @@ public interface CachedData {
     * @return a future
     * @since 4.0
     */
-    @Nonnull
-    CompletableFuture<Void> reloadMeta();
+    @NonNull CompletableFuture<Void> reloadMeta();

    /**
     * Pre-calculates and caches {@link PermissionData} and {@link MetaData}
@@ -356,7 +346,7 @@ public interface CachedData {
     * @param contexts a set of contexts
     * @throws NullPointerException if contexts is null
     */
-    default void preCalculate(@Nonnull Set<Contexts> contexts) {
+    default void preCalculate(@NonNull Set<Contexts> contexts) {
        contexts.forEach(this::preCalculate);
    }

@@ -372,7 +362,14 @@ public interface CachedData {
     * @param contexts the contexts to pre-calculate for
     * @throws NullPointerException if contexts is null
     */
-    void preCalculate(@Nonnull Contexts contexts);
+    default void preCalculate(@NonNull Contexts contexts) {
+        Objects.requireNonNull(contexts, "contexts");
+
+        // pre-calculate just by requesting the data from this cache.
+        // if the data isn't already loaded, it will be calculated.
+        getPermissionData(contexts);
+        getMetaData(contexts);
+    }

    /**
     * Invalidates any cached {@link PermissionData} instances mapped to the given
@@ -381,7 +378,7 @@ public interface CachedData {
     * @param contexts the contexts to invalidate for
     * @since 4.0
     */
-    void invalidatePermissions(@Nonnull Contexts contexts);
+    void invalidatePermissions(@NonNull Contexts contexts);

    /**
     * Invalidates any cached {@link MetaData} instances mapped to the given
@@ -390,7 +387,7 @@ public interface CachedData {
     * @param contexts the contexts to invalidate for
     * @since 4.0
     */
-    void invalidateMeta(@Nonnull MetaContexts contexts);
+    void invalidateMeta(@NonNull MetaContexts contexts);

    /**
     * Invalidates any cached {@link MetaData} instances mapped to the given
@@ -399,7 +396,7 @@ public interface CachedData {
     * @param contexts the contexts to invalidate for
     * @since 4.0
     */
-    void invalidateMeta(@Nonnull Contexts contexts);
+    void invalidateMeta(@NonNull Contexts contexts);

    /**
     * Invalidates all cached {@link PermissionData} instances.
@@ -27,7 +27,7 @@ package me.lucko.luckperms.api.caching;

 import me.lucko.luckperms.api.Contexts;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * Holds cached lookup data in a specific set of contexts.
@@ -41,7 +41,6 @@ public interface CachedDataContainer {
     *
     * @return the contexts this container is caching
     */
-    @Nonnull
-    Contexts getContexts();
+    @NonNull Contexts getContexts();

 }
@@ -28,10 +28,9 @@ package me.lucko.luckperms.api.caching;
 import me.lucko.luckperms.api.Contexts;
 import me.lucko.luckperms.api.metastacking.MetaStackDefinition;

-import java.util.Objects;
+import org.checkerframework.checker.nullness.qual.NonNull;

-import javax.annotation.Nonnull;
-import javax.annotation.concurrent.Immutable;
+import java.util.Objects;

 /**
 * Encapsulates the options and settings for a meta lookup.
@@ -41,18 +40,17 @@ import javax.annotation.concurrent.Immutable;
 *
 * @since 3.2
 */
-@Immutable
 public final class MetaContexts {

    /**
     * Creates a new meta contexts instance
     *
-     * @param contexts the standard contexts for the query
+     * @param contexts              the standard contexts for the query
     * @param prefixStackDefinition the prefix stack definition to be used
     * @param suffixStackDefinition the suffix stack definition to be used
     * @return the new instance
     */
-    public static MetaContexts of(@Nonnull Contexts contexts, @Nonnull MetaStackDefinition prefixStackDefinition, @Nonnull MetaStackDefinition suffixStackDefinition) {
+    public static MetaContexts of(@NonNull Contexts contexts, @NonNull MetaStackDefinition prefixStackDefinition, @NonNull MetaStackDefinition suffixStackDefinition) {
        return new MetaContexts(contexts, prefixStackDefinition, suffixStackDefinition);
    }

@@ -66,35 +64,31 @@ public final class MetaContexts {
    /**
     * Creates a new meta contexts instance
     *
-     * @param contexts the standard contexts for the query
+     * @param contexts              the standard contexts for the query
     * @param prefixStackDefinition the prefix stack definition to be used
     * @param suffixStackDefinition the suffix stack definition to be used
     */
-    public MetaContexts(@Nonnull Contexts contexts, @Nonnull MetaStackDefinition prefixStackDefinition, @Nonnull MetaStackDefinition suffixStackDefinition) {
+    public MetaContexts(@NonNull Contexts contexts, @NonNull MetaStackDefinition prefixStackDefinition, @NonNull MetaStackDefinition suffixStackDefinition) {
        this.contexts = Objects.requireNonNull(contexts, "contexts");
        this.prefixStackDefinition = Objects.requireNonNull(prefixStackDefinition, "prefixStackDefinition");
        this.suffixStackDefinition = Objects.requireNonNull(suffixStackDefinition, "suffixStackDefinition");
        this.hashCode = calculateHashCode();
    }

-    @Nonnull
-    public Contexts getContexts() {
+    public @NonNull Contexts getContexts() {
        return this.contexts;
    }

-    @Nonnull
-    public MetaStackDefinition getPrefixStackDefinition() {
+    public @NonNull MetaStackDefinition getPrefixStackDefinition() {
        return this.prefixStackDefinition;
    }

-    @Nonnull
-    public MetaStackDefinition getSuffixStackDefinition() {
+    public @NonNull MetaStackDefinition getSuffixStackDefinition() {
        return this.suffixStackDefinition;
    }

    @Override
-    @Nonnull
-    public String toString() {
+    public @NonNull String toString() {
        return "MetaContexts(" +
                "contexts=" + this.getContexts() + ", " +
                "prefixStackDefinition=" + this.getPrefixStackDefinition() + ", " +
@@ -29,12 +29,12 @@ import com.google.common.collect.ListMultimap;

 import me.lucko.luckperms.api.metastacking.MetaStackDefinition;

+import org.checkerframework.checker.nullness.qual.NonNull;
+import org.checkerframework.checker.nullness.qual.Nullable;
+
 import java.util.Map;
 import java.util.SortedMap;

-import javax.annotation.Nonnull;
-import javax.annotation.Nullable;
-
 /**
 * Holds cached meta lookup data for a specific set of contexts.
 *
@@ -47,8 +47,7 @@ public interface MetaData extends CachedDataContainer {
     *
     * @return the contexts this container is caching
     */
-    @Nonnull
-    MetaContexts getMetaContexts();
+    @NonNull MetaContexts getMetaContexts();

    /**
     * Gets an immutable copy of the meta this user has.
@@ -62,8 +61,7 @@ public interface MetaData extends CachedDataContainer {
     * @return an immutable multimap of meta
     * @since 3.3
     */
-    @Nonnull
-    ListMultimap<String, String> getMetaMultimap();
+    @NonNull ListMultimap<String, String> getMetaMultimap();

    /**
     * Gets an immutable copy of the meta this user has.
@@ -73,8 +71,7 @@ public interface MetaData extends CachedDataContainer {
     *
     * @return an immutable map of meta
     */
-    @Nonnull
-    Map<String, String> getMeta();
+    @NonNull Map<String, String> getMeta();

    /**
     * Gets an immutable sorted map of all of the prefixes the user has, whereby the first value is the highest priority
@@ -82,8 +79,7 @@ public interface MetaData extends CachedDataContainer {
     *
     * @return a sorted map of prefixes
     */
-    @Nonnull
-    SortedMap<Integer, String> getPrefixes();
+    @NonNull SortedMap<Integer, String> getPrefixes();

    /**
     * Gets an immutable sorted map of all of the suffixes the user has, whereby the first value is the highest priority
@@ -91,24 +87,21 @@ public interface MetaData extends CachedDataContainer {
     *
     * @return a sorted map of suffixes
     */
-    @Nonnull
-    SortedMap<Integer, String> getSuffixes();
+    @NonNull SortedMap<Integer, String> getSuffixes();

    /**
     * Gets the user's highest priority prefix, or null if the user has no prefixes
     *
     * @return a prefix string, or null
     */
-    @Nullable
-    String getPrefix();
+    @Nullable String getPrefix();

    /**
     * Gets the user's highest priority suffix, or null if the user has no suffixes
     *
     * @return a suffix string, or null
     */
-    @Nullable
-    String getSuffix();
+    @Nullable String getSuffix();

    /**
     * Gets the definition used for the prefix stack
@@ -116,8 +109,7 @@ public interface MetaData extends CachedDataContainer {
     * @return the definition used for the prefix stack
     * @since 3.2
     */
-    @Nonnull
-    MetaStackDefinition getPrefixStackDefinition();
+    @NonNull MetaStackDefinition getPrefixStackDefinition();

    /**
     * Gets the definition used for the suffix stack
@@ -125,7 +117,6 @@ public interface MetaData extends CachedDataContainer {
     * @return the definition used for the suffix stack
     * @since 3.2
     */
-    @Nonnull
-    MetaStackDefinition getSuffixStackDefinition();
+    @NonNull MetaStackDefinition getSuffixStackDefinition();

 }
@@ -27,9 +27,9 @@ package me.lucko.luckperms.api.caching;

 import me.lucko.luckperms.api.Tristate;

-import java.util.Map;
+import org.checkerframework.checker.nullness.qual.NonNull;

-import javax.annotation.Nonnull;
+import java.util.Map;

 /**
 * Holds cached permission lookup data for a specific set of contexts.
@@ -45,8 +45,7 @@ public interface PermissionData extends CachedDataContainer {
     * @return a tristate result
     * @throws NullPointerException if permission is null
     */
-    @Nonnull
-    Tristate getPermissionValue(@Nonnull String permission);
+    @NonNull Tristate getPermissionValue(@NonNull String permission);

    /**
     * Invalidates the underlying permission calculator cache.
@@ -60,7 +59,6 @@ public interface PermissionData extends CachedDataContainer {
     *
     * @return an immutable set of permissions
     */
-    @Nonnull
-    Map<String, Boolean> getImmutableBacking();
+    @NonNull Map<String, Boolean> getImmutableBacking();

 }
@@ -29,12 +29,12 @@ import com.google.common.collect.ImmutableSet;
 import com.google.common.collect.Multimap;
 import com.google.common.collect.SetMultimap;

+import org.checkerframework.checker.nullness.qual.NonNull;
+
 import java.util.Collection;
 import java.util.Objects;
 import java.util.Set;

-import javax.annotation.Nonnull;
-
 abstract class AbstractContextSet implements ContextSet {

    protected abstract SetMultimap<String, String> backing();
@@ -42,19 +42,18 @@ abstract class AbstractContextSet implements ContextSet {
    protected abstract void copyTo(SetMultimap<String, String> other);

    @Override
-    public boolean containsKey(@Nonnull String key) {
+    public boolean containsKey(@NonNull String key) {
        return backing().containsKey(sanitizeKey(key));
    }

-    @Nonnull
    @Override
-    public Set<String> getValues(@Nonnull String key) {
+    public @NonNull Set<String> getValues(@NonNull String key) {
        Collection<String> values = backing().asMap().get(sanitizeKey(key));
        return values != null ? ImmutableSet.copyOf(values) : ImmutableSet.of();
    }

    @Override
-    public boolean has(@Nonnull String key, @Nonnull String value) {
+    public boolean has(@NonNull String key, @NonNull String value) {
        return backing().containsEntry(sanitizeKey(key), sanitizeValue(value));
    }

@@ -25,7 +25,7 @@

 package me.lucko.luckperms.api.context;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * Calculates whether contexts are applicable to a {@link T subject}.
@@ -44,7 +44,6 @@ public interface ContextCalculator<T> {
     * @return the map
     * @since 2.13
     */
-    @Nonnull
-    MutableContextSet giveApplicableContext(@Nonnull T subject, @Nonnull MutableContextSet accumulator);
+    @NonNull MutableContextSet giveApplicableContext(@NonNull T subject, @NonNull MutableContextSet accumulator);

 }
@@ -28,9 +28,9 @@ package me.lucko.luckperms.api.context;
 import me.lucko.luckperms.api.Contexts;
 import me.lucko.luckperms.api.User;

-import java.util.Optional;
+import org.checkerframework.checker.nullness.qual.NonNull;

-import javax.annotation.Nonnull;
+import java.util.Optional;

 /**
 * Manages {@link ContextCalculator}s, and calculates applicable contexts for a
@@ -44,9 +44,9 @@ import javax.annotation.Nonnull;
 *
 * <p></p>
 * <ul>
- *     <li>{@code org.bukkit.entity.Player}</li>
- *     <li>{@code net.md_5.bungee.api.connection.ProxiedPlayer}</li>
- *     <li>{@code org.spongepowered.api.service.permission.Subject}</li>
+ * <li>{@code org.bukkit.entity.Player}</li>
+ * <li>{@code net.md_5.bungee.api.connection.ProxiedPlayer}</li>
+ * <li>{@code org.spongepowered.api.service.permission.Subject}</li>
 * </ul>
 *
 * @since 4.0
@@ -59,8 +59,7 @@ public interface ContextManager {
     * @param subject the subject
     * @return the applicable context for the subject
     */
-    @Nonnull
-    ImmutableContextSet getApplicableContext(@Nonnull Object subject);
+    @NonNull ImmutableContextSet getApplicableContext(@NonNull Object subject);

    /**
     * Queries the ContextManager for current context values for the subject.
@@ -68,8 +67,7 @@ public interface ContextManager {
     * @param subject the subject
     * @return the applicable context for the subject
     */
-    @Nonnull
-    Contexts getApplicableContexts(@Nonnull Object subject);
+    @NonNull Contexts getApplicableContexts(@NonNull Object subject);

    /**
     * Queries the ContextManager for current context values for the given User.
@@ -83,8 +81,7 @@ public interface ContextManager {
     * @param user the user
     * @return the applicable context for the subject
     */
-    @Nonnull
-    Optional<ImmutableContextSet> lookupApplicableContext(@Nonnull User user);
+    @NonNull Optional<ImmutableContextSet> lookupApplicableContext(@NonNull User user);

    /**
     * Queries the ContextManager for current context values for the given User.
@@ -98,8 +95,7 @@ public interface ContextManager {
     * @param user the user
     * @return the applicable context for the subject
     */
-    @Nonnull
-    Optional<Contexts> lookupApplicableContexts(@Nonnull User user);
+    @NonNull Optional<Contexts> lookupApplicableContexts(@NonNull User user);

    /**
     * Gets the contexts from the static calculators in this manager.
@@ -109,8 +105,7 @@ public interface ContextManager {
     *
     * @return the current active static contexts
     */
-    @Nonnull
-    ImmutableContextSet getStaticContext();
+    @NonNull ImmutableContextSet getStaticContext();

    /**
     * Gets the contexts from the static calculators in this manager.
@@ -120,8 +115,7 @@ public interface ContextManager {
     *
     * @return the current active static contexts
     */
-    @Nonnull
-    Contexts getStaticContexts();
+    @NonNull Contexts getStaticContexts();

    /**
     * Forms a {@link Contexts} instance from an {@link ImmutableContextSet}.
@@ -129,12 +123,11 @@ public interface ContextManager {
     * <p>This method relies on the plugins configuration to form the
     * {@link Contexts} instance.</p>
     *
-     * @param subject the reference subject
+     * @param subject    the reference subject
     * @param contextSet the context set
     * @return a contexts instance
     */
-    @Nonnull
-    Contexts formContexts(@Nonnull Object subject, @Nonnull ImmutableContextSet contextSet);
+    @NonNull Contexts formContexts(@NonNull Object subject, @NonNull ImmutableContextSet contextSet);

    /**
     * Forms a {@link Contexts} instance from an {@link ImmutableContextSet}.
@@ -145,15 +138,14 @@ public interface ContextManager {
     * @param contextSet the context set
     * @return a contexts instance
     */
-    @Nonnull
-    Contexts formContexts(@Nonnull ImmutableContextSet contextSet);
+    @NonNull Contexts formContexts(@NonNull ImmutableContextSet contextSet);

    /**
     * Registers a context calculator with the manager.
     *
     * @param calculator the calculator
     */
-    void registerCalculator(@Nonnull ContextCalculator<?> calculator);
+    void registerCalculator(@NonNull ContextCalculator<?> calculator);

    /**
     * Registers a static context calculator with the manager.
@@ -162,13 +154,13 @@ public interface ContextManager {
     *
     * @param calculator the calculator
     */
-    void registerStaticCalculator(@Nonnull StaticContextCalculator calculator);
+    void registerStaticCalculator(@NonNull StaticContextCalculator calculator);

    /**
     * Invalidates the lookup cache for a given subject
     *
     * @param subject the subject
     */
-    void invalidateCache(@Nonnull Object subject);
+    void invalidateCache(@NonNull Object subject);

 }
@@ -27,14 +27,14 @@ package me.lucko.luckperms.api.context;

 import com.google.common.collect.Multimap;

+import org.checkerframework.checker.nullness.qual.NonNull;
+
 import java.util.Iterator;
 import java.util.Map;
 import java.util.Objects;
 import java.util.Optional;
 import java.util.Set;

-import javax.annotation.Nonnull;
-
 /**
 * A set of contexts.
 *
@@ -71,24 +71,22 @@ public interface ContextSet extends Iterable<Map.Entry<String, String>> {
     * @return a new ImmutableContextSet containing one context pair
     * @throws NullPointerException if key or value is null
     */
-    @Nonnull
-    static ImmutableContextSet singleton(@Nonnull String key, @Nonnull String value) {
+    static @NonNull ImmutableContextSet singleton(@NonNull String key, @NonNull String value) {
        return ImmutableContextSet.singleton(key, value);
    }

    /**
     * Creates an {@link ImmutableContextSet} from two context pairs.
     *
-     * @param key1 the first key
+     * @param key1   the first key
     * @param value1 the first value
-     * @param key2 the second key
+     * @param key2   the second key
     * @param value2 the second value
     * @return a new ImmutableContextSet containing the two pairs
     * @throws NullPointerException if any of the keys or values are null
     * @since 3.1
     */
-    @Nonnull
-    static ImmutableContextSet of(@Nonnull String key1, @Nonnull String value1, @Nonnull String key2, @Nonnull String value2) {
+    static @NonNull ImmutableContextSet of(@NonNull String key1, @NonNull String value1, @NonNull String key2, @NonNull String value2) {
        return ImmutableContextSet.of(key1, value1, key2, value2);
    }

@@ -99,8 +97,7 @@ public interface ContextSet extends Iterable<Map.Entry<String, String>> {
     * @return a new ImmutableContextSet representing the pairs in the iterable
     * @throws NullPointerException if the iterable is null
     */
-    @Nonnull
-    static ImmutableContextSet fromEntries(@Nonnull Iterable<? extends Map.Entry<String, String>> iterable) {
+    static @NonNull ImmutableContextSet fromEntries(@NonNull Iterable<? extends Map.Entry<String, String>> iterable) {
        return ImmutableContextSet.fromEntries(iterable);
    }

@@ -111,8 +108,7 @@ public interface ContextSet extends Iterable<Map.Entry<String, String>> {
     * @return a new ImmutableContextSet representing the pairs from the map
     * @throws NullPointerException if the map is null
     */
-    @Nonnull
-    static ImmutableContextSet fromMap(@Nonnull Map<String, String> map) {
+    static @NonNull ImmutableContextSet fromMap(@NonNull Map<String, String> map) {
        return ImmutableContextSet.fromMap(map);
    }

@@ -124,8 +120,7 @@ public interface ContextSet extends Iterable<Map.Entry<String, String>> {
     * @throws NullPointerException if the multimap is null
     * @since 2.16
     */
-    @Nonnull
-    static ImmutableContextSet fromMultimap(@Nonnull Multimap<String, String> multimap) {
+    static @NonNull ImmutableContextSet fromMultimap(@NonNull Multimap<String, String> multimap) {
        return ImmutableContextSet.fromMultimap(multimap);
    }

@@ -138,8 +133,7 @@ public interface ContextSet extends Iterable<Map.Entry<String, String>> {
     * @return a new ImmutableContextSet with the same content and the one provided
     * @throws NullPointerException if contextSet is null
     */
-    @Nonnull
-    static ImmutableContextSet fromSet(@Nonnull ContextSet contextSet) {
+    static @NonNull ImmutableContextSet fromSet(@NonNull ContextSet contextSet) {
        return ImmutableContextSet.fromSet(contextSet);
    }

@@ -148,8 +142,7 @@ public interface ContextSet extends Iterable<Map.Entry<String, String>> {
     *
     * @return an empty ImmutableContextSet
     */
-    @Nonnull
-    static ImmutableContextSet empty() {
+    static @NonNull ImmutableContextSet empty() {
        return ImmutableContextSet.empty();
    }

@@ -170,8 +163,7 @@ public interface ContextSet extends Iterable<Map.Entry<String, String>> {
     *
     * @return an immutable representation of this set
     */
-    @Nonnull
-    ImmutableContextSet makeImmutable();
+    @NonNull ImmutableContextSet makeImmutable();

    /**
     * Creates a mutable copy of this {@link ContextSet}.
@@ -182,8 +174,7 @@ public interface ContextSet extends Iterable<Map.Entry<String, String>> {
     * @return a mutable ContextSet
     * @since 2.16
     */
-    @Nonnull
-    MutableContextSet mutableCopy();
+    @NonNull MutableContextSet mutableCopy();

    /**
     * Returns a {@link Set} of {@link Map.Entry}s representing the current
@@ -194,8 +185,7 @@ public interface ContextSet extends Iterable<Map.Entry<String, String>> {
     *
     * @return an immutable set
     */
-    @Nonnull
-    Set<Map.Entry<String, String>> toSet();
+    @NonNull Set<Map.Entry<String, String>> toSet();

    /**
     * Returns a {@link Map} <b>loosely</b> representing the current state of
@@ -214,9 +204,8 @@ public interface ContextSet extends Iterable<Map.Entry<String, String>> {
     * @return an immutable map
     * @deprecated because the resultant map may not contain all data in the ContextSet
     */
-    @Nonnull
    @Deprecated
-    Map<String, String> toMap();
+    @NonNull Map<String, String> toMap();

    /**
     * Returns a {@link Multimap} representing the current state of this
@@ -228,8 +217,7 @@ public interface ContextSet extends Iterable<Map.Entry<String, String>> {
     * @return a multimap
     * @since 2.16
     */
-    @Nonnull
-    Multimap<String, String> toMultimap();
+    @NonNull Multimap<String, String> toMultimap();

    /**
     * Returns an {@link Iterator} over each of the context pairs in this set.
@@ -241,9 +229,8 @@ public interface ContextSet extends Iterable<Map.Entry<String, String>> {
     *
     * @return an iterator
     */
-    @Nonnull
    @Override
-    Iterator<Map.Entry<String, String>> iterator();
+    @NonNull Iterator<Map.Entry<String, String>> iterator();

    /**
     * Returns if the {@link ContextSet} contains at least one value for the
@@ -253,7 +240,7 @@ public interface ContextSet extends Iterable<Map.Entry<String, String>> {
     * @return true if the set contains a value for the key
     * @throws NullPointerException if the key is null
     */
-    boolean containsKey(@Nonnull String key);
+    boolean containsKey(@NonNull String key);

    /**
     * Returns a {@link Set} of the values mapped to the given key.
@@ -265,8 +252,7 @@ public interface ContextSet extends Iterable<Map.Entry<String, String>> {
     * @return a set of values
     * @throws NullPointerException if the key is null
     */
-    @Nonnull
-    Set<String> getValues(@Nonnull String key);
+    @NonNull Set<String> getValues(@NonNull String key);

    /**
     * Returns any value from this {@link ContextSet} matching the key, if present.
@@ -278,8 +264,7 @@ public interface ContextSet extends Iterable<Map.Entry<String, String>> {
     * @return an optional containing any match
     * @since 3.1
     */
-    @Nonnull
-    default Optional<String> getAnyValue(@Nonnull String key) {
+    default @NonNull Optional<String> getAnyValue(@NonNull String key) {
        return getValues(key).stream().findAny();
    }

@@ -291,7 +276,7 @@ public interface ContextSet extends Iterable<Map.Entry<String, String>> {
     * @return true if the set contains the context pair
     * @throws NullPointerException if the key or value is null
     */
-    boolean has(@Nonnull String key, @Nonnull String value);
+    boolean has(@NonNull String key, @NonNull String value);

    /**
     * Returns if the {@link ContextSet} contains a given context pairing.
@@ -300,7 +285,7 @@ public interface ContextSet extends Iterable<Map.Entry<String, String>> {
     * @return true if the set contains the context pair
     * @throws NullPointerException if the key or value is null
     */
-    default boolean has(@Nonnull Map.Entry<String, String> entry) {
+    default boolean has(Map.@NonNull Entry<String, String> entry) {
        Objects.requireNonNull(entry, "entry");
        return has(entry.getKey(), entry.getValue());
    }
@@ -317,7 +302,7 @@ public interface ContextSet extends Iterable<Map.Entry<String, String>> {
     * @return true if all entries in this set are also in the other set
     * @since 3.1
     */
-    default boolean isSatisfiedBy(@Nonnull ContextSet other) {
+    default boolean isSatisfiedBy(@NonNull ContextSet other) {
        if (this == other) {
            return true;
        }
@@ -30,21 +30,19 @@ import com.google.common.collect.ImmutableSetMultimap;
 import com.google.common.collect.Multimap;
 import com.google.common.collect.SetMultimap;

+import org.checkerframework.checker.nullness.qual.NonNull;
+
 import java.util.Iterator;
 import java.util.Map;
 import java.util.Objects;
 import java.util.Set;
 import java.util.Spliterator;

-import javax.annotation.Nonnull;
-import javax.annotation.concurrent.Immutable;
-
 /**
 * An immutable implementation of {@link ContextSet}.
 *
 * @since 2.16
 */
-@Immutable
 public final class ImmutableContextSet extends AbstractContextSet implements ContextSet {
    private static final ImmutableContextSet EMPTY = new ImmutableContextSet(ImmutableSetMultimap.of());

@@ -54,8 +52,7 @@ public final class ImmutableContextSet extends AbstractContextSet implements Con
     * @return a new ImmutableContextSet builder
     * @since 4.1
     */
-    @Nonnull
-    public static Builder builder() {
+    public static @NonNull Builder builder() {
        return new Builder();
    }

@@ -67,8 +64,7 @@ public final class ImmutableContextSet extends AbstractContextSet implements Con
     * @return a new ImmutableContextSet containing one context pair
     * @throws NullPointerException if key or value is null
     */
-    @Nonnull
-    public static ImmutableContextSet singleton(@Nonnull String key, @Nonnull String value) {
+    public static @NonNull ImmutableContextSet singleton(@NonNull String key, @NonNull String value) {
        return new ImmutableContextSet(ImmutableSetMultimap.of(sanitizeKey(key), sanitizeValue(value)));
    }

@@ -83,8 +79,7 @@ public final class ImmutableContextSet extends AbstractContextSet implements Con
     * @throws NullPointerException if any of the keys or values are null
     * @since 3.1
     */
-    @Nonnull
-    public static ImmutableContextSet of(@Nonnull String key1, @Nonnull String value1, @Nonnull String key2, @Nonnull String value2) {
+    public static @NonNull ImmutableContextSet of(@NonNull String key1, @NonNull String value1, @NonNull String key2, @NonNull String value2) {
        return new ImmutableContextSet(ImmutableSetMultimap.of(
                sanitizeKey(key1),
                sanitizeValue(value1),
@@ -100,8 +95,7 @@ public final class ImmutableContextSet extends AbstractContextSet implements Con
     * @return a new ImmutableContextSet representing the pairs in the iterable
     * @throws NullPointerException if the iterable is null
     */
-    @Nonnull
-    public static ImmutableContextSet fromEntries(@Nonnull Iterable<? extends Map.Entry<String, String>> iterable) {
+    public static @NonNull ImmutableContextSet fromEntries(@NonNull Iterable<? extends Map.Entry<String, String>> iterable) {
        Objects.requireNonNull(iterable, "iterable");
        ImmutableContextSet.Builder builder = builder();
        for (Map.Entry<String, String> entry : iterable) {
@@ -117,8 +111,7 @@ public final class ImmutableContextSet extends AbstractContextSet implements Con
     * @return a new ImmutableContextSet representing the pairs from the map
     * @throws NullPointerException if the map is null
     */
-    @Nonnull
-    public static ImmutableContextSet fromMap(@Nonnull Map<String, String> map) {
+    public static @NonNull ImmutableContextSet fromMap(@NonNull Map<String, String> map) {
        return fromEntries(Objects.requireNonNull(map, "map").entrySet());
    }

@@ -130,8 +123,7 @@ public final class ImmutableContextSet extends AbstractContextSet implements Con
     * @throws NullPointerException if the multimap is null
     * @since 2.16
     */
-    @Nonnull
-    public static ImmutableContextSet fromMultimap(@Nonnull Multimap<String, String> multimap) {
+    public static @NonNull ImmutableContextSet fromMultimap(@NonNull Multimap<String, String> multimap) {
        return fromEntries(Objects.requireNonNull(multimap, "multimap").entries());
    }

@@ -144,8 +136,7 @@ public final class ImmutableContextSet extends AbstractContextSet implements Con
     * @return a new ImmutableContextSet with the same content and the one provided
     * @throws NullPointerException if contextSet is null
     */
-    @Nonnull
-    public static ImmutableContextSet fromSet(@Nonnull ContextSet contextSet) {
+    public static @NonNull ImmutableContextSet fromSet(@NonNull ContextSet contextSet) {
        return Objects.requireNonNull(contextSet, "contextSet").makeImmutable();
    }

@@ -154,8 +145,7 @@ public final class ImmutableContextSet extends AbstractContextSet implements Con
     *
     * @return an empty ImmutableContextSet
     */
-    @Nonnull
-    public static ImmutableContextSet empty() {
+    public static @NonNull ImmutableContextSet empty() {
        return EMPTY;
    }

@@ -182,29 +172,25 @@ public final class ImmutableContextSet extends AbstractContextSet implements Con
        return true;
    }

-    @Nonnull
-    @Override
-    @Deprecated // This set is already immutable!
-    public ImmutableContextSet makeImmutable() {
+    @Deprecated
+    @Override // This set is already immutable!
+    public @NonNull ImmutableContextSet makeImmutable() {
        return this;
    }

-    @Nonnull
    @Override
-    public MutableContextSet mutableCopy() {
+    public @NonNull MutableContextSet mutableCopy() {
        return MutableContextSet.fromSet(this);
    }

-    @Nonnull
    @Override
-    public Set<Map.Entry<String, String>> toSet() {
+    public @NonNull Set<Map.Entry<String, String>> toSet() {
        return this.map.entries();
    }

-    @Nonnull
-    @Override
    @Deprecated
-    public Map<String, String> toMap() {
+    @Override
+    public @NonNull Map<String, String> toMap() {
        ImmutableMap.Builder<String, String> m = ImmutableMap.builder();
        for (Map.Entry<String, String> e : this.map.entries()) {
            m.put(e.getKey(), e.getValue());
@@ -212,15 +198,13 @@ public final class ImmutableContextSet extends AbstractContextSet implements Con
        return m.build();
    }

-    @Nonnull
    @Override
-    public Multimap<String, String> toMultimap() {
+    public @NonNull Multimap<String, String> toMultimap() {
        return this.map;
    }

-    @Nonnull
    @Override
-    public Iterator<Map.Entry<String, String>> iterator() {
+    public @NonNull Iterator<Map.Entry<String, String>> iterator() {
        return this.map.entries().iterator();
    }

@@ -271,8 +255,7 @@ public final class ImmutableContextSet extends AbstractContextSet implements Con
         * @throws NullPointerException if the key or value is null
         * @see MutableContextSet#add(String, String)
         */
-        @Nonnull
-        public Builder add(@Nonnull String key, @Nonnull String value) {
+        public @NonNull Builder add(@NonNull String key, @NonNull String value) {
            put(sanitizeKey(key), sanitizeValue(value));
            return this;
        }
@@ -285,8 +268,7 @@ public final class ImmutableContextSet extends AbstractContextSet implements Con
         * @throws NullPointerException if the entry is null
         * @see MutableContextSet#add(Map.Entry)
         */
-        @Nonnull
-        public Builder add(@Nonnull Map.Entry<String, String> entry) {
+        public @NonNull Builder add(Map.@NonNull Entry<String, String> entry) {
            Objects.requireNonNull(entry, "entry");
            add(entry.getKey(), entry.getValue());
            return this;
@@ -300,8 +282,7 @@ public final class ImmutableContextSet extends AbstractContextSet implements Con
         * @throws NullPointerException if iterable is null
         * @see MutableContextSet#addAll(Iterable)
         */
-        @Nonnull
-        public Builder addAll(@Nonnull Iterable<? extends Map.Entry<String, String>> iterable) {
+        public @NonNull Builder addAll(@NonNull Iterable<? extends Map.Entry<String, String>> iterable) {
            for (Map.Entry<String, String> e : Objects.requireNonNull(iterable, "iterable")) {
                add(e);
            }
@@ -316,8 +297,7 @@ public final class ImmutableContextSet extends AbstractContextSet implements Con
         * @throws NullPointerException if the map is null
         * @see MutableContextSet#addAll(Map)
         */
-        @Nonnull
-        public Builder addAll(@Nonnull Map<String, String> map) {
+        public @NonNull Builder addAll(@NonNull Map<String, String> map) {
            addAll(Objects.requireNonNull(map, "map").entrySet());
            return this;
        }
@@ -331,8 +311,7 @@ public final class ImmutableContextSet extends AbstractContextSet implements Con
         * @since 3.4
         * @see MutableContextSet#addAll(Multimap)
         */
-        @Nonnull
-        public Builder addAll(@Nonnull Multimap<String, String> multimap) {
+        public @NonNull Builder addAll(@NonNull Multimap<String, String> multimap) {
            addAll(Objects.requireNonNull(multimap, "multimap").entries());
            return this;
        }
@@ -345,8 +324,7 @@ public final class ImmutableContextSet extends AbstractContextSet implements Con
         * @throws NullPointerException if the contextSet is null
         * @see MutableContextSet#addAll(ContextSet)
         */
-        @Nonnull
-        public Builder addAll(@Nonnull ContextSet contextSet) {
+        public @NonNull Builder addAll(@NonNull ContextSet contextSet) {
            Objects.requireNonNull(contextSet, "contextSet");
            if (contextSet instanceof AbstractContextSet) {
                AbstractContextSet other = ((AbstractContextSet) contextSet);
@@ -365,8 +343,7 @@ public final class ImmutableContextSet extends AbstractContextSet implements Con
         *
         * @return an {@link ImmutableContextSet} from the builder
         */
-        @Nonnull
-        public ImmutableContextSet build() {
+        public @NonNull ImmutableContextSet build() {
            if (this.builder == null) {
                return empty();
            } else {
@@ -33,14 +33,14 @@ import com.google.common.collect.Multimap;
 import com.google.common.collect.Multimaps;
 import com.google.common.collect.SetMultimap;

+import org.checkerframework.checker.nullness.qual.NonNull;
+
 import java.util.Iterator;
 import java.util.Map;
 import java.util.Objects;
 import java.util.Set;
 import java.util.Spliterator;

-import javax.annotation.Nonnull;
-
 /**
 * A mutable implementation of {@link ContextSet}.
 *
@@ -56,8 +56,7 @@ public final class MutableContextSet extends AbstractContextSet implements Conte
     * @return a new MutableContextSet containing one context pair
     * @throws NullPointerException if key or value is null
     */
-    @Nonnull
-    public static MutableContextSet singleton(@Nonnull String key, @Nonnull String value) {
+    public static @NonNull MutableContextSet singleton(@NonNull String key, @NonNull String value) {
        Objects.requireNonNull(key, "key");
        Objects.requireNonNull(value, "value");
        MutableContextSet set = MutableContextSet.create();
@@ -76,8 +75,7 @@ public final class MutableContextSet extends AbstractContextSet implements Conte
     * @throws NullPointerException if any of the keys or values are null
     * @since 3.1
     */
-    @Nonnull
-    public static MutableContextSet of(@Nonnull String key1, @Nonnull String value1, @Nonnull String key2, @Nonnull String value2) {
+    public static @NonNull MutableContextSet of(@NonNull String key1, @NonNull String value1, @NonNull String key2, @NonNull String value2) {
        Objects.requireNonNull(key1, "key1");
        Objects.requireNonNull(value1, "value1");
        Objects.requireNonNull(key2, "key2");
@@ -95,8 +93,7 @@ public final class MutableContextSet extends AbstractContextSet implements Conte
     * @return a new MutableContextSet representing the pairs in the iterable
     * @throws NullPointerException if the iterable is null
     */
-    @Nonnull
-    public static MutableContextSet fromEntries(@Nonnull Iterable<? extends Map.Entry<String, String>> iterable) {
+    public static @NonNull MutableContextSet fromEntries(@NonNull Iterable<? extends Map.Entry<String, String>> iterable) {
        Objects.requireNonNull(iterable, "iterable");
        MutableContextSet set = create();
        set.addAll(iterable);
@@ -110,8 +107,7 @@ public final class MutableContextSet extends AbstractContextSet implements Conte
     * @return a new MutableContextSet representing the pairs from the map
     * @throws NullPointerException if the map is null
     */
-    @Nonnull
-    public static MutableContextSet fromMap(@Nonnull Map<String, String> map) {
+    public static @NonNull MutableContextSet fromMap(@NonNull Map<String, String> map) {
        Objects.requireNonNull(map, "map");
        MutableContextSet set = create();
        set.addAll(map);
@@ -126,8 +122,7 @@ public final class MutableContextSet extends AbstractContextSet implements Conte
     * @throws NullPointerException if the multimap is null
     * @since 2.16
     */
-    @Nonnull
-    public static MutableContextSet fromMultimap(@Nonnull Multimap<String, String> multimap) {
+    public static @NonNull MutableContextSet fromMultimap(@NonNull Multimap<String, String> multimap) {
        Objects.requireNonNull(multimap, "multimap");
        MutableContextSet set = create();
        set.addAll(multimap);
@@ -143,8 +138,7 @@ public final class MutableContextSet extends AbstractContextSet implements Conte
     * @return a new MutableContextSet with the same content and the one provided
     * @throws NullPointerException if contextSet is null
     */
-    @Nonnull
-    public static MutableContextSet fromSet(@Nonnull ContextSet contextSet) {
+    public static @NonNull MutableContextSet fromSet(@NonNull ContextSet contextSet) {
        Objects.requireNonNull(contextSet, "contextSet");

        if (contextSet instanceof ImmutableContextSet) {
@@ -164,8 +158,7 @@ public final class MutableContextSet extends AbstractContextSet implements Conte
     *
     * @return a new MutableContextSet
     */
-    @Nonnull
-    public static MutableContextSet create() {
+    public static @NonNull MutableContextSet create() {
        return new MutableContextSet();
    }

@@ -196,9 +189,8 @@ public final class MutableContextSet extends AbstractContextSet implements Conte
        return false;
    }

-    @Nonnull
    @Override
-    public ImmutableContextSet makeImmutable() {
+    public @NonNull ImmutableContextSet makeImmutable() {
        // if the map is empty, don't create a new instance
        if (this.map.isEmpty()) {
            return ImmutableContextSet.empty();
@@ -208,27 +200,24 @@ public final class MutableContextSet extends AbstractContextSet implements Conte
        }
    }

-    @Nonnull
    @Override
-    public MutableContextSet mutableCopy() {
+    public @NonNull MutableContextSet mutableCopy() {
        synchronized (this.map) {
            return new MutableContextSet(this.map);
        }
    }

-    @Nonnull
    @Override
-    public Set<Map.Entry<String, String>> toSet() {
+    public @NonNull Set<Map.Entry<String, String>> toSet() {
        synchronized (this.map) {
            // map.entries() returns immutable Map.Entry instances, so we can just call copyOf
            return ImmutableSet.copyOf(this.map.entries());
        }
    }

-    @Nonnull
-    @Override
    @Deprecated
-    public Map<String, String> toMap() {
+    @Override
+    public @NonNull Map<String, String> toMap() {
        ImmutableMap.Builder<String, String> m = ImmutableMap.builder();
        synchronized (this.map) {
            for (Map.Entry<String, String> e : this.map.entries()) {
@@ -238,17 +227,15 @@ public final class MutableContextSet extends AbstractContextSet implements Conte
        return m.build();
    }

-    @Nonnull
    @Override
-    public Multimap<String, String> toMultimap() {
+    public @NonNull Multimap<String, String> toMultimap() {
        synchronized (this.map) {
            return ImmutableSetMultimap.copyOf(this.map);
        }
    }

-    @Nonnull
    @Override
-    public Iterator<Map.Entry<String, String>> iterator() {
+    public @NonNull Iterator<Map.Entry<String, String>> iterator() {
        return toSet().iterator();
    }

@@ -264,7 +251,7 @@ public final class MutableContextSet extends AbstractContextSet implements Conte
     * @param value the value to add
     * @throws NullPointerException if the key or value is null
     */
-    public void add(@Nonnull String key, @Nonnull String value) {
+    public void add(@NonNull String key, @NonNull String value) {
        this.map.put(sanitizeKey(key), sanitizeValue(value));
    }

@@ -274,7 +261,7 @@ public final class MutableContextSet extends AbstractContextSet implements Conte
     * @param entry the entry to add
     * @throws NullPointerException if the entry is null
     */
-    public void add(@Nonnull Map.Entry<String, String> entry) {
+    public void add(Map.@NonNull Entry<String, String> entry) {
        Objects.requireNonNull(entry, "entry");
        add(entry.getKey(), entry.getValue());
    }
@@ -285,7 +272,7 @@ public final class MutableContextSet extends AbstractContextSet implements Conte
     * @param iterable an iterable of key value context pairs
     * @throws NullPointerException if iterable is null
     */
-    public void addAll(@Nonnull Iterable<? extends Map.Entry<String, String>> iterable) {
+    public void addAll(@NonNull Iterable<? extends Map.Entry<String, String>> iterable) {
        for (Map.Entry<String, String> e : Objects.requireNonNull(iterable, "iterable")) {
            add(e);
        }
@@ -297,7 +284,7 @@ public final class MutableContextSet extends AbstractContextSet implements Conte
     * @param map the map to add from
     * @throws NullPointerException if the map is null
     */
-    public void addAll(@Nonnull Map<String, String> map) {
+    public void addAll(@NonNull Map<String, String> map) {
        addAll(Objects.requireNonNull(map, "map").entrySet());
    }

@@ -308,7 +295,7 @@ public final class MutableContextSet extends AbstractContextSet implements Conte
     * @throws NullPointerException if the map is null
     * @since 3.4
     */
-    public void addAll(@Nonnull Multimap<String, String> multimap) {
+    public void addAll(@NonNull Multimap<String, String> multimap) {
        addAll(Objects.requireNonNull(multimap, "multimap").entries());
    }

@@ -318,7 +305,7 @@ public final class MutableContextSet extends AbstractContextSet implements Conte
     * @param contextSet the set to add from
     * @throws NullPointerException if the contextSet is null
     */
-    public void addAll(@Nonnull ContextSet contextSet) {
+    public void addAll(@NonNull ContextSet contextSet) {
        Objects.requireNonNull(contextSet, "contextSet");
        if (contextSet instanceof AbstractContextSet) {
            AbstractContextSet other = ((AbstractContextSet) contextSet);
@@ -337,7 +324,7 @@ public final class MutableContextSet extends AbstractContextSet implements Conte
     * @param value the value to remove
     * @throws NullPointerException if the key or value is null
     */
-    public void remove(@Nonnull String key, @Nonnull String value) {
+    public void remove(@NonNull String key, @NonNull String value) {
        this.map.remove(sanitizeKey(key), sanitizeValue(value));
    }

@@ -347,7 +334,7 @@ public final class MutableContextSet extends AbstractContextSet implements Conte
     * @param key the key to remove
     * @throws NullPointerException if the key is null
     */
-    public void removeAll(@Nonnull String key) {
+    public void removeAll(@NonNull String key) {
        this.map.removeAll(sanitizeKey(key));
    }

@@ -25,8 +25,8 @@

 package me.lucko.luckperms.api.context;

-import javax.annotation.Nonnull;
-import javax.annotation.Nullable;
+import org.checkerframework.checker.nullness.qual.NonNull;
+import org.checkerframework.checker.nullness.qual.Nullable;

 /**
 * Extension of {@link ContextCalculator} which provides the same context
@@ -43,20 +43,18 @@ public interface StaticContextCalculator extends ContextCalculator<Object> {
     * @param accumulator a map of contexts to add to
     * @return the map
     */
-    @Nonnull
-    MutableContextSet giveApplicableContext(@Nonnull MutableContextSet accumulator);
+    @NonNull MutableContextSet giveApplicableContext(@NonNull MutableContextSet accumulator);

    /**
     * Gives the subject all of the applicable contexts they meet
     *
-     * @param subject the subject to add contexts to
+     * @param subject     the subject to add contexts to
     * @param accumulator a map of contexts to add to
     * @return the map
     */
-    @Nonnull
-    @Override
    @Deprecated
-    default MutableContextSet giveApplicableContext(@Nullable Object subject, @Nonnull MutableContextSet accumulator) {
+    @Override
+    default @NonNull MutableContextSet giveApplicableContext(@Nullable Object subject, @NonNull MutableContextSet accumulator) {
        return giveApplicableContext(accumulator);
    }

@@ -25,9 +25,9 @@

 package me.lucko.luckperms.api.event;

-import java.util.concurrent.atomic.AtomicBoolean;
+import org.checkerframework.checker.nullness.qual.NonNull;

-import javax.annotation.Nonnull;
+import java.util.concurrent.atomic.AtomicBoolean;

 /**
 * Represents an event that can be cancelled
@@ -39,8 +39,7 @@ public interface Cancellable {
     *
     * @return the cancellation
     */
-    @Nonnull
-    AtomicBoolean getCancellationState();
+    @NonNull @Param(-1) AtomicBoolean getCancellationState();

    /**
     * Returns true if the event is currently cancelled.
@@ -25,11 +25,11 @@

 package me.lucko.luckperms.api.event;

+import org.checkerframework.checker.nullness.qual.NonNull;
+
 import java.util.Set;
 import java.util.function.Consumer;

-import javax.annotation.Nonnull;
-
 /**
 * The LuckPerms event bus.
 *
@@ -51,8 +51,7 @@ public interface EventBus {
     * @param <T>        the event class
     * @return an event handler instance representing this subscription
     */
-    @Nonnull
-    <T extends LuckPermsEvent> EventHandler<T> subscribe(@Nonnull Class<T> eventClass, @Nonnull Consumer<? super T> handler);
+    @NonNull <T extends LuckPermsEvent> EventHandler<T> subscribe(@NonNull Class<T> eventClass, @NonNull Consumer<? super T> handler);

    /**
     * Registers a new subscription to the given event.
@@ -72,8 +71,7 @@ public interface EventBus {
     * @param handler    the event handler
     * @return an event handler instance representing this subscription
     */
-    @Nonnull
-    <T extends LuckPermsEvent> EventHandler<T> subscribe(Object plugin, @Nonnull Class<T> eventClass, @Nonnull Consumer<? super T> handler);
+    @NonNull <T extends LuckPermsEvent> EventHandler<T> subscribe(Object plugin, @NonNull Class<T> eventClass, @NonNull Consumer<? super T> handler);

    /**
     * Gets a set of all registered handlers for a given event.
@@ -82,7 +80,6 @@ public interface EventBus {
     * @param <T>        the event class
     * @return an immutable set of event handlers
     */
-    @Nonnull
-    <T extends LuckPermsEvent> Set<EventHandler<T>> getHandlers(@Nonnull Class<T> eventClass);
+    @NonNull <T extends LuckPermsEvent> Set<EventHandler<T>> getHandlers(@NonNull Class<T> eventClass);

 }
@@ -25,9 +25,9 @@

 package me.lucko.luckperms.api.event;

-import java.util.function.Consumer;
+import org.checkerframework.checker.nullness.qual.NonNull;

-import javax.annotation.Nonnull;
+import java.util.function.Consumer;

 /**
 * Represents a subscription to a {@link LuckPermsEvent}.
@@ -41,8 +41,7 @@ public interface EventHandler<T extends LuckPermsEvent> extends AutoCloseable {
     *
     * @return the event class
     */
-    @Nonnull
-    Class<T> getEventClass();
+    @NonNull Class<T> getEventClass();

    /**
     * Returns true if this handler is active
@@ -63,8 +62,7 @@ public interface EventHandler<T extends LuckPermsEvent> extends AutoCloseable {
     *
     * @return the event consumer
     */
-    @Nonnull
-    Consumer<? super T> getConsumer();
+    @NonNull Consumer<? super T> getConsumer();

    /**
     * Gets the number of times this handler has been called
@@ -27,7 +27,7 @@ package me.lucko.luckperms.api.event;

 import me.lucko.luckperms.api.LuckPermsApi;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * A superinterface for all LuckPerms events.
@@ -41,7 +41,6 @@ public interface LuckPermsEvent {
     *
     * @return the api instance
     */
-    @Nonnull
-    LuckPermsApi getApi();
+    @NonNull LuckPermsApi getApi();

 }
@@ -23,21 +23,27 @@
 *  SOFTWARE.
 */

-package me.lucko.luckperms.common.listener;
+package me.lucko.luckperms.api.event;

-import java.util.Set;
-import java.util.UUID;
+import java.lang.annotation.ElementType;
+import java.lang.annotation.Retention;
+import java.lang.annotation.RetentionPolicy;
+import java.lang.annotation.Target;

 /**
- * Handles incoming connections on the platform
+ * Represents the position of a parameter within an event.
+ *
+ * <p>This is an implementation detail and should not be relied upon.</p>
 */
-public interface ConnectionListener {
+@Target(ElementType.METHOD)
+@Retention(RetentionPolicy.RUNTIME)
+public @interface Param {

    /**
-     * Gets the unique players which have connected to the server since it started.
+     * Gets the index of the parameter.
     *
-     * @return the unique connections
+     * @return the index
     */
-    Set<UUID> getUniqueConnections();
+    int value();

 }
@@ -27,7 +27,7 @@ package me.lucko.luckperms.api.event;

 import me.lucko.luckperms.api.event.source.Source;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * Represents an event with a {@link Source}.
@@ -44,7 +44,6 @@ public interface Sourced {
     *
     * @return the source
     */
-    @Nonnull
-    Source getSource();
+    @NonNull @Param(-1) Source getSource();

 }
@@ -28,8 +28,9 @@ package me.lucko.luckperms.api.event.group;
 import me.lucko.luckperms.api.Group;
 import me.lucko.luckperms.api.caching.GroupData;
 import me.lucko.luckperms.api.event.LuckPermsEvent;
+import me.lucko.luckperms.api.event.Param;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * Called when a groups {@link GroupData} is loaded.
@@ -43,15 +44,13 @@ public interface GroupCacheLoadEvent extends LuckPermsEvent {
     *
     * @return the group
     */
-    @Nonnull
-    Group getGroup();
+    @NonNull @Param(0) Group getGroup();

    /**
     * Gets the data that was loaded
     *
     * @return the loaded data
     */
-    @Nonnull
-    GroupData getLoadedData();
+    @NonNull @Param(1) GroupData getLoadedData();

 }
@@ -27,9 +27,10 @@ package me.lucko.luckperms.api.event.group;

 import me.lucko.luckperms.api.Group;
 import me.lucko.luckperms.api.event.LuckPermsEvent;
+import me.lucko.luckperms.api.event.Param;
 import me.lucko.luckperms.api.event.cause.CreationCause;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * Called when a group is created
@@ -41,15 +42,13 @@ public interface GroupCreateEvent extends LuckPermsEvent {
     *
     * @return the new group
     */
-    @Nonnull
-    Group getGroup();
+    @NonNull @Param(0) Group getGroup();

    /**
     * Gets the cause of the creation
     *
     * @return the cause of the creation
     */
-    @Nonnull
-    CreationCause getCause();
+    @NonNull @Param(1) CreationCause getCause();

 }
@@ -28,8 +28,9 @@ package me.lucko.luckperms.api.event.group;
 import me.lucko.luckperms.api.Group;
 import me.lucko.luckperms.api.caching.GroupData;
 import me.lucko.luckperms.api.event.LuckPermsEvent;
+import me.lucko.luckperms.api.event.Param;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * Called when a groups cached data is refreshed
@@ -43,15 +44,13 @@ public interface GroupDataRecalculateEvent extends LuckPermsEvent {
     *
     * @return the group
     */
-    @Nonnull
-    Group getGroup();
+    @NonNull @Param(0) Group getGroup();

    /**
     * Gets the data that was recalculated
     *
     * @return the data
     */
-    @Nonnull
-    GroupData getData();
+    @NonNull @Param(1) GroupData getData();

 }
@@ -27,11 +27,12 @@ package me.lucko.luckperms.api.event.group;

 import me.lucko.luckperms.api.Node;
 import me.lucko.luckperms.api.event.LuckPermsEvent;
+import me.lucko.luckperms.api.event.Param;
 import me.lucko.luckperms.api.event.cause.DeletionCause;

-import java.util.Set;
+import org.checkerframework.checker.nullness.qual.NonNull;

-import javax.annotation.Nonnull;
+import java.util.Set;

 /**
 * Called when a group is deleted
@@ -43,23 +44,20 @@ public interface GroupDeleteEvent extends LuckPermsEvent {
     *
     * @return the name of the deleted group
     */
-    @Nonnull
-    String getGroupName();
+    @NonNull @Param(0) String getGroupName();

    /**
     * Gets an immutable copy of the groups existing data
     *
     * @return a copy of the groups existing data
     */
-    @Nonnull
-    Set<Node> getExistingData();
+    @NonNull @Param(1) Set<Node> getExistingData();

    /**
     * Gets the cause of the deletion
     *
     * @return the cause of the deletion
     */
-    @Nonnull
-    DeletionCause getCause();
+    @NonNull @Param(2) DeletionCause getCause();

 }
@@ -27,8 +27,9 @@ package me.lucko.luckperms.api.event.group;

 import me.lucko.luckperms.api.Group;
 import me.lucko.luckperms.api.event.LuckPermsEvent;
+import me.lucko.luckperms.api.event.Param;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * Called when a group is loaded into memory from the storage.
@@ -42,7 +43,6 @@ public interface GroupLoadEvent extends LuckPermsEvent {
     *
     * @return the group that was loaded
     */
-    @Nonnull
-    Group getGroup();
+    @NonNull @Param(0) Group getGroup();

 }
@@ -28,8 +28,9 @@ package me.lucko.luckperms.api.event.log;
 import me.lucko.luckperms.api.LogEntry;
 import me.lucko.luckperms.api.event.Cancellable;
 import me.lucko.luckperms.api.event.LuckPermsEvent;
+import me.lucko.luckperms.api.event.Param;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * Called when a log entry is about to be sent to notifiable players on the platform
@@ -41,8 +42,7 @@ public interface LogBroadcastEvent extends LuckPermsEvent, Cancellable {
     *
     * @return the log entry to be broadcasted
     */
-    @Nonnull
-    LogEntry getEntry();
+    @NonNull @Param(0) LogEntry getEntry();

    /**
     * Gets where the log entry originated from.
@@ -50,8 +50,7 @@ public interface LogBroadcastEvent extends LuckPermsEvent, Cancellable {
     * @return the origin of the log
     * @since 3.3
     */
-    @Nonnull
-    Origin getOrigin();
+    @NonNull @Param(1) Origin getOrigin();

    /**
     * Represents where a log entry is from
@@ -28,11 +28,12 @@ package me.lucko.luckperms.api.event.log;
 import me.lucko.luckperms.api.LogEntry;
 import me.lucko.luckperms.api.event.Cancellable;
 import me.lucko.luckperms.api.event.LuckPermsEvent;
+import me.lucko.luckperms.api.event.Param;
+
+import org.checkerframework.checker.nullness.qual.NonNull;

 import java.util.UUID;

-import javax.annotation.Nonnull;
-
 /**
 * Called when a log is about to be published to the network via the MessagingService
 *
@@ -45,15 +46,13 @@ public interface LogNetworkPublishEvent extends LuckPermsEvent, Cancellable {
     *
     * @return the id of the log entry being published
     */
-    @Nonnull
-    UUID getLogId();
+    @NonNull @Param(0) UUID getLogId();

    /**
     * Gets the log entry to be published
     *
     * @return the log entry to be published
     */
-    @Nonnull
-    LogEntry getEntry();
+    @NonNull @Param(1) LogEntry getEntry();

 }
@@ -29,8 +29,9 @@ import me.lucko.luckperms.api.Entity;
 import me.lucko.luckperms.api.LogEntry;
 import me.lucko.luckperms.api.event.Cancellable;
 import me.lucko.luckperms.api.event.LuckPermsEvent;
+import me.lucko.luckperms.api.event.Param;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * Called when a log entry is about to be sent to specific notifiable object on
@@ -49,24 +50,21 @@ public interface LogNotifyEvent extends LuckPermsEvent, Cancellable {
     *
     * @return the log entry to be sent
     */
-    @Nonnull
-    LogEntry getEntry();
+    @NonNull @Param(0) LogEntry getEntry();

    /**
     * Gets where the log entry originated from.
     *
     * @return the origin of the log
     */
-    @Nonnull
-    Origin getOrigin();
+    @NonNull @Param(1) Origin getOrigin();

    /**
     * Gets the object to be notified.
     *
     * @return the object to notify
     */
-    @Nonnull
-    Entity getNotifiable();
+    @NonNull @Param(2) Entity getNotifiable();

    /**
     * Represents where a log entry is from
@@ -28,8 +28,9 @@ package me.lucko.luckperms.api.event.log;
 import me.lucko.luckperms.api.LogEntry;
 import me.lucko.luckperms.api.event.Cancellable;
 import me.lucko.luckperms.api.event.LuckPermsEvent;
+import me.lucko.luckperms.api.event.Param;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * Called when a log is about to be published to the storage file/table
@@ -41,7 +42,6 @@ public interface LogPublishEvent extends LuckPermsEvent, Cancellable {
     *
     * @return the log entry to be published
     */
-    @Nonnull
-    LogEntry getEntry();
+    @NonNull @Param(0) LogEntry getEntry();

 }
@@ -27,11 +27,12 @@ package me.lucko.luckperms.api.event.log;

 import me.lucko.luckperms.api.LogEntry;
 import me.lucko.luckperms.api.event.LuckPermsEvent;
+import me.lucko.luckperms.api.event.Param;
+
+import org.checkerframework.checker.nullness.qual.NonNull;

 import java.util.UUID;

-import javax.annotation.Nonnull;
-
 /**
 * Called when a log entry is received via the MessagingService
 *
@@ -44,15 +45,13 @@ public interface LogReceiveEvent extends LuckPermsEvent {
     *
     * @return the id of the log entry being received
     */
-    @Nonnull
-    UUID getLogId();
+    @NonNull @Param(0) UUID getLogId();

    /**
     * Gets the log entry being received
     *
     * @return the log entry being received
     */
-    @Nonnull
-    LogEntry getEntry();
+    @NonNull @Param(1) LogEntry getEntry();

 }
@@ -26,8 +26,9 @@
 package me.lucko.luckperms.api.event.node;

 import me.lucko.luckperms.api.Node;
+import me.lucko.luckperms.api.event.Param;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * Called when a node is added to a holder
@@ -39,7 +40,6 @@ public interface NodeAddEvent extends NodeMutateEvent {
     *
     * @return the node that was added
     */
-    @Nonnull
-    Node getNode();
+    @NonNull @Param(3) Node getNode();

 }
@@ -25,14 +25,17 @@

 package me.lucko.luckperms.api.event.node;

+import me.lucko.luckperms.api.Group;
 import me.lucko.luckperms.api.Node;
 import me.lucko.luckperms.api.PermissionHolder;
+import me.lucko.luckperms.api.User;
 import me.lucko.luckperms.api.event.LuckPermsEvent;
+import me.lucko.luckperms.api.event.Param;
+
+import org.checkerframework.checker.nullness.qual.NonNull;

 import java.util.Set;

-import javax.annotation.Nonnull;
-
 /**
 * Called when a node is added to/removed from a user/group
 */
@@ -43,24 +46,21 @@ public interface NodeMutateEvent extends LuckPermsEvent {
     *
     * @return the event target
     */
-    @Nonnull
-    PermissionHolder getTarget();
+    @NonNull @Param(0) PermissionHolder getTarget();

    /**
     * Gets an immutable copy of the holders data before the change
     *
     * @return the data before the change
     */
-    @Nonnull
-    Set<Node> getDataBefore();
+    @NonNull @Param(1) Set<Node> getDataBefore();

    /**
     * Gets an immutable copy of the holders data after the change
     *
     * @return the data after the change
     */
-    @Nonnull
-    Set<Node> getDataAfter();
+    @NonNull @Param(2) Set<Node> getDataAfter();

    /**
     * Gets whether the target of this event is a {@link me.lucko.luckperms.api.User}
@@ -69,7 +69,9 @@ public interface NodeMutateEvent extends LuckPermsEvent {
     *
     * @return if the event is targeting a user
     */
-    boolean isUser();
+    default boolean isUser() {
+        return getTarget() instanceof User;
+    }

    /**
     * Gets whether the target of this event is a {@link me.lucko.luckperms.api.Group}
@@ -78,6 +80,8 @@ public interface NodeMutateEvent extends LuckPermsEvent {
     *
     * @return if the event is targeting a group
     */
-    boolean isGroup();
+    default boolean isGroup() {
+        return getTarget() instanceof Group;
+    }

 }
@@ -26,8 +26,9 @@
 package me.lucko.luckperms.api.event.node;

 import me.lucko.luckperms.api.Node;
+import me.lucko.luckperms.api.event.Param;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * Called when a node is removed from a holder
@@ -39,7 +40,6 @@ public interface NodeRemoveEvent extends NodeMutateEvent {
     *
     * @return the node that was removed
     */
-    @Nonnull
-    Node getNode();
+    @NonNull @Param(3) Node getNode();

 }
@@ -23,40 +23,45 @@
 *  SOFTWARE.
 */

-package me.lucko.luckperms.common.event.impl;
+package me.lucko.luckperms.api.event.player;

-import me.lucko.luckperms.api.LogEntry;
-import me.lucko.luckperms.api.event.log.LogReceiveEvent;
-import me.lucko.luckperms.common.event.AbstractEvent;
+import me.lucko.luckperms.api.PlayerSaveResult;
+import me.lucko.luckperms.api.event.LuckPermsEvent;
+import me.lucko.luckperms.api.event.Param;
+import me.lucko.luckperms.api.manager.UserManager;
+
+import org.checkerframework.checker.nullness.qual.NonNull;

 import java.util.UUID;

-import javax.annotation.Nonnull;
+/**
+ * Called when player data is saved to the storage.
+ *
+ * <p>Data can be saved using {@link UserManager#savePlayerData(UUID, String)}.</p>
+ *
+ * @since 4.3
+ */
+public interface PlayerDataSaveEvent extends LuckPermsEvent {

-public class EventLogReceive extends AbstractEvent implements LogReceiveEvent {
+    /**
+     * Gets the unique ID that was saved.
+     *
+     * @return the uuid
+     */
+    @NonNull @Param(0) UUID getUuid();

-    private final UUID logId;
-    private final LogEntry entry;
+    /**
+     * Gets the username that was saved.
+     *
+     * @return the username
+     */
+    @NonNull @Param(1) String getUsername();

-    public EventLogReceive(UUID logId, LogEntry entry) {
-        this.logId = logId;
-        this.entry = entry;
-    }
+    /**
+     * Gets the result of the operation.
+     *
+     * @return the result
+     */
+    @NonNull @Param(2) PlayerSaveResult getResult();

-    @Nonnull
-    @Override
-    public UUID getLogId() {
-        return this.logId;
-    }
-
-    @Nonnull
-    @Override
-    public LogEntry getEntry() {
-        return this.entry;
-    }
-
-    @Override
-    public String toString() {
-        return "LogReceiveEvent(logId=" + this.getLogId() + ", entry=" + this.getEntry() + ")";
-    }
 }
@@ -27,7 +27,7 @@ package me.lucko.luckperms.api.event.source;

 import me.lucko.luckperms.api.Entity;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * Represents an {@link Entity} which was the {@link Source} of something.
@@ -41,7 +41,6 @@ public interface EntitySource extends Source {
     *
     * @return the entity
     */
-    @Nonnull
-    Entity getEntity();
+    @NonNull Entity getEntity();

 }
@@ -27,7 +27,7 @@ package me.lucko.luckperms.api.event.source;

 import me.lucko.luckperms.api.Entity;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * Represents the source of an event.
@@ -43,8 +43,7 @@ public interface Source {
     *
     * @return the type
     */
-    @Nonnull
-    Type getType();
+    @NonNull Type getType();

    /**
     * Represents a type of source
@@ -27,11 +27,12 @@ package me.lucko.luckperms.api.event.sync;

 import me.lucko.luckperms.api.event.Cancellable;
 import me.lucko.luckperms.api.event.LuckPermsEvent;
+import me.lucko.luckperms.api.event.Param;
+
+import org.checkerframework.checker.nullness.qual.NonNull;

 import java.util.UUID;

-import javax.annotation.Nonnull;
-
 /**
 * Called before a received network sync task runs
 */
@@ -42,7 +43,6 @@ public interface PreNetworkSyncEvent extends LuckPermsEvent, Cancellable {
     *
     * @return the id of the sync request
     */
-    @Nonnull
-    UUID getSyncId();
+    @NonNull @Param(0) UUID getSyncId();

 }
@@ -27,9 +27,10 @@ package me.lucko.luckperms.api.event.track;

 import me.lucko.luckperms.api.Track;
 import me.lucko.luckperms.api.event.LuckPermsEvent;
+import me.lucko.luckperms.api.event.Param;
 import me.lucko.luckperms.api.event.cause.CreationCause;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * Called when a track is created
@@ -41,15 +42,13 @@ public interface TrackCreateEvent extends LuckPermsEvent {
     *
     * @return the new track
     */
-    @Nonnull
-    Track getTrack();
+    @NonNull @Param(0) Track getTrack();

    /**
     * Gets the cause of the creation
     *
     * @return the cause of the creation
     */
-    @Nonnull
-    CreationCause getCause();
+    @NonNull @Param(1) CreationCause getCause();

 }
@@ -26,11 +26,12 @@
 package me.lucko.luckperms.api.event.track;

 import me.lucko.luckperms.api.event.LuckPermsEvent;
+import me.lucko.luckperms.api.event.Param;
 import me.lucko.luckperms.api.event.cause.DeletionCause;

-import java.util.List;
+import org.checkerframework.checker.nullness.qual.NonNull;

-import javax.annotation.Nonnull;
+import java.util.List;

 /**
 * Called when a track is deleted
@@ -42,23 +43,20 @@ public interface TrackDeleteEvent extends LuckPermsEvent {
     *
     * @return the name of the deleted track
     */
-    @Nonnull
-    String getTrackName();
+    @NonNull @Param(0) String getTrackName();

    /**
     * Gets an immutable copy of the tracks existing data
     *
     * @return a copy of the tracks existing data
     */
-    @Nonnull
-    List<String> getExistingData();
+    @NonNull @Param(1) List<String> getExistingData();

    /**
     * Gets the cause of the deletion
     *
     * @return the cause of the deletion
     */
-    @Nonnull
-    DeletionCause getCause();
+    @NonNull @Param(2) DeletionCause getCause();

 }
@@ -27,8 +27,9 @@ package me.lucko.luckperms.api.event.track;

 import me.lucko.luckperms.api.Track;
 import me.lucko.luckperms.api.event.LuckPermsEvent;
+import me.lucko.luckperms.api.event.Param;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * Called when a track is loaded into memory from the storage.
@@ -42,7 +43,6 @@ public interface TrackLoadEvent extends LuckPermsEvent {
     *
     * @return the track that was loaded
     */
-    @Nonnull
-    Track getTrack();
+    @NonNull @Param(0) Track getTrack();

 }
@@ -25,7 +25,9 @@

 package me.lucko.luckperms.api.event.track.mutate;

-import javax.annotation.Nonnull;
+import me.lucko.luckperms.api.event.Param;
+
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * Called when a group is added to a track
@@ -37,7 +39,6 @@ public interface TrackAddGroupEvent extends TrackMutateEvent {
     *
     * @return the group that was added
     */
-    @Nonnull
-    String getGroup();
+    @NonNull @Param(3) String getGroup();

 }
@@ -27,11 +27,12 @@ package me.lucko.luckperms.api.event.track.mutate;

 import me.lucko.luckperms.api.Track;
 import me.lucko.luckperms.api.event.LuckPermsEvent;
+import me.lucko.luckperms.api.event.Param;
+
+import org.checkerframework.checker.nullness.qual.NonNull;

 import java.util.List;

-import javax.annotation.Nonnull;
-
 /**
 * Called when a track is changed
 */
@@ -42,23 +43,20 @@ public interface TrackMutateEvent extends LuckPermsEvent {
     *
     * @return the track that was mutated
     */
-    @Nonnull
-    Track getTrack();
+    @NonNull @Param(0) Track getTrack();

    /**
     * Gets an immutable copy of the tracks data before the change
     *
     * @return the data before the change
     */
-    @Nonnull
-    List<String> getDataBefore();
+    @NonNull @Param(1) List<String> getDataBefore();

    /**
     * Gets an immutable copy of the tracks data after the change
     *
     * @return the data after the change
     */
-    @Nonnull
-    List<String> getDataAfter();
+    @NonNull @Param(2) List<String> getDataAfter();

 }
@@ -25,7 +25,9 @@

 package me.lucko.luckperms.api.event.track.mutate;

-import javax.annotation.Nonnull;
+import me.lucko.luckperms.api.event.Param;
+
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * Called when a group is removed from a track
@@ -37,7 +39,6 @@ public interface TrackRemoveGroupEvent extends TrackMutateEvent {
     *
     * @return the group that was removed
     */
-    @Nonnull
-    String getGroup();
+    @NonNull @Param(3) String getGroup();

 }
@@ -28,11 +28,12 @@ package me.lucko.luckperms.api.event.user;
 import me.lucko.luckperms.api.User;
 import me.lucko.luckperms.api.caching.UserData;
 import me.lucko.luckperms.api.event.LuckPermsEvent;
+import me.lucko.luckperms.api.event.Param;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
- * Called when a users {@link me.lucko.luckperms.api.caching.UserData} is loaded.
+ * Called when a users {@link UserData} is loaded.
 */
 public interface UserCacheLoadEvent extends LuckPermsEvent {

@@ -41,15 +42,13 @@ public interface UserCacheLoadEvent extends LuckPermsEvent {
     *
     * @return the user
     */
-    @Nonnull
-    User getUser();
+    @NonNull @Param(0) User getUser();

    /**
     * Gets the data that was loaded
     *
     * @return the loaded data
     */
-    @Nonnull
-    UserData getLoadedData();
+    @NonNull @Param(1) UserData getLoadedData();

 }
@@ -28,8 +28,9 @@ package me.lucko.luckperms.api.event.user;
 import me.lucko.luckperms.api.User;
 import me.lucko.luckperms.api.caching.UserData;
 import me.lucko.luckperms.api.event.LuckPermsEvent;
+import me.lucko.luckperms.api.event.Param;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * Called when a users cached data is refreshed
@@ -41,15 +42,13 @@ public interface UserDataRecalculateEvent extends LuckPermsEvent {
     *
     * @return the user
     */
-    @Nonnull
-    User getUser();
+    @NonNull @Param(0) User getUser();

    /**
     * Gets the data that was recalculated
     *
     * @return the data
     */
-    @Nonnull
-    UserData getData();
+    @NonNull @Param(1) UserData getData();

 }
@@ -26,11 +26,12 @@
 package me.lucko.luckperms.api.event.user;

 import me.lucko.luckperms.api.event.LuckPermsEvent;
+import me.lucko.luckperms.api.event.Param;
+
+import org.checkerframework.checker.nullness.qual.NonNull;

 import java.util.UUID;

-import javax.annotation.Nonnull;
-
 /**
 * Called when the user logs into the network for the first time.
 *
@@ -49,15 +50,13 @@ public interface UserFirstLoginEvent extends LuckPermsEvent {
     *
     * @return the uuid of the user
     */
-    @Nonnull
-    UUID getUuid();
+    @NonNull @Param(0) UUID getUuid();

    /**
     * Gets the username of the user
     *
     * @return the username of the user
     */
-    @Nonnull
-    String getUsername();
+    @NonNull @Param(1) String getUsername();

 }
@@ -27,8 +27,9 @@ package me.lucko.luckperms.api.event.user;

 import me.lucko.luckperms.api.User;
 import me.lucko.luckperms.api.event.LuckPermsEvent;
+import me.lucko.luckperms.api.event.Param;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * Called when a user is loaded into memory from the storage.
@@ -40,7 +41,6 @@ public interface UserLoadEvent extends LuckPermsEvent {
     *
     * @return the user that was loaded
     */
-    @Nonnull
-    User getUser();
+    @NonNull @Param(0) User getUser();

 }
@@ -27,11 +27,12 @@ package me.lucko.luckperms.api.event.user;

 import me.lucko.luckperms.api.User;
 import me.lucko.luckperms.api.event.LuckPermsEvent;
+import me.lucko.luckperms.api.event.Param;
+
+import org.checkerframework.checker.nullness.qual.NonNull;

 import java.util.UUID;

-import javax.annotation.Nonnull;
-
 /**
 * Called when LuckPerms has finished processing a certain Player's connection.
 *
@@ -51,22 +52,20 @@ public interface UserLoginProcessEvent extends LuckPermsEvent {
     *
     * @return the uuid of the connection which was processed
     */
-    @Nonnull
-    UUID getUuid();
+    @NonNull @Param(0) UUID getUuid();

    /**
     * Gets the username of the connection which was processed
     *
     * @return the username of the connection which was processed
     */
-    @Nonnull
-    String getUsername();
+    @NonNull @Param(1) String getUsername();

    /**
     * Gets the resultant User instance which was loaded.
     *
     * @return the user instance
     */
-    User getUser();
+    @NonNull @Param(2) User getUser();

 }
@@ -25,6 +25,8 @@

 package me.lucko.luckperms.api.event.user.track;

+import org.checkerframework.checker.nullness.qual.NonNull;
+
 /**
 * Called when a user is demoted down a track.
 *
@@ -32,4 +34,8 @@ package me.lucko.luckperms.api.event.user.track;
 */
 public interface UserDemoteEvent extends UserTrackEvent {

+    @Override
+    default @NonNull TrackAction getAction() {
+        return TrackAction.DEMOTION;
+    }
 }
@@ -25,6 +25,8 @@

 package me.lucko.luckperms.api.event.user.track;

+import org.checkerframework.checker.nullness.qual.NonNull;
+
 /**
 * Called when a user is promoted up a track.
 *
@@ -32,4 +34,8 @@ package me.lucko.luckperms.api.event.user.track;
 */
 public interface UserPromoteEvent extends UserTrackEvent {

+    @Override
+    default @NonNull TrackAction getAction() {
+        return TrackAction.PROMOTION;
+    }
 }
@@ -28,11 +28,12 @@ package me.lucko.luckperms.api.event.user.track;
 import me.lucko.luckperms.api.Track;
 import me.lucko.luckperms.api.User;
 import me.lucko.luckperms.api.event.LuckPermsEvent;
+import me.lucko.luckperms.api.event.Param;
 import me.lucko.luckperms.api.event.Sourced;

-import java.util.Optional;
+import org.checkerframework.checker.nullness.qual.NonNull;

-import javax.annotation.Nonnull;
+import java.util.Optional;

 /**
 * Called when a user interacts with a track through a promotion or demotion
@@ -44,24 +45,21 @@ public interface UserTrackEvent extends LuckPermsEvent, Sourced {
     *
     * @return the track involved in the event
     */
-    @Nonnull
-    Track getTrack();
+    @NonNull @Param(0) Track getTrack();

    /**
     * Gets the user who was promoted or demoted
     *
     * @return the user involved in the event
     */
-    @Nonnull
-    User getUser();
+    @NonNull @Param(1) User getUser();

    /**
     * Gets the action performed
     *
     * @return the action performed
     */
-    @Nonnull
-    TrackAction getAction();
+    @NonNull TrackAction getAction();

    /**
     * Gets the group the user was promoted/demoted from.
@@ -70,15 +68,13 @@ public interface UserTrackEvent extends LuckPermsEvent, Sourced {
     *
     * @return the group the user was promoted/demoted from
     */
-    @Nonnull
-    Optional<String> getGroupFrom();
+    @NonNull @Param(2) Optional<String> getGroupFrom();

    /**
     * Gets the group the user was promoted/demoted to
     *
     * @return the group the user was promoted/demoted to
     */
-    @Nonnull
-    Optional<String> getGroupTo();
+    @NonNull @Param(3) Optional<String> getGroupTo();

 }
@@ -29,6 +29,9 @@ import me.lucko.luckperms.api.Group;
 import me.lucko.luckperms.api.HeldPermission;
 import me.lucko.luckperms.api.Storage;

+import org.checkerframework.checker.nullness.qual.NonNull;
+import org.checkerframework.checker.nullness.qual.Nullable;
+
 import java.util.List;
 import java.util.Optional;
 import java.util.Set;
@@ -36,9 +39,6 @@ import java.util.concurrent.CompletableFuture;
 import java.util.concurrent.Executor;
 import java.util.function.Consumer;

-import javax.annotation.Nonnull;
-import javax.annotation.Nullable;
-
 /**
 * Represents the object responsible for managing {@link Group} instances.
 *
@@ -74,8 +74,7 @@ public interface GroupManager {
     * @throws NullPointerException if the name is null
     * @since 4.1
     */
-    @Nonnull
-    CompletableFuture<Group> createAndLoadGroup(@Nonnull String name);
+    @NonNull CompletableFuture<Group> createAndLoadGroup(@NonNull String name);

    /**
     * Loads a group from the plugin's storage provider into memory.
@@ -95,8 +94,7 @@ public interface GroupManager {
     * @throws NullPointerException if the name is null
     * @since 4.1
     */
-    @Nonnull
-    CompletableFuture<Optional<Group>> loadGroup(@Nonnull String name);
+    @NonNull CompletableFuture<Optional<Group>> loadGroup(@NonNull String name);

    /**
     * Saves a group's data back to the plugin's storage provider.
@@ -115,8 +113,7 @@ public interface GroupManager {
     * @throws IllegalStateException if the group instance was not obtained from LuckPerms.
     * @since 4.1
     */
-    @Nonnull
-    CompletableFuture<Void> saveGroup(@Nonnull Group group);
+    @NonNull CompletableFuture<Void> saveGroup(@NonNull Group group);

    /**
     * Permanently deletes a group from the plugin's storage provider.
@@ -127,15 +124,13 @@ public interface GroupManager {
     * <p>Unlike the method in {@link Storage}, when a group cannot be deleted,
     * the future will be {@link CompletableFuture completed exceptionally}.</p>
     *
-     *
     * @param group the group to delete
     * @return a future to encapsulate the operation.
     * @throws NullPointerException  if group is null
     * @throws IllegalStateException if the group instance was not obtained from LuckPerms.
     * @since 4.1
     */
-    @Nonnull
-    CompletableFuture<Void> deleteGroup(@Nonnull Group group);
+    @NonNull CompletableFuture<Void> deleteGroup(@NonNull Group group);

    /**
     * Loads all groups into memory.
@@ -149,8 +144,7 @@ public interface GroupManager {
     * @return a future to encapsulate the operation.
     * @since 4.1
     */
-    @Nonnull
-    CompletableFuture<Void> loadAllGroups();
+    @NonNull CompletableFuture<Void> loadAllGroups();

    /**
     * Searches for a list of groups with a given permission.
@@ -160,8 +154,7 @@ public interface GroupManager {
     * @throws NullPointerException if the permission is null
     * @since 4.2
     */
-    @Nonnull
-    CompletableFuture<List<HeldPermission<String>>> getWithPermission(@Nonnull String permission);
+    @NonNull CompletableFuture<List<HeldPermission<String>>> getWithPermission(@NonNull String permission);

    /**
     * Gets a loaded group.
@@ -170,8 +163,7 @@ public interface GroupManager {
     * @return a {@link Group} object, if one matching the name exists, or null if not
     * @throws NullPointerException if the name is null
     */
-    @Nullable
-    Group getGroup(@Nonnull String name);
+    @Nullable Group getGroup(@NonNull String name);

    /**
     * Gets a loaded group.
@@ -182,8 +174,7 @@ public interface GroupManager {
     * @return an optional {@link Group} object
     * @throws NullPointerException if the name is null
     */
-    @Nonnull
-    default Optional<Group> getGroupOpt(@Nonnull String name) {
+    default @NonNull Optional<Group> getGroupOpt(@NonNull String name) {
        return Optional.ofNullable(getGroup(name));
    }

@@ -192,8 +183,7 @@ public interface GroupManager {
     *
     * @return a {@link Set} of {@link Group} objects
     */
-    @Nonnull
-    Set<Group> getLoadedGroups();
+    @NonNull Set<Group> getLoadedGroups();

    /**
     * Check if a group is loaded in memory
@@ -202,6 +192,6 @@ public interface GroupManager {
     * @return true if the group is loaded
     * @throws NullPointerException if the name is null
     */
-    boolean isLoaded(@Nonnull String name);
+    boolean isLoaded(@NonNull String name);

 }
@@ -28,15 +28,15 @@ package me.lucko.luckperms.api.manager;
 import me.lucko.luckperms.api.Storage;
 import me.lucko.luckperms.api.Track;

+import org.checkerframework.checker.nullness.qual.NonNull;
+import org.checkerframework.checker.nullness.qual.Nullable;
+
 import java.util.Optional;
 import java.util.Set;
 import java.util.concurrent.CompletableFuture;
 import java.util.concurrent.Executor;
 import java.util.function.Consumer;

-import javax.annotation.Nonnull;
-import javax.annotation.Nullable;
-
 /**
 * Represents the object responsible for managing {@link Track} instances.
 *
@@ -72,8 +72,7 @@ public interface TrackManager {
     * @throws NullPointerException if the name is null
     * @since 4.1
     */
-    @Nonnull
-    CompletableFuture<Track> createAndLoadTrack(@Nonnull String name);
+    @NonNull CompletableFuture<Track> createAndLoadTrack(@NonNull String name);

    /**
     * Loads a track from the plugin's storage provider into memory.
@@ -93,8 +92,7 @@ public interface TrackManager {
     * @throws NullPointerException if the name is null
     * @since 4.1
     */
-    @Nonnull
-    CompletableFuture<Optional<Track>> loadTrack(@Nonnull String name);
+    @NonNull CompletableFuture<Optional<Track>> loadTrack(@NonNull String name);

    /**
     * Saves a track's data back to the plugin's storage provider.
@@ -113,8 +111,7 @@ public interface TrackManager {
     * @throws IllegalStateException if the track instance was not obtained from LuckPerms.
     * @since 4.1
     */
-    @Nonnull
-    CompletableFuture<Void> saveTrack(@Nonnull Track track);
+    @NonNull CompletableFuture<Void> saveTrack(@NonNull Track track);

    /**
     * Permanently deletes a track from the plugin's storage provider.
@@ -125,15 +122,13 @@ public interface TrackManager {
     * <p>Unlike the method in {@link Storage}, when a track cannot be deleted,
     * the future will be {@link CompletableFuture completed exceptionally}.</p>
     *
-     *
     * @param track the track to delete
     * @return a future to encapsulate the operation.
     * @throws NullPointerException  if track is null
     * @throws IllegalStateException if the track instance was not obtained from LuckPerms.
     * @since 4.1
     */
-    @Nonnull
-    CompletableFuture<Void> deleteTrack(@Nonnull Track track);
+    @NonNull CompletableFuture<Void> deleteTrack(@NonNull Track track);

    /**
     * Loads all tracks into memory.
@@ -147,8 +142,7 @@ public interface TrackManager {
     * @return a future to encapsulate the operation.
     * @since 4.1
     */
-    @Nonnull
-    CompletableFuture<Void> loadAllTracks();
+    @NonNull CompletableFuture<Void> loadAllTracks();

    /**
     * Gets a loaded track.
@@ -157,8 +151,7 @@ public interface TrackManager {
     * @return a {@link Track} object, if one matching the name exists, or null if not
     * @throws NullPointerException if the name is null
     */
-    @Nullable
-    Track getTrack(@Nonnull String name);
+    @Nullable Track getTrack(@NonNull String name);

    /**
     * Gets a loaded track.
@@ -169,8 +162,7 @@ public interface TrackManager {
     * @return an optional {@link Track} object
     * @throws NullPointerException if the name is null
     */
-    @Nonnull
-    default Optional<Track> getTrackOpt(@Nonnull String name) {
+    default @NonNull Optional<Track> getTrackOpt(@NonNull String name) {
        return Optional.ofNullable(getTrack(name));
    }

@@ -179,8 +171,7 @@ public interface TrackManager {
     *
     * @return a {@link Set} of {@link Track} objects
     */
-    @Nonnull
-    Set<Track> getLoadedTracks();
+    @NonNull Set<Track> getLoadedTracks();

    /**
     * Check if a track is loaded in memory
@@ -189,6 +180,6 @@ public interface TrackManager {
     * @return true if the track is loaded
     * @throws NullPointerException if the name is null
     */
-    boolean isLoaded(@Nonnull String name);
+    boolean isLoaded(@NonNull String name);

 }
@@ -30,6 +30,9 @@ import me.lucko.luckperms.api.PlayerSaveResult;
 import me.lucko.luckperms.api.Storage;
 import me.lucko.luckperms.api.User;

+import org.checkerframework.checker.nullness.qual.NonNull;
+import org.checkerframework.checker.nullness.qual.Nullable;
+
 import java.util.List;
 import java.util.Optional;
 import java.util.Set;
@@ -38,9 +41,6 @@ import java.util.concurrent.CompletableFuture;
 import java.util.concurrent.Executor;
 import java.util.function.Consumer;

-import javax.annotation.Nonnull;
-import javax.annotation.Nullable;
-
 /**
 * Represents the object responsible for managing {@link User} instances.
 *
@@ -71,14 +71,13 @@ public interface UserManager {
     * <p>Unlike the method in {@link Storage}, when a user cannot be loaded,
     * the future will be {@link CompletableFuture completed exceptionally}.</p>
     *
-     * @param uuid the uuid of the user
+     * @param uuid     the uuid of the user
     * @param username the username, if known
     * @return the resultant user
     * @throws NullPointerException if the uuid is null
     * @since 4.1
     */
-    @Nonnull
-    CompletableFuture<User> loadUser(@Nonnull UUID uuid, @Nullable String username);
+    @NonNull CompletableFuture<User> loadUser(@NonNull UUID uuid, @Nullable String username);

    /**
     * Loads a user from the plugin's storage provider into memory.
@@ -95,8 +94,7 @@ public interface UserManager {
     * @throws NullPointerException if the uuid is null
     * @since 4.1
     */
-    @Nonnull
-    default CompletableFuture<User> loadUser(@Nonnull UUID uuid) {
+    default @NonNull CompletableFuture<User> loadUser(@NonNull UUID uuid) {
        return loadUser(uuid, null);
    }

@@ -111,8 +109,7 @@ public interface UserManager {
     * @throws IllegalArgumentException if the username is invalid
     * @since 4.2
     */
-    @Nonnull
-    CompletableFuture<UUID> lookupUuid(@Nonnull String username);
+    @NonNull CompletableFuture<UUID> lookupUuid(@NonNull String username);

    /**
     * Uses the LuckPerms cache to find a username for the given uuid.
@@ -123,8 +120,7 @@ public interface UserManager {
     * @throws IllegalArgumentException if the username is invalid
     * @since 4.2
     */
-    @Nonnull
-    CompletableFuture<String> lookupUsername(@Nonnull UUID uuid);
+    @NonNull CompletableFuture<String> lookupUsername(@NonNull UUID uuid);

    /**
     * Saves a user's data back to the plugin's storage provider.
@@ -143,8 +139,7 @@ public interface UserManager {
     * @throws IllegalStateException if the user instance was not obtained from LuckPerms.
     * @since 4.1
     */
-    @Nonnull
-    CompletableFuture<Void> saveUser(@Nonnull User user);
+    @NonNull CompletableFuture<Void> saveUser(@NonNull User user);

    /**
     * Saves data about a player to the uuid caching system.
@@ -156,8 +151,7 @@ public interface UserManager {
     * @throws IllegalArgumentException if the username is invalid
     * @since 4.2
     */
-    @Nonnull
-    CompletableFuture<PlayerSaveResult> savePlayerData(@Nonnull UUID uuid, @Nonnull String username);
+    @NonNull CompletableFuture<PlayerSaveResult> savePlayerData(@NonNull UUID uuid, @NonNull String username);

    /**
     * Gets a set all "unique" user UUIDs.
@@ -167,8 +161,7 @@ public interface UserManager {
     * @return a set of uuids
     * @since 4.2
     */
-    @Nonnull
-    CompletableFuture<Set<UUID>> getUniqueUsers();
+    @NonNull CompletableFuture<Set<UUID>> getUniqueUsers();

    /**
     * Searches for a list of users with a given permission.
@@ -178,8 +171,7 @@ public interface UserManager {
     * @throws NullPointerException if the permission is null
     * @since 4.2
     */
-    @Nonnull
-    CompletableFuture<List<HeldPermission<UUID>>> getWithPermission(@Nonnull String permission);
+    @NonNull CompletableFuture<List<HeldPermission<UUID>>> getWithPermission(@NonNull String permission);

    /**
     * Gets a loaded user.
@@ -188,8 +180,7 @@ public interface UserManager {
     * @return a {@link User} object, if one matching the uuid is loaded, or null if not
     * @throws NullPointerException if the uuid is null
     */
-    @Nullable
-    User getUser(@Nonnull UUID uuid);
+    @Nullable User getUser(@NonNull UUID uuid);

    /**
     * Gets a loaded user.
@@ -198,8 +189,7 @@ public interface UserManager {
     * @return an optional {@link User} object
     * @throws NullPointerException if the uuid is null
     */
-    @Nonnull
-    default Optional<User> getUserOpt(@Nonnull UUID uuid) {
+    default @NonNull Optional<User> getUserOpt(@NonNull UUID uuid) {
        return Optional.ofNullable(getUser(uuid));
    }

@@ -210,8 +200,7 @@ public interface UserManager {
     * @return a {@link User} object, if one matching the uuid is loaded, or null if not
     * @throws NullPointerException if the name is null
     */
-    @Nullable
-    User getUser(@Nonnull String name);
+    @Nullable User getUser(@NonNull String name);

    /**
     * Gets a loaded user.
@@ -220,8 +209,7 @@ public interface UserManager {
     * @return an optional {@link User} object
     * @throws NullPointerException if the name is null
     */
-    @Nonnull
-    default Optional<User> getUserOpt(@Nonnull String name) {
+    default @NonNull Optional<User> getUserOpt(@NonNull String name) {
        return Optional.ofNullable(getUser(name));
    }

@@ -230,8 +218,7 @@ public interface UserManager {
     *
     * @return a {@link Set} of {@link User} objects
     */
-    @Nonnull
-    Set<User> getLoadedUsers();
+    @NonNull Set<User> getLoadedUsers();

    /**
     * Check if a user is loaded in memory
@@ -240,7 +227,7 @@ public interface UserManager {
     * @return true if the user is loaded
     * @throws NullPointerException if the uuid is null
     */
-    boolean isLoaded(@Nonnull UUID uuid);
+    boolean isLoaded(@NonNull UUID uuid);

    /**
     * Unload a user from the internal storage, if they're not currently online.
@@ -248,6 +235,6 @@ public interface UserManager {
     * @param user the user to unload
     * @throws NullPointerException if the user is null
     */
-    void cleanupUser(@Nonnull User user);
+    void cleanupUser(@NonNull User user);

 }
@@ -28,11 +28,11 @@ package me.lucko.luckperms.api.messenger;
 import me.lucko.luckperms.api.messenger.message.Message;
 import me.lucko.luckperms.api.messenger.message.OutgoingMessage;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * Encapsulates the LuckPerms system which accepts incoming {@link Message}s
- * from implementations of {@link MessengerProvider}.
+ * from implementations of {@link Messenger}.
 *
 * @since 4.1
 */
@@ -52,7 +52,7 @@ public interface IncomingMessageConsumer {
     * @param message the message
     * @return true if the message was accepted by the plugin
     */
-    boolean consumeIncomingMessage(@Nonnull Message message);
+    boolean consumeIncomingMessage(@NonNull Message message);

    /**
     * Consumes a message in an encoded string format.
@@ -73,6 +73,6 @@ public interface IncomingMessageConsumer {
     * @param encodedString the encoded string
     * @return true if the message was accepted by the plugin
     */
-    boolean consumeIncomingMessageAsString(@Nonnull String encodedString);
+    boolean consumeIncomingMessageAsString(@NonNull String encodedString);

 }
@@ -28,7 +28,7 @@ package me.lucko.luckperms.api.messenger;
 import me.lucko.luckperms.api.messenger.message.Message;
 import me.lucko.luckperms.api.messenger.message.OutgoingMessage;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * Represents an object which dispatches {@link OutgoingMessage}s.
@@ -45,12 +45,18 @@ public interface Messenger extends AutoCloseable {
     * of the interfaces extending {@link Message} in the
     * 'api.messenger.message.type' package.</p>
     *
+     * <p>3rd party implementations are encouraged to implement this method with consideration
+     * that new types may be added in the future.</p>
+     *
     * <p>This call is always made async.</p>
     *
     * @param outgoingMessage the outgoing message
     */
-    void sendOutgoingMessage(@Nonnull OutgoingMessage outgoingMessage);
+    void sendOutgoingMessage(@NonNull OutgoingMessage outgoingMessage);

+    /**
+     * Performs the necessary action to gracefully shutdown the messenger.
+     */
    @Override
    default void close() {

@@ -27,7 +27,7 @@ package me.lucko.luckperms.api.messenger;

 import me.lucko.luckperms.api.LuckPermsApi;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * Represents a provider for {@link Messenger} instances.
@@ -35,8 +35,8 @@ import javax.annotation.Nonnull;
 * <p>Users wishing to provide their own implementation for the plugins
 * "Messaging Service" should implement and register this interface.</p>
 *
- * @since 4.1
 * @see LuckPermsApi#registerMessengerProvider(MessengerProvider)
+ * @since 4.1
 */
 public interface MessengerProvider {

@@ -45,8 +45,7 @@ public interface MessengerProvider {
     *
     * @return the provider name
     */
-    @Nonnull
-    String getName();
+    @NonNull String getName();

    /**
     * Creates and returns a new {@link Messenger} instance, which passes
@@ -59,7 +58,6 @@ public interface MessengerProvider {
     *                                incoming messages to
     * @return a new messenger agent instance
     */
-    @Nonnull
-    Messenger obtain(@Nonnull IncomingMessageConsumer incomingMessageConsumer);
+    @NonNull Messenger obtain(@NonNull IncomingMessageConsumer incomingMessageConsumer);

 }
@@ -27,9 +27,9 @@ package me.lucko.luckperms.api.messenger.message;

 import me.lucko.luckperms.api.messenger.Messenger;

-import java.util.UUID;
+import org.checkerframework.checker.nullness.qual.NonNull;

-import javax.annotation.Nonnull;
+import java.util.UUID;

 /**
 * Represents a message sent received via a {@link Messenger}.
@@ -46,7 +46,6 @@ public interface Message {
     *
     * @return the id of the message
     */
-    @Nonnull
-    UUID getId();
+    @NonNull UUID getId();

 }
@@ -27,7 +27,7 @@ package me.lucko.luckperms.api.messenger.message;

 import me.lucko.luckperms.api.messenger.IncomingMessageConsumer;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * Represents an outgoing {@link Message}.
@@ -56,7 +56,6 @@ public interface OutgoingMessage extends Message {
     *
     * @return an encoded string form of the message
     */
-    @Nonnull
-    String asEncodedString();
+    @NonNull String asEncodedString();

 }
@@ -28,7 +28,7 @@ package me.lucko.luckperms.api.messenger.message.type;
 import me.lucko.luckperms.api.LogEntry;
 import me.lucko.luckperms.api.messenger.message.Message;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * Represents an "log" message.
@@ -44,7 +44,6 @@ public interface LogMessage extends Message {
     *
     * @return the log entry
     */
-    @Nonnull
-    LogEntry getLogEntry();
+    @NonNull LogEntry getLogEntry();

 }
@@ -27,9 +27,9 @@ package me.lucko.luckperms.api.messenger.message.type;

 import me.lucko.luckperms.api.messenger.message.Message;

-import java.util.UUID;
+import org.checkerframework.checker.nullness.qual.NonNull;

-import javax.annotation.Nonnull;
+import java.util.UUID;

 /**
 * Represents an "user update" message.
@@ -45,7 +45,6 @@ public interface UserUpdateMessage extends Message {
     *
     * @return the user
     */
-    @Nonnull
-    UUID getUser();
+    @NonNull UUID getUser();

 }
@@ -25,10 +25,9 @@

 package me.lucko.luckperms.api.metastacking;

-import java.util.List;
+import org.checkerframework.checker.nullness.qual.NonNull;

-import javax.annotation.Nonnull;
-import javax.annotation.concurrent.Immutable;
+import java.util.List;

 /**
 * Represents a meta stack model, consisting of a chain of elements, separated by spacers.
@@ -41,7 +40,6 @@ import javax.annotation.concurrent.Immutable;
 *
 * @since 2.3
 */
-@Immutable
 public interface MetaStackDefinition {

    /**
@@ -49,31 +47,27 @@ public interface MetaStackDefinition {
     *
     * @return the elements in this stack
     */
-    @Nonnull
-    List<MetaStackElement> getElements();
+    @NonNull List<MetaStackElement> getElements();

    /**
     * Gets the spacer string added before any stack elements
     *
     * @return the start spacer
     */
-    @Nonnull
-    String getStartSpacer();
+    @NonNull String getStartSpacer();

    /**
     * Gets the spacer added between stack elements
     *
     * @return the middle spacer
     */
-    @Nonnull
-    String getMiddleSpacer();
+    @NonNull String getMiddleSpacer();

    /**
     * Gets the spacer added after any stack elements
     *
     * @return the end spacer
     */
-    @Nonnull
-    String getEndSpacer();
+    @NonNull String getEndSpacer();

 }
@@ -29,11 +29,10 @@ import me.lucko.luckperms.api.ChatMetaType;
 import me.lucko.luckperms.api.LocalizedNode;
 import me.lucko.luckperms.api.Node;

-import java.util.Map;
+import org.checkerframework.checker.nullness.qual.NonNull;
+import org.checkerframework.checker.nullness.qual.Nullable;

-import javax.annotation.Nonnull;
-import javax.annotation.Nullable;
-import javax.annotation.concurrent.Immutable;
+import java.util.Map;

 /**
 * Represents an element within a {@link MetaStackDefinition}.
@@ -42,7 +41,6 @@ import javax.annotation.concurrent.Immutable;
 *
 * @since 3.2
 */
-@Immutable
 public interface MetaStackElement {

    /**
@@ -50,11 +48,11 @@ public interface MetaStackElement {
     *
     * <p>The element being considered can be retrieved using {@link ChatMetaType#getEntry(Node)}.</p>
     *
-     * @param node the node being considered
-     * @param type the type of entry being accumulated
+     * @param node    the node being considered
+     * @param type    the type of entry being accumulated
     * @param current the current value being used. If this returns true, the current value will be replaced by this entry
     * @return true if the node should be accumulated into this element, replacing the current value
     */
-    boolean shouldAccumulate(@Nonnull LocalizedNode node, @Nonnull ChatMetaType type, @Nullable Map.Entry<Integer, String> current);
+    boolean shouldAccumulate(@NonNull LocalizedNode node, @NonNull ChatMetaType type, Map.@Nullable Entry<Integer, String> current);

 }
@@ -25,11 +25,11 @@

 package me.lucko.luckperms.api.metastacking;

+import org.checkerframework.checker.nullness.qual.NonNull;
+
 import java.util.List;
 import java.util.Optional;

-import javax.annotation.Nonnull;
-
 /**
 * Factory to create meta stack elements and definitions.
 *
@@ -43,8 +43,7 @@ public interface MetaStackFactory {
     * @param definition the definition
     * @return the parsed element, if present
     */
-    @Nonnull
-    Optional<MetaStackElement> fromString(@Nonnull String definition);
+    @NonNull Optional<MetaStackElement> fromString(@NonNull String definition);

    /**
     * Parses a list of {@link MetaStackElement}s from string, using the pre-defined elements in the plugin.
@@ -54,19 +53,17 @@ public interface MetaStackFactory {
     * @param definitions the definition strings
     * @return a list of parsed elements
     */
-    @Nonnull
-    List<MetaStackElement> fromStrings(@Nonnull List<String> definitions);
+    @NonNull List<MetaStackElement> fromStrings(@NonNull List<String> definitions);

    /**
     * Creates a new {@link MetaStackDefinition} with the given properties.
     *
-     * @param elements the elements to be included in the stack.
-     * @param startSpacer the spacer to be included at the start of the stacks output
+     * @param elements     the elements to be included in the stack.
+     * @param startSpacer  the spacer to be included at the start of the stacks output
     * @param middleSpacer the spacer to be included between stack elements
-     * @param endSpacer the spacer to be included at the end of the stacks output
+     * @param endSpacer    the spacer to be included at the end of the stacks output
     * @return the new stack definition instance
     */
-    @Nonnull
-    MetaStackDefinition createDefinition(@Nonnull List<MetaStackElement> elements, @Nonnull String startSpacer, @Nonnull String middleSpacer, @Nonnull String endSpacer);
+    @NonNull MetaStackDefinition createDefinition(@NonNull List<MetaStackElement> elements, @NonNull String startSpacer, @NonNull String middleSpacer, @NonNull String endSpacer);

 }
@@ -25,10 +25,7 @@

 package me.lucko.luckperms.api.nodetype;

-import java.lang.reflect.ParameterizedType;
-import java.lang.reflect.Type;
-
-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * Marks an instance used as a key for a {@link NodeType}.
@@ -46,11 +43,9 @@ public interface NodeTypeKey<N extends NodeType> {
     *
     * @return the name of the represented type
     */
-    @Nonnull
-    default String getTypeName() {
-        ParameterizedType thisType = (ParameterizedType) getClass().getGenericSuperclass();
-        Type nodeType = thisType.getActualTypeArguments()[0];
-        return ((Class) nodeType).getSimpleName();
+    default @NonNull String getTypeName() {
+        String name = getClass().getName();
+        return name.substring(name.lastIndexOf('.') + 1, name.length() - 2);
    }

 }
@@ -29,7 +29,7 @@ import me.lucko.luckperms.api.Node;
 import me.lucko.luckperms.api.nodetype.NodeType;
 import me.lucko.luckperms.api.nodetype.NodeTypeKey;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * A sub-type of {@link Node} used to mark the display name of the node's holder.
@@ -48,7 +48,6 @@ public interface DisplayNameType extends NodeType {
     *
     * @return the display name
     */
-    @Nonnull
-    String getDisplayName();
+    @NonNull String getDisplayName();

 }
@@ -29,7 +29,7 @@ import me.lucko.luckperms.api.Node;
 import me.lucko.luckperms.api.nodetype.NodeType;
 import me.lucko.luckperms.api.nodetype.NodeTypeKey;

-import javax.annotation.Nonnull;
+import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * A sub-type of {@link Node} used to mark that the holder of the node should inherit
@@ -51,7 +51,6 @@ public interface InheritanceType extends NodeType {
     *
     * @return the name of the group
     */
-    @Nonnull
-    String getGroupName();
+    @NonNull String getGroupName();

 }
@@ -29,9 +29,9 @@ import me.lucko.luckperms.api.Node;
 import me.lucko.luckperms.api.nodetype.NodeType;
 import me.lucko.luckperms.api.nodetype.NodeTypeKey;

-import java.util.Map;
+import org.checkerframework.checker.nullness.qual.NonNull;

-import javax.annotation.Nonnull;
+import java.util.Map;

 /**
 * A sub-type of {@link Node} used to store meta assignments.
@@ -50,16 +50,14 @@ public interface MetaType extends NodeType, Map.Entry<String, String> {
     *
     * @return the meta key
     */
-    @Nonnull
-    String getKey();
+    @NonNull String getKey();

    /**
     * Gets the meta value.
     *
     * @return the meta value
     */
-    @Nonnull
-    String getValue();
+    @NonNull String getValue();

    @Override
    @Deprecated
@@ -29,9 +29,9 @@ import me.lucko.luckperms.api.Node;
 import me.lucko.luckperms.api.nodetype.NodeType;
 import me.lucko.luckperms.api.nodetype.NodeTypeKey;

-import java.util.Map;
+import org.checkerframework.checker.nullness.qual.NonNull;

-import javax.annotation.Nonnull;
+import java.util.Map;

 /**
 * A sub-type of {@link Node} used to store prefix assignments.
@@ -57,15 +57,13 @@ public interface PrefixType extends NodeType {
     *
     * @return the prefix
     */
-    @Nonnull
-    String getPrefix();
+    @NonNull String getPrefix();

    /**
     * Gets a representation of this instance as a {@link Map.Entry}.
     *
     * @return a map entry representation of the priority and prefix string
     */
-    @Nonnull
-    Map.Entry<Integer, String> getAsEntry();
+    Map.@NonNull Entry<Integer, String> getAsEntry();

 }
Author	SHA1	Message	Date
Luck	aa4139f0b7	Release v4.3.0	2018-09-23 15:04:15 +01:00
Luck	e2f1c1e16b	Remove specific dependency on Sponge API 8 in the mod info file (#1221 )	2018-09-23 15:02:02 +01:00
Luck	1e7ef15e54	Fix class cast exception in NodeTypeKey#getTypeName (#1230 )	2018-09-23 14:38:27 +01:00
Luck	1cc156d6f6	Fix buildscript deployment config for Sonatype	2018-09-23 14:30:09 +01:00
Imortalshard	1445a3faec	Fix typo in the description of /lp export (#1229 )	2018-09-23 11:28:48 +01:00
Luck	1cf90692cf	fix typo	2018-09-21 21:53:06 +01:00
Luck	5656bea3f9	Update for latest Velocity changes	2018-09-21 21:36:44 +01:00
Luck	8e344cea76	Cleanup imports	2018-09-20 21:17:28 +01:00
Luck	5b1229c882	Fix slf4j loading (#1224 )	2018-09-20 21:17:00 +01:00
Luck	834649b390	Improve Uuids#parse thanks @kashike	2018-09-20 15:30:22 +01:00
Luck	d28170cc3f	Operations to add new nodes with differing values should just override existing entries, instead of failing with the ALREADY_HAS result (#1198 )	2018-09-20 11:25:31 +01:00
Luck	04e511026d	Detach from permissible objects with a 1 tick delay after player quit to allow plugins listening on monitor to still access data (#1220 )	2018-09-20 11:07:20 +01:00
Luck	11a3ecbba0	Fix NPE when loading subject data on Sponge (#1222 )	2018-09-20 10:51:00 +01:00
Luck	51b7bb93e2	Don't trigger Vault primary thread checks whilst the server is starting up	2018-09-20 10:44:04 +01:00
Luck	9fd2028d9f	Update for velocity API changes	2018-09-19 21:47:22 +01:00
Luck	75f0f40d70	NonNull annotations should be on the same line in interfaces	2018-09-19 21:12:54 +01:00
Luck	3941c77826	Move from findbugs jsr305 to checker-qual	2018-09-19 20:40:25 +01:00
Luck	4987ca3ba2	Update gradle wrapper and buildscript plugins	2018-09-19 19:41:47 +01:00
Jesse Boyd	5fa481337a	Fix Nukkit permissible null handling - fixes #1191 (#1213 )	2018-09-12 08:58:55 +01:00
Luck	dd78e0e271	Separate formatted vs plain display name methods in User/Group Continues from `3c773e6823` & #1210	2018-09-10 16:54:16 +01:00
Max Lee	3c773e6823	Use neutral color to display potentially colored names - fixes #1170 (#1210 )	2018-09-10 16:41:14 +01:00
Luck	54a9ff9256	Fix compile	2018-09-09 11:31:17 +01:00
Luck	eca6b09b18	Refactor config keys	2018-09-05 20:49:37 +01:00
Luck	b7f8ca6658	Add 'require-sender-group-membership-to-modify' config option (#1140 )	2018-09-05 15:49:29 +01:00
Luck	ffdeadac8a	Add flag to prevent users from being added/removed to/from the first group on a track (#1151 )	2018-09-05 15:15:15 +01:00
Luck	9e769904bf	Expose TemporaryMergeBehaviour in the API (#1189 )	2018-09-05 11:11:45 +01:00
Luck	8d79ec6b07	Rename "update task" --> "sync task"	2018-09-04 21:11:03 +01:00
Luck	3a123f9fa1	Avoid running full "update tasks" unless they're absolutely needed. Process Vault API "set" requests immediately instead of in the background	2018-09-04 21:02:40 +01:00
Luck	f0c0328919	Refactor MetaAccumulator to be a little more sane	2018-09-04 20:33:22 +01:00
Luck	274286032a	Update banner in readme	2018-09-03 13:56:49 +01:00
Luck	76517374a2	Only allow import and export commands to interact with files in the data directory (#1193 )	2018-09-02 22:20:29 +01:00
Luck	2da027f3b5	Revert MySQL driver update	2018-09-02 20:04:26 +01:00
Luck	fd712f131f	Update some dependency versions	2018-09-02 16:36:28 +01:00
Luck	38613e5f03	Update gitignore	2018-08-31 20:27:13 +01:00
Luck	ec9060ba80	Update messenger log prefixes	2018-08-31 20:22:31 +01:00
Luck	502d57e744	Optimize event generation & other misc improvements	2018-08-27 22:11:03 +01:00
Luck	c59f2ab37a	Don't call NodeAddEvent when adding the default group (#1177 )	2018-08-27 22:06:49 +01:00
Luck	066189b486	Send a more informative message when the bulkupdate command is used from in-game	2018-08-27 18:25:57 +01:00
Luck	37d4f92c33	Update README	2018-08-27 09:37:41 +01:00
Luck	55220e9d10	Add support for Velocity	2018-08-26 22:24:46 +01:00
Luck	bd41286076	Fix build	2018-08-26 18:45:33 +01:00
Luck	d84484c81c	Optimize event generation and posting	2018-08-26 10:28:04 +01:00
Luck	fec87d4c6a	Update event to 3.0.0	2018-08-16 20:01:26 +01:00
Luck	68ec6e2a37	Include license file in the jar	2018-08-14 10:56:07 +01:00
Luck	a784ad435f	Fix applying group web editor changes (#1148 )	2018-08-13 22:44:10 +01:00
Luck	cc9bb34873	Update kyori/event dependency	2018-08-13 20:03:57 +01:00
Luck	2ddae5a96c	Avoid UnsupportedEncodingException in LilyPad messenger	2018-08-12 10:40:26 +01:00
Luck	40ad489c56	Update build instructions in the README file	2018-08-12 10:27:21 +01:00
Luck	56c55d71fe	Improve the way Vault lookups for offline players are handled (#1150 ) 1) The behaviour of online player lookups is unchanged - this only affects offline player data lookups. 2) If a 'offline player' request is made from an async task, it is executed normally. (the data is loaded) 3) If a 'offline player' request that cannot be immediately fulfilled is made from the main server thread, an exception is thrown. The exception thrown as a result of 3) can be disabled in the config. In this case, the query is performed anyway. The main server thread will block until the request is complete.	2018-08-11 19:02:57 +01:00
Luck	56df15caf8	Add bukkit-legacy build back to the main project	2018-08-11 18:11:30 +01:00
Luck	03720665b3	Move to gradle	2018-08-10 20:39:29 +01:00
Luck	445429b510	Update text & asm dependencies (fixes #1142 ) Updates for: https://github.com/KyoriPowered/text/commit/6a7fda4e80c48884ba957ffaadd9219cead7962d	2018-08-10 00:09:34 +01:00
Luck	c698587d53	Update text dependency	2018-08-09 22:43:41 +01:00
Luck	b8466beee5	Completely rework event system * Event instances are now generated dynamically at runtime. The performance hit of creating proxies at runtime is negligible in this case. * A better EventBus implementation is now being used internally, API contracts are unaffected.	2018-08-09 22:39:33 +01:00
Luck	efa666445f	Initial support for Brigadier (1.13 commands) on Bukkit servers	2018-08-08 23:42:25 +01:00
Luck	59c7d77525	Load a up-to-date copy of group data before applying web editor changes	2018-08-08 09:00:35 +01:00
Luck	553da9baf6	Give some form of feedback when trying to applyedits no changes instead of no feedback at all (#1136 )	2018-08-08 08:58:58 +01:00
Luck	486ddb3d42	Don't include any database properties by default (#1134 )	2018-08-05 21:59:48 +01:00
Luck	381922fc02	Don't schedule a delayed load task on startup (#1130 )	2018-08-05 21:51:53 +01:00
Luck	4c891f0819	Fix uppercase inconsistencies when saving users to SQL storage types (#1119 ) This was causing issues when user data was saved, without UUID data being saved beforehand. This commit fixes the way the inconsistency was introduced, and ensures that inconsistent data is made consistent in the future. (when players next login)	2018-07-30 11:22:50 -07:00
Luck	4b3d11ccab	More SQL escaping (#1091 & #1126 ) Follows from `c16630c32b`	2018-07-29 17:14:48 -07:00
Luck	b1ab465991	Misc cleanup	2018-07-29 17:12:05 -07:00
Luck	952e41ad3d	A few API javadoc improvements	2018-07-27 23:45:16 -07:00
Luck	ae3adf23c8	Fail silently when plugins use reflection to add to dummy permissibles (#1117 )	2018-07-24 21:21:05 -07:00
Luck	c3cdf387c2	Fix NPE in GroupManager migration (#1118 )	2018-07-24 21:03:14 -07:00
Luck	054f0c2c19	Don't add spaces when formatting 'concise' durations	2018-07-24 20:59:49 -07:00
Luck	2f7fce2109	Change name of Sql messenger table for compat with changes in the previous commit	2018-07-15 22:49:09 -07:00
Luck	07469599bc	Rewrite MessagingService message encoding, conform to new plugin message channel requirements * This change isn't backwards compatible with previous builds, this version (& onwards) won't be able to "communicate" with old versions, and vice versa. * MC 1.13 requires plugin message channel names to be namespaced, so 'lpuc' has been changed to 'luckperms:update' * The channel names for Redis/Lily types have also changed, I've been wanting to change the encoded format to something a bit saner for a while, and this seemed like a good time. Changing the channel names a) keeps things consistent and b) means I don't have to worry about old versions interpreting the new format.	2018-07-15 22:42:37 -07:00
Luck	08869da96e	misc cleanup	2018-07-13 17:51:30 -07:00
Luck	7540a6eb70	Send more appropriate message when no users/groups match the editor type (#1096 )	2018-07-13 17:51:21 -07:00
Luck	fa2d9357b8	Allow FileWatcher locations to be registered after initial start	2018-07-12 17:10:07 -07:00
Luck	247a40b65f	Fix /lp search for groups when using separated file storage (#1094 )	2018-07-12 16:35:32 -07:00
Luck	01ffe6ad5a	Use synchronized wrapper for NodeMap backings	2018-07-12 09:50:10 -07:00
Luck	da3c2a073a	cleanup build scripts a bit	2018-07-11 13:09:54 -07:00
Luck	c16630c32b	Escape groups keyword in SQL queries to (hopefully) fix compat with newer versions of MySQL (#1091 )	2018-07-11 13:09:20 -07:00
Luck	bf64f465a8	Re-implement Bukkit 1.7.10 support	2018-07-01 15:31:28 +01:00
Luck	c1e0b874d8	Add helpful message for users on first startup	2018-07-01 15:07:25 +01:00
Luck	c780266139	Add setprefix, setsuffix, settempprefix, settempsuffix commands (#840 )	2018-06-27 17:31:30 +01:00
Luck	08454d58d0	Refactor the way contexts are cached on all platforms (#1071 ) Should fix memory leak issues on BungeeCord	2018-06-27 00:17:53 +01:00
Luck	ee13540d78	Specify explicit global context in generated LP commands when default contexts have been set (#1062 )	2018-06-26 14:39:18 +01:00
Luck	b4ff54ffc8	Apply Sponge implicit wildcards after Bukkit style .* wildcards	2018-06-24 21:44:12 +01:00
Luck	6f96d9c7d1	Fix NPE on Nukkit load (#1059 )	2018-06-19 23:40:13 +01:00
Luck	5b66436519	Ensure the backing of LPSubscriptionValueMaps is a WeakHashMap (#1058 )	2018-06-19 23:35:19 +01:00
Luck	379eb00b69	Cleanup various message localisations, limit the size of the PermissionRegistry	2018-06-19 22:27:47 +01:00
Luck	2036127612	Cleanup action log formatting and pagination This is technically a breaking change to the API, but I'm fairly certain hardly anyone is using these methods (the pagination functionality in the API is pretty useless) Can always add them back if it becomes an issue	2018-06-18 16:08:18 +01:00
Luck	957a918bd6	Fix loading on BungeeCord (#1054 )	2018-06-17 09:49:38 +01:00
Luck	6b305972ac	Fix sponge build script	2018-06-16 18:41:53 +01:00
Luck	b8c06904ab	Make logging messages a bit less obnoxious * Removed the ascii text from the startup output * Log messages (but not command output) is now sent through the standard server logger, and isn't colored	2018-06-16 16:59:59 +01:00
Luck	6baa472567	Fix context cache invalidation	2018-06-15 19:07:24 +01:00
Luck	9c449def57	Update some dependency versions	2018-06-13 20:29:00 +01:00
Luck	e4acc133a9	remove some unnecessary interfaces	2018-06-12 18:18:38 +01:00
Luck	8b181b877b	Nukkit stores attachments in a set, not a list (#1046 )	2018-06-11 17:07:56 +01:00
Luck	194790110d	Fix node comparison for identical nodes except the value	2018-06-07 19:58:55 +01:00
Luck	c0cd197bd0	Don't allow players to be added to SubscriptionValueMaps	2018-06-07 17:01:25 +01:00
Luck	30ce269873	Disable hikari leak detection (#1033 )	2018-06-05 20:33:23 +01:00
Luck	4d7be13c16	Don't reload Sponge persisted subjects when a save is pending	2018-06-03 20:46:50 +01:00
Luck	8cf0f7da5f	Improve buffering code	2018-06-03 19:36:55 +01:00
Luck	bcb4e5ca64	cleanup various comparators	2018-05-31 23:41:37 +01:00
Luck	07b003c77a	Make the dummy attachment a bit more compatible still MassiveCore lol	2018-05-31 01:01:29 +01:00
Luck	f4e4f727dd	More workarounds to support MassiveCore's modification of attachment permissions via reflection	2018-05-31 00:51:33 +01:00
Luck	644c53a074	Add workaround for plugins adding/removing/modifying permission attachments via reflection (#1024 ) who knows why they're doing it, it's not even beneficial for performance...	2018-05-30 13:17:41 +01:00
Luck	c66622bd09	Fix regex permissions not really working properly (#1021 )	2018-05-29 14:45:21 +01:00
Luck	adf0589184	Print a more informative error when the port is missing from the address arg (#1022 )	2018-05-29 12:40:39 +01:00
Luck	b7de0e6901	Provide an even more detailed warning message when other uuids are present for a username We can use the data we have about the UUID and the platform LP is running on to infer more information about what the problem (potentially) is.	2018-05-25 17:05:26 +01:00
Luck	a90e59ec97	Fix cache invalidation when group data changes (#1010 )	2018-05-21 18:48:47 +01:00
Luck	f4882c1c14	Cache the creation of Sponge proxy subject classes (#1005 )	2018-05-17 18:19:45 +01:00
Luck	ed3b5a3cf1	Lazily load contexts cache for Sponge subjects (#1005 )	2018-05-17 18:12:20 +01:00
Luck	45e2b7b12f	Fix saving node attributes on non-YAML file storage types (#999 )	2018-05-16 19:30:05 +01:00
Luck	1e73986c60	Allow world rewrite options to apply in (some) Vault lookups (#990 )	2018-05-16 19:04:09 +01:00
Luck	20f992110d	Refactor meta stack element creation, add 'from_group' & 'not_from_group' elements	2018-05-16 18:46:25 +01:00
Luck	18f09f9862	Various performance optimizations	2018-05-15 00:08:23 +01:00
Luck	a0be1c7c48	Fix issue with Bukkit attachment permissions never being removed (#991 )	2018-05-13 14:06:23 +01:00
DoNotSpamPls	5d34661a15	Readme fixes (#993 )	2018-05-12 16:40:54 +01:00
Luck	1639879106	Reduce the number of config lookups required when calculating contexts	2018-05-12 01:51:38 +01:00
Luck	a927ca659f	Reduce the throughput on the contexts cache in ContextManager (#929 )	2018-05-12 01:34:34 +01:00
Luck	804c884d8b	Update configurate to fix issues with legacy guava compat	2018-05-06 19:57:50 +01:00
Luck	51f215b8ac	Add PlayerDataSaveEvent	2018-05-06 18:21:14 +01:00