Implement luckiness calculator

2021-04-01 06:45:02 +01:00
843 changed files with 7692 additions and 27007 deletions
@@ -0,0 +1,40 @@
+---
+name: 🐛  Bug report
+about: Report an issue or problem with the way LuckPerms is functioning.
+title: ''
+labels: 'type: issue'
+assignees: ''
+
+---
+
+<!-- Before going any further, please check to see if your problem is fixed by updating to a newer version. The latest development builds can be obtained from https://luckperms.net/download -->
+
+### Description
+<!-- Please provide a short description of the issue in the space below. -->
+
+
+### Reproduction steps
+<!-- Please provide a clear list of steps we should follow in order to reproduce your issue in the space below. -->
+
+1. Open the config.yml file and set example to true.
+2. Restart the server
+3. Run /lp user example permission set x from the console
+4. See error
+
+
+### Expected behaviour
+<!-- Please provide a description of what you expected to happen. -->
+
+
+### Environment details
+<!-- Please provide details of your server environment in the space below. -->
+
+* Server type/version: `ExampleSpigot` running version `1.12.2` build `???`
+* LuckPerms version: `v???`
+
+<!-- If you think it would be helpful, please also include a "pastebin" of any relevant config files or log output. You should use https://gist.github.com/ to upload, then include the link below. -->
+
+
+### Any other relevant details
+<!-- Please include any other relevant details in the space below. -->
+
@@ -1,63 +0,0 @@
-name: 🐛  Bug report
-description: Report an issue or problem with the way LuckPerms is functioning.
-labels: ["type: issue"]
-body:
-  - type: markdown
-    attributes:
-      value: "Before going any further, please check to see if your problem is fixed by updating to a newer version. The latest development builds can be obtained from https://luckperms.net/download"
-  - type: textarea
-    id: description
-    attributes:
-      label: Description
-      description: Please provide a short description of the issue in the space below.
-      placeholder: Description goes here!
-    validations:
-      required: true
-  - type: textarea
-    id: reproduction-steps
-    attributes:
-      label: Reproduction Steps
-      description: Please provide a clear list of steps we should follow in order to reproduce your issue in the space below.
-      placeholder: List reproduction steps here, please!
-    validations:
-      required: true
-  - type: textarea
-    id: expected-behaviour
-    attributes:
-      label: Expected Behaviour
-      description: Please provide a description of what you expected to happen.
-      placeholder: Describe what you expected to happen here.
-    validations:
-      required: true
-  - type: input
-    id: server-info
-    attributes:
-      label: Server Details
-      description: Please provide a description of the server details, including type and specific version number.
-      placeholder: 'git-Paper-124 (MC: 1.17.1)'
-    validations:
-      required: true
-  - type: input
-    id: luckperms-version
-    attributes:
-      label: LuckPerms Version
-      description: Please provide the specific, precise version number of LuckPerms you are using to reproduce this bug.
-      placeholder: v5.3.70
-    validations:
-      required: true
-  - type: textarea
-    id: logs
-    attributes:
-      label: Logs and Configs
-      description: If you think it would be helpful, please also include a pastebin of any relevant config files or log output. You should use https://gist.github.com/ to upload, then include the link below. 
-      placeholder: https://gist.github.com/HelpfulBugReporter/ThisIsAVeryUsefulLogOutput
-    validations:
-      required: false
-  - type: textarea
-    id: extra-info
-    attributes:
-      label: Extra Details
-      description: Please include any other relevant details in the space below.
-      placeholder: I was able to reproduce this only at the end of a rainbow.
-    validations:
-      required: false
@@ -0,0 +1,16 @@
+---
+name: 💡  Suggestion
+about: Suggest an idea for an improvement or change to LuckPerms.
+title: ''
+labels: 'type: suggestion'
+assignees: ''
+
+---
+
+### Description
+<!-- Please provide a short description of the suggestion in the space below. -->
+
+
+### Proposed behaviour
+<!-- Please provide a short explanation of how the feature should work / be changed, and how this will effect the project. -->
+
@@ -1,31 +0,0 @@
-name: 💡  Suggestion
-description: Suggest an idea for an improvement or change to LuckPerms.
-labels: ["type: suggestion"]
-body:
-  - type: markdown
-    attributes:
-      value: "Before going any further, please check to see if your suggestion has already been added by checking the [wiki](https://luckperms.net/wiki/Home). The latest development builds can be obtained from https://luckperms.net/download"
-  - type: textarea
-    id: description
-    attributes:
-      label: Description
-      description: Please provide a short description of your suggestion in the space below.
-      placeholder: Hello, perhaps you should...
-    validations:
-      required: true
-  - type: textarea
-    id: proposed-behaviour
-    attributes:
-      label: Proposed Behaviour
-      description: Please provide a short explanation of how the feature should work / be changed, and how this will affect the project.
-      placeholder: It would...
-    validations:
-      required: true
-  - type: textarea
-    id: extra-info
-    attributes:
-      label: Extra Details
-      description: Please include any other relevant details in the space below.
-      placeholder: I think this should only function at the end of a rainbow.
-    validations:
-      required: false
@@ -1,127 +0,0 @@
-name: Build Gradle and Publish Docker image
-
-on:
-  push:
-    branches:
-      - 'master'
-    tags:
-      - 'v*'
-  pull_request:
-    branches:
-      - 'master'
-
-env:
-  REGISTRY: ghcr.io
-  IMAGE_NAME: ${{ github.repository }}
-
-jobs:
-  build-gradle:
-    runs-on: ubuntu-latest
-    steps:
-      - name: Checkout repository
-        uses: actions/checkout@v3
-        with:
-          fetch-depth: 0
-
-      - name: validate gradle wrapper
-        uses: gradle/wrapper-validation-action@v1
-
-      - uses: actions/setup-java@v3
-        with:
-          distribution: 'temurin'
-          java-version: '17'
-
-      - name: Setup Gradle
-        uses: gradle/gradle-build-action@v2
-
-      - name: Run build and tests with Gradle wrapper
-        run: ./gradlew test build -PdockerTests
-
-      - name: Publish test report
-        uses: mikepenz/action-junit-report@v3
-        if: success() || failure()
-        with:
-          report_paths: '**/build/test-results/test/TEST-*.xml'
-          annotate_notice: true
-          detailed_summary: true
-
-      - name: Upload all artifacts
-        uses: actions/upload-artifact@v3
-        with:
-          name: jars
-          path: |
-            bukkit/loader/build/libs/LuckPerms-Bukkit-*.jar
-            bukkit-legacy/loader/build/libs/LuckPerms-Bukkit-Legacy-*.jar
-            bungee/loader/build/libs/LuckPerms-Bungee-*.jar
-            sponge/loader/build/libs/LuckPerms-Sponge-*.jar
-            nukkit/loader/build/libs/LuckPerms-Nukkit-*.jar
-            velocity/build/libs/LuckPerms-Velocity-*.jar
-            fabric/build/libs/LuckPerms-Fabric-*.jar
-            forge/loader/build/libs/LuckPerms-Forge-*.jar
-            standalone/loader/build/libs/LuckPerms-Standalone-*.jar
-
-      - name: Upload standalone artifact
-        uses: actions/upload-artifact@v3
-        with:
-          name: standalone-binary
-          path: standalone/loader/build/libs/LuckPerms-Standalone-*.jar
-
-
-  build-docker:
-    needs: build-gradle
-    runs-on: ubuntu-latest
-    permissions:
-      contents: read
-      packages: write
-
-    steps:
-      - name: Checkout repository
-        uses: actions/checkout@v3
-        with:
-          fetch-depth: 0
-
-      - name: Retrieve saved standalone jar artifact
-        uses: actions/download-artifact@v3
-        with:
-          name: standalone-binary
-          path: standalone/docker/
-
-      - name: Set up QEMU
-        uses: docker/setup-qemu-action@v1
-
-      - name: Set up Docker Buildx
-        uses: docker/setup-buildx-action@v1
-
-      - name: Log in to the Container registry
-        uses: docker/login-action@v1
-        with:
-          registry: ${{ env.REGISTRY }}
-          username: ${{ github.actor }}
-          password: ${{ secrets.GITHUB_TOKEN }}
-
-      - name: Determine the LuckPerms version
-        run: |
-          echo "luckperms_version=$(git describe --tags | awk -F "-" '{print $1 "." $2}')" >> "$GITHUB_ENV"
-
-      - name: Extract metadata for Docker
-        id: meta
-        uses: docker/metadata-action@v3
-        with:
-          images: ${{ env.REGISTRY }}/${{ env.IMAGE_NAME }}
-          flavor: |
-            latest=${{ github.ref == 'refs/heads/master' }}
-          tags: |
-            type=schedule
-            type=ref,event=branch
-            type=ref,event=tag
-            type=ref,event=pr
-            type=raw,enable=${{ github.ref == 'refs/heads/master' }},value=${{ env.luckperms_version }}
-
-      - name: Build and push Docker image
-        uses: docker/build-push-action@v2
-        with:
-          context: standalone/docker/
-          platforms: linux/amd64,linux/arm64
-          push: ${{ github.event_name != 'pull_request' }}
-          tags: ${{ steps.meta.outputs.tags }}
-          labels: ${{ steps.meta.outputs.labels }}
@@ -1,6 +1,3 @@
-### LuckPerms Standalone ###
-standalone/loader/data/
-
 ### Gradle & IntelliJ ###
 .gradle/
 /.idea/
