Exclude CODEOWNERS from language statistics

# Conflicts:
#	common/src/main/java/ac/grim/grimac/checks/impl/combat/Reach.java

.editorconfig

@@ -7,6 +7,8 @@ indent_style = space
 insert_final_newline = true
 max_line_length = 100
 tab_width = 4
+end_of_line = crlf
+trim_trailing_whitespace = true
 
 # intellij specific configuration
 ij_continuation_indent_size = 8
@@ -15,3 +17,6 @@ ij_formatter_on_tag = @formatter:on
 ij_formatter_tags_enabled = false
 ij_smart_tabs = false
 ij_wrap_on_typing = false
+
+[.github/ISSUE_TEMPLATE/*.yml]
+indent_size = 2

.gitattributes

@@ -0,0 +1 @@
+.gitea/CODEOWNERS linguist-detectable=false

.gitea/CODEOWNERS

@@ -0,0 +1,3 @@
+# Pull requests request review from the eligible staff team(s).
+# One approval from either an Admin or Developer satisfies branch protection.
+*   @trueog/Admins @trueog/Developers

.github/ISSUE_TEMPLATE/bypass.yml

@@ -5,7 +5,7 @@ body:
   - type: textarea
     attributes:
       label: Describe the bypass and how to replicate it
-      description: Provide a video, or detailed instructions to help replicate it.
+      description: Provide a video or detailed instructions to help replicate it.
       placeholder: |
         Step 1. Do this
         Step 2. Then do this
@@ -17,8 +17,8 @@ body:
     attributes:
       label: Grim version
       description: |
-        Provide the git commit or the version in the jar's name.
-      placeholder: A example would be b85c2d0 or grim version 2.3.5, etc
+        Provide the version of Grim running on the server. You can run `/grim version` to check.
+      placeholder: Examples include 2.3.73-8ddd7f1, 2.3.72, etc.
     validations:
       required: true
 
@@ -26,15 +26,23 @@ body:
     attributes:
       label: Server version
       description: |
-        Run `/version` on your server to check. Provide both the server type (paper, airplane, etc) & the minecraft version.
+        Run `/version` on your server to check. Provide both the server type (ie Paper, Airplane, etc) & the Minecraft version.
+    validations:
+      required: true
+
+  - type: input
+    attributes:
+      label: Client version
+      description: |
+        Provide the version of your Minecraft client. Also provide the protocol version if changed.
     validations:
       required: true
 
   - type: textarea
     attributes:
-      label: Plugins
+      label: Plugins/Mods
       description: |
-        List all plugins or mods running on your server. To list plugins on bukkit, run `/plugins`.
+        List all plugins or mods installed on your server. To list plugins on Bukkit-based servers, run `/plugins`.
         If your server is behind a proxy such as Bungeecord, you should list the proxy's plugins as well.
     validations:
       required: true

.github/ISSUE_TEMPLATE/config.yml

@@ -0,0 +1 @@
+blank_issues_enabled: false

.github/ISSUE_TEMPLATE/crash-or-error.yml

@@ -6,7 +6,7 @@ body:
     attributes:
       label: Stack trace
       description: |
-        Please include the entire stack trace with no attachments. If you would like, you can use a text pasting site such as https://paste.gg.
+        Provide the entire stack trace with no attachments. If you would like, you can use a text pasting site such as https://mclo.gs.
       placeholder: Paste the stack trace or link to the stack trace here
     validations:
       required: true
@@ -14,7 +14,7 @@ body:
   - type: textarea
     attributes:
       label: How to replicate it if known
-      description: A video, or detailed instructions to help replicate the issue would help.
+      description: A video or detailed instructions to help replicate the issue would help.
       placeholder: |
         Step 1. Do this
         Step 2. Then do this
@@ -26,8 +26,8 @@ body:
     attributes:
       label: Grim version
       description: |
-        Provide the git commit or the version in the jar's name.
-      placeholder: A example would be b85c2d0 or grim version 2.3.5, etc
+        Provide the version of Grim running on the server. You can run `/grim version` to check.
+      placeholder: Examples include 2.3.73-8ddd7f1, 2.3.72, etc.
     validations:
       required: true
 
@@ -35,27 +35,27 @@ body:
     attributes:
       label: Server version
       description: |
-        Run `/version` on your server to check. Provide both the server type (paper, airplane, etc) & the minecraft version.
+        Run `/version` on your server to check. Provide both the server type (ie Paper, Airplane, etc) & the Minecraft version.
     validations:
       required: true
 
   - type: dropdown
     id: type
     attributes:
-        label: Affected Platform(s)
-        description: Which platforms does this issue affect? If you aren't sure, select both.
-        options:
-            - "Bukkit"
-            - "Fabric"
-        multiple: true
+      label: Affected Platform(s)
+      description: Which platforms does this issue affect? If you aren't sure, select both.
+      options:
+        - "Bukkit"
+        - "Fabric"
+      multiple: true
     validations:
-        required: true
+      required: true
 
   - type: textarea
     attributes:
-      label: Plugins/mods
+      label: Plugins/Mods
       description: |
-        List all plugins or mods running on your server. To list plugins on bukkit, run `/plugins`.
+        List all plugins or mods installed on your server. To list plugins on Bukkit-based servers, run `/plugins`.
         If your server is behind a proxy such as Bungeecord, you should list the proxy's plugins as well.
     validations:
       required: true

.github/ISSUE_TEMPLATE/false-positive.yml

@@ -20,8 +20,8 @@ body:
     attributes:
       label: Grim version
       description: |
-        Provide the git commit or the version in the jar's name.
-      placeholder: A example would be b85c2d0 or grim version 2.3.5, etc
+        Provide the version of Grim running on the server. You can run `/grim version` to check.
+      placeholder: Examples include 2.3.73-8ddd7f1, 2.3.72, etc.
     validations:
       required: true
 
@@ -29,7 +29,7 @@ body:
     attributes:
       label: Server version
       description: |
-        Run `/version` on your server to check. Provide both the server type (paper, airplane, etc) & the minecraft version.
+        Run `/version` on your server to check. Provide both the server type (ie Paper, Airplane, etc) & the Minecraft version.
     validations:
       required: true
 
@@ -37,16 +37,16 @@ body:
     attributes:
       label: Client version
       description: |
-        Provide the version of your Minecraft client. You must be able to reproduce the false positive using the vanilla client. 
-        Lunar Client, TLauncher etc. will result in your issue being closed.
+        Provide the version of your Minecraft client. You must be able to reproduce the false positive using a vanilla client.
+        Using Lunar Client, TLauncher, Fabric, etc, will result in your issue being closed.
     validations:
       required: true
 
   - type: textarea
     attributes:
-      label: Plugins
+      label: Plugins/Mods
       description: |
-        List all plugins or mods running on your server. To list plugins on bukkit, run `/plugins`.
+        List all plugins or mods installed on your server. To list plugins on Bukkit-based servers, run `/plugins`.
         If your server is behind a proxy such as Bungeecord, you should list the proxy's plugins as well.
     validations:
       required: true

.github/ISSUE_TEMPLATE/feature.yml

@@ -5,6 +5,6 @@ body:
   - type: textarea
     attributes:
       label: Feature description
-      description: Describe the feature and why it should be added
+      description: Describe the feature and why it should be added.
     validations:
-      required: true
+      required: true

.github/ISSUE_TEMPLATE/other-issues.yml

@@ -5,7 +5,7 @@ body:
   - type: textarea
     attributes:
       label: Describe the bug and how to replicate it
-      description: A video, or detailed instructions to help replicate the issue would help.
+      description: A video or detailed instructions to help replicate the issue would help.
       placeholder: |
         Step 1. Do this
         Step 2. Then do this
@@ -16,9 +16,8 @@ body:
   - type: textarea
     attributes:
       label: Grim version
-      description: |
-        Provide the git commit or the version in the jar's name.
-      placeholder: A example would be b85c2d0 or grim version 2.3.5, etc
+      description: Provide the version of Grim running on the server. You can run `/grim version` to check.
+      placeholder: Examples include 2.3.73-8ddd7f1, 2.3.72, etc.
     validations:
       required: true
 
@@ -26,27 +25,27 @@ body:
     attributes:
       label: Server version
       description: |
-        Run `/version` on your server to check. Provide both the server type (paper, airplane, etc) & the minecraft version.
+        Run `/version` on your server to check. Provide both the server type (ie Paper, Airplane, etc) & the Minecraft version.
     validations:
       required: true
 
   - type: dropdown
     id: type
     attributes:
-        label: Affected Platform(s)
-        description: Which platforms does this issue affect? If you aren't sure, select both.
-        options:
-            - "Bukkit"
-            - "Fabric"
-        multiple: true
+      label: Affected Platform(s)
+      description: Which platforms does this issue affect? If you aren't sure, select both.
+      options:
+        - "Bukkit"
+        - "Fabric"
+      multiple: true
     validations:
-        required: true
+      required: true
 
   - type: textarea
     attributes:
-      label: Plugins
+      label: Plugins/Mods
       description: |
-        List all plugins or mods running on your server. To list plugins on bukkit, run `/plugins`.
+        List all plugins or mods installed on your server. To list plugins on Bukkit-based servers, run `/plugins`.
         If your server is behind a proxy such as Bungeecord, you should list the proxy's plugins as well.
     validations:
       required: true

.github/ISSUE_TEMPLATE/simulation-false-positive.yml

@@ -19,8 +19,15 @@ body:
   - type: input
     attributes:
       label: Grim Log
-      description: |
-        Provide the generated /grim log link.
+      description: Provide the generated /grim log link.
+    validations:
+      required: true
+
+  - type: textarea
+    attributes:
+      label: Grim version
+      description: Provide the version of Grim running on the server. You can run `/grim version` to check.
+      placeholder: Examples include 2.3.73-8ddd7f1, 2.3.72, etc.
     validations:
       required: true
 
@@ -28,7 +35,7 @@ body:
     attributes:
       label: Server version
       description: |
-        Run `/version` on your server to check. Provide both the server type (paper, airplane, etc) & the minecraft version.
+        Run `/version` on your server to check. Provide both the server type (ie Paper, Airplane, etc) & the Minecraft version.
     validations:
       required: true
 
@@ -36,16 +43,16 @@ body:
     attributes:
       label: Client version
       description: |
-        Provide the version of your Minecraft client. You must be able to reproduce the false positive using the vanilla client. 
-        Lunar Client, TLauncher etc. will result in your issue being closed.
+        Provide the version of your Minecraft client. You must be able to reproduce the false positive using a vanilla client.
+        Using Lunar Client, TLauncher, Fabric, etc, will result in your issue being closed.
     validations:
       required: true
 
   - type: textarea
     attributes:
-      label: Plugins
+      label: Plugins/Mods
       description: |
-        List all plugins or mods running on your server. To list plugins on bukkit, run `/plugins`.
+        List all plugins or mods installed on your server. To list plugins on Bukkit-based servers, run `/plugins`.
         If your server is behind a proxy such as Bungeecord, you should list the proxy's plugins as well.
     validations:
-        required: true
+      required: true

.github/workflows/build-and-publish.yml

@@ -0,0 +1,253 @@
+name: Reusable Build and Publish
+
+on:
+    workflow_call:
+        inputs:
+            runner:
+                description: 'The runner to use for the build job.'
+                required: false
+                type: string
+                default: 'ubuntu-latest'
+            is-release-build:
+                description: 'True if this is a release build, which affects versioning and publishing.'
+                required: true
+                type: boolean
+            release-version-type:
+                description: 'The version type for Modrinth (e.g., release, beta, alpha).'
+                required: true
+                type: string
+            release-changelog:
+                description: 'The changelog content for the release.'
+                required: false
+                type: string
+                default: ''
+            run-build-scan:
+                description: 'Whether to enable Develocity build scan'
+                required: true
+                type: boolean
+            publish-artifacts:
+                description: 'If true, uploads artifacts to Modrinth and other distribution sites.'
+                required: true
+                type: boolean
+        secrets:
+            MODRINTH_ID:
+                required: false
+            MODRINTH_TOKEN:
+                required: false
+            BUILD_UPDATE_TOKEN:
+                required: true
+            BUILD_UPDATE_URL:
+                required: true
+
+jobs:
+    build:
+        runs-on: ${{ inputs.runner }}
+        permissions:
+            # Grant permission to write contents (for creating tags/refs)
+            contents: write
+            packages: write
+
+        env:
+            MODRINTH_ID: ${{ secrets.MODRINTH_ID }}
+            MODRINTH_TOKEN: ${{ secrets.MODRINTH_TOKEN }}
+            GRIM_MAIN_BRANCH: ${{ github.event.repository.default_branch }}
+            BUILD_UPDATE_TOKEN: ${{ secrets.BUILD_UPDATE_TOKEN }}
+
+        steps:
+            - name: Checkout repository with full history
+              uses: actions/checkout@v4
+              with:
+                  persist-credentials: false
+                  fetch-depth: 200
+
+            - name: Set up JDK 21
+              uses: actions/setup-java@v4
+              with:
+                  java-version: '21'
+                  distribution: 'temurin'
+                  server-id: github
+                  settings-path: ${{ github.workspace }}
+
+            - name: Setup Gradle
+              uses: gradle/actions/setup-gradle@v3
+              with:
+                  cache-read-only: ${{ github.ref_name != github.event.repository.default_branch && github.ref_name != 'dev' }}
+
+            - name: Cache Gradle Loom Cache
+              uses: actions/cache@v4
+              with:
+                  path: .gradle/loom-cache
+                  key: loom-cache-${{ runner.os }}-${{ hashFiles('**/libs.versions.toml', 'fabric/**/build.gradle.kts') }}
+                  restore-keys: |
+                      loom-cache-${{ runner.os }}
+
+            - name: Set Git Environment Variables
+              run: |
+                  echo "GRIM_GIT_REPO=${{ github.event.repository.name }}" >> $GITHUB_ENV
+                  echo "GRIM_GIT_ORG=${{ github.repository_owner }}" >> $GITHUB_ENV
+
+            - name: Compute version and Build with Gradle
+              id: build_with_version
+              run: |
+                  # If any command in a pipeline fails, treat the pipeline as failed.
+                  set -o pipefail
+
+                  BUILD_ARGS="build"
+                  if [[ "${{ inputs.is-release-build }}" == "true" ]]; then
+                    BUILD_ARGS="$BUILD_ARGS -Prelease=true"
+                  fi
+                  # Conditionally add the --scan flag
+                  if [[ "${{ inputs.run-build-scan }}" == "true" ]]; then
+                    BUILD_ARGS="$BUILD_ARGS --scan"
+                  fi
+
+                  echo "Running: ./gradlew printVersion $BUILD_ARGS"
+
+                  # Use 'tee' to both display the output live and save it to a file for parsing
+                  VERSION=$(./gradlew printVersion $BUILD_ARGS | tee build-output.log | grep '^VERSION=' | tail -n1 | cut -d'=' -f2)
+
+                  echo "build_version=$VERSION" >> $GITHUB_OUTPUT
+                  echo "BUILD_VERSION=$VERSION" >> $GITHUB_ENV
+                  echo "Build version: $VERSION"
+
+            - name: Publish Build Scan URL
+              # Only run this step if a scan was requested
+              if: ${{ inputs.run-build-scan == true }}
+              run: |
+                  # Search the log file for the build scan URL and extract it
+                  SCAN_URL=$(grep -o 'https://gradle.com/s/[^ ]*' build-output.log | head -n 1 || true)
+
+                  if [[ -n "$SCAN_URL" ]]; then
+                    echo "Found build scan URL: $SCAN_URL"
+                    # Post a highly visible link to the Job Summary page
+                    echo "### Gradle Build Scan" >> $GITHUB_STEP_SUMMARY
+                    echo "A public build scan has been published and is available for inspection." >> $GITHUB_STEP_SUMMARY
+                    echo "" >> $GITHUB_STEP_SUMMARY
+                    echo "[**View Build Scan →**]($SCAN_URL)" >> $GITHUB_STEP_SUMMARY
+                  else
+                    echo "Could not find build scan URL in build-output.log" >> $GITHUB_STEP_SUMMARY
+                  fi
+
+            - name: Upload Bukkit Artifact
+              uses: actions/upload-artifact@v6
+              with:
+                  name: grimac-bukkit
+                  path: ${{ github.workspace }}/bukkit/build/libs/grimac-bukkit-${{ env.BUILD_VERSION }}.jar
+                  if-no-files-found: error
+
+            - name: Upload Fabric Artifact
+              uses: actions/upload-artifact@v6
+              with:
+                  name: grimac-fabric
+                  path: ${{ github.workspace }}/fabric/build/libs/grimac-fabric-${{ env.BUILD_VERSION }}.jar
+                  if-no-files-found: error
+
+            #------------------------------------------------------------------
+            #  Generate/Prepare Changelog
+            #------------------------------------------------------------------
+            - name: Generate Changelog with git-cliff (for Dev Builds)
+              if: ${{ !inputs.is-release-build }}
+              id: git-cliff
+              uses: orhun/git-cliff-action@v4
+              with:
+                  config: cliff.toml
+                  args: --verbose ${{ github.event.before }}..${{ github.event.after }}
+              env:
+                  GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} # git-cliff needs a token
+
+            - name: Prepare Final Changelog
+              id: changelog
+              # Add this 'env' block to safely load the changelogs as environment variables
+              env:
+                  RELEASE_NOTES: ${{ inputs.release-changelog }}
+                  DEV_NOTES: ${{ steps.git-cliff.outputs.content }}
+              run: |
+                  NOTES=""
+                  if [[ "${{ inputs.is-release-build }}" == "true" ]]; then
+                    # Use the environment variable for the release changelog.
+                    # The double quotes are critical to preserve newlines!
+                    NOTES="$RELEASE_NOTES"
+                  else
+                    # Use the environment variable for the dev changelog.
+                    NOTES="$DEV_NOTES"
+                  fi
+
+                  if [ -z "$NOTES" ]; then
+                    NOTES="* No new code changes in this build."
+                  fi
+
+                  # This is the correct way to set a multi-line output variable in Actions
+                  {
+                    echo "notes<<EOF"
+                    echo "$NOTES"
+                    echo "EOF"
+                  } >> "$GITHUB_OUTPUT"
+
+            - name: Generate build number
+              if: contains(fromJSON('["merge","release","main","lightning"]'), github.ref_name) && inputs.publish-artifacts == true && env.MODRINTH_ID != '' && env.MODRINTH_TOKEN != ''
+              id: buildnumber
+              uses: onyxmueller/build-tag-number@v1
+              with:
+                  token: ${{ secrets.GITHUB_TOKEN }}
+
+            #------------------------------------------------------------------
+            #  Publish to Modrinth & API
+            #------------------------------------------------------------------
+            - name: Publish Bukkit to Modrinth
+              if: ${{ inputs.publish-artifacts == true && env.MODRINTH_ID != '' && env.MODRINTH_TOKEN != '' }}
+              uses: Kir-Antipov/mc-publish@v3.3
+              with:
+                  modrinth-id: ${{ secrets.MODRINTH_ID }}
+                  modrinth-token: ${{ secrets.MODRINTH_TOKEN }}
+                  modrinth-featured: ${{ inputs.is-release-build }} # Only feature release builds
+
+                  files: bukkit/build/libs/grimac-bukkit-${{ env.BUILD_VERSION }}.jar
+                  name: LightningGrim Anticheat (Bukkit) ${{ env.BUILD_VERSION }}-b${{ steps.buildnumber.outputs.build_number }}
+                  version: ${{ env.BUILD_VERSION }}-b${{ steps.buildnumber.outputs.build_number }}
+                  version-type: ${{ inputs.release-version-type }}
+                  changelog: ${{ steps.changelog.outputs.notes }}
+                  loaders: |
+                      bukkit
+                      spigot
+                      paper
+                      folia
+                      purpur
+                  game-versions: '>=1.7 <=1.21.11'
+
+            - name: Publish Fabric to Modrinth
+              if: ${{ inputs.publish-artifacts == true && env.MODRINTH_ID != '' && env.MODRINTH_TOKEN != '' }}
+              uses: Kir-Antipov/mc-publish@v3.3
+              with:
+                  modrinth-id:    ${{ secrets.MODRINTH_ID }}
+                  modrinth-token: ${{ secrets.MODRINTH_TOKEN }}
+                  modrinth-featured: ${{ inputs.is-release-build }} # Only feature release builds
+
+                  files: fabric/build/libs/grimac-fabric-${{ env.BUILD_VERSION }}.jar
+                  name: LightningGrim Anticheat (Fabric) ${{ env.BUILD_VERSION }}-b${{ steps.buildnumber.outputs.build_number }}
+                  version: ${{ env.BUILD_VERSION }}-b${{ steps.buildnumber.outputs.build_number }}
+                  version-type: ${{ inputs.release-version-type }}
+                  changelog: ${{ steps.changelog.outputs.notes }}
+                  loaders: |
+                      fabric
+                  game-versions: '>=1.16.1 <=1.21.10'
+                  dependencies: 'fabric-api'
+
+            - name: Publish to API
+              if: ${{ inputs.publish-artifacts == true && env.BUILD_UPDATE_TOKEN != '' && inputs.is-release-build == false }}
+              run: |
+                  curl -X POST \
+                    -H "Content-Type: application/json" \
+                    -H "User-Agent: GrimACBuilder/v1" \
+                    -H "Authorization: Bearer ${{ secrets.BUILD_UPDATE_TOKEN }}" \
+                    -d '{"org": "${{ github.repository_owner }}", "repository": "${{ github.event.repository.name }}", "branch": "${{ github.ref_name }}", "version": "${{ env.BUILD_VERSION }}"}' \
+                    "${{ secrets.BUILD_UPDATE_URL }}"
+
+            - name: Publish Release to API
+              if: ${{ inputs.publish-artifacts == true && env.BUILD_UPDATE_TOKEN != '' && inputs.is-release-build == true }}
+              run: |
+                  curl -X POST \
+                    -H "Content-Type: application/json" \
+                    -H "User-Agent: GrimACBuilder/v1" \
+                    -H "Authorization: Bearer ${{ secrets.BUILD_UPDATE_TOKEN }}" \
+                    -d '{"org": "${{ github.repository_owner }}", "repository": "${{ github.event.repository.name }}", "branch": "release", "version": "${{ env.BUILD_VERSION }}"}' \
+                    "${{ secrets.BUILD_UPDATE_URL }}"

.github/workflows/codeql-analysis.yml

@@ -12,60 +12,64 @@
 name: "CodeQL"
 
 on:
-  push:
-    branches: [ master ]
-  pull_request:
-    # The branches below must be a subset of the branches above
-    branches: [ master ]
-  schedule:
-    - cron: '30 15 * * 2'
+    workflow_dispatch:
+    push:
+        branches: [dev]
 
 jobs:
-  analyze:
-    name: Analyze
-    runs-on: ubuntu-latest
-    permissions:
-      actions: read
-      contents: read
-      security-events: write
+    analyze:
+        name: Analyze
+        runs-on: tenki-standard-autoscale
+        permissions:
+            actions: read
+            contents: read
+            security-events: write
 
-    strategy:
-      fail-fast: false
-      matrix:
-        language: [ 'java' ]
-        # CodeQL supports [ 'cpp', 'csharp', 'go', 'java', 'javascript', 'python' ]
-        # Learn more:
-        # https://docs.github.com/en/free-pro-team@latest/github/finding-security-vulnerabilities-and-errors-in-your-code/configuring-code-scanning#changing-the-languages-that-are-analyzed
+        strategy:
+            fail-fast: false
+            matrix:
+                language: ['java-kotlin']
+                # CodeQL supports [ 'cpp', 'csharp', 'go', 'java', 'javascript', 'python' ]
+                # Learn more:
+                # https://docs.github.com/en/free-pro-team@latest/github/finding-security-vulnerabilities-and-errors-in-your-code/configuring-code-scanning#changing-the-languages-that-are-analyzed
 
-    steps:
-      - name: Checkout repository
-        uses: actions/checkout@v4
+        steps:
+            - name: Checkout repository
+              uses: actions/checkout@v5
 
-      # Initializes the CodeQL tools for scanning.
-      - name: Initialize CodeQL
-        uses: github/codeql-action/init@v3
-        with:
-          languages: ${{ matrix.language }}
-          # If you wish to specify custom queries, you can do so here or in a config file.
-          # By default, queries listed here will override any specified in a config file.
-          # Prefix the list here with "+" to use these queries and those in the config file.
-          # queries: ./path/to/local/query, your-org/your-repo/queries@main
+            # Initializes the CodeQL tools for scanning.
+            - name: Initialize CodeQL
+              uses: github/codeql-action/init@v3
+              with:
+                  languages: ${{ matrix.language }}
+                  # If you wish to specify custom queries, you can do so here or in a config file.
+                  # By default, queries listed here will override any specified in a config file.
+                  # Prefix the list here with "+" to use these queries and those in the config file.
+                  # queries: ./path/to/local/query, your-org/your-repo/queries@main
 
-      # Autobuild attempts to build any compiled languages  (C/C++, C#, or Java).
-      # If this step fails, then you should remove it and run the build manually (see below)
-      - name: Autobuild
-        uses: github/codeql-action/autobuild@v3
+            - name: Set up JDK 21
+              uses: actions/setup-java@v5
+              with:
+                  java-version: '21'
+                  distribution: 'temurin'
+                  server-id: github # Value of the distributionManagement/repository/id field of the pom.xml
+                  settings-path: ${{ github.workspace }} # location for the settings.xml file
 
-      # ℹ️ Command-line programs to run using the OS shell.
-      # 📚 https://git.io/JvXDl
+            # Autobuild attempts to build any compiled languages  (C/C++, C#, or Java).
+            # If this step fails, then you should remove it and run the build manually (see below)
+            - name: Autobuild
+              uses: github/codeql-action/autobuild@v3
 
-      # ✏️ If the Autobuild fails above, remove it and uncomment the following three lines
-      #    and modify them (or add more) to build your code if your project
-      #    uses a compiled language
+            # ℹ️ Command-line programs to run using the OS shell.
+            # 📚 https://git.io/JvXDl
 
-      #- run: |
-      #   make bootstrap
-      #   make release
+            # ✏️ If the Autobuild fails above, remove it and uncomment the following three lines
+            #    and modify them (or add more) to build your code if your project
+            #    uses a compiled language
 
-      - name: Perform CodeQL Analysis
-        uses: github/codeql-action/analyze@v3
+            #- run: |
+            #   make bootstrap
+            #   make release
+
+            - name: Perform CodeQL Analysis
+              uses: github/codeql-action/analyze@v3

.github/workflows/gradle-publish.yml

@@ -1,59 +1,49 @@
-# This workflow uses actions that are not certified by GitHub.
-# They are provided by a third-party and are governed by
-# separate terms of service, privacy policy, and support
-# documentation.
-# This workflow will build a package using Gradle and then publish it to GitHub packages when a release is created
-# For more information see: https://github.com/actions/setup-java/blob/main/docs/advanced-usage.md#Publishing-using-gradle
-
-name: Gradle Package
-
-on: [push, workflow_dispatch]
+name: Gradle Package (Development)
 
+on:
+    workflow_dispatch:
+        inputs:
+            run-build-scan:
+                description: 'Run with public build scan (adds --scan)'
+                required: true
+                type: boolean
+                default: false
+            publish-artifacts:
+                description: 'Upload to Modrinth & API? (Uncheck for a dry run)'
+                type: boolean
+                default: true
+            runner:
+                description: 'The runner to use for the build job.'
+                required: false
+                type: string
+                default: 'tenki-standard-autoscale'
+    push:
+        branches:
+            - 'main'
+            - '2.0'
+            - 'dev'
+            - 'release'
+            - 'release/**'
+            - 'lightning'
+        paths:
+            - 'bukkit/**'
+            - 'common/**'
+            - 'fabric/**'
+            - '**.java'
+            - '**.kt'
+            - '**.kts'
+            - '**.toml'
 jobs:
-  build:
-    runs-on: ubuntu-latest
-    permissions:
-      contents: read
-      packages: write
-
-    steps:
-    - uses: actions/checkout@v4
-
-    - name: Set up JDK 21
-      uses: actions/setup-java@v4
-      with:
-        java-version: '21'
-        distribution: 'temurin'
-        server-id: github # Value of the distributionManagement/repository/id field of the pom.xml
-        settings-path: ${{ github.workspace }} # location for the settings.xml file
-
-    - name: Setup Gradle
-      uses: gradle/actions/setup-gradle@v4 # Handles Gradle wrapper validation and basic caching
-
-    - name: Cache Gradle Loom Cache
-      uses: actions/cache@v4
-      with:
-        path: .gradle/loom-cache # Path to Loom cache relative to workspace root
-        key: loom-cache-${{ runner.os }}-${{ hashFiles('**/libs.versions.toml', 'fabric/**/build.gradle.kts') }}
-        restore-keys: |
-          loom-cache-${{ runner.os }}
-
-    - name: Build with Gradle (Actual Build for Artifacts)
-      run: ./gradlew build
-
-    - name: Upload Bukkit Artifact
-      uses: actions/upload-artifact@v4
-      with:
-        name: grimac-bukkit
-        # Adding if-no-files-found for robustness
-        path: ${{ github.workspace }}/bukkit/build/libs/grimac-bukkit-*.jar
-        if-no-files-found: error
-
-    - name: Upload Fabric Artifact
-      uses: actions/upload-artifact@v4
-      with:
-        name: grimac-fabric
-        # Adding if-no-files-found for robustness
-        path: ${{ github.workspace }}/fabric/build/libs/grimac-fabric-*.jar
-        if-no-files-found: error
-
+    call-build-workflow:
+        uses: ./.github/workflows/build-and-publish.yml
+        with:
+            runner: ${{ github.event.inputs.runner || 'tenki-standard-autoscale' }}
+            is-release-build: false
+            release-version-type: alpha # Dev builds are always 'alpha'
+            run-build-scan: ${{ github.event.inputs.run-build-scan == 'true' }}
+            publish-artifacts: ${{ github.event.inputs.publish-artifacts != 'false' }}
+        secrets:
+            MODRINTH_ID: ${{ secrets.MODRINTH_ID }}
+            MODRINTH_TOKEN: ${{ secrets.MODRINTH_TOKEN }}
+            BUILD_UPDATE_TOKEN: ${{ secrets.BUILD_UPDATE_TOKEN }}
+            BUILD_UPDATE_URL: ${{ secrets.BUILD_UPDATE_URL }}

.github/workflows/release.yml

@@ -0,0 +1,22 @@
+name: Publish Release
+
+on:
+    release:
+        types: [published]
+
+jobs:
+    call-release-workflow:
+        uses: ./.github/workflows/build-and-publish.yml
+        with:
+            runner: 'tenki-standard-autoscale'
+            is-release-build: true
+            release-version-type: release # Official releases are 'release'
+            # Use the body of the GitHub Release as the changelog
+            release-changelog: ${{ github.event.release.body }}
+            run-build-scan: false
+            publish-artifacts: true
+        secrets:
+            MODRINTH_ID: ${{ secrets.MODRINTH_ID }}
+            MODRINTH_TOKEN: ${{ secrets.MODRINTH_TOKEN }}
+            BUILD_UPDATE_TOKEN: ${{ secrets.BUILD_UPDATE_TOKEN }}
+            BUILD_UPDATE_URL: ${{ secrets.BUILD_UPDATE_URL }}

.gitignore

@@ -73,3 +73,5 @@ nb-configuration.xml
 ##############################
 run/
 fabric/run/
+
+.ignored/

CONTRIBUTING.md