@@ -8,7 +8,7 @@ Before reporting a bug or issue, please make sure that the issue is actually bei

 If you're unsure, feel free to ask using the above resources BEFORE making a report.

-Bugs or issues should be reported using the [GitHub Issues tab](https://github.com/LuckPerms/LuckPerms/issues).
+Bugs or issues should be reported using the [GitHub Issues tab](https://github.com/lucko/LuckPerms/issues).

 ### :pencil: Want to contribute code?
 #### Pull Requests
@@ -23,4 +23,4 @@ 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, Fabric, Forge, Nukkit, Sponge & Velocity** - Each use the common module to implement plugins on the respective server platforms.
+* **Bukkit, BungeeCord, Sponge, Nukkit, Velocity & Fabric** - Each use the common module to implement plugins on the respective server platforms.
@@ -1,4 +1,4 @@
-![](https://raw.githubusercontent.com/LuckPerms/branding/master/banner/banner.png "Banner")
+![](https://i.imgur.com/ToguFkQ.png "Banner")
 # LuckPerms
 [![Build Status](https://ci.lucko.me/job/LuckPerms/badge/icon)](https://ci.lucko.me/job/LuckPerms/)
 [![javadoc](https://javadoc.io/badge2/net.luckperms/api/javadoc.svg)](https://javadoc.io/doc/net.luckperms/api)
@@ -24,24 +24,18 @@ For more information, see the wiki article on [Why LuckPerms?](https://luckperms
 LuckPerms uses Gradle to handle dependencies & building.

 #### Requirements
-* Java 17 JDK or newer
+* Java 8 JDK or newer
 * Git

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

 You can find the output jars in the `loader/build/libs` or `build/libs` directories.

-## Tests
-There are some automated tests which run during each build.
-
-* Unit tests are defined in [`common/src/test`](https://github.com/LuckPerms/LuckPerms/tree/master/common/src/test)
-* Integration tests are defined in [`standalone/src/test`](https://github.com/LuckPerms/LuckPerms/tree/master/standalone/src/test).
-
 ## Contributing
 #### Pull Requests
 If you make any changes or improvements to the plugin which you think would be beneficial to others, please consider making a pull request to merge your changes back into the upstream project. (especially if your changes are bug fixes!)
@@ -53,7 +47,7 @@ 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, Fabric, Forge, Nukkit, Sponge & Velocity** - Each use the common module to implement plugins on the respective server platforms.
+* **Bukkit, BungeeCord, Sponge, Fabric, 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/LuckPerms/LuckPerms/blob/master/LICENSE.txt) for more info.
+LuckPerms is licensed under the permissive MIT license. Please see [`LICENSE.txt`](https://github.com/lucko/LuckPerms/blob/master/LICENSE.txt) for more info.
@@ -1,15 +1,9 @@
 group = 'net.luckperms'
-project.version = '5.4'
-
-jar {
-    manifest {
-        attributes('Automatic-Module-Name': 'net.luckperms.api')
-    }
-}
+project.version = '5.3'

 dependencies {
-    compileOnly 'org.checkerframework:checker-qual:3.21.2'
-    compileOnly 'org.jetbrains:annotations:23.1.0'
+    compileOnly 'org.checkerframework:checker-qual:3.8.0'
+    compileOnly 'org.jetbrains:annotations:20.1.0'
 }

 // Only used occasionally for deployment - not needed for normal builds.
@@ -23,17 +17,11 @@ if (project.hasProperty('sonatypeUsername') && project.hasProperty('sonatypePass
        options.encoding = 'UTF-8'
        options.charSet = 'UTF-8'
        options.links(
+                'https://docs.oracle.com/javase/8/docs/api/',
                'https://checkerframework.org/api/',
-                'https://javadoc.io/static/org.jetbrains/annotations/23.0.0/'
+                'https://javadoc.io/static/org.jetbrains/annotations/20.1.0/'
        )
        options.addStringOption('Xdoclint:none', '-quiet')
-        options.addStringOption('-since', '5.0,5.1,5.2,5.3,5.4')
-
-        if (JavaVersion.current() > JavaVersion.VERSION_1_8) {
-            options.links.add('https://docs.oracle.com/en/java/javase/11/docs/api/')
-        } else {
-            options.links.add('https://docs.oracle.com/javase/8/docs/api/')
-        }
    }

    task javadocJar(type: Jar, dependsOn: javadoc) {
@@ -51,69 +39,65 @@ if (project.hasProperty('sonatypeUsername') && project.hasProperty('sonatypePass
        archives sourcesJar
    }

-    publishing {
-        repositories {
-            maven {
-                def releasesRepoUrl = 'https://oss.sonatype.org/service/local/staging/deploy/maven2/'
-                def snapshotsRepoUrl = 'https://oss.sonatype.org/content/repositories/snapshots/'
-                url = version.endsWith('SNAPSHOT') ? snapshotsRepoUrl : releasesRepoUrl
-
-                credentials {
-                    username sonatypeUsername
-                    password sonatypePassword
-                }
-            }
-        }
-
-        publications {
-            mavenJava(MavenPublication) {
-                artifactId = 'api'
-
-                from components.java
-                artifact sourcesJar
-                artifact javadocJar
-
-                pom {
-                    name = 'LuckPerms API'
-                    description = 'A permissions plugin for Minecraft servers.'
-                    url = 'https://luckperms.net'
-
-                    licenses {
-                        license {
-                            name = 'MIT'
-                            url = 'https://opensource.org/licenses/MIT'
-                        }
-                    }
-
-                    developers {
-                        developer {
-                            id = 'lucko'
-                            name = 'Luck'
-                            url = 'https://lucko.me'
-                            email = 'git@lucko.me'
-                        }
-                    }
-
-                    scm {
-                        connection = 'scm:git:https://github.com/LuckPerms/LuckPerms.git'
-                        developerConnection = 'scm:git:git@github.com:LuckPerms/LuckPerms.git'
-                        url = 'https://github.com/LuckPerms/LuckPerms'
-                    }
-
-                    issueManagement {
-                        system = 'GitHub'
-                        url = 'https://github.com/LuckPerms/LuckPerms/issues'
-                    }
-                }
-            }
-        }
-    }
-
    signing {
        useGpgCmd()
        sign configurations.archives
-        sign publishing.publications.mavenJava
-        required = true
+    }
+
+    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)
+                }
+
+                snapshotRepository(url: 'https://oss.sonatype.org/content/repositories/snapshots/') {
+                    authentication(userName: sonatypeUsername, password: sonatypePassword)
+                }
+
+                pom {
+                    project {
+                        name 'LuckPerms API'
+                        description 'A permissions plugin for Minecraft servers.'
+                        url 'https://luckperms.net'
+
+                        licenses {
+                            license {
+                                name 'MIT'
+                                url 'https://opensource.org/licenses/MIT'
+                            }
+                        }
+
+                        developers {
+                            developer {
+                                id 'lucko'
+                                name 'Luck'
+                                url 'https://lucko.me'
+                                email 'git@lucko.me'
+                            }
+                        }
+
+                        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'
+                        }
+                    }
+                }
+            }
+        }
    }
 }
 */
@@ -8,11 +8,10 @@
    <li><a target="_top" href="https://luckperms.net">Project Website</a></li>
    <li><a target="_top" href="https://luckperms.net/wiki/Developer-API">Wiki - API Introduction</a></li>
    <li><a target="_top" href="https://luckperms.net/wiki/Developer-API-Usage">Wiki - API Usage</a></li>
-    <li><a target="_top" href="https://github.com/LuckPerms/LuckPerms">Source Code</a></li>
 </ul>
 <h4>Maven</h4>
 <p>
-    The API artifact is deployed to Maven Central with
+    The API artifact is deployed to Maven Central under
    <a target="_top" href="https://search.maven.org/artifact/net.luckperms/api">
        group id: <b><code>net.luckperms</code></b>, artifact id: <b><code>api</code></b>.
    </a>
@@ -46,6 +46,7 @@ import net.luckperms.api.platform.PluginMetadata;
 import net.luckperms.api.query.QueryOptionsRegistry;
 import net.luckperms.api.track.Track;
 import net.luckperms.api.track.TrackManager;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.jetbrains.annotations.ApiStatus.Internal;

@@ -48,7 +48,6 @@ public final class LuckPermsProvider {
     * @throws IllegalStateException if the API is not loaded yet
     */
    public static @NonNull LuckPerms get() {
-        LuckPerms instance = LuckPermsProvider.instance;
        if (instance == null) {
            throw new NotLoadedException();
        }
@@ -77,9 +76,9 @@ public final class LuckPermsProvider {
        private static final String MESSAGE = "The LuckPerms API isn't loaded yet!\n" +
                "This could be because:\n" +
                "  a) the LuckPerms plugin is not installed or it failed to enable\n" +
-                "  b) the plugin in the stacktrace does not declare a dependency on LuckPerms\n" +
-                "  c) the plugin in the stacktrace is retrieving the API before the plugin 'enable' phase\n" +
-                "     (call the #get method in onEnable, not the constructor!)\n";
+                "  b) your plugin does not declare a dependency on LuckPerms\n" +
+                "  c) you are attempting to use the API before plugins reach the 'enable' phase\n" +
+                "     (call the #get method in onEnable, not in your plugin constructor!)\n";

        NotLoadedException() {
            super(MESSAGE);
@@ -26,6 +26,7 @@
 package net.luckperms.api.actionlog;

 import net.luckperms.api.LuckPermsProvider;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.checkerframework.checker.nullness.qual.Nullable;

@@ -26,6 +26,7 @@
 package net.luckperms.api.actionlog;

 import net.luckperms.api.messaging.MessagingService;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 import java.util.concurrent.CompletableFuture;
@@ -26,14 +26,11 @@
 package net.luckperms.api.cacheddata;

 import net.luckperms.api.query.QueryOptions;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**
 * Holds cached lookup data for a given set of query options.
- *
- * <p>All calls will account for inheritance, as well as any default data
- * provided by the platform. These calls are heavily cached and are therefore
- * fast.</p>
 */
 public interface CachedData {

@@ -30,6 +30,7 @@ import net.luckperms.api.model.PermissionHolder;
 import net.luckperms.api.model.group.Group;
 import net.luckperms.api.model.user.User;
 import net.luckperms.api.query.QueryOptions;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 import java.util.concurrent.CompletableFuture;
@@ -38,7 +39,7 @@ import java.util.concurrent.CompletableFuture;
 * Holds cached permission and meta lookup data for a {@link PermissionHolder}.
 *
 * <p>All calls will account for inheritance, as well as any default data
- * provided by the platform. These calls are heavily cached and are therefore
+ * provided by the platform. This calls are heavily cached and are therefore
 * fast.</p>
 */
 public interface CachedDataManager {
@@ -80,10 +81,11 @@ public interface CachedDataManager {
     * <p>For {@link User}s, the most appropriate query options will be their
     * {@link ContextManager#getQueryOptions(User) current active query options} if the
     * corresponding player is online, and otherwise, will fallback to
-     * {@link ContextManager#getStaticQueryOptions() the current static query options}.</p>
+     * {@link ContextManager#getStaticQueryOptions() the current static query options}
+     * if they are offline.</p>
     *
     * <p>For {@link Group}s, the most appropriate query options will always be
-     * {@link ContextManager#getStaticQueryOptions() the current static query options}.</p>
+     * {@link ContextManager#getStaticQueryOptions()} the current static query options.</p>
     *
     * @return a permission data instance
     * @since 5.1
@@ -97,10 +99,11 @@ public interface CachedDataManager {
     * <p>For {@link User}s, the most appropriate query options will be their
     * {@link ContextManager#getQueryOptions(User) current active query options} if the
     * corresponding player is online, and otherwise, will fallback to
-     * {@link ContextManager#getStaticQueryOptions() the current static query options}.</p>
+     * {@link ContextManager#getStaticQueryOptions() the current static query options}
+     * if they are offline.</p>
     *
     * <p>For {@link Group}s, the most appropriate query options will always be
-     * {@link ContextManager#getStaticQueryOptions() the current static query options}.</p>
+     * {@link ContextManager#getStaticQueryOptions()} the current static query options.</p>
     *
     * @return a meta data instance
     * @since 5.1
@@ -115,7 +118,7 @@ public interface CachedDataManager {
    void invalidate();

    /**
-     * Invalidates all underlying permission calculators.
+     * Invalidates all of the underlying Permission calculators.
     *
     * <p>Can be called to allow for an update in defaults.</p>
     */
@@ -26,10 +26,7 @@
 package net.luckperms.api.cacheddata;

 import net.luckperms.api.metastacking.MetaStackDefinition;
-import net.luckperms.api.node.types.MetaNode;
-import net.luckperms.api.node.types.PrefixNode;
-import net.luckperms.api.node.types.SuffixNode;
-import net.luckperms.api.node.types.WeightNode;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.checkerframework.checker.nullness.qual.Nullable;
 import org.jetbrains.annotations.Unmodifiable;
@@ -42,40 +39,16 @@ import java.util.function.Function;

 /**
 * Holds cached meta lookup data for a specific set of contexts.
- * 
- * <p>Meta data refers to {@link PrefixNode prefixes}, {@link SuffixNode suffixes} and
- * {@link MetaNode meta (options)} held by a permission holder.</p>
- *
- * <p>All calls will account for inheritance, as well as any default data
- * provided by the platform. These calls are heavily cached and are therefore
- * fast.</p>
 */
 public interface CachedMetaData extends CachedData {

-    /**
-     * Query a meta value for the given {@code key}.
-     * 
-     * <p>This method will always return a {@link Result}, but the
-     * {@link Result#result() inner result} {@link String} will be null if a value
-     * for the given key was not found.</p>
-     *
-     * @param key the key
-     * @return a result containing the value
-     * @since 5.4
-     */
-    @NonNull Result<String, MetaNode> queryMetaValue(@NonNull String key);
-
    /**
     * Gets a value for the given meta key.
-     * 
-     * <p>If no such meta value exists for the given key, {@code null} is returned.</p>
     *
     * @param key the key
     * @return the value
     */
-    default @Nullable String getMetaValue(@NonNull String key) {
-        return queryMetaValue(key).result();
-    }
+    @Nullable String getMetaValue(@NonNull String key);

    /**
     * Gets a value for the given meta key, and runs it through the given {@code transformer}.
@@ -108,121 +81,37 @@ public interface CachedMetaData extends CachedData {
    }

    /**
-     * Query for a prefix.
-     * 
-     * <p>This method uses the rules defined by the {@link #getPrefixStackDefinition() prefix stack}
-     * to produce a {@link String} output.</p>
-     * 
-     * <p>Assuming the default configuration is used, this will usually be the value of the
-     * holder's highest priority prefix node.</p>
-     * 
-     * <p>This method will always return a {@link Result}, but the
-     * {@link Result#result() inner result} {@link String} will be null if
-     * a the resultant prefix stack contained no elements.</p>
-     *
-     * @return a result containing the prefix
-     * @since 5.4
-     */
-    @NonNull Result<String, PrefixNode> queryPrefix();
-
-    /**
-     * Gets the prefix.
-     * 
-     * <p>This method uses the rules defined by the {@link #getPrefixStackDefinition() prefix stack}
-     * to produce a {@link String} output.</p>
-     * 
-     * <p>Assuming the default configuration is used, this will usually be the value of the
-     * holder's highest priority prefix node.</p>
-     * 
-     * <p>If the resultant prefix stack contained no elements, {@code null} is returned.</p>
+     * Gets the holder's highest priority prefix, or null if the holder has no prefixes
     *
     * @return a prefix string, or null
     */
-    default @Nullable String getPrefix() {
-        return queryPrefix().result();
-    }
+    @Nullable String getPrefix();

    /**
-     * Query for a suffix.
-     *
-     * <p>This method uses the rules defined by the {@link #getSuffixStackDefinition() suffix stack}
-     * to produce a {@link String} output.</p>
-     *
-     * <p>Assuming the default configuration is used, this will usually be the value of the
-     * holder's highest priority suffix node.</p>
-     *
-     * <p>This method will always return a {@link Result}, but the
-     * {@link Result#result() inner result} {@link String} will be null if
-     * a the resultant suffix stack contained no elements.</p>
-     *
-     * @return a result containing the suffix
-     * @since 5.4
-     */
-    @NonNull Result<String, SuffixNode> querySuffix();
-
-    /**
-     * Gets the suffix.
-     *
-     * <p>This method uses the rules defined by the {@link #getSuffixStackDefinition() suffix stack}
-     * to produce a {@link String} output.</p>
-     *
-     * <p>Assuming the default configuration is used, this will usually be the value of the
-     * holder's highest priority suffix node.</p>
-     *
-     * <p>If the resultant suffix stack contained no elements, {@code null} is returned.</p>
+     * Gets the holder's highest priority suffix, or null if the holder has no suffixes
     *
     * @return a suffix string, or null
     */
-    default @Nullable String getSuffix() {
-        return querySuffix().result();
-    }
+    @Nullable String getSuffix();

    /**
-     * Query for a weight.
+     * Gets an immutable copy of the meta this holder has.
     *
-     * <p>This method will always return a {@link Result}, and the
-     * {@link Result#result() inner result} {@link Integer} will never be null.
-     * A value of {@code 0} is equivalent to null.</p>
-     *
-     * @return a result containing the weight
-     * @since 5.5
-     */
-    @NonNull Result<Integer, WeightNode> queryWeight();
-
-    /**
-     * Gets the weight.
-     *
-     * <p>If the there is no defined weight, {@code 0} is returned.</p>
-     *
-     * @return the weight
-     * @since 5.5
-     */
-    default int getWeight() {
-        return queryWeight().result();
-    }
-
-    /**
-     * Gets a map of all accumulated {@link MetaNode meta}.
-     *
-     * <p>Prefer using the {@link #getMetaValue(String)} method for querying values.</p>
-     *
-     * @return a map of meta
+     * @return an immutable map of meta
     */
    @NonNull @Unmodifiable Map<String, List<String>> getMeta();

    /**
-     * Gets a sorted map of all accumulated {@link PrefixNode prefixes}.
-     *
-     * <p>Prefer using the {@link #getPrefix()} method for querying.</p>
+     * Gets an immutable sorted map of all of the prefixes the holder has, whereby the first
+     * value is the highest priority prefix.
     *
     * @return a sorted map of prefixes
     */
    @NonNull @Unmodifiable SortedMap<Integer, String> getPrefixes();

    /**
-     * Gets a sorted map of all accumulated {@link SuffixNode suffixes}.
-     *
-     * <p>Prefer using the {@link #getSuffix()} method for querying.</p>
+     * Gets an immutable sorted map of all of the suffixes the holder has, whereby the first
+     * value is the highest priority suffix.
     *
     * @return a sorted map of suffixes
     */
@@ -239,14 +128,14 @@ public interface CachedMetaData extends CachedData {
    @Nullable String getPrimaryGroup();

    /**
-     * Gets the definition used for the prefix stack.
+     * Gets the definition used for the prefix stack
     *
     * @return the definition used for the prefix stack
     */
    @NonNull MetaStackDefinition getPrefixStackDefinition();

    /**
-     * Gets the definition used for the suffix stack.
+     * Gets the definition used for the suffix stack
     *
     * @return the definition used for the suffix stack
     */
@@ -25,8 +25,8 @@

 package net.luckperms.api.cacheddata;

-import net.luckperms.api.node.Node;
 import net.luckperms.api.util.Tristate;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.jetbrains.annotations.Unmodifiable;

@@ -34,39 +34,17 @@ import java.util.Map;

 /**
 * Holds cached permission lookup data for a specific set of contexts.
- *
- * <p>All calls will account for inheritance, as well as any default data
- * provided by the platform. These calls are heavily cached and are therefore
- * fast.</p>
 */
 public interface CachedPermissionData extends CachedData {

    /**
-     * Performs a permission check for the given {@code permission} node.
-     *
-     * <p>This check is equivalent to the "hasPermission" method call on most platforms.
-     * You can use {@link Tristate#asBoolean()} if you need a truthy result.</p>
-     *
-     * @param permission the permission node
-     * @return a result containing the tristate
-     * @throws NullPointerException if permission is null
-     * @since 5.4
-     */
-    @NonNull Result<Tristate, Node> queryPermission(@NonNull String permission);
-
-    /**
-     * Performs a permission check for the given {@code permission} node.
-     *
-     * <p>This check is equivalent to the "hasPermission" method call on most platforms.
-     * You can use {@link Tristate#asBoolean()} if you need a truthy result.</p>
+     * Gets a permission check result for the given permission node.
     *
     * @param permission the permission node
     * @return a tristate result
     * @throws NullPointerException if permission is null
     */
-    default @NonNull Tristate checkPermission(@NonNull String permission) {
-        return queryPermission(permission).result();
-    }
+    @NonNull Tristate checkPermission(@NonNull String permission);

    /**
     * Invalidates the underlying permission calculator cache.
@@ -1,81 +0,0 @@
-/*
- * 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 net.luckperms.api.cacheddata;
-
-import net.luckperms.api.node.Node;
-import org.checkerframework.checker.nullness.qual.Nullable;
-
-/**
- * Represents the result of a cached data lookup.
- *
- * <p>You can find "the holder that has the node that caused this result"
- * using the following code:</p>
- * <p></p>
- * <blockquote>
- * <pre>
- * public static {@link net.luckperms.api.model.PermissionHolder.Identifier} holderThatHasTheNodeThatCausedTheResult(Result&lt;?, ?&gt; result) {
- *     {@link Node} node = result.node();
- *     if (node == null) {
- *         return null;
- *     }
- *     {@link net.luckperms.api.node.metadata.types.InheritanceOriginMetadata} origin = node.getMetadata(InheritanceOriginMetadata.KEY).orElse(null);
- *     if (origin == null) {
- *         return null;
- *     }
- *     return origin.getOrigin();
- * }
- * </pre>
- *
- * <p>Combined with the node itself, this is all the information needed to determine
- * the root cause of the result.</p>
- * </blockquote>
- *
- * <p>The nullability of {@link #result()} is purposely undefined to allow the
- * flexibility for methods using {@link Result} to declare it. In general, if the {@code T} type
- * has a nullable/undefined value, then the return of {@link #result()} will be non-null,
- * and if not, it will be nullable.</p>
- *
- * @param <T> the result type
- * @param <N> the node type
- * @since 5.4
- */
-public interface Result<T, N extends Node> {
-
-    /**
-     * Gets the underlying result.
-     *
-     * @return the underlying result
-     */
-    T result();
-
-    /**
-     * Gets the node that caused the result.
-     *
-     * @return the causing node
-     */
-    @Nullable N node();
-
-}
@@ -24,7 +24,7 @@
 */

 /**
- * Caches permission checks and meta lookups for {@link net.luckperms.api.model.user.User}s and
+ * CachedData lookup API for {@link net.luckperms.api.model.user.User}s and
 * {@link net.luckperms.api.model.group.Group}s.
 */
 package net.luckperms.api.cacheddata;
@@ -28,6 +28,7 @@ package net.luckperms.api.context;
 import net.luckperms.api.model.user.User;
 import net.luckperms.api.query.QueryMode;
 import net.luckperms.api.query.QueryOptions;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.jetbrains.annotations.ApiStatus.Internal;

@@ -26,6 +26,7 @@
 package net.luckperms.api.context;

 import net.luckperms.api.LuckPermsProvider;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 import java.util.Objects;
@@ -118,7 +119,7 @@ public interface ImmutableContextSet extends ContextSet {
        }

        /**
-         * Adds all the contexts in another {@link ContextSet} to the set.
+         * Adds of of the contexts in another {@link ContextSet} to the set.
         *
         * @param contextSet the set to add from
         * @return the builder
@@ -26,6 +26,7 @@
 package net.luckperms.api.context;

 import net.luckperms.api.LuckPermsProvider;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 import java.util.Objects;
@@ -93,7 +94,7 @@ public interface MutableContextSet extends ContextSet {
    }

    /**
-     * Adds all the contexts in another {@link ContextSet} to this set.
+     * Adds of of the contexts in another {@link ContextSet} to this set.
     *
     * @param contextSet the set to add from
     * @throws NullPointerException if the contextSet is null
@@ -26,6 +26,7 @@
 package net.luckperms.api.event;

 import net.luckperms.api.LuckPerms;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**
@@ -28,6 +28,7 @@ package net.luckperms.api.event.context;
 import net.luckperms.api.context.ContextManager;
 import net.luckperms.api.event.LuckPermsEvent;
 import net.luckperms.api.event.util.Param;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 import java.util.Optional;
@@ -28,6 +28,7 @@ package net.luckperms.api.event.extension;
 import net.luckperms.api.event.LuckPermsEvent;
 import net.luckperms.api.event.util.Param;
 import net.luckperms.api.extension.Extension;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**
@@ -29,6 +29,7 @@ import net.luckperms.api.cacheddata.CachedDataManager;
 import net.luckperms.api.event.LuckPermsEvent;
 import net.luckperms.api.event.util.Param;
 import net.luckperms.api.model.group.Group;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**
@@ -29,6 +29,7 @@ import net.luckperms.api.event.LuckPermsEvent;
 import net.luckperms.api.event.cause.CreationCause;
 import net.luckperms.api.event.util.Param;
 import net.luckperms.api.model.group.Group;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**
@@ -29,6 +29,7 @@ import net.luckperms.api.cacheddata.CachedDataManager;
 import net.luckperms.api.event.LuckPermsEvent;
 import net.luckperms.api.event.util.Param;
 import net.luckperms.api.model.group.Group;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**
@@ -29,6 +29,7 @@ import net.luckperms.api.event.LuckPermsEvent;
 import net.luckperms.api.event.cause.DeletionCause;
 import net.luckperms.api.event.util.Param;
 import net.luckperms.api.node.Node;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 import java.util.Set;
@@ -28,6 +28,7 @@ package net.luckperms.api.event.group;
 import net.luckperms.api.event.LuckPermsEvent;
 import net.luckperms.api.event.util.Param;
 import net.luckperms.api.model.group.Group;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**
@@ -29,6 +29,7 @@ import net.luckperms.api.actionlog.Action;
 import net.luckperms.api.event.LuckPermsEvent;
 import net.luckperms.api.event.type.Cancellable;
 import net.luckperms.api.event.util.Param;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**
@@ -29,6 +29,7 @@ import net.luckperms.api.actionlog.Action;
 import net.luckperms.api.event.LuckPermsEvent;
 import net.luckperms.api.event.type.Cancellable;
 import net.luckperms.api.event.util.Param;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 import java.util.UUID;
@@ -30,6 +30,7 @@ import net.luckperms.api.event.LuckPermsEvent;
 import net.luckperms.api.event.type.Cancellable;
 import net.luckperms.api.event.util.Param;
 import net.luckperms.api.platform.PlatformEntity;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**
@@ -29,6 +29,7 @@ import net.luckperms.api.actionlog.Action;
 import net.luckperms.api.event.LuckPermsEvent;
 import net.luckperms.api.event.type.Cancellable;
 import net.luckperms.api.event.util.Param;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**
@@ -28,6 +28,7 @@ package net.luckperms.api.event.log;
 import net.luckperms.api.actionlog.Action;
 import net.luckperms.api.event.LuckPermsEvent;
 import net.luckperms.api.event.util.Param;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 import java.util.UUID;
@@ -27,6 +27,7 @@ package net.luckperms.api.event.node;

 import net.luckperms.api.event.util.Param;
 import net.luckperms.api.node.Node;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.jetbrains.annotations.Unmodifiable;

@@ -27,6 +27,7 @@ package net.luckperms.api.event.node;

 import net.luckperms.api.event.util.Param;
 import net.luckperms.api.node.Node;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.jetbrains.annotations.Unmodifiable;

@@ -32,6 +32,7 @@ import net.luckperms.api.model.data.DataType;
 import net.luckperms.api.model.group.Group;
 import net.luckperms.api.model.user.User;
 import net.luckperms.api.node.Node;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.jetbrains.annotations.Unmodifiable;

@@ -27,6 +27,7 @@ package net.luckperms.api.event.node;

 import net.luckperms.api.event.util.Param;
 import net.luckperms.api.node.Node;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.jetbrains.annotations.Unmodifiable;

@@ -29,6 +29,7 @@ import net.luckperms.api.event.LuckPermsEvent;
 import net.luckperms.api.event.util.Param;
 import net.luckperms.api.model.PlayerSaveResult;
 import net.luckperms.api.model.user.UserManager;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 import java.util.UUID;
@@ -29,6 +29,7 @@ import net.luckperms.api.event.LuckPermsEvent;
 import net.luckperms.api.event.util.Param;
 import net.luckperms.api.model.user.User;
 import net.luckperms.api.util.Result;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.checkerframework.checker.nullness.qual.Nullable;

@@ -28,6 +28,7 @@ package net.luckperms.api.event.player.lookup;
 import net.luckperms.api.event.LuckPermsEvent;
 import net.luckperms.api.event.type.ResultEvent;
 import net.luckperms.api.event.util.Param;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 import java.util.Objects;
@@ -55,22 +56,6 @@ public interface UniqueIdDetermineTypeEvent extends LuckPermsEvent, ResultEvent<
     */
    String TYPE_UNAUTHENTICATED = "unauthenticated";

-    /**
-     * The players UUID most likely belongs to a NPC (non-player character).
-     *
-     * <p>Usually indicated by the UUID being {@link UUID#version() version} 2.</p>
-     *
-     * @since 5.4
-     */
-    String TYPE_NPC = "npc";
-
-    /**
-     * Unknown UUID type.
-     *
-     * @since 5.4
-     */
-    String TYPE_UNKNOWN = "unknown";
-
    /**
     * Gets the {@link UUID unique id} being queried.
     *
@@ -28,6 +28,7 @@ package net.luckperms.api.event.player.lookup;
 import net.luckperms.api.event.LuckPermsEvent;
 import net.luckperms.api.event.type.ResultEvent;
 import net.luckperms.api.event.util.Param;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.checkerframework.checker.nullness.qual.Nullable;

@@ -28,6 +28,7 @@ package net.luckperms.api.event.player.lookup;
 import net.luckperms.api.event.LuckPermsEvent;
 import net.luckperms.api.event.type.ResultEvent;
 import net.luckperms.api.event.util.Param;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.checkerframework.checker.nullness.qual.Nullable;

@@ -27,6 +27,7 @@ package net.luckperms.api.event.player.lookup;

 import net.luckperms.api.event.LuckPermsEvent;
 import net.luckperms.api.event.util.Param;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 import java.util.concurrent.atomic.AtomicBoolean;
@@ -26,6 +26,7 @@
 package net.luckperms.api.event.source;

 import net.luckperms.api.platform.PlatformEntity;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**
@@ -26,6 +26,7 @@
 package net.luckperms.api.event.source;

 import net.luckperms.api.platform.PlatformEntity;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**
@@ -28,6 +28,7 @@ package net.luckperms.api.event.sync;
 import net.luckperms.api.event.LuckPermsEvent;
 import net.luckperms.api.event.type.Cancellable;
 import net.luckperms.api.event.util.Param;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 import java.util.UUID;
@@ -29,6 +29,7 @@ import net.luckperms.api.event.LuckPermsEvent;
 import net.luckperms.api.event.cause.CreationCause;
 import net.luckperms.api.event.util.Param;
 import net.luckperms.api.track.Track;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**
@@ -28,6 +28,7 @@ package net.luckperms.api.event.track;
 import net.luckperms.api.event.LuckPermsEvent;
 import net.luckperms.api.event.cause.DeletionCause;
 import net.luckperms.api.event.util.Param;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 import java.util.List;
@@ -28,6 +28,7 @@ package net.luckperms.api.event.track;
 import net.luckperms.api.event.LuckPermsEvent;
 import net.luckperms.api.event.util.Param;
 import net.luckperms.api.track.Track;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**
@@ -26,6 +26,7 @@
 package net.luckperms.api.event.track.mutate;

 import net.luckperms.api.event.util.Param;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**
@@ -28,6 +28,7 @@ package net.luckperms.api.event.track.mutate;
 import net.luckperms.api.event.LuckPermsEvent;
 import net.luckperms.api.event.util.Param;
 import net.luckperms.api.track.Track;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.jetbrains.annotations.Unmodifiable;

@@ -26,6 +26,7 @@
 package net.luckperms.api.event.track.mutate;

 import net.luckperms.api.event.util.Param;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**
@@ -26,6 +26,7 @@
 package net.luckperms.api.event.type;

 import net.luckperms.api.event.util.Param;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 import java.util.concurrent.atomic.AtomicBoolean;
@@ -26,6 +26,7 @@
 package net.luckperms.api.event.type;

 import net.luckperms.api.event.util.Param;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 import java.util.concurrent.atomic.AtomicReference;
@@ -27,6 +27,7 @@ package net.luckperms.api.event.type;

 import net.luckperms.api.event.source.Source;
 import net.luckperms.api.event.util.Param;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**
@@ -29,6 +29,7 @@ import net.luckperms.api.cacheddata.CachedDataManager;
 import net.luckperms.api.event.LuckPermsEvent;
 import net.luckperms.api.event.util.Param;
 import net.luckperms.api.model.user.User;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**
@@ -29,6 +29,7 @@ import net.luckperms.api.cacheddata.CachedDataManager;
 import net.luckperms.api.event.LuckPermsEvent;
 import net.luckperms.api.event.util.Param;
 import net.luckperms.api.model.user.User;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**
@@ -27,6 +27,7 @@ package net.luckperms.api.event.user;

 import net.luckperms.api.event.LuckPermsEvent;
 import net.luckperms.api.event.util.Param;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 import java.util.UUID;
@@ -28,6 +28,7 @@ package net.luckperms.api.event.user;
 import net.luckperms.api.event.LuckPermsEvent;
 import net.luckperms.api.event.util.Param;
 import net.luckperms.api.model.user.User;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**
@@ -29,6 +29,7 @@ import net.luckperms.api.event.LuckPermsEvent;
 import net.luckperms.api.event.type.Cancellable;
 import net.luckperms.api.event.util.Param;
 import net.luckperms.api.model.user.User;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**
@@ -30,6 +30,7 @@ import net.luckperms.api.event.type.Sourced;
 import net.luckperms.api.event.util.Param;
 import net.luckperms.api.model.user.User;
 import net.luckperms.api.track.Track;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 import java.util.Optional;
@@ -27,6 +27,7 @@ package net.luckperms.api.messaging;

 import net.luckperms.api.LuckPerms;
 import net.luckperms.api.model.user.User;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**
@@ -27,6 +27,7 @@ package net.luckperms.api.messenger;

 import net.luckperms.api.messenger.message.Message;
 import net.luckperms.api.messenger.message.OutgoingMessage;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.jetbrains.annotations.ApiStatus.NonExtendable;

@@ -27,6 +27,7 @@ package net.luckperms.api.messenger;

 import net.luckperms.api.messenger.message.Message;
 import net.luckperms.api.messenger.message.OutgoingMessage;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.jetbrains.annotations.ApiStatus.OverrideOnly;

@@ -26,6 +26,7 @@
 package net.luckperms.api.messenger;

 import net.luckperms.api.LuckPerms;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.jetbrains.annotations.ApiStatus.OverrideOnly;

@@ -26,6 +26,7 @@
 package net.luckperms.api.messenger.message;

 import net.luckperms.api.messenger.Messenger;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.jetbrains.annotations.ApiStatus.NonExtendable;

@@ -26,6 +26,7 @@
 package net.luckperms.api.messenger.message;

 import net.luckperms.api.messenger.IncomingMessageConsumer;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**
@@ -27,6 +27,7 @@ package net.luckperms.api.messenger.message.type;

 import net.luckperms.api.actionlog.Action;
 import net.luckperms.api.messenger.message.Message;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**
@@ -26,6 +26,7 @@
 package net.luckperms.api.messenger.message.type;

 import net.luckperms.api.messenger.message.Message;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 import java.util.UUID;
@@ -26,6 +26,7 @@
 package net.luckperms.api.metastacking;

 import net.luckperms.api.query.OptionKey;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.jetbrains.annotations.Unmodifiable;

@@ -27,6 +27,7 @@ package net.luckperms.api.metastacking;

 import net.luckperms.api.node.ChatMetaType;
 import net.luckperms.api.node.types.ChatMetaNode;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.checkerframework.checker.nullness.qual.Nullable;

@@ -35,6 +35,7 @@ import net.luckperms.api.node.Node;
 import net.luckperms.api.node.NodeType;
 import net.luckperms.api.query.Flag;
 import net.luckperms.api.query.QueryOptions;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.jetbrains.annotations.Unmodifiable;

@@ -26,6 +26,7 @@
 package net.luckperms.api.model;

 import net.luckperms.api.model.user.UserManager;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.checkerframework.checker.nullness.qual.Nullable;
 import org.jetbrains.annotations.Unmodifiable;
@@ -29,6 +29,7 @@ import net.luckperms.api.model.PermissionHolder;
 import net.luckperms.api.node.Node;
 import net.luckperms.api.track.Track;
 import net.luckperms.api.util.Result;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**
@@ -35,6 +35,7 @@ import net.luckperms.api.model.user.UserManager;
 import net.luckperms.api.node.Node;
 import net.luckperms.api.node.NodeEqualityPredicate;
 import net.luckperms.api.util.Tristate;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.jetbrains.annotations.Unmodifiable;

@@ -27,6 +27,7 @@ package net.luckperms.api.model.group;

 import net.luckperms.api.model.PermissionHolder;
 import net.luckperms.api.query.QueryOptions;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.checkerframework.checker.nullness.qual.Nullable;

@@ -28,6 +28,7 @@ package net.luckperms.api.model.group;
 import net.luckperms.api.node.HeldNode;
 import net.luckperms.api.node.Node;
 import net.luckperms.api.node.matcher.NodeMatcher;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.checkerframework.checker.nullness.qual.Nullable;
 import org.jetbrains.annotations.Unmodifiable;
@@ -27,6 +27,7 @@ package net.luckperms.api.model.user;

 import net.luckperms.api.model.PermissionHolder;
 import net.luckperms.api.model.data.DataMutateResult;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.checkerframework.checker.nullness.qual.Nullable;

@@ -29,6 +29,7 @@ import net.luckperms.api.model.PlayerSaveResult;
 import net.luckperms.api.node.HeldNode;
 import net.luckperms.api.node.Node;
 import net.luckperms.api.node.matcher.NodeMatcher;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.checkerframework.checker.nullness.qual.Nullable;
 import org.jetbrains.annotations.Unmodifiable;
@@ -28,9 +28,8 @@ package net.luckperms.api.node;
 import net.luckperms.api.node.types.ChatMetaNode;
 import net.luckperms.api.node.types.PrefixNode;
 import net.luckperms.api.node.types.SuffixNode;
-import org.checkerframework.checker.nullness.qual.NonNull;

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

 /**
 * Represents a type of chat meta
@@ -71,7 +70,7 @@ public enum ChatMetaType {
    private final NodeType<? extends ChatMetaNode<?, ?>> nodeType;

    ChatMetaType(NodeType<? extends ChatMetaNode<?, ?>> nodeType) {
-        this.name = nodeType.name().toLowerCase(Locale.ROOT);
+        this.name = nodeType.name().toLowerCase();
        this.nodeType = nodeType;
    }

@@ -26,6 +26,7 @@
 package net.luckperms.api.node;

 import net.luckperms.api.model.PermissionHolder;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**
@@ -36,6 +36,7 @@ import net.luckperms.api.node.types.PrefixNode;
 import net.luckperms.api.node.types.RegexPermissionNode;
 import net.luckperms.api.node.types.SuffixNode;
 import net.luckperms.api.node.types.WeightNode;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.checkerframework.checker.nullness.qual.Nullable;
 import org.jetbrains.annotations.ApiStatus.NonExtendable;
@@ -27,6 +27,7 @@ package net.luckperms.api.node;

 import net.luckperms.api.context.ContextSet;
 import net.luckperms.api.node.metadata.NodeMetadataKey;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.checkerframework.checker.nullness.qual.Nullable;
 import org.jetbrains.annotations.ApiStatus.NonExtendable;
@@ -33,6 +33,7 @@ import net.luckperms.api.node.types.PrefixNode;
 import net.luckperms.api.node.types.RegexPermissionNode;
 import net.luckperms.api.node.types.SuffixNode;
 import net.luckperms.api.node.types.WeightNode;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.jetbrains.annotations.ApiStatus.Internal;

@@ -34,6 +34,7 @@ import net.luckperms.api.node.types.PrefixNode;
 import net.luckperms.api.node.types.RegexPermissionNode;
 import net.luckperms.api.node.types.SuffixNode;
 import net.luckperms.api.node.types.WeightNode;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 import java.util.Objects;
@@ -30,6 +30,7 @@ import net.luckperms.api.node.Node;
 import net.luckperms.api.node.NodeEqualityPredicate;
 import net.luckperms.api.node.NodeType;
 import net.luckperms.api.node.types.MetaNode;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.jetbrains.annotations.ApiStatus.NonExtendable;

@@ -29,6 +29,7 @@ import net.luckperms.api.node.Node;
 import net.luckperms.api.node.NodeEqualityPredicate;
 import net.luckperms.api.node.NodeType;
 import net.luckperms.api.node.types.MetaNode;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.jetbrains.annotations.ApiStatus.Internal;

@@ -27,7 +27,6 @@ package net.luckperms.api.node.metadata;

 import org.checkerframework.checker.nullness.qual.NonNull;

-import java.util.Locale;
 import java.util.Objects;

 final class SimpleNodeMetadataKey<T> implements NodeMetadataKey<T> {
@@ -35,7 +34,7 @@ final class SimpleNodeMetadataKey<T> implements NodeMetadataKey<T> {
    private final Class<T> type;

    SimpleNodeMetadataKey(String name, Class<T> type) {
-        this.name = name.toLowerCase(Locale.ROOT);
+        this.name = name.toLowerCase();
        this.type = type;
    }

@@ -26,9 +26,9 @@
 package net.luckperms.api.node.metadata.types;

 import net.luckperms.api.model.PermissionHolder;
-import net.luckperms.api.model.data.DataType;
 import net.luckperms.api.node.Node;
 import net.luckperms.api.node.metadata.NodeMetadataKey;
+
 import org.checkerframework.checker.nullness.qual.NonNull;
 import org.jetbrains.annotations.ApiStatus.NonExtendable;

@@ -56,15 +56,6 @@ public interface InheritanceOriginMetadata {
     */
    PermissionHolder.@NonNull Identifier getOrigin();

-    /**
-     * Gets the {@link DataType type} of the {@link net.luckperms.api.model.data.NodeMap}
-     * the node was inherited from.
-     *
-     * @return the type of the NodeMap the node was inherited from.
-     * @since 5.4
-     */
-    @NonNull DataType getDataType();
-
    /**
     * Gets whether the associated node was inherited from another holder.
     *
@@ -28,6 +28,7 @@ package net.luckperms.api.node.types;
 import net.luckperms.api.node.ChatMetaType;
 import net.luckperms.api.node.NodeBuilder;
 import net.luckperms.api.node.ScopedNode;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**
@@ -30,6 +30,7 @@ import net.luckperms.api.node.Node;
 import net.luckperms.api.node.NodeBuilder;
 import net.luckperms.api.node.NodeType;
 import net.luckperms.api.node.ScopedNode;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**
@@ -78,7 +79,6 @@ public interface DisplayNameNode extends ScopedNode<DisplayNameNode, DisplayName
         *
         * @param displayName the display name
         * @return the builder
-         * @throws IllegalArgumentException if {@code displayName} is empty
         */
        @NonNull Builder displayName(@NonNull String displayName);

@@ -31,6 +31,7 @@ import net.luckperms.api.node.Node;
 import net.luckperms.api.node.NodeBuilder;
 import net.luckperms.api.node.NodeType;
 import net.luckperms.api.node.ScopedNode;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**
@@ -95,7 +96,6 @@ public interface InheritanceNode extends ScopedNode<InheritanceNode, Inheritance
         *
         * @param group the group name
         * @return the builder
-         * @throws IllegalArgumentException if {@code group} is not a valid group name
         */
        @NonNull Builder group(@NonNull String group);

@@ -30,6 +30,7 @@ import net.luckperms.api.node.Node;
 import net.luckperms.api.node.NodeBuilder;
 import net.luckperms.api.node.NodeType;
 import net.luckperms.api.node.ScopedNode;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**
@@ -86,7 +87,6 @@ public interface MetaNode extends ScopedNode<MetaNode, MetaNode.Builder> {
         *
         * @param key the meta key
         * @return the builder
-         * @throws IllegalArgumentException if {@code key} is empty
         */
        @NonNull Builder key(@NonNull String key);

@@ -30,6 +30,7 @@ import net.luckperms.api.node.Node;
 import net.luckperms.api.node.NodeBuilder;
 import net.luckperms.api.node.NodeType;
 import net.luckperms.api.node.ScopedNode;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 import java.util.OptionalInt;
@@ -103,7 +104,6 @@ public interface PermissionNode extends ScopedNode<PermissionNode, PermissionNod
         *
         * @param permission the permission
         * @return the builder
-         * @throws IllegalArgumentException if {@code permission} is empty
         */
        @NonNull Builder permission(@NonNull String permission);

@@ -28,6 +28,7 @@ package net.luckperms.api.node.types;
 import net.luckperms.api.LuckPermsProvider;
 import net.luckperms.api.node.Node;
 import net.luckperms.api.node.NodeType;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**
@@ -30,6 +30,7 @@ import net.luckperms.api.node.Node;
 import net.luckperms.api.node.NodeBuilder;
 import net.luckperms.api.node.NodeType;
 import net.luckperms.api.node.ScopedNode;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 import java.util.Optional;
@@ -100,7 +101,6 @@ public interface RegexPermissionNode extends ScopedNode<RegexPermissionNode, Reg
         *
         * @param pattern the pattern
         * @return the builder
-         * @throws IllegalArgumentException if {@code pattern} is empty
         */
        @NonNull Builder pattern(@NonNull String pattern);

@@ -28,6 +28,7 @@ package net.luckperms.api.node.types;
 import net.luckperms.api.LuckPermsProvider;
 import net.luckperms.api.node.Node;
 import net.luckperms.api.node.NodeType;
+
 import org.checkerframework.checker.nullness.qual.NonNull;

 /**