@@ -0,0 +1,40 @@
+# Contributing to LightningGrim
+
+Thank you for your interest in contributing to LightningGrim. This document outlines the guidelines for
+making pull
+requests to the project. *We're usually pretty lenient with pull requests, but this guide will help
+make the process go more smoothly.*
+
+### Pull Request Guidelines
+
+- **Compatibility**
+  - Any changes must be compatible with
+  the [supported environments](https://github.com/GrimAnticheat/Grim/wiki/Supported-environments) (Spigot, Paper, Folia, Fabric, etc.)
+  - The plugin must be compatible with minecraft versions 1.8 and higher. Exempting checks from specific versions is acceptable.
+  - The plugin must be able to run on Java 17 or higher. Changes that don't support Java 17 at runtime will not be accepted.
+
+- **Non-acceptable pull requests**
+  - Heuristic-based checks will not be accepted; however, basic rate limiting such as blocking attacks based on CPS is acceptable.
+  - Checks that can be easily circumvented that don't block packets or fix anything are likely to not be accepted.
+  - Checks or features that are too specific to a single environment or minecraft version are likely to not be accepted.
+  - Changes that require large or unnecessary dependencies will likely not be accepted.
+
+- **Pull request formatting**
+  - Create a new branch for your feature or fix when forking the repository.
+  - Reference related issues in your pull request description if applicable.
+  - Write clear and descriptive commit messages.
+
+- **Code styling**
+  - Add code comments for complex logic or significant changes.
+  - Try to keep your code clean and avoid duplication.
+  - Thoroughly test your changes before submitting your pull request.
+
+### Development Notes
+
+- LightningGrim is built using [Gradle](https://gradle.org/) kotlin scripts.
+- Java 21 is currently required to build the project. A minimum of Java 17 is required to run it.
+
+### Questions & Support
+
+- Join our [Discord](https://discord.grim.ac) if you have questions or need assistance.
+- Refer to the [Wiki](https://github.com/GrimAnticheat/Grim/wiki) for project documentation.

LightningGrim_build.log

@@ -0,0 +1,96 @@
+To honour the JVM settings for this build a single-use Daemon process will be forked. For more on this, please refer to https://docs.gradle.org/8.14.3/userguide/gradle_daemon.html#sec:disabling_the_daemon in the Gradle documentation.
+Daemon will be stopped at the end of the build 
+
+> Configure project :buildSrc
+WARNING: Unsupported Kotlin plugin version.
+The `embedded-kotlin` and `kotlin-dsl` plugins rely on features of Kotlin `2.0.21` that might work differently than in the requested version `2.2.20`.
+
+> Task :buildSrc:checkKotlinGradlePluginConfigurationErrors SKIPPED
+> Task :buildSrc:generateExternalPluginSpecBuilders UP-TO-DATE
+> Task :buildSrc:extractPrecompiledScriptPluginPlugins UP-TO-DATE
+> Task :buildSrc:compilePluginsBlocks UP-TO-DATE
+> Task :buildSrc:generatePrecompiledScriptPluginAccessors UP-TO-DATE
+> Task :buildSrc:generateScriptPluginAdapters UP-TO-DATE
+> Task :buildSrc:pluginDescriptors UP-TO-DATE
+> Task :buildSrc:processResources UP-TO-DATE
+
+> Task :buildSrc:compileKotlin
+w: Language version 1.8 is deprecated and its support will be removed in a future version of Kotlin
+
+> Task :buildSrc:compileJava NO-SOURCE
+> Task :buildSrc:compileGroovy NO-SOURCE
+> Task :buildSrc:classes UP-TO-DATE
+> Task :buildSrc:jar
+
+> Configure project :
+Build configuration:
+    shadePE            = false
+    relocate           = true
+    mavenLocalOverride = true
+    release            = false
+    version            = 2.3.73-HEAD-bebcab7+lite
+
+> Task :clean
+> Task :bukkit:clean
+> Task :common:clean
+> Task :assemble UP-TO-DATE
+> Task :check UP-TO-DATE
+> Task :bukkit:generateBukkitPluginDescription
+> Task :bukkit:processResources
+> Task :bukkit:generateTestEffectiveLombokConfig
+> Task :common:processResources
+> Task :bukkit:sourcesJar
+> Task :spotlessInternalRegisterDependencies
+> Task :bukkit:spotlessJava
+> Task :bukkit:spotlessJavaApply
+> Task :bukkit:spotlessJavaCheck
+> Task :bukkit:spotlessKotlinGradle
+> Task :bukkit:spotlessKotlinGradleApply
+> Task :bukkit:spotlessKotlinGradleCheck
+> Task :bukkit:spotlessCheck
+> Task :bukkit:processTestResources NO-SOURCE
+> Task :bukkit:spotlessApply
+> Task :common:sourcesJar
+> Task :common:spotlessJava
+> Task :common:spotlessJavaApply
+> Task :common:spotlessJavaCheck
+> Task :common:spotlessKotlinGradle
+> Task :common:spotlessKotlinGradleApply
+> Task :common:spotlessKotlinGradleCheck
+> Task :common:spotlessCheck
+> Task :common:generateTestEffectiveLombokConfig
+> Task :common:processTestResources NO-SOURCE
+> Task :common:spotlessApply
+
+> Task :common:eclipseClasspath
+Could not resolve: com.github.retrooper:packetevents-api:2.11.2-SNAPSHOT
+
+> Task :common:eclipseJdt
+> Task :common:eclipseProject
+> Task :common:eclipse
+> Task :bukkit:generateEffectiveLombokConfig
+> Task :common:generateEffectiveLombokConfig
+> Task :common:compileJava FAILED
+
+FAILURE: Build failed with an exception.
+
+* What went wrong:
+Execution failed for task ':common:compileJava'.
+> Could not resolve all files for configuration ':common:compileClasspath'.
+   > Could not find com.github.retrooper:packetevents-api:2.11.2-SNAPSHOT.
+     Searched in the following locations:
+       - https://repo.grim.ac/snapshots/com/github/retrooper/packetevents-api/2.11.2-SNAPSHOT/maven-metadata.xml
+       - https://repo.grim.ac/snapshots/com/github/retrooper/packetevents-api/2.11.2-SNAPSHOT/packetevents-api-2.11.2-SNAPSHOT.pom
+       - file:/home/admin/.m2/repository/com/github/retrooper/packetevents-api/2.11.2-SNAPSHOT/maven-metadata.xml
+       - file:/home/admin/.m2/repository/com/github/retrooper/packetevents-api/2.11.2-SNAPSHOT/packetevents-api-2.11.2-SNAPSHOT.pom
+     Required by:
+         project :common
+
+* Try:
+> Run with --stacktrace option to get the stack trace.
+> Run with --info or --debug option to get more log output.
+> Run with --scan to get full insights.
+> Get more help at https://help.gradle.org.
+
+BUILD FAILED in 17s
+38 actionable tasks: 31 executed, 7 up-to-date

README.md

@@ -1,42 +1,75 @@
-# GrimAC
+<div align="center">
+ <h1>LightningGrim</h1>
 
-GrimAC is an open source Minecraft anticheat designed for 1.21 and supports 1.8-1.21. It is free while in beta. It will eventually become paid and/or will include offering additional subscription based paid checks. Geyser players are fully exempt.
+ <div>
+  <a href="https://github.com/GrimAnticheat/Grim/actions/workflows/gradle-publish.yml">
+   <img alt="Workflow" src="https://img.shields.io/github/actions/workflow/status/GrimAnticheat/Grim/gradle-publish.yml?style=flat&logo=github"/>
+  </a>&nbsp;&nbsp;
+  <a href="https://modrinth.com/plugin/grimac">
+   <img alt="Modrinth" src="https://img.shields.io/modrinth/v/LJNGWSvH?style=flat&label=version&logo=modrinth">
+  </a>&nbsp;&nbsp;
+  <a href="https://modrinth.com/plugin/grimac#download">
+   <img alt="Downloads" src="https://img.shields.io/modrinth/dt/LJNGWSvH?style=flat&logo=modrinth&label=downloads&link=https%3A%2F%2Fmodrinth.com%2Fplugin%2Fgrimac%23download">
+  </a>&nbsp;&nbsp;
+  <a href="https://discord.grim.ac">
+   <img alt="Discord" src="https://img.shields.io/discord/811396969670901800?style=flat&label=discord&logo=discord">
+  </a>
+ </div>
+ <br>
+</div>
 
-This project is considered feature complete for the 2.0 (open-source) branch of this project. If you would like a bugfix or enhancement and cannot sponsor the work, pull requests are welcome.
+LightningGrim is an open source Minecraft anticheat fork designed to support the latest versions of Minecraft.
+It currently supports minecraft versions 1.8–1.21. Geyser players are fully exempt from the anticheat to prevent false positives.
+This project is considered feature-complete for the 2.0 (open-source) branch. If you would like a bug fix or enhancement and cannot sponsor the work, pull requests are welcome.
+A premium version is planned, which will offer additional subscription-based paid checks, such as heuristics.
 
-## Downloads
-- [Modrinth](https://modrinth.com/plugin/grimac)
-- [Hangar](https://hangar.papermc.io/GrimAnticheat/GrimAnticheat)
-- [SpigotMC](https://www.spigotmc.org/resources/grim-anticheat.99923/)
-- *For bleeding edge builds use GitHub artifacts*: [Bukkit](https://nightly.link/GrimAnticheat/Grim/workflows/gradle-publish/2.0/grimac-bukkit.zip), [Fabric](https://nightly.link/GrimAnticheat/Grim/workflows/gradle-publish/2.0/grimac-fabric.zip)
+## Changes over GrimAC:
 
-## Installation notes
-> [!WARNING]
-> Java 17 is now required. More information [here](https://github.com/GrimAnticheat/Grim/wiki/Updating-to-Java-17).
-- Paper, Spigot, Folia, and Fabric are currently supported.
-- If you use Geyser, place Floodgate on the backend server so grim can exempt bedrock players. Grim cannot access the Floodgate API if it is on the proxy.
-- If you use ViaVersion, it should be on the backend server as movement is highly dependent on client version.
+- WallHit - checks for hitting through walls
+- EntityPierce - checks for hitting through other entities
+- Bukkit Event Piston Optimizations
+- Optimized Hitbox/Collision Boxes
+- Optimized Reach calculations
+- Checks for PacketOrder (patches lots of noslow, autoblocks, etc...)
+- Checks for Inventory interactions (prevents using crafting slots as extra storage, moving while having inventory open, etc...)
 
-## Support & wiki information
-- Support & discussion: [Discord](https://discord.com/invite/kqQAhTmkUF)
-- Report issues: [Issues](https://github.com/GrimAnticheat/Grim/issues/new/choose)
-- Wiki & examples: [Wiki](https://github.com/GrimAnticheat/Grim/wiki)
+## Requirements & Installation
 
-## Developer API
-Grim's API allows you to integrate Grim into your own plugins.
-- API repository: [GrimAPI](https://github.com/GrimAnticheat/GrimAPI)
-- Wiki info: [Wiki](https://github.com/GrimAnticheat/GrimAPI)
+- Java 17 or higher. *For more details, see [Updating-to-Java-17](https://github.com/GrimAnticheat/Grim/wiki/Updating-to-Java-17).*
+- A Spigot, Paper, Folia, or Fabric server environment. *For more details, see [Supported-environments](https://github.com/GrimAnticheat/Grim/wiki/Supported-environments).*
 
-## How to compile
+If you use a proxy such as Velocity or Bungeecord:
+- If you use Geyser, Floodgate must be installed on the backend server (where Grim is) so Grim can access the Floodgate API.
+- If you use ViaVersion, it must be installed on the backend server (where Grim is) ONLY.
+  Grim does not support having ViaVersion installed on the proxy, even if it is also installed on the backend.
 
-1. `git clone https://github.com/GrimAnticheat/Grim.git`
-2. `cd Grim`
-3. `./gradlew build`
+## Resources
+
+- For documentation and examples visit the [Wiki](https://github.com/GrimAnticheat/Grim/wiki).
+- For answers to commonly asked questions visit the [FAQ](https://github.com/GrimAnticheat/Grim/wiki/FAQ).
+- For community support and project discussion join our [Discord](https://discord.grim.ac).
+
+## Pull Requests
+
+See [Contributing](CONTRIBUTING.md) for more information about contributing and what our guidelines
+are.
+
+## Developer Plugin API
+
+LightningGrim's plugin API allows you to integrate LightningGrim into your own plugins. Visit
+the [plugin API repository](https://github.com/GrimAnticheat/GrimAPI) for the source code and more
+information.
+
+## Compiling From Source
+
+1. `git clone https://github.com/Axionize/LightningGrim`
+2. `cd LightningGrim`
+3. `./gradlew build clean eclipse --warning-mode all`
 4. The final jars will compile into the `<platform>/build/libs` folders
 
-## Grim supremacy
+## LightningGrim Supremacy
 
-What makes Grim stand out against other anticheats?
+What makes LightningGrim stand out against other anticheats?
 
 ### Movement Simulation Engine
 
@@ -44,7 +77,9 @@ What makes Grim stand out against other anticheats?
     * This covers everything from basic walking, swimming, knockback, cobwebs, to bubble columns
     * It even covers riding entities from boats to pigs to striders
 * Built upon covering edge cases to confirm accuracy
-* 1.13+ clients on 1.13+ servers, 1.12- clients on 1.13+ servers, 1.13+ clients on 1.12- servers, and 1.12- clients on 1.12- servers are all supported regardless of the large technical changes between these versions.
+* 1.13+ clients on 1.13+ servers, 1.12- clients on 1.13+ servers, 1.13+ clients on 1.12- servers,
+  and 1.12- clients on 1.12- servers are all supported regardless of the large technical changes
+  between these versions.
 * The order of collisions depends on the client version and is correct
 * Accounts for minor bounding box differences between versions, for example:
     * Single glass panes will be a + shape for 1.7-1.8 players and * for 1.9+ players
@@ -71,7 +106,8 @@ What makes Grim stand out against other anticheats?
 * Per player, the cache allows for multithreaded design
 * Sending players fake blocks with packets is safe and does not lead to falses
 * The world is recreated for each player to allow lag compensation
-* Client sided blocks cause no issues with packet based blocks. Block glitching does not false the anticheat.
+* Client sided blocks cause no issues with packet based blocks. Block glitching does not false the
+  anticheat.
 
 ### Latency compensation
 

build.gradle.kts

@@ -1,32 +1,42 @@
 /**
- * ┏━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┓
- * ┃        GrimAC Build Configuration     ┃
- * ┗━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┛
+ *          GrimAC Build Configuration
  *
- * 🔧 Build Flags:
- * -PshadePE=true      → Enables 'lite' mode
- * -Prelocate=false    → Adds 'no_relocate' modifier
- * -Prelease=true      → Removes commit/modifiers for release build
+ * Build Flags:
+ * -PshadePE=true   - Bundles PacketEvents into the jar (default: external/lite).
+ * -Prelocate=false - Adds 'no_relocate' modifier
+ * -Prelease=true   - Removes commit/modifiers for release build
  *
  * Logic in: buildSrc/versioning/BuildConfig.kt & VersionUtil.kt
  */
 
+plugins {
+    base
+}
+
+import org.gradle.api.tasks.Copy
+import org.gradle.api.tasks.compile.JavaCompile
 import versioning.BuildConfig
 import versioning.VersionUtil
 
 BuildConfig.init(project)
 
-val baseVersion = "2.3.72"
+val baseVersion = "2.3.73"
 group = "ac.grim.grimac"
 version = VersionUtil.computeVersion(baseVersion)
-description = "Libre simulation anticheat designed for 1.21 with 1.8–1.21 support, powered by PacketEvents 2.0."
+description = "Libre simulation anticheat designed for 1.21 with 1.8–1.21 support, powered by PacketEvents 2.11.2."
 
-println("⚙️  Build configuration:")
-println("     shadePE             = ${BuildConfig.shadePE}")
-println("     relocate            = ${BuildConfig.relocate}")
-println("     mavenLocalOverride  = ${BuildConfig.mavenLocalOverride}")
-println("     release             = ${BuildConfig.release}")
-println("     version             = $version")
+ext["timestamp"] = System.currentTimeMillis().toString()
+ext["git_branch"] = VersionUtil.getGitBranch(true)
+ext["git_commit"] = VersionUtil.getGitCommitHash(true)
+ext["git_org"] = System.getenv("GRIM_GIT_ORG") ?: VersionUtil.getGitUser()
+ext["git_repo"] = System.getenv("GRIM_GIT_REPO") ?: "Grim"
+
+println("Build configuration:")
+println("    shadePE            = ${BuildConfig.shadePE}")
+println("    relocate           = ${BuildConfig.relocate}")
+println("    mavenLocalOverride = ${BuildConfig.mavenLocalOverride}")
+println("    release            = ${BuildConfig.release}")
+println("    version            = $version")
 
 tasks.register("printVersion") {
     group = "versioning"
@@ -42,4 +52,17 @@ subprojects {
         options.isFork = true
         options.isIncremental = true
     }
-}
+}
+
+tasks.register<Copy>("copyBukkitJarToRoot") {
+    dependsOn(":bukkit:shadowJar")
+    from(project(":bukkit").layout.buildDirectory.dir("libs")) {
+        include("LightningGrim-bukkit-$version.jar") // current build only; stale jars from prior commits would collide on rename
+        rename("LightningGrim-bukkit-.*\\.jar", "LightningGrim-$baseVersion.jar")
+    }
+    into(layout.buildDirectory.dir("libs"))
+}
+
+tasks.named("build") {
+    dependsOn("copyBukkitJarToRoot")
+}

buildSrc/build.gradle.kts

@@ -1,11 +1,10 @@
 plugins {
     `kotlin-dsl`
-    kotlin("plugin.serialization") version "1.9.0" // Use the same Kotlin version as your main project
+    kotlin("plugin.serialization") version "2.2.20"
 }
 
 repositories {
     mavenCentral()
-    gradlePluginPortal()
 }
 
 dependencies {

buildSrc/src/main/kotlin/exclusive.kt

@@ -0,0 +1,42 @@
+import org.gradle.api.artifacts.dsl.RepositoryHandler
+import org.gradle.api.artifacts.repositories.MavenArtifactRepository
+import org.gradle.api.artifacts.repositories.InclusiveRepositoryContentDescriptor
+import org.gradle.kotlin.dsl.maven
+import versioning.BuildConfig
+
+/**
+ * Registers a repository with strict content exclusivity.
+ *
+ * Logic:
+ * 1. If BuildConfig.mavenLocalOverride is true, checks Maven Local first.
+ * 2. Checks the provided remote URL second.
+ * 3. Enforces strict filtering so Gradle never checks this repo for other deps,
+ *    and never checks other repos (like Central) for these deps.
+ */
+fun RepositoryHandler.exclusive(
+    url: String,
+    repoConfig: MavenArtifactRepository.() -> Unit = {},
+    filterConfig: InclusiveRepositoryContentDescriptor.() -> Unit
+) {
+    // We define the repositories inside the function scope so they are added to the handler
+    val remote = maven(url, repoConfig)
+
+    exclusive(remote, filterConfig)
+}
+
+fun RepositoryHandler.exclusive(
+    remote: MavenArtifactRepository,
+    filterConfig: InclusiveRepositoryContentDescriptor.() -> Unit
+) {
+    // We access BuildConfig here again to check for local override
+    val local = if (BuildConfig.mavenLocalOverride) mavenLocal() else null
+
+    exclusiveContent {
+        if (local != null) {
+            forRepositories(local, remote)
+        } else {
+            forRepositories(remote)
+        }
+        filter(filterConfig)
+    }
+}

buildSrc/src/main/kotlin/grim.base-conventions.gradle.kts

@@ -1,3 +1,5 @@
+import versioning.BuildConfig
+
 plugins {
     `java-library`
     id("io.freefair.lombok")
@@ -16,11 +18,6 @@ java {
     withJavadocJar()
 }
 
-// Lombok configuration
-//lombok {
-//    version.set("1.18.30") // Use the version from your version catalog if available
-//}
-
 // Spotless configuration
 spotless {
     java {
@@ -38,7 +35,6 @@ spotless {
     }
 }
 
-
 tasks {
     withType<JavaCompile>().configureEach {
         options.encoding = "UTF-8"
@@ -52,11 +48,28 @@ tasks {
 
     // Process resources (e.g., for plugin metadata files)
     processResources {
-        inputs.property("version", project.version)
-        filesMatching(listOf( // "plugin.yml",  Bukkit variable expansion taken care of by net.minecrell.plugin-yml.bukkit
-            "bungee.yml", "velocity-plugin.json", "fabric.mod.json")
+        val properties = mapOf(
+            "timestamp" to rootProject.ext["timestamp"],
+            "version" to project.version.toString(),
+            "git_commit" to rootProject.ext["git_commit"],
+            "git_branch" to rootProject.ext["git_branch"],
+            "git_repo" to rootProject.ext["git_repo"],
+            "git_org" to rootProject.ext["git_org"],
+            "build_shade_pe" to BuildConfig.shadePE,
+            "build_relocate" to BuildConfig.relocate,
+            "build_release" to BuildConfig.release,
+        )
+
+        properties.forEach { (key, value) -> inputs.property(key, value) }
+
+        filesMatching(
+            listOf(
+                "bungee.yml",
+                "velocity-plugin.json",
+                "grimac.properties"
+            )
         ) {
-            expand("version" to project.version)
+            expand(properties)
         }
     }
 

buildSrc/src/main/kotlin/grim.jij-conventions.gradle.kts

@@ -95,7 +95,7 @@ fun extractEmbeddedJars(jarFile: File, project: Project): Set<DependencyIdentifi
                                     ?: throw IllegalStateException("Missing 'id' in fabric.mod.json of embedded JAR: $entryName")
                                 val idWithDots = id.replace("_", ".")
                                 val lastDotIndex = idWithDots.lastIndexOf(".")
-                                val group = if (lastDotIndex != -1) idWithDots.substring(0, lastDotIndex) else ""
+                                val group = if (lastDotIndex != -1) idWithDots.take(lastDotIndex) else ""
 
                                 // Extract name and version from the JAR filename
                                 val jarName = entryName.substringAfterLast("/").removeSuffix(".jar")
@@ -127,7 +127,7 @@ fun parseNameAndVersionFromJarName(jarName: String): Pair<String, String>? {
     val versionStartIndex = versionStartMatch.range.first
 
     // Split the JAR name into name and version at the version start index
-    val name = jarName.substring(0, versionStartIndex)
+    val name = jarName.take(versionStartIndex)
     val version = jarName.substring(versionStartIndex + 1) // Skip the hyphen
 
     return if (name.isNotEmpty() && version.isNotEmpty()) {
@@ -300,7 +300,7 @@ project.afterEvaluate {
     val allEmbeddedDependencies = mutableMapOf<DependencyIdentifier, File>()
     val actuallyIncludedDependencies = mutableSetOf<DependencyIdentifier>()
 
-    fun collectAllEmbeddedDependencies(dependencies: Set<ResolvedDependency>) {
+    fun collectAllEmbeddedDependenciesForAfterEvaluate(dependencies: Set<ResolvedDependency>) {
         dependencies.forEach { dep ->
             val classifier = dep.moduleArtifacts.firstOrNull()?.classifier ?: ""
             val depKey = "${dep.moduleGroup}:${dep.moduleName}:${dep.moduleVersion}${if (classifier.isNotEmpty()) ":$classifier" else ""}"
@@ -318,13 +318,13 @@ project.afterEvaluate {
                         )
                     }
                 }
-                collectAllEmbeddedDependencies(dep.children)
+                collectAllEmbeddedDependenciesForAfterEvaluate(dep.children)
             }
         }
     }
 
     processed.clear()
-    collectAllEmbeddedDependencies(resolvedDependencies)
+    collectAllEmbeddedDependenciesForAfterEvaluate(resolvedDependencies)
 
     processed.clear()
     processDependencies(
@@ -345,7 +345,7 @@ tasks.withType<Jar>().configureEach {
         val allEmbeddedDependencies = mutableMapOf<DependencyIdentifier, File>()
         val actuallyIncludedDependencies = mutableSetOf<DependencyIdentifier>()
 
-        fun collectAllEmbeddedDependencies(dependencies: Set<ResolvedDependency>) {
+        fun collectAllEmbeddedDependenciesForJar(dependencies: Set<ResolvedDependency>) {
             dependencies.forEach { dep ->
                 val depKey = "${dep.moduleGroup}:${dep.moduleName}:${dep.moduleVersion}"
                 if (!processed.contains(depKey)) {
@@ -362,13 +362,13 @@ tasks.withType<Jar>().configureEach {
                             )
                         }
                     }
-                    collectAllEmbeddedDependencies(dep.children)
+                    collectAllEmbeddedDependenciesForJar(dep.children)
                 }
             }
         }
 
         processed.clear()
-        collectAllEmbeddedDependencies(resolvedDependencies)
+        collectAllEmbeddedDependenciesForJar(resolvedDependencies)
 
         processed.clear()
 

buildSrc/src/main/kotlin/grim.shadow-conventions.gradle.kts

@@ -5,17 +5,16 @@ plugins {
     id("com.gradleup.shadow")
 }
 
-// Create a configuration for shading dependencies from common
-val shadowCommon: Configuration by project.configurations.creating {
-    isCanBeResolved = true
-    isCanBeConsumed = false
-    extendsFrom(project.configurations.getByName("implementation"))
-}
-
 tasks.named<ShadowJar>("shadowJar") {
-    minimize()
-    archiveFileName.set("${rootProject.name}-${project.name}-${rootProject.version}.jar")
-    from(shadowCommon) // Use from() instead of direct assignment
+    minimize {
+        // The PostgreSQL JDBC driver is loaded reflectively via java.sql ServiceLoader,
+        // so minimize() sees no direct references and would strip it. Keep it whole.
+        exclude(dependency("org.postgresql:postgresql:.*"))
+        // Same for the SLF4J NOP provider, loaded via SLF4JServiceProvider ServiceLoader.
+        exclude(dependency("org.slf4j:slf4j-nop:.*"))
+    }
+    archiveFileName = "${rootProject.name}-${project.name}-${rootProject.version}.jar"
+    duplicatesStrategy = DuplicatesStrategy.EXCLUDE
 
     if (BuildConfig.relocate) {
         if (BuildConfig.shadePE) {
@@ -38,6 +37,8 @@ tasks.named<ShadowJar>("shadowJar") {
         relocate("org.jetbrains", "ac.grim.grimac.shaded.jetbrains")
         relocate("org.incendo", "ac.grim.grimac.shaded.incendo")
         relocate("io.leangen.geantyref", "ac.grim.grimac.shaded.geantyref") // Required by cloud
+        relocate("com.zaxxer", "ac.grim.grimac.shaded.zaxxer") // Database history
+        relocate("org.postgresql", "ac.grim.grimac.shaded.postgresql") // Database history (JDBC driver); service file rewritten by mergeServiceFiles
     }
     mergeServiceFiles()
 }

buildSrc/src/main/kotlin/versioning/BuildConfig.kt

@@ -2,6 +2,11 @@ package versioning
 
 import org.gradle.api.Project
 import org.gradle.internal.extensions.stdlib.toDefaultLowerCase
+import versioning.BuildConfig.init
+import versioning.BuildConfig.mavenLocalOverride
+import versioning.BuildConfig.release
+import versioning.BuildConfig.relocate
+import versioning.BuildConfig.shadePE
 
 /**
  * BuildConfig provides access to user-defined build flags that control how a Grim
@@ -10,9 +15,9 @@ import org.gradle.internal.extensions.stdlib.toDefaultLowerCase
  *
  * Flags can be defined in three ways:
  *
- *   ① JVM system properties  (-Dflag=value)
- *   ② Gradle project properties  (-Pflag=value or in gradle.properties)
- *   ③ Environment variables  (FLAG=value)
+ *   1. JVM system properties  (-Dflag=value)
+ *   2. Gradle project properties  (-Pflag=value or in gradle.properties)
+ *   3. Environment variables  (FLAG=value)
  *
  * You can use these to enable/disable features like shading, relocation, or release mode.
  *
@@ -33,7 +38,7 @@ import org.gradle.internal.extensions.stdlib.toDefaultLowerCase
  * ./gradlew build -DshadePE=true -Drelease=true
  * ```
  *
- * @property shadePE  If true, shades PacketEvents into the jar. Default: true.
+ * @property shadePE  Bundles PacketEvents into the jar when true; consumes an external provider when false (default; emits the `-lite` artifact).
  * @property relocate If true, relocates shaded dependencies to avoid conflicts. Default: true.
  * @property release  If true, omits commit hash and modifiers from version string. Default: false.
  * @property mavenLocalOverride If true, will make artifacts in mavenLocal() will be used instead of their remote counterparts for this build. Default: false
@@ -48,32 +53,23 @@ object BuildConfig {
      * ```
      */
     fun init(project: Project) {
-        _shadePE = resolveBool(project, "shadePE", altKey = "SHADE_PE", default = true)
+        _shadePE = resolveBool(project, "shadePE", altKey = "SHADE_PE", default = false)
         _relocate = resolveBool(project, "relocate", altKey = "RELOCATE_JAR", default = true)
         _release = resolveBool(project, "release", default = false)
-        _mavenLocalOverride = resolveBool(project, "mavenLocalOverride", default = false)
+        _mavenLocalOverride = resolveBool(project, "mavenLocalOverride", altKey = "MAVEN_LOCAL_OVERRIDE", default = false)
     }
 
     // Unified resolution logic (System > Gradle > Env)
     private fun resolveRaw(project: Project, key: String): String? =
-        System.getProperty(key)                       // ① JVM   (-Dkey=value)
-            ?: project.findProperty(key)?.toString()  // ② Gradle (-Pkey=value or gradle.properties)
-            ?: System.getenv(key.uppercase())         // ③ ENV   (KEY=value)
+        System.getProperty(key)                      // 1. JVM   (-Dkey=value)
+            ?: project.findProperty(key)?.toString() // 2. Gradle (-Pkey=value or gradle.properties)
+            ?: System.getenv(key.uppercase())        // 3. ENV   (KEY=value)
 
     private fun resolveBool(project: Project, key: String, altKey: String? = null, default: Boolean): Boolean {
-        val primaryValue = resolveRaw(project, key)?.toDefaultLowerCase()?.toBooleanStrictOrNull()
-        if (primaryValue != null) {
-            return primaryValue
-        }
-
-        if (altKey != null) {
-            val altValue = resolveRaw(project, altKey)?.toDefaultLowerCase()?.toBooleanStrictOrNull()
-            if (altValue != null) {
-                return altValue
-            }
-        }
-
-        return default
+        return resolveRaw(project, key)?.toDefaultLowerCase()?.toBooleanStrictOrNull()
+            ?: altKey?.let {
+                resolveRaw(project, it)?.toDefaultLowerCase()?.toBooleanStrictOrNull()
+            } ?: default
     }
 
     // Private backing vars (nullable because we can't use lateinit with primitives)
@@ -82,7 +78,7 @@ object BuildConfig {
     private var _release: Boolean? = null
     private var _mavenLocalOverride: Boolean? = null
 
-    /** If true, shades PacketEvents into the jar. Default: true. */
+    /** If true, shades PacketEvents into the jar. Default: false (external provider). */
     val shadePE: Boolean get() = _shadePE
         ?: error("BuildConfig.shadePE accessed before init() was called")
 
@@ -96,4 +92,5 @@ object BuildConfig {
 
     val mavenLocalOverride: Boolean get() = _mavenLocalOverride
         ?: error("BuildConfig.release accessed before init() was called")
-}
+
+}

buildSrc/src/main/kotlin/versioning/VersionUtil.kt

@@ -3,7 +3,7 @@ package versioning
 import java.io.ByteArrayOutputStream
 
 /**
- * Utility for computing the version string of GrimAC artifacts.
+ * Utility for computing the version string of LightningGrim artifacts.
  *
  * The version string is constructed based on:
  * - A base semantic version (e.g., "2.3.72")
@@ -37,29 +37,31 @@ object VersionUtil {
         val modifiers = buildList {
             if (!BuildConfig.shadePE) add("lite")
             if (!BuildConfig.relocate) add("no_relocate")
-        }.takeIf { it.isNotEmpty() }?.joinToString("-")
+        }.joinToString("-").takeIf { it.isNotEmpty() }
 
         return buildString {
             append(baseVersion)
             append("-")
             branch?.let { append("$it-") }
             append(commitHash)
-            modifiers?.let { append("+$modifiers") }
+            modifiers?.let { append("+$it") }
         }
     }
 
     /**
-     * Retrieves the current Git commit as a short hash.
+     * Retrieves the current Git commit
      */
-    private fun getGitCommitHash(): String {
+    fun getGitCommitHash(full: Boolean = false): String {
         val stdout = ByteArrayOutputStream()
-        ProcessBuilder("git", "rev-parse", "--short", "HEAD")
+        val command = listOfNotNull("git", "rev-parse", if (full) null else "--short", "HEAD")
+        ProcessBuilder(command)
             .redirectErrorStream(true)
             .start()
             .apply { waitFor() }
             .inputStream
             .use { stdout.writeBytes(it.readAllBytes()) }
-        return stdout.toString().trim()
+        val fullCommit = stdout.toString().trim()
+        return fullCommit.take(minOf(fullCommit.length, 7))
     }
 
     /**
@@ -69,7 +71,7 @@ object VersionUtil {
      * Any slash (/) in the branch name is replaced with an underscore (_)
      * to avoid filesystem issues.
      */
-    private fun getGitBranch(): String? {
+    fun getGitBranch(raw: Boolean = false): String? {
         val stdout = ByteArrayOutputStream()
 
         ProcessBuilder("git", "rev-parse", "--abbrev-ref", "HEAD")
@@ -78,11 +80,35 @@ object VersionUtil {
             .apply { waitFor() }
             .inputStream.use { stdout.writeBytes(it.readAllBytes()) }
 
+        if (raw) return stdout.toString().trim()
+
         val branch = stdout.toString().trim()
+            .replace(Regex("[^a-zA-Z0-9_.-]+"), "_")
+            .replace(Regex("_{2,}"), "_")
+            .trim(' ', '.', '_', '-')
+            .removePrefix("heads_")
+
+        val mainBranch = System.getenv("GRIM_MAIN_BRANCH") ?: "2.0"
 
         return when (branch) {
-            "main", "2.0" -> null                    // ← ignore these branches
-            else           -> branch.replace("/", "_")
+            "lightning" -> null
+            "main", mainBranch -> null                    // ← ignore these branches
+            else -> branch
         }
     }
-}
+
+    fun getGitUser(): String {
+        try {
+            val stdout = ByteArrayOutputStream()
+            ProcessBuilder("git", "config", "user.name")
+                .redirectErrorStream(true)
+                .start()
+                .apply { waitFor() }
+                .inputStream.use { stdout.writeBytes(it.readAllBytes()) }
+            return stdout.toString().trim()
+        } catch (_: Exception) {
+            return "unknown"
+        }
+    }
+
+}

bukkit/build.gradle.kts

@@ -5,31 +5,41 @@ plugins {
     `maven-publish`
     grim.`base-conventions`
     grim.`shadow-conventions`
-    id("net.minecrell.plugin-yml.bukkit") version "0.6.0"
+    id("de.eldoria.plugin-yml.bukkit") version "0.8.0"
+    id("xyz.jpenilla.run-paper") version "3.0.0-beta.1"
 }
 
 repositories {
-    if (BuildConfig.mavenLocalOverride) {
-        mavenLocal()
+    // 1. Fallback for non-exclusive deps (e.g. Maven Central deps)
+    if (BuildConfig.mavenLocalOverride) mavenLocal()
+
+    // 2. Exclusive Repositories (One HTTP request per dep)
+    exclusive("https://repo.papermc.io/repository/maven-public/", { name = "papermc" }) {
+        includeGroup("io.papermc.paper")
+        includeGroup("net.md-5")
     }
-    maven {
-        name = "papermc"
-        url = uri("https://repo.papermc.io/repository/maven-public/")
+
+    exclusive("https://libraries.minecraft.net", { mavenContent { releasesOnly() } }) {
+        includeModule("com.mojang", "brigadier")
     }
-    maven("https://hub.spigotmc.org/nexus/content/repositories/snapshots/") // Spigot
-    maven("https://repo.grim.ac/snapshots") { // Grim API
-        content {
-            includeGroup("ac.grim.grimac")
-            includeGroup("com.github.retrooper")
-        }
+
+    exclusive("https://repo.extendedclip.com/content/repositories/placeholderapi/") {
+        includeGroup("me.clip")
     }
-    maven("https://repo.viaversion.com") // ViaVersion
-    maven("https://nexus.scarsz.me/content/repositories/releases") // Configuralize
-    maven("https://repo.opencollab.dev/maven-snapshots/") // Floodgate
-    maven("https://repo.opencollab.dev/maven-releases/") // Cumulus (for Floodgate)
-    maven("https://s01.oss.sonatype.org/content/repositories/snapshots/")
-    maven("https://repo.extendedclip.com/content/repositories/placeholderapi/") // placeholderapi
-    mavenCentral() // FastUtil
+
+    exclusive("https://repo.codemc.io/repository/maven-releases/", { mavenContent { releasesOnly() } }) {
+        includeGroup("com.github.retrooper")
+    }
+
+    exclusive("https://repo.grim.ac/snapshots") {
+        includeGroup("ac.grim.grimac")
+    }
+
+    exclusive("https://nexus.scarsz.me/content/repositories/releases", { mavenContent { releasesOnly() } }) {
+        includeGroup("github.scarsz")
+    }
+
+    mavenCentral()
 }
 
 dependencies {
@@ -42,22 +52,24 @@ dependencies {
         compileOnly(libs.packetevents.spigot)
     }
     implementation(libs.cloud.paper)
-    implementation(libs.adventure.platform.bukkit)
-
+    implementation(libs.grim.bukkit.internal)
+    implementation("org.slf4j:slf4j-nop:2.0.17") // Provide SLF4J NOP provider to suppress missing-provider warning.
+    implementation("org.slf4j:slf4j-api:2.0.17") // Bundle a relocated SLF4J API for shaded dependencies.
     implementation(project(":common"))
     shadow(project(":common"))
 }
 
 bukkit {
-    name = "GrimAC"
-    author = "GrimAC"
+    name = "LightningGrim"
+    author = "LightningGrim"
     main = "ac.grim.grimac.platform.bukkit.GrimACBukkitLoaderPlugin"
     website = "https://grim.ac/"
     apiVersion = "1.13"
     foliaSupported = true
 
     if (!BuildConfig.shadePE) {
-        depend = listOf("packetevents")
+        // Bukkit `depend` is case-sensitive; canonical PE plugin name is PascalCase.
+        depend = listOf("PacketEvents")
     }
 
     softDepend = listOf(
@@ -71,6 +83,7 @@ bukkit {
         "floodgate",
         "FastLogin",
         "PlaceholderAPI",
+        "ZNPCsPlus",
     )
 
     permissions {
@@ -125,24 +138,49 @@ bukkit {
         }
 
         register("grim.verbose") {
-            description = "Receive verbose alerts for violations. Requires grim.alerts"
+            description = "Receive verbose alerts for violations"
             default = Permission.Default.OP
         }
 
         register("grim.verbose.enable-on-join") {
             description =
-                "Enable verbose alerts on join. Requires grim.alerts and grim.alerts.enable-on-join"
+                "Enable verbose alerts on join"
+            default = Permission.Default.FALSE
+        }
+
+        register("grim.list") {
+            description =
+                "Shows lists of specific data"
             default = Permission.Default.FALSE
         }
     }
 }
 
 publishing.publications.create<MavenPublication>("maven") {
-    artifact(tasks["shadowJar"])
+    artifact(tasks.named("shadowJar"))
 }
 
-tasks.shadowJar {
-    manifest {
-        attributes["paperweight-mappings-namespace"] = "mojang"
+apply(from = "$projectDir/gradle/packetevents-provider.gradle.kts")
+
+@Suppress("UNCHECKED_CAST")
+val registerStagePE = project.extra["registerStagePacketEventsTask"]
+    as (org.gradle.api.Project, String, java.io.File) -> org.gradle.api.tasks.TaskProvider<Copy>
+
+val stagePEJar = registerStagePE(project, "stageLocalPacketEvents", file("run"))
+
+tasks {
+    runServer {
+        minecraftVersion("1.21.11")
+        // External mode stages the locally built True-OG PacketEvents JAR.
+        if (!BuildConfig.shadePE) {
+            dependsOn(stagePEJar)
+        }
+    }
+
+    shadowJar {
+        mergeServiceFiles()
+        manifest {
+            attributes["paperweight-mappings-namespace"] = "mojang"
+        }
     }
 }

bukkit/gradle/packetevents-provider.gradle.kts

@@ -0,0 +1,76 @@
+// True-OG PacketEvents JAR staging convention.
+// Resolution order (first hit wins): -PtruogPacketEventsJar, TRUEOG_PACKETEVENTS_JAR env, default glob under ~/eclipse-workspace/PacketEvents/spigot/build/libs (case-insensitive, latest by mtime). See context/refs/packetevents-provider.md.
+
+import java.nio.file.Files
+import java.nio.file.Path
+import java.nio.file.Paths
+import java.util.Comparator
+
+fun resolveTrueOgPacketEventsJar(project: Project): File {
+    val home = System.getProperty("user.home")
+    val defaultDir = Paths.get(home, "eclipse-workspace", "PacketEvents", "spigot", "build", "libs")
+
+    // Every resolution path is reported in the failure message regardless of which got tried.
+    val propValue = (project.findProperty("truogPacketEventsJar") as? String)?.takeIf { it.isNotBlank() } ?: "<unset>"
+    val envValue = System.getenv("TRUEOG_PACKETEVENTS_JAR")?.takeIf { it.isNotBlank() } ?: "<unset>"
+    val attempts = listOf(
+        "Gradle property truogPacketEventsJar=$propValue",
+        "Environment variable TRUEOG_PACKETEVENTS_JAR=$envValue",
+        "Default glob $defaultDir/PacketEvents-*.jar (case-insensitive; falls back to packetevents-spigot-*.jar; latest by mtime)",
+    )
+
+    val gradleProp = (project.findProperty("truogPacketEventsJar") as? String)?.trim()
+    if (!gradleProp.isNullOrEmpty()) {
+        val f = File(gradleProp)
+        if (f.isFile) return f
+    }
+
+    val envVar = System.getenv("TRUEOG_PACKETEVENTS_JAR")?.trim()
+    if (!envVar.isNullOrEmpty()) {
+        val f = File(envVar)
+        if (f.isFile) return f
+    }
+
+    if (Files.isDirectory(defaultDir)) {
+        val nameRegex = Regex("(?i)(PacketEvents|packetevents-spigot|packetevents)-.*\\.jar")
+        val match = Files.list(defaultDir).use { stream ->
+            stream
+                .filter { nameRegex.matches(it.fileName.toString()) }
+                .sorted(Comparator.comparingLong<Path> { Files.getLastModifiedTime(it).toMillis() }.reversed())
+                .findFirst()
+                .orElse(null)
+        }
+        if (match != null) return match.toFile()
+    }
+
+    throw GradleException(
+        buildString {
+            appendLine("True-OG PacketEvents JAR not found. Tried:")
+            attempts.forEach { appendLine("  - $it") }
+            appendLine("Provide one of:")
+            appendLine("  ./gradlew -PtruogPacketEventsJar=/abs/path/to/PacketEvents.jar ...")
+            appendLine("  TRUEOG_PACKETEVENTS_JAR=/abs/path/to/PacketEvents.jar ./gradlew ...")
+            appendLine("  Build the fork at \${user.home}/eclipse-workspace/PacketEvents (default location).")
+        },
+    )
+}
+
+
+// Registers a Copy task that stages the resolved JAR into runDir/plugins/PacketEvents.jar. See context/refs/packetevents-provider.md for the consumer integration snippet.
+val registerStagePacketEventsTask: (Project, String, File) -> TaskProvider<Copy> = { p, taskName, runDir ->
+    p.tasks.register(taskName, Copy::class.java) {
+        group = "packetevents"
+        description = "Stage the locally built True-OG PacketEvents JAR into plugins/."
+        // Eager filesystem resolution at configuration time is incompatible with the config cache.
+        notCompatibleWithConfigurationCache(
+            "Resolves the JAR via filesystem inspection during configuration.",
+        )
+        val jar = resolveTrueOgPacketEventsJar(p)
+        from(jar)
+        into(runDir.resolve("plugins"))
+        rename { "PacketEvents.jar" }
+    }
+}
+
+extra["resolveTrueOgPacketEventsJar"] = { p: Project -> resolveTrueOgPacketEventsJar(p) }
+extra["registerStagePacketEventsTask"] = registerStagePacketEventsTask

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/BukkitPlatformPlugin.java

@@ -20,6 +20,7 @@ public class BukkitPlatformPlugin implements PlatformPlugin {
         return plugin.getName();
     }
 
+    @SuppressWarnings("deprecation")
     @Override
     public String getVersion() {
         return plugin.getDescription().getVersion();

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/BukkitPlatformServer.java

@@ -1,5 +1,7 @@
 package ac.grim.grimac.platform.bukkit;
 
+import ac.grim.grimac.GrimAPI;
+import ac.grim.grimac.platform.api.Platform;
 import ac.grim.grimac.platform.api.PlatformServer;
 import ac.grim.grimac.platform.api.sender.Sender;
 import io.github.retrooper.packetevents.util.SpigotReflectionUtil;
@@ -7,6 +9,7 @@ import org.bukkit.Bukkit;
 import org.bukkit.command.CommandSender;
 
 
+
 public class BukkitPlatformServer implements PlatformServer {
 
     @Override
@@ -32,6 +35,10 @@ public class BukkitPlatformServer implements PlatformServer {
 
     @Override
     public double getTPS() {
+        // Folia throws UnsupportedOperationException on calling getTPS(), there is no API for getting TPS on Folia
+        if (GrimAPI.INSTANCE.getPlatform() == Platform.FOLIA) {
+            return Double.NaN;
+        }
         return SpigotReflectionUtil.getTPS();
     }
 }

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/GrimACBukkitLoaderPlugin.java

@@ -1,21 +1,25 @@
 package ac.grim.grimac.platform.bukkit;
 
-import ac.grim.grimac.api.plugin.BasicGrimPlugin;
 import ac.grim.grimac.GrimAPI;
+import ac.grim.grimac.GrimExternalAPI;
 import ac.grim.grimac.api.GrimAPIProvider;
 import ac.grim.grimac.api.GrimAbstractAPI;
+import ac.grim.grimac.api.event.EventBus;
 import ac.grim.grimac.api.plugin.GrimPlugin;
+import ac.grim.grimac.command.CloudCommandService;
+import ac.grim.grimac.internal.platform.bukkit.resolver.BukkitResolverRegistrar;
 import ac.grim.grimac.manager.init.Initable;
+import ac.grim.grimac.manager.init.load.PacketEventsInit;
 import ac.grim.grimac.manager.init.start.ExemptOnlinePlayersOnReload;
 import ac.grim.grimac.manager.init.start.StartableInitable;
 import ac.grim.grimac.platform.api.Platform;
 import ac.grim.grimac.platform.api.PlatformLoader;
 import ac.grim.grimac.platform.api.PlatformServer;
+import ac.grim.grimac.platform.api.command.CommandService;
 import ac.grim.grimac.platform.api.manager.ItemResetHandler;
 import ac.grim.grimac.platform.api.manager.MessagePlaceHolderManager;
-import ac.grim.grimac.platform.api.manager.ParserDescriptorFactory;
-import ac.grim.grimac.platform.api.manager.PermissionRegistrationManager;
 import ac.grim.grimac.platform.api.manager.PlatformPluginManager;
+import ac.grim.grimac.platform.api.manager.cloud.CloudCommandAdapter;
 import ac.grim.grimac.platform.api.player.PlatformPlayerFactory;
 import ac.grim.grimac.platform.api.scheduler.PlatformScheduler;
 import ac.grim.grimac.platform.api.sender.Sender;
@@ -33,9 +37,11 @@ import ac.grim.grimac.platform.bukkit.scheduler.bukkit.BukkitPlatformScheduler;
 import ac.grim.grimac.platform.bukkit.scheduler.folia.FoliaPlatformScheduler;
 import ac.grim.grimac.platform.bukkit.sender.BukkitSenderFactory;
 import ac.grim.grimac.platform.bukkit.utils.placeholder.PlaceholderAPIExpansion;
+import ac.grim.grimac.utils.anticheat.LogUtil;
 import ac.grim.grimac.utils.lazy.LazyHolder;
 import com.github.retrooper.packetevents.PacketEventsAPI;
 import io.github.retrooper.packetevents.factory.spigot.SpigotPacketEventsBuilder;
+import lombok.Getter;
 import org.bukkit.Bukkit;
 import org.bukkit.command.CommandSender;
 import org.bukkit.plugin.ServicePriority;
@@ -46,35 +52,36 @@ import org.incendo.cloud.brigadier.CloudBrigadierManager;
 import org.incendo.cloud.bukkit.CloudBukkitCapabilities;
 import org.incendo.cloud.execution.ExecutionCoordinator;
 import org.incendo.cloud.paper.LegacyPaperCommandManager;
-import org.incendo.cloud.setting.Configurable;
-import org.jetbrains.annotations.NotNull;
+
 
 public final class GrimACBukkitLoaderPlugin extends JavaPlugin implements PlatformLoader {
 
     public static GrimACBukkitLoaderPlugin LOADER;
 
     private final LazyHolder<PlatformScheduler> scheduler = LazyHolder.simple(this::createScheduler);
-    private final LazyHolder<PacketEventsAPI<?>> packetEvents = LazyHolder.simple(() -> SpigotPacketEventsBuilder.build(this));
+    private final LazyHolder<PacketEventsAPI<?>> packetEvents = LazyHolder.simple(() -> {
+        // External mode reuses the installed provider's API (plugin.yml depend orders load before this).
+        if (PacketEventsInit.isShadePE()) {
+            return SpigotPacketEventsBuilder.build(this);
+        }
+        return com.github.retrooper.packetevents.PacketEvents.getAPI();
+    });
     private final LazyHolder<BukkitSenderFactory> senderFactory = LazyHolder.simple(BukkitSenderFactory::new);
-    private final LazyHolder<CommandManager<Sender>> commandManager = LazyHolder.simple(this::createCommandManager);
     private final LazyHolder<ItemResetHandler> itemResetHandler = LazyHolder.simple(BukkitItemResetHandler::new);
+    private final LazyHolder<CommandService> commandService = LazyHolder.simple(this::createCommandService);
+    private final CloudCommandAdapter commandAdapter = new BukkitParserDescriptorFactory();
 
-    private final PlatformPlayerFactory playerFactory = new BukkitPlatformPlayerFactory();
-    private final ParserDescriptorFactory parserFactory = new BukkitParserDescriptorFactory();
-    private final PlatformPluginManager platformPluginManager = new BukkitPlatformPluginManager();
-    private final GrimPlugin plugin;
-    private final PlatformServer platformServer = new BukkitPlatformServer();
-    private final MessagePlaceHolderManager messagePlaceHolderManager = new BukkitMessagePlaceHolderManager();
-    private final BukkitPermissionRegistrationManager bukkitPermissionRegistrationManager = new BukkitPermissionRegistrationManager();
+    @Getter private final PlatformPlayerFactory platformPlayerFactory = new BukkitPlatformPlayerFactory();
+    @Getter private final PlatformPluginManager pluginManager = new BukkitPlatformPluginManager();
+    @Getter private final GrimPlugin plugin;
+    @Getter private final PlatformServer platformServer = new BukkitPlatformServer();
+    @Getter private final MessagePlaceHolderManager messagePlaceHolderManager = new BukkitMessagePlaceHolderManager();
+    @Getter private final BukkitPermissionRegistrationManager permissionManager = new BukkitPermissionRegistrationManager();
 
     public GrimACBukkitLoaderPlugin() {
-        this.plugin = new BasicGrimPlugin(
-                this.getLogger(),
-                this.getDataFolder(),
-                this.getDescription().getVersion(),
-                this.getDescription().getDescription(),
-                this.getDescription().getAuthors()
-        );
+        BukkitResolverRegistrar registrar = new BukkitResolverRegistrar();
+        registrar.registerAll(GrimAPI.INSTANCE.getExtensionManager());
+        this.plugin = registrar.resolvePlugin(this);
     }
 
     @Override
@@ -112,82 +119,133 @@ public final class GrimACBukkitLoaderPlugin extends JavaPlugin implements Platfo
         return scheduler.get();
     }
 
-    @Override
-    public PlatformPlayerFactory getPlatformPlayerFactory() {
-        return playerFactory;
-    }
-
-    @Override
-    public ParserDescriptorFactory getParserDescriptorFactory() {
-        return parserFactory;
-    }
-
     @Override
     public PacketEventsAPI<?> getPacketEvents() {
         return packetEvents.get();
     }
 
-    @Override
-    public CommandManager<Sender> getCommandManager() {
-        return commandManager.get();
-    }
-
     @Override
     public ItemResetHandler getItemResetHandler() {
         return itemResetHandler.get();
     }
 
+    @Override
+    public CommandService getCommandService() {
+        return commandService.get();
+    }
+
     @Override
     public SenderFactory<CommandSender> getSenderFactory() {
         return senderFactory.get();
     }
 
-    @Override
-    public GrimPlugin getPlugin() {
-        return plugin;
-    }
-
-    @Override
-    public PlatformPluginManager getPluginManager() {
-        return platformPluginManager;
-    }
-
-    @Override
-    public PlatformServer getPlatformServer() {
-        return platformServer;
-    }
-
     @Override
     public void registerAPIService() {
-        GrimAPIProvider.init(GrimAPI.INSTANCE.getExternalAPI());
-        Bukkit.getServicesManager().register(GrimAbstractAPI.class, GrimAPI.INSTANCE.getExternalAPI(), GrimACBukkitLoaderPlugin.LOADER, ServicePriority.Normal);
-    }
+        final GrimExternalAPI externalAPI = GrimAPI.INSTANCE.getExternalAPI();
+        final EventBus eventBus = externalAPI.getEventBus();
+        final ac.grim.grimac.api.plugin.GrimPlugin context = GrimAPI.INSTANCE.getGrimPlugin();
 
-    @Override
-    public @NotNull MessagePlaceHolderManager getMessagePlaceHolderManager() {
-        return messagePlaceHolderManager;
-    }
+        eventBus.subscribe(context, ac.grim.grimac.api.event.events.GrimJoinEvent.class, (event) -> {
+            ac.grim.grimac.api.events.GrimJoinEvent bukkitEvent =
+                    new ac.grim.grimac.api.events.GrimJoinEvent(event.getUser());
 
-    @Override
-    public PermissionRegistrationManager getPermissionManager() {
-        return bukkitPermissionRegistrationManager;
+            Bukkit.getPluginManager().callEvent(bukkitEvent);
+        });
+
+        eventBus.subscribe(context, ac.grim.grimac.api.event.events.GrimQuitEvent.class, (event) -> {
+            ac.grim.grimac.api.events.GrimQuitEvent bukkitEvent =
+                    new ac.grim.grimac.api.events.GrimQuitEvent(event.getUser());
+
+            Bukkit.getPluginManager().callEvent(bukkitEvent);
+        });
+
+        eventBus.subscribe(context, ac.grim.grimac.api.event.events.GrimReloadEvent.class, (event) -> {
+            ac.grim.grimac.api.events.GrimReloadEvent bukkitEvent =
+                    new ac.grim.grimac.api.events.GrimReloadEvent(event.isSuccess());
+
+            Bukkit.getPluginManager().callEvent(bukkitEvent);
+        });
+
+        eventBus.subscribe(context, ac.grim.grimac.api.event.events.FlagEvent.class, (event) -> {
+            ac.grim.grimac.api.events.FlagEvent bukkitEvent =
+                    new ac.grim.grimac.api.events.FlagEvent(
+                            event.getUser(),
+                            event.getCheck(),
+                            event.getVerbose()
+                    );
+
+            Bukkit.getPluginManager().callEvent(bukkitEvent);
+
+            if (bukkitEvent.isCancelled()) {
+                event.setCancelled(true);
+            }
+        });
+
+        eventBus.subscribe(context, ac.grim.grimac.api.event.events.CommandExecuteEvent.class, (event) -> {
+            ac.grim.grimac.api.events.CommandExecuteEvent bukkitEvent =
+                    new ac.grim.grimac.api.events.CommandExecuteEvent(
+                            event.getUser(),
+                            event.getCheck(),
+                            event.getVerbose(),
+                            event.getCommand()
+                    );
+
+            Bukkit.getPluginManager().callEvent(bukkitEvent);
+
+            if (bukkitEvent.isCancelled()) {
+                event.setCancelled(true);
+            }
+        });
+
+        eventBus.subscribe(context, ac.grim.grimac.api.event.events.CompletePredictionEvent.class, (event) -> {
+            // Note: New event doesn't have verbose, passing null or check name is standard fallback
+            ac.grim.grimac.api.events.CompletePredictionEvent bukkitEvent =
+                    new ac.grim.grimac.api.events.CompletePredictionEvent(
+                            event.getUser(),
+                            event.getCheck(),
+                            "",
+                            event.getOffset()
+                    );
+
+            Bukkit.getPluginManager().callEvent(bukkitEvent);
+
+            if (bukkitEvent.isCancelled()) {
+                event.setCancelled(true);
+            }
+        });
+
+        GrimAPIProvider.init(externalAPI);
+        Bukkit.getServicesManager().register(GrimAbstractAPI.class, externalAPI, GrimACBukkitLoaderPlugin.LOADER, ServicePriority.Normal);
     }
 
     private PlatformScheduler createScheduler() {
         return GrimAPI.INSTANCE.getPlatform() == Platform.FOLIA ? new FoliaPlatformScheduler() : new BukkitPlatformScheduler();
     }
 
-    private CommandManager<Sender> createCommandManager() {
+    private CommandService createCommandService() {
+        try {
+            return new CloudCommandService(this::createCloudCommandManager, commandAdapter);
+        } catch (Throwable t) {
+            LogUtil.warn("CRITICAL: Failed to initialize Command Framework. " +
+                    "Grim will continue to run with no commands.", t);
+            return () -> {};
+        }
+    }
+
+    private CommandManager<Sender> createCloudCommandManager() {
         LegacyPaperCommandManager<Sender> manager = new LegacyPaperCommandManager<>(
                 this,
                 ExecutionCoordinator.simpleCoordinator(),
                 senderFactory.get()
         );
         if (manager.hasCapability(CloudBukkitCapabilities.NATIVE_BRIGADIER)) {
-            manager.registerBrigadier();
-            CloudBrigadierManager<Sender, ?> cbm = manager.brigadierManager();
-            Configurable<BrigadierSetting> settings = cbm.settings();
-            settings.set(BrigadierSetting.FORCE_EXECUTABLE, true);
+            try {
+                manager.registerBrigadier();
+                CloudBrigadierManager<Sender, ?> cbm = manager.brigadierManager();
+                cbm.settings().set(BrigadierSetting.FORCE_EXECUTABLE, true);
+            } catch (Throwable t) {
+                LogUtil.error("Failed to register Brigadier native completions. Falling back to standard completions.", t);
+            }
         } else if (manager.hasCapability(CloudBukkitCapabilities.ASYNCHRONOUS_COMPLETION)) {
             manager.registerAsynchronousCompletions();
         }

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/command/BukkitPlayerSelectorAdapter.java

@@ -4,16 +4,15 @@ import ac.grim.grimac.GrimAPI;
 import ac.grim.grimac.platform.api.command.PlayerSelector;
 import ac.grim.grimac.platform.api.sender.Sender;
 import ac.grim.grimac.platform.bukkit.sender.BukkitSenderFactory;
+import lombok.RequiredArgsConstructor;
+import org.incendo.cloud.bukkit.data.SinglePlayerSelector;
 
 import java.util.Collection;
 import java.util.Collections;
 
+@RequiredArgsConstructor
 public class BukkitPlayerSelectorAdapter implements PlayerSelector {
-    private final org.incendo.cloud.bukkit.data.SinglePlayerSelector bukkitSelector;
-
-    public BukkitPlayerSelectorAdapter(org.incendo.cloud.bukkit.data.SinglePlayerSelector bukkitSelector) {
-        this.bukkitSelector = bukkitSelector;
-    }
+    private final SinglePlayerSelector bukkitSelector;
 
     @Override
     public boolean isSingle() {

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/entity/BukkitGrimEntity.java

@@ -6,8 +6,10 @@ import ac.grim.grimac.platform.bukkit.utils.convert.BukkitConversionUtils;
 import ac.grim.grimac.platform.bukkit.utils.reflection.PaperUtils;
 import ac.grim.grimac.platform.bukkit.world.BukkitPlatformWorld;
 import ac.grim.grimac.utils.math.Location;
+import com.github.retrooper.packetevents.PacketEvents;
+import com.github.retrooper.packetevents.manager.server.ServerVersion;
 import org.bukkit.entity.Entity;
-import org.checkerframework.checker.nullness.qual.NonNull;
+import org.jetbrains.annotations.NotNull;
 
 import java.util.Objects;
 import java.util.UUID;
@@ -15,12 +17,13 @@ import java.util.concurrent.CompletableFuture;
 
 public class BukkitGrimEntity implements GrimEntity {
 
+    protected static final boolean CAN_USE_DIRECT_GETTERS = PacketEvents.getAPI().getServerManager().getVersion().isNewerThan(ServerVersion.V_1_20_1) && PaperUtils.PAPER;
+
     private final Entity entity;
     private BukkitPlatformWorld bukkitPlatformWorld;
 
     public BukkitGrimEntity(Entity entity) {
-        Objects.requireNonNull(entity);
-        this.entity = entity;
+        this.entity = Objects.requireNonNull(entity);
     }
 
     public Entity getBukkitEntity() {
@@ -44,7 +47,7 @@ public class BukkitGrimEntity implements GrimEntity {
     }
 
     @Override
-    @NonNull
+    @NotNull
     public Entity getNative() {
         return entity;
     }
@@ -60,7 +63,6 @@ public class BukkitGrimEntity implements GrimEntity {
         if (bukkitPlatformWorld == null || !bukkitPlatformWorld.getBukkitWorld().equals(entity.getWorld())) {
             bukkitPlatformWorld = new BukkitPlatformWorld(entity.getWorld());
         }
-
         return bukkitPlatformWorld;
     }
 
@@ -76,4 +78,26 @@ public class BukkitGrimEntity implements GrimEntity {
                 location.getPitch()
         );
     }
+
+    @Override
+    public double distanceSquared(double oX, double oY, double oZ) {
+        if (CAN_USE_DIRECT_GETTERS) {
+            double x = this.entity.getX();
+            double y = this.entity.getY();
+            double z = this.entity.getZ();
+            double distX = (x - oX) * (x - oX);
+            double distY = (y - oY) * (y - oY);
+            double distZ = (z - oZ) * (z - oZ);
+            return distX + distY + distZ;
+        } else {
+            org.bukkit.Location location = this.entity.getLocation();
+            double x = location.getX();
+            double y = location.getY();
+            double z = location.getZ();
+            double distX = (x - oX) * (x - oX);
+            double distY = (y - oY) * (y - oY);
+            double distZ = (z - oZ) * (z - oZ);
+            return distX + distY + distZ;
+        }
+    }
 }

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/events/PistonEvent.java

@@ -4,7 +4,7 @@ import ac.grim.grimac.GrimAPI;
 import ac.grim.grimac.platform.bukkit.utils.convert.BukkitConversionUtils;
 import ac.grim.grimac.player.GrimPlayer;
 import ac.grim.grimac.utils.collisions.datatypes.SimpleCollisionBox;
-import ac.grim.grimac.utils.data.PistonData;
+import ac.grim.grimac.utils.data.PistonTemplate;
 import com.github.retrooper.packetevents.protocol.world.BlockFace;
 import org.bukkit.Material;
 import org.bukkit.block.Block;
@@ -18,8 +18,19 @@ import java.util.ArrayList;
 import java.util.List;
 
 public class PistonEvent implements Listener {
-    Material SLIME_BLOCK = Material.getMaterial("SLIME_BLOCK");
-    Material HONEY_BLOCK = Material.getMaterial("HONEY_BLOCK");
+
+    private final Material SLIME_BLOCK = Material.getMaterial("SLIME_BLOCK");
+    private final Material HONEY_BLOCK = Material.getMaterial("HONEY_BLOCK");
+
+    private static final double MAX_HORIZONTAL_DISTANCE = 24.0;
+    private static final double MAX_VERTICAL_DISTANCE = 64.0;
+
+    // accuracy isn't that important, it's close enough and performant
+    private static boolean isCloseEnough(int x, int y, int z, double posX, double posY, double posZ) {
+        return Math.abs(x - posX) <= MAX_HORIZONTAL_DISTANCE
+                && Math.abs(y - posY) <= MAX_VERTICAL_DISTANCE
+                && Math.abs(z - posZ) <= MAX_HORIZONTAL_DISTANCE;
+    }
 
     @EventHandler(priority = EventPriority.MONITOR, ignoreCancelled = true)
     public void onPistonPushEvent(BlockPistonExtendEvent event) {
@@ -55,17 +66,8 @@ public class PistonEvent implements Listener {
                         piston.getY() + event.getDirection().getModY(),
                         piston.getZ() + event.getDirection().getModZ()));
 
-        boolean finalHasSlimeBlock = hasSlimeBlock;
-        boolean finalHasHoneyBlock = hasHoneyBlock;
-        for (GrimPlayer player : GrimAPI.INSTANCE.getPlayerDataManager().getEntries()) {
-            int lastTrans = player.lastTransactionSent.get();
-            player.runSafely(() -> {
-                if (player.compensatedWorld.isChunkLoaded(event.getBlock().getX() >> 4, event.getBlock().getZ() >> 4)) {
-                    PistonData data = new PistonData(BukkitConversionUtils.fromBukkitFace(event.getDirection()), boxes, lastTrans, true, finalHasSlimeBlock, finalHasHoneyBlock);
-                    player.latencyUtils.addRealTimeTaskAsync(lastTrans, () -> player.compensatedWorld.activePistons.add(data));
-                }
-            });
-        }
+        PistonTemplate data = new PistonTemplate(BukkitConversionUtils.fromBukkitFace(event.getDirection()), boxes, true, hasSlimeBlock, hasHoneyBlock);
+        addPistonData(data, piston.getX(), piston.getY(), piston.getZ());
     }
 
     // For some unknown reason, bukkit handles this stupidly
@@ -87,9 +89,8 @@ public class PistonEvent implements Listener {
         BlockFace face = BukkitConversionUtils.fromBukkitFace(event.getDirection());
 
         // The event was called without blocks and is therefore in the right direction
+        Block piston = event.getBlock();
         if (event.getBlocks().isEmpty()) {
-            Block piston = event.getBlock();
-
             // Add bounding box of the actual piston head pushing
             boxes.add(new SimpleCollisionBox(0, 0, 0, 1, 1, 1, true)
                     .offset(piston.getX() + face.getModX(),
@@ -113,15 +114,18 @@ public class PistonEvent implements Listener {
             }
         }
 
-        boolean finalHasSlimeBlock = hasSlimeBlock;
-        boolean finalHasHoneyBlock = hasHoneyBlock;
+        PistonTemplate data = new PistonTemplate(face, boxes, false, hasSlimeBlock, hasHoneyBlock);
+        addPistonData(data, piston.getX(), piston.getY(), piston.getZ());
+    }
+
+    private void addPistonData(PistonTemplate pistonTemplate, int blockX, int blockY, int blockZ) {
         for (GrimPlayer player : GrimAPI.INSTANCE.getPlayerDataManager().getEntries()) {
+            if (player.compensatedWorld.isChunkLoaded(blockX >> 4, blockZ >> 4) || !isCloseEnough(blockX, blockY, blockZ, player.x, player.y, player.z)) continue;
+
             int lastTrans = player.lastTransactionSent.get();
-            player.runSafely(() -> {
-                if (player.compensatedWorld.isChunkLoaded(event.getBlock().getX() >> 4, event.getBlock().getZ() >> 4)) {
-                    PistonData data = new PistonData(BukkitConversionUtils.fromBukkitFace(event.getDirection()), boxes, lastTrans, false, finalHasSlimeBlock, finalHasHoneyBlock);
-                    player.latencyUtils.addRealTimeTaskAsync(lastTrans, () -> player.compensatedWorld.activePistons.add(data));
-                }
+
+            player.latencyUtils.addRealTimeTaskAsync(lastTrans, () -> {
+                player.compensatedWorld.addPiston(pistonTemplate, lastTrans);
             });
         }
     }

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/initables/BukkitBStats.java

@@ -2,15 +2,14 @@ package ac.grim.grimac.platform.bukkit.initables;
 
 import ac.grim.grimac.manager.init.start.StartableInitable;
 import ac.grim.grimac.platform.bukkit.GrimACBukkitLoaderPlugin;
+import ac.grim.grimac.utils.anticheat.Constants;
 import io.github.retrooper.packetevents.bstats.bukkit.Metrics;
 
 public class BukkitBStats implements StartableInitable {
     @Override
     public void start() {
-        int pluginId = 12820; // <-- Replace with the id of your plugin!
         try {
-            new Metrics(GrimACBukkitLoaderPlugin.LOADER, pluginId);
-        } catch (Exception ignored) {
-        }
+            new Metrics(GrimACBukkitLoaderPlugin.LOADER, Constants.BSTATS_PLUGIN_ID);
+        } catch (Exception ignored) {}
     }
 }

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/initables/BukkitTickEndEvent.java

@@ -8,6 +8,7 @@ import ac.grim.grimac.platform.bukkit.utils.reflection.PaperUtils;
 import ac.grim.grimac.player.GrimPlayer;
 import ac.grim.grimac.utils.anticheat.LogUtil;
 import ac.grim.grimac.utils.lists.HookedListWrapper;
+import ac.grim.grimac.utils.reflection.ReflectionUtils;
 import com.github.retrooper.packetevents.PacketEvents;
 import com.github.retrooper.packetevents.manager.server.ServerVersion;
 import com.github.retrooper.packetevents.util.reflection.Reflection;
@@ -25,6 +26,20 @@ import java.util.List;
 @SuppressWarnings(value = {"unchecked", "deprecated"})
 public class BukkitTickEndEvent extends AbstractTickEndEvent implements Listener {
 
+    private Boolean getLateBindState() {
+        Class<?> spigotConfig = ReflectionUtils.getClass("org.spigotmc.SpigotConfig");
+        // ReflectionUtils.getField(class, name) handles the loop and setAccessible
+        Field field = ReflectionUtils.getField(spigotConfig, "lateBind");
+
+        if (field == null) return null;
+
+        try {
+            return (boolean) field.get(null);
+        } catch (Exception ignored) {
+            return null;
+        }
+    }
+
     @Override
     public void start() {
         if (!super.shouldInjectEndTick()) {
@@ -38,9 +53,18 @@ public class BukkitTickEndEvent extends AbstractTickEndEvent implements Listener
             PaperUtils.registerTickEndEvent(this, this::tickAllFoliaPlayers);
             return;
         }
-        // if it fails to inject, try to use paper's event
-        if (!injectWithReflection() && !PaperUtils.registerTickEndEvent(this, this::tickAllPlayers)) {
+        // if it fails to register Paper event, try to inject via reflection
+        if (!PaperUtils.registerTickEndEvent(this, this::tickAllPlayers) && !injectWithReflection()) {
             LogUtil.error("Failed to inject into the end of tick event!");
+
+            if (PacketEvents.getAPI().getServerManager().getVersion().isOlderThan(ServerVersion.V_1_14_4)) {
+                Boolean lateBind = getLateBindState();
+                if (lateBind == null) {
+                    LogUtil.error("Failed to determine the late-bind state. Perhaps you are using a custom server fork? Check the fork configuration for a late-bind option and disable it.");
+                } else if (lateBind) {
+                    LogUtil.error("Injection failed because the late-bind option is enabled. Disable it in spigot.yml.");
+                }
+            }
         }
     }
 
@@ -65,6 +89,7 @@ public class BukkitTickEndEvent extends AbstractTickEndEvent implements Listener
         // Inject so we can add the final transaction pre-flush event
         try {
             Object connection = SpigotReflectionUtil.getMinecraftServerConnectionInstance();
+            if (connection == null) return false;
 
             Field connectionsList = Reflection.getField(connection.getClass(), List.class, 1);
             List<Object> endOfTickObject = (List<Object>) connectionsList.get(connection);
@@ -86,7 +111,7 @@ public class BukkitTickEndEvent extends AbstractTickEndEvent implements Listener
             Unsafe unsafe = (Unsafe) unsafeField.get(null);
             unsafe.putObject(connection, unsafe.objectFieldOffset(connectionsList), wrapper);
         } catch (NoSuchFieldException | IllegalAccessException e) {
-            LogUtil.exception("Failed to inject into the end of tick event via reflection", e);
+            LogUtil.error("Failed to inject into the end of tick event via reflection", e);
             return false;
         }
         return true;

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/manager/BukkitItemResetHandler.java

@@ -2,95 +2,212 @@ package ac.grim.grimac.platform.bukkit.manager;
 
 import ac.grim.grimac.platform.api.manager.ItemResetHandler;
 import ac.grim.grimac.platform.api.player.PlatformPlayer;
-import ac.grim.grimac.platform.bukkit.player.BukkitPlatformPlayer;
 import ac.grim.grimac.platform.bukkit.utils.reflection.PaperUtils;
+import ac.grim.grimac.utils.reflection.ReflectionUtils;
 import com.github.retrooper.packetevents.PacketEvents;
 import com.github.retrooper.packetevents.manager.server.ServerVersion;
-import lombok.SneakyThrows;
+import com.github.retrooper.packetevents.protocol.player.InteractionHand;
 import org.bukkit.Bukkit;
 import org.bukkit.entity.LivingEntity;
 import org.bukkit.entity.Player;
-import org.jetbrains.annotations.NotNull;
+import org.bukkit.inventory.EquipmentSlot;
 import org.jetbrains.annotations.Nullable;
 
+import java.lang.reflect.InvocationTargetException;
 import java.lang.reflect.Method;
+import java.util.Objects;
+import java.util.function.Consumer;
+import java.util.function.Function;
+import java.util.function.Predicate;
 
 public class BukkitItemResetHandler implements ItemResetHandler {
-    // resets item usage, then returns whether the player was using an item
-    private final @NotNull ItemUsageReset resetItemUsage = createItemUsageResetFunction();
+    private static final Consumer<Player> resetItemUsage;
+    private static final Predicate<Player> isUsingItem;
+    private static final Function<Player, InteractionHand> getItemUsageHand;
 
-    @SneakyThrows
+    @Override
     public void resetItemUsage(@Nullable PlatformPlayer player) {
-        if (player != null) {
-            resetItemUsage.accept(((BukkitPlatformPlayer) player).getNative());
-        }
+        if (player != null) resetItemUsage.accept((Player) player.getNative());
     }
 
-    @SneakyThrows
-    private @NotNull ItemUsageReset createItemUsageResetFunction() {
-        ServerVersion version = PacketEvents.getAPI().getServerManager().getVersion();
-        if (version.isNewerThan(ServerVersion.V_1_17) && PaperUtils.PAPER) {
-            if (version.isOlderThan(ServerVersion.V_1_19)) {
-                return LivingEntity::clearActiveItem;
+    @Override
+    public @Nullable InteractionHand getItemUsageHand(@Nullable PlatformPlayer player) {
+        return player == null ? null : getItemUsageHand.apply((Player) player.getNative());
+    }
+
+    @Override
+    public boolean isUsingItem(@Nullable PlatformPlayer player) {
+        return player != null && isUsingItem.test((Player) player.getNative());
+    }
+
+    static {
+        final ServerVersion version = PacketEvents.getAPI().getServerManager().getVersion();
+
+        boolean legacy = version.isOlderThanOrEquals(ServerVersion.V_1_8_8);
+
+        try {
+            Method getHandle_ = null;
+
+            if (PaperUtils.PAPER && version.isNewerThanOrEquals(ServerVersion.V_1_20_5)) {
+                Class<?> cls = ReflectionUtils.getClass("org.bukkit.craftbukkit.entity.CraftLivingEntity");
+                if (cls != null) getHandle_ = cls.getMethod("getHandle");
             }
-            Method setLivingEntityFlag = Class.forName(version.isOlderThan(ServerVersion.V_1_20_5) ? "net.minecraft.world.entity.EntityLiving" : "net.minecraft.world.entity.LivingEntity")
-                    .getDeclaredMethod(version.isOlderThan(ServerVersion.V_1_20_5) ? "c" : "setLivingEntityFlag", int.class, boolean.class);
-            Method getHandle = (version.isOlderThan(ServerVersion.V_1_20_5)
-                    ? Class.forName("org.bukkit.craftbukkit." + Bukkit.getServer().getClass().getPackageName().split("\\.")[3] + ".entity.CraftPlayer")
-                    : Class.forName("org.bukkit.craftbukkit.entity.CraftPlayer")
-            ).getMethod("getHandle");
 
-            setLivingEntityFlag.setAccessible(true);
+            boolean obfuscated = getHandle_ == null;
+            String nmsPackage = obfuscated ? Bukkit.getServer().getClass().getPackageName().split("\\.")[3] : null;
 
-            return player -> {
-                // don't trigger gameevents
-                setLivingEntityFlag.invoke(getHandle.invoke(player), 1, false);
-                player.clearActiveItem();
-            };
-        }
+            if (obfuscated) {
+                String clazzName = legacy ? "CraftHumanEntity" : "CraftLivingEntity";
+                getHandle_ = Class.forName("org.bukkit.craftbukkit." + nmsPackage + ".entity." + clazzName).getMethod("getHandle");
+            }
 
-        if (version == ServerVersion.V_1_8_8) {
-            Class<?> EntityHuman = Class.forName("net.minecraft.server.v1_8_R3.EntityHuman");
-            Method getHandle = Class.forName("org.bukkit.craftbukkit.v1_8_R3.entity.CraftPlayer").getMethod("getHandle");
-            Method clearActiveItem = EntityHuman.getMethod("bV");
-            Method isUsingItem = EntityHuman.getMethod("bS");
+            final Method getHandle = getHandle_;
+            Class<?> clazz = getHandle_.getReturnType();
 
-            return player -> {
-                Object handle = getHandle.invoke(player);
-                clearActiveItem.invoke(handle);
+            if (version.isNewerThanOrEquals(ServerVersion.V_1_10)) {
+                isUsingItem = Player::isHandRaised;
+            } else {
+                Method method = clazz.getMethod(switch (Objects.requireNonNull(nmsPackage)) {
+                    case "v1_8_R3" -> "bS";
+                    case "v1_9_R1" -> "cs";
+                    case "v1_9_R2" -> "ct";
+                    default -> throw new IllegalStateException("You are using an unsupported server version! (" + version.getReleaseName() + ")");
+                });
+                isUsingItem = player -> {
+                    try {
+                        return (boolean) method.invoke(getHandle.invoke(player));
+                    } catch (IllegalAccessException | InvocationTargetException e) {
+                        throw new RuntimeException(e);
+                    }
+                };
+            }
 
-                // in 1.8 we need to resync item usage manually,
-                // only do so if the player was using an item
-                if ((boolean) isUsingItem.invoke(handle)) {
-                    player.updateInventory();
+            if (legacy) {
+                getItemUsageHand = player -> isUsingItem.test(player) ? InteractionHand.MAIN_HAND : null;
+            } else if (PaperUtils.PAPER && version.isNewerThanOrEquals(ServerVersion.V_1_16_5)) {
+                getItemUsageHand = player -> player.isHandRaised()
+                        ? player.getHandRaised() == EquipmentSlot.OFF_HAND
+                        ? InteractionHand.OFF_HAND
+                        : InteractionHand.MAIN_HAND
+                        : null;
+            } else {
+                Method method = clazz.getMethod(switch (Objects.requireNonNull(nmsPackage)) {
+                    case "v1_9_R1" -> "ct";
+                    case "v1_9_R2" -> "cu";
+                    case "v1_10_R1" -> "cy";
+                    case "v1_11_R1" -> "cz";
+                    case "v1_12_R1" -> "cH";
+                    case "v1_13_R1", "v1_13_R2", "v1_14_R1" -> "cU";
+                    case "v1_15_R1", "v1_16_R1", "v1_16_R2", "v1_16_R3", "v1_17_R1" -> "getRaisedHand";
+                    case "v1_18_R1" -> "eM";
+                    case "v1_18_R2" -> "eN";
+                    case "v1_19_R1" -> "eU";
+                    case "v1_19_R2" -> "fa";
+                    case "v1_19_R3" -> "ff";
+                    case "v1_20_R1" -> "fj";
+                    case "v1_20_R2" -> "fn";
+                    case "v1_20_R3" -> "fo";
+                    case "v1_20_R4" -> "fw";
+                    case "v1_21_R1" -> "fs";
+                    case "v1_21_R2", "v1_21_R3", "v1_21_R4" -> "fA";
+                    case "v1_21_R5" -> "fH";
+                    case "v1_21_R6" -> "fP";
+                    default -> throw new IllegalStateException("You are using an unsupported server version! (" + version.getReleaseName() + ")");
+                });
+                getItemUsageHand = player -> {
+                    try {
+                        return isUsingItem.test(player)
+                                ? ((Enum<?>) method.invoke(getHandle.invoke(player))).ordinal() == 0
+                                ? InteractionHand.MAIN_HAND
+                                : InteractionHand.OFF_HAND
+                                : null;
+                    } catch (IllegalAccessException | InvocationTargetException e) {
+                        throw new RuntimeException(e);
+                    }
+                };
+            }
+
+            Method setLivingEntityFlag;
+
+            if (version.isNewerThanOrEquals(ServerVersion.V_1_19)) {
+                String name = obfuscated ? "c" : "setLivingEntityFlag";
+                setLivingEntityFlag = clazz.getDeclaredMethod(name, int.class, boolean.class);
+                setLivingEntityFlag.setAccessible(true);
+            } else {
+                setLivingEntityFlag = null;
+            }
+
+            if (PaperUtils.PAPER && version.isNewerThan(ServerVersion.V_1_17)) {
+                resetItemUsage = setLivingEntityFlag == null ? LivingEntity::clearActiveItem : player -> {
+                    try {
+                        setLivingEntityFlag.invoke(getHandle.invoke(player), 1, false);
+                    } catch (IllegalAccessException | InvocationTargetException e) {
+                        throw new RuntimeException(e);
+                    }
+                    player.clearActiveItem();
+                };
+            } else {
+                Method method = clazz.getMethod(switch (Objects.requireNonNull(nmsPackage)) {
+                    case "v1_8_R3" -> "bV";
+                    case "v1_9_R1" -> "cz";
+                    case "v1_9_R2" -> "cA";
+                    case "v1_10_R1" -> "cE";
+                    case "v1_11_R1" -> "cF";
+                    case "v1_12_R1" -> "cN";
+                    case "v1_13_R1", "v1_13_R2" -> "da";
+                    case "v1_14_R1" -> "dp";
+                    case "v1_15_R1" -> "dH";
+                    case "v1_16_R1", "v1_16_R2", "v1_16_R3", "v1_17_R1" -> "clearActiveItem";
+                    case "v1_18_R1" -> "eR";
+                    case "v1_18_R2" -> "eS";
+                    case "v1_19_R1" -> "eZ";
+                    case "v1_19_R2" -> "ff";
+                    case "v1_19_R3" -> "fk";
+                    case "v1_20_R1" -> "fo";
+                    case "v1_20_R2" -> "fs";
+                    case "v1_20_R3" -> "ft";
+                    case "v1_20_R4" -> "fB";
+                    case "v1_21_R1" -> "fx";
+                    case "v1_21_R2", "v1_21_R3", "v1_21_R4" -> "fF";
+                    case "v1_21_R5" -> "fM";
+                    case "v1_21_R6" -> "fU";
+                    default -> throw new IllegalStateException("You are using an unsupported server version! (" + version.getReleaseName() + ")");
+                });
+
+                if (legacy) { // 1.8.8
+                    resetItemUsage = player -> {
+                        try {
+                            method.invoke(getHandle.invoke(player));
+
+                            // in 1.8 we need to resync item usage manually,
+                            // only do so if the player is using an item
+                            if (isUsingItem.test(player)) player.updateInventory();
+                        } catch (IllegalAccessException | InvocationTargetException e) {
+                            throw new RuntimeException(e);
+                        }
+                    };
+                } else if (setLivingEntityFlag == null) { // 1.9-1.18.2
+                    resetItemUsage = player -> {
+                        try {
+                            method.invoke(getHandle.invoke(player));
+                        } catch (IllegalAccessException | InvocationTargetException e) {
+                            throw new RuntimeException(e);
+                        }
+                    };
+                } else { // 1.19+
+                    resetItemUsage = player -> {
+                        try {
+                            Object handle = getHandle.invoke(player);
+                            setLivingEntityFlag.invoke(handle, 1, false);
+                            method.invoke(handle);
+                        } catch (IllegalAccessException | InvocationTargetException e) {
+                            throw new RuntimeException(e);
+                        }
+                    };
                 }
-            };
-        }
-
-        String nmsPackage = Bukkit.getServer().getClass().getPackageName().split("\\.")[3];
-        String livingEntityPackage = version.isNewerThan(ServerVersion.V_1_16_5) ? "net.minecraft.world.entity.EntityLiving" : "net.minecraft.server." + nmsPackage + ".EntityLiving";
-        Method getHandle = Class.forName("org.bukkit.craftbukkit." + nmsPackage + ".entity.CraftPlayer").getMethod("getHandle");
-        Method clearActiveItem = Class.forName(livingEntityPackage).getMethod(
-            switch (nmsPackage) {
-                case "v1_9_R1" -> "cz";
-                case "v1_9_R2" -> "cA";
-                case "v1_10_R1" -> "cE";
-                case "v1_11_R1" -> "cF";
-                case "v1_12_R1" -> "cN";
-                case "v1_13_R1", "v1_13_R2" -> "da";
-                case "v1_14_R1" -> "dp";
-                case "v1_15_R1" -> "dH";
-                case "v1_16_R1", "v1_16_R2", "v1_16_R3", "v1_17_R1" -> "clearActiveItem";
-                case "v1_18_R1" -> "eR";
-                case "v1_18_R2" -> "eS";
-                default -> throw new IllegalStateException("You are using an unsupported server version! (" + version.getReleaseName() + ")");
             }
-        );
-
-        return player -> clearActiveItem.invoke(getHandle.invoke(player));
-    }
-
-    private interface ItemUsageReset {
-        void accept(@NotNull Player player) throws Throwable;
+        } catch (Throwable t) {
+            throw t instanceof RuntimeException e ? e : new RuntimeException(t);
+        }
     }
 }

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/manager/BukkitMessagePlaceHolderManager.java

@@ -5,14 +5,14 @@ import ac.grim.grimac.platform.api.player.PlatformPlayer;
 import ac.grim.grimac.platform.bukkit.player.BukkitPlatformPlayer;
 import com.github.retrooper.packetevents.util.reflection.Reflection;
 import me.clip.placeholderapi.PlaceholderAPI;
-import org.checkerframework.checker.nullness.qual.NonNull;
-import org.checkerframework.checker.nullness.qual.Nullable;
+import org.jetbrains.annotations.NotNull;
+import org.jetbrains.annotations.Nullable;
 
 public class BukkitMessagePlaceHolderManager implements MessagePlaceHolderManager {
     public static final boolean hasPlaceholderAPI = Reflection.getClassByNameWithoutException("me.clip.placeholderapi.PlaceholderAPI") != null;
 
     @Override
-    public @NonNull String replacePlaceholders(@Nullable PlatformPlayer player, @NonNull String string) {
+    public @NotNull String replacePlaceholders(@Nullable PlatformPlayer player, @NotNull String string) {
         if (!hasPlaceholderAPI) return string;
         return PlaceholderAPI.setPlaceholders(player instanceof BukkitPlatformPlayer bukkitPlatformPlayer ? bukkitPlatformPlayer.getBukkitPlayer() : null, string);
     }

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/manager/BukkitParserDescriptorFactory.java

@@ -1,17 +1,45 @@
 package ac.grim.grimac.platform.bukkit.manager;
 
 import ac.grim.grimac.platform.api.command.PlayerSelector;
-import ac.grim.grimac.platform.api.manager.ParserDescriptorFactory;
+import ac.grim.grimac.platform.api.manager.cloud.CloudCommandAdapter;
 import ac.grim.grimac.platform.api.sender.Sender;
 import ac.grim.grimac.platform.bukkit.command.BukkitPlayerSelectorParser;
+import org.bukkit.Bukkit;
+import org.bukkit.command.CommandSender;
+import org.bukkit.entity.Player;
+import org.incendo.cloud.bukkit.BukkitCommandContextKeys;
 import org.incendo.cloud.parser.ParserDescriptor;
+import org.incendo.cloud.suggestion.Suggestion;
+import org.incendo.cloud.suggestion.SuggestionProvider;
 
-public class BukkitParserDescriptorFactory implements ParserDescriptorFactory {
+import java.util.ArrayList;
+import java.util.List;
+import java.util.concurrent.CompletableFuture;
 
-    BukkitPlayerSelectorParser<Sender> bukkitPlayerSelectorParser = new BukkitPlayerSelectorParser<>();
+public class BukkitParserDescriptorFactory implements CloudCommandAdapter {
+
+    // Parser is kept as a field because its stateless
+    private final BukkitPlayerSelectorParser<Sender> bukkitPlayerSelectorParser = new BukkitPlayerSelectorParser<>();
 
     @Override
-    public ParserDescriptor<Sender, PlayerSelector> getSinglePlayer() {
+    public ParserDescriptor<Sender, PlayerSelector> singlePlayerSelectorParser() {
         return bukkitPlayerSelectorParser.descriptor();
     }
+
+    // TODO (Cross-platform) better + brigadier suggestions
+    @Override
+    public SuggestionProvider<Sender> onlinePlayerSuggestions() {
+        return (context, input) -> {
+            List<Suggestion> suggestions = new ArrayList<>();
+
+            for(Player player : Bukkit.getOnlinePlayers()) {
+                CommandSender bukkit = context.get(BukkitCommandContextKeys.BUKKIT_COMMAND_SENDER);
+                if (!(bukkit instanceof Player) || ((Player)bukkit).canSee(player)) {
+                    suggestions.add(Suggestion.suggestion(player.getName()));
+                }
+            }
+
+            return CompletableFuture.completedFuture(suggestions);
+        };
+    }
 }

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/manager/BukkitPlatformPluginManager.java

@@ -5,7 +5,7 @@ import ac.grim.grimac.platform.api.manager.PlatformPluginManager;
 import ac.grim.grimac.platform.bukkit.BukkitPlatformPlugin;
 import org.bukkit.Bukkit;
 import org.bukkit.plugin.Plugin;
-import org.checkerframework.checker.nullness.qual.Nullable;
+import org.jetbrains.annotations.Nullable;
 
 public class BukkitPlatformPluginManager implements PlatformPluginManager {
 

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/player/BukkitOfflinePlatformPlayer.java

@@ -0,0 +1,34 @@
+package ac.grim.grimac.platform.bukkit.player;
+
+import ac.grim.grimac.platform.api.player.OfflinePlatformPlayer;
+import lombok.RequiredArgsConstructor;
+import org.bukkit.OfflinePlayer;
+import org.jetbrains.annotations.NotNull;
+
+import java.util.Objects;
+import java.util.UUID;
+
+@RequiredArgsConstructor
+public class BukkitOfflinePlatformPlayer implements OfflinePlatformPlayer {
+    private final OfflinePlayer offlinePlayer;
+
+    @Override
+    public boolean isOnline() {
+        return offlinePlayer.isOnline();
+    }
+
+    @Override
+    public @NotNull String getName() {
+        return Objects.requireNonNull(offlinePlayer.getName());
+    }
+
+    @Override
+    public @NotNull UUID getUniqueId() {
+        return offlinePlayer.getUniqueId();
+    }
+
+    @Override
+    public boolean equals(Object o) {
+        return o instanceof OfflinePlatformPlayer player && this.getUniqueId().equals(player.getUniqueId());
+    }
+}

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/player/BukkitPlatformInventory.java

@@ -3,16 +3,14 @@ package ac.grim.grimac.platform.bukkit.player;
 import ac.grim.grimac.platform.api.player.PlatformInventory;
 import com.github.retrooper.packetevents.protocol.item.ItemStack;
 import io.github.retrooper.packetevents.util.SpigotConversionUtil;
+import lombok.RequiredArgsConstructor;
 import org.bukkit.entity.Player;
 
+@RequiredArgsConstructor
 public class BukkitPlatformInventory implements PlatformInventory {
 
     private final Player bukkitPlayer;
 
-    public BukkitPlatformInventory(Player bukkitPlayer) {
-        this.bukkitPlayer = bukkitPlayer;
-    }
-
     @Override
     public ItemStack getItemInHand() {
         return SpigotConversionUtil.fromBukkitItemStack(bukkitPlayer.getInventory().getItemInHand());

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/player/BukkitPlatformPlayer.java

@@ -1,6 +1,7 @@
 package ac.grim.grimac.platform.bukkit.player;
 
 import ac.grim.grimac.platform.api.entity.GrimEntity;
+import ac.grim.grimac.platform.api.player.BlockTranslator;
 import ac.grim.grimac.platform.api.player.PlatformInventory;
 import ac.grim.grimac.platform.api.player.PlatformPlayer;
 import ac.grim.grimac.platform.api.sender.Sender;
@@ -9,34 +10,44 @@ import ac.grim.grimac.platform.bukkit.entity.BukkitGrimEntity;
 import ac.grim.grimac.platform.bukkit.utils.anticheat.MultiLibUtil;
 import ac.grim.grimac.platform.bukkit.utils.convert.BukkitConversionUtils;
 import ac.grim.grimac.platform.bukkit.utils.reflection.PaperUtils;
+import ac.grim.grimac.utils.common.arguments.CommonGrimArguments;
 import ac.grim.grimac.utils.math.Location;
+import com.github.retrooper.packetevents.PacketEvents;
 import com.github.retrooper.packetevents.protocol.player.GameMode;
+import com.github.retrooper.packetevents.protocol.player.User;
 import com.github.retrooper.packetevents.util.Vector3d;
 import io.github.retrooper.packetevents.util.SpigotConversionUtil;
 import lombok.Getter;
-import net.kyori.adventure.platform.bukkit.BukkitAudiences;
 import net.kyori.adventure.text.Component;
 import org.bukkit.World;
 import org.bukkit.entity.Player;
 import org.bukkit.permissions.Permission;
 import org.bukkit.permissions.PermissionDefault;
-import org.checkerframework.checker.nullness.qual.NonNull;
-import org.checkerframework.checker.nullness.qual.Nullable;
+import org.jetbrains.annotations.NotNull;
+import org.jetbrains.annotations.Nullable;
 
 import java.util.UUID;
 import java.util.concurrent.CompletableFuture;
 
 public class BukkitPlatformPlayer extends BukkitGrimEntity implements PlatformPlayer {
-    private static final BukkitAudiences audiences = BukkitAudiences.create(GrimACBukkitLoaderPlugin.LOADER);
 
     @Getter
     private final Player bukkitPlayer;
+    @Getter
     private final PlatformInventory inventory;
 
+    private final @Nullable User user;
+
     public BukkitPlatformPlayer(Player bukkitPlayer) {
         super(bukkitPlayer);
         this.bukkitPlayer = bukkitPlayer;
         this.inventory = new BukkitPlatformInventory(bukkitPlayer);
+        if (CommonGrimArguments.USE_CHAT_FAST_BYPASS.value()) {
+            Object channel = PacketEvents.getAPI().getProtocolManager().getChannel(bukkitPlayer.getUniqueId());
+            this.user = PacketEvents.getAPI().getProtocolManager().getUser(channel);
+        } else {
+            this.user = null;
+        }
     }
 
     @Override
@@ -66,12 +77,21 @@ public class BukkitPlatformPlayer extends BukkitGrimEntity implements PlatformPl
 
     @Override
     public void sendMessage(String message) {
-        bukkitPlayer.sendMessage(message);
+        if (CommonGrimArguments.USE_CHAT_FAST_BYPASS.value() && user != null) {
+            user.sendMessage(message);
+        } else {
+            bukkitPlayer.sendMessage(message);
+        }
     }
 
     @Override
     public void sendMessage(Component message) {
-        audiences.player(bukkitPlayer).sendMessage(message);
+        if (CommonGrimArguments.USE_CHAT_FAST_BYPASS.value() && user != null) {
+            user.sendMessage(message);
+        } else {
+            // Paper/Purpur Player is natively an Adventure Audience, so no platform shim is needed
+            bukkitPlayer.sendMessage(message);
+        }
     }
 
     @Override
@@ -91,13 +111,12 @@ public class BukkitPlatformPlayer extends BukkitGrimEntity implements PlatformPl
 
     @Override
     public Vector3d getPosition() {
-        org.bukkit.Location location = this.bukkitPlayer.getLocation();
-        return new Vector3d(location.getX(), location.getY(), location.getZ());
-    }
-
-    @Override
-    public PlatformInventory getInventory() {
-        return inventory;
+        if (CAN_USE_DIRECT_GETTERS) {
+            return new Vector3d(this.bukkitPlayer.getX(), this.bukkitPlayer.getY(), this.bukkitPlayer.getZ());
+        } else {
+            org.bukkit.Location location = this.bukkitPlayer.getLocation();
+            return new Vector3d(location.getX(), location.getY(), location.getZ());
+        }
     }
 
     @Override
@@ -151,7 +170,12 @@ public class BukkitPlatformPlayer extends BukkitGrimEntity implements PlatformPl
     }
 
     @Override
-    @NonNull
+    public BlockTranslator getBlockTranslator() {
+        return BlockTranslator.IDENTITY;
+    }
+
+    @Override
+    @NotNull
     public Player getNative() {
         return this.bukkitPlayer;
     }

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/player/BukkitPlatformPlayerFactory.java

@@ -1,41 +1,40 @@
 package ac.grim.grimac.platform.bukkit.player;
 
 import ac.grim.grimac.platform.api.player.AbstractPlatformPlayerFactory;
+import ac.grim.grimac.platform.api.player.OfflinePlatformPlayer;
 import ac.grim.grimac.platform.api.player.PlatformPlayer;
 import org.bukkit.Bukkit;
+import org.bukkit.OfflinePlayer;
 import org.bukkit.entity.Player;
 import org.jetbrains.annotations.NotNull;
 
+import java.util.Arrays;
 import java.util.Collection;
 import java.util.UUID;
 
 
 public class BukkitPlatformPlayerFactory extends AbstractPlatformPlayerFactory<Player> {
+
     @Override
     protected Player getNativePlayer(@NotNull UUID uuid) {
         return Bukkit.getPlayer(uuid);
     }
 
+    @Override
+    protected Player getNativePlayer(@NotNull String name) {
+        return Bukkit.getPlayer(name);
+    }
+
     @Override
     protected PlatformPlayer createPlatformPlayer(@NotNull Player nativePlayer) {
         return new BukkitPlatformPlayer(nativePlayer);
     }
 
-    @Override
-    protected boolean isNativePlayerType(@NotNull Object playerObject) {
-        return playerObject instanceof Player;
-    }
-
     @Override
     protected UUID getPlayerUUID(@NotNull Player nativePlayer) {
         return nativePlayer.getUniqueId();
     }
 
-    @Override
-    protected Class<Player> getNativePlayerClass() {
-        return Player.class;
-    }
-
     // The cast is safe because Bukkit.getOnlinePlayers() is guaranteed to contain Player or its subtypes,
     // and we're only reading from it.
     @Override
@@ -44,4 +43,26 @@ public class BukkitPlatformPlayerFactory extends AbstractPlatformPlayerFactory<P
         // Cast Collection<? extends Player> to Collection<Player>
         return (Collection<Player>) Bukkit.getOnlinePlayers();
     }
+
+    @Override
+    public OfflinePlatformPlayer getOfflineFromUUID(@NotNull UUID uuid) {
+        OfflinePlayer offlinePlayer = Bukkit.getOfflinePlayer(uuid);
+        return new BukkitOfflinePlatformPlayer(offlinePlayer);
+    }
+
+    @Override
+    public OfflinePlatformPlayer getOfflineFromName(@NotNull String name) {
+        OfflinePlayer offlinePlayer = Bukkit.getOfflinePlayer(name);
+        return new BukkitOfflinePlatformPlayer(offlinePlayer);
+    }
+
+    @Override
+    public Collection<OfflinePlatformPlayer> getOfflinePlayers() {
+        OfflinePlayer[] offlinePlayers = Bukkit.getOfflinePlayers();
+        BukkitOfflinePlatformPlayer[] offlinePlatformPlayers = new BukkitOfflinePlatformPlayer[offlinePlayers.length];
+        for (int i = 0; i < offlinePlayers.length; i++) {
+            offlinePlatformPlayers[i] = new BukkitOfflinePlatformPlayer(offlinePlayers[i]);
+        }
+        return Arrays.asList(offlinePlatformPlayers);
+    }
 }

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/scheduler/bukkit/BukkitAsyncScheduler.java

@@ -22,29 +22,31 @@ public class BukkitAsyncScheduler implements AsyncScheduler {
 
     @Override
     public TaskHandle runDelayed(@NotNull GrimPlugin plugin, @NotNull Runnable task, long delay, @NotNull TimeUnit timeUnit) {
-        return new BukkitTaskHandle(bukkitScheduler.runTaskLaterAsynchronously(GrimACBukkitLoaderPlugin.LOADER,
+        return new BukkitTaskHandle(bukkitScheduler.runTaskLaterAsynchronously(
+                GrimACBukkitLoaderPlugin.LOADER,
                 task,
-                PlatformScheduler.convertTimeToTicks(delay, timeUnit))
-        );
+                PlatformScheduler.convertTimeToTicks(delay, timeUnit)
+        ));
     }
 
     @Override
     public TaskHandle runAtFixedRate(@NotNull GrimPlugin plugin, @NotNull Runnable task, long delay, long period, @NotNull TimeUnit timeUnit) {
-        return new BukkitTaskHandle(
-                bukkitScheduler.runTaskTimerAsynchronously(
-                        GrimACBukkitLoaderPlugin.LOADER,
-                        task,
-                        PlatformScheduler.convertTimeToTicks(delay, timeUnit),
-                        PlatformScheduler.convertTimeToTicks(period, timeUnit)
-                )
-        );
+        return new BukkitTaskHandle(bukkitScheduler.runTaskTimerAsynchronously(
+                GrimACBukkitLoaderPlugin.LOADER,
+                task,
+                PlatformScheduler.convertTimeToTicks(delay, timeUnit),
+                PlatformScheduler.convertTimeToTicks(period, timeUnit)
+        ));
     }
 
     @Override
     public TaskHandle runAtFixedRate(@NotNull GrimPlugin plugin, @NotNull Runnable task, long initialDelayTicks, long periodTicks) {
-        return new BukkitTaskHandle(
-                bukkitScheduler.runTaskTimerAsynchronously(GrimACBukkitLoaderPlugin.LOADER, task, initialDelayTicks, periodTicks)
-        );
+        return new BukkitTaskHandle(bukkitScheduler.runTaskTimerAsynchronously(
+                GrimACBukkitLoaderPlugin.LOADER,
+                task,
+                initialDelayTicks,
+                periodTicks
+        ));
     }
 
     @Override

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/scheduler/bukkit/BukkitEntityScheduler.java

@@ -11,26 +11,25 @@ import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 
 public class BukkitEntityScheduler implements EntityScheduler {
-
-    private final BukkitScheduler bukkitScheduler = Bukkit.getScheduler();
+    private final BukkitScheduler scheduler = Bukkit.getScheduler();
 
     @Override
     public void execute(@NotNull GrimEntity entity, @NotNull GrimPlugin plugin, @NotNull Runnable run, @Nullable Runnable retired, long delay) {
-        bukkitScheduler.runTaskLater(GrimACBukkitLoaderPlugin.LOADER, run, delay);
+        scheduler.runTaskLater(GrimACBukkitLoaderPlugin.LOADER, run, delay);
     }
 
     @Override
     public TaskHandle run(@NotNull GrimEntity entity, @NotNull GrimPlugin plugin, @NotNull Runnable task, @Nullable Runnable retired) {
-        return new BukkitTaskHandle(bukkitScheduler.runTask(GrimACBukkitLoaderPlugin.LOADER, task));
+        return new BukkitTaskHandle(scheduler.runTask(GrimACBukkitLoaderPlugin.LOADER, task));
     }
 
     @Override
     public TaskHandle runDelayed(@NotNull GrimEntity entity, @NotNull GrimPlugin plugin, @NotNull Runnable task, @Nullable Runnable retired, long delayTicks) {
-        return new BukkitTaskHandle(bukkitScheduler.runTaskLater(GrimACBukkitLoaderPlugin.LOADER, task, delayTicks));
+        return new BukkitTaskHandle(scheduler.runTaskLater(GrimACBukkitLoaderPlugin.LOADER, task, delayTicks));
     }
 
     @Override
     public TaskHandle runAtFixedRate(@NotNull GrimEntity entity, @NotNull GrimPlugin plugin, @NotNull Runnable task, @Nullable Runnable retired, long initialDelayTicks, long periodTicks) {
-        return new BukkitTaskHandle(bukkitScheduler.runTaskTimer(GrimACBukkitLoaderPlugin.LOADER, task, initialDelayTicks, periodTicks));
+        return new BukkitTaskHandle(scheduler.runTaskTimer(GrimACBukkitLoaderPlugin.LOADER, task, initialDelayTicks, periodTicks));
     }
 }

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/scheduler/bukkit/BukkitGlobalRegionScheduler.java

@@ -13,8 +13,8 @@ public class BukkitGlobalRegionScheduler implements GlobalRegionScheduler {
     private final BukkitScheduler bukkitScheduler = Bukkit.getScheduler();
 
     @Override
-    public void execute(@NotNull GrimPlugin plugin, @NotNull Runnable run) {
-        bukkitScheduler.runTask(GrimACBukkitLoaderPlugin.LOADER, run);
+    public void execute(@NotNull GrimPlugin plugin, @NotNull Runnable task) {
+        bukkitScheduler.runTask(GrimACBukkitLoaderPlugin.LOADER, task);
     }
 
     @Override

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/scheduler/bukkit/BukkitPlatformScheduler.java

@@ -1,36 +1,13 @@
 package ac.grim.grimac.platform.bukkit.scheduler.bukkit;
 
-import ac.grim.grimac.platform.api.scheduler.AsyncScheduler;
-import ac.grim.grimac.platform.api.scheduler.EntityScheduler;
-import ac.grim.grimac.platform.api.scheduler.GlobalRegionScheduler;
 import ac.grim.grimac.platform.api.scheduler.PlatformScheduler;
-import ac.grim.grimac.platform.api.scheduler.RegionScheduler;
-import org.checkerframework.checker.nullness.qual.NonNull;
+import lombok.Getter;
+import org.jetbrains.annotations.NotNull;
 
+@Getter
 public class BukkitPlatformScheduler implements PlatformScheduler {
-
-    private final BukkitAsyncScheduler bukkitAsyncScheduler = new BukkitAsyncScheduler();
-    private final BukkitGlobalRegionScheduler bukkitGlobalRegionScheduler = new BukkitGlobalRegionScheduler();
-    private final BukkitEntityScheduler bukkitEntityScheduler = new BukkitEntityScheduler();
-    private final BukkitRegionScheduler bukkitRegionScheduler = new BukkitRegionScheduler();
-
-    @Override
-    public @NonNull AsyncScheduler getAsyncScheduler() {
-        return bukkitAsyncScheduler;
-    }
-
-    @Override
-    public @NonNull GlobalRegionScheduler getGlobalRegionScheduler() {
-        return bukkitGlobalRegionScheduler;
-    }
-
-    @Override
-    public @NonNull EntityScheduler getEntityScheduler() {
-        return bukkitEntityScheduler;
-    }
-
-    @Override
-    public @NonNull RegionScheduler getRegionScheduler() {
-        return bukkitRegionScheduler;
-    }
+    private final @NotNull BukkitAsyncScheduler asyncScheduler = new BukkitAsyncScheduler();
+    private final @NotNull BukkitGlobalRegionScheduler globalRegionScheduler = new BukkitGlobalRegionScheduler();
+    private final @NotNull BukkitEntityScheduler entityScheduler = new BukkitEntityScheduler();
+    private final @NotNull BukkitRegionScheduler regionScheduler = new BukkitRegionScheduler();
 }

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/scheduler/bukkit/BukkitRegionScheduler.java

@@ -15,13 +15,13 @@ public class BukkitRegionScheduler implements RegionScheduler {
     private final BukkitScheduler bukkitScheduler = Bukkit.getScheduler();
 
     @Override
-    public void execute(@NotNull GrimPlugin plugin, @NotNull PlatformWorld world, int chunkX, int chunkZ, @NotNull Runnable run) {
-        bukkitScheduler.runTask(GrimACBukkitLoaderPlugin.LOADER, run);
+    public void execute(@NotNull GrimPlugin plugin, @NotNull PlatformWorld world, int chunkX, int chunkZ, @NotNull Runnable task) {
+        bukkitScheduler.runTask(GrimACBukkitLoaderPlugin.LOADER, task);
     }
 
     @Override
-    public void execute(@NotNull GrimPlugin plugin, @NotNull Location location, @NotNull Runnable run) {
-        bukkitScheduler.runTask(GrimACBukkitLoaderPlugin.LOADER, run);
+    public void execute(@NotNull GrimPlugin plugin, @NotNull Location location, @NotNull Runnable task) {
+        bukkitScheduler.runTask(GrimACBukkitLoaderPlugin.LOADER, task);
     }
 
     @Override

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/scheduler/bukkit/BukkitTaskHandle.java

@@ -2,27 +2,32 @@ package ac.grim.grimac.platform.bukkit.scheduler.bukkit;
 
 import ac.grim.grimac.platform.api.scheduler.TaskHandle;
 import org.bukkit.scheduler.BukkitTask;
+import org.jetbrains.annotations.Contract;
+import org.jetbrains.annotations.NotNull;
+
+import java.util.Objects;
 
 public class BukkitTaskHandle implements TaskHandle {
 
-    private final BukkitTask bukkitTask;
+    private final @NotNull BukkitTask task;
 
-    public BukkitTaskHandle(BukkitTask bukkitTask) {
-        this.bukkitTask = bukkitTask;
+    @Contract(pure = true)
+    public BukkitTaskHandle(@NotNull BukkitTask task) {
+        this.task = Objects.requireNonNull(task);
     }
 
     @Override
     public boolean isSync() {
-        return bukkitTask.isSync();
+        return task.isSync();
     }
 
     @Override
     public boolean isCancelled() {
-        return bukkitTask.isCancelled();
+        return task.isCancelled();
     }
 
     @Override
     public void cancel() {
-        bukkitTask.cancel();
+        task.cancel();
     }
 }

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/scheduler/folia/FoliaAsyncScheduler.java

@@ -11,53 +11,47 @@ import java.util.concurrent.TimeUnit;
 
 public class FoliaAsyncScheduler implements AsyncScheduler {
 
-    private final io.papermc.paper.threadedregions.scheduler.AsyncScheduler asyncScheduler = Bukkit.getAsyncScheduler();
+    private final io.papermc.paper.threadedregions.scheduler.AsyncScheduler scheduler = Bukkit.getAsyncScheduler();
 
     @Override
     public TaskHandle runNow(@NotNull GrimPlugin plugin, @NotNull Runnable task) {
-        return new FoliaTaskHandle(asyncScheduler.runNow(GrimACBukkitLoaderPlugin.LOADER, (ignored) -> task.run()));
+        return new FoliaTaskHandle(scheduler.runNow(GrimACBukkitLoaderPlugin.LOADER, ignored -> task.run()));
     }
 
     @Override
     public TaskHandle runDelayed(@NotNull GrimPlugin plugin, @NotNull Runnable task, long delay, @NotNull TimeUnit timeUnit) {
-        return new FoliaTaskHandle(
-                asyncScheduler.runDelayed(
-                        GrimACBukkitLoaderPlugin.LOADER,
-                        (ignored) -> task.run(),
-                        delay,
-                        timeUnit
-                )
-        );
+        return new FoliaTaskHandle(scheduler.runDelayed(
+                GrimACBukkitLoaderPlugin.LOADER,
+                ignored -> task.run(),
+                delay,
+                timeUnit
+        ));
     }
 
     @Override
     public TaskHandle runAtFixedRate(@NotNull GrimPlugin plugin, @NotNull Runnable task, long delay, long period, @NotNull TimeUnit timeUnit) {
-        return new FoliaTaskHandle(
-                asyncScheduler.runAtFixedRate(
-                        GrimACBukkitLoaderPlugin.LOADER,
-                        (ignored) -> task.run(),
-                        delay,
-                        period,
-                        timeUnit
-                )
-        );
+        return new FoliaTaskHandle(scheduler.runAtFixedRate(
+                GrimACBukkitLoaderPlugin.LOADER,
+                ignored -> task.run(),
+                delay,
+                period,
+                timeUnit
+        ));
     }
 
     @Override
     public TaskHandle runAtFixedRate(@NotNull GrimPlugin plugin, @NotNull Runnable task, long initialDelayTicks, long periodTicks) {
-        return new FoliaTaskHandle(
-                asyncScheduler.runAtFixedRate(
-                        GrimACBukkitLoaderPlugin.LOADER,
-                        (ignored) -> task.run(),
-                        initialDelayTicks * 50,
-                        periodTicks * 50,
-                        TimeUnit.MILLISECONDS
-                )
-        );
+        return new FoliaTaskHandle(scheduler.runAtFixedRate(
+                GrimACBukkitLoaderPlugin.LOADER,
+                ignored -> task.run(),
+                initialDelayTicks * 50,
+                periodTicks * 50,
+                TimeUnit.MILLISECONDS
+        ));
     }
 
     @Override
     public void cancel(@NotNull GrimPlugin plugin) {
-        asyncScheduler.cancelTasks(GrimACBukkitLoaderPlugin.LOADER);
+        scheduler.cancelTasks(GrimACBukkitLoaderPlugin.LOADER);
     }
 }

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/scheduler/folia/FoliaEntityScheduler.java

@@ -6,30 +6,50 @@ import ac.grim.grimac.platform.api.scheduler.EntityScheduler;
 import ac.grim.grimac.platform.api.scheduler.TaskHandle;
 import ac.grim.grimac.platform.bukkit.GrimACBukkitLoaderPlugin;
 import ac.grim.grimac.platform.bukkit.entity.BukkitGrimEntity;
+import io.papermc.paper.threadedregions.scheduler.ScheduledTask;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 
 public class FoliaEntityScheduler implements EntityScheduler {
 
     @Override
-    public void execute(@NotNull GrimEntity entity, @NotNull GrimPlugin plugin, @NotNull Runnable run, @Nullable Runnable retired, long delay) {
-        ((BukkitGrimEntity) entity).getBukkitEntity().getScheduler().execute(GrimACBukkitLoaderPlugin.LOADER, run, retired, delay);
+    public void execute(@NotNull GrimEntity entity, @NotNull GrimPlugin plugin, @NotNull Runnable task, @Nullable Runnable retired, long delay) {
+        ((BukkitGrimEntity) entity).getBukkitEntity().getScheduler().execute(GrimACBukkitLoaderPlugin.LOADER, task, retired, delay);
     }
 
     @Override
     public TaskHandle run(@NotNull GrimEntity entity, @NotNull GrimPlugin plugin, @NotNull Runnable task, @Nullable Runnable retired) {
-        return new FoliaTaskHandle(((BukkitGrimEntity) entity).getBukkitEntity().getScheduler().run(GrimACBukkitLoaderPlugin.LOADER, (ignored) -> task.run(), retired));
+        ScheduledTask scheduled = ((BukkitGrimEntity) entity).getBukkitEntity().getScheduler().run(
+                GrimACBukkitLoaderPlugin.LOADER,
+                ignored -> task.run(),
+                retired
+        );
+
+        return scheduled == null ? null : new FoliaTaskHandle(scheduled);
     }
 
     @Override
     public TaskHandle runDelayed(@NotNull GrimEntity entity, @NotNull GrimPlugin plugin, @NotNull Runnable task, @Nullable Runnable retired, long delayTicks) {
-        return new FoliaTaskHandle(
-                ((BukkitGrimEntity) entity).getBukkitEntity().getScheduler().runDelayed(GrimACBukkitLoaderPlugin.LOADER, (ignored) -> task.run(), retired, delayTicks)
+        ScheduledTask scheduled = ((BukkitGrimEntity) entity).getBukkitEntity().getScheduler().runDelayed(
+                GrimACBukkitLoaderPlugin.LOADER,
+                ignored -> task.run(),
+                retired,
+                delayTicks
         );
+
+        return scheduled == null ? null : new FoliaTaskHandle(scheduled);
     }
 
     @Override
     public TaskHandle runAtFixedRate(@NotNull GrimEntity entity, @NotNull GrimPlugin plugin, @NotNull Runnable task, @Nullable Runnable retired, long initialDelayTicks, long periodTicks) {
-        return new FoliaTaskHandle(((BukkitGrimEntity) entity).getBukkitEntity().getScheduler().runAtFixedRate(GrimACBukkitLoaderPlugin.LOADER, (ignored) -> task.run(), retired, initialDelayTicks, periodTicks));
+        ScheduledTask scheduled = ((BukkitGrimEntity) entity).getBukkitEntity().getScheduler().runAtFixedRate(
+                GrimACBukkitLoaderPlugin.LOADER,
+                ignored -> task.run(),
+                retired,
+                initialDelayTicks,
+                periodTicks
+        );
+
+        return scheduled == null ? null : new FoliaTaskHandle(scheduled);
     }
 }

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/scheduler/folia/FoliaGlobalRegionScheduler.java

@@ -12,23 +12,23 @@ public class FoliaGlobalRegionScheduler implements GlobalRegionScheduler {
     private final io.papermc.paper.threadedregions.scheduler.GlobalRegionScheduler globalRegionScheduler = Bukkit.getGlobalRegionScheduler();
 
     @Override
-    public void execute(@NotNull GrimPlugin plugin, @NotNull Runnable run) {
-        globalRegionScheduler.execute(GrimACBukkitLoaderPlugin.LOADER, run);
+    public void execute(@NotNull GrimPlugin plugin, @NotNull Runnable task) {
+        globalRegionScheduler.execute(GrimACBukkitLoaderPlugin.LOADER, task);
     }
 
     @Override
     public TaskHandle run(@NotNull GrimPlugin plugin, @NotNull Runnable task) {
-        return new FoliaTaskHandle(globalRegionScheduler.run(GrimACBukkitLoaderPlugin.LOADER, (ignored) -> task.run()));
+        return new FoliaTaskHandle(globalRegionScheduler.run(GrimACBukkitLoaderPlugin.LOADER, ignored -> task.run()));
     }
 
     @Override
     public TaskHandle runDelayed(@NotNull GrimPlugin plugin, @NotNull Runnable task, long delay) {
-        return new FoliaTaskHandle(globalRegionScheduler.runDelayed(GrimACBukkitLoaderPlugin.LOADER, (ignored) -> task.run(), delay));
+        return new FoliaTaskHandle(globalRegionScheduler.runDelayed(GrimACBukkitLoaderPlugin.LOADER, ignored -> task.run(), delay));
     }
 
     @Override
     public TaskHandle runAtFixedRate(@NotNull GrimPlugin plugin, @NotNull Runnable task, long initialDelayTicks, long periodTicks) {
-        return new FoliaTaskHandle(globalRegionScheduler.runAtFixedRate(GrimACBukkitLoaderPlugin.LOADER, (ignored) -> task.run(), initialDelayTicks, periodTicks));
+        return new FoliaTaskHandle(globalRegionScheduler.runAtFixedRate(GrimACBukkitLoaderPlugin.LOADER, ignored -> task.run(), initialDelayTicks, periodTicks));
     }
 
     @Override

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/scheduler/folia/FoliaPlatformScheduler.java

@@ -1,36 +1,13 @@
 package ac.grim.grimac.platform.bukkit.scheduler.folia;
 
-import ac.grim.grimac.platform.api.scheduler.AsyncScheduler;
-import ac.grim.grimac.platform.api.scheduler.EntityScheduler;
-import ac.grim.grimac.platform.api.scheduler.GlobalRegionScheduler;
 import ac.grim.grimac.platform.api.scheduler.PlatformScheduler;
-import ac.grim.grimac.platform.api.scheduler.RegionScheduler;
-import org.checkerframework.checker.nullness.qual.NonNull;
+import lombok.Getter;
+import org.jetbrains.annotations.NotNull;
 
+@Getter
 public class FoliaPlatformScheduler implements PlatformScheduler {
-
-    private final FoliaAsyncScheduler foliaAsyncScheduler = new FoliaAsyncScheduler();
-    private final FoliaGlobalRegionScheduler foliaGlobalRegionScheduler = new FoliaGlobalRegionScheduler();
-    private final FoliaEntityScheduler foliaEntityScheduler = new FoliaEntityScheduler();
-    private final FoliaRegionScheduler foliaRegionScheduler = new FoliaRegionScheduler();
-
-    @Override
-    public @NonNull AsyncScheduler getAsyncScheduler() {
-        return foliaAsyncScheduler;
-    }
-
-    @Override
-    public @NonNull GlobalRegionScheduler getGlobalRegionScheduler() {
-        return foliaGlobalRegionScheduler;
-    }
-
-    @Override
-    public @NonNull EntityScheduler getEntityScheduler() {
-        return foliaEntityScheduler;
-    }
-
-    @Override
-    public @NonNull RegionScheduler getRegionScheduler() {
-        return foliaRegionScheduler;
-    }
+    private final @NotNull FoliaAsyncScheduler asyncScheduler = new FoliaAsyncScheduler();
+    private final @NotNull FoliaGlobalRegionScheduler globalRegionScheduler = new FoliaGlobalRegionScheduler();
+    private final @NotNull FoliaEntityScheduler entityScheduler = new FoliaEntityScheduler();
+    private final @NotNull FoliaRegionScheduler regionScheduler = new FoliaRegionScheduler();
 }

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/scheduler/folia/FoliaRegionScheduler.java

@@ -15,19 +15,24 @@ public class FoliaRegionScheduler implements RegionScheduler {
     private final io.papermc.paper.threadedregions.scheduler.RegionScheduler regionScheduler = Bukkit.getRegionScheduler();
 
     @Override
-    public void execute(@NotNull GrimPlugin plugin, @NotNull PlatformWorld world, int chunkX, int chunkZ, @NotNull Runnable run) {
-        regionScheduler.execute(GrimACBukkitLoaderPlugin.LOADER, ((BukkitPlatformWorld) world).getBukkitWorld(), chunkX, chunkZ, run);
+    public void execute(@NotNull GrimPlugin plugin, @NotNull PlatformWorld world, int chunkX, int chunkZ, @NotNull Runnable task) {
+        regionScheduler.execute(GrimACBukkitLoaderPlugin.LOADER, ((BukkitPlatformWorld) world).getBukkitWorld(), chunkX, chunkZ, task);
     }
 
     @Override
-    public void execute(@NotNull GrimPlugin plugin, @NotNull Location location, @NotNull Runnable run) {
-        execute(plugin, location.getWorld(), location.getBlockX() >> 4, location.getBlockZ() >> 4, run);
-
+    public void execute(@NotNull GrimPlugin plugin, @NotNull Location location, @NotNull Runnable task) {
+        execute(plugin, location.getWorld(), location.getBlockX() >> 4, location.getBlockZ() >> 4, task);
     }
 
     @Override
     public TaskHandle run(@NotNull GrimPlugin plugin, @NotNull PlatformWorld world, int chunkX, int chunkZ, @NotNull Runnable task) {
-        return new FoliaTaskHandle(regionScheduler.run(GrimACBukkitLoaderPlugin.LOADER, ((BukkitPlatformWorld) world).getBukkitWorld(), chunkX, chunkZ, (ignored) -> task.run()));
+        return new FoliaTaskHandle(regionScheduler.run(
+                GrimACBukkitLoaderPlugin.LOADER,
+                ((BukkitPlatformWorld) world).getBukkitWorld(),
+                chunkX,
+                chunkZ,
+                ignored -> task.run()
+        ));
     }
 
     @Override
@@ -37,8 +42,14 @@ public class FoliaRegionScheduler implements RegionScheduler {
 
     @Override
     public TaskHandle runDelayed(@NotNull GrimPlugin plugin, @NotNull PlatformWorld world, int chunkX, int chunkZ, @NotNull Runnable task, long delayTicks) {
-        return new FoliaTaskHandle(regionScheduler.runDelayed(GrimACBukkitLoaderPlugin.LOADER,
-                ((BukkitPlatformWorld) world).getBukkitWorld(), chunkX, chunkZ, (ignored) -> task.run(), delayTicks));
+        return new FoliaTaskHandle(regionScheduler.runDelayed(
+                GrimACBukkitLoaderPlugin.LOADER,
+                ((BukkitPlatformWorld) world).getBukkitWorld(),
+                chunkX,
+                chunkZ,
+                ignored -> task.run(),
+                delayTicks
+        ));
     }
 
     @Override
@@ -48,12 +59,27 @@ public class FoliaRegionScheduler implements RegionScheduler {
 
     @Override
     public TaskHandle runAtFixedRate(@NotNull GrimPlugin plugin, @NotNull PlatformWorld world, int chunkX, int chunkZ, @NotNull Runnable task, long initialDelayTicks, long periodTicks) {
-        return new FoliaTaskHandle(regionScheduler.runAtFixedRate(GrimACBukkitLoaderPlugin.LOADER,
-                ((BukkitPlatformWorld) world).getBukkitWorld(), chunkX, chunkZ, (ignored) -> task.run(), initialDelayTicks, periodTicks));
+        return new FoliaTaskHandle(regionScheduler.runAtFixedRate(
+                GrimACBukkitLoaderPlugin.LOADER,
+                ((BukkitPlatformWorld) world).getBukkitWorld(),
+                chunkX,
+                chunkZ,
+                ignored -> task.run(),
+                initialDelayTicks,
+                periodTicks
+        ));
     }
 
     @Override
     public TaskHandle runAtFixedRate(@NotNull GrimPlugin plugin, @NotNull Location location, @NotNull Runnable task, long initialDelayTicks, long periodTicks) {
-        return runAtFixedRate(plugin, location.getWorld(), location.getBlockX() >> 4, location.getBlockZ() >> 4, task, initialDelayTicks, periodTicks);
+        return runAtFixedRate(
+                plugin,
+                location.getWorld(),
+                location.getBlockX() >> 4,
+                location.getBlockZ() >> 4,
+                task,
+                initialDelayTicks,
+                periodTicks
+        );
     }
 }

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/scheduler/folia/FoliaTaskHandle.java

@@ -2,13 +2,18 @@ package ac.grim.grimac.platform.bukkit.scheduler.folia;
 
 import ac.grim.grimac.platform.api.scheduler.TaskHandle;
 import io.papermc.paper.threadedregions.scheduler.ScheduledTask;
+import org.jetbrains.annotations.Contract;
+import org.jetbrains.annotations.NotNull;
+
+import java.util.Objects;
 
 public class FoliaTaskHandle implements TaskHandle {
 
-    private final ScheduledTask task;
+    private final @NotNull ScheduledTask task;
 
-    public FoliaTaskHandle(ScheduledTask task) {
-        this.task = task;
+    @Contract(pure = true)
+    public FoliaTaskHandle(@NotNull ScheduledTask task) {
+        this.task = Objects.requireNonNull(task);
     }
 
     @Override

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/sender/BukkitSenderFactory.java

@@ -3,8 +3,6 @@ package ac.grim.grimac.platform.bukkit.sender;
 import ac.grim.grimac.GrimAPI;
 import ac.grim.grimac.platform.api.sender.Sender;
 import ac.grim.grimac.platform.api.sender.SenderFactory;
-import ac.grim.grimac.platform.bukkit.GrimACBukkitLoaderPlugin;
-import net.kyori.adventure.platform.bukkit.BukkitAudiences;
 import net.kyori.adventure.text.Component;
 import org.bukkit.command.CommandSender;
 import org.bukkit.command.ConsoleCommandSender;
@@ -12,17 +10,12 @@ import org.bukkit.command.RemoteConsoleCommandSender;
 import org.bukkit.entity.Player;
 import org.bukkit.permissions.Permission;
 import org.bukkit.permissions.PermissionDefault;
-import org.checkerframework.checker.nullness.qual.NonNull;
 import org.incendo.cloud.SenderMapper;
+import org.jetbrains.annotations.NotNull;
 
 import java.util.UUID;
 
 public class BukkitSenderFactory extends SenderFactory<CommandSender> implements SenderMapper<CommandSender, Sender> {
-    private final BukkitAudiences audiences;
-
-    public BukkitSenderFactory() {
-        this.audiences = BukkitAudiences.create(GrimACBukkitLoaderPlugin.LOADER);
-    }
 
     @Override
     protected String getName(CommandSender sender) {
@@ -31,7 +24,7 @@ public class BukkitSenderFactory extends SenderFactory<CommandSender> implements
 
     @Override
     protected UUID getUniqueId(CommandSender sender) {
-        return sender instanceof Player ? ((Player) sender).getUniqueId() : Sender.CONSOLE_UUID;
+        return sender instanceof Player player ? player.getUniqueId() : Sender.CONSOLE_UUID;
     }
 
     @Override
@@ -42,12 +35,13 @@ public class BukkitSenderFactory extends SenderFactory<CommandSender> implements
     @Override
     protected void sendMessage(CommandSender sender, Component message) {
         // we can safely send async for players and the console - otherwise, send it sync
+        // Paper/Purpur CommandSender is natively an Adventure Audience, so no platform shim is needed
         if (sender instanceof Player || sender instanceof ConsoleCommandSender || sender instanceof RemoteConsoleCommandSender) {
-            this.audiences.sender(sender).sendMessage(message);
+            sender.sendMessage(message);
         } else {
             GrimAPI.INSTANCE.getScheduler().getGlobalRegionScheduler().run(
                     GrimAPI.INSTANCE.getGrimPlugin(),
-                    () -> this.audiences.sender(sender).sendMessage(message)
+                    () -> sender.sendMessage(message)
             );
         }
     }
@@ -78,12 +72,12 @@ public class BukkitSenderFactory extends SenderFactory<CommandSender> implements
     }
 
     @Override
-    public @NonNull Sender map(@NonNull CommandSender base) {
+    public @NotNull Sender map(@NotNull CommandSender base) {
         return this.wrap(base);
     }
 
     @Override
-    public @NonNull CommandSender reverse(@NonNull Sender mapped) {
+    public @NotNull CommandSender reverse(@NotNull Sender mapped) {
         return this.unwrap(mapped);
     }
 }

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/utils/anticheat/MultiLibUtil.java

@@ -1,5 +1,7 @@
 package ac.grim.grimac.platform.bukkit.utils.anticheat;
 
+import ac.grim.grimac.utils.anticheat.LogUtil;
+import ac.grim.grimac.utils.reflection.ReflectionUtils;
 import com.github.retrooper.packetevents.PacketEvents;
 import com.github.retrooper.packetevents.manager.server.ServerVersion;
 import org.bukkit.entity.Player;
@@ -8,27 +10,17 @@ import java.lang.reflect.Method;
 
 public class MultiLibUtil {
 
-    public final static Method externalPlayerMethod = getMethod(Player.class, "isExternalPlayer");
+    public final static Method externalPlayerMethod = ReflectionUtils.getMethod(Player.class, "isExternalPlayer");
     private static final boolean IS_PRE_1_18 = PacketEvents.getAPI().getServerManager().getVersion().isOlderThan(ServerVersion.V_1_18);
 
-    public static Method getMethod(Class<?> clazz, String methodName) {
-        try {
-            return clazz.getMethod(methodName);
-        } catch (NoSuchMethodException e) {
-            return null;
-        }
-    }
-
     // TODO: cache external players for better performance, but this only matters for people using multi-lib
     public static boolean isExternalPlayer(Player player) {
         if (externalPlayerMethod == null || IS_PRE_1_18) return false;
         try {
             return (boolean) externalPlayerMethod.invoke(player);
         } catch (Exception e) {
-            e.printStackTrace();
+            LogUtil.error("Failed to invoke external player method", e);
             return false;
         }
     }
-
-
 }

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/utils/convert/BukkitConversionUtils.java

@@ -4,9 +4,14 @@ import ac.grim.grimac.platform.api.permissions.PermissionDefaultValue;
 import ac.grim.grimac.platform.bukkit.world.BukkitPlatformWorld;
 import ac.grim.grimac.utils.math.Location;
 import com.github.retrooper.packetevents.protocol.world.BlockFace;
+import org.bukkit.permissions.PermissionDefault;
+import org.jetbrains.annotations.Contract;
+import org.jetbrains.annotations.Nullable;
 
 public class BukkitConversionUtils {
+    @Contract("null -> null; !null -> new")
     public static org.bukkit.Location toBukkitLocation(Location location) {
+        if (location == null) return null;
         return new org.bukkit.Location(((BukkitPlatformWorld) location.getWorld()).getBukkitWorld(), location.getX(), location.getY(), location.getZ(), location.getYaw(), location.getPitch());
     }
 
@@ -15,20 +20,17 @@ public class BukkitConversionUtils {
      *
      * @return The corresponding Bukkit PermissionDefault.
      */
-    public static org.bukkit.permissions.PermissionDefault toBukkitPermissionDefault(PermissionDefaultValue permissionDefaultValue) {
+    @Contract(value = "null -> null; !null -> !null", pure = true)
+    public static @Nullable PermissionDefault toBukkitPermissionDefault(@Nullable PermissionDefaultValue permissionDefaultValue) {
+        if (permissionDefaultValue == null) return null;
         return switch (permissionDefaultValue) {
-            case TRUE -> org.bukkit.permissions.PermissionDefault.TRUE;
-            case FALSE -> org.bukkit.permissions.PermissionDefault.FALSE;
-            case OP -> org.bukkit.permissions.PermissionDefault.OP;
-            case NOT_OP -> org.bukkit.permissions.PermissionDefault.NOT_OP;
+            case TRUE -> PermissionDefault.TRUE;
+            case FALSE -> PermissionDefault.FALSE;
+            case OP -> PermissionDefault.OP;
+            case NOT_OP -> PermissionDefault.NOT_OP;
         };
     }
 
-    /**
-     * For use with bukkit events only
-     * Grim is not meant to be restrained by bukkit!
-     */
-    @Deprecated
     public static BlockFace fromBukkitFace(org.bukkit.block.BlockFace face) {
         return switch (face) {
             case NORTH -> BlockFace.NORTH;

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/utils/reflection/PaperUtils.java

@@ -12,41 +12,29 @@ import org.bukkit.event.Listener;
 import java.util.concurrent.CompletableFuture;
 
 public class PaperUtils {
-
-    public static final boolean PAPER;
-    public static final boolean TICK_END_EVENT;
-    public static final boolean ASYNC_TELEPORT;
-
-    static {
-        PAPER = ReflectionUtils.hasClass("com.destroystokyo.paper.PaperConfig") || ReflectionUtils.hasClass("io.papermc.paper.configuration.Configuration");
-        TICK_END_EVENT = ReflectionUtils.hasClass("com.destroystokyo.paper.event.server.ServerTickEndEvent");
-        ASYNC_TELEPORT = ReflectionUtils.hasMethod(Entity.class, "teleportAsync", org.bukkit.Location.class);
-    }
+    public static final boolean PAPER = ReflectionUtils.hasClass("com.destroystokyo.paper.PaperConfig")
+            || ReflectionUtils.hasClass("io.papermc.paper.configuration.Configuration");
 
     public static CompletableFuture<Boolean> teleportAsync(final Entity entity, final Location location) {
-        if (PAPER) {
-            return entity.teleportAsync(location);
-        } else {
-            return CompletableFuture.completedFuture(entity.teleport(location));
-        }
+        return PAPER ? entity.teleportAsync(location) : CompletableFuture.completedFuture(entity.teleport(location));
     }
 
     @SuppressWarnings("unchecked")
     public static boolean registerTickEndEvent(Listener listener, Runnable runnable) {
-        if (TICK_END_EVENT) {
-            try {
-                Class<?> clazz = ReflectionUtils.getClass("com.destroystokyo.paper.event.server.ServerTickEndEvent");
-                if (clazz == null) return false;
-                GrimACBukkitLoaderPlugin.LOADER.getServer().getPluginManager().registerEvent((Class<? extends Event>) clazz,
-                        listener,
-                        EventPriority.NORMAL,
-                        (l, event) -> runnable.run(), GrimACBukkitLoaderPlugin.LOADER);
-                return true;
-            } catch (Exception e) {
-                LogUtil.exception("Failed to register tick end event", e);
-            }
+        try {
+            Class<?> clazz = ReflectionUtils.getClass("com.destroystokyo.paper.event.server.ServerTickEndEvent");
+            if (clazz == null) return false;
+            GrimACBukkitLoaderPlugin.LOADER.getServer().getPluginManager().registerEvent(
+                    (Class<? extends Event>) clazz,
+                    listener,
+                    EventPriority.NORMAL,
+                    (l, event) -> runnable.run(),
+                    GrimACBukkitLoaderPlugin.LOADER
+            );
+            return true;
+        } catch (Exception e) {
+            LogUtil.error("Failed to register tick end event", e);
         }
         return false;
     }
-
 }

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/world/BukkitPlatformChunk.java

@@ -1,24 +1,26 @@
 package ac.grim.grimac.platform.bukkit.world;
 
+import ac.grim.grimac.GrimAPI;
+import ac.grim.grimac.platform.api.Platform;
 import ac.grim.grimac.platform.api.world.PlatformChunk;
 import com.github.retrooper.packetevents.PacketEvents;
 import com.github.retrooper.packetevents.manager.server.ServerVersion;
 import com.github.retrooper.packetevents.protocol.world.states.WrappedBlockState;
+import lombok.RequiredArgsConstructor;
 import org.bukkit.Chunk;
 import org.bukkit.block.Block;
 import org.bukkit.block.data.BlockData;
 import org.jetbrains.annotations.NotNull;
 
 import java.util.HashMap;
+import java.util.Map;
+import java.util.concurrent.ConcurrentHashMap;
 
+@RequiredArgsConstructor
 public class BukkitPlatformChunk implements PlatformChunk {
-    private static final HashMap<BlockData, Integer> blockDataToId = new HashMap<>();
+    private static final Map<BlockData, Integer> blockDataToId = GrimAPI.INSTANCE.getPlatform() == Platform.FOLIA ? new ConcurrentHashMap<>() : new HashMap<>();
     private static final boolean isFlat = PacketEvents.getAPI().getServerManager().getVersion().isNewerThanOrEquals(ServerVersion.V_1_13);
-    private final Chunk chunk;
-
-    public BukkitPlatformChunk(@NotNull Chunk chunkAt) {
-        this.chunk = chunkAt;
-    }
+    private final @NotNull Chunk chunk;
 
     @Override
     public int getBlockID(int x, int y, int z) {
@@ -26,6 +28,11 @@ public class BukkitPlatformChunk implements PlatformChunk {
 
         return isFlat // Cache blockDataToID because Strings are expensive
                 ? blockDataToId.computeIfAbsent(block.getBlockData(), data -> WrappedBlockState.getByString(PacketEvents.getAPI().getServerManager().getVersion().toClientVersion(), data.getAsString(false)).getGlobalId())
-                : (block.getType().getId() << 4) | block.getData();
+                : getLegacyBlockID(block);
+    }
+
+    @SuppressWarnings({ "deprecation", "UnstableApiUsage" })
+    private static int getLegacyBlockID(@NotNull Block block) {
+        return (block.getType().getId() << 4) | block.getData();
     }
 }

bukkit/src/main/java/ac/grim/grimac/platform/bukkit/world/BukkitPlatformWorld.java

@@ -2,24 +2,26 @@ package ac.grim.grimac.platform.bukkit.world;
 
 import ac.grim.grimac.platform.api.world.PlatformChunk;
 import ac.grim.grimac.platform.api.world.PlatformWorld;
+import com.github.retrooper.packetevents.PacketEvents;
+import com.github.retrooper.packetevents.manager.server.ServerVersion;
 import com.github.retrooper.packetevents.protocol.world.states.WrappedBlockState;
 import io.github.retrooper.packetevents.util.SpigotConversionUtil;
 import lombok.Getter;
+import lombok.RequiredArgsConstructor;
 import org.bukkit.Bukkit;
 import org.bukkit.World;
+import org.bukkit.block.Block;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 
 import java.util.UUID;
 
 @Getter
+@RequiredArgsConstructor
 public class BukkitPlatformWorld implements PlatformWorld {
 
-    private final World bukkitWorld;
-
-    public BukkitPlatformWorld(@NotNull World world) {
-        this.bukkitWorld = world;
-    }
+    private static final boolean LEGACY_SERVER_VERSION = PacketEvents.getAPI().getServerManager().getVersion().isOlderThanOrEquals(ServerVersion.V_1_12_2);
+    private final @NotNull World bukkitWorld;
 
     @Override
     public boolean isChunkLoaded(int chunkX, int chunkZ) {
@@ -28,7 +30,14 @@ public class BukkitPlatformWorld implements PlatformWorld {
 
     @Override
     public WrappedBlockState getBlockAt(int x, int y, int z) {
-        return SpigotConversionUtil.fromBukkitBlockData(bukkitWorld.getBlockAt(x, y, z).getBlockData());
+        if (LEGACY_SERVER_VERSION) {
+            Block block = bukkitWorld.getBlockAt(x, y, z);
+            @SuppressWarnings({"deprecation", "UnstableApiUsage"})
+            int blockId = (block.getType().getId() << 4) | block.getData();
+            return WrappedBlockState.getByGlobalId(blockId);
+        } else {
+            return SpigotConversionUtil.fromBukkitBlockData(bukkitWorld.getBlockAt(x, y, z).getBlockData());
+        }
     }
 
     @Override

cliff.toml

@@ -0,0 +1,47 @@
+# cliff.toml - DEV BUILDS - Now with Author Info
+
+[changelog]
+body = """
+## Changes in this Build (since previous push)
+{%- for group, commits in commits | group_by(attribute="group") %}
+    ### {{ group | striptags | upper_first }}
+    {%- for commit in commits %}
+        - {{ commit.message | upper_first }} - *by {{ commit.author.name }}* {% if commit.breaking %}**[BREAKING]**{% endif %} \
+        {%- if commit.remote and commit.remote.url -%}
+            ([`{{ commit.id | truncate(length=7, end="") }}`]({{ commit.remote.url }}/commit/{{- commit.id -}}))
+        {%- else -%}
+            (`{{ commit.id | truncate(length=7, end="") }}`)
+        {%- endif -%}
+    {% endfor %}
+{% endfor %}
+"""
+trim = true
+
+[git]
+# Your previous settings here are perfect, no changes needed
+filter_unconventional = false
+skip_unconventional = false
+remote = "https://github.com/GrimAnticheat/Grim"
+commit_parsers = [
+    { message = "^feat", group = "🚀 Features" },
+    { message = "^fix", group = "🐛 Fixes" },
+    { message = "^perf", group = "⚡️ Performance" },
+    { message = "^refactor", group = "♻️ Refactoring" },
+    { message = "^docs", group = "📖 Documentation" },
+    { message = "^test", group = "✅ Tests" },
+    { message = "^chore", group = "🤖 Maintenance & CI" },
+    { body = ".*BREAKING CHANGE.*", group = "🚨 Breaking Changes" },
+    { message = ".*", group = "Miscellaneous" } # CHANGED THIS LINE
+]
+commit_preprocessors = [
+    { pattern = 'src/main/java/ac/grim/grimac/checks/.*', group = "✔️ Checks" },
+    { pattern = 'src/main/java/ac/grim/grimac/api/.*', group = "🔌 API" },
+    { pattern = 'src/main/java/ac/grim/grimac/platform/.*', group = "🏗️ Platform" },
+    { pattern = 'src/main/java/ac/grim/grimac/engine/.*', group = "⚙️ Core Engine" },
+    { pattern = 'src/main/java/ac/grim/grimac/config/.*', group = "⚙️ Configuration" },
+    { pattern = 'pom.xml', group = "🤖 Maintenance & CI" },
+    { pattern = '.github/.*', group = "🤖 Maintenance & CI" },
+    { pattern = 'build.gradle.kts', group = "🤖 Maintenance & CI" },
+    { pattern = 'src/.*', group = "⚙️ Core Engine" },
+]
+sort_commits = "oldest"

common/build.gradle.kts

@@ -1,33 +1,52 @@
 import versioning.BuildConfig
 
 plugins {
+    eclipse
     `maven-publish`
     grim.`base-conventions`
 }
 
 repositories {
-    if (BuildConfig.mavenLocalOverride) {
-        mavenLocal()
+    // We still call mavenLocal() conditionally at the top for non-exclusive deps (general fallback)
+    if (BuildConfig.mavenLocalOverride) mavenLocal()
+
+    // Grim API
+    exclusive("https://repo.grim.ac/snapshots") {
+        includeGroup("ac.grim.grimac")
     }
-    maven {
-        name = "papermc"
-        url = uri("https://repo.papermc.io/repository/maven-public/")
+
+    // PacketEvents
+    exclusive("https://repo.codemc.io/repository/maven-releases/", { mavenContent { releasesOnly() } }) {
+        includeGroup("com.github.retrooper")
     }
-    maven("https://hub.spigotmc.org/nexus/content/repositories/snapshots/") // Spigot
-    maven("https://repo.grim.ac/snapshots") { // Grim API
-        content {
-            includeGroup("ac.grim.grimac")
-            includeGroup("com.github.retrooper")
-        }
+
+    // ViaVersion
+    exclusive("https://repo.viaversion.com", { mavenContent { releasesOnly() } }) {
+        includeGroup("com.viaversion")
     }
-    maven("https://repo.viaversion.com") // ViaVersion
-    maven("https://nexus.scarsz.me/content/repositories/releases") // Configuralize
-    maven("https://repo.opencollab.dev/maven-snapshots/") // Floodgate
-    maven("https://repo.opencollab.dev/maven-releases/") // Cumulus (for Floodgate)
-    maven("https://s01.oss.sonatype.org/content/repositories/snapshots/")
-    mavenCentral() // FastUtil
+
+    // Configuralize
+    exclusive("https://nexus.scarsz.me/content/repositories/releases", { mavenContent { releasesOnly() } }) {
+        includeGroup("github.scarsz")
+    }
+
+    // Cumulus
+    exclusive("https://repo.opencollab.dev/maven-releases/", { mavenContent { releasesOnly() } }) {
+        includeGroup("org.geysermc.api")
+    }
+
+    // Floodgate
+    exclusive("https://repo.opencollab.dev/maven-snapshots/", { mavenContent { snapshotsOnly() } }) {
+        includeGroup("org.geysermc.floodgate")
+        includeGroup("org.geysermc.cumulus")
+        includeModule("org.geysermc", "common")
+        includeModule("org.geysermc", "geyser-parent")
+    }
+
+    mavenCentral()
 }
 
+
 dependencies {
     if (BuildConfig.shadePE) {
         api(libs.packetevents.api)
@@ -35,20 +54,32 @@ dependencies {
         compileOnly(libs.packetevents.api)
     }
     api(libs.cloud.core)
-    api("org.incendo:cloud-processors-requirements:1.0.0-rc.1")
-    api("github.scarsz:configuralize:1.4.1:slim") {
+    api(libs.cloud.processors.requirements)
+    api(libs.configuralize) {
+        artifact {
+            classifier = "slim"
+        }
         exclude(group = "org.yaml", module = "snakeyaml")
     }
-    // Bump snakeyaml (transitive dep of configuralize) 1.29 -> 2.2 for geyser-fabric
-    api("org.yaml:snakeyaml:2.2")
+    // Bump snakeyaml (transitive dep of configuralize) 1.29 -> 2.2+ for geyser-fabric
+    api(libs.snakeyaml)
     api(libs.fastutil)
-    api(libs.adventure.text.minimessage)
+    // Provided by the server at runtime; bundling it would shadow the platform's own Adventure
+    compileOnly(libs.adventure.text.minimessage)
     api(libs.jetbrains.annotations)
+    api(libs.hikaricp)
+    api(libs.postgresql) // JDBC driver for /grim history postgresql backend
 
-    api("ac.grim.grimac:GrimAPI:1.1.0.0")
+    api(libs.grim.api)
+    api(libs.grim.internal)
+    compileOnly(libs.grim.internal.shims)
+
+    compileOnly(libs.geyser.base.api) {
+        isTransitive = false // messes with guava otherwise
+    }
 
     compileOnly(libs.floodgate.api)
-    compileOnly(libs.via.version.api)
+    compileOnly(libs.viaversion)
     compileOnly(libs.netty)
 }
 

common/src/main/java/ac/grim/grimac/GrimAPI.java

@@ -1,8 +1,9 @@
 package ac.grim.grimac;
 
 import ac.grim.grimac.api.event.EventBus;
-import ac.grim.grimac.api.event.OptimizedEventBus;
 import ac.grim.grimac.api.plugin.GrimPlugin;
+import ac.grim.grimac.internal.plugin.resolver.GrimExtensionManager;
+import ac.grim.grimac.internal.event.OptimizedEventBus;
 import ac.grim.grimac.manager.AlertManagerImpl;
 import ac.grim.grimac.manager.DiscordManager;
 import ac.grim.grimac.manager.InitManager;
@@ -10,28 +11,24 @@ import ac.grim.grimac.manager.SpectateManager;
 import ac.grim.grimac.manager.TickManager;
 import ac.grim.grimac.manager.config.BaseConfigManager;
 import ac.grim.grimac.manager.init.Initable;
+import ac.grim.grimac.manager.violationdatabase.ViolationDatabaseManager;
 import ac.grim.grimac.platform.api.Platform;
 import ac.grim.grimac.platform.api.PlatformLoader;
 import ac.grim.grimac.platform.api.PlatformServer;
+import ac.grim.grimac.platform.api.command.CommandService;
 import ac.grim.grimac.platform.api.manager.ItemResetHandler;
 import ac.grim.grimac.platform.api.manager.MessagePlaceHolderManager;
-import ac.grim.grimac.platform.api.manager.ParserDescriptorFactory;
 import ac.grim.grimac.platform.api.manager.PermissionRegistrationManager;
 import ac.grim.grimac.platform.api.manager.PlatformPluginManager;
 import ac.grim.grimac.platform.api.player.PlatformPlayerFactory;
 import ac.grim.grimac.platform.api.scheduler.PlatformScheduler;
-import ac.grim.grimac.platform.api.sender.Sender;
 import ac.grim.grimac.platform.api.sender.SenderFactory;
 import ac.grim.grimac.utils.anticheat.PlayerDataManager;
+import ac.grim.grimac.utils.common.arguments.CommonGrimArguments;
 import ac.grim.grimac.utils.reflection.ReflectionUtils;
 import lombok.Getter;
-import org.incendo.cloud.CommandManager;
 import org.jetbrains.annotations.NotNull;
 
-import java.util.Collections;
-import java.util.HashMap;
-import java.util.Map;
-
 @Getter
 public final class GrimAPI {
     public static final GrimAPI INSTANCE = new GrimAPI();
@@ -44,8 +41,10 @@ public final class GrimAPI {
     private final DiscordManager discordManager;
     private final PlayerDataManager playerDataManager;
     private final TickManager tickManager;
+    private final GrimExtensionManager extensionManager;
     private final EventBus eventBus;
     private final GrimExternalAPI externalAPI;
+    private ViolationDatabaseManager violationDatabaseManager;
     private PlatformLoader loader;
     @Getter
     private InitManager initManager;
@@ -58,27 +57,25 @@ public final class GrimAPI {
         this.discordManager = new DiscordManager();
         this.playerDataManager = new PlayerDataManager();
         this.tickManager = new TickManager();
-        this.eventBus = new OptimizedEventBus();
+        this.extensionManager = new GrimExtensionManager();
+        this.eventBus = new OptimizedEventBus(extensionManager);
         this.externalAPI = new GrimExternalAPI(this);
     }
 
+    // the order matters
     private static Platform detectPlatform() {
-        final Map<String, Platform> platforms = Map.of(
-                "io.papermc.paper.threadedregions.RegionizedServer", Platform.FOLIA,
-                "org.bukkit.Bukkit", Platform.BUKKIT,
-                "net.fabricmc.loader.api.FabricLoader", Platform.FABRIC
-        );
-
-        return platforms.entrySet().stream()
-                .filter(entry -> ReflectionUtils.hasClass(entry.getKey()))
-                .map(Map.Entry::getValue)
-                .findFirst()
-                .orElseThrow(() -> new IllegalStateException("Unknown platform!"));
+        Platform override = CommonGrimArguments.PLATFORM_OVERRIDE.value();
+        if (override != null) return override;
+        if (ReflectionUtils.hasClass("io.papermc.paper.threadedregions.RegionizedServer")) return Platform.FOLIA;
+        if (ReflectionUtils.hasClass("org.bukkit.Bukkit")) return Platform.BUKKIT;
+        if (ReflectionUtils.hasClass("net.fabricmc.loader.api.FabricLoader")) return Platform.FABRIC;
+        throw new IllegalStateException("Unknown platform!");
     }
 
     public void load(PlatformLoader platformLoader, Initable... platformSpecificInitables) {
         this.loader = platformLoader;
-        this.initManager = new InitManager(loader.getPacketEvents(), loader::getCommandManager, platformSpecificInitables);
+        this.violationDatabaseManager = new ViolationDatabaseManager(getGrimPlugin());
+        this.initManager = new InitManager(loader.getPacketEvents(), platformSpecificInitables);
         this.initManager.load();
         this.initialized = true;
     }
@@ -101,10 +98,6 @@ public final class GrimAPI {
         return loader.getPlatformPlayerFactory();
     }
 
-    public ParserDescriptorFactory getParserDescriptors() {
-        return loader.getParserDescriptorFactory();
-    }
-
     public GrimPlugin getGrimPlugin() {
         return loader.getPlugin();
     }
@@ -129,8 +122,8 @@ public final class GrimAPI {
         return loader.getMessagePlaceHolderManager();
     }
 
-    public CommandManager<Sender> getCommandManager() {
-        return loader.getCommandManager();
+    public CommandService getCommandService() {
+        return loader.getCommandService();
     }
 
     private void checkInitialized() {
@@ -142,4 +135,8 @@ public final class GrimAPI {
     public PermissionRegistrationManager getPermissionManager() {
         return loader.getPermissionManager();
     }
+
+    public GrimExtensionManager getExtensionManager() {
+        return extensionManager;
+    }
 }

common/src/main/java/ac/grim/grimac/GrimExternalAPI.java

@@ -1,21 +1,21 @@
 package ac.grim.grimac;
 
 import ac.grim.grimac.api.GrimAbstractAPI;
-import ac.grim.grimac.api.plugin.GrimPluginDescription;
 import ac.grim.grimac.api.GrimUser;
 import ac.grim.grimac.api.alerts.AlertManager;
 import ac.grim.grimac.api.config.ConfigManager;
 import ac.grim.grimac.api.event.EventBus;
 import ac.grim.grimac.api.event.events.GrimReloadEvent;
+import ac.grim.grimac.api.plugin.GrimPlugin;
 import ac.grim.grimac.manager.config.ConfigManagerFileImpl;
 import ac.grim.grimac.manager.init.start.StartableInitable;
 import ac.grim.grimac.player.GrimPlayer;
 import ac.grim.grimac.utils.anticheat.LogUtil;
-import ac.grim.grimac.utils.chat.ChatUtil;
+import ac.grim.grimac.utils.anticheat.MessageUtil;
 import ac.grim.grimac.utils.common.ConfigReloadObserver;
-import com.github.retrooper.packetevents.netty.channel.ChannelHelper;
 import lombok.Getter;
-import org.checkerframework.checker.nullness.qual.NonNull;
+import org.bukkit.entity.Player;
+import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 
 import java.util.Map;
@@ -43,10 +43,15 @@ public class GrimExternalAPI implements GrimAbstractAPI, ConfigReloadObserver, S
     }
 
     @Override
-    public @NonNull EventBus getEventBus() {
+    public @NotNull EventBus getEventBus() {
         return api.getEventBus();
     }
 
+    @Override
+    public @Nullable GrimUser getGrimUser(Player player) {
+        return getGrimUser(player.getUniqueId());
+    }
+
     @Override
     public @Nullable GrimUser getGrimUser(UUID uuid) {
         return api.getPlayerDataManager().getPlayer(uuid);
@@ -72,8 +77,7 @@ public class GrimExternalAPI implements GrimAbstractAPI, ConfigReloadObserver, S
 
     @Override
     public String getGrimVersion() {
-        GrimPluginDescription description = GrimAPI.INSTANCE.getGrimPlugin().getDescription();
-        return description.getVersion();
+        return api.getGrimPlugin().getDescription().getVersion();
     }
 
     @Override
@@ -110,6 +114,11 @@ public class GrimExternalAPI implements GrimAbstractAPI, ConfigReloadObserver, S
         return GrimAPI.INSTANCE.getTickManager().currentTick;
     }
 
+    @Override
+    public @NotNull GrimPlugin getGrimPlugin(@NotNull Object o) {
+        return this.api.getExtensionManager().getPlugin(o);
+    }
+
     // on load, load the config & register the service
     public void load() {
         reload(configManagerFile);
@@ -123,7 +132,7 @@ public class GrimExternalAPI implements GrimAbstractAPI, ConfigReloadObserver, S
         try {
             GrimAPI.INSTANCE.getConfigManager().start();
         } catch (Exception e) {
-            e.printStackTrace();
+            LogUtil.error("Failed to start config manager.", e);
         }
     }
 
@@ -159,7 +168,7 @@ public class GrimExternalAPI implements GrimAbstractAPI, ConfigReloadObserver, S
                         () -> GrimAPI.INSTANCE.getEventBus().post(new GrimReloadEvent(true)));
             return true;
         } catch (Exception e) {
-            e.printStackTrace();
+            LogUtil.error("Failed to reload config", e);
         }
         if (started)
             GrimAPI.INSTANCE.getScheduler().getAsyncScheduler().runNow(GrimAPI.INSTANCE.getGrimPlugin(),
@@ -181,14 +190,12 @@ public class GrimExternalAPI implements GrimAbstractAPI, ConfigReloadObserver, S
         GrimAPI.INSTANCE.getAlertManager().reload(configManager);
         GrimAPI.INSTANCE.getDiscordManager().reload();
         GrimAPI.INSTANCE.getSpectateManager().reload();
+        GrimAPI.INSTANCE.getViolationDatabaseManager().reload();
         // Don't reload players if the plugin hasn't started yet
         if (!started) return;
         // Reload checks for all players
-        for (GrimPlayer grimPlayer : GrimAPI.INSTANCE.getPlayerDataManager().getEntries()) {
-            ChannelHelper.runInEventLoop(grimPlayer.user.getChannel(), () -> {
-                grimPlayer.updatePermissions();
-                grimPlayer.reload(configManager);
-            });
+        for (GrimPlayer player : GrimAPI.INSTANCE.getPlayerDataManager().getEntries()) {
+            player.runSafely(() -> player.reload(configManager));
         }
     }
 
@@ -203,7 +210,7 @@ public class GrimExternalAPI implements GrimAbstractAPI, ConfigReloadObserver, S
         variableReplacements.putIfAbsent("%tps%", user -> String.format("%.2f", GrimAPI.INSTANCE.getPlatformServer().getTPS()));
         variableReplacements.putIfAbsent("%version%", GrimUser::getVersionName);
         // static variables
-        staticReplacements.put("%prefix%", ChatUtil.translateAlternateColorCodes('&', GrimAPI.INSTANCE.getConfigManager().getPrefix()));
+        staticReplacements.put("%prefix%", MessageUtil.translateAlternateColorCodes('&', GrimAPI.INSTANCE.getConfigManager().getPrefix()));
         staticReplacements.putIfAbsent("%grim_version%", getGrimVersion());
     }
 }

common/src/main/java/ac/grim/grimac/checks/Check.java

@@ -8,13 +8,15 @@ import ac.grim.grimac.player.GrimPlayer;
 import com.github.retrooper.packetevents.protocol.packettype.PacketType;
 import com.github.retrooper.packetevents.protocol.packettype.PacketTypeCommon;
 import com.github.retrooper.packetevents.protocol.player.ClientVersion;
-import com.github.retrooper.packetevents.wrapper.play.client.WrapperPlayClientPlayerFlying;
+import com.github.retrooper.packetevents.protocol.player.DiggingAction;
 import lombok.Getter;
 import lombok.Setter;
 import org.jetbrains.annotations.NotNull;
 
 import java.util.Objects;
 
+import static com.github.retrooper.packetevents.wrapper.play.client.WrapperPlayClientPlayerFlying.isFlying;
+
 // Class from https://github.com/Tecnio/AntiCheatBase/blob/master/src/main/java/me/tecnio/anticheat/check/Check.java
 @Getter
 public class Check extends GrimProcessor implements AbstractCheck {
@@ -59,22 +61,19 @@ public class Check extends GrimProcessor implements AbstractCheck {
     }
 
     public boolean shouldModifyPackets() {
-        return isEnabled && !player.disableGrim && !player.noModifyPacketPermission && !exemptPermission;
+        return isEnabled
+                && !player.disableGrim
+                && !player.noModifyPacketPermission
+                && !noModifyPacketPermission
+                && !exemptPermission;
     }
 
-    public void updatePermissions() {
-        if (player.platformPlayer == null || configName == null) return;
-        GrimAPI.INSTANCE.getScheduler().getEntityScheduler().run(
-                player.platformPlayer,
-                GrimAPI.INSTANCE.getGrimPlugin(),
-                () -> {
-                    final String id = configName.toLowerCase();
-                    exemptPermission = player.platformPlayer.hasPermission("grim.exempt." + id);
-                    noSetbackPermission = player.platformPlayer.hasPermission("grim.nosetback." + id);
-                    noModifyPacketPermission = player.platformPlayer.hasPermission("grim.nomodifypacket." + id);
-                },
-                () -> {}
-        );
+    public final void updatePermissions() {
+        if (configName == null || player.platformPlayer == null) return;
+        final String id = configName.toLowerCase();
+        exemptPermission = player.platformPlayer.hasPermission("grim.exempt." + id);
+        noSetbackPermission = player.platformPlayer.hasPermission("grim.nosetback." + id);
+        noModifyPacketPermission = player.platformPlayer.hasPermission("grim.nomodifypacket." + id);
     }
 
     public final boolean flagAndAlert(String verbose) {
@@ -136,14 +135,13 @@ public class Check extends GrimProcessor implements AbstractCheck {
     }
 
     @Override
-    public void reload(ConfigManager configuration) {
+    public final void reload(ConfigManager configuration) {
         decay = configuration.getDoubleElse(configName + ".decay", decay);
         setbackVL = configuration.getDoubleElse(configName + ".setbackvl", setbackVL);
         displayName = configuration.getStringElse(configName + ".displayname", checkName);
         description = configuration.getStringElse(configName + ".description", description);
 
         if (setbackVL == -1) setbackVL = Double.MAX_VALUE;
-        updatePermissions();
         onReload(configuration);
     }
 
@@ -171,13 +169,15 @@ public class Check extends GrimProcessor implements AbstractCheck {
         return offset > 0.001 ? String.format("%.5f", offset) : String.format("%.2E", offset);
     }
 
-    public boolean isTransaction(PacketTypeCommon packetType) {
+    public static boolean isTransaction(PacketTypeCommon packetType) {
         return packetType == PacketType.Play.Client.PONG ||
                 packetType == PacketType.Play.Client.WINDOW_CONFIRMATION;
     }
 
-    public boolean isFlying(PacketTypeCommon packetType) {
-        return WrapperPlayClientPlayerFlying.isFlying(packetType);
+    public static boolean isAsync(PacketTypeCommon packetType) {
+        return packetType == PacketType.Play.Client.KEEP_ALIVE
+                || packetType == PacketType.Play.Client.CHUNK_BATCH_ACK
+                || packetType == PacketType.Play.Client.RESOURCE_PACK_STATUS;
     }
 
     public boolean isUpdate(PacketTypeCommon packetType) {
@@ -209,4 +209,10 @@ public class Check extends GrimProcessor implements AbstractCheck {
         return isFlying(packetType);
     }
 
+    // prevent causing exploits with packet cancelling (ie noslow)
+    public boolean canCancel(DiggingAction action) {
+        return action != DiggingAction.RELEASE_USE_ITEM
+                // we check client version here because 1.8- doesn't predict dropping items, so we can cancel them. (see CompensatedInventory)
+                && (action != DiggingAction.DROP_ITEM && action != DiggingAction.DROP_ITEM_STACK || player.getClientVersion().isOlderThanOrEquals(ClientVersion.V_1_8));
+    }
 }

common/src/main/java/ac/grim/grimac/checks/debug/AbstractDebugHandler.java

@@ -4,19 +4,11 @@ import ac.grim.grimac.checks.Check;
 import ac.grim.grimac.player.GrimPlayer;
 
 public abstract class AbstractDebugHandler extends Check {
-
-    protected final GrimPlayer grimPlayer;
-
-    public AbstractDebugHandler(GrimPlayer grimPlayer) {
-        super(grimPlayer);
-        this.grimPlayer = grimPlayer;
+    public AbstractDebugHandler(GrimPlayer player) {
+        super(player);
     }
 
-    public GrimPlayer getPlayer() {
-        return grimPlayer;
-    }
-
-    public abstract void toggleListener(GrimPlayer player);
+    public abstract boolean toggleListener(GrimPlayer player);
 
     public abstract boolean toggleConsoleOutput();
 }

common/src/main/java/ac/grim/grimac/checks/debug/HitboxDebugHandler.java

@@ -0,0 +1,209 @@
+package ac.grim.grimac.checks.debug;
+
+import ac.grim.grimac.player.GrimPlayer;
+import ac.grim.grimac.utils.collisions.datatypes.CollisionBox;
+import ac.grim.grimac.utils.collisions.datatypes.NoCollisionBox;
+import ac.grim.grimac.utils.collisions.datatypes.SimpleCollisionBox;
+import ac.grim.grimac.utils.data.Pair;
+import ac.grim.grimac.utils.math.Vector3dm;
+import com.github.retrooper.packetevents.netty.buffer.ByteBufHelper;
+import com.github.retrooper.packetevents.wrapper.play.server.WrapperPlayServerPluginMessage;
+import io.netty.buffer.ByteBuf;
+import io.netty.buffer.Unpooled;
+
+import java.util.HashSet;
+import java.util.List;
+import java.util.Map;
+import java.util.Set;
+import java.util.concurrent.CopyOnWriteArraySet;
+
+/**
+ * Handles debug visualization of hitboxes and reach calculations for GrimAC.
+ * Sends debug data to clients through plugin messages that can be visualized by compatible clients.
+ */
+public class HitboxDebugHandler extends AbstractDebugHandler {
+
+    /**
+     * Set of players currently listening to hitbox debug data
+     */
+    private final Set<GrimPlayer> listeners = new CopyOnWriteArraySet<>(new HashSet<>());
+
+    /**
+     * Creates a new HitboxDebugHandler for the specified player
+     *
+     * @param grimPlayer The GrimAC player instance to debug
+     */
+    public HitboxDebugHandler(GrimPlayer grimPlayer) {
+        super(grimPlayer);
+    }
+
+    /**
+     * Toggles whether a player receives hitbox debug visualization data.
+     * If the player is already listening, they will be removed. If not, they will be added.
+     *
+     * @param player The player to toggle debug visualization for
+     * @return {@code true} if the player is now listening (was added),
+     *         {@code false} if the player is no longer listening (was removed).
+     */
+    @Override
+    public boolean toggleListener(GrimPlayer player) {
+        boolean wasPresent = listeners.remove(player);
+
+        if (wasPresent) {
+            return false;
+        } else {
+            listeners.add(player);
+            return true;
+        }
+    }
+
+    @Override
+    public boolean toggleConsoleOutput() {
+        throw new UnsupportedOperationException();
+    }
+
+    /**
+     * Sends debug visualization data for reach calculations to all listening players.
+     * The data includes hitboxes, target entities, look vectors, and eye heights used in reach calculations.
+     *
+     * @param hitboxes Map of entity IDs to their collision boxes
+     * @param targetEntities Set of entity IDs that are considered targets
+     * @param lookVecsAndEyeHeights List of pairs containing look vectors and their corresponding eye heights
+     * @param baseX Base x coordinate before eye height adjustments
+     * @param baseY Base y coordinate before eye height adjustments
+     * @param baseZ Base z coordinate before eye height adjustments
+     * @param isPrediction Whether these hitboxes are from a prediction calculation
+     *
+     * Packet Format (Version 1):
+     * - Byte: Version (0)
+     * - Byte: Global flags
+     *   - Bit 0: isPrediction
+     *   - Bits 1-7: Reserved
+     * - Double: Player reach/interact distance
+     * - Vector3d: Base position (3 doubles)
+     * - VarInt: Number of ray traces
+     * - For each ray trace:
+     *   - Double: Eye height delta
+     *   - Vector3d: Look vector (3 doubles)
+     * - VarInt: Number of hitboxes
+     * - For each hitbox:
+     *   - VarInt: Entity ID
+     *   - Byte: Flags
+     *     - Bit 0: Is target entity
+     *     - Bit 1: Is no collision
+     *     - Bits 2-7: Reserved
+     *   - If not NoCollisionBox:
+     *     - Double: minX
+     *     - Double: minY
+     *     - Double: minZ
+     *     - Double: maxX
+     *     - Double: maxY
+     *     - Double: maxZ
+     */
+    public void sendHitboxData(Map<Integer, CollisionBox> hitboxes, Set<Integer> targetEntities,
+                               List<Pair<Vector3dm, Double>> lookVecsAndEyeHeights, double baseX, double baseY, double baseZ,
+                               boolean isPrediction, double reachDistance) {
+        if (!isEnabled()) return;
+
+        ByteBuf buffer = Unpooled.buffer();
+        try {
+            // Version Header
+            buffer.writeByte(1);
+
+            // Global Flags Header
+            // Pack boolean flags into a single byte
+            byte global_flags = 0;
+            global_flags |= (isPrediction ? 1 : 0);     // Bit 0: are the hitboxes from a prediction?
+            // Bits 2-7 reserved for future use
+            buffer.writeByte(global_flags);
+
+            // Write reach distance
+            buffer.writeDouble(reachDistance);
+
+            // Write base position
+            writeVector(buffer, baseX, baseY, baseZ);
+
+            // Write number of ray traces
+            ByteBufHelper.writeVarInt(buffer, lookVecsAndEyeHeights.size());
+
+            // Write all possible ray traces
+            for (Pair<Vector3dm, Double> pair : lookVecsAndEyeHeights) {
+                Vector3dm lookVec = pair.first();
+                double eyeHeight = pair.second();
+
+                // Write eye height delta and look vector
+                // we make them 3 meters shorter because all of the vectors passed into here are made 3 meters longer
+                // then they need to be so grim can report correct reach distance for too far away hits
+                buffer.writeDouble(eyeHeight);
+                writeVector(buffer, lookVec.getX(), lookVec.getY(), lookVec.getZ());
+            }
+
+            // Write number of hitboxes
+            ByteBufHelper.writeVarInt(buffer, hitboxes.size());
+
+            // Write hitbox data
+            for (Map.Entry<Integer, CollisionBox> entry : hitboxes.entrySet()) {
+                int entityId = entry.getKey();
+                CollisionBox box = entry.getValue();
+
+                // Write entity ID
+                ByteBufHelper.writeVarInt(buffer, entityId);
+
+                // Pack boolean flags into a single byte
+                byte flags = 0;
+                flags |= (targetEntities.contains(entityId) ? 1 : 0);     // Bit 0: Is target
+                flags |= (box == NoCollisionBox.INSTANCE ? 2 : 0);        // Bit 1: Is no collision
+                // Bits 2-7 reserved for future use
+                buffer.writeByte(flags);
+
+                // Write box coordinates if it's not a NoCollisionBox
+                if ((flags & 2) == 0) {
+                    SimpleCollisionBox simpleCollisionBox = (SimpleCollisionBox) box;
+                    buffer.writeDouble(simpleCollisionBox.minX);
+                    buffer.writeDouble(simpleCollisionBox.minY);
+                    buffer.writeDouble(simpleCollisionBox.minZ);
+                    buffer.writeDouble(simpleCollisionBox.maxX);
+                    buffer.writeDouble(simpleCollisionBox.maxY);
+                    buffer.writeDouble(simpleCollisionBox.maxZ);
+                }
+            }
+
+            // Convert buffer to byte array
+            byte[] data = new byte[buffer.readableBytes()];
+            buffer.readBytes(data);
+
+            // Create and send packet
+            WrapperPlayServerPluginMessage packet = new WrapperPlayServerPluginMessage(
+                    "grim:debug_hitbox",
+                    data
+            );
+
+            // Send to all listeners
+            for (GrimPlayer listener : listeners) {
+                if (listener != null) {
+                    listener.user.sendPacket(packet);
+                }
+            }
+        } finally {
+            // Release buffer to prevent memory leaks
+            buffer.release();
+        }
+    }
+
+    public boolean isEnabled() {
+        return !listeners.isEmpty();
+    }
+
+    /**
+     * Helper method to write vector coordinates to the ByteBuf
+     * @param buffer The buffer to write to
+     * @param x The x coordinate to write
+     * @param y The y coordinate to write
+     * @param z The z coordinate to write
+     */
+    private void writeVector(ByteBuf buffer, double x, double y, double z) {
+        buffer.writeDouble(x);
+        buffer.writeDouble(y);
+        buffer.writeDouble(z);
+    }
+}

common/src/main/java/ac/grim/grimac/checks/impl/aim/AimDuplicateLook.java

@@ -8,7 +8,7 @@ import ac.grim.grimac.utils.anticheat.update.RotationUpdate;
 
 @CheckData(name = "AimDuplicateLook")
 public class AimDuplicateLook extends Check implements RotationCheck {
-    boolean exempt = false;
+    private boolean exempt;
 
     public AimDuplicateLook(GrimPlayer playerData) {
         super(playerData);

common/src/main/java/ac/grim/grimac/checks/impl/aim/AimModulo360.java

@@ -28,7 +28,7 @@ public class AimModulo360 extends Check implements RotationCheck {
             return;
         }
 
-        if (player.xRot < 360 && player.xRot > -360 && Math.abs(rotationUpdate.getDeltaXRot()) > 320 && Math.abs(lastDeltaYaw) < 30) {
+        if (player.yaw < 360 && player.yaw > -360 && Math.abs(rotationUpdate.getDeltaXRot()) > 320 && Math.abs(lastDeltaYaw) < 30) {
             flagAndAlert();
         } else {
             reward();

common/src/main/java/ac/grim/grimac/checks/impl/aim/processor/AimProcessor.java

@@ -8,7 +8,6 @@ import ac.grim.grimac.utils.data.Pair;
 import ac.grim.grimac.utils.lists.RunningMode;
 import ac.grim.grimac.utils.math.GrimMath;
 
-
 public class AimProcessor extends Check implements RotationCheck {
 
     private static final int SIGNIFICANT_SAMPLES_THRESHOLD = 15;
@@ -19,10 +18,10 @@ public class AimProcessor extends Check implements RotationCheck {
     public double divisorY;
     public double modeX, modeY;
     public double deltaDotsX, deltaDotsY;
-    RunningMode xRotMode = new RunningMode(TOTAL_SAMPLES_THRESHOLD);
-    RunningMode yRotMode = new RunningMode(TOTAL_SAMPLES_THRESHOLD);
-    float lastXRot;
-    float lastYRot;
+    private final RunningMode xRotMode = new RunningMode(TOTAL_SAMPLES_THRESHOLD);
+    private final RunningMode yRotMode = new RunningMode(TOTAL_SAMPLES_THRESHOLD);
+    private float lastXRot;
+    private float lastYRot;
 
     public AimProcessor(GrimPlayer playerData) {
         super(playerData);
@@ -38,7 +37,6 @@ public class AimProcessor extends Check implements RotationCheck {
     public void process(final RotationUpdate rotationUpdate) {
         rotationUpdate.setProcessor(this);
 
-
         float deltaXRot = rotationUpdate.getDeltaXRotABS();
 
         this.divisorX = GrimMath.gcd(deltaXRot, lastXRot);
@@ -47,7 +45,6 @@ public class AimProcessor extends Check implements RotationCheck {
             this.lastXRot = deltaXRot;
         }
 
-
         float deltaYRot = rotationUpdate.getDeltaYRotABS();
 
         this.divisorY = GrimMath.gcd(deltaYRot, lastYRot);

common/src/main/java/ac/grim/grimac/checks/impl/badpackets/BadPacketsA.java

@@ -10,7 +10,7 @@ import com.github.retrooper.packetevents.wrapper.play.client.WrapperPlayClientHe
 
 @CheckData(name = "BadPacketsA", description = "Sent duplicate slot id")
 public class BadPacketsA extends Check implements PacketCheck {
-    int lastSlot = -1;
+    private int lastSlot = -1;
 
     public BadPacketsA(final GrimPlayer player) {
         super(player);

common/src/main/java/ac/grim/grimac/checks/impl/badpackets/BadPacketsC.java

@@ -6,26 +6,21 @@ import ac.grim.grimac.checks.type.PacketCheck;
 import ac.grim.grimac.player.GrimPlayer;
 import com.github.retrooper.packetevents.event.PacketReceiveEvent;
 import com.github.retrooper.packetevents.protocol.packettype.PacketType;
-import com.github.retrooper.packetevents.protocol.player.GameMode;
-import com.github.retrooper.packetevents.wrapper.play.client.WrapperPlayClientInteractEntity;
+import com.github.retrooper.packetevents.wrapper.play.client.WrapperPlayClientEntityAction;
+import org.jetbrains.annotations.NotNull;
 
-@CheckData(name = "BadPacketsC", description = "Interacted with self")
+@CheckData(name = "BadPacketsC", description = "Tried to wake up while not sleeping", experimental = true)
 public class BadPacketsC extends Check implements PacketCheck {
-    public BadPacketsC(GrimPlayer player) {
+    public BadPacketsC(@NotNull GrimPlayer player) {
         super(player);
     }
 
     @Override
     public void onPacketReceive(PacketReceiveEvent event) {
-        if (event.getPacketType() == PacketType.Play.Client.INTERACT_ENTITY) {
-            if (player.gamemode == GameMode.SPECTATOR) return;
-            if (new WrapperPlayClientInteractEntity(event).getEntityId() == player.entityID) {
-                // Instant ban
-                if (flagAndAlert() && shouldModifyPackets()) {
-                    event.setCancelled(true);
-                    player.onPacketCancel();
-                }
-            }
+        if (event.getPacketType() == PacketType.Play.Client.ENTITY_ACTION
+                && new WrapperPlayClientEntityAction(event).getAction() == WrapperPlayClientEntityAction.Action.LEAVE_BED
+                && !player.isInBed) {
+            flagAndAlert();
         }
     }
 }

common/src/main/java/ac/grim/grimac/checks/impl/badpackets/BadPacketsD.java

@@ -22,15 +22,13 @@ public class BadPacketsD extends Check implements PacketCheck {
             final float pitch = new WrapperPlayClientPlayerFlying(event).getLocation().getPitch();
             if (pitch > 90 || pitch < -90) {
                 // Ban.
-                if (flagAndAlert("pitch=" + pitch)) {
-                    if (shouldModifyPackets()) {
-                        // prevent other checks from using an invalid pitch
-                        if (player.yRot > 90) player.yRot = 90;
-                        if (player.yRot < -90) player.yRot = -90;
+                if (flagAndAlert("pitch=" + pitch) && shouldModifyPackets()) {
+                    // prevent other checks from using an invalid pitch
+                    if (player.pitch > 90) player.pitch = 90;
+                    if (player.pitch < -90) player.pitch = -90;
 
-                        event.setCancelled(true);
-                        player.onPacketCancel();
-                    }
+                    event.setCancelled(true);
+                    player.onPacketCancel();
                 }
             }
         }

common/src/main/java/ac/grim/grimac/checks/impl/badpackets/BadPacketsE.java

@@ -14,6 +14,8 @@ import com.github.retrooper.packetevents.wrapper.play.client.WrapperPlayClientPl
 @CheckData(name = "BadPacketsE")
 public class BadPacketsE extends Check implements PacketCheck {
     private int noReminderTicks;
+    private final int maxNoReminderTicks = player.getClientVersion().isOlderThanOrEquals(ClientVersion.V_1_8) ? 20 : 19;
+    private final boolean isViaPleaseStopUsingProtocolHacksOnYourServer = player.getClientVersion().isNewerThanOrEquals(ClientVersion.V_1_21_2) || PacketEvents.getAPI().getServerManager().getVersion().isNewerThanOrEquals(ServerVersion.V_1_21_2);
 
     public BadPacketsE(GrimPlayer player) {
         super(player);
@@ -21,20 +23,17 @@ public class BadPacketsE extends Check implements PacketCheck {
 
     @Override
     public void onPacketReceive(PacketReceiveEvent event) {
-        final boolean isViaPleaseStopUsingProtocolHacksOnYourServer = player.getClientVersion().isNewerThanOrEquals(ClientVersion.V_1_21_2) || PacketEvents.getAPI().getServerManager().getVersion().isNewerThanOrEquals(ServerVersion.V_1_21_2);
         if (event.getPacketType() == PacketType.Play.Client.PLAYER_POSITION_AND_ROTATION ||
                 event.getPacketType() == PacketType.Play.Client.PLAYER_POSITION) {
             noReminderTicks = 0;
         } else if (WrapperPlayClientPlayerFlying.isFlying(event.getPacketType()) && !player.packetStateData.lastPacketWasTeleport) {
-            noReminderTicks++;
+            if (++noReminderTicks > maxNoReminderTicks) {
+                flagAndAlert("ticks=" + noReminderTicks);
+            }
         } else if (event.getPacketType() == PacketType.Play.Client.STEER_VEHICLE
                 || (isViaPleaseStopUsingProtocolHacksOnYourServer && player.inVehicle())) {
             noReminderTicks = 0; // Exempt vehicles
         }
-
-        if (noReminderTicks > 20) {
-            flagAndAlert("ticks=" + noReminderTicks); // ban?  I don't know how this would false
-        }
     }
 
     public void handleRespawn() {

common/src/main/java/ac/grim/grimac/checks/impl/badpackets/BadPacketsH.java

@@ -54,15 +54,11 @@ public class BadPacketsH extends BlockPlaceCheck {
     }
 
     public boolean shouldCancel(int sequence) {
-        if (isSupportedVersion && sequence != lastSequence + 1) {
-            if (flagAndAlert("expected=" + (lastSequence + 1) + ", id=" + sequence) && shouldModifyPackets()) {
-                lastSequence = sequence;
-                return true;
-            }
-        }
-
+        int expected = lastSequence + 1;
         lastSequence = sequence;
-        return false;
+        return isSupportedVersion && sequence != expected
+                && flagAndAlert("expected=" + expected + ", id=" + sequence)
+                && shouldModifyPackets();
     }
 
     public void onWorldChange() {

common/src/main/java/ac/grim/grimac/checks/impl/badpackets/BadPacketsI.java

@@ -16,13 +16,11 @@ public class BadPacketsI extends Check implements PacketCheck {
 
     @Override
     public void onPacketReceive(PacketReceiveEvent event) {
-        if (event.getPacketType() == PacketType.Play.Client.PLAYER_ABILITIES) {
-            if (new WrapperPlayClientPlayerAbilities(event).isFlying() && !player.canFly) {
-                if (flagAndAlert() && shouldModifyPackets()) {
-                    event.setCancelled(true);
-                    player.onPacketCancel();
-                }
-            }
+        if (event.getPacketType() == PacketType.Play.Client.PLAYER_ABILITIES
+                && new WrapperPlayClientPlayerAbilities(event).isFlying() && !player.canFly
+                && flagAndAlert() && shouldModifyPackets()) {
+            event.setCancelled(true);
+            player.onPacketCancel();
         }
     }
 }

common/src/main/java/ac/grim/grimac/checks/impl/badpackets/BadPacketsJ.java

@@ -2,7 +2,7 @@ package ac.grim.grimac.checks.impl.badpackets;
 
 import ac.grim.grimac.checks.Check;
 import ac.grim.grimac.checks.CheckData;
-import ac.grim.grimac.checks.type.PostPredictionCheck;
+import ac.grim.grimac.checks.type.PacketCheck;
 import ac.grim.grimac.player.GrimPlayer;
 import ac.grim.grimac.utils.data.HeadRotation;
 import com.github.retrooper.packetevents.PacketEvents;
@@ -10,14 +10,13 @@ import com.github.retrooper.packetevents.event.PacketReceiveEvent;
 import com.github.retrooper.packetevents.manager.server.ServerVersion;
 import com.github.retrooper.packetevents.protocol.packettype.PacketType;
 import com.github.retrooper.packetevents.protocol.player.ClientVersion;
-import com.github.retrooper.packetevents.protocol.player.GameMode;
 import com.github.retrooper.packetevents.wrapper.play.client.WrapperPlayClientUseItem;
 
 import java.util.ArrayList;
 import java.util.List;
 
-@CheckData(name = "BadPacketsJ", description = "Rotation in use item packet did not match tick rotation", experimental = true)
-public class BadPacketsJ extends Check implements PostPredictionCheck {
+@CheckData(name = "BadPacketsJ", description = "Rotation in use item packet did not match tick rotation")
+public class BadPacketsJ extends Check implements PacketCheck {
     private final List<HeadRotation> rotations = new ArrayList<>();
 
     public BadPacketsJ(GrimPlayer player) {
@@ -26,7 +25,7 @@ public class BadPacketsJ extends Check implements PostPredictionCheck {
 
     @Override
     public void onPacketReceive(PacketReceiveEvent event) {
-        if (player.gamemode == GameMode.SPECTATOR) {
+        if (!player.cameraEntity.isSelf()) {
             rotations.clear();
             return;
         }
@@ -41,12 +40,12 @@ public class BadPacketsJ extends Check implements PostPredictionCheck {
             // due to tick skipping, the rotations sent could be last tick's
             boolean allowLast = player.canSkipTicks() && (event.getPacketType() == PacketType.Play.Client.PLAYER_POSITION_AND_ROTATION || event.getPacketType() == PacketType.Play.Client.PLAYER_ROTATION);
             for (HeadRotation rotation : rotations) {
-                if (rotation.getYaw() == player.xRot && rotation.getPitch() == player.yRot) {
+                if (rotation.yaw() == player.yaw && rotation.pitch() == player.pitch) {
                     allowLast = false;
                     continue;
                 }
 
-                if (rotation.getYaw() == player.lastXRot && rotation.getPitch() == player.lastYRot && allowLast) {
+                if (rotation.yaw() == player.lastYaw && rotation.pitch() == player.lastPitch && allowLast) {
                     continue;
                 }
 

common/src/main/java/ac/grim/grimac/checks/impl/badpackets/BadPacketsK.java

@@ -16,13 +16,11 @@ public class BadPacketsK extends Check implements PacketCheck {
 
     @Override
     public void onPacketReceive(PacketReceiveEvent event) {
-        if (event.getPacketType() == PacketType.Play.Client.SPECTATE) {
-            if (player.gamemode != GameMode.SPECTATOR) {
-                if (flagAndAlert() && shouldModifyPackets()) {
-                    event.setCancelled(true);
-                    player.onPacketCancel();
-                }
-            }
+        if (event.getPacketType() == PacketType.Play.Client.SPECTATE
+                && player.gamemode != GameMode.SPECTATOR
+                && flagAndAlert() && shouldModifyPackets()) {
+            event.setCancelled(true);
+            player.onPacketCancel();
         }
     }
 }

common/src/main/java/ac/grim/grimac/checks/impl/badpackets/BadPacketsL.java

@@ -43,7 +43,7 @@ public class BadPacketsL extends Check implements PacketCheck {
                         + ", face=" + packet.getBlockFace()
                         + ", sequence=" + packet.getSequence()
                         + ", action=" + packet.getAction().toString().toLowerCase(Locale.ROOT)
-                ) && shouldModifyPackets() && packet.getAction() != DiggingAction.RELEASE_USE_ITEM) {
+                ) && shouldModifyPackets() && canCancel(packet.getAction())) {
                     event.setCancelled(true);
                     player.onPacketCancel();
                 }

common/src/main/java/ac/grim/grimac/checks/impl/badpackets/BadPacketsM.java

@@ -0,0 +1,64 @@
+package ac.grim.grimac.checks.impl.badpackets;
+
+import ac.grim.grimac.checks.Check;
+import ac.grim.grimac.checks.CheckData;
+import ac.grim.grimac.checks.type.PacketCheck;
+import ac.grim.grimac.player.GrimPlayer;
+import com.github.retrooper.packetevents.event.PacketReceiveEvent;
+import com.github.retrooper.packetevents.event.PacketSendEvent;
+import com.github.retrooper.packetevents.protocol.packettype.PacketType;
+import com.github.retrooper.packetevents.protocol.player.ClientVersion;
+import com.github.retrooper.packetevents.protocol.player.Combat;
+import com.github.retrooper.packetevents.wrapper.play.client.WrapperPlayClientClientStatus;
+import com.github.retrooper.packetevents.wrapper.play.server.WrapperPlayServerChangeGameState;
+import com.github.retrooper.packetevents.wrapper.play.server.WrapperPlayServerCombatEvent;
+import com.github.retrooper.packetevents.wrapper.play.server.WrapperPlayServerDeathCombatEvent;
+
+@CheckData(name = "BadPacketsM", description = "Tried to respawn while alive", experimental = true)
+public class BadPacketsM extends Check implements PacketCheck {
+    public BadPacketsM(final GrimPlayer player) {
+        super(player);
+    }
+
+    // not a boolean because the server could send packets that cause
+    // the client to send a respawn packet before it receives the first
+    private int exempt;
+
+    @Override
+    public void onPacketReceive(PacketReceiveEvent event) {
+        if (event.getPacketType() == PacketType.Play.Client.CLIENT_STATUS && new WrapperPlayClientClientStatus(event).getAction() == WrapperPlayClientClientStatus.Action.PERFORM_RESPAWN) {
+            if (exempt > 0) {
+                exempt--;
+                return;
+            }
+
+            if (!player.compensatedEntities.self.isDead) {
+                if (flagAndAlert() && shouldModifyPackets()) {
+                    event.setCancelled(true);
+                    player.onPacketCancel();
+                }
+            }
+        }
+    }
+
+    @Override
+    public void onPacketSend(PacketSendEvent event) {
+        if (event.getPacketType() == PacketType.Play.Server.CHANGE_GAME_STATE && player.getClientVersion().isNewerThanOrEquals(ClientVersion.V_1_9)
+                && new WrapperPlayServerChangeGameState(event).getReason() == WrapperPlayServerChangeGameState.Reason.WIN_GAME) {
+            player.addRealTimeTaskNow(() -> exempt++);
+        }
+
+        if (event.getPacketType() == PacketType.Play.Server.DEATH_COMBAT_EVENT && player.getClientVersion().isNewerThanOrEquals(ClientVersion.V_1_9)) {
+            if (new WrapperPlayServerDeathCombatEvent(event).getPlayerId() == player.entityID) {
+                player.addRealTimeTaskNow(() -> exempt++);
+            }
+        }
+
+        if (event.getPacketType() == PacketType.Play.Server.COMBAT_EVENT && player.getClientVersion().isNewerThanOrEquals(ClientVersion.V_1_9)) {
+            WrapperPlayServerCombatEvent packet = new WrapperPlayServerCombatEvent(event);
+            if (packet.getCombat() == Combat.ENTITY_DEAD && packet.getPlayerId() == player.entityID) {
+                player.addRealTimeTaskNow(() -> exempt++);
+            }
+        }
+    }
+}

common/src/main/java/ac/grim/grimac/checks/impl/badpackets/BadPacketsN.java

@@ -2,11 +2,10 @@ package ac.grim.grimac.checks.impl.badpackets;
 
 import ac.grim.grimac.checks.Check;
 import ac.grim.grimac.checks.CheckData;
-import ac.grim.grimac.checks.type.PacketCheck;
 import ac.grim.grimac.player.GrimPlayer;
 
 @CheckData(name = "BadPacketsN", setback = 0)
-public class BadPacketsN extends Check implements PacketCheck {
+public class BadPacketsN extends Check {
     public BadPacketsN(final GrimPlayer player) {
         super(player);
     }

common/src/main/java/ac/grim/grimac/checks/impl/badpackets/BadPacketsO.java

@@ -4,7 +4,6 @@ import ac.grim.grimac.checks.Check;
 import ac.grim.grimac.checks.CheckData;
 import ac.grim.grimac.checks.type.PacketCheck;
 import ac.grim.grimac.player.GrimPlayer;
-import ac.grim.grimac.utils.data.Pair;
 import com.github.retrooper.packetevents.event.PacketReceiveEvent;
 import com.github.retrooper.packetevents.event.PacketSendEvent;
 import com.github.retrooper.packetevents.protocol.packettype.PacketType;
@@ -12,11 +11,10 @@ import com.github.retrooper.packetevents.wrapper.play.client.WrapperPlayClientKe
 import com.github.retrooper.packetevents.wrapper.play.server.WrapperPlayServerKeepAlive;
 
 import java.util.LinkedList;
-import java.util.Queue;
 
 @CheckData(name = "BadPacketsO")
 public class BadPacketsO extends Check implements PacketCheck {
-    Queue<Pair<Long, Long>> keepaliveMap = new LinkedList<>();
+    private final LinkedList<Long> keepalives = new LinkedList<>();
 
     public BadPacketsO(GrimPlayer player) {
         super(player);
@@ -25,37 +23,30 @@ public class BadPacketsO extends Check implements PacketCheck {
     @Override
     public void onPacketSend(PacketSendEvent event) {
         if (event.getPacketType() == PacketType.Play.Server.KEEP_ALIVE) {
-            WrapperPlayServerKeepAlive packet = new WrapperPlayServerKeepAlive(event);
-            keepaliveMap.add(new Pair<>(packet.getId(), System.nanoTime()));
+            keepalives.add(new WrapperPlayServerKeepAlive(event).getId());
         }
     }
 
     @Override
     public void onPacketReceive(PacketReceiveEvent event) {
         if (event.getPacketType() == PacketType.Play.Client.KEEP_ALIVE) {
-            WrapperPlayClientKeepAlive packet = new WrapperPlayClientKeepAlive(event);
+            final long id = new WrapperPlayClientKeepAlive(event).getId();
 
-            long id = packet.getId();
-            boolean hasID = false;
+            for (long keepalive : keepalives) {
+                if (keepalive == id) {
+                    // Found the ID, remove stuff until we get to it (to stop very slow memory leaks)
+                    Long data;
+                    do {
+                        data = keepalives.poll();
+                    } while (data != null && data != id);
 
-            for (Pair<Long, Long> iterator : keepaliveMap) {
-                if (iterator.first() == id) {
-                    hasID = true;
-                    break;
+                    return;
                 }
             }
 
-            if (!hasID) {
-                if (flagAndAlert("id=" + id) && shouldModifyPackets()) {
-                    event.setCancelled(true);
-                    player.onPacketCancel();
-                }
-            } else { // Found the ID, remove stuff until we get to it (to stop very slow memory leaks)
-                Pair<Long, Long> data;
-                do {
-                    data = keepaliveMap.poll();
-                    if (data == null) break;
-                } while (data.first() != id);
+            if (flagAndAlert("id=" + id) && shouldModifyPackets()) {
+                event.setCancelled(true);
+                player.onPacketCancel();
             }
         }
     }

common/src/main/java/ac/grim/grimac/checks/impl/badpackets/BadPacketsP.java

@@ -11,7 +11,7 @@ import com.github.retrooper.packetevents.wrapper.play.client.WrapperPlayClientCl
 import com.github.retrooper.packetevents.wrapper.play.client.WrapperPlayClientClickWindow.WindowClickType;
 import com.github.retrooper.packetevents.wrapper.play.server.WrapperPlayServerOpenWindow;
 
-@CheckData(name = "BadPacketsP", experimental = true)
+@CheckData(name = "BadPacketsP", description = "Invalid window click packet", experimental = true)
 public class BadPacketsP extends Check implements PacketCheck {
 
     private int containerType = -1;

common/src/main/java/ac/grim/grimac/checks/impl/badpackets/BadPacketsR.java

@@ -6,7 +6,6 @@ import ac.grim.grimac.checks.type.PacketCheck;
 import ac.grim.grimac.player.GrimPlayer;
 import com.github.retrooper.packetevents.event.PacketReceiveEvent;
 import com.github.retrooper.packetevents.protocol.packettype.PacketType;
-import com.github.retrooper.packetevents.protocol.player.GameMode;
 
 @CheckData(name = "BadPacketsR", decay = 0.25, experimental = true)
 public class BadPacketsR extends Check implements PacketCheck {
@@ -25,11 +24,12 @@ public class BadPacketsR extends Check implements PacketCheck {
             long ms = (player.getPlayerClockAtLeast() - clock) / 1000000L;
             long diff = (System.currentTimeMillis() - lastTransTime);
             if (diff > 2000 && ms > 2000) {
-                if (positions == 0 && clock != 0 && player.gamemode != GameMode.SPECTATOR && !player.compensatedEntities.self.isDead) {
+                if (positions == 0 && clock != 0 && player.cameraEntity.isSelf() && !player.compensatedEntities.self.isDead) {
                     flag("time=" + ms + "ms, " + "lst=" + diff + "ms, positions=" + positions);
                 } else {
                     reward();
                 }
+                player.compensatedEntities.entitiesRemovedThisTick.clear();
                 player.compensatedWorld.removeInvalidPistonLikeStuff(oldTransId);
                 positions = 0;
                 clock = player.getPlayerClockAtLeast();

common/src/main/java/ac/grim/grimac/checks/impl/badpackets/BadPacketsS.java

@@ -4,6 +4,9 @@ import ac.grim.grimac.checks.Check;
 import ac.grim.grimac.checks.CheckData;
 import ac.grim.grimac.checks.type.PacketCheck;
 import ac.grim.grimac.player.GrimPlayer;
+import com.github.retrooper.packetevents.event.PacketReceiveEvent;
+import com.github.retrooper.packetevents.protocol.packettype.PacketType;
+import com.github.retrooper.packetevents.wrapper.play.client.WrapperPlayClientWindowConfirmation;
 
 @CheckData(name = "BadPacketsS")
 public class BadPacketsS extends Check implements PacketCheck {
@@ -11,4 +14,13 @@ public class BadPacketsS extends Check implements PacketCheck {
         super(player);
     }
 
+    @Override
+    public void onPacketReceive(PacketReceiveEvent event) {
+        if (event.getPacketType() == PacketType.Play.Client.WINDOW_CONFIRMATION
+                && !new WrapperPlayClientWindowConfirmation(event).isAccepted()
+                && flagAndAlert() && shouldModifyPackets()) {
+            event.setCancelled(true);
+            player.onPacketCancel();
+        }
+    }
 }

common/src/main/java/ac/grim/grimac/checks/impl/badpackets/BadPacketsT.java

@@ -10,20 +10,25 @@ import com.github.retrooper.packetevents.protocol.attribute.Attributes;
 import com.github.retrooper.packetevents.protocol.entity.type.EntityTypes;
 import com.github.retrooper.packetevents.protocol.packettype.PacketType;
 import com.github.retrooper.packetevents.protocol.player.ClientVersion;
+import com.github.retrooper.packetevents.util.Vector3f;
 import com.github.retrooper.packetevents.wrapper.play.client.WrapperPlayClientInteractEntity;
 
 @CheckData(name = "BadPacketsT")
 public class BadPacketsT extends Check implements PacketCheck {
-    // 1.7 and 1.8 seem to have different hitbox "expansion" values than 1.9+
-    // https://github.com/GrimAnticheat/Grim/pull/1274#issuecomment-1872458702
-    // https://github.com/GrimAnticheat/Grim/pull/1274#issuecomment-1872533497
-    private final boolean hasLegacyExpansion = player.getClientVersion().isOlderThan(ClientVersion.V_1_9);
-    private final double maxHorizontalDisplacement = 0.3001 + (hasLegacyExpansion ? 0.1 : 0);
-    private final double minVerticalDisplacement = -0.0001 - (hasLegacyExpansion ? 0.1 : 0);
-    private final double maxVerticalDisplacement = 1.8001 + (hasLegacyExpansion ? 0.1 : 0);
+
+    private final double maxHorizontalDisplacement;
+    private final double minVerticalDisplacement;
+    private final double maxVerticalDisplacement;
 
     public BadPacketsT(final GrimPlayer player) {
         super(player);
+        // 1.7 and 1.8 seem to have different hitbox "expansion" values than 1.9+
+        // https://github.com/GrimAnticheat/Grim/pull/1274#issuecomment-1872458702
+        // https://github.com/GrimAnticheat/Grim/pull/1274#issuecomment-1872533497
+        double expansion = player.getClientVersion().isOlderThan(ClientVersion.V_1_9) ? 0.1 : 0;
+        maxHorizontalDisplacement = 0.3001 + expansion;
+        minVerticalDisplacement = -0.0001 - expansion;
+        maxVerticalDisplacement = 1.8001 + expansion;
     }
 
     @Override
@@ -31,37 +36,39 @@ public class BadPacketsT extends Check implements PacketCheck {
         if (event.getPacketType().equals(PacketType.Play.Client.INTERACT_ENTITY)) {
             final WrapperPlayClientInteractEntity wrapper = new WrapperPlayClientInteractEntity(event);
             // Only INTERACT_AT actually has an interaction vector
-            wrapper.getTarget().ifPresent(targetVector -> {
-                final PacketEntity packetEntity = player.compensatedEntities.getEntity(wrapper.getEntityId());
-                // Don't continue if the compensated entity hasn't been resolved
-                if (packetEntity == null) {
-                    return;
-                }
+            if (wrapper.getAction() != WrapperPlayClientInteractEntity.InteractAction.INTERACT_AT) return;
+            Vector3f targetVector = wrapper.getTarget().orElse(null);
+            if (targetVector == null) return; // shouldn't ever happen, but whatever
 
-                // Make sure our target entity is actually a player (Player NPCs work too)
-                if (!EntityTypes.PLAYER.equals(packetEntity.getType())) {
-                    // We can't check for any entity that is not a player
-                    return;
-                }
+            final PacketEntity packetEntity = player.compensatedEntities.getEntity(wrapper.getEntityId());
+            // Don't continue if the compensated entity hasn't been resolved
+            if (packetEntity == null) {
+                return;
+            }
 
-                // Perform the interaction vector check
-                // TODO:
-                //  27/12/2023 - Dynamic values for more than just one entity type?
-                //  28/12/2023 - Player-only is fine
-                //  30/12/2023 - Expansions differ in 1.9+
-                final float scale = (float) packetEntity.getAttributeValue(Attributes.SCALE);
-                if (targetVector.y > (minVerticalDisplacement * scale) && targetVector.y < (maxVerticalDisplacement * scale)
-                        && Math.abs(targetVector.x) < (maxHorizontalDisplacement * scale)
-                        && Math.abs(targetVector.z) < (maxHorizontalDisplacement * scale)) {
-                    return;
-                }
+            // Make sure our target entity is actually a player (Player NPCs work too)
+            if (!EntityTypes.PLAYER.equals(packetEntity.type)) {
+                // We can't check for any entity that is not a player
+                return;
+            }
 
-                // Log the vector
-                final String verbose = String.format("%.5f/%.5f/%.5f",
-                        targetVector.x, targetVector.y, targetVector.z);
-                // We could pretty much ban the player at this point
-                flagAndAlert(verbose);
-            });
+            // Perform the interaction vector check
+            // TODO:
+            //  27/12/2023 - Dynamic values for more than just one entity type?
+            //  28/12/2023 - Player-only is fine
+            //  30/12/2023 - Expansions differ in 1.9+
+            final float scale = (float) packetEntity.getAttributeValue(Attributes.SCALE);
+            if (targetVector.y > (minVerticalDisplacement * scale) && targetVector.y < (maxVerticalDisplacement * scale)
+                    && Math.abs(targetVector.x) < (maxHorizontalDisplacement * scale)
+                    && Math.abs(targetVector.z) < (maxHorizontalDisplacement * scale)) {
+                return;
+            }
+
+            // Log the vector
+            final String verbose = String.format("%.5f/%.5f/%.5f",
+                    targetVector.x, targetVector.y, targetVector.z);
+            // We could pretty much ban the player at this point
+            flagAndAlert(verbose);
         }
     }
 }

common/src/main/java/ac/grim/grimac/checks/impl/badpackets/BadPacketsW.java

@@ -2,11 +2,10 @@ package ac.grim.grimac.checks.impl.badpackets;
 
 import ac.grim.grimac.checks.Check;
 import ac.grim.grimac.checks.CheckData;
-import ac.grim.grimac.checks.type.PacketCheck;
 import ac.grim.grimac.player.GrimPlayer;
 
 @CheckData(name = "BadPacketsW", description = "Interacted with non-existent entity", experimental = true)
-public class BadPacketsW extends Check implements PacketCheck {
+public class BadPacketsW extends Check {
     public BadPacketsW(GrimPlayer player) {
         super(player);
     }

common/src/main/java/ac/grim/grimac/checks/impl/badpackets/BadPacketsX.java

@@ -7,7 +7,6 @@ import ac.grim.grimac.player.GrimPlayer;
 import ac.grim.grimac.utils.anticheat.update.PredictionComplete;
 import com.github.retrooper.packetevents.event.PacketReceiveEvent;
 import com.github.retrooper.packetevents.protocol.packettype.PacketType;
-import com.github.retrooper.packetevents.protocol.player.GameMode;
 import com.github.retrooper.packetevents.wrapper.play.client.WrapperPlayClientEntityAction;
 
 @CheckData(name = "BadPacketsX", experimental = true)
@@ -42,7 +41,7 @@ public class BadPacketsX extends Check implements PostPredictionCheck {
 
     @Override
     public void onPacketReceive(PacketReceiveEvent event) {
-        if (player.gamemode == GameMode.SPECTATOR || isTickPacket(event.getPacketType())) {
+        if (!player.cameraEntity.isSelf() || isTickPacket(event.getPacketType())) {
             sprint = sneak = false;
             return;
         }

common/src/main/java/ac/grim/grimac/checks/impl/badpackets/BadPacketsZ.java

@@ -0,0 +1,32 @@
+package ac.grim.grimac.checks.impl.badpackets;
+
+import ac.grim.grimac.checks.Check;
+import ac.grim.grimac.checks.CheckData;
+import ac.grim.grimac.checks.type.PacketCheck;
+import ac.grim.grimac.player.GrimPlayer;
+import com.github.retrooper.packetevents.event.PacketReceiveEvent;
+import com.github.retrooper.packetevents.protocol.packettype.PacketType;
+
+@CheckData(name = "BadPacketsZ", experimental = true)
+public class BadPacketsZ extends Check implements PacketCheck {
+    private boolean sent;
+
+    public BadPacketsZ(GrimPlayer player) {
+        super(player);
+    }
+
+    @Override
+    public void onPacketReceive(PacketReceiveEvent event) {
+        if (event.getPacketType() == PacketType.Play.Client.CLIENT_TICK_END) {
+            sent = false;
+        }
+
+        if (event.getPacketType() == PacketType.Play.Client.PLAYER_INPUT) {
+            if (sent) {
+                flagAndAlert();
+            }
+
+            sent = true;
+        }
+    }
+}

common/src/main/java/ac/grim/grimac/checks/impl/baritone/Baritone.java

@@ -9,7 +9,7 @@ import ac.grim.grimac.utils.anticheat.update.RotationUpdate;
 import ac.grim.grimac.utils.data.HeadRotation;
 import ac.grim.grimac.utils.math.GrimMath;
 
-// This check has been patched by Baritone for a long time and it also seems to false with cinematic camera now, so it is disabled.
+// This check has been patched by Baritone for a long time, and it also seems to false with cinematic camera now, so it is disabled.
 @CheckData(name = "Baritone")
 public class Baritone extends Check implements RotationCheck {
     private int verbose;
@@ -23,10 +23,10 @@ public class Baritone extends Check implements RotationCheck {
         final HeadRotation from = rotationUpdate.getFrom();
         final HeadRotation to = rotationUpdate.getTo();
 
-        final float deltaPitch = Math.abs(to.getPitch() - from.getPitch());
+        final float deltaPitch = Math.abs(to.pitch() - from.pitch());
 
-        // Baritone works with small degrees, limit to 1 degrees to pick up on baritone slightly moving aim to bypass anticheats
-        if (rotationUpdate.getDeltaXRot() == 0 && deltaPitch > 0 && deltaPitch < 1 && Math.abs(to.getPitch()) != 90.0f) {
+        // Baritone works with small degrees, limit to 1 degree to pick up on baritone slightly moving aim to bypass anticheats
+        if (rotationUpdate.getDeltaXRot() == 0 && deltaPitch > 0 && deltaPitch < 1 && Math.abs(to.pitch()) != 90.0f) {
             if (rotationUpdate.getProcessor().divisorY < GrimMath.MINIMUM_DIVISOR) {
                 verbose++;
                 if (verbose > 8) {

common/src/main/java/ac/grim/grimac/checks/impl/breaking/AirLiquidBreak.java

@@ -12,7 +12,7 @@ import com.github.retrooper.packetevents.protocol.player.DiggingAction;
 import com.github.retrooper.packetevents.protocol.world.states.type.StateType;
 import com.github.retrooper.packetevents.protocol.world.states.type.StateTypes;
 import com.github.retrooper.packetevents.util.Vector3i;
-import org.checkerframework.checker.nullness.qual.NonNull;
+import org.jetbrains.annotations.NotNull;
 
 @CheckData(name = "AirLiquidBreak", description = "Breaking a block that cannot be broken")
 public class AirLiquidBreak extends Check implements BlockBreakCheck {
@@ -20,8 +20,8 @@ public class AirLiquidBreak extends Check implements BlockBreakCheck {
     private int lastTick;
     private boolean didLastFlag;
     // Initialize to non-null values to prevent NPE when checking for blockType properties and if position equals old position
-    private @NonNull Vector3i lastBreakLoc = new Vector3i();
-    private @NonNull StateType lastBlockType = StateTypes.AIR;
+    private @NotNull Vector3i lastBreakLoc = new Vector3i();
+    private @NotNull StateType lastBlockType = StateTypes.AIR;
 
     public AirLiquidBreak(GrimPlayer player) {
         super(player);
@@ -51,7 +51,7 @@ public class AirLiquidBreak extends Check implements BlockBreakCheck {
         lastBlockType = block;
 
         // the block does not have a hitbox
-        boolean invalid = (block == StateTypes.LIGHT && !(player.getInventory().getHeldItem().is(ItemTypes.LIGHT) || player.getInventory().getOffHand().is(ItemTypes.LIGHT)))
+        boolean invalid = (block == StateTypes.LIGHT && !(player.inventory.getHeldItem().is(ItemTypes.LIGHT) || player.inventory.getOffHand().is(ItemTypes.LIGHT)))
                 || block.isAir()
                 || block == StateTypes.WATER
                 || block == StateTypes.LAVA

common/src/main/java/ac/grim/grimac/checks/impl/breaking/FarBreak.java

@@ -10,7 +10,6 @@ import ac.grim.grimac.utils.math.Vector3dm;
 import ac.grim.grimac.utils.math.VectorUtils;
 import com.github.retrooper.packetevents.protocol.attribute.Attributes;
 import com.github.retrooper.packetevents.protocol.player.DiggingAction;
-import com.github.retrooper.packetevents.protocol.player.GameMode;
 
 @CheckData(name = "FarBreak", description = "Breaking blocks too far away", experimental = true)
 public class FarBreak extends Check implements BlockBreakCheck {
@@ -20,15 +19,14 @@ public class FarBreak extends Check implements BlockBreakCheck {
 
     @Override
     public void onBlockBreak(BlockBreak blockBreak) {
-        if (player.gamemode == GameMode.SPECTATOR || player.inVehicle() || blockBreak.action == DiggingAction.CANCELLED_DIGGING)
+        if (!player.cameraEntity.isSelf() || player.inVehicle() || blockBreak.action == DiggingAction.CANCELLED_DIGGING)
             return; // falses
 
         double min = Double.MAX_VALUE;
         for (double d : player.getPossibleEyeHeights()) {
             SimpleCollisionBox box = new SimpleCollisionBox(blockBreak.position);
-            Vector3dm eyes = new Vector3dm(player.x, player.y + d, player.z);
-            Vector3dm best = VectorUtils.cutBoxToVector(eyes, box);
-            min = Math.min(min, eyes.distanceSquared(best));
+            Vector3dm best = VectorUtils.cutBoxToVector(player.x, player.y + d, player.z, box);
+            min = Math.min(min, best.distanceSquared(player.x, player.y + d, player.z));
         }
 
         // getPickRange() determines this?

common/src/main/java/ac/grim/grimac/checks/impl/breaking/FastBreak.java

@@ -7,7 +7,7 @@ import ac.grim.grimac.player.GrimPlayer;
 import ac.grim.grimac.utils.anticheat.update.BlockBreak;
 import ac.grim.grimac.utils.math.GrimMath;
 import ac.grim.grimac.utils.nmsutil.BlockBreakSpeed;
-import ac.grim.grimac.utils.reflection.ViaVersionUtil;
+import ac.grim.grimac.utils.viaversion.ViaVersionUtil;
 import com.github.retrooper.packetevents.PacketEvents;
 import com.github.retrooper.packetevents.event.PacketReceiveEvent;
 import com.github.retrooper.packetevents.protocol.packettype.PacketType;
@@ -53,7 +53,7 @@ public class FastBreak extends Check implements BlockBreakCheck {
     @Override
     public void onBlockBreak(BlockBreak blockBreak) {
         if (blockBreak.action == DiggingAction.START_DIGGING) {
-            if (!ViaVersionUtil.isAvailable()) {
+            if (!ViaVersionUtil.isAvailable) {
                 // Exempt all blocks that do not exist in the player version
                 final WrappedBlockState defaultState = WrappedBlockState.getDefaultState(player.getClientVersion(), blockBreak.block.getType());
                 if (defaultState.getType() == StateTypes.AIR || EXEMPT_STATES.contains(defaultState.getType())) {
@@ -80,7 +80,8 @@ public class FastBreak extends Check implements BlockBreakCheck {
                 blockDelayBalance += 300 - breakDelay;
             }
 
-            if (blockDelayBalance > 1000) { // If more than a second of advantage
+            // Skip flagging clients whose tick rate is unobservable, like MultiBreak does
+            if (blockDelayBalance > 1000 && player.isTickingReliablyFor(3)) { // If more than a second of advantage
                 if (flagAndAlert("delay=" + breakDelay + "ms, type=" + blockBreak.block.getType()) && shouldModifyPackets()) {
                     blockBreak.cancel();
                 }
@@ -102,7 +103,8 @@ public class FastBreak extends Check implements BlockBreakCheck {
                 blockBreakBalance += diff;
             }
 
-            if (blockBreakBalance > 1000) { // If more than a second of advantage
+            // Skip flagging clients whose tick rate is unobservable, like MultiBreak does
+            if (blockBreakBalance > 1000 && player.isTickingReliablyFor(3)) { // If more than a second of advantage
                 if (flagAndAlert("diff=" + diff + "ms, balance=" + blockBreakBalance + "ms, type=" + blockBreak.block.getType()) && shouldModifyPackets()) {
                     blockBreak.cancel();
                 }

common/src/main/java/ac/grim/grimac/checks/impl/breaking/MultiBreak.java

@@ -9,7 +9,6 @@ import ac.grim.grimac.utils.anticheat.update.BlockBreak;
 import ac.grim.grimac.utils.anticheat.update.PredictionComplete;
 import com.github.retrooper.packetevents.event.PacketReceiveEvent;
 import com.github.retrooper.packetevents.protocol.player.DiggingAction;
-import com.github.retrooper.packetevents.protocol.player.GameMode;
 import com.github.retrooper.packetevents.protocol.world.BlockFace;
 import com.github.retrooper.packetevents.util.Vector3i;
 
@@ -53,7 +52,7 @@ public class MultiBreak extends Check implements BlockBreakCheck {
 
     @Override
     public void onPacketReceive(PacketReceiveEvent event) {
-        if (player.gamemode == GameMode.SPECTATOR || isTickPacket(event.getPacketType())) {
+        if (!player.cameraEntity.isSelf() || isTickPacket(event.getPacketType())) {
             hasBroken = false;
         }
     }

common/src/main/java/ac/grim/grimac/checks/impl/breaking/PositionBreakB.java

@@ -9,7 +9,7 @@ import com.github.retrooper.packetevents.protocol.player.ClientVersion;
 import com.github.retrooper.packetevents.protocol.player.DiggingAction;
 import com.github.retrooper.packetevents.protocol.world.BlockFace;
 
-@CheckData(name = "PositionBreakB", experimental = true)
+@CheckData(name = "PositionBreakB")
 public class PositionBreakB extends Check implements BlockBreakCheck {
     private final int releaseFace = player.getClientVersion().isNewerThanOrEquals(ClientVersion.V_1_8) ? 0 : 255;
     private BlockFace lastFace;

common/src/main/java/ac/grim/grimac/checks/impl/breaking/RotationBreak.java

@@ -13,7 +13,6 @@ import ac.grim.grimac.utils.nmsutil.ReachUtils;
 import com.github.retrooper.packetevents.protocol.attribute.Attributes;
 import com.github.retrooper.packetevents.protocol.player.ClientVersion;
 import com.github.retrooper.packetevents.protocol.player.DiggingAction;
-import com.github.retrooper.packetevents.protocol.player.GameMode;
 import com.github.retrooper.packetevents.protocol.world.BlockFace;
 import com.github.retrooper.packetevents.util.Vector3d;
 import com.github.retrooper.packetevents.util.Vector3f;
@@ -34,7 +33,7 @@ public class RotationBreak extends Check implements BlockBreakCheck {
 
     @Override
     public void onBlockBreak(BlockBreak blockBreak) {
-        if (player.gamemode == GameMode.SPECTATOR)
+        if (!player.cameraEntity.isSelf())
             return; // you don't send flying packets when spectating entities
         if (player.inVehicle()) return; // falses
         if (blockBreak.action == DiggingAction.CANCELLED_DIGGING) return; // falses
@@ -50,7 +49,7 @@ public class RotationBreak extends Check implements BlockBreakCheck {
 
     @Override
     public void onPostFlyingBlockBreak(BlockBreak blockBreak) {
-        if (player.gamemode == GameMode.SPECTATOR)
+        if (!player.cameraEntity.isSelf())
             return; // you don't send flying packets when spectating entities
         if (player.inVehicle()) return; // falses
         if (blockBreak.action == DiggingAction.CANCELLED_DIGGING) return; // falses
@@ -92,18 +91,18 @@ public class RotationBreak extends Check implements BlockBreakCheck {
         // End checking if the player is in the block
 
         List<Vector3f> possibleLookDirs = new ArrayList<>(Arrays.asList(
-                new Vector3f(player.lastXRot, player.yRot, 0),
-                new Vector3f(player.xRot, player.yRot, 0)
+                new Vector3f(player.lastYaw, player.pitch, 0),
+                new Vector3f(player.yaw, player.pitch, 0)
         ));
 
         // 1.9+ players could be a tick behind because we don't get skipped ticks
         if (player.getClientVersion().isNewerThanOrEquals(ClientVersion.V_1_9)) {
-            possibleLookDirs.add(new Vector3f(player.lastXRot, player.lastYRot, 0));
+            possibleLookDirs.add(new Vector3f(player.lastYaw, player.lastPitch, 0));
         }
 
         // 1.7 players do not have any of these issues! They are always on the latest look vector
         if (player.getClientVersion().isOlderThan(ClientVersion.V_1_8)) {
-            possibleLookDirs = Collections.singletonList(new Vector3f(player.xRot, player.yRot, 0));
+            possibleLookDirs = Collections.singletonList(new Vector3f(player.yaw, player.pitch, 0));
         }
 
         final double distance = player.compensatedEntities.self.getAttributeValue(Attributes.BLOCK_INTERACTION_RANGE);

common/src/main/java/ac/grim/grimac/checks/impl/chat/ChatA.java

@@ -0,0 +1,33 @@
+package ac.grim.grimac.checks.impl.chat;
+
+import ac.grim.grimac.checks.Check;
+import ac.grim.grimac.checks.CheckData;
+import ac.grim.grimac.checks.type.PacketCheck;
+import ac.grim.grimac.player.GrimPlayer;
+import com.github.retrooper.packetevents.PacketEvents;
+import com.github.retrooper.packetevents.event.PacketReceiveEvent;
+import com.github.retrooper.packetevents.manager.server.ServerVersion;
+import com.github.retrooper.packetevents.protocol.packettype.PacketType;
+import com.github.retrooper.packetevents.wrapper.play.client.WrapperPlayClientTabComplete;
+
+@CheckData(name = "ChatA", experimental = true)
+public class ChatA extends Check implements PacketCheck {
+
+    public ChatA(GrimPlayer player) {
+        super(player);
+    }
+
+    @Override
+    public void onPacketReceive(PacketReceiveEvent event) {
+        if (event.getPacketType() == PacketType.Play.Client.TAB_COMPLETE && PacketEvents.getAPI().getServerManager().getVersion().isNewerThanOrEquals(ServerVersion.V_1_13)) {
+            WrapperPlayClientTabComplete wrapper = new WrapperPlayClientTabComplete(event);
+            String text = wrapper.getText();
+            if (text.equals("/") || text.trim().isEmpty()) {
+                if (flagAndAlert("")) {
+                    event.setCancelled(true);
+                    player.onPacketCancel();
+                }
+            }
+        }
+    }
+}

common/src/main/java/ac/grim/grimac/checks/impl/chat/ChatB.java

@@ -1,4 +1,4 @@
-package ac.grim.grimac.checks.impl.exploit;
+package ac.grim.grimac.checks.impl.chat;
 
 import ac.grim.grimac.checks.Check;
 import ac.grim.grimac.checks.CheckData;
@@ -14,21 +14,18 @@ import com.github.retrooper.packetevents.wrapper.play.client.WrapperPlayClientCh
 
 // this can false from click events, but I doubt this would actually
 // happen unless they're trying to flag, or if the server is set up badly
-@CheckData(name = "ExploitC", description = "Invalid chat message")
-public class ExploitC extends Check implements PacketCheck {
-    public ExploitC(GrimPlayer playerData) {
-        super(playerData);
+@CheckData(name = "ChatB", description = "Invalid chat message")
+public class ChatB extends Check implements PacketCheck {
+    public ChatB(GrimPlayer player) {
+        super(player);
     }
 
     @Override
     public void onPacketReceive(PacketReceiveEvent event) {
         if (event.getPacketType() == PacketType.Play.Client.CHAT_MESSAGE) {
             String message = new WrapperPlayClientChatMessage(event).getMessage();
-            if (message.isEmpty() || !message.trim().equals(message) || message.startsWith("/") && PacketEvents.getAPI().getServerManager().getVersion().isNewerThanOrEquals(ServerVersion.V_1_19)) {
-                if (flagAndAlert("message=" + message)) {
-                    event.setCancelled(true);
-                    player.onPacketCancel();
-                }
+            if (checkChatMessage(message)) {
+                event.setCancelled(true);
             }
         }
 
@@ -43,6 +40,7 @@ public class ExploitC extends Check implements PacketCheck {
         }
 
         if (event.getPacketType() == PacketType.Play.Client.CHAT_COMMAND) {
+            // TODO make previa after making wrapper parse by client version instead of server version
             String command = "/" + new WrapperPlayClientChatCommand(event).getCommand();
             if (!command.trim().equals(command)) {
                 if (flagAndAlert("command=" + command)) {
@@ -52,4 +50,15 @@ public class ExploitC extends Check implements PacketCheck {
             }
         }
     }
+
+    // returns whether the packet should be cancelled
+    public boolean checkChatMessage(String message) {
+        if (message.isEmpty() || !message.trim().equals(message) || message.startsWith("/") && PacketEvents.getAPI().getServerManager().getVersion().isNewerThanOrEquals(ServerVersion.V_1_19)) {
+            if (flagAndAlert("message=" + message) && shouldModifyPackets()) {
+                player.onPacketCancel();
+                return true;
+            }
+        }
+        return false;
+    }
 }