v1.3.2

use runAutoTest

.github/ISSUE_TEMPLATE.md

@@ -0,0 +1 @@
+<!-- No UwU's or OwO's allowed -->

.github/ISSUE_TEMPLATE/bug.md

@@ -1,35 +0,0 @@
----
-name: Bug report
-about: Please file a separate report for each issue
-title: Please add a brief but descriptive title
-labels: bug
-assignees: ''
----
-
-## Some information
-Operating system:
-Java version:
-Minecraft version:
-Baritone version:
-Other mods (if used): 
-
-## Exception, error or logs
-Please find your `latest.log` or `debug.log` in this folder and attach it to the issue
-
-Linux: `~/.minecraft/logs/`
-
-Windows: `%appdata%/.minecraft/logs/`
-
-Mac: `/Library/Application\ Support/minecraft/logs/`
-
-## How to reproduce
-Add your steps to reproduce the issue/bug experienced here.
-
-## Modified settings
-To get the modified settings run `#modified` in game
-
-## Final checklist
-- [x] I know how to properly use check boxes 
-- [ ] I have included the version of Minecraft I'm running, baritone's version and forge mods (if used).
-- [ ] I have included logs, exceptions and / or steps to reproduce the issue.
-- [ ] I have not used any OwO's or UwU's in this issue.

.github/ISSUE_TEMPLATE/question.md

@@ -1,14 +0,0 @@
----
-name: Question
-about: Please file a separate report for each question
-title: Please add a brief but descriptive title
-labels: question
-assignees: ''
----
-
-## What do you need help with?
-With as much detail as possible, describe your question and what you may need help with.
-
-## Final checklist
-- [x] I know how to properly use check boxes 
-- [ ] I have not used any OwO's or UwU's in this issue.

.github/ISSUE_TEMPLATE/suggestion.md

@@ -1,20 +0,0 @@
----
-name: Suggestion
-about: Please file a separate report for each suggestion
-title: Please add a brief but descriptive title
-labels: enhancement
-assignees: ''
----
-
-## Describe your suggestion
-With as much detail as possible, describe what your suggestion would do for Baritone.
-
-## Settings
-If applicable, what settings/customizability should be offered to tweak the functionality of your suggestion.
-
-## Context
-Describe how your suggestion would improve Baritone, or the reason behind it being added.
-
-## Final checklist
-- [x] I know how to properly use check boxes 
-- [ ] I have not used any OwO's or UwU's in this issue.

.github/workflows/gradle_build.yml

@@ -1,39 +0,0 @@
-# This workflow will build a Java project with Gradle
-# For more information see: https://help.github.com/actions/language-and-framework-guides/building-and-testing-java-with-gradle
-
-name: Java CI with Gradle
-
-on:
-  push:
-  pull_request:
-
-jobs:
-  build:
-
-    runs-on: ubuntu-latest
-
-    steps:
-    - uses: actions/checkout@v2
-    - name: Set up JDK 8
-      uses: actions/setup-java@v2
-      with:
-        java-version: '8'
-        distribution: 'adopt'
-        
-    - name: Grant execute permission for gradlew
-      run: chmod +x gradlew
-      
-    - name: Build with Gradle
-      run: ./gradlew build
-      
-    - name: Archive Artifacts
-      uses: actions/upload-artifact@v2
-      with:
-        name: Artifacts
-        path: dist/
-        
-    - name: Archive mapping.txt
-      uses: actions/upload-artifact@v2
-      with:
-        name: Mappings
-        path: build/tmp/proguard/mapping.txt

.github/workflows/run_tests.yml

@@ -1,26 +0,0 @@
-
-name: Tests
-
-on:
-  push:
-  pull_request:
-
-jobs:
-  test:
-
-    runs-on: ubuntu-latest
-
-    steps:
-    - uses: actions/checkout@v2
-    - name: Set up JDK 9
-      uses: actions/setup-java@v2
-      with:
-        java-version: '8'
-        distribution: 'adopt'
-        
-    - name: Grant execute permission for gradlew
-      run: chmod +x gradlew
-      
-    - name: Executing tests
-      run: ./gradlew test
-    

.gitignore

@@ -11,19 +11,13 @@ build/
 classes/
 *.class
 
+/out
+
 # IntelliJ Files
 .idea/
 *.iml
-*.ipr
-*.iws
 /logs/
 
-# Eclipse Files
-.classpath
-.project
-.settings/
-baritone_Client.launch
-
 # Copyright Files
 !/.idea/copyright/Baritone.xml
 !/.idea/copyright/profiles_settings.xml

.gitmessage

@@ -1,31 +0,0 @@
-<emoji> <title> (<ticket>)
-
-# 📝 Update README.md (WD-1234)
-# ✅ Add unit test for inputs (WD-1234)
-
-# <emoji> can be:
-# 🎨 :art: when improving structure of the code
-# ⚡️ :zap: when improving performance
-# 🔥 :fire: when removing code or files
-# ✨ :sparkles: when introducing new features
-# 🚧 :construction: when work in progress
-# 🔨 :hammer: when refactoring code
-# 📝 :memo: when writing docs
-# 💄 :lipstick: when updating the UI and style files
-# 📈 :chart_with_upwards_trend: when adding analytics or tracking code
-# 🌐 :globe_with_meridians: when adding internationalization and localization
-# ✏️ :pencil2: when fixing typos
-# 🚚 :truck: when moving or renaming files
-# ✅ :white_check_mark: when adding tests
-
-# 👌 :ok_hand: when updating code due to code review changes
-# 🐛 :bug: when fixing a bug
-# 🚑 :ambulance: when doing a critical hotfix
-# 🚨 :rotating_light: when removing linter warnings
-
-# 🔀 :twisted_rightwards_arrows: when merging branches
-# ⬆️ :arrow_up: when upgrading dependencies
-# ⬇️ :arrow_down: when downgrading dependencies
-# 🔧 :wrench: when changing configuration files
-# 🔖 :bookmark: when releasing / version tagging
-# 💚 :green_heart: when fixing the CI build

.travis.yml

@@ -0,0 +1,29 @@
+language: java
+
+sudo: required
+
+services:
+- docker
+
+install:
+- travis_retry docker build -t cabaletta/baritone .
+
+script:
+- docker run --rm cabaletta/baritone ./gradlew javadoc
+- docker run --name baritone cabaletta/baritone /bin/sh -c "/sbin/start-stop-daemon --start --quiet --pidfile /tmp/custom_xvfb_99.pid --make-pidfile --background --exec /usr/bin/Xvfb -- :99 -screen 0 128x128x24 -ac +extension GLX +render; DISPLAY=:99 BARITONE_AUTO_TEST=true ./gradlew runAutoTest; cat /code/autotest/success"
+- docker cp baritone:/code/dist dist
+- ls dist
+- cat dist/checksums.txt
+
+deploy:
+  provider: releases
+  api_key:
+    secure: YOuiXoJNpB4bW89TQoY2IGXg0tqOKls55YMXsSPU6Mx8WzRu8CjjO/A8KA9nGfNrKM+NucjiKr/h53O2Dp2uyy0i0SLvav/G0MaBMeB1NlPRwFopi6tVPNaoZsvr8NW4BIURhspckYLpOTYWnfmOkIv8q7AxrjUZWPKDlq0dte20UxEqUE6msHJ7U9XlKo/4fX40kvWMfwGI2hTyAtL0cRT1QPsd+uW3OQjAPcQj+jKaWld46V8pBK8g9Qde9mo8HC9NBv97zw1bBF1EFkynW569kElHvaS2Opl2QLGaf66guDbpnqDpGHMhQrDdxsZHJ4RksyITn+8A9UArmbkU35BxKqBeQqOWxod2+M0axdLh1pvX43Q1t9n7RiZBf7GvV8vkXL5Sjf8v6Y4LqkJGhvQkTUwpH+0knwrE761DMCtBC34AiWG70D4u7msmhurkflr9kmRHSj/3lyJ1Q2lkt8L+FOAlQBVs64vXTsfgc6Yge7N0O3UD5hCkrDNoz3BzhNBdCkbdxdKCGip71UZgUNkPy9o3ui8jATNj9ypx3+U8ovqP0XWlJqUZmyeXyNGW9NrLeCkRLTlLnZ/dv6OPONa1oAu4TwF1w5A+TGRFZcZjH/PnZKZDQ1OYQOR6drLKRYdr2unvuf5KUKUGqZ7aYtLGhP0rBvGWddRV7DSmX/s=
+  all_branches: true
+  file_glob: true
+  file:
+  - dist/*
+  skip_cleanup: true
+  on:
+    tags: true
+    repo: cabaletta/baritone

CODE_OF_CONDUCT.md

@@ -14,10 +14,10 @@ appearance, race, religion, or sexual identity and orientation.
 Examples of behavior that contributes to creating a positive environment
 include:
 
-* No Anime (including uwu's or owo's)
+* No Anime
 * Using welcoming and inclusive language
 * Being respectful of differing viewpoints and experiences
-* Giving and gracefully accepting constructive criticism
+* Gracefully accepting constructive criticism
 * Focusing on what is best for the community
 * Showing empathy towards other community members
 
@@ -29,7 +29,7 @@ Examples of unacceptable behavior by participants include:
 * ~~Trolling, insulting/derogatory comments, and personal or political attacks~~
 * Public or private harassment
 * Publishing others' private information, such as a physical or electronic
- address, without explicit permission or consent
+ address, without explicit permission
 * Other conduct which could reasonably be considered inappropriate in a
  professional setting
 

Dockerfile

@@ -1,5 +1,7 @@
 FROM debian:stretch
 
+RUN echo 'deb http://deb.debian.org/debian stretch-backports main' > /etc/apt/sources.list.d/stretch-backports.list
+
 ENV DEBIAN_FRONTEND noninteractive
 
 RUN apt update -y
@@ -8,8 +10,14 @@ RUN apt install \
           openjdk-8-jdk \
           --assume-yes
 
+RUN apt install -qq --assume-yes mesa-utils libgl1-mesa-glx libxcursor1 libxrandr2 libxxf86vm1 x11-xserver-utils xfonts-base xserver-common
+
 COPY . /code
 
 WORKDIR /code
 
+# this .deb is specially patched to support lwjgl
+# source: https://github.com/tectonicus/tectonicus/issues/60#issuecomment-154239173
+RUN dpkg -i scripts/xvfb_1.16.4-1_amd64.deb
+
 RUN ./gradlew build

README.md

@@ -1,65 +1,37 @@
 # Baritone
-<p align="center">
-  <a href="https://github.com/cabaletta/baritone/releases/"><img src="https://img.shields.io/github/downloads/cabaletta/baritone/total.svg" alt="GitHub All Releases"/></a>
-</p>
+[![Build Status](https://travis-ci.com/cabaletta/baritone.svg?branch=master)](https://travis-ci.com/cabaletta/baritone)
+[![Release](https://img.shields.io/github/release/cabaletta/baritone.svg)](https://github.com/cabaletta/baritone/releases)
+[![License](https://img.shields.io/badge/license-LGPL--3.0%20with%20anime%20exception-green.svg)](LICENSE)
+[![Codacy Badge](https://api.codacy.com/project/badge/Grade/a73d037823b64a5faf597a18d71e3400)](https://www.codacy.com/app/leijurv/baritone?utm_source=github.com&amp;utm_medium=referral&amp;utm_content=cabaletta/baritone&amp;utm_campaign=Badge_Grade)
+[![HitCount](http://hits.dwyl.com/cabaletta/baritone.svg)](http://hits.dwyl.com/cabaletta/baritone)
+[![Code of Conduct](https://img.shields.io/badge/%E2%9D%A4-code%20of%20conduct-blue.svg?style=flat)](https://github.com/cabaletta/baritone/blob/master/CODE_OF_CONDUCT.md)
+[![Known Vulnerabilities](https://snyk.io/test/github/cabaletta/baritone/badge.svg?targetFile=build.gradle)](https://snyk.io/test/github/cabaletta/baritone?targetFile=build.gradle)
+[![Contributions welcome](https://img.shields.io/badge/contributions-welcome-brightgreen.svg?style=flat)](https://github.com/cabaletta/baritone/issues)
+[![Issues](https://img.shields.io/github/issues/cabaletta/baritone.svg)](https://github.com/cabaletta/baritone/issues/)
+[![GitHub issues-closed](https://img.shields.io/github/issues-closed/cabaletta/baritone.svg)](https://github.com/cabaletta/baritone/issues?q=is%3Aissue+is%3Aclosed)
+[![Pull Requests](https://img.shields.io/github/issues-pr/cabaletta/baritone.svg)](https://github.com/cabaletta/baritone/pulls/)
+![Code size](https://img.shields.io/github/languages/code-size/cabaletta/baritone.svg)
+![GitHub repo size](https://img.shields.io/github/repo-size/cabaletta/baritone.svg)
+![](https://tokei.rs/b1/github/cabaletta/baritone?category=code)
+[![Minecraft](https://img.shields.io/badge/MC-1.13.2-brightgreen.svg)](https://minecraft.gamepedia.com/1.13.2)
+[![GitHub contributors](https://img.shields.io/github/contributors/cabaletta/baritone.svg)](https://github.com/cabaletta/baritone/graphs/contributors/)
+[![GitHub commits](https://img.shields.io/github/commits-since/cabaletta/baritone/v1.0.0.svg)](https://github.com/cabaletta/baritone/commit/)
+[![Impact integration](https://img.shields.io/badge/Impact%20integration-v1.2.5%20/%20v1.3.0-brightgreen.svg)](https://impactdevelopment.github.io/)
+[![WWE integration](https://img.shields.io/badge/WWE%20%22integration%22-master%3F-green.svg)](https://wweclient.com/)
+[![KAMI integration](https://img.shields.io/badge/KAMI%20integration-v1.0.0-red.svg)](https://github.com/zeroeightysix/KAMI/)
+[![Future integration](https://img.shields.io/badge/Future%20integration-Soon™%3F%3F%3F-red.svg)](https://futureclient.net/)
+[![ForgeHax integration](https://img.shields.io/badge/ForgeHax%20integration-Soon™-red.svg)](https://github.com/fr1kin/ForgeHax)
+[![forthebadge](https://forthebadge.com/images/badges/built-with-swag.svg)](http://forthebadge.com)
+[![forthebadge](https://forthebadge.com/images/badges/mom-made-pizza-rolls.svg)](http://forthebadge.com)
 
-<p align="center">
-  <img src="https://img.shields.io/badge/MC-1.12.2-brightgreen.svg" alt="Minecraft"/>
-  <img src="https://img.shields.io/badge/MC-1.13.2-brightgreen.svg" alt="Minecraft"/>
-  <img src="https://img.shields.io/badge/MC-1.14.4-brightgreen.svg" alt="Minecraft"/>
-  <img src="https://img.shields.io/badge/MC-1.15.2-brightgreen.svg" alt="Minecraft"/>
-  <img src="https://img.shields.io/badge/MC-1.16.5-brightgreen.svg" alt="Minecraft"/>
-</p>
+A Minecraft pathfinder bot. 
 
-<p align="center">
-  <a href="https://travis-ci.com/cabaletta/baritone/"><img src="https://travis-ci.com/cabaletta/baritone.svg?branch=master" alt="Build Status"/></a>
-  <a href="https://github.com/cabaletta/baritone/releases/"><img src="https://img.shields.io/github/release/cabaletta/baritone.svg" alt="Release"/></a>
-  <a href="LICENSE"><img src="https://img.shields.io/badge/license-LGPL--3.0%20with%20anime%20exception-green.svg" alt="License"/></a>
-  <a href="https://www.codacy.com/app/leijurv/baritone?utm_source=github.com&amp;utm_medium=referral&amp;utm_content=cabaletta/baritone&amp;utm_campaign=Badge_Grade"><img src="https://api.codacy.com/project/badge/Grade/a73d037823b64a5faf597a18d71e3400" alt="Codacy Badge"/></a>
-  <a href="https://github.com/cabaletta/baritone/blob/master/CODE_OF_CONDUCT.md"><img src="https://img.shields.io/badge/%E2%9D%A4-code%20of%20conduct-blue.svg?style=flat" alt="Code of Conduct"/></a>
-  <a href="https://snyk.io/test/github/cabaletta/baritone?targetFile=build.gradle"><img src="https://snyk.io/test/github/cabaletta/baritone/badge.svg?targetFile=build.gradle" alt="Known Vulnerabilities"/></a>
-  <a href="https://github.com/cabaletta/baritone/issues/"><img src="https://img.shields.io/badge/contributions-welcome-brightgreen.svg?style=flat" alt="Contributions welcome"/></a>
-  <a href="https://github.com/cabaletta/baritone/issues/"><img src="https://img.shields.io/github/issues/cabaletta/baritone.svg" alt="Issues"/></a>
-  <a href="https://github.com/cabaletta/baritone/issues?q=is%3Aissue+is%3Aclosed"><img src="https://img.shields.io/github/issues-closed/cabaletta/baritone.svg" alt="GitHub issues-closed"/></a>
-  <a href="https://github.com/cabaletta/baritone/pulls/"><img src="https://img.shields.io/github/issues-pr/cabaletta/baritone.svg" alt="Pull Requests"/></a>
-  <a href="https://github.com/cabaletta/baritone/graphs/contributors/"><img src="https://img.shields.io/github/contributors/cabaletta/baritone.svg" alt="GitHub contributors"/></a>
-  <a href="https://github.com/cabaletta/baritone/commit/"><img src="https://img.shields.io/github/commits-since/cabaletta/baritone/v1.0.0.svg" alt="GitHub commits"/></a>
-  <img src="https://img.shields.io/github/languages/code-size/cabaletta/baritone.svg" alt="Code size"/>
-  <img src="https://img.shields.io/github/repo-size/cabaletta/baritone.svg" alt="GitHub repo size"/>
-  <img src="https://tokei.rs/b1/github/cabaletta/baritone?category=code" alt="Lines of Code"/>
-</p>
-
-<p align="center">
-  <a href="https://impactclient.net/"><img src="https://img.shields.io/badge/Impact%20integration-v1.2.14%20/%20v1.3.8%20/%20v1.4.6%20/%20v1.5.3%20/%20v1.6.3-brightgreen.svg" alt="Impact integration"/></a>
-  <a href="https://github.com/kami-blue/client"><img src="https://img.shields.io/badge/KAMI%20Blue%20integration-v1.2.14--master-green" alt="KAMI Blue integration"/></a>
-  <a href="https://github.com/fr1kin/ForgeHax/"><img src="https://img.shields.io/badge/ForgeHax%20%22integration%22-scuffed-yellow.svg" alt="ForgeHax integration"/></a>
-  <a href="https://aristois.net/"><img src="https://img.shields.io/badge/Aristois%20add--on%20integration-v1.6.3-green.svg" alt="Aristois add-on integration"/></a>
-  <a href="https://rootnet.dev/"><img src="https://img.shields.io/badge/rootNET%20integration-v1.2.14-green.svg" alt="rootNET integration"/></a>
-  <a href="https://futureclient.net/"><img src="https://img.shields.io/badge/Future%20integration-v1.2.12%20%2F%20v1.3.6%20%2F%20v1.4.4-red" alt="Future integration"/></a>
-  <a href="https://rusherhack.org/"><img src="https://img.shields.io/badge/RusherHack%20integration-v1.2.14-green" alt="RusherHack integration"/></a>
-</p>
-
-<p align="center">
-  <a href="http://forthebadge.com/"><img src="https://forthebadge.com/images/badges/built-with-swag.svg" alt="forthebadge"/></a>
-  <a href="http://forthebadge.com/"><img src="https://forthebadge.com/images/badges/mom-made-pizza-rolls.svg" alt="forthebadge"/></a>
-</p>
-
-A Minecraft pathfinder bot.
-
-[**Baritone Discord Server**](http://discord.gg/s6fRBAUpmr)
-
-Baritone is the pathfinding system used in [Impact](https://impactclient.net/) since 4.4. There's a [showcase video](https://youtu.be/CZkLXWo4Fg4) made by @Adovin#0730 on Baritone which I recommend. [Here's](https://www.youtube.com/watch?v=StquF69-_wI) a (very old!) video I made showing off what it can do. [Tutorial playlist](https://www.youtube.com/playlist?list=PLnwnJ1qsS7CoQl9Si-RTluuzCo_4Oulpa)
-
-The easiest way to install Baritone is to install [Impact](https://impactclient.net/), which comes with Baritone. The second easiest way (for 1.12.2 only) is to install the v1.2.* `api-forge` jar from [releases](https://github.com/cabaletta/baritone/releases). **For 1.12.2 Forge, just click [here](https://github.com/cabaletta/baritone/releases/download/v1.2.15/baritone-api-forge-1.2.15.jar)**. Otherwise, see [Installation & setup](SETUP.md). Once Baritone is installed, look [here](USAGE.md) for instructions on how to use it.
-
-For 1.15.2, [click here](https://www.youtube.com/watch?v=j1qKtCZFURM) and see description. If you need Forge 1.15.2, look [here](https://github.com/cabaletta/baritone/releases/tag/v1.5.3), follow the instructions, and get the `api-forge` jar.
-
-For 1.16.5, [click here](https://www.youtube.com/watch?v=_4eVJ9Qz2J8) and see description. If you need Forge or Fabric 1.16.5, look [here](https://github.com/cabaletta/baritone/releases/tag/v1.6.3) and get the `api-forge` or `api-fabric` jar. **For 1.16.5 Fabric, just click [here](https://github.com/cabaletta/baritone/releases/download/v1.6.3/baritone-api-fabric-1.6.3.jar)**.
+Baritone is the pathfinding system used in [Impact](https://impactdevelopment.github.io/) since 4.4. There's a [showcase video](https://www.youtube.com/watch?v=yI8hgW_m6dQ) made by @Adovin#3153 on Baritone's integration into Impact. [Here's](https://www.youtube.com/watch?v=StquF69-_wI) a video I made showing off what it can do.
 
 This project is an updated version of [MineBot](https://github.com/leijurv/MineBot/),
-the original version of the bot for Minecraft 1.8.9, rebuilt for 1.12.2 through 1.16.5. Baritone focuses on reliability and particularly performance (it's over [30x faster](https://github.com/cabaletta/baritone/pull/180#issuecomment-423822928) than MineBot at calculating paths).
+the original version of the bot for Minecraft 1.8.9, rebuilt for 1.12.2. Baritone focuses on reliability and particularly performance (it's over [30x faster](https://github.com/cabaletta/baritone/pull/180#issuecomment-423822928) than MineBot at calculating paths).
 
-Have committed at least once a day from Aug 1, 2018, to Aug 1, 2019.
+Have committed at least once a day for the last 8 months =D 🦀
 
 1Leijurv3DWTrGAfmmiTphjhXLvQiHg7K2
 
@@ -71,23 +43,19 @@ Here are some links to help to get started:
 
 - [Installation & setup](SETUP.md)
 
-- [API Javadocs](https://baritone.leijurv.com/)
+- [API Javadocs](https://baritone.leijurv.com)
 
 - [Settings](https://baritone.leijurv.com/baritone/api/Settings.html#field.detail)
 
-- [Usage (chat control)](USAGE.md)
-
-## Stars over time
-
-[![Stargazers over time](https://starchart.cc/cabaletta/baritone.svg)](https://starchart.cc/cabaletta/baritone)
+- [Baritone chat control usage](USAGE.md)
 
 # API
 
-The API is heavily documented, you can find the Javadocs for the latest release [here](https://baritone.leijurv.com/).
+The API is heavily documented, you can find the Javadocs for the latest release [here](https://baritone.leijurv.com).
 Please note that usage of anything located outside of the ``baritone.api`` package is not supported by the API release
 jar.
 
-Below is an example of basic usage for changing some settings, and then pathing to an X/Z goal.
+Below is an example of basic usage for changing some settings, and then pathing to a X/Z goal.
 
 ```
 BaritoneAPI.getSettings().allowSprint.value = true;
@@ -100,22 +68,12 @@ BaritoneAPI.getProvider().getPrimaryBaritone().getCustomGoalProcess().setGoalAnd
 
 ## Can I use Baritone as a library in my custom utility client?
 
-That's what it's for, sure! (As long as usage complies with the LGPL 3.0 License)
+That's what it's for, sure! (As long as usage is in compliance with the LGPL 3 License)
 
 ## How is it so fast?
 
-Magic. (Hours of [leijurv](https://github.com/leijurv/) enduring excruciating pain)
-
-### Additional Special Thanks To:
-
-![YourKit-Logo](https://www.yourkit.com/images/yklogo.png)
-
-YourKit supports open source projects with innovative and intelligent tools for monitoring and profiling Java and .NET applications.
-
-YourKit is the creator of the [YourKit Java Profiler](https://www.yourkit.com/java/profiler/), [YourKit .NET Profiler](https://www.yourkit.com/.net/profiler/), and [YourKit YouMonitor](https://www.yourkit.com/youmonitor/).
-
-We thank them for granting Baritone an OSS license so that we can make our software the best it can be.
+Magic. (Hours of [leijurv](https://github.com/leijurv) enduring excruciating pain)
 
 ## Why is it called Baritone?
 
-It's named for FitMC's deep sultry voice.
+It's named for FitMC's deep sultry voice. 

SETUP.md

@@ -1,19 +1,11 @@
 # Installation
 
-The easiest way to install Baritone is to install [Impact](https://impactclient.net/), which comes with Baritone.
-
-You can also use a custom version json for Minecraft, with the [1.14.4](https://www.dropbox.com/s/rkml3hjokd3qv0m/1.14.4-Baritone.zip?dl=1) version or the [1.15.2](https://www.dropbox.com/s/8rx6f0kts9hvd4f/1.15.2-Baritone.zip?dl=1) version or the [1.16.5](https://www.dropbox.com/s/i6f292o2i7o9acp/1.16.5-Baritone.zip?dl=1) version.
-
-Once Baritone is installed, look [here](USAGE.md) for instructions on how to use it.
-
 ## Prebuilt official releases
 These releases are not always completely up to date with latest features, and are only released from `master`. (so if you want `backfill-2` branch for example, you'll have to build it yourself)
 
 Link to the releases page: [Releases](https://github.com/cabaletta/baritone/releases)
 
-v1.2.* is for 1.12.2, v1.3.* is for 1.13.2, v1.4.* is for 1.14.4, v1.5.* is for 1.15.2, v1.6.* is for 1.16.2 or 1.16.4 or 1.16.5 (LOL)
-
-Any official release will be GPG signed by leijurv (44A3EA646EADAC6A). Please verify that the hash of the file you download is in `checksums.txt` and that `checksums_signed.asc` is a valid signature by that public keys of `checksums.txt`. 
+Any official release will be GPG signed by leijurv (44A3EA646EADAC6A) and ZeroMemes (73A788379A197567). Please verify that the hash of the file you download is in `checksums.txt` and that `checksums_signed.asc` is a valid signature by those two public keys of `checksums.txt`. 
 
 The build is fully deterministic and reproducible, and you can verify Travis did it properly by running `docker build --no-cache -t cabaletta/baritone .` yourself and comparing the shasum. This works identically on Travis, Mac, and Linux (if you have docker on Windows, I'd be grateful if you could let me know if it works there too).
 
@@ -32,6 +24,11 @@ If another one of your Forge mods has a Baritone integration, you want `baritone
 - **Forge Standalone**: Same as Standalone, but packaged for Forge. This should be used when Baritone is your only Forge mod, or none of your other Forge mods integrate with Baritone.
 - **Unoptimized**: Nothing is obfuscated. This shouldn't be used ever in production.
 
+## More Info
+To replace out Impact 4.5's Baritone build with a customized one, build Baritone as above then copy & **rename** `dist/baritone-api-$VERSION$.jar` into `minecraft/libraries/cabaletta/baritone-api/1.2/baritone-api-1.2.jar`, replacing the jar that was previously there. You also need to edit `minecraft/versions/1.12.2-Impact_4.5/1.12.2-Impact_4.5.json`, find the line `"name": "cabaletta:baritone-api:1.2"`, remove the comma from the end, and **entirely remove the NEXT line** (starts with `"url"`). **Restart your launcher** then load as normal. 
+
+You can verify whether or not it worked by running `.b version` in chat (only valid in Impact). It should print out the version that you downloaded. Note: The version that comes with 4.5 is `v1.2.3`.
+
 ## Build it yourself
 - Clone or download Baritone
 
@@ -42,29 +39,6 @@ If another one of your Forge mods has a Baritone integration, you want `baritone
 ## Command Line
 On Mac OSX and Linux, use `./gradlew` instead of `gradlew`.
 
-If you have errors with a package missing please make sure you have setup your environment, and are using Oracle JDK 8.
-
-To check which java you are using do 
-`java -version` in a command prompt or terminal.
-If you are using anything above OpenJDK 8, it might not work because the Java distributions above JDK 8 using may not have the needed javax classes.
-
-Open JDK 8 download: https://openjdk.java.net/install/
-#### macOS guide
-In order to get JDK 8, Try running the following command:
-`% /usr/libexec/java_home -V`
-If it doesn't work try this guide: https://stackoverflow.com/questions/46513639/how-to-downgrade-java-from-9-to-8-on-a-macos-eclipse-is-not-running-with-java-9
-
-If you see something like
-
-`% 1.8.0_VERSION, x86_64:	"Java SE 8"	/Library/Java/JavaVirtualMachines/jdk1.8.0_VERSION.jdk/Contents/Home`
-
-in the list then you've got JDK 8 installed. 
-In order to get JDK 8 running in the **current terminal window** you will have to run this command: 
-
-`% export JAVA_HOME=$(/usr/libexec/java_home -v 1.8)`
-
-To add OpenJDK 8 to your PATH add the export line to the end of your `.zshrc / .bashrc` if you want it to apply to each new terminal. If you're using bash change the .bachrc and if you're using zsh change the .zshrc
-
 Setting up the Environment:
 
 ```
@@ -72,18 +46,6 @@ $ gradlew setupDecompWorkspace
 $ gradlew --refresh-dependencies
 ```
 
-Building Baritone: 
-
-```
-$ gradlew build
-```
-
-For minecraft 1.15.2+, run the following instead to include the Forge jars:
-
-```
-$ gradlew build -Pbaritone.forge_build
-```
-
 Running Baritone:
 
 ```
@@ -117,10 +79,20 @@ For information on how to build baritone, see [Building Baritone](#building-bari
   
   ![Image](https://i.imgur.com/hrLhG9u.png)
 
+# Building
+
+Make sure that you have properly [setup](#setup) the environment before trying to build it.
+
+## Command Line
+
+```
+$ gradlew build
+```
+
 ## IntelliJ
 
 - Navigate to the gradle tasks on the right tab as follows
 
   ![Image](https://i.imgur.com/PE6r9iN.png)
 
-- Double click on **build** to run it
+- Double click on **build** to run it

USAGE.md

@@ -2,59 +2,52 @@
 
 # Prefix
 
-Baritone's chat control prefix is `#` by default. In Impact, you can also use `.b` as a prefix. (for example, `.b click` instead of `#click`)
+Baritone commands can by default be typed in the chatbox. However if you make a typo, like typing "gola 10000 10000" instead of goal it goes into public chat, which is bad.
 
-Baritone commands can also by default be typed in the chatbox. However if you make a typo, like typing "gola 10000 10000" instead of "goal" it goes into public chat, which is bad, so using `#` is suggested.
+Therefore you can use a prefix before your messages.
 
-To disable direct chat control (with no prefix), turn off the `chatControl` setting. To disable chat control with the `#` prefix, turn off the `prefixControl` setting. In Impact, `.b` cannot be disabled. Be careful that you don't leave yourself with all control methods disabled (if you do, reset your settings by deleting the file `minecraft/baritone/settings.txt` and relaunching).
+On Baritone v1.1.0 and newer: The prefix is `#` by default. Anything beginning with `#` isn't sent, and is only interpreted by Baritone.
+For older than v1.1.0, `#` must be enabled by toggling on the `prefix` setting.
 
-# For Baritone 1.2.10+, 1.3.5+, 1.4.2+
+**Only** in Impact is `.b` also a valid prefix. In 4.4, `#` does **not** work, neither does saying the commands directly in chat. `#` works by default in 4.5 (not 4.4).
 
-Lots of the commands have changed, BUT `#help` is improved vastly (its clickable! commands have tab completion! oh my!).
+Other clients like Kami and Asuna have their own custom things (like `-path`), and can disable direct chat control entirely.
 
-Try `#help` I promise it won't just send you back here =)
 
-"wtf where is cleararea" -> look at `#help sel`
-
-"wtf where is goto death, goto waypoint" -> look at `#help wp` 
-
-just look at `#help` lmao
-
-Watch this [showcase video](https://youtu.be/CZkLXWo4Fg4)!
 
 # Commands
 
-[Tutorial playlist](https://www.youtube.com/playlist?list=PLnwnJ1qsS7CoQl9Si-RTluuzCo_4Oulpa)
-
 **All** of these commands may need a prefix before them, as above ^.
 
-`help`
+`help` for (rudimentary) help. You can see what it says [here](https://github.com/cabaletta/baritone/blob/master/src/main/java/baritone/utils/ExampleBaritoneControl.java#L53).
 
 To toggle a boolean setting, just say its name in chat (for example saying `allowBreak` toggles whether Baritone will consider breaking blocks). For a numeric setting, say its name then the new value (like `primaryTimeoutMS 250`). It's case insensitive. To reset a setting to its default value, say `acceptableThrowawayItems reset`. To reset all settings, say `reset`. To see all settings that have been modified from their default values, say `modified`.
 
+
+
+
 Some common examples:
 - `thisway 1000` then `path` to go in the direction you're facing for a thousand blocks
-- `goal x y z` or `goal x z` or `goal y`, then `path` to set a goal to a certain coordinate then path to it
-- `goto x y z` or `goto x z` or `goto y` to go to a certain coordinate (in a single step, starts going immediately)
+- `goal x y z` or `goal x z` or `goal y`, then `path` to go to a certain coordinate
 - `goal` to set the goal to your player's feet
 - `goal clear` to clear the goal
 - `cancel` or `stop` to stop everything
 - `goto portal` or `goto ender_chest` or `goto block_type` to go to a block. (in Impact, `.goto` is an alias for `.b goto` for the most part)
-- `mine diamond_ore iron_ore` to mine diamond ore or iron ore (turn on the setting `legitMine` to only mine ores that it can actually see. It will explore randomly around y=11 until it finds them.) An amount of blocks can also be specified, for example, `mine 64 diamond_ore`.
-- `click` to click your destination on the screen. Right click path to on top of the block, left click to path into it (either at foot level or eye level), and left click and drag to select an area (`#help sel` to see what you can do with that selection).
-- `follow player playerName` to follow a player. `follow players` to follow any players in range (combine with Kill Aura for a fun time). `follow entities` to follow any entities. `follow entity pig` to follow entities of a specific type.
-- `wp` for waypoints. A "tag" is like "home" (created automatically on right clicking a bed) or "death" (created automatically on death) or "user" (has to be created manually). So you might want `#wp save user coolbiome`, then to set the goal `#wp goal coolbiome` then `#path` to path to it. For death, `#wp goal death` will list waypoints under the "death" tag (remember stuff is clickable!)
-- `build` to build a schematic. `build blah.schematic` will load `schematics/blah.schematic` and build it with the origin being your player feet. `build blah.schematic x y z` to set the origin. Any of those can be relative to your player (`~ 69 ~-420` would build at x=player x, y=69, z=player z-420).
-- `schematica` to build the schematic that is currently open in schematica
-- `tunnel` to dig and make a tunnel, 1x2. It will only deviate from the straight line if necessary such as to avoid lava. For a dumber tunnel that is really just cleararea, you can `tunnel 3 2 100`, to clear an area 3 high, 2 wide, and 100 deep.
-- `farm` to automatically harvest, replant, or bone meal crops. Use `farm <range>` or `farm <range> <waypoint>` to limit the max distance from the starting point or a waypoint. 
+- `mine diamond_ore` to mine diamond ore (turn on the setting `legitMine` to only mine ores that it can actually see. It will explore randomly around y=11 until it finds them.)
+- `click` to click your destination on the screen. Right click path to on top of the block, left click to path into it (either at foot level or eye level), and left click and drag to clear all blocks from an area.
+- `follow playerName` to follow a player. `follow` to follow the entity you're looking at (only works if it hitting range). `followplayers` to follow any players in range (combine with Kill Aura for a fun time).
+- `save waypointName` to save a waypoint. `goto waypointName` to go to it.
+- `build` to build a schematic. `build blah` will load `schematics/blah.schematic` and build it with the origin being your player feet. `build blah x y z` to set the origin. Any of those can be relative to your player (`~ 69 ~-420` would build at x=player x, y=69, z=player z-420).
+- `tunnel` to dig just straight ahead and make a tunnel
+- `farm` to automatically harvest, replant, or bone meal crops
 - `axis` to go to an axis or diagonal axis at y=120 (`axisHeight` is a configurable setting, defaults to 120).
 - `explore x z` to explore the world from the origin of x,z. Leave out x and z to default to player feet. This will continually path towards the closest chunk to the origin that it's never seen before. `explorefilter filter.json` with optional invert can be used to load in a list of chunks to load.
 - `invert` to invert the current goal and path. This gets as far away from it as possible, instead of as close as possible. For example, do `goal` then `invert` to run as far as possible from where you're standing at the start.
 - `version` to get the version of Baritone you're running
 - `damn` daniel
 
-For the rest of the commands, you can take a look at the code [here](https://baritone.leijurv.com/baritone/api/Settings.html).
+
+For the rest of the commands, you can take a look at the code [here](https://github.com/cabaletta/baritone/blob/master/src/main/java/baritone/utils/ExampleBaritoneControl.java).
 
 All the settings and documentation are <a href="https://github.com/cabaletta/baritone/blob/master/src/api/java/baritone/api/Settings.java">here</a>. If you find HTML easier to read than Javadoc, you can look <a href="https://baritone.leijurv.com/baritone/api/Settings.html#field.detail">here</a>.
 
@@ -75,8 +68,6 @@ There are about a hundred settings, but here are some fun / interesting / import
 - `worldExploringChunkOffset`
 - `acceptableThrowawayItems`
 - `blocksToAvoidBreaking`
-- `mineScanDroppedItems`
-- `allowDiagonalAscend`
 
 
 
@@ -96,4 +87,4 @@ So you'll need to use the `#` prefix or edit `baritone/settings.txt` in your Min
 
 ## Why can I do `.goto x z` in Impact but nowhere else? Why can I do `-path to x z` in KAMI but nowhere else?
 These are custom commands that they added; those aren't from Baritone.
-The equivalent you're looking for is `goto x z`.
+The equivalent you're looking for is `goal x z` then `path`.

build.gradle

@@ -16,34 +16,33 @@
  */
 
 group 'baritone'
-version '1.2.15'
+version '1.3.2'
 
 buildscript {
     repositories {
+        jcenter()
         maven {
             name = 'forge'
             url = 'http://files.minecraftforge.net/maven'
         }
         maven {
-            name = 'SpongePowered'
-            url = 'https://repo.spongepowered.org/repository/maven-public/'
+            name = 'impactdevelopment-repo'
+            url = 'https://impactdevelopment.github.io/maven/'
         }
-        jcenter()
     }
 
     dependencies {
-        classpath 'net.minecraftforge.gradle:ForgeGradle:2.3-SNAPSHOT'
-        classpath 'org.spongepowered:mixingradle:0.6-SNAPSHOT'
+        classpath group: 'com.github.ImpactDevelopment', name: 'ForgeGradle', version: '3.0.115'
+        classpath group: 'com.github.ImpactDevelopment', name: 'MixinGradle', version: '0.6.2'
     }
 }
 
-
 import baritone.gradle.task.CreateDistTask
 import baritone.gradle.task.ProguardTask
+import org.apache.tools.ant.taskdefs.condition.Os
 
 apply plugin: 'java'
-apply plugin: 'maven'
-apply plugin: 'net.minecraftforge.gradle.tweaker-client'
+apply plugin: 'net.minecraftforge.gradle'
 apply plugin: 'org.spongepowered.mixin'
 
 sourceCompatibility = targetCompatibility = '1.8'
@@ -53,27 +52,67 @@ compileJava {
 }
 
 sourceSets {
-    launch {
-        compileClasspath += main.compileClasspath + main.runtimeClasspath + main.output
-    }
-
-    schematica_api {
+    api {
         compileClasspath += main.compileClasspath
     }
-
     main {
-        compileClasspath += schematica_api.output
+        compileClasspath += api.output
+    }
+    test {
+        compileClasspath += main.compileClasspath + main.runtimeClasspath + main.output
+        runtimeClasspath += main.compileClasspath + main.runtimeClasspath + main.output
+    }
+    launch {
+        compileClasspath += main.compileClasspath + main.runtimeClasspath + main.output
+        runtimeClasspath += main.compileClasspath + main.runtimeClasspath + main.output
     }
 }
 
-minecraft {
-    version = '1.12.2'
-    mappings = 'stable_39'
-    tweakClass = 'baritone.launch.BaritoneTweaker'
-    runDir = 'run'
+task sourceJar(type: Jar, dependsOn: classes) {
+    classifier = 'sources'
+    from sourceSets.api.allSource
+}
 
-    // The sources jar should use SRG names not MCP to ensure compatibility with all mappings
-    makeObfSourceJar = true
+minecraft {
+    mappings channel: 'snapshot', version: '20190307-1.13.1'
+    reobfMappings 'notch'
+
+    runs {
+        client {
+            workingDirectory project.file('run')
+            source sourceSets.launch
+
+            main 'baritone.launch.LaunchTesting'
+
+            environment 'assetIndex', '{asset_index}'
+            environment 'assetDirectory', downloadAssets.output
+            environment 'nativesDirectory', extractNatives.output
+
+            environment 'tweakClass', 'baritone.launch.BaritoneTweaker'
+
+            if (Os.isFamily(Os.FAMILY_MAC)) {
+                jvmArgs "-XstartOnFirstThread"
+            }
+        }
+
+        autoTest {
+            workingDirectory project.file('autotest')
+            source sourceSets.launch
+
+            main 'baritone.launch.LaunchTesting'
+
+            environment 'assetIndex', '{asset_index}'
+            environment 'assetDirectory', downloadAssets.output
+            environment 'nativesDirectory', extractNatives.output
+
+            environment 'tweakClass', 'baritone.launch.BaritoneTweaker'
+            environment 'BARITONE_AUTO_TEST', 'true'
+
+            if (Os.isFamily(Os.FAMILY_MAC)) {
+                jvmArgs "-XstartOnFirstThread"
+            }
+        }
+    }
 }
 
 repositories {
@@ -81,7 +120,7 @@ repositories {
 
     maven {
         name = 'spongepowered-repo'
-        url = 'https://repo.spongepowered.org/repository/maven-public/'
+        url = 'http://repo.spongepowered.org/maven/'
     }
 
     maven {
@@ -91,6 +130,12 @@ repositories {
 }
 
 dependencies {
+    minecraft 'com.github.ImpactDevelopment:Vanilla:1.13.2'
+
+    runtime launchCompile('net.minecraft:launchwrapper:1.12') {
+        exclude module: 'lwjgl'
+    }
+    runtime launchCompile('org.ow2.asm:asm-debug-all:5.2')
     runtime launchCompile('com.github.ImpactDevelopment:SimpleTweaker:1.2')
     runtime launchCompile('org.spongepowered:mixin:0.7.11-SNAPSHOT') {
         // Mixin includes a lot of dependencies that are too up-to-date
@@ -100,6 +145,7 @@ dependencies {
         exclude module: 'commons-io'
         exclude module: 'log4j-core'
     }
+
     testImplementation 'junit:junit:4.12'
 }
 
@@ -118,12 +164,6 @@ javadoc {
 
 jar {
     from sourceSets.launch.output, sourceSets.api.output
-
-    if (!getProject().hasProperty("baritone.forge_build")) {
-        exclude "**/BaritoneForgeModXD.class"
-        exclude "**/mods.toml"
-    }
-
     preserveFileTimestamps = false
     reproducibleFileOrder = true
 
@@ -145,25 +185,3 @@ task proguard(type: ProguardTask) {
 task createDist(type: CreateDistTask, dependsOn: proguard)
 
 build.finalizedBy(createDist)
-
-install {
-    def jarApiName = String.format("%s-api-%s", rootProject.name, version.toString())
-    def jarApiForgeName = String.format("%s-api-forge-%s", rootProject.name, version.toString())
-    def jarSAName = String.format("%s-standalone-%s", rootProject.name, version.toString())
-    def jarSAForgeName = String.format("%s-standalone-forge-%s", rootProject.name, version.toString())
-
-    artifacts {
-        archives file("$buildDir/libs/"+jarApiName+".jar")
-        archives file("$buildDir/libs/"+jarApiForgeName+".jar")
-        archives file("$buildDir/libs/"+jarSAName+".jar")
-        archives file("$buildDir/libs/"+jarSAForgeName+".jar")
-    }
-    repositories.mavenInstaller {
-		addFilter('api') { artifact, file -> artifact.name == "baritone-api" }
-		addFilter('api-forge') { artifact, file -> artifact.name == "baritone-api-forge" }
-		addFilter('standalone') { artifact, file -> artifact.name == "baritone-standalone" }
-		addFilter('standalone-forge') { artifact, file -> artifact.name == "baritone-standalone-forge" }
-    }
-}
-
-install.dependsOn(build)

buildSrc/src/main/java/baritone/gradle/task/BaritoneGradleTask.java

@@ -40,17 +40,13 @@ class BaritoneGradleTask extends DefaultTask {
             PROGUARD_STANDALONE_CONFIG      = "standalone.pro",
             PROGUARD_EXPORT_PATH            = "proguard_out.jar",
 
-            TEMP_LIBRARY_DIR = "tempLibraries/",
-
             ARTIFACT_STANDARD         = "%s-%s.jar",
             ARTIFACT_UNOPTIMIZED      = "%s-unoptimized-%s.jar",
             ARTIFACT_API              = "%s-api-%s.jar",
-            ARTIFACT_STANDALONE       = "%s-standalone-%s.jar",
-            ARTIFACT_FORGE_API        = "%s-api-forge-%s.jar",
-            ARTIFACT_FORGE_STANDALONE = "%s-standalone-forge-%s.jar";
+            ARTIFACT_STANDALONE       = "%s-standalone-%s.jar";
 
     protected String artifactName, artifactVersion;
-    protected Path artifactPath, artifactUnoptimizedPath, artifactApiPath, artifactStandalonePath, artifactForgeApiPath, artifactForgeStandalonePath, proguardOut;
+    protected Path artifactPath, artifactUnoptimizedPath, artifactApiPath, artifactStandalonePath, proguardOut;
 
     protected void verifyArtifacts() throws IllegalStateException {
         this.artifactName = getProject().getName();
@@ -60,8 +56,6 @@ class BaritoneGradleTask extends DefaultTask {
         this.artifactUnoptimizedPath     = this.getBuildFile(formatVersion(ARTIFACT_UNOPTIMIZED));
         this.artifactApiPath             = this.getBuildFile(formatVersion(ARTIFACT_API));
         this.artifactStandalonePath      = this.getBuildFile(formatVersion(ARTIFACT_STANDALONE));
-        this.artifactForgeApiPath        = this.getBuildFile(formatVersion(ARTIFACT_FORGE_API));
-        this.artifactForgeStandalonePath = this.getBuildFile(formatVersion(ARTIFACT_FORGE_STANDALONE));
 
         this.proguardOut = this.getTemporaryFile(PROGUARD_EXPORT_PATH);
 

buildSrc/src/main/java/baritone/gradle/task/CreateDistTask.java

@@ -45,8 +45,6 @@ public class CreateDistTask extends BaritoneGradleTask {
         Path api             = getRelativeFile("dist/" + formatVersion(ARTIFACT_API));
         Path standalone      = getRelativeFile("dist/" + formatVersion(ARTIFACT_STANDALONE));
         Path unoptimized     = getRelativeFile("dist/" + formatVersion(ARTIFACT_UNOPTIMIZED));
-        Path forgeApi        = getRelativeFile("dist/" + formatVersion(ARTIFACT_FORGE_API));
-        Path forgeStandalone = getRelativeFile("dist/" + formatVersion(ARTIFACT_FORGE_STANDALONE));
 
         // NIO will not automatically create directories
         Path dir = getRelativeFile("dist/");
@@ -58,11 +56,9 @@ public class CreateDistTask extends BaritoneGradleTask {
         Files.copy(this.artifactApiPath,             api,             REPLACE_EXISTING);
         Files.copy(this.artifactStandalonePath,      standalone,      REPLACE_EXISTING);
         Files.copy(this.artifactUnoptimizedPath,     unoptimized,     REPLACE_EXISTING);
-        Files.copy(this.artifactForgeApiPath,        forgeApi,        REPLACE_EXISTING);
-        Files.copy(this.artifactForgeStandalonePath, forgeStandalone, REPLACE_EXISTING);
 
         // Calculate all checksums and format them like "shasum"
-        List<String> shasum = Stream.of(api, forgeApi, standalone, forgeStandalone, unoptimized)
+        List<String> shasum = Stream.of(api, standalone, unoptimized)
                 .map(path -> sha1(path) + "  " + path.getFileName().toString())
                 .collect(Collectors.toList());
 

buildSrc/src/main/java/baritone/gradle/task/ProguardTask.java

@@ -18,34 +18,18 @@
 package baritone.gradle.task;
 
 import baritone.gradle.util.Determinizer;
-import baritone.gradle.util.MappingType;
-import baritone.gradle.util.ReobfWrapper;
 import org.apache.commons.io.IOUtils;
-import org.gradle.api.JavaVersion;
-import org.gradle.api.NamedDomainObjectContainer;
-import org.gradle.api.artifacts.Configuration;
-import org.gradle.api.artifacts.Dependency;
-import org.gradle.api.internal.file.IdentityFileResolver;
-import org.gradle.api.internal.plugins.DefaultConvention;
+import org.gradle.api.plugins.JavaPluginConvention;
 import org.gradle.api.tasks.Input;
 import org.gradle.api.tasks.TaskAction;
-import org.gradle.api.tasks.TaskCollection;
-import org.gradle.api.tasks.compile.ForkOptions;
-import org.gradle.api.tasks.compile.JavaCompile;
-import org.gradle.internal.Pair;
-import org.gradle.internal.jvm.Jvm;
-import org.gradle.internal.jvm.inspection.DefaultJvmVersionDetector;
-import org.gradle.process.internal.DefaultExecActionFactory;
 
 import java.io.*;
-import java.lang.reflect.Field;
 import java.net.URL;
 import java.nio.file.Files;
 import java.nio.file.Path;
-import java.util.*;
-import java.util.regex.Matcher;
-import java.util.regex.Pattern;
-import java.util.stream.Collectors;
+import java.util.ArrayList;
+import java.util.List;
+import java.util.stream.Stream;
 import java.util.zip.ZipEntry;
 import java.util.zip.ZipFile;
 
@@ -57,18 +41,12 @@ import static java.nio.file.StandardCopyOption.REPLACE_EXISTING;
  */
 public class ProguardTask extends BaritoneGradleTask {
 
-    private static final Pattern TEMP_LIBRARY_PATTERN = Pattern.compile("-libraryjars 'tempLibraries\\/([a-zA-Z0-9/_\\-\\.]+)\\.jar'");
-
     @Input
     private String url;
 
     @Input
     private String extract;
 
-    private List<String> requiredLibraries;
-
-    private File mixin;
-
     @TaskAction
     protected void exec() throws Exception {
         super.verifyArtifacts();
@@ -78,7 +56,6 @@ public class ProguardTask extends BaritoneGradleTask {
         downloadProguard();
         extractProguard();
         generateConfigs();
-        acquireDependencies();
         proguardApi();
         proguardStandalone();
         cleanup();
@@ -89,7 +66,7 @@ public class ProguardTask extends BaritoneGradleTask {
             Files.delete(this.artifactUnoptimizedPath);
         }
 
-        Determinizer.determinize(this.artifactPath.toString(), this.artifactUnoptimizedPath.toString(), Optional.empty());
+        Determinizer.determinize(this.artifactPath.toString(), this.artifactUnoptimizedPath.toString());
     }
 
     private void downloadProguard() throws Exception {
@@ -109,101 +86,6 @@ public class ProguardTask extends BaritoneGradleTask {
         }
     }
 
-    private String getJavaBinPathForProguard() throws Exception {
-        String path;
-        try {
-            path = findJavaPathByGradleConfig();
-            if (path != null) return path;
-        }
-        catch (Exception ex) {
-            System.err.println("Unable to find java by javaCompile options");
-            ex.printStackTrace();
-        }
-
-        try {
-            path = findJavaByJavaHome();
-            if (path != null) return path;
-        }
-        catch(Exception ex) {
-            System.err.println("Unable to find java by JAVA_HOME");
-            ex.printStackTrace();
-        }
-
-
-        path = findJavaByGradleCurrentRuntime();
-        if (path != null) return path;
-        
-        throw new Exception("Unable to find java to determine ProGuard libraryjars. Please specify forkOptions.executable in javaCompile," +
-                " JAVA_HOME environment variable, or make sure to run Gradle with the correct JDK (a v1.8 only)");
-    }
-
-    private String findJavaByGradleCurrentRuntime() {
-        String path = Jvm.current().getJavaExecutable().getAbsolutePath();
-
-        if (this.validateJavaVersion(path)) {
-            System.out.println("Using Gradle's runtime Java for ProGuard");
-            return path;
-        }
-        return null;
-    }
-
-    private String findJavaByJavaHome() {
-        final String javaHomeEnv = System.getenv("JAVA_HOME");
-        if (javaHomeEnv != null) {
-
-            String path = Jvm.forHome(new File(javaHomeEnv)).getJavaExecutable().getAbsolutePath();
-            if (this.validateJavaVersion(path)) {
-                System.out.println("Detected Java path by JAVA_HOME");
-                return path;
-            }
-        }
-        return null;
-    }
-
-    private String findJavaPathByGradleConfig() {
-        final TaskCollection<JavaCompile> javaCompiles = super.getProject().getTasks().withType(JavaCompile.class);
-
-        final JavaCompile compileTask = javaCompiles.iterator().next();
-        final ForkOptions forkOptions = compileTask.getOptions().getForkOptions();
-
-        if (forkOptions != null) {
-            String javacPath = forkOptions.getExecutable();
-            if (javacPath != null) {
-                File javacFile = new File(javacPath);
-                if (javacFile.exists()) {
-                    File[] maybeJava = javacFile.getParentFile().listFiles(new FilenameFilter() {
-                        @Override
-                        public boolean accept(File dir, String name) {
-                            return name.equals("java");
-                        }
-                    });
-
-                    if (maybeJava != null && maybeJava.length > 0) {
-                        String path = maybeJava[0].getAbsolutePath();
-                        if (this.validateJavaVersion(path)) {
-                            System.out.println("Detected Java path by forkOptions");
-                            return path;
-                        }
-                    }
-                }
-            }
-        }
-        return null;
-    }
-
-    private boolean validateJavaVersion(String java) {
-        final JavaVersion javaVersion = new DefaultJvmVersionDetector(new DefaultExecActionFactory(new IdentityFileResolver())).getJavaVersion(java);
-
-        if (!javaVersion.getMajorVersion().equals("8")) {
-            System.out.println("Failed to validate Java version " + javaVersion.toString() + " [" + java + "] for ProGuard libraryjars");
-            // throw new RuntimeException("Java version incorrect: " + javaVersion.getMajorVersion() + " for " + java);
-            return false;
-        }
-
-        System.out.println("Validated Java version " + javaVersion.toString() + " [" + java + "] for ProGuard libraryjars");
-        return true;
-    }
-
     private void generateConfigs() throws Exception {
         Files.copy(getRelativeFile(PROGUARD_CONFIG_TEMPLATE), getTemporaryFile(PROGUARD_CONFIG_DEST), REPLACE_EXISTING);
 
@@ -213,10 +95,23 @@ public class ProguardTask extends BaritoneGradleTask {
         template.add(1, "-outjars " + this.getTemporaryFile(PROGUARD_EXPORT_PATH));
 
         // Acquire the RT jar using "java -verbose". This doesn't work on Java 9+
-        Process p = new ProcessBuilder(this.getJavaBinPathForProguard(), "-verbose").start();
+        Process p = new ProcessBuilder("java", "-verbose").start();
         String out = IOUtils.toString(p.getInputStream(), "UTF-8").split("\n")[0].split("Opened ")[1].replace("]", "");
         template.add(2, "-libraryjars '" + out + "'");
 
+        // Discover all of the libraries that we will need to acquire from gradle
+        acquireDependencies().forEach(f -> {
+            if (f.toString().endsWith("-recomp.jar")) {
+                // remove MCP mapped jar
+                return;
+            }
+            if (f.toString().endsWith("client-extra.jar")) {
+                // go from the extra to the original downloaded client
+                f = new File(f.getParentFile(), "client.jar");
+            }
+            template.add(2, "-libraryjars '" + f + "'");
+        });
+
         // API config doesn't require any changes from the changes that we made to the template
         Files.write(getTemporaryFile(PROGUARD_API_CONFIG), template);
 
@@ -224,165 +119,20 @@ public class ProguardTask extends BaritoneGradleTask {
         List<String> standalone = new ArrayList<>(template);
         standalone.removeIf(s -> s.contains("# this is the keep api"));
         Files.write(getTemporaryFile(PROGUARD_STANDALONE_CONFIG), standalone);
-
-        // Discover all of the libraries that we will need to acquire from gradle
-        this.requiredLibraries = new ArrayList<>();
-        template.forEach(line -> {
-            if (!line.startsWith("#")) {
-                Matcher m = TEMP_LIBRARY_PATTERN.matcher(line);
-                if (m.find()) {
-                    this.requiredLibraries.add(m.group(1));
-                }
-            }
-        });
     }
 
-    private void acquireDependencies() throws Exception {
-
-        // Create a map of all of the dependencies that we are able to access in this project
-        // Likely a better way to do this, I just pair the dependency with the first valid configuration
-        Map<String, Pair<Configuration, Dependency>> dependencyLookupMap = new HashMap<>();
-        getProject().getConfigurations().stream().filter(Configuration::isCanBeResolved).forEach(config ->
-                config.getAllDependencies().forEach(dependency ->
-                        dependencyLookupMap.putIfAbsent(dependency.getName() + "-" + dependency.getVersion(), Pair.of(config, dependency))));
-
-        // Create the directory if it doesn't already exist
-        Path tempLibraries = getTemporaryFile(TEMP_LIBRARY_DIR);
-        if (!Files.exists(tempLibraries)) {
-            Files.createDirectory(tempLibraries);
-        }
-
-        // Iterate the required libraries to copy them to tempLibraries
-        for (String lib : this.requiredLibraries) {
-            // copy from the forgegradle cache
-            if (lib.equals("minecraft")) {
-                Path cachedJar = getMinecraftJar();
-                Path inTempDir = getTemporaryFile("tempLibraries/minecraft.jar");
-                // TODO: maybe try not to copy every time
-                Files.copy(cachedJar, inTempDir, REPLACE_EXISTING);
-
-                continue;
-            }
-
-            // Find a configuration/dependency pair that matches the desired library
-            Pair<Configuration, Dependency> pair = null;
-            for (Map.Entry<String, Pair<Configuration, Dependency>> entry : dependencyLookupMap.entrySet()) {
-                if (entry.getKey().startsWith(lib)) {
-                    pair = entry.getValue();
-                }
-            }
-
-            // The pair must be non-null
-            Objects.requireNonNull(pair);
-
-            // Find the library jar file, and copy it to tempLibraries
-            for (File file : pair.getLeft().files(pair.getRight())) {
-                if (file.getName().startsWith(lib)) {
-                    if (lib.contains("mixin")) {
-                        mixin = file;
-                    }
-                    Files.copy(file.toPath(), getTemporaryFile("tempLibraries/" + lib + ".jar"), REPLACE_EXISTING);
-                }
-            }
-        }
-        if (mixin == null) {
-            throw new IllegalStateException("Unable to find mixin jar");
-        }
-    }
-
-    // a bunch of epic stuff to get the path to the cached jar
-    private Path getMinecraftJar() throws Exception {
-        MappingType mappingType;
-        try {
-            mappingType = getMappingType();
-        } catch (Exception e) {
-            System.err.println("Failed to get mapping type, assuming NOTCH.");
-            mappingType = MappingType.NOTCH;
-        }
-
-        String suffix;
-        switch (mappingType) {
-            case NOTCH:
-                suffix = "";
-                break;
-            case SEARGE:
-                suffix = "-srgBin";
-                break;
-            case CUSTOM:
-                throw new IllegalStateException("Custom mappings not supported!");
-            default:
-                throw new IllegalStateException("Unknown mapping type: " + mappingType);
-        }
-
-        DefaultConvention convention = (DefaultConvention) this.getProject().getConvention();
-        Object extension = convention.getAsMap().get("minecraft");
-        Objects.requireNonNull(extension);
-
-        // for some reason cant use Class.forName
-        Class<?> class_baseExtension = extension.getClass().getSuperclass().getSuperclass().getSuperclass(); // <-- cursed
-        Field f_replacer = class_baseExtension.getDeclaredField("replacer");
-        f_replacer.setAccessible(true);
-        Object replacer = f_replacer.get(extension);
-        Class<?> class_replacementProvider = replacer.getClass();
-        Field replacement_replaceMap = class_replacementProvider.getDeclaredField("replaceMap");
-        replacement_replaceMap.setAccessible(true);
-
-        Map<String, Object> replacements = (Map) replacement_replaceMap.get(replacer);
-        String cacheDir = replacements.get("CACHE_DIR").toString() + "/net/minecraft";
-        String mcVersion = replacements.get("MC_VERSION").toString();
-        String mcpInsert = replacements.get("MAPPING_CHANNEL").toString() + "/" + replacements.get("MAPPING_VERSION").toString();
-        String fullJarName = "minecraft-" + mcVersion + suffix + ".jar";
-
-        String baseDir = String.format("%s/minecraft/%s/", cacheDir, mcVersion);
-
-        String jarPath;
-        if (mappingType == MappingType.SEARGE) {
-            jarPath = String.format("%s/%s/%s", baseDir, mcpInsert, fullJarName);
-        } else {
-            jarPath = baseDir + fullJarName;
-        }
-        jarPath = jarPath
-                .replace("/", File.separator)
-                .replace("\\", File.separator); // hecking regex
-
-        return new File(jarPath).toPath();
-    }
-
-    // throws IllegalStateException if mapping type is ambiguous or it fails to find it
-    private MappingType getMappingType() {
-        // if it fails to find this then its probably a forgegradle version problem
-        Set<Object> reobf = (NamedDomainObjectContainer<Object>) this.getProject().getExtensions().getByName("reobf");
-
-        List<MappingType> mappingTypes = getUsedMappingTypes(reobf);
-        long mappingTypesUsed = mappingTypes.size();
-        if (mappingTypesUsed == 0) {
-            throw new IllegalStateException("Failed to find mapping type (no jar task?)");
-        }
-        if (mappingTypesUsed > 1) {
-            throw new IllegalStateException("Ambiguous mapping type (multiple jars with different mapping types?)");
-        }
-
-        return mappingTypes.get(0);
-    }
-
-    private List<MappingType> getUsedMappingTypes(Set<Object> reobf) {
-        return reobf.stream()
-                .map(ReobfWrapper::new)
-                .map(ReobfWrapper::getMappingType)
-                .distinct()
-                .collect(Collectors.toList());
+    private Stream<File> acquireDependencies() {
+        return getProject().getConvention().getPlugin(JavaPluginConvention.class).getSourceSets().findByName("launch").getRuntimeClasspath().getFiles().stream().filter(File::isFile);
     }
 
     private void proguardApi() throws Exception {
         runProguard(getTemporaryFile(PROGUARD_API_CONFIG));
-        Determinizer.determinize(this.proguardOut.toString(), this.artifactApiPath.toString(), Optional.empty());
-        Determinizer.determinize(this.proguardOut.toString(), this.artifactForgeApiPath.toString(), Optional.of(mixin));
+        Determinizer.determinize(this.proguardOut.toString(), this.artifactApiPath.toString());
     }
 
     private void proguardStandalone() throws Exception {
         runProguard(getTemporaryFile(PROGUARD_STANDALONE_CONFIG));
-        Determinizer.determinize(this.proguardOut.toString(), this.artifactStandalonePath.toString(), Optional.empty());
-        Determinizer.determinize(this.proguardOut.toString(), this.artifactForgeStandalonePath.toString(), Optional.of(mixin));
+        Determinizer.determinize(this.proguardOut.toString(), this.artifactStandalonePath.toString());
     }
 
     private void cleanup() {

buildSrc/src/main/java/baritone/gradle/util/Determinizer.java

@@ -22,7 +22,10 @@ import com.google.gson.stream.JsonReader;
 import com.google.gson.stream.JsonWriter;
 
 import java.io.*;
-import java.util.*;
+import java.util.ArrayList;
+import java.util.Comparator;
+import java.util.List;
+import java.util.Map;
 import java.util.jar.JarEntry;
 import java.util.jar.JarFile;
 import java.util.jar.JarOutputStream;
@@ -36,7 +39,7 @@ import java.util.stream.Collectors;
  */
 public class Determinizer {
 
-    public static void determinize(String inputPath, String outputPath, Optional<File> toInclude) throws IOException {
+    public static void determinize(String inputPath, String outputPath) throws IOException {
         System.out.println("Running Determinizer");
         System.out.println(" Input path: " + inputPath);
         System.out.println(" Output path: " + outputPath);
@@ -63,30 +66,10 @@ public class Determinizer {
                 if (entry.getName().endsWith(".refmap.json")) {
                     JsonObject object = new JsonParser().parse(new InputStreamReader(jarFile.getInputStream(entry))).getAsJsonObject();
                     jos.write(writeSorted(object).getBytes());
-                } else if (entry.getName().equals("META-INF/MANIFEST.MF") && toInclude.isPresent()) { // only replace for forge jar
-                    ByteArrayOutputStream cancer = new ByteArrayOutputStream();
-                    copy(jarFile.getInputStream(entry), cancer);
-                    String manifest = new String(cancer.toByteArray());
-                    if (!manifest.contains("baritone.launch.BaritoneTweaker")) {
-                        throw new IllegalStateException("unable to replace");
-                    }
-                    manifest = manifest.replace("baritone.launch.BaritoneTweaker", "org.spongepowered.asm.launch.MixinTweaker");
-                    jos.write(manifest.getBytes());
                 } else {
                     copy(jarFile.getInputStream(entry), jos);
                 }
             }
-            if (toInclude.isPresent()) {
-                try (JarFile mixin = new JarFile(toInclude.get())) {
-                    for (JarEntry entry : mixin.stream().sorted(Comparator.comparing(JarEntry::getName)).collect(Collectors.toList())) {
-                        if (entry.getName().startsWith("META-INF") && !entry.getName().startsWith("META-INF/services")) {
-                            continue;
-                        }
-                        jos.putNextEntry(entry);
-                        copy(mixin.getInputStream(entry), jos);
-                    }
-                }
-            }
             jos.finish();
         }
     }

buildSrc/src/main/java/baritone/gradle/util/MappingType.java

@@ -1,29 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.gradle.util;
-
-/**
- * All credits go to AsmLibGradle and its contributors.
- *
- * @see <a href="https://github.com/pozzed/AsmLibGradle/blob/8f917dbc3939eab7a3d9daf54d9d285fdf34f4b2/src/main/java/net/futureclient/asmlib/forgegradle/MappingType.java">Original Source</a>
- */
-public enum MappingType {
-    SEARGE,
-    NOTCH,
-    CUSTOM // forgegradle
-}

buildSrc/src/main/java/baritone/gradle/util/ReobfWrapper.java

@@ -1,63 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.gradle.util;
-
-import java.lang.reflect.Field;
-import java.util.Objects;
-
-/**
- * All credits go to AsmLibGradle and its contributors.
- *
- * @see <a href="https://github.com/pozzed/AsmLibGradle/blob/8f917dbc3939eab7a3d9daf54d9d285fdf34f4b2/src/main/java/net/futureclient/asmlib/forgegradle/ReobfWrapper.java">Original Source</a>
- */
-public class ReobfWrapper {
-
-    private final Object instance;
-    private final Class<?> type;
-
-    public ReobfWrapper(Object instance) {
-        this.instance = instance;
-        Objects.requireNonNull(instance);
-        this.type = instance.getClass();
-    }
-
-    public String getName() {
-        try {
-            Field nameField = type.getDeclaredField("name");
-            nameField.setAccessible(true);
-            return (String) nameField.get(this.instance);
-        } catch (ReflectiveOperationException ex) {
-            throw new IllegalStateException(ex);
-        }
-    }
-
-    public MappingType getMappingType() {
-        try {
-            Field enumField = type.getDeclaredField("mappingType");
-            enumField.setAccessible(true);
-            Enum<?> aEnum = (Enum<?>) enumField.get(this.instance);
-            MappingType mappingType = MappingType.values()[aEnum.ordinal()];
-            if (!aEnum.name().equals(mappingType.name())) {
-                throw new IllegalStateException("ForgeGradle ReobfMappingType is not equivalent to MappingType (version error?)");
-            }
-            return mappingType;
-        } catch (ReflectiveOperationException ex) {
-            throw new IllegalStateException(ex);
-        }
-    }
-}

scripts/proguard.pro

@@ -14,6 +14,8 @@
 
 # lwjgl is weird
 -dontwarn org.lwjgl.**
+# also lwjgl lol
+-dontwarn module-info
 
 -keep class baritone.api.** { *; } # this is the keep api
 
@@ -23,13 +25,6 @@
 
 -keep class baritone.api.utils.MyChunkPos { *; } # even in standalone we need to keep this for gson reflect
 
-# Keep any class or member annotated with @KeepName so we dont have to put everything in the script
--keep,allowobfuscation @interface baritone.KeepName
--keep @baritone.KeepName class *
--keepclassmembers class * {
-    @baritone.KeepName *;
-}
-
 # setting names are reflected from field names, so keep field names
 -keepclassmembers class baritone.api.Settings {
     public <fields>;    
@@ -38,61 +33,6 @@
 # need to keep mixin names
 -keep class baritone.launch.** { *; }
 
-#try to keep usage of schematica in separate classes
--keep class baritone.utils.schematic.schematica.**
-#proguard doesnt like it when it cant find our fake schematica classes
--dontwarn baritone.utils.schematic.schematica.**
-
-# copy all necessary libraries into tempLibraries to build
-
-# The correct jar will be copied from the forgegradle cache based on the mapping type being compiled with
--libraryjars 'tempLibraries/minecraft.jar'
-
--libraryjars 'tempLibraries/SimpleTweaker-1.2.jar'
-
--libraryjars 'tempLibraries/authlib-1.5.25.jar'
--libraryjars 'tempLibraries/codecjorbis-20101023.jar'
--libraryjars 'tempLibraries/codecwav-20101023.jar'
--libraryjars 'tempLibraries/commons-codec-1.10.jar'
--libraryjars 'tempLibraries/commons-compress-1.8.1.jar'
--libraryjars 'tempLibraries/commons-io-2.5.jar'
--libraryjars 'tempLibraries/commons-lang3-3.5.jar'
--libraryjars 'tempLibraries/commons-logging-1.1.3.jar'
--libraryjars 'tempLibraries/fastutil-7.1.0.jar'
--libraryjars 'tempLibraries/gson-2.8.0.jar'
--libraryjars 'tempLibraries/guava-21.0.jar'
--libraryjars 'tempLibraries/httpclient-4.3.3.jar'
--libraryjars 'tempLibraries/httpcore-4.3.2.jar'
--libraryjars 'tempLibraries/icu4j-core-mojang-51.2.jar'
--libraryjars 'tempLibraries/jinput-2.0.5.jar'
--libraryjars 'tempLibraries/jna-4.4.0.jar'
--libraryjars 'tempLibraries/jopt-simple-5.0.3.jar'
--libraryjars 'tempLibraries/jsr305-3.0.1.jar'
--libraryjars 'tempLibraries/jutils-1.0.0.jar'
--libraryjars 'tempLibraries/libraryjavasound-20101123.jar'
--libraryjars 'tempLibraries/librarylwjglopenal-20100824.jar'
--libraryjars 'tempLibraries/log4j-api-2.8.1.jar'
--libraryjars 'tempLibraries/log4j-core-2.8.1.jar'
-
-# startsWith is used to check the library, and mac/linux differ in which version they use
-# this is FINE
--libraryjars 'tempLibraries/lwjgl-.jar'
--libraryjars 'tempLibraries/lwjgl_util-.jar'
-
--libraryjars 'tempLibraries/netty-all-4.1.9.Final.jar'
--libraryjars 'tempLibraries/oshi-core-1.1.jar'
--libraryjars 'tempLibraries/patchy-1.2.jar'
--libraryjars 'tempLibraries/platform-3.4.0.jar'
--libraryjars 'tempLibraries/realms-1.10.22.jar'
--libraryjars 'tempLibraries/soundsystem-20120107.jar'
--libraryjars 'tempLibraries/text2speech-1.10.3.jar'
-
--libraryjars 'tempLibraries/mixin-0.7.11-SNAPSHOT.jar'
--libraryjars 'tempLibraries/launchwrapper-1.11.jar' # TODO why does only 1.11.jar exist?
-
-
-
-
 # Keep - Applications. Keep all application classes, along with their 'main'
 # methods.
 -keepclasseswithmembers public class * {
@@ -188,19 +128,19 @@
 -assumenosideeffects public class java.lang.* extends java.lang.Number {
     public static java.lang.String toString(byte);
     public static java.lang.Byte valueOf(byte);
-#    public static byte parseByte(java.lang.String);
-#    public static byte parseByte(java.lang.String,int);
-#    public static java.lang.Byte valueOf(java.lang.String,int);
-#    public static java.lang.Byte valueOf(java.lang.String);
-#    public static java.lang.Byte decode(java.lang.String);
+    public static byte parseByte(java.lang.String);
+    public static byte parseByte(java.lang.String,int);
+    public static java.lang.Byte valueOf(java.lang.String,int);
+    public static java.lang.Byte valueOf(java.lang.String);
+    public static java.lang.Byte decode(java.lang.String);
     public int compareTo(java.lang.Byte);
     public static java.lang.String toString(short);
-#    public static short parseShort(java.lang.String);
-#    public static short parseShort(java.lang.String,int);
-#    public static java.lang.Short valueOf(java.lang.String,int);
-#    public static java.lang.Short valueOf(java.lang.String);
+    public static short parseShort(java.lang.String);
+    public static short parseShort(java.lang.String,int);
+    public static java.lang.Short valueOf(java.lang.String,int);
+    public static java.lang.Short valueOf(java.lang.String);
     public static java.lang.Short valueOf(short);
-#    public static java.lang.Short decode(java.lang.String);
+    public static java.lang.Short decode(java.lang.String);
     public static short reverseBytes(short);
     public int compareTo(java.lang.Short);
     public static java.lang.String toString(int,int);
@@ -208,10 +148,10 @@
     public static java.lang.String toOctalString(int);
     public static java.lang.String toBinaryString(int);
     public static java.lang.String toString(int);
-#    public static int parseInt(java.lang.String,int);
-#    public static int parseInt(java.lang.String);
-#    public static java.lang.Integer valueOf(java.lang.String,int);
-#    public static java.lang.Integer valueOf(java.lang.String);
+    public static int parseInt(java.lang.String,int);
+    public static int parseInt(java.lang.String);
+    public static java.lang.Integer valueOf(java.lang.String,int);
+    public static java.lang.Integer valueOf(java.lang.String);
     public static java.lang.Integer valueOf(int);
     public static java.lang.Integer getInteger(java.lang.String);
     public static java.lang.Integer getInteger(java.lang.String,int);
@@ -233,12 +173,12 @@
     public static java.lang.String toOctalString(long);
     public static java.lang.String toBinaryString(long);
     public static java.lang.String toString(long);
-#    public static long parseLong(java.lang.String,int);
-#    public static long parseLong(java.lang.String);
-#    public static java.lang.Long valueOf(java.lang.String,int);
-#    public static java.lang.Long valueOf(java.lang.String);
+    public static long parseLong(java.lang.String,int);
+    public static long parseLong(java.lang.String);
+    public static java.lang.Long valueOf(java.lang.String,int);
+    public static java.lang.Long valueOf(java.lang.String);
     public static java.lang.Long valueOf(long);
-#    public static java.lang.Long decode(java.lang.String);
+    public static java.lang.Long decode(java.lang.String);
     public static java.lang.Long getLong(java.lang.String);
     public static java.lang.Long getLong(java.lang.String,long);
     public static java.lang.Long getLong(java.lang.String,java.lang.Long);
@@ -255,9 +195,9 @@
     public int compareTo(java.lang.Long);
     public static java.lang.String toString(float);
     public static java.lang.String toHexString(float);
-#    public static java.lang.Float valueOf(java.lang.String);
+    public static java.lang.Float valueOf(java.lang.String);
     public static java.lang.Float valueOf(float);
-#    public static float parseFloat(java.lang.String);
+    public static float parseFloat(java.lang.String);
     public static boolean isNaN(float);
     public static boolean isInfinite(float);
     public static int floatToIntBits(float);
@@ -269,9 +209,9 @@
     public int compareTo(java.lang.Float);
     public static java.lang.String toString(double);
     public static java.lang.String toHexString(double);
-#    public static java.lang.Double valueOf(java.lang.String);
-#    public static java.lang.Double valueOf(double);
-#    public static double parseDouble(java.lang.String);
+    public static java.lang.Double valueOf(java.lang.String);
+    public static java.lang.Double valueOf(double);
+    public static double parseDouble(java.lang.String);
     public static boolean isNaN(double);
     public static boolean isInfinite(double);
     public static long doubleToLongBits(double);
@@ -381,5 +321,3 @@
     public java.lang.String substring(int);
     public java.lang.String substring(int,int);
 }
-
--printmapping mapping.txt

src/api/java/baritone/api/BaritoneAPI.java

@@ -34,12 +34,12 @@ public final class BaritoneAPI {
     private static final Settings settings;
 
     static {
-        settings = new Settings();
-        SettingsUtil.readAndApply(settings);
-
         ServiceLoader<IBaritoneProvider> baritoneLoader = ServiceLoader.load(IBaritoneProvider.class);
         Iterator<IBaritoneProvider> instances = baritoneLoader.iterator();
         provider = instances.next();
+
+        settings = new Settings();
+        SettingsUtil.readAndApply(settings);
     }
 
     public static IBaritoneProvider getProvider() {

src/api/java/baritone/api/IBaritone.java

@@ -20,11 +20,9 @@ package baritone.api;
 import baritone.api.behavior.ILookBehavior;
 import baritone.api.behavior.IPathingBehavior;
 import baritone.api.cache.IWorldProvider;
-import baritone.api.command.manager.ICommandManager;
 import baritone.api.event.listener.IEventBus;
 import baritone.api.pathing.calc.IPathingControlManager;
 import baritone.api.process.*;
-import baritone.api.selection.ISelectionManager;
 import baritone.api.utils.IInputOverrideHandler;
 import baritone.api.utils.IPlayerContext;
 
@@ -34,6 +32,15 @@ import baritone.api.utils.IPlayerContext;
  */
 public interface IBaritone {
 
+    /**
+     * Call as soon as Minecraft is ready, initializes all of the processes, behaviors, etc. This will
+     * only effectively be ran once, any additional calls are redundant because the initialization state
+     * is saved.
+     * <p>
+     * Or whenever your overeager utility client wants.
+     */
+    void init();
+
     /**
      * @return The {@link IPathingBehavior} instance
      * @see IPathingBehavior
@@ -121,20 +128,5 @@ public interface IBaritone {
      */
     IEventBus getGameEventHandler();
 
-    /**
-     * @return The {@link ISelectionManager} instance
-     * @see ISelectionManager
-     */
-    ISelectionManager getSelectionManager();
-
-    /**
-     * @return The {@link ICommandManager} instance
-     * @see ICommandManager
-     */
-    ICommandManager getCommandManager();
-
-    /**
-     * Open click
-     */
     void openClick();
 }

src/api/java/baritone/api/IBaritoneProvider.java

@@ -18,16 +18,12 @@
 package baritone.api;
 
 import baritone.api.cache.IWorldScanner;
-import baritone.api.command.ICommand;
-import baritone.api.command.ICommandSystem;
-import baritone.api.schematic.ISchematicSystem;
 import net.minecraft.client.entity.EntityPlayerSP;
 
 import java.util.List;
-import java.util.Objects;
 
 /**
- * Provides the present {@link IBaritone} instances, as well as non-baritone instance related APIs.
+ * Provides the present {@link IBaritone} instances
  *
  * @author leijurv
  */
@@ -61,7 +57,7 @@ public interface IBaritoneProvider {
      */
     default IBaritone getBaritoneForPlayer(EntityPlayerSP player) {
         for (IBaritone baritone : getAllBaritones()) {
-            if (Objects.equals(player, baritone.getPlayerContext().player())) {
+            if (player.equals(baritone.getPlayerContext().player())) {
                 return baritone;
             }
         }
@@ -75,17 +71,4 @@ public interface IBaritoneProvider {
      * @return The {@link IWorldScanner} instance.
      */
     IWorldScanner getWorldScanner();
-
-    /**
-     * Returns the {@link ICommandSystem} instance. This is not bound to a specific {@link IBaritone}
-     * instance because {@link ICommandSystem} itself controls global behavior for {@link ICommand}s.
-     *
-     * @return The {@link ICommandSystem} instance.
-     */
-    ICommandSystem getCommandSystem();
-
-    /**
-     * @return The {@link ISchematicSystem} instance.
-     */
-    ISchematicSystem getSchematicSystem();
 }

src/api/java/baritone/api/Settings.java

@@ -17,10 +17,8 @@
 
 package baritone.api;
 
-import baritone.api.utils.NotificationHelper;
 import baritone.api.utils.SettingsUtil;
 import baritone.api.utils.TypeUtils;
-import baritone.api.utils.gui.BaritoneToast;
 import net.minecraft.block.Block;
 import net.minecraft.client.Minecraft;
 import net.minecraft.init.Blocks;
@@ -32,10 +30,9 @@ import java.awt.*;
 import java.lang.reflect.Field;
 import java.lang.reflect.ParameterizedType;
 import java.lang.reflect.Type;
-import java.util.List;
 import java.util.*;
+import java.util.List;
 import java.util.function.Consumer;
-import java.util.function.BiConsumer;
 
 /**
  * Baritone's settings. Settings apply to all Baritone instances.
@@ -64,19 +61,6 @@ public final class Settings {
      */
     public final Setting<Boolean> allowInventory = new Setting<>(false);
 
-    /**
-     * Disable baritone's auto-tool at runtime, but still assume that another mod will provide auto tool functionality
-     * <p>
-     * Specifically, path calculation will still assume that an auto tool will run at execution time, even though
-     * Baritone itself will not do that.
-     */
-    public final Setting<Boolean> assumeExternalAutoTool = new Setting<>(false);
-
-    /**
-     * Automatically select the best available tool
-     */
-    public final Setting<Boolean> autoTool = new Setting<>(true);
-
     /**
      * It doesn't actually take twenty ticks to place a block, this cost is so high
      * because we want to generally conserve blocks which might be limited.
@@ -140,13 +124,6 @@ public final class Settings {
      */
     public final Setting<Boolean> allowJumpAt256 = new Setting<>(false);
 
-    /**
-     * This should be monetized it's so good
-     * <p>
-     * Defaults to true, but only actually takes effect if allowParkour is also true
-     */
-    public final Setting<Boolean> allowParkourAscend = new Setting<>(true);
-
     /**
      * Allow descending diagonally
      * <p>
@@ -156,13 +133,6 @@ public final class Settings {
      */
     public final Setting<Boolean> allowDiagonalDescend = new Setting<>(false);
 
-    /**
-     * Allow diagonal ascending
-     * <p>
-     * Actually pretty safe, much safer than diagonal descend tbh
-     */
-    public final Setting<Boolean> allowDiagonalAscend = new Setting<>(false);
-
     /**
      * Allow mining the block directly beneath its feet
      * <p>
@@ -174,10 +144,10 @@ public final class Settings {
      * Blocks that Baritone is allowed to place (as throwaway, for sneak bridging, pillaring, etc.)
      */
     public final Setting<List<Item>> acceptableThrowawayItems = new Setting<>(new ArrayList<>(Arrays.asList(
-            Item.getItemFromBlock(Blocks.DIRT),
-            Item.getItemFromBlock(Blocks.COBBLESTONE),
-            Item.getItemFromBlock(Blocks.NETHERRACK),
-            Item.getItemFromBlock(Blocks.STONE)
+            Blocks.DIRT.asItem(),
+            Blocks.COBBLESTONE.asItem(),
+            Blocks.NETHERRACK.asItem(),
+            Blocks.STONE.asItem()
     )));
 
     /**
@@ -193,59 +163,10 @@ public final class Settings {
     public final Setting<List<Block>> blocksToAvoidBreaking = new Setting<>(new ArrayList<>(Arrays.asList( // TODO can this be a HashSet or ImmutableSet?
             Blocks.CRAFTING_TABLE,
             Blocks.FURNACE,
-            Blocks.LIT_FURNACE,
             Blocks.CHEST,
-            Blocks.TRAPPED_CHEST,
-            Blocks.STANDING_SIGN,
-            Blocks.WALL_SIGN
+            Blocks.TRAPPED_CHEST
     )));
 
-    /**
-     * A list of blocks to be treated as if they're air.
-     * <p>
-     * If a schematic asks for air at a certain position, and that position currently contains a block on this list, it will be treated as correct.
-     */
-    public final Setting<List<Block>> buildIgnoreBlocks = new Setting<>(new ArrayList<>(Arrays.asList(
-
-    )));
-
-    /**
-     * A list of blocks to be treated as correct.
-     * <p>
-     * If a schematic asks for any block on this list at a certain position, it will be treated as correct, regardless of what it currently is.
-     */
-    public final Setting<List<Block>> buildSkipBlocks = new Setting<>(new ArrayList<>(Arrays.asList(
-
-    )));
-
-    /**
-     * A mapping of blocks to blocks treated as correct in their position
-     * <p>
-     * If a schematic asks for a block on this mapping, all blocks on the mapped list will be accepted at that location as well
-     */
-    public final Setting<Map<Block, List<Block>>> buildValidSubstitutes = new Setting<>(new HashMap<>());
-
-    /**
-     * A mapping of blocks to blocks to be built instead
-     * <p>
-     * If a schematic asks for a block on this mapping, Baritone will place the first placeable block in the mapped list
-     */
-    public final Setting<Map<Block, List<Block>>> buildSubstitutes = new Setting<>(new HashMap<>());
-
-    /**
-     * A list of blocks to become air
-     * <p>
-     * If a schematic asks for a block on this list, only air will be accepted at that location (and nothing on buildIgnoreBlocks)
-     */
-    public final Setting<List<Block>> okIfAir = new Setting<>(new ArrayList<>(Arrays.asList(
-
-    )));
-
-    /**
-     * If this is true, the builder will treat all non-air blocks as correct. It will only place new blocks.
-     */
-    public final Setting<Boolean> buildIgnoreExisting = new Setting<>(false);
-
     /**
      * If this setting is true, Baritone will never break a block that is adjacent to an unsupported falling block.
      * <p>
@@ -257,7 +178,7 @@ public final class Settings {
      * Enables some more advanced vine features. They're honestly just gimmicks and won't ever be needed in real
      * pathing scenarios. And they can cause Baritone to get trapped indefinitely in a strange scenario.
      * <p>
-     * Almost never turn this on lol
+     * Never turn this on lol
      */
     public final Setting<Boolean> allowVines = new Setting<>(false);
 
@@ -296,7 +217,7 @@ public final class Settings {
     /**
      * If we overshoot a traverse and end up one block beyond the destination, mark it as successful anyway.
      * <p>
-     * This helps with speed exceeding 20m/s
+     * This helps with speed at >=20m/s
      */
     public final Setting<Boolean> overshootTraverse = new Setting<>(true);
 
@@ -310,16 +231,6 @@ public final class Settings {
      */
     public final Setting<Integer> rightClickSpeed = new Setting<>(4);
 
-    /**
-     * Block reach distance
-     */
-    public final Setting<Float> blockReachDistance = new Setting<>(4.5f);
-
-    /**
-     * How many degrees to randomize the pitch and yaw every tick. Set to 0 to disable
-     */
-    public final Setting<Double> randomLooking = new Setting<>(0.01d);
-
     /**
      * This is the big A* setting.
      * As long as your cost heuristic is an *underestimate*, it's guaranteed to find you the best path.
@@ -478,11 +389,6 @@ public final class Settings {
      */
     public final Setting<Boolean> simplifyUnloadedYCoord = new Setting<>(true);
 
-    /**
-     * Whenever a block changes, repack the whole chunk that it's in
-     */
-    public final Setting<Boolean> repackOnAnyBlockChange = new Setting<>(true);
-
     /**
      * If a movement takes this many ticks more than its initial cost estimate, cancel it
      */
@@ -555,18 +461,6 @@ public final class Settings {
      */
     public final Setting<Boolean> backfill = new Setting<>(false);
 
-    /**
-     * Shows popup message in the upper right corner, similarly to when you make an advancement
-     */
-    public final Setting<Boolean> logAsToast = new Setting<>(false);
-
-    /**
-     * The time of how long the message in the pop-up will display
-     * <p>
-     * If below 1000L (1sec), it's better to disable this
-     */
-    public final Setting<Long> toastTimer = new Setting<>(5000L);
-
     /**
      * Print all the debug messages to chat
      */
@@ -579,31 +473,20 @@ public final class Settings {
     public final Setting<Boolean> chatControl = new Setting<>(true);
 
     /**
-     * Some clients like Impact try to force chatControl to off, so here's a second setting to do it anyway
+     * A second override over chatControl to force it on
      */
-    public final Setting<Boolean> chatControlAnyway = new Setting<>(false);
+    public final Setting<Boolean> removePrefix = new Setting<>(false);
 
     /**
      * Render the path
      */
     public final Setting<Boolean> renderPath = new Setting<>(true);
 
-    /**
-     * Render the path as a line instead of a frickin thingy
-     */
-    public final Setting<Boolean> renderPathAsLine = new Setting<>(false);
-
     /**
      * Render the goal
      */
     public final Setting<Boolean> renderGoal = new Setting<>(true);
 
-    /**
-     * Render the goal as a sick animated thingy instead of just a box
-     * (also controls animation of GoalXZ if {@link #renderGoalXZBeacon} is enabled)
-     */
-    public final Setting<Boolean> renderGoalAnimated = new Setting<>(true);
-
     /**
      * Render selection boxes
      */
@@ -673,7 +556,7 @@ public final class Settings {
 
     /**
      * When GetToBlockProcess or MineProcess fails to calculate a path, instead of just giving up, mark the closest instance
-     * of that block as "unreachable" and go towards the next closest. GetToBlock expands this search to the whole "vein"; MineProcess does not.
+     * of that block as "unreachable" and go towards the next closest. GetToBlock expands this seaarch to the whole "vein"; MineProcess does not.
      * This is because MineProcess finds individual impossible blocks (like one block in a vein that has gravel on top then lava, so it can't break)
      * Whereas GetToBlock should blacklist the whole "vein" if it can't get to any of them.
      */
@@ -699,51 +582,10 @@ public final class Settings {
     public final Setting<Float> cachedChunksOpacity = new Setting<>(0.5f);
 
     /**
-     * Whether or not to allow you to run Baritone commands with the prefix
+     * Whether or not to use the "#" command prefix
      */
     public final Setting<Boolean> prefixControl = new Setting<>(true);
 
-    /**
-     * The command prefix for chat control
-     */
-    public final Setting<String> prefix = new Setting<>("#");
-
-    /**
-     * Use a short Baritone prefix [B] instead of [Baritone] when logging to chat
-     */
-    public final Setting<Boolean> shortBaritonePrefix = new Setting<>(false);
-
-    /**
-     * Echo commands to chat when they are run
-     */
-    public final Setting<Boolean> echoCommands = new Setting<>(true);
-
-    /**
-     * Censor coordinates in goals and block positions
-     */
-    public final Setting<Boolean> censorCoordinates = new Setting<>(false);
-
-    /**
-     * Censor arguments to ran commands, to hide, for example, coordinates to #goal
-     */
-    public final Setting<Boolean> censorRanCommands = new Setting<>(false);
-
-    /**
-     * Stop using tools just before they are going to break.
-     */
-    public final Setting<Boolean> itemSaver = new Setting<>(false);
-
-    /**
-     * Durability to leave on the tool when using itemSaver
-     */
-    public final Setting<Integer> itemSaverThreshold = new Setting<>(10);
-
-    /**
-     * Always prefer silk touch tools over regular tools. This will not sacrifice speed, but it will always prefer silk
-     * touch tools over other tools of the same speed. This includes always choosing ANY silk touch tool over your hand.
-     */
-    public final Setting<Boolean> preferSilkTouch = new Setting<>(false);
-
     /**
      * Don't stop walking forward when you need to break blocks in your way
      */
@@ -781,24 +623,7 @@ public final class Settings {
     public final Setting<Integer> maxCachedWorldScanCount = new Setting<>(10);
 
     /**
-     * Sets the minimum y level whilst mining - set to 0 to turn off.
-     */
-    public final Setting<Integer> minYLevelWhileMining = new Setting<>(0);
-
-    /**
-     * This will only allow baritone to mine exposed ores, can be used to stop ore obfuscators on servers that use them.
-     */
-    public final Setting<Boolean> allowOnlyExposedOres = new Setting<>(false);
-
-    /**
-     * When allowOnlyExposedOres is enabled this is the distance around to search.
-     * <p>
-     * It is recommended to keep this value low, as it dramatically increases calculation times.
-     */
-    public final Setting<Integer> allowOnlyExposedOresDistance = new Setting<>(1);
-
-    /**
-     * When GetToBlock or non-legit Mine doesn't know any locations for the desired block, explore randomly instead of giving up.
+     * When GetToBlock doesn't know any locations for the desired block, explore randomly instead of giving up.
      */
     public final Setting<Boolean> exploreForBlocks = new Setting<>(true);
 
@@ -822,12 +647,7 @@ public final class Settings {
     public final Setting<Integer> exploreMaintainY = new Setting<>(64);
 
     /**
-     * Replant normal Crops while farming and leave cactus and sugarcane to regrow
-     */
-    public final Setting<Boolean> replantCrops = new Setting<>(true);
-
-    /**
-     * Replant nether wart while farming. This setting only has an effect when replantCrops is also enabled
+     * Replant nether wart while farming
      */
     public final Setting<Boolean> replantNetherWart = new Setting<>(false);
 
@@ -850,44 +670,11 @@ public final class Settings {
      */
     public final Setting<Boolean> layerOrder = new Setting<>(false);
 
-    /**
-     * How high should the individual layers be?
-     */
-    public final Setting<Integer> layerHeight = new Setting<>(1);
-
-    /**
-     * Start building the schematic at a specific layer.
-     * Can help on larger builds when schematic wants to break things its already built
-     */
-    public final Setting<Integer> startAtLayer = new Setting<>(0);
-
-    /**
-     * If a layer is unable to be constructed, just skip it.
-     */
-    public final Setting<Boolean> skipFailedLayers = new Setting<>(false);
-
-    /**
-     * Only build the selected part of schematics
-     */
-     public final Setting<Boolean> buildOnlySelection = new Setting<>(false);
-
     /**
      * How far to move before repeating the build. 0 to disable repeating on a certain axis, 0,0,0 to disable entirely
      */
     public final Setting<Vec3i> buildRepeat = new Setting<>(new Vec3i(0, 0, 0));
 
-    /**
-     * How many times to buildrepeat. -1 for infinite.
-     */
-    public final Setting<Integer> buildRepeatCount = new Setting<>(-1);
-
-    /**
-     * Don't notify schematics that they are moved.
-     * e.g. replacing will replace the same spots for every repetition
-     * Mainly for backward compatibility.
-     */
-    public final Setting<Boolean> buildRepeatSneaky = new Setting<>(true);
-
     /**
      * Allow standing above a block while mining it, in BuilderProcess
      * <p>
@@ -902,71 +689,11 @@ public final class Settings {
      */
     public final Setting<Boolean> goalBreakFromAbove = new Setting<>(false);
 
-    /**
-     * Build in map art mode, which makes baritone only care about the top block in each column
-     */
-    public final Setting<Boolean> mapArtMode = new Setting<>(false);
-
-    /**
-     * Override builder's behavior to not attempt to correct blocks that are currently water
-     */
-    public final Setting<Boolean> okIfWater = new Setting<>(false);
-
-    /**
-     * The set of incorrect blocks can never grow beyond this size
-     */
-    public final Setting<Integer> incorrectSize = new Setting<>(100);
-
-    /**
-     * Multiply the cost of breaking a block that's correct in the builder's schematic by this coefficient
-     */
-    public final Setting<Double> breakCorrectBlockPenaltyMultiplier = new Setting<>(10d);
-
-    /**
-     * When this setting is true, build a schematic with the highest X coordinate being the origin, instead of the lowest
-     */
-    public final Setting<Boolean> schematicOrientationX = new Setting<>(false);
-
-    /**
-     * When this setting is true, build a schematic with the highest Y coordinate being the origin, instead of the lowest
-     */
-    public final Setting<Boolean> schematicOrientationY = new Setting<>(false);
-
-    /**
-     * When this setting is true, build a schematic with the highest Z coordinate being the origin, instead of the lowest
-     */
-    public final Setting<Boolean> schematicOrientationZ = new Setting<>(false);
-
-    /**
-     * The fallback used by the build command when no extension is specified. This may be useful if schematics of a
-     * particular format are used often, and the user does not wish to have to specify the extension with every usage.
-     */
-    public final Setting<String> schematicFallbackExtension = new Setting<>("schematic");
-
-    /**
-     * Distance to scan every tick for updates. Expanding this beyond player reach distance (i.e. setting it to 6 or above)
-     * is only necessary in very large schematics where rescanning the whole thing is costly.
-     */
-    public final Setting<Integer> builderTickScanRadius = new Setting<>(5);
-
     /**
      * While mining, should it also consider dropped items of the correct type as a pathing destination (as well as ore blocks)?
      */
     public final Setting<Boolean> mineScanDroppedItems = new Setting<>(true);
 
-    /**
-     * While mining, wait this number of milliseconds after mining an ore to see if it will drop an item
-     * instead of immediately going onto the next one
-     * <p>
-     * Thanks Louca
-     */
-    public final Setting<Long> mineDropLoiterDurationMSThanksLouca = new Setting<>(250L);
-
-    /**
-     * Trim incorrect positions too far away, helps performance but hurts reliability in very large schematics
-     */
-    public final Setting<Boolean> distanceTrim = new Setting<>(true);
-
     /**
      * Cancel the current path if the goal has changed, and the path originally ended in the goal but doesn't anymore.
      * <p>
@@ -996,7 +723,6 @@ public final class Settings {
      * Disallow MineBehavior from using X-Ray to see where the ores are. Turn this option on to force it to mine "legit"
      * where it will only mine an ore once it can actually see it, so it won't do or know anything that a normal player
      * couldn't. If you don't want it to look like you're X-Raying, turn this on
-     * This will always explore, regardless of exploreForBlocks
      */
     public final Setting<Boolean> legitMine = new Setting<>(false);
 
@@ -1081,21 +807,7 @@ public final class Settings {
      * via {@link Consumer#andThen(Consumer)} or it can completely be overriden via setting
      * {@link Setting#value};
      */
-    public final Setting<Consumer<ITextComponent>> logger = new Setting<>(Minecraft.getMinecraft().ingameGUI.getChatGUI()::printChatMessage);
-
-    /**
-     * The function that is called when Baritone will send a desktop notification. This function can be added to
-     * via {@link Consumer#andThen(Consumer)} or it can completely be overriden via setting
-     * {@link Setting#value};
-     */
-    public final Setting<BiConsumer<String, Boolean>> notifier = new Setting<>(NotificationHelper::notify);
-
-    /**
-     * The function that is called when Baritone will show a toast. This function can be added to
-     * via {@link Consumer#andThen(Consumer)} or it can completely be overriden via setting
-     * {@link Setting#value};
-     */
-    public final Setting<BiConsumer<ITextComponent, ITextComponent>> toaster = new Setting<>(BaritoneToast::addOrUpdate);
+    public final Setting<Consumer<ITextComponent>> logger = new Setting<>(Minecraft.getInstance().ingameGUI.getChatGUI()::printChatMessage);
 
     /**
      * The size of the box that is rendered when the current goal is a GoalYLevel
@@ -1142,85 +854,6 @@ public final class Settings {
      */
     public final Setting<Color> colorGoalBox = new Setting<>(Color.GREEN);
 
-    /**
-     * The color of the goal box when it's inverted
-     */
-    public final Setting<Color> colorInvertedGoalBox = new Setting<>(Color.RED);
-
-    /**
-     * The color of all selections
-     */
-    public final Setting<Color> colorSelection = new Setting<>(Color.CYAN);
-
-    /**
-     * The color of the selection pos 1
-     */
-    public final Setting<Color> colorSelectionPos1 = new Setting<>(Color.BLACK);
-
-    /**
-     * The color of the selection pos 2
-     */
-    public final Setting<Color> colorSelectionPos2 = new Setting<>(Color.ORANGE);
-
-    /**
-     * The opacity of the selection. 0 is completely transparent, 1 is completely opaque
-     */
-    public final Setting<Float> selectionOpacity = new Setting<>(.5f);
-
-    /**
-     * Line width of the goal when rendered, in pixels
-     */
-    public final Setting<Float> selectionLineWidth = new Setting<>(2F);
-
-    /**
-     * Render selections
-     */
-    public final Setting<Boolean> renderSelection = new Setting<>(true);
-
-    /**
-     * Ignore depth when rendering selections
-     */
-    public final Setting<Boolean> renderSelectionIgnoreDepth = new Setting<>(true);
-
-    /**
-     * Render selection corners
-     */
-    public final Setting<Boolean> renderSelectionCorners = new Setting<>(true);
-
-    /**
-     * Use sword to mine.
-     */
-    public final Setting<Boolean> useSwordToMine = new Setting<>(true);
-
-    /**
-     * Desktop notifications
-     */
-    public final Setting<Boolean> desktopNotifications = new Setting<>(false);
-
-    /**
-     * Desktop notification on path complete
-     */
-    public final Setting<Boolean> notificationOnPathComplete = new Setting<>(true);
-
-    /**
-     * Desktop notification on farm fail
-     */
-    public final Setting<Boolean> notificationOnFarmFail = new Setting<>(true);
-
-    /**
-     * Desktop notification on build finished
-     */
-    public final Setting<Boolean> notificationOnBuildFinished = new Setting<>(true);
-
-    /**
-     * Desktop notification on explore finished
-     */
-    public final Setting<Boolean> notificationOnExploreFinished = new Setting<>(true);
-
-    /**
-     * Desktop notification on mine fail
-     */
-    public final Setting<Boolean> notificationOnMineFail = new Setting<>(true);
 
     /**
      * A map of lowercase setting field names to their respective setting
@@ -1235,7 +868,6 @@ public final class Settings {
     public final Map<Setting<?>, Type> settingTypes;
 
     public final class Setting<T> {
-
         public T value;
         public final T defaultValue;
         private String name;
@@ -1318,10 +950,10 @@ public final class Settings {
     }
 
     @SuppressWarnings("unchecked")
-    public <T> List<Setting<T>> getAllValuesByType(Class<T> cla$$) {
+    public <T> List<Setting<T>> getAllValuesByType(Class<T> klass) {
         List<Setting<T>> result = new ArrayList<>();
         for (Setting<?> setting : allSettings) {
-            if (setting.getValueClass().equals(cla$$)) {
+            if (setting.getValueClass().equals(klass)) {
                 result.add((Setting<T>) setting);
             }
         }

src/api/java/baritone/api/behavior/IPathingBehavior.java

@@ -58,32 +58,15 @@ public interface IPathingBehavior extends IBehavior {
         return Optional.of(current.getPath().ticksRemainingFrom(start));
     }
 
-    /**
-     * Returns the estimated remaining ticks to the current goal.
-     * Given that the return type is an optional, {@link Optional#empty()}
-     * will be returned in the case that there is no current goal.
-     *
-     * @return The estimated remaining ticks to the current goal.
-     */
-    Optional<Double> estimatedTicksToGoal();
-
     /**
      * @return The current pathing goal
      */
     Goal getGoal();
 
     /**
-     * @return Whether or not a path is currently being executed. This will be false if there's currently a pause.
-     * @see #hasPath()
+     * @return Whether or not a path is currently being executed.
      */
-    boolean isPathing();
-
-    /**
-     * @return If there is a current path. Note that the path is not necessarily being executed, for example when there
-     * is a pause in effect.
-     * @see #isPathing()
-     */
-    default boolean hasPath() {
+    default boolean isPathing() {
         return getCurrent() != null;
     }
 

src/api/java/baritone/api/cache/ICachedWorld.java

@@ -81,4 +81,6 @@ public interface ICachedWorld {
      * in a new thread by default.
      */
     void save();
+
+
 }

src/api/java/baritone/api/cache/IWaypoint.java

@@ -17,9 +17,12 @@
 
 package baritone.api.cache;
 
-import baritone.api.utils.BetterBlockPos;
+import net.minecraft.util.math.BlockPos;
+import org.apache.commons.lang3.ArrayUtils;
 
-import java.util.*;
+import java.util.Arrays;
+import java.util.Collections;
+import java.util.List;
 
 /**
  * A marker for a position in the world.
@@ -57,7 +60,7 @@ public interface IWaypoint {
      *
      * @return The block position of this waypoint
      */
-    BetterBlockPos getLocation();
+    BlockPos getLocation();
 
     enum Tag {
 
@@ -89,48 +92,20 @@ public interface IWaypoint {
         /**
          * The names for the tag, anything that the tag can be referred to as.
          */
-        public final String[] names;
+        private final String[] names;
 
         Tag(String... names) {
             this.names = names;
         }
 
         /**
-         * @return A name that can be passed to {@link #getByName(String)} to retrieve this tag
-         */
-        public String getName() {
-            return names[0];
-        }
-
-        /**
-         * Gets a tag by one of its names.
+         * Finds a tag from one of the names that could be used to identify said tag.
          *
-         * @param name The name to search for.
-         * @return The tag, if found, or null.
+         * @param name The name of the tag
+         * @return The tag, if one is found, otherwise, {@code null}
          */
-        public static Tag getByName(String name) {
-            for (Tag action : Tag.values()) {
-                for (String alias : action.names) {
-                    if (alias.equalsIgnoreCase(name)) {
-                        return action;
-                    }
-                }
-            }
-
-            return null;
-        }
-
-        /**
-         * @return All tag names.
-         */
-        public static String[] getAllNames() {
-            Set<String> names = new HashSet<>();
-
-            for (Tag tag : Tag.values()) {
-                names.addAll(Arrays.asList(tag.names));
-            }
-
-            return names.toArray(new String[0]);
+        public static Tag fromString(String name) {
+            return TAG_LIST.stream().filter(tag -> ArrayUtils.contains(tag.names, name.toLowerCase())).findFirst().orElse(null);
         }
     }
 }

src/api/java/baritone/api/cache/IWorldScanner.java

@@ -17,7 +17,6 @@
 
 package baritone.api.cache;
 
-import baritone.api.utils.BlockOptionalMetaLookup;
 import baritone.api.utils.IPlayerContext;
 import net.minecraft.block.Block;
 import net.minecraft.util.math.BlockPos;
@@ -34,63 +33,28 @@ public interface IWorldScanner {
     /**
      * Scans the world, up to the specified max chunk radius, for the specified blocks.
      *
-     * @param ctx             The {@link IPlayerContext} containing player and world info that the scan is based upon
-     * @param filter          The blocks to scan for
+     * @param ctx             The {@link IPlayerContext} containing player and world info that the
+     *                        scan is based upon
+     * @param blocks          The blocks to scan for
      * @param max             The maximum number of blocks to scan before cutoff
-     * @param yLevelThreshold If a block is found within this Y level, the current result will be returned, if the value
-     *                        is negative, then this condition doesn't apply.
+     * @param yLevelThreshold If a block is found within this Y level, the current result will be
+     *                        returned, if the value is negative, then this condition doesn't apply.
      * @param maxSearchRadius The maximum chunk search radius
      * @return The matching block positions
      */
-    List<BlockPos> scanChunkRadius(IPlayerContext ctx, BlockOptionalMetaLookup filter, int max, int yLevelThreshold, int maxSearchRadius);
-
-    default List<BlockPos> scanChunkRadius(IPlayerContext ctx, List<Block> filter, int max, int yLevelThreshold, int maxSearchRadius) {
-        return scanChunkRadius(ctx, new BlockOptionalMetaLookup(filter.toArray(new Block[0])), max, yLevelThreshold, maxSearchRadius);
-    }
+    List<BlockPos> scanChunkRadius(IPlayerContext ctx, List<Block> blocks, int max, int yLevelThreshold, int maxSearchRadius);
 
     /**
      * Scans a single chunk for the specified blocks.
      *
-     * @param ctx             The {@link IPlayerContext} containing player and world info that the scan is based upon
-     * @param filter          The blocks to scan for
-     * @param pos             The position of the target chunk
-     * @param max             The maximum number of blocks to scan before cutoff
-     * @param yLevelThreshold If a block is found within this Y level, the current result will be returned, if the value
-     *                        is negative, then this condition doesn't apply.
-     * @return The matching block positions
-     */
-    List<BlockPos> scanChunk(IPlayerContext ctx, BlockOptionalMetaLookup filter, ChunkPos pos, int max, int yLevelThreshold);
-
-    /**
-     * Scans a single chunk for the specified blocks.
-     *
-     * @param ctx             The {@link IPlayerContext} containing player and world info that the scan is based upon
+     * @param ctx             The {@link IPlayerContext} containing player and world info that the
+     *                        scan is based upon
      * @param blocks          The blocks to scan for
      * @param pos             The position of the target chunk
      * @param max             The maximum number of blocks to scan before cutoff
-     * @param yLevelThreshold If a block is found within this Y level, the current result will be returned, if the value
-     *                        is negative, then this condition doesn't apply.
+     * @param yLevelThreshold If a block is found within this Y level, the current result will be
+     *                        returned, if the value is negative, then this condition doesn't apply.
      * @return The matching block positions
      */
-    default List<BlockPos> scanChunk(IPlayerContext ctx, List<Block> blocks, ChunkPos pos, int max, int yLevelThreshold) {
-        return scanChunk(ctx, new BlockOptionalMetaLookup(blocks), pos, max, yLevelThreshold);
-    }
-
-    /**
-     * Overload of {@link #repack(IPlayerContext, int)} where the value of the {@code range} parameter is {@code 40}.
-     *
-     * @param ctx The player, describing the origin
-     * @return The amount of chunks successfully queued for repacking
-     */
-    int repack(IPlayerContext ctx);
-
-    /**
-     * Queues the chunks in a square formation around the specified player, using the specified
-     * range, which represents 1/2 the square's dimensions, where the player is in the center.
-     *
-     * @param ctx   The player, describing the origin
-     * @param range The range to repack
-     * @return The amount of chunks successfully queued for repacking
-     */
-    int repack(IPlayerContext ctx, int range);
+    List<BlockPos> scanChunk(IPlayerContext ctx, List<Block> blocks, ChunkPos pos, int max, int yLevelThreshold);
 }

src/api/java/baritone/api/cache/Waypoint.java

@@ -17,7 +17,7 @@
 
 package baritone.api.cache;
 
-import baritone.api.utils.BetterBlockPos;
+import net.minecraft.util.math.BlockPos;
 
 import java.util.Date;
 
@@ -31,9 +31,9 @@ public class Waypoint implements IWaypoint {
     private final String name;
     private final Tag tag;
     private final long creationTimestamp;
-    private final BetterBlockPos location;
+    private final BlockPos location;
 
-    public Waypoint(String name, Tag tag, BetterBlockPos location) {
+    public Waypoint(String name, Tag tag, BlockPos location) {
         this(name, tag, location, System.currentTimeMillis());
     }
 
@@ -46,7 +46,7 @@ public class Waypoint implements IWaypoint {
      * @param location          The waypoint location
      * @param creationTimestamp When the waypoint was created
      */
-    public Waypoint(String name, Tag tag, BetterBlockPos location, long creationTimestamp) {
+    public Waypoint(String name, Tag tag, BlockPos location, long creationTimestamp) {
         this.name = name;
         this.tag = tag;
         this.location = location;
@@ -55,7 +55,7 @@ public class Waypoint implements IWaypoint {
 
     @Override
     public int hashCode() {
-        return name.hashCode() ^ tag.hashCode() ^ location.hashCode() ^ Long.hashCode(creationTimestamp);
+        return name.hashCode() + tag.hashCode() + location.hashCode(); //lol
     }
 
     @Override
@@ -74,18 +74,13 @@ public class Waypoint implements IWaypoint {
     }
 
     @Override
-    public BetterBlockPos getLocation() {
+    public BlockPos getLocation() {
         return this.location;
     }
 
     @Override
     public String toString() {
-        return String.format(
-                "%s %s %s",
-                name,
-                BetterBlockPos.from(location).toString(),
-                new Date(creationTimestamp).toString()
-        );
+        return name + " " + location.toString() + " " + new Date(creationTimestamp).toString();
     }
 
     @Override

src/api/java/baritone/api/command/Command.java

@@ -1,67 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command;
-
-import baritone.api.IBaritone;
-import baritone.api.utils.IPlayerContext;
-
-import java.util.Collections;
-import java.util.List;
-import java.util.Locale;
-import java.util.stream.Collectors;
-import java.util.stream.Stream;
-
-/**
- * A default implementation of {@link ICommand} which provides easy access to the
- * command's bound {@link IBaritone} instance, {@link IPlayerContext} and an easy
- * way to provide multiple valid command execution names through the default constructor.
- * <p>
- * So basically, you should use it because it provides a small amount of boilerplate,
- * but you're not forced to use it.
- *
- * @author LoganDark
- * @see ICommand
- */
-public abstract class Command implements ICommand {
-
-    protected IBaritone baritone;
-    protected IPlayerContext ctx;
-
-    /**
-     * The names of this command. This is what you put after the command prefix.
-     */
-    protected final List<String> names;
-
-    /**
-     * Creates a new Baritone control command.
-     *
-     * @param names The names of this command. This is what you put after the command prefix.
-     */
-    protected Command(IBaritone baritone, String... names) {
-        this.names = Collections.unmodifiableList(Stream.of(names)
-                .map(s -> s.toLowerCase(Locale.US))
-                .collect(Collectors.toList()));
-        this.baritone = baritone;
-        this.ctx = baritone.getPlayerContext();
-    }
-
-    @Override
-    public final List<String> getNames() {
-        return this.names;
-    }
-}

src/api/java/baritone/api/command/IBaritoneChatControl.java

@@ -1,42 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command;
-
-import baritone.api.Settings;
-
-import java.util.UUID;
-
-/**
- * @author Brady
- * @since 9/26/2019
- */
-public interface IBaritoneChatControl {
-
-    /**
-     * In certain cases chat components need to execute commands for you. For example, the paginator automatically runs
-     * commands when you click the forward and back arrows to show you the previous/next page.
-     * <p>
-     * If the prefix is changed in the meantime, then the command will go to chat. That's no good. So here's a permanent
-     * prefix that forces a command to run, regardless of the current prefix, chat/prefix control being enabled, etc.
-     * <p>
-     * If used right (by both developers and users), it should be impossible to expose a command accidentally to the
-     * server. As a rule of thumb, if you have a clickable chat component, always use this prefix. If you're suggesting
-     * a command (a component that puts text into your text box, or something else), use {@link Settings#prefix}.
-     */
-    String FORCE_COMMAND_PREFIX = String.format("<<%s>>", UUID.randomUUID().toString());
-}

src/api/java/baritone/api/command/ICommand.java

@@ -1,67 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command;
-
-import baritone.api.command.argument.IArgConsumer;
-import baritone.api.command.exception.CommandException;
-import baritone.api.utils.Helper;
-
-import java.util.List;
-import java.util.stream.Stream;
-
-/**
- * The base for a command.
- *
- * @author Brady
- * @since 10/7/2019
- */
-public interface ICommand extends Helper {
-
-    /**
-     * Called when this command is executed.
-     */
-    void execute(String label, IArgConsumer args) throws CommandException;
-
-    /**
-     * Called when the command needs to tab complete. Return a Stream representing the entries to put in the completions
-     * list.
-     */
-    Stream<String> tabComplete(String label, IArgConsumer args) throws CommandException;
-
-    /**
-     * @return A <b>single-line</b> string containing a short description of this command's purpose.
-     */
-    String getShortDesc();
-
-    /**
-     * @return A list of lines that will be printed by the help command when the user wishes to view them.
-     */
-    List<String> getLongDesc();
-
-    /**
-     * @return A list of the names that can be accepted to have arguments passed to this command
-     */
-    List<String> getNames();
-
-    /**
-     * @return {@code true} if this command should be hidden from the help menu
-     */
-    default boolean hiddenFromHelp() {
-        return false;
-    }
-}

src/api/java/baritone/api/command/ICommandSystem.java

@@ -1,29 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command;
-
-import baritone.api.command.argparser.IArgParserManager;
-
-/**
- * @author Brady
- * @since 10/4/2019
- */
-public interface ICommandSystem {
-
-    IArgParserManager getParserManager();
-}

src/api/java/baritone/api/command/argparser/IArgParser.java

@@ -1,60 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.argparser;
-
-import baritone.api.command.argument.ICommandArgument;
-
-public interface IArgParser<T> {
-
-    /**
-     * @return the class of this parser.
-     */
-    Class<T> getTarget();
-
-    /**
-     * A stateless argument parser is just that. It takes a {@link ICommandArgument} and outputs its type.
-     */
-    interface Stateless<T> extends IArgParser<T> {
-
-        /**
-         * @param arg The argument to parse.
-         * @return What it was parsed into.
-         * @throws RuntimeException if you want the parsing to fail. The exception will be caught and turned into an
-         *                          appropriate error.
-         */
-        T parseArg(ICommandArgument arg) throws Exception;
-    }
-
-    /**
-     * A stated argument parser is similar to a stateless one. It also takes a {@link ICommandArgument}, but it also
-     * takes a second argument that can be any type, referred to as the state.
-     */
-    interface Stated<T, S> extends IArgParser<T> {
-
-        Class<S> getStateType();
-
-        /**
-         * @param arg   The argument to parse.
-         * @param state Can be anything.
-         * @return What it was parsed into.
-         * @throws RuntimeException if you want the parsing to fail. The exception will be caught and turned into an
-         *                          appropriate error.
-         */
-        T parseArg(ICommandArgument arg, S state) throws Exception;
-    }
-}

src/api/java/baritone/api/command/argparser/IArgParserManager.java

@@ -1,69 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.argparser;
-
-import baritone.api.command.argument.ICommandArgument;
-import baritone.api.command.exception.CommandInvalidTypeException;
-import baritone.api.command.registry.Registry;
-
-/**
- * Used to retrieve {@link IArgParser} instances from the registry, by their target class.
- * It can be assumed that a {@link IArgParser} exists for {@link Integer}, {@link Long},
- * {@link Float}, {@link Double} and {@link Boolean}.
- *
- * @author Brady
- * @since 10/4/2019
- */
-public interface IArgParserManager {
-
-    /**
-     * @param type The type trying to be parsed
-     * @return A parser that can parse arguments into this class, if found.
-     */
-    <T> IArgParser.Stateless<T> getParserStateless(Class<T> type);
-
-    /**
-     * @param type The type trying to be parsed
-     * @return A parser that can parse arguments into this class, if found.
-     */
-    <T, S> IArgParser.Stated<T, S> getParserStated(Class<T> type, Class<S> stateKlass);
-
-    /**
-     * Attempt to parse the specified argument with a stateless {@link IArgParser} that outputs the specified class.
-     *
-     * @param type The type to try and parse the argument into.
-     * @param arg  The argument to parse.
-     * @return An instance of the specified class.
-     * @throws CommandInvalidTypeException If the parsing failed
-     */
-    <T> T parseStateless(Class<T> type, ICommandArgument arg) throws CommandInvalidTypeException;
-
-    /**
-     * Attempt to parse the specified argument with a stated {@link IArgParser} that outputs the specified class.
-     *
-     * @param type  The type to try and parse the argument into.
-     * @param arg   The argument to parse.
-     * @param state The state to pass to the {@link IArgParser.Stated}.
-     * @return An instance of the specified class.
-     * @throws CommandInvalidTypeException If the parsing failed
-     * @see IArgParser.Stated
-     */
-    <T, S> T parseStated(Class<T> type, Class<S> stateKlass, ICommandArgument arg, S state) throws CommandInvalidTypeException;
-
-    Registry<IArgParser> getRegistry();
-}

src/api/java/baritone/api/command/argument/IArgConsumer.java

@@ -1,594 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.argument;
-
-import baritone.api.command.ICommand;
-import baritone.api.command.argparser.IArgParser;
-import baritone.api.command.datatypes.IDatatype;
-import baritone.api.command.datatypes.IDatatypeFor;
-import baritone.api.command.datatypes.IDatatypePost;
-import baritone.api.command.exception.CommandException;
-import baritone.api.command.exception.CommandInvalidTypeException;
-import baritone.api.command.exception.CommandNotEnoughArgumentsException;
-import baritone.api.command.exception.CommandTooManyArgumentsException;
-import baritone.api.utils.Helper;
-import net.minecraft.util.EnumFacing;
-
-import java.util.Deque;
-import java.util.LinkedList;
-import java.util.stream.Stream;
-
-/**
- * The {@link IArgConsumer} is how {@link ICommand}s read the arguments passed to them. This class has many benefits:
- *
- * <ul>
- * <li>Mutability. The whole concept of the {@link IArgConsumer}} is to let you gradually consume arguments in any way
- * you'd like. You can change your consumption based on earlier arguments, for subcommands for example.</li>
- * <li>You don't need to keep track of your consumption. The {@link IArgConsumer}} keeps track of the arguments you
- * consume so that it can throw detailed exceptions whenever something is out of the ordinary. Additionally, if you
- * need to retrieve an argument after you've already consumed it - look no further than {@link #consumed()}!</li>
- * <li>Easy retrieval of many different types. If you need to retrieve an instance of an int or float for example,
- * look no further than {@link #getAs(Class)}. If you need a more powerful way of retrieving data, try out the many
- * {@code getDatatype...} methods.</li>
- * <li>It's very easy to throw detailed exceptions. The {@link IArgConsumer}} has many different methods that can
- * enforce the number of arguments, the type of arguments, and more, throwing different types of
- * {@link CommandException}s if something seems off. You're recommended to do all validation and store all needed
- * data in variables BEFORE logging any data to chat via {@link Helper#logDirect(String)}, so that the error
- * handlers can do their job and log the error to chat.</li>
- * </ul>
- */
-public interface IArgConsumer {
-
-    LinkedList<ICommandArgument> getArgs();
-
-    Deque<ICommandArgument> getConsumed();
-
-    /**
-     * @param num The number of arguments to check for
-     * @return {@code true} if there are <i>at least</i> {@code num} arguments left in this {@link IArgConsumer}}
-     * @see #hasAny()
-     * @see #hasAtMost(int)
-     * @see #hasExactly(int)
-     */
-    boolean has(int num);
-
-    /**
-     * @return {@code true} if there is <i>at least</i> 1 argument left in this {@link IArgConsumer}}
-     * @see #has(int)
-     * @see #hasAtMostOne()
-     * @see #hasExactlyOne()
-     */
-    boolean hasAny();
-
-    /**
-     * @param num The number of arguments to check for
-     * @return {@code true} if there are <i>at most</i> {@code num} arguments left in this {@link IArgConsumer}}
-     * @see #has(int)
-     * @see #hasAtMost(int)
-     * @see #hasExactly(int)
-     */
-    boolean hasAtMost(int num);
-
-    /**
-     * @return {@code true} if there is <i>at most</i> 1 argument left in this {@link IArgConsumer}}
-     * @see #hasAny()
-     * @see #hasAtMostOne()
-     * @see #hasExactlyOne()
-     */
-    boolean hasAtMostOne();
-
-    /**
-     * @param num The number of arguments to check for
-     * @return {@code true} if there are <i>exactly</i> {@code num} arguments left in this {@link IArgConsumer}}
-     * @see #has(int)
-     * @see #hasAtMost(int)
-     */
-    boolean hasExactly(int num);
-
-    /**
-     * @return {@code true} if there is <i>exactly</i> 1 argument left in this {@link IArgConsumer}}
-     * @see #hasAny()
-     * @see #hasAtMostOne()
-     */
-    boolean hasExactlyOne();
-
-    /**
-     * @param index The index to peek
-     * @return The argument at index {@code index} in this {@link IArgConsumer}}, with 0 being the next one. This does not
-     * mutate the {@link IArgConsumer}}
-     * @throws CommandNotEnoughArgumentsException If there is less than {@code index + 1} arguments left
-     * @see #peek()
-     * @see #peekString(int)
-     * @see #peekAs(Class, int)
-     * @see #get()
-     */
-    ICommandArgument peek(int index) throws CommandNotEnoughArgumentsException;
-
-    /**
-     * @return The next argument in this {@link IArgConsumer}}. This does not mutate the {@link IArgConsumer}}
-     * @throws CommandNotEnoughArgumentsException If there is less than one argument left
-     * @see #peek(int)
-     * @see #peekString()
-     * @see #peekAs(Class)
-     * @see #get()
-     */
-    ICommandArgument peek() throws CommandNotEnoughArgumentsException;
-
-    /**
-     * @param index The index to peek
-     * @param type  The type to check for
-     * @return If an ArgParser.Stateless for the specified {@code type} would succeed in parsing the next
-     * argument
-     * @throws CommandNotEnoughArgumentsException If there is less than {@code index + 1} arguments left
-     * @see #peek()
-     * @see #getAs(Class)
-     */
-    boolean is(Class<?> type, int index) throws CommandNotEnoughArgumentsException;
-
-    /**
-     * @param type The type to check for
-     * @return If an ArgParser.Stateless for the specified {@code type} would succeed in parsing the next
-     * argument
-     * @throws CommandNotEnoughArgumentsException If there is less than one argument left
-     * @see #peek()
-     * @see #getAs(Class)
-     */
-    boolean is(Class<?> type) throws CommandNotEnoughArgumentsException;
-
-    /**
-     * @param index The index to peek
-     * @return The value of the argument at index {@code index} in this {@link IArgConsumer}}, with 0 being the next one
-     * This does not mutate the {@link IArgConsumer}}
-     * @throws CommandNotEnoughArgumentsException If there is less than {@code index + 1} arguments left
-     * @see #peek()
-     * @see #peekString()
-     */
-    String peekString(int index) throws CommandNotEnoughArgumentsException;
-
-    /**
-     * @return The value of the next argument in this {@link IArgConsumer}}. This does not mutate the {@link IArgConsumer}}
-     * @throws CommandNotEnoughArgumentsException If there is less than one argument left
-     * @see #peekString(int)
-     * @see #getString()
-     */
-    String peekString() throws CommandNotEnoughArgumentsException;
-
-    /**
-     * @param index     The index to peek
-     * @param enumClass The class to search
-     * @return From the specified enum class, an enum constant of that class. The enum constant's name will match the
-     * next argument's value
-     * @throws java.util.NoSuchElementException If the constant couldn't be found
-     * @see #peekEnumOrNull(Class)
-     * @see #getEnum(Class)
-     * @see ICommandArgument#getEnum(Class)
-     */
-    <E extends Enum<?>> E peekEnum(Class<E> enumClass, int index) throws CommandInvalidTypeException, CommandNotEnoughArgumentsException;
-
-    /**
-     * @param enumClass The class to search
-     * @return From the specified enum class, an enum constant of that class. The enum constant's name will match the
-     * next argument's value
-     * @throws CommandInvalidTypeException If the constant couldn't be found
-     * @see #peekEnumOrNull(Class)
-     * @see #getEnum(Class)
-     * @see ICommandArgument#getEnum(Class)
-     */
-    <E extends Enum<?>> E peekEnum(Class<E> enumClass) throws CommandInvalidTypeException, CommandNotEnoughArgumentsException;
-
-    /**
-     * @param index     The index to peek
-     * @param enumClass The class to search
-     * @return From the specified enum class, an enum constant of that class. The enum constant's name will match the
-     * next argument's value. If no constant could be found, null
-     * @see #peekEnum(Class)
-     * @see #getEnumOrNull(Class)
-     * @see ICommandArgument#getEnum(Class)
-     */
-    <E extends Enum<?>> E peekEnumOrNull(Class<E> enumClass, int index) throws CommandNotEnoughArgumentsException;
-
-    /**
-     * @param enumClass The class to search
-     * @return From the specified enum class, an enum constant of that class. The enum constant's name will match the
-     * next argument's value. If no constant could be found, null
-     * @see #peekEnum(Class)
-     * @see #getEnumOrNull(Class)
-     * @see ICommandArgument#getEnum(Class)
-     */
-    <E extends Enum<?>> E peekEnumOrNull(Class<E> enumClass) throws CommandNotEnoughArgumentsException;
-
-    /**
-     * Tries to use a <b>stateless</b> {@link IArgParser} to parse the argument at the specified index into the specified
-     * class
-     * <p>
-     * A critical difference between {@link IDatatype}s and {@link IArgParser}s is how many arguments they can take.
-     * While {@link IArgParser}s always operate on a single argument's value, {@link IDatatype}s get access to the entire
-     * {@link IArgConsumer}}.
-     *
-     * @param type  The type to peek as
-     * @param index The index to peek
-     * @return An instance of the specified type
-     * @throws CommandInvalidTypeException If the parsing failed
-     * @see IArgParser
-     * @see #peekAs(Class)
-     * @see #peekAsOrDefault(Class, Object, int)
-     * @see #peekAsOrNull(Class, int)
-     */
-    <T> T peekAs(Class<T> type, int index) throws CommandInvalidTypeException, CommandNotEnoughArgumentsException;
-
-    /**
-     * Tries to use a <b>stateless</b> {@link IArgParser} to parse the next argument into the specified class
-     * <p>
-     * A critical difference between {@link IDatatype}s and {@link IArgParser}s is how many arguments they can take.
-     * While {@link IArgParser}s always operate on a single argument's value, {@link IDatatype}s get access to the entire
-     * {@link IArgConsumer}}.
-     *
-     * @param type The type to peek as
-     * @return An instance of the specified type
-     * @throws CommandInvalidTypeException If the parsing failed
-     * @see IArgParser
-     * @see #peekAs(Class, int)
-     * @see #peekAsOrDefault(Class, Object)
-     * @see #peekAsOrNull(Class)
-     */
-    <T> T peekAs(Class<T> type) throws CommandInvalidTypeException, CommandNotEnoughArgumentsException;
-
-    /**
-     * Tries to use a <b>stateless</b> {@link IArgParser} to parse the argument at the specified index into the specified
-     * class
-     * <p>
-     * A critical difference between {@link IDatatype}s and {@link IArgParser}s is how many arguments they can take.
-     * While {@link IArgParser}s always operate on a single argument's value, {@link IDatatype}s get access to the entire
-     * {@link IArgConsumer}}.
-     *
-     * @param type  The type to peek as
-     * @param def   The value to return if the argument can't be parsed
-     * @param index The index to peek
-     * @return An instance of the specified type, or {@code def} if it couldn't be parsed
-     * @see IArgParser
-     * @see #peekAsOrDefault(Class, Object)
-     * @see #peekAs(Class, int)
-     * @see #peekAsOrNull(Class, int)
-     */
-    <T> T peekAsOrDefault(Class<T> type, T def, int index) throws CommandNotEnoughArgumentsException;
-
-    /**
-     * Tries to use a <b>stateless</b> {@link IArgParser} to parse the next argument into the specified class
-     * <p>
-     * A critical difference between {@link IDatatype}s and {@link IArgParser}s is how many arguments they can take.
-     * While {@link IArgParser}s always operate on a single argument's value, {@link IDatatype}s get access to the entire
-     * {@link IArgConsumer}}.
-     *
-     * @param type The type to peek as
-     * @param def  The value to return if the argument can't be parsed
-     * @return An instance of the specified type, or {@code def} if it couldn't be parsed
-     * @see IArgParser
-     * @see #peekAsOrDefault(Class, Object, int)
-     * @see #peekAs(Class)
-     * @see #peekAsOrNull(Class)
-     */
-    <T> T peekAsOrDefault(Class<T> type, T def) throws CommandNotEnoughArgumentsException;
-
-    /**
-     * Tries to use a <b>stateless</b> {@link IArgParser} to parse the argument at the specified index into the specified
-     * class
-     * <p>
-     * A critical difference between {@link IDatatype}s and {@link IArgParser}s is how many arguments they can take.
-     * While {@link IArgParser}s always operate on a single argument's value, {@link IDatatype}s get access to the entire
-     * {@link IArgConsumer}}.
-     *
-     * @param type  The type to peek as
-     * @param index The index to peek
-     * @return An instance of the specified type, or {@code null} if it couldn't be parsed
-     * @see IArgParser
-     * @see #peekAsOrNull(Class)
-     * @see #peekAs(Class, int)
-     * @see #peekAsOrDefault(Class, Object, int)
-     */
-    <T> T peekAsOrNull(Class<T> type, int index) throws CommandNotEnoughArgumentsException;
-
-    /**
-     * Tries to use a <b>stateless</b> {@link IArgParser} to parse the next argument into the specified class
-     * <p>
-     * A critical difference between {@link IDatatype}s and {@link IArgParser}s is how many arguments they can take.
-     * While {@link IArgParser}s always operate on a single argument's value, {@link IDatatype}s get access to the entire
-     * {@link IArgConsumer}}.
-     *
-     * @param type The type to peek as
-     * @return An instance of the specified type, or {@code null} if it couldn't be parsed
-     * @see IArgParser
-     * @see #peekAsOrNull(Class, int)
-     * @see #peekAs(Class)
-     * @see #peekAsOrDefault(Class, Object)
-     */
-    <T> T peekAsOrNull(Class<T> type) throws CommandNotEnoughArgumentsException;
-
-    <T> T peekDatatype(IDatatypeFor<T> datatype) throws CommandInvalidTypeException, CommandNotEnoughArgumentsException;
-
-    <T, O> T peekDatatype(IDatatypePost<T, O> datatype) throws CommandInvalidTypeException, CommandNotEnoughArgumentsException;
-
-    <T, O> T peekDatatype(IDatatypePost<T, O> datatype, O original) throws CommandInvalidTypeException, CommandNotEnoughArgumentsException;
-
-    <T> T peekDatatypeOrNull(IDatatypeFor<T> datatype);
-
-    <T, O> T peekDatatypeOrNull(IDatatypePost<T, O> datatype);
-
-    <T, O, D extends IDatatypePost<T, O>> T peekDatatypePost(D datatype, O original) throws CommandInvalidTypeException, CommandNotEnoughArgumentsException;
-
-    <T, O, D extends IDatatypePost<T, O>> T peekDatatypePostOrDefault(D datatype, O original, T def);
-
-    <T, O, D extends IDatatypePost<T, O>> T peekDatatypePostOrNull(D datatype, O original);
-
-    /**
-     * Attempts to get the specified {@link IDatatypeFor} from this ArgConsumer
-     * <p>
-     * A critical difference between {@link IDatatype}s and {@link IArgParser}s is how many arguments they can take.
-     * While {@link IArgParser}s always operate on a single argument's value, {@link IDatatype}s get access to the entire
-     * {@link IArgConsumer}}.
-     * <p>
-     * Since this is a peek operation, this ArgConsumer will not be mutated by any call to this method.
-     *
-     * @param datatype The datatype to get
-     * @return The datatype instance
-     * @see IDatatype
-     * @see IDatatypeFor
-     */
-    <T, D extends IDatatypeFor<T>> T peekDatatypeFor(Class<D> datatype);
-
-    /**
-     * Attempts to get the specified {@link IDatatypeFor} from this ArgConsumer
-     * <p>
-     * A critical difference between {@link IDatatype}s and {@link IArgParser}s is how many arguments they can take.
-     * While {@link IArgParser}s always operate on a single argument's value, {@link IDatatype}s get access to the entire
-     * {@link IArgConsumer}}.
-     * <p>
-     * Since this is a peek operation, this ArgConsumer will not be mutated by any call to this method.
-     *
-     * @param datatype The datatype to get
-     * @param def      The default value
-     * @return The datatype instance, or {@code def} if it throws an exception
-     * @see IDatatype
-     * @see IDatatypeFor
-     */
-    <T, D extends IDatatypeFor<T>> T peekDatatypeForOrDefault(Class<D> datatype, T def);
-
-    /**
-     * Attempts to get the specified {@link IDatatypeFor} from this ArgConsumer
-     * <p>
-     * A critical difference between {@link IDatatype}s and {@link IArgParser}s is how many arguments they can take.
-     * While {@link IArgParser}s always operate on a single argument's value, {@link IDatatype}s get access to the entire
-     * {@link IArgConsumer}}.
-     * <p>
-     * Since this is a peek operation, this ArgConsumer will not be mutated by any call to this method.
-     *
-     * @param datatype The datatype to get
-     * @return The datatype instance, or {@code null} if it throws an exception
-     * @see IDatatype
-     * @see IDatatypeFor
-     */
-    <T, D extends IDatatypeFor<T>> T peekDatatypeForOrNull(Class<D> datatype);
-
-    /**
-     * Gets the next argument and returns it. This consumes the first argument so that subsequent calls will return
-     * later arguments
-     *
-     * @return The next argument
-     * @throws CommandNotEnoughArgumentsException If there's less than one argument left
-     */
-    ICommandArgument get() throws CommandNotEnoughArgumentsException;
-
-    /**
-     * Gets the value of the next argument and returns it. This consumes the first argument so that subsequent calls
-     * will return later arguments
-     *
-     * @return The value of the next argument
-     * @throws CommandNotEnoughArgumentsException If there's less than one argument left
-     */
-    String getString() throws CommandNotEnoughArgumentsException;
-
-    /**
-     * Gets an enum value from the enum class with the same name as the next argument's value
-     * <p>
-     * For example if you getEnum as an {@link EnumFacing}, and the next argument's value is "up", this will return
-     * {@link EnumFacing#UP}
-     *
-     * @param enumClass The enum class to search
-     * @return An enum constant of that class with the same name as the next argument's value
-     * @throws CommandInvalidTypeException If the constant couldn't be found
-     * @see #peekEnum(Class)
-     * @see #getEnumOrNull(Class)
-     * @see ICommandArgument#getEnum(Class)
-     */
-    <E extends Enum<?>> E getEnum(Class<E> enumClass) throws CommandInvalidTypeException, CommandNotEnoughArgumentsException;
-
-    /**
-     * Gets an enum value from the enum class with the same name as the next argument's value
-     * <p>
-     * For example if you getEnum as an {@link EnumFacing}, and the next argument's value is "up", this will return
-     * {@link EnumFacing#UP}
-     *
-     * @param enumClass The enum class to search
-     * @param def       The default value
-     * @return An enum constant of that class with the same name as the next argument's value, or {@code def} if it
-     * couldn't be found
-     * @see #getEnum(Class)
-     * @see #getEnumOrNull(Class)
-     * @see #peekEnumOrNull(Class)
-     * @see ICommandArgument#getEnum(Class)
-     */
-    <E extends Enum<?>> E getEnumOrDefault(Class<E> enumClass, E def) throws CommandNotEnoughArgumentsException;
-
-    /**
-     * Gets an enum value from the enum class with the same name as the next argument's value
-     * <p>
-     * For example if you getEnum as an {@link EnumFacing}, and the next argument's value is "up", this will return
-     * {@link EnumFacing#UP}
-     *
-     * @param enumClass The enum class to search
-     * @return An enum constant of that class with the same name as the next argument's value, or {@code null} if it
-     * couldn't be found
-     * @see #getEnum(Class)
-     * @see #getEnumOrDefault(Class, Enum)
-     * @see #peekEnumOrNull(Class)
-     * @see ICommandArgument#getEnum(Class)
-     */
-    <E extends Enum<?>> E getEnumOrNull(Class<E> enumClass) throws CommandNotEnoughArgumentsException;
-
-    /**
-     * Tries to use a <b>stateless</b> {@link IArgParser} to parse the next argument into the specified class
-     * <p>
-     * A critical difference between {@link IDatatype}s and {@link IArgParser}s is how many arguments they can take.
-     * While {@link IArgParser}s always operate on a single argument's value, {@link IDatatype}s get access to the entire
-     * {@link IArgConsumer}}.
-     *
-     * @param type The type to peek as
-     * @return An instance of the specified type
-     * @throws CommandInvalidTypeException If the parsing failed
-     * @see IArgParser
-     * @see #get()
-     * @see #getAsOrDefault(Class, Object)
-     * @see #getAsOrNull(Class)
-     * @see #peekAs(Class)
-     * @see #peekAsOrDefault(Class, Object, int)
-     * @see #peekAsOrNull(Class, int)
-     */
-    <T> T getAs(Class<T> type) throws CommandInvalidTypeException, CommandNotEnoughArgumentsException;
-
-    /**
-     * Tries to use a <b>stateless</b> {@link IArgParser} to parse the next argument into the specified class
-     * <p>
-     * A critical difference between {@link IDatatype}s and {@link IArgParser}s is how many arguments they can take.
-     * While {@link IArgParser}s always operate on a single argument's value, {@link IDatatype}s get access to the entire
-     * {@link IArgConsumer}}.
-     *
-     * @param type The type to peek as
-     * @param def  The default value
-     * @return An instance of the specified type, or {@code def} if it couldn't be parsed
-     * @see IArgParser
-     * @see #get()
-     * @see #getAs(Class)
-     * @see #getAsOrNull(Class)
-     * @see #peekAs(Class)
-     * @see #peekAsOrDefault(Class, Object, int)
-     * @see #peekAsOrNull(Class, int)
-     */
-    <T> T getAsOrDefault(Class<T> type, T def) throws CommandNotEnoughArgumentsException;
-
-    /**
-     * Tries to use a <b>stateless</b> {@link IArgParser} to parse the next argument into the specified class
-     * <p>
-     * A critical difference between {@link IDatatype}s and {@link IArgParser}s is how many arguments they can take.
-     * While {@link IArgParser}s always operate on a single argument's value, {@link IDatatype}s get access to the entire
-     * {@link IArgConsumer}}.
-     *
-     * @param type The type to peek as
-     * @return An instance of the specified type, or {@code null} if it couldn't be parsed
-     * @see IArgParser
-     * @see #get()
-     * @see #getAs(Class)
-     * @see #getAsOrDefault(Class, Object)
-     * @see #peekAs(Class)
-     * @see #peekAsOrDefault(Class, Object, int)
-     * @see #peekAsOrNull(Class, int)
-     */
-    <T> T getAsOrNull(Class<T> type) throws CommandNotEnoughArgumentsException;
-
-    <T, O, D extends IDatatypePost<T, O>> T getDatatypePost(D datatype, O original) throws CommandInvalidTypeException, CommandNotEnoughArgumentsException;
-
-    <T, O, D extends IDatatypePost<T, O>> T getDatatypePostOrDefault(D datatype, O original, T _default);
-
-    <T, O, D extends IDatatypePost<T, O>> T getDatatypePostOrNull(D datatype, O original);
-
-    <T, D extends IDatatypeFor<T>> T getDatatypeFor(D datatype) throws CommandInvalidTypeException, CommandNotEnoughArgumentsException;
-
-    <T, D extends IDatatypeFor<T>> T getDatatypeForOrDefault(D datatype, T def);
-
-    <T, D extends IDatatypeFor<T>> T getDatatypeForOrNull(D datatype);
-
-    <T extends IDatatype> Stream<String> tabCompleteDatatype(T datatype);
-
-    /**
-     * Returns the "raw rest" of the string. For example, from a string <code>arg1 arg2&nbsp;&nbsp;arg3</code>, split
-     * into three {@link ICommandArgument}s {@code "arg1"}, {@code "arg2"}, and {@code "arg3"}:
-     *
-     * <ul>
-     * <li>{@code rawRest()} would return <code>arg1 arg2&nbsp;&nbsp;arg3</code></li>
-     * <li>After calling {@link #get()}, {@code rawRest()} would return <code>arg2&nbsp;&nbsp;arg3</code> (note the
-     * double space - it is preserved!)</li>
-     * <li>After calling {@link #get()} again, {@code rawRest()} would return {@code "arg3"}</li>
-     * <li>After calling {@link #get()} one last time, {@code rawRest()} would return {@code ""}</li>
-     * </ul>
-     *
-     * @return The "raw rest" of the string.
-     */
-    String rawRest();
-
-    /**
-     * @param min The minimum amount of arguments to require.
-     * @throws CommandNotEnoughArgumentsException If there are less than {@code min} arguments left.
-     * @see #requireMax(int)
-     * @see #requireExactly(int)
-     */
-    void requireMin(int min) throws CommandNotEnoughArgumentsException;
-
-    /**
-     * @param max The maximum amount of arguments allowed.
-     * @throws CommandTooManyArgumentsException If there are more than {@code max} arguments left.
-     * @see #requireMin(int)
-     * @see #requireExactly(int)
-     */
-    void requireMax(int max) throws CommandTooManyArgumentsException;
-
-    /**
-     * @param args The exact amount of arguments to require.
-     * @throws CommandNotEnoughArgumentsException If there are less than {@code args} arguments left.
-     * @throws CommandTooManyArgumentsException   If there are more than {@code args} arguments left.
-     * @see #requireMin(int)
-     * @see #requireMax(int)
-     */
-    void requireExactly(int args) throws CommandException;
-
-    /**
-     * @return If this {@link IArgConsumer}} has consumed at least one argument.
-     * @see #consumed()
-     * @see #consumedString()
-     */
-    boolean hasConsumed();
-
-    /**
-     * @return The last argument this {@link IArgConsumer}} has consumed, or an "unknown" argument, indicated by a
-     * comamnd argument index that has a value of {@code -1}, if no arguments have been consumed yet.
-     * @see #consumedString()
-     * @see #hasConsumed()
-     */
-    ICommandArgument consumed();
-
-    /**
-     * @return The value of thelast argument this {@link IArgConsumer}} has consumed, or an empty string if no arguments
-     * have been consumed yet
-     * @see #consumed()
-     * @see #hasConsumed()
-     */
-    String consumedString();
-
-    /**
-     * @return A copy of this {@link IArgConsumer}}. It has the same arguments (both consumed and not), but does not
-     * affect or mutate this instance. Useful for the various {@code peek} functions
-     */
-    IArgConsumer copy();
-}

src/api/java/baritone/api/command/argument/ICommandArgument.java

@@ -1,101 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.argument;
-
-import baritone.api.command.argparser.IArgParser;
-import baritone.api.command.exception.CommandInvalidTypeException;
-import net.minecraft.util.EnumFacing;
-
-/**
- * A {@link ICommandArgument} is an immutable object representing one command argument. It contains data on the index of
- * that argument, its value, and the rest of the string that argument was found in
- * <p>
- * You're recommended to use {@link IArgConsumer}}s to handle these.
- *
- * @author Brady
- * @since 10/2/2019
- */
-public interface ICommandArgument {
-
-    /**
-     * @return The index of this command argument in the list of command arguments generated
-     */
-    int getIndex();
-
-    /**
-     * @return The raw value of just this argument
-     */
-    String getValue();
-
-    /**
-     * @return The raw value of the remaining arguments after this one was captured
-     */
-    String getRawRest();
-
-    /**
-     * Gets an enum value from the enum class with the same name as this argument's value
-     * <p>
-     * For example if you getEnum as an {@link EnumFacing}, and this argument's value is "up", it will return {@link
-     * EnumFacing#UP}
-     *
-     * @param enumClass The enum class to search
-     * @return An enum constant of that class with the same name as this argument's value
-     * @throws CommandInvalidTypeException If the constant couldn't be found
-     * @see IArgConsumer#peekEnum(Class)
-     * @see IArgConsumer#peekEnum(Class, int)
-     * @see IArgConsumer#peekEnumOrNull(Class)
-     * @see IArgConsumer#peekEnumOrNull(Class, int)
-     * @see IArgConsumer#getEnum(Class)
-     * @see IArgConsumer#getEnumOrNull(Class)
-     */
-    <E extends Enum<?>> E getEnum(Class<E> enumClass) throws CommandInvalidTypeException;
-
-    /**
-     * Tries to use a <b>stateless</b> {@link IArgParser} to parse this argument into the specified class
-     *
-     * @param type The class to parse this argument into
-     * @return An instance of the specified type
-     * @throws CommandInvalidTypeException If the parsing failed
-     */
-    <T> T getAs(Class<T> type) throws CommandInvalidTypeException;
-
-    /**
-     * Tries to use a <b>stateless</b> {@link IArgParser} to parse this argument into the specified class
-     *
-     * @param type The class to parse this argument into
-     * @return If the parser succeeded
-     */
-    <T> boolean is(Class<T> type);
-
-    /**
-     * Tries to use a <b>stated</b> {@link IArgParser} to parse this argument into the specified class
-     *
-     * @param type The class to parse this argument into
-     * @return An instance of the specified type
-     * @throws CommandInvalidTypeException If the parsing failed
-     */
-    <T, S> T getAs(Class<T> type, Class<S> stateType, S state) throws CommandInvalidTypeException;
-
-    /**
-     * Tries to use a <b>stated</b> {@link IArgParser} to parse this argument into the specified class
-     *
-     * @param type The class to parse this argument into
-     * @return If the parser succeeded
-     */
-    <T, S> boolean is(Class<T> type, Class<S> stateType, S state);
-}

src/api/java/baritone/api/command/datatypes/BlockById.java

@@ -1,53 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.datatypes;
-
-import baritone.api.command.exception.CommandException;
-import baritone.api.command.helpers.TabCompleteHelper;
-import net.minecraft.block.Block;
-import net.minecraft.init.Blocks;
-import net.minecraft.util.ResourceLocation;
-
-import java.util.stream.Stream;
-
-public enum BlockById implements IDatatypeFor<Block> {
-    INSTANCE;
-
-    @Override
-    public Block get(IDatatypeContext ctx) throws CommandException {
-        ResourceLocation id = new ResourceLocation(ctx.getConsumer().getString());
-        Block block;
-        if ((block = Block.REGISTRY.getObject(id)) == Blocks.AIR) {
-            throw new IllegalArgumentException("no block found by that id");
-        }
-        return block;
-    }
-
-    @Override
-    public Stream<String> tabComplete(IDatatypeContext ctx) throws CommandException {
-        return new TabCompleteHelper()
-                .append(
-                        Block.REGISTRY.getKeys()
-                                .stream()
-                                .map(Object::toString)
-                )
-                .filterPrefixNamespaced(ctx.getConsumer().getString())
-                .sortAlphabetically()
-                .stream();
-    }
-}

src/api/java/baritone/api/command/datatypes/EntityClassById.java

@@ -1,61 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.datatypes;
-
-import baritone.api.command.exception.CommandException;
-import baritone.api.command.helpers.TabCompleteHelper;
-import net.minecraft.entity.Entity;
-import net.minecraft.entity.EntityList;
-import net.minecraft.util.ResourceLocation;
-
-import java.util.stream.Stream;
-
-public enum EntityClassById implements IDatatypeFor<Class<? extends Entity>> {
-    INSTANCE;
-
-    @Override
-    public Class<? extends Entity> get(IDatatypeContext ctx) throws CommandException {
-        ResourceLocation id = new ResourceLocation(ctx.getConsumer().getString());
-        Class<? extends Entity> entity;
-        try {
-            entity = EntityList.REGISTRY.getObject(id);
-        } catch (NoSuchFieldError e) {
-            // Forge removes EntityList.REGISTRY field and provides the getClass method as a replacement
-            // See https://github.com/MinecraftForge/MinecraftForge/blob/1.12.x/patches/minecraft/net/minecraft/entity/EntityList.java.patch
-            try {
-                entity = (Class<? extends Entity>) EntityList.class.getMethod("getClass", ResourceLocation.class).invoke(null, id);
-            } catch (Exception ex) {
-                throw new RuntimeException("EntityList.REGISTRY does not exist and failed to call the Forge-replacement method", ex);
-            }
-        }
-
-        if (entity == null) {
-            throw new IllegalArgumentException("no entity found by that id");
-        }
-        return entity;
-    }
-
-    @Override
-    public Stream<String> tabComplete(IDatatypeContext ctx) throws CommandException {
-        return new TabCompleteHelper()
-                .append(EntityList.getEntityNameList().stream().map(Object::toString))
-                .filterPrefixNamespaced(ctx.getConsumer().getString())
-                .sortAlphabetically()
-                .stream();
-    }
-}

src/api/java/baritone/api/command/datatypes/ForBlockOptionalMeta.java

@@ -1,37 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.datatypes;
-
-import baritone.api.command.exception.CommandException;
-import baritone.api.utils.BlockOptionalMeta;
-
-import java.util.stream.Stream;
-
-public enum ForBlockOptionalMeta implements IDatatypeFor<BlockOptionalMeta> {
-    INSTANCE;
-
-    @Override
-    public BlockOptionalMeta get(IDatatypeContext ctx) throws CommandException {
-        return new BlockOptionalMeta(ctx.getConsumer().getString());
-    }
-
-    @Override
-    public Stream<String> tabComplete(IDatatypeContext ctx) {
-        return ctx.getConsumer().tabCompleteDatatype(BlockById.INSTANCE);
-    }
-}

src/api/java/baritone/api/command/datatypes/ForEnumFacing.java

@@ -1,43 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.datatypes;
-
-import baritone.api.command.exception.CommandException;
-import baritone.api.command.helpers.TabCompleteHelper;
-import net.minecraft.util.EnumFacing;
-
-import java.util.Locale;
-import java.util.stream.Stream;
-
-public enum ForEnumFacing implements IDatatypeFor<EnumFacing> {
-    INSTANCE;
-
-    @Override
-    public EnumFacing get(IDatatypeContext ctx) throws CommandException {
-        return EnumFacing.valueOf(ctx.getConsumer().getString().toUpperCase(Locale.US));
-    }
-
-    @Override
-    public Stream<String> tabComplete(IDatatypeContext ctx) throws CommandException {
-        return new TabCompleteHelper()
-                .append(Stream.of(EnumFacing.values())
-                        .map(EnumFacing::getName).map(String::toLowerCase))
-                .filterPrefix(ctx.getConsumer().getString())
-                .stream();
-    }
-}

src/api/java/baritone/api/command/datatypes/ForWaypoints.java

@@ -1,81 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.datatypes;
-
-import baritone.api.IBaritone;
-import baritone.api.cache.IWaypoint;
-import baritone.api.cache.IWaypointCollection;
-import baritone.api.command.exception.CommandException;
-import baritone.api.command.helpers.TabCompleteHelper;
-
-import java.util.Comparator;
-import java.util.stream.Stream;
-
-public enum ForWaypoints implements IDatatypeFor<IWaypoint[]> {
-    INSTANCE;
-
-    @Override
-    public IWaypoint[] get(IDatatypeContext ctx) throws CommandException {
-        final String input = ctx.getConsumer().getString();
-        final IWaypoint.Tag tag = IWaypoint.Tag.getByName(input);
-
-        // If the input doesn't resolve to a valid tag, resolve by name
-        return tag == null
-                ? getWaypointsByName(ctx.getBaritone(), input)
-                : getWaypointsByTag(ctx.getBaritone(), tag);
-    }
-
-    @Override
-    public Stream<String> tabComplete(IDatatypeContext ctx) throws CommandException {
-        return new TabCompleteHelper()
-                .append(getWaypointNames(ctx.getBaritone()))
-                .sortAlphabetically()
-                .prepend(IWaypoint.Tag.getAllNames())
-                .filterPrefix(ctx.getConsumer().getString())
-                .stream();
-    }
-
-    public static IWaypointCollection waypoints(IBaritone baritone) {
-        return baritone.getWorldProvider().getCurrentWorld().getWaypoints();
-    }
-
-    public static IWaypoint[] getWaypoints(IBaritone baritone) {
-        return waypoints(baritone).getAllWaypoints().stream()
-                .sorted(Comparator.comparingLong(IWaypoint::getCreationTimestamp).reversed())
-                .toArray(IWaypoint[]::new);
-    }
-
-    public static String[] getWaypointNames(IBaritone baritone) {
-        return Stream.of(getWaypoints(baritone))
-                .map(IWaypoint::getName)
-                .filter(name -> !name.isEmpty())
-                .toArray(String[]::new);
-    }
-
-    public static IWaypoint[] getWaypointsByTag(IBaritone baritone, IWaypoint.Tag tag) {
-        return waypoints(baritone).getByTag(tag).stream()
-                .sorted(Comparator.comparingLong(IWaypoint::getCreationTimestamp).reversed())
-                .toArray(IWaypoint[]::new);
-    }
-
-    public static IWaypoint[] getWaypointsByName(IBaritone baritone, String name) {
-        return Stream.of(getWaypoints(baritone))
-                .filter(waypoint -> waypoint.getName().equalsIgnoreCase(name))
-                .toArray(IWaypoint[]::new);
-    }
-}

src/api/java/baritone/api/command/datatypes/IDatatype.java

@@ -1,56 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.datatypes;
-
-import baritone.api.command.argparser.IArgParser;
-import baritone.api.command.argument.IArgConsumer;
-import baritone.api.command.exception.CommandException;
-
-import java.util.stream.Stream;
-
-/**
- * An {@link IDatatype} is similar to an {@link IArgParser} in the sense that it is capable of consuming an argument
- * to transform it into a usable form as the code desires.
- * <p>
- * A fundamental difference is that an {@link IDatatype} is capable of consuming multiple arguments. For example,
- * {@link RelativeBlockPos} is an {@link IDatatypePost} which requires at least 3 {@link RelativeCoordinate} arguments
- * to be specified.
- * <p>
- * Another difference is that an {@link IDatatype} can be tab-completed, providing comprehensive auto completion
- * that can substitute based on existing input or provide possibilities for the next piece of input.
- *
- * @see IDatatypeContext
- * @see IDatatypeFor
- * @see IDatatypePost
- */
-public interface IDatatype {
-
-    /**
-     * Attempts to complete missing or partial input provided through the {@link IArgConsumer}} provided by
-     * {@link IDatatypeContext#getConsumer()} in order to aide the user in executing commands.
-     * <p>
-     * One benefit over datatypes over {@link IArgParser}s is that instead of each command trying to guess what values
-     * the datatype will accept, or simply not tab completing at all, datatypes that support tab completion can provide
-     * accurate information using the same methods used to parse arguments in the first place.
-     *
-     * @param ctx The argument consumer to tab complete
-     * @return A stream representing the strings that can be tab completed. DO NOT INCLUDE SPACES IN ANY STRINGS.
-     * @see IArgConsumer#tabCompleteDatatype(IDatatype)
-     */
-    Stream<String> tabComplete(IDatatypeContext ctx) throws CommandException;
-}

src/api/java/baritone/api/command/datatypes/IDatatypeContext.java

@@ -1,46 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.datatypes;
-
-import baritone.api.IBaritone;
-import baritone.api.command.argument.IArgConsumer;
-
-/**
- * Provides an {@link IDatatype} with contextual information so
- * that it can perform the desired operation on the target level.
- *
- * @author Brady
- * @see IDatatype
- * @since 9/26/2019
- */
-public interface IDatatypeContext {
-
-    /**
-     * Provides the {@link IBaritone} instance that is associated with the action relating to datatype handling.
-     *
-     * @return The context {@link IBaritone} instance.
-     */
-    IBaritone getBaritone();
-
-    /**
-     * Provides the {@link IArgConsumer}} to fetch input information from.
-     *
-     * @return The context {@link IArgConsumer}}.
-     */
-    IArgConsumer getConsumer();
-}

src/api/java/baritone/api/command/datatypes/IDatatypeFor.java

@@ -1,43 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.datatypes;
-
-import baritone.api.command.exception.CommandException;
-
-import java.util.function.Supplier;
-
-/**
- * An {@link IDatatype} which acts as a {@link Supplier}, in essence. The only difference
- * is that it requires an {@link IDatatypeContext} to be provided due to the expectation that
- * implementations of {@link IDatatype} are singletons.
- */
-public interface IDatatypeFor<T> extends IDatatype {
-
-    /**
-     * Consumes the desired amount of arguments from the specified {@link IDatatypeContext}, and
-     * then returns the parsed value. This method will more than likely return a {@link IllegalArgumentException}
-     * if the expected input does not conform to a parseable value. As far as a {@link CommandException} being
-     * thrown is concerned, see the note below for specifics.
-     *
-     * @param ctx The context
-     * @return The parsed data-type
-     * @throws CommandException If there was an issue parsing using another type or arguments could not be polled.
-     * @see IDatatypeContext
-     */
-    T get(IDatatypeContext ctx) throws CommandException;
-}

src/api/java/baritone/api/command/datatypes/IDatatypePost.java

@@ -1,41 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.datatypes;
-
-import baritone.api.command.exception.CommandException;
-
-import java.util.function.Function;
-
-/**
- * An {@link IDatatype} which acts as a {@link Function}, in essence. The only difference
- * is that it requires an {@link IDatatypeContext} to be provided due to the expectation that
- * implementations of {@link IDatatype} are singletons.
- */
-public interface IDatatypePost<T, O> extends IDatatype {
-
-    /**
-     * Takes the expected input and transforms it based on the value held by {@code original}. If {@code original}
-     * is null, it is expected that the implementation of this method has a case to handle it, such that a
-     * {@link NullPointerException} will never be thrown as a result.
-     *
-     * @param ctx      The datatype context
-     * @param original The transformable value
-     * @return The transformed value
-     */
-    T apply(IDatatypeContext ctx, O original) throws CommandException;
-}

src/api/java/baritone/api/command/datatypes/IDatatypePostFunction.java

@@ -1,29 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.datatypes;
-
-import baritone.api.command.exception.CommandException;
-
-/**
- * @author Brady
- * @since 9/26/2019
- */
-public interface IDatatypePostFunction<T, O> {
-
-    T apply(O original) throws CommandException;
-}

src/api/java/baritone/api/command/datatypes/NearbyPlayer.java

@@ -1,55 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.datatypes;
-
-import baritone.api.IBaritone;
-import baritone.api.command.exception.CommandException;
-import baritone.api.command.helpers.TabCompleteHelper;
-import net.minecraft.entity.player.EntityPlayer;
-
-import java.util.List;
-import java.util.stream.Stream;
-
-/**
- * An {@link IDatatype} used to resolve nearby players, those within
- * render distance of the target {@link IBaritone} instance.
- */
-public enum NearbyPlayer implements IDatatypeFor<EntityPlayer> {
-    INSTANCE;
-
-    @Override
-    public EntityPlayer get(IDatatypeContext ctx) throws CommandException {
-        final String username = ctx.getConsumer().getString();
-        return getPlayers(ctx).stream()
-                .filter(s -> s.getName().equalsIgnoreCase(username))
-                .findFirst().orElse(null);
-    }
-
-    @Override
-    public Stream<String> tabComplete(IDatatypeContext ctx) throws CommandException {
-        return new TabCompleteHelper()
-                .append(getPlayers(ctx).stream().map(EntityPlayer::getName))
-                .filterPrefix(ctx.getConsumer().getString())
-                .sortAlphabetically()
-                .stream();
-    }
-
-    private static List<EntityPlayer> getPlayers(IDatatypeContext ctx) {
-        return ctx.getBaritone().getPlayerContext().world().playerEntities;
-    }
-}

src/api/java/baritone/api/command/datatypes/RelativeBlockPos.java

@@ -1,57 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.datatypes;
-
-import baritone.api.command.argument.IArgConsumer;
-import baritone.api.command.exception.CommandException;
-import baritone.api.utils.BetterBlockPos;
-
-import java.util.stream.Stream;
-
-public enum RelativeBlockPos implements IDatatypePost<BetterBlockPos, BetterBlockPos> {
-    INSTANCE;
-
-    @Override
-    public BetterBlockPos apply(IDatatypeContext ctx, BetterBlockPos origin) throws CommandException {
-        if (origin == null) {
-            origin = BetterBlockPos.ORIGIN;
-        }
-
-        final IArgConsumer consumer = ctx.getConsumer();
-        return new BetterBlockPos(
-                consumer.getDatatypePost(RelativeCoordinate.INSTANCE, (double) origin.x),
-                consumer.getDatatypePost(RelativeCoordinate.INSTANCE, (double) origin.y),
-                consumer.getDatatypePost(RelativeCoordinate.INSTANCE, (double) origin.z)
-        );
-    }
-
-    @Override
-    public Stream<String> tabComplete(IDatatypeContext ctx) throws CommandException {
-        final IArgConsumer consumer = ctx.getConsumer();
-        if (consumer.hasAny() && !consumer.has(4)) {
-            while (consumer.has(2)) {
-                if (consumer.peekDatatypeOrNull(RelativeCoordinate.INSTANCE) == null) {
-                    break;
-                }
-                consumer.get();
-            }
-            return consumer.tabCompleteDatatype(RelativeCoordinate.INSTANCE);
-        }
-        return Stream.empty();
-    }
-}

src/api/java/baritone/api/command/datatypes/RelativeCoordinate.java

@@ -1,64 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.datatypes;
-
-import baritone.api.command.argument.IArgConsumer;
-import baritone.api.command.exception.CommandException;
-
-import java.util.regex.Matcher;
-import java.util.regex.Pattern;
-import java.util.stream.Stream;
-
-public enum RelativeCoordinate implements IDatatypePost<Double, Double> {
-    INSTANCE;
-    private static Pattern PATTERN = Pattern.compile("^(~?)([+-]?(?:\\d+(?:\\.\\d*)?|\\.\\d+)([k-k]?)|)$");
-
-    @Override
-    public Double apply(IDatatypeContext ctx, Double origin) throws CommandException {
-        if (origin == null) {
-            origin = 0.0D;
-        }
-
-        Matcher matcher = PATTERN.matcher(ctx.getConsumer().getString());
-        if (!matcher.matches()) {
-            throw new IllegalArgumentException("pattern doesn't match");
-        }
-
-        boolean isRelative = !matcher.group(1).isEmpty();
-
-        double offset = matcher.group(2).isEmpty() ? 0 : Double.parseDouble(matcher.group(2).replaceAll("k", ""));
-
-        if (matcher.group(2).contains("k")) {
-            offset *= 1000;
-        }
-
-        if (isRelative) {
-            return origin + offset;
-        }
-        return offset;
-    }
-
-    @Override
-    public Stream<String> tabComplete(IDatatypeContext ctx) throws CommandException {
-        final IArgConsumer consumer = ctx.getConsumer();
-        if (!consumer.has(2) && consumer.getString().matches("^(~|$)")) {
-            return Stream.of("~");
-        }
-        return Stream.empty();
-    }
-}

src/api/java/baritone/api/command/datatypes/RelativeFile.java

@@ -1,103 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.datatypes;
-
-import baritone.api.command.argument.IArgConsumer;
-import baritone.api.command.exception.CommandException;
-
-import java.io.File;
-import java.io.IOException;
-import java.io.UncheckedIOException;
-import java.nio.file.FileSystems;
-import java.nio.file.InvalidPathException;
-import java.nio.file.Path;
-import java.util.Locale;
-import java.util.Objects;
-import java.util.stream.Stream;
-
-import static baritone.api.utils.Helper.HELPER;
-
-public enum RelativeFile implements IDatatypePost<File, File> {
-    INSTANCE;
-
-    @Override
-    public File apply(IDatatypeContext ctx, File original) throws CommandException {
-        if (original == null) {
-            original = new File("./");
-        }
-
-        Path path;
-        try {
-            path = FileSystems.getDefault().getPath(ctx.getConsumer().getString());
-        } catch (InvalidPathException e) {
-            throw new IllegalArgumentException("invalid path");
-        }
-        return getCanonicalFileUnchecked(original.toPath().resolve(path).toFile());
-    }
-
-    @Override
-    public Stream<String> tabComplete(IDatatypeContext ctx) {
-        return Stream.empty();
-    }
-
-    /**
-     * Seriously
-     *
-     * @param file File
-     * @return Canonical file of file
-     * @author LoganDark
-     */
-    private static File getCanonicalFileUnchecked(File file) {
-        try {
-            return file.getCanonicalFile();
-        } catch (IOException e) {
-            throw new UncheckedIOException(e);
-        }
-    }
-
-    public static Stream<String> tabComplete(IArgConsumer consumer, File base0) throws CommandException {
-        // I will not make the caller deal with this, seriously
-        // Tab complete code is beautiful and I'm not going to bloat it with dumb ass checked exception bullshit -LoganDark
-
-        // lol owned -Brady
-
-        File base = getCanonicalFileUnchecked(base0);
-        String currentPathStringThing = consumer.getString();
-        Path currentPath = FileSystems.getDefault().getPath(currentPathStringThing);
-        Path basePath = currentPath.isAbsolute() ? currentPath.getRoot() : base.toPath();
-        boolean useParent = !currentPathStringThing.isEmpty() && !currentPathStringThing.endsWith(File.separator);
-        File currentFile = currentPath.isAbsolute() ? currentPath.toFile() : new File(base, currentPathStringThing);
-        return Stream.of(Objects.requireNonNull(getCanonicalFileUnchecked(
-                useParent
-                        ? currentFile.getParentFile()
-                        : currentFile
-        ).listFiles()))
-                .map(f -> (currentPath.isAbsolute() ? f : basePath.relativize(f.toPath()).toString()) +
-                        (f.isDirectory() ? File.separator : ""))
-                .filter(s -> s.toLowerCase(Locale.US).startsWith(currentPathStringThing.toLowerCase(Locale.US)))
-                .filter(s -> !s.contains(" "));
-    }
-
-    public static File gameDir() {
-        File gameDir = HELPER.mc.gameDir.getAbsoluteFile();
-        if (gameDir.getName().equals(".")) {
-            return gameDir.getParentFile();
-        }
-        return gameDir;
-    }
-}

src/api/java/baritone/api/command/datatypes/RelativeGoal.java

@@ -1,64 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.datatypes;
-
-import baritone.api.command.argument.IArgConsumer;
-import baritone.api.command.exception.CommandException;
-import baritone.api.pathing.goals.Goal;
-import baritone.api.pathing.goals.GoalBlock;
-import baritone.api.pathing.goals.GoalXZ;
-import baritone.api.pathing.goals.GoalYLevel;
-import baritone.api.utils.BetterBlockPos;
-
-import java.util.stream.Stream;
-
-public enum RelativeGoal implements IDatatypePost<Goal, BetterBlockPos> {
-    INSTANCE;
-
-    @Override
-    public Goal apply(IDatatypeContext ctx, BetterBlockPos origin) throws CommandException {
-        if (origin == null) {
-            origin = BetterBlockPos.ORIGIN;
-        }
-
-        final IArgConsumer consumer = ctx.getConsumer();
-
-        GoalBlock goalBlock = consumer.peekDatatypePostOrNull(RelativeGoalBlock.INSTANCE, origin);
-        if (goalBlock != null) {
-            return goalBlock;
-        }
-
-        GoalXZ goalXZ = consumer.peekDatatypePostOrNull(RelativeGoalXZ.INSTANCE, origin);
-        if (goalXZ != null) {
-            return goalXZ;
-        }
-
-        GoalYLevel goalYLevel = consumer.peekDatatypePostOrNull(RelativeGoalYLevel.INSTANCE, origin);
-        if (goalYLevel != null) {
-            return goalYLevel;
-        }
-
-        // when the user doesn't input anything, default to the origin
-        return new GoalBlock(origin);
-    }
-
-    @Override
-    public Stream<String> tabComplete(IDatatypeContext ctx) {
-        return ctx.getConsumer().tabCompleteDatatype(RelativeCoordinate.INSTANCE);
-    }
-}

src/api/java/baritone/api/command/datatypes/RelativeGoalBlock.java

@@ -1,53 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.datatypes;
-
-import baritone.api.command.argument.IArgConsumer;
-import baritone.api.command.exception.CommandException;
-import baritone.api.pathing.goals.GoalBlock;
-import baritone.api.utils.BetterBlockPos;
-import net.minecraft.util.math.MathHelper;
-
-import java.util.stream.Stream;
-
-public enum RelativeGoalBlock implements IDatatypePost<GoalBlock, BetterBlockPos> {
-    INSTANCE;
-
-    @Override
-    public GoalBlock apply(IDatatypeContext ctx, BetterBlockPos origin) throws CommandException {
-        if (origin == null) {
-            origin = BetterBlockPos.ORIGIN;
-        }
-
-        final IArgConsumer consumer = ctx.getConsumer();
-        return new GoalBlock(
-                MathHelper.floor(consumer.getDatatypePost(RelativeCoordinate.INSTANCE, (double) origin.x)),
-                MathHelper.floor(consumer.getDatatypePost(RelativeCoordinate.INSTANCE, (double) origin.y)),
-                MathHelper.floor(consumer.getDatatypePost(RelativeCoordinate.INSTANCE, (double) origin.z))
-        );
-    }
-
-    @Override
-    public Stream<String> tabComplete(IDatatypeContext ctx) {
-        final IArgConsumer consumer = ctx.getConsumer();
-        if (consumer.hasAtMost(3)) {
-            return consumer.tabCompleteDatatype(RelativeCoordinate.INSTANCE);
-        }
-        return Stream.empty();
-    }
-}

src/api/java/baritone/api/command/datatypes/RelativeGoalXZ.java

@@ -1,52 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.datatypes;
-
-import baritone.api.command.argument.IArgConsumer;
-import baritone.api.command.exception.CommandException;
-import baritone.api.pathing.goals.GoalXZ;
-import baritone.api.utils.BetterBlockPos;
-import net.minecraft.util.math.MathHelper;
-
-import java.util.stream.Stream;
-
-public enum RelativeGoalXZ implements IDatatypePost<GoalXZ, BetterBlockPos> {
-    INSTANCE;
-
-    @Override
-    public GoalXZ apply(IDatatypeContext ctx, BetterBlockPos origin) throws CommandException {
-        if (origin == null) {
-            origin = BetterBlockPos.ORIGIN;
-        }
-
-        final IArgConsumer consumer = ctx.getConsumer();
-        return new GoalXZ(
-                MathHelper.floor(consumer.getDatatypePost(RelativeCoordinate.INSTANCE, (double) origin.x)),
-                MathHelper.floor(consumer.getDatatypePost(RelativeCoordinate.INSTANCE, (double) origin.z))
-        );
-    }
-
-    @Override
-    public Stream<String> tabComplete(IDatatypeContext ctx) {
-        final IArgConsumer consumer = ctx.getConsumer();
-        if (consumer.hasAtMost(2)) {
-            return consumer.tabCompleteDatatype(RelativeCoordinate.INSTANCE);
-        }
-        return Stream.empty();
-    }
-}

src/api/java/baritone/api/command/datatypes/RelativeGoalYLevel.java

@@ -1,50 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.datatypes;
-
-import baritone.api.command.argument.IArgConsumer;
-import baritone.api.command.exception.CommandException;
-import baritone.api.pathing.goals.GoalYLevel;
-import baritone.api.utils.BetterBlockPos;
-import net.minecraft.util.math.MathHelper;
-
-import java.util.stream.Stream;
-
-public enum RelativeGoalYLevel implements IDatatypePost<GoalYLevel, BetterBlockPos> {
-    INSTANCE;
-
-    @Override
-    public GoalYLevel apply(IDatatypeContext ctx, BetterBlockPos origin) throws CommandException {
-        if (origin == null) {
-            origin = BetterBlockPos.ORIGIN;
-        }
-
-        return new GoalYLevel(
-                MathHelper.floor(ctx.getConsumer().getDatatypePost(RelativeCoordinate.INSTANCE, (double) origin.y))
-        );
-    }
-
-    @Override
-    public Stream<String> tabComplete(IDatatypeContext ctx) {
-        final IArgConsumer consumer = ctx.getConsumer();
-        if (consumer.hasAtMost(1)) {
-            return consumer.tabCompleteDatatype(RelativeCoordinate.INSTANCE);
-        }
-        return Stream.empty();
-    }
-}

src/api/java/baritone/api/command/exception/CommandErrorMessageException.java

@@ -1,29 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.exception;
-
-public abstract class CommandErrorMessageException extends CommandException {
-
-    protected CommandErrorMessageException(String reason) {
-        super(reason);
-    }
-
-    protected CommandErrorMessageException(String reason, Throwable cause) {
-        super(reason, cause);
-    }
-}

src/api/java/baritone/api/command/exception/CommandException.java

@@ -1,29 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.exception;
-
-public abstract class CommandException extends Exception implements ICommandException {
-
-    protected CommandException(String reason) {
-        super(reason);
-    }
-
-    protected CommandException(String reason, Throwable cause) {
-        super(reason, cause);
-    }
-}

src/api/java/baritone/api/command/exception/CommandInvalidArgumentException.java

@@ -1,43 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.exception;
-
-import baritone.api.command.argument.ICommandArgument;
-
-public abstract class CommandInvalidArgumentException extends CommandErrorMessageException {
-
-    public final ICommandArgument arg;
-
-    protected CommandInvalidArgumentException(ICommandArgument arg, String message) {
-        super(formatMessage(arg, message));
-        this.arg = arg;
-    }
-
-    protected CommandInvalidArgumentException(ICommandArgument arg, String message, Throwable cause) {
-        super(formatMessage(arg, message), cause);
-        this.arg = arg;
-    }
-
-    private static String formatMessage(ICommandArgument arg, String message) {
-        return String.format(
-                "Error at argument #%s: %s",
-                arg.getIndex() == -1 ? "<unknown>" : Integer.toString(arg.getIndex() + 1),
-                message
-        );
-    }
-}

src/api/java/baritone/api/command/exception/CommandInvalidStateException.java

@@ -1,25 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.exception;
-
-public class CommandInvalidStateException extends CommandErrorMessageException {
-
-    public CommandInvalidStateException(String reason) {
-        super(reason);
-    }
-}

src/api/java/baritone/api/command/exception/CommandInvalidTypeException.java

@@ -1,39 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.exception;
-
-import baritone.api.command.argument.ICommandArgument;
-
-public class CommandInvalidTypeException extends CommandInvalidArgumentException {
-
-    public CommandInvalidTypeException(ICommandArgument arg, String expected) {
-        super(arg, String.format("Expected %s", expected));
-    }
-
-    public CommandInvalidTypeException(ICommandArgument arg, String expected, Throwable cause) {
-        super(arg, String.format("Expected %s", expected), cause);
-    }
-
-    public CommandInvalidTypeException(ICommandArgument arg, String expected, String got) {
-        super(arg, String.format("Expected %s, but got %s instead", expected, got));
-    }
-
-    public CommandInvalidTypeException(ICommandArgument arg, String expected, String got, Throwable cause) {
-        super(arg, String.format("Expected %s, but got %s instead", expected, got), cause);
-    }
-}

src/api/java/baritone/api/command/exception/CommandNoParserForTypeException.java

@@ -1,25 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.exception;
-
-public class CommandNoParserForTypeException extends CommandUnhandledException {
-
-    public CommandNoParserForTypeException(Class<?> klass) {
-        super(String.format("Could not find a handler for type %s", klass.getSimpleName()));
-    }
-}

src/api/java/baritone/api/command/exception/CommandNotEnoughArgumentsException.java

@@ -1,25 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.exception;
-
-public class CommandNotEnoughArgumentsException extends CommandErrorMessageException {
-
-    public CommandNotEnoughArgumentsException(int minArgs) {
-        super(String.format("Not enough arguments (expected at least %d)", minArgs));
-    }
-}

src/api/java/baritone/api/command/exception/CommandNotFoundException.java

@@ -1,40 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.exception;
-
-import baritone.api.command.ICommand;
-import baritone.api.command.argument.ICommandArgument;
-
-import java.util.List;
-
-import static baritone.api.utils.Helper.HELPER;
-
-public class CommandNotFoundException extends CommandException {
-
-    public final String command;
-
-    public CommandNotFoundException(String command) {
-        super(String.format("Command not found: %s", command));
-        this.command = command;
-    }
-
-    @Override
-    public void handle(ICommand command, List<ICommandArgument> args) {
-        HELPER.logDirect(getMessage());
-    }
-}

src/api/java/baritone/api/command/exception/CommandTooManyArgumentsException.java

@@ -1,25 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.exception;
-
-public class CommandTooManyArgumentsException extends CommandErrorMessageException {
-
-    public CommandTooManyArgumentsException(int maxArgs) {
-        super(String.format("Too many arguments (expected at most %d)", maxArgs));
-    }
-}

src/api/java/baritone/api/command/exception/CommandUnhandledException.java

@@ -1,46 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.exception;
-
-import baritone.api.command.ICommand;
-import baritone.api.command.argument.ICommandArgument;
-import net.minecraft.util.text.TextFormatting;
-
-import java.util.List;
-
-import static baritone.api.utils.Helper.HELPER;
-
-public class CommandUnhandledException extends RuntimeException implements ICommandException {
-
-    public CommandUnhandledException(String message) {
-        super(message);
-    }
-
-    public CommandUnhandledException(Throwable cause) {
-        super(cause);
-    }
-
-    @Override
-    public void handle(ICommand command, List<ICommandArgument> args) {
-        HELPER.logDirect("An unhandled exception occurred. " +
-                        "The error is in your game's log, please report this at https://github.com/cabaletta/baritone/issues",
-                TextFormatting.RED);
-
-        this.printStackTrace();
-    }
-}

src/api/java/baritone/api/command/exception/ICommandException.java

@@ -1,55 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.exception;
-
-import baritone.api.command.ICommand;
-import baritone.api.command.argument.ICommandArgument;
-import net.minecraft.util.text.TextFormatting;
-
-import java.util.List;
-
-import static baritone.api.utils.Helper.HELPER;
-
-/**
- * The base for a Baritone Command Exception, checked or unchecked. Provides a
- * {@link #handle(ICommand, List)} method that is used to provide useful output
- * to the user for diagnosing issues that may have occurred during execution.
- * <p>
- * Anything implementing this interface should be assignable to {@link Exception}.
- *
- * @author Brady
- * @since 9/20/2019
- */
-public interface ICommandException {
-
-    /**
-     * @return The exception details
-     * @see Exception#getMessage()
-     */
-    String getMessage();
-
-    /**
-     * Called when this exception is thrown, to handle the exception.
-     *
-     * @param command The command that threw it.
-     * @param args    The arguments the command was called with.
-     */
-    default void handle(ICommand command, List<ICommandArgument> args) {
-        HELPER.logDirect(this.getMessage(), TextFormatting.RED);
-    }
-}

src/api/java/baritone/api/command/helpers/Paginator.java

@@ -1,184 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.helpers;
-
-import baritone.api.command.argument.IArgConsumer;
-import baritone.api.command.exception.CommandException;
-import baritone.api.command.exception.CommandInvalidTypeException;
-import baritone.api.utils.Helper;
-import net.minecraft.util.text.ITextComponent;
-import net.minecraft.util.text.TextComponentString;
-import net.minecraft.util.text.TextFormatting;
-import net.minecraft.util.text.event.ClickEvent;
-import net.minecraft.util.text.event.HoverEvent;
-
-import java.util.Arrays;
-import java.util.List;
-import java.util.function.Function;
-
-public class Paginator<E> implements Helper {
-
-    public final List<E> entries;
-    public int pageSize = 8;
-    public int page = 1;
-
-    public Paginator(List<E> entries) {
-        this.entries = entries;
-    }
-
-    public Paginator(E... entries) {
-        this.entries = Arrays.asList(entries);
-    }
-
-    public Paginator<E> setPageSize(int pageSize) {
-        this.pageSize = pageSize;
-        return this;
-    }
-
-    public int getMaxPage() {
-        return (entries.size() - 1) / pageSize + 1;
-    }
-
-    public boolean validPage(int page) {
-        return page > 0 && page <= getMaxPage();
-    }
-
-    public Paginator<E> skipPages(int pages) {
-        page += pages;
-        return this;
-    }
-
-    public void display(Function<E, ITextComponent> transform, String commandPrefix) {
-        int offset = (page - 1) * pageSize;
-        for (int i = offset; i < offset + pageSize; i++) {
-            if (i < entries.size()) {
-                logDirect(transform.apply(entries.get(i)));
-            } else {
-                logDirect("--", TextFormatting.DARK_GRAY);
-            }
-        }
-        boolean hasPrevPage = commandPrefix != null && validPage(page - 1);
-        boolean hasNextPage = commandPrefix != null && validPage(page + 1);
-        ITextComponent prevPageComponent = new TextComponentString("<<");
-        if (hasPrevPage) {
-            prevPageComponent.getStyle()
-                    .setClickEvent(new ClickEvent(
-                            ClickEvent.Action.RUN_COMMAND,
-                            String.format("%s %d", commandPrefix, page - 1)
-                    ))
-                    .setHoverEvent(new HoverEvent(
-                            HoverEvent.Action.SHOW_TEXT,
-                            new TextComponentString("Click to view previous page")
-                    ));
-        } else {
-            prevPageComponent.getStyle().setColor(TextFormatting.DARK_GRAY);
-        }
-        ITextComponent nextPageComponent = new TextComponentString(">>");
-        if (hasNextPage) {
-            nextPageComponent.getStyle()
-                    .setClickEvent(new ClickEvent(
-                            ClickEvent.Action.RUN_COMMAND,
-                            String.format("%s %d", commandPrefix, page + 1)
-                    ))
-                    .setHoverEvent(new HoverEvent(
-                            HoverEvent.Action.SHOW_TEXT,
-                            new TextComponentString("Click to view next page")
-                    ));
-        } else {
-            nextPageComponent.getStyle().setColor(TextFormatting.DARK_GRAY);
-        }
-        ITextComponent pagerComponent = new TextComponentString("");
-        pagerComponent.getStyle().setColor(TextFormatting.GRAY);
-        pagerComponent.appendSibling(prevPageComponent);
-        pagerComponent.appendText(" | ");
-        pagerComponent.appendSibling(nextPageComponent);
-        pagerComponent.appendText(String.format(" %d/%d", page, getMaxPage()));
-        logDirect(pagerComponent);
-    }
-
-    public void display(Function<E, ITextComponent> transform) {
-        display(transform, null);
-    }
-
-    public static <T> void paginate(IArgConsumer consumer, Paginator<T> pagi, Runnable pre, Function<T, ITextComponent> transform, String commandPrefix) throws CommandException {
-        int page = 1;
-        consumer.requireMax(1);
-        if (consumer.hasAny()) {
-            page = consumer.getAs(Integer.class);
-            if (!pagi.validPage(page)) {
-                throw new CommandInvalidTypeException(
-                        consumer.consumed(),
-                        String.format(
-                                "a valid page (1-%d)",
-                                pagi.getMaxPage()
-                        ),
-                        consumer.consumed().getValue()
-                );
-            }
-        }
-        pagi.skipPages(page - pagi.page);
-        if (pre != null) {
-            pre.run();
-        }
-        pagi.display(transform, commandPrefix);
-    }
-
-    public static <T> void paginate(IArgConsumer consumer, List<T> elems, Runnable pre, Function<T, ITextComponent> transform, String commandPrefix) throws CommandException {
-        paginate(consumer, new Paginator<>(elems), pre, transform, commandPrefix);
-    }
-
-    public static <T> void paginate(IArgConsumer consumer, T[] elems, Runnable pre, Function<T, ITextComponent> transform, String commandPrefix) throws CommandException {
-        paginate(consumer, Arrays.asList(elems), pre, transform, commandPrefix);
-    }
-
-    public static <T> void paginate(IArgConsumer consumer, Paginator<T> pagi, Function<T, ITextComponent> transform, String commandPrefix) throws CommandException {
-        paginate(consumer, pagi, null, transform, commandPrefix);
-    }
-
-    public static <T> void paginate(IArgConsumer consumer, List<T> elems, Function<T, ITextComponent> transform, String commandPrefix) throws CommandException {
-        paginate(consumer, new Paginator<>(elems), null, transform, commandPrefix);
-    }
-
-    public static <T> void paginate(IArgConsumer consumer, T[] elems, Function<T, ITextComponent> transform, String commandPrefix) throws CommandException {
-        paginate(consumer, Arrays.asList(elems), null, transform, commandPrefix);
-    }
-
-    public static <T> void paginate(IArgConsumer consumer, Paginator<T> pagi, Runnable pre, Function<T, ITextComponent> transform) throws CommandException {
-        paginate(consumer, pagi, pre, transform, null);
-    }
-
-    public static <T> void paginate(IArgConsumer consumer, List<T> elems, Runnable pre, Function<T, ITextComponent> transform) throws CommandException {
-        paginate(consumer, new Paginator<>(elems), pre, transform, null);
-    }
-
-    public static <T> void paginate(IArgConsumer consumer, T[] elems, Runnable pre, Function<T, ITextComponent> transform) throws CommandException {
-        paginate(consumer, Arrays.asList(elems), pre, transform, null);
-    }
-
-    public static <T> void paginate(IArgConsumer consumer, Paginator<T> pagi, Function<T, ITextComponent> transform) throws CommandException {
-        paginate(consumer, pagi, null, transform, null);
-    }
-
-    public static <T> void paginate(IArgConsumer consumer, List<T> elems, Function<T, ITextComponent> transform) throws CommandException {
-        paginate(consumer, new Paginator<>(elems), null, transform, null);
-    }
-
-    public static <T> void paginate(IArgConsumer consumer, T[] elems, Function<T, ITextComponent> transform) throws CommandException {
-        paginate(consumer, Arrays.asList(elems), null, transform, null);
-    }
-}

src/api/java/baritone/api/command/helpers/TabCompleteHelper.java

@@ -1,287 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.helpers;
-
-import baritone.api.BaritoneAPI;
-import baritone.api.Settings;
-import baritone.api.command.argument.IArgConsumer;
-import baritone.api.command.manager.ICommandManager;
-import baritone.api.event.events.TabCompleteEvent;
-import baritone.api.utils.SettingsUtil;
-import net.minecraft.util.ResourceLocation;
-
-import java.util.Comparator;
-import java.util.List;
-import java.util.Locale;
-import java.util.function.Function;
-import java.util.function.Predicate;
-import java.util.stream.Stream;
-
-/**
- * The {@link TabCompleteHelper} is a <b>single-use</b> object that helps you handle tab completion. It includes helper
- * methods for appending and prepending streams, sorting, filtering by prefix, and so on.
- * <p>
- * The recommended way to use this class is:
- * <ul>
- * <li>Create a new instance with the empty constructor</li>
- * <li>Use {@code append}, {@code prepend} or {@code add<something>} methods to add completions</li>
- * <li>Sort using {@link #sort(Comparator)} or {@link #sortAlphabetically()} and then filter by prefix using
- * {@link #filterPrefix(String)}</li>
- * <li>Get the stream using {@link #stream()}</li>
- * <li>Pass it up to whatever's calling your tab complete function (i.e.
- * {@link ICommandManager#tabComplete(String)} or {@link IArgConsumer}#tabCompleteDatatype(IDatatype)})</li>
- * </ul>
- * <p>
- * For advanced users: if you're intercepting {@link TabCompleteEvent}s directly, use {@link #build()} instead for an
- * array.
- */
-public class TabCompleteHelper {
-
-    private Stream<String> stream;
-
-    public TabCompleteHelper(String[] base) {
-        stream = Stream.of(base);
-    }
-
-    public TabCompleteHelper(List<String> base) {
-        stream = base.stream();
-    }
-
-    public TabCompleteHelper() {
-        stream = Stream.empty();
-    }
-
-    /**
-     * Appends the specified stream to this {@link TabCompleteHelper} and returns it for chaining
-     *
-     * @param source The stream to append
-     * @return This {@link TabCompleteHelper} after having appended the stream
-     * @see #append(String...)
-     * @see #append(Class)
-     */
-    public TabCompleteHelper append(Stream<String> source) {
-        stream = Stream.concat(stream, source);
-        return this;
-    }
-
-    /**
-     * Appends the specified strings to this {@link TabCompleteHelper} and returns it for chaining
-     *
-     * @param source The stream to append
-     * @return This {@link TabCompleteHelper} after having appended the strings
-     * @see #append(Stream)
-     * @see #append(Class)
-     */
-    public TabCompleteHelper append(String... source) {
-        return append(Stream.of(source));
-    }
-
-    /**
-     * Appends all values of the specified enum to this {@link TabCompleteHelper} and returns it for chaining
-     *
-     * @param num The enum to append the values of
-     * @return This {@link TabCompleteHelper} after having appended the values
-     * @see #append(Stream)
-     * @see #append(String...)
-     */
-    public TabCompleteHelper append(Class<? extends Enum<?>> num) {
-        return append(
-                Stream.of(num.getEnumConstants())
-                        .map(Enum::name)
-                        .map(String::toLowerCase)
-        );
-    }
-
-    /**
-     * Prepends the specified stream to this {@link TabCompleteHelper} and returns it for chaining
-     *
-     * @param source The stream to prepend
-     * @return This {@link TabCompleteHelper} after having prepended the stream
-     * @see #prepend(String...)
-     * @see #prepend(Class)
-     */
-    public TabCompleteHelper prepend(Stream<String> source) {
-        stream = Stream.concat(source, stream);
-        return this;
-    }
-
-    /**
-     * Prepends the specified strings to this {@link TabCompleteHelper} and returns it for chaining
-     *
-     * @param source The stream to prepend
-     * @return This {@link TabCompleteHelper} after having prepended the strings
-     * @see #prepend(Stream)
-     * @see #prepend(Class)
-     */
-    public TabCompleteHelper prepend(String... source) {
-        return prepend(Stream.of(source));
-    }
-
-    /**
-     * Prepends all values of the specified enum to this {@link TabCompleteHelper} and returns it for chaining
-     *
-     * @param num The enum to prepend the values of
-     * @return This {@link TabCompleteHelper} after having prepended the values
-     * @see #prepend(Stream)
-     * @see #prepend(String...)
-     */
-    public TabCompleteHelper prepend(Class<? extends Enum<?>> num) {
-        return prepend(
-                Stream.of(num.getEnumConstants())
-                        .map(Enum::name)
-                        .map(String::toLowerCase)
-        );
-    }
-
-    /**
-     * Apply the specified {@code transform} to every element <b>currently</b> in this {@link TabCompleteHelper} and
-     * return this object for chaining
-     *
-     * @param transform The transform to apply
-     * @return This {@link TabCompleteHelper}
-     */
-    public TabCompleteHelper map(Function<String, String> transform) {
-        stream = stream.map(transform);
-        return this;
-    }
-
-    /**
-     * Apply the specified {@code filter} to every element <b>currently</b> in this {@link TabCompleteHelper} and return
-     * this object for chaining
-     *
-     * @param filter The filter to apply
-     * @return This {@link TabCompleteHelper}
-     */
-    public TabCompleteHelper filter(Predicate<String> filter) {
-        stream = stream.filter(filter);
-        return this;
-    }
-
-    /**
-     * Apply the specified {@code sort} to every element <b>currently</b> in this {@link TabCompleteHelper} and return
-     * this object for chaining
-     *
-     * @param comparator The comparator to use
-     * @return This {@link TabCompleteHelper}
-     */
-    public TabCompleteHelper sort(Comparator<String> comparator) {
-        stream = stream.sorted(comparator);
-        return this;
-    }
-
-    /**
-     * Sort every element <b>currently</b> in this {@link TabCompleteHelper} alphabetically and return this object for
-     * chaining
-     *
-     * @return This {@link TabCompleteHelper}
-     */
-    public TabCompleteHelper sortAlphabetically() {
-        return sort(String.CASE_INSENSITIVE_ORDER);
-    }
-
-    /**
-     * Filter out any element that doesn't start with {@code prefix} and return this object for chaining
-     *
-     * @param prefix The prefix to filter for
-     * @return This {@link TabCompleteHelper}
-     */
-    public TabCompleteHelper filterPrefix(String prefix) {
-        return filter(x -> x.toLowerCase(Locale.US).startsWith(prefix.toLowerCase(Locale.US)));
-    }
-
-    /**
-     * Filter out any element that doesn't start with {@code prefix} and return this object for chaining
-     * <p>
-     * Assumes every element in this {@link TabCompleteHelper} is a {@link ResourceLocation}
-     *
-     * @param prefix The prefix to filter for
-     * @return This {@link TabCompleteHelper}
-     */
-    public TabCompleteHelper filterPrefixNamespaced(String prefix) {
-        return filterPrefix(new ResourceLocation(prefix).toString());
-    }
-
-    /**
-     * @return An array containing every element in this {@link TabCompleteHelper}
-     * @see #stream()
-     */
-    public String[] build() {
-        return stream.toArray(String[]::new);
-    }
-
-    /**
-     * @return A stream containing every element in this {@link TabCompleteHelper}
-     * @see #build()
-     */
-    public Stream<String> stream() {
-        return stream;
-    }
-
-    /**
-     * Appends every command in the specified {@link ICommandManager} to this {@link TabCompleteHelper}
-     *
-     * @param manager A command manager
-     * @return This {@link TabCompleteHelper}
-     */
-    public TabCompleteHelper addCommands(ICommandManager manager) {
-        return append(manager.getRegistry().descendingStream()
-                .flatMap(command -> command.getNames().stream())
-                .distinct()
-        );
-    }
-
-    /**
-     * Appends every setting in the {@link Settings} to this {@link TabCompleteHelper}
-     *
-     * @return This {@link TabCompleteHelper}
-     */
-    public TabCompleteHelper addSettings() {
-        return append(
-                BaritoneAPI.getSettings().allSettings.stream()
-                        .filter(s -> !SettingsUtil.javaOnlySetting(s))
-                        .map(Settings.Setting::getName)
-                        .sorted(String.CASE_INSENSITIVE_ORDER)
-        );
-    }
-
-    /**
-     * Appends every modified setting in the {@link Settings} to this {@link TabCompleteHelper}
-     *
-     * @return This {@link TabCompleteHelper}
-     */
-    public TabCompleteHelper addModifiedSettings() {
-        return append(
-                SettingsUtil.modifiedSettings(BaritoneAPI.getSettings()).stream()
-                        .map(Settings.Setting::getName)
-                        .sorted(String.CASE_INSENSITIVE_ORDER)
-        );
-    }
-
-    /**
-     * Appends every {@link Boolean} setting in the {@link Settings} to this {@link TabCompleteHelper}
-     *
-     * @return This {@link TabCompleteHelper}
-     */
-    public TabCompleteHelper addToggleableSettings() {
-        return append(
-                BaritoneAPI.getSettings().getAllValuesByType(Boolean.class).stream()
-                        .map(Settings.Setting::getName)
-                        .sorted(String.CASE_INSENSITIVE_ORDER)
-        );
-    }
-}

src/api/java/baritone/api/command/manager/ICommandManager.java

@@ -1,52 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.manager;
-
-import baritone.api.IBaritone;
-import baritone.api.command.ICommand;
-import baritone.api.command.argument.ICommandArgument;
-import baritone.api.command.registry.Registry;
-import net.minecraft.util.Tuple;
-
-import java.util.List;
-import java.util.stream.Stream;
-
-/**
- * @author Brady
- * @since 9/21/2019
- */
-public interface ICommandManager {
-
-    IBaritone getBaritone();
-
-    Registry<ICommand> getRegistry();
-
-    /**
-     * @param name The command name to search for.
-     * @return The command, if found.
-     */
-    ICommand getCommand(String name);
-
-    boolean execute(String string);
-
-    boolean execute(Tuple<String, List<ICommandArgument>> expanded);
-
-    Stream<String> tabComplete(Tuple<String, List<ICommandArgument>> expanded);
-
-    Stream<String> tabComplete(String prefix);
-}

src/api/java/baritone/api/command/registry/Registry.java

@@ -1,135 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.command.registry;
-
-import java.util.*;
-import java.util.function.Consumer;
-import java.util.stream.Stream;
-import java.util.stream.StreamSupport;
-
-/**
- * This registry class allows for registration and unregistration of a certain type. This is mainly designed for use by
- * event handlers where newly registered ones are encountered first during iteration and can therefore override older
- * ones. In Baritone, this is used for commands and argument parsers so that mods and addons can extend Baritone's
- * functionality without resorting to hacks, wrappers, or mixins.
- *
- * @param <V> The entry type that will be stored in this registry. This can be anything, really - preferably anything
- *            that works as a HashMap key, as that's what's used to keep track of which entries are registered or not.
- */
-public class Registry<V> {
-
-    /**
-     * An internal linked list of all the entries that are currently registered. This is a linked list so that entries
-     * can be inserted at the beginning, which means that newer entries are encountered first during iteration. This is
-     * an important property of the registry that makes it more useful than a simple list, and also the reason it does
-     * not just use a map.
-     */
-    private final Deque<V> _entries = new LinkedList<>();
-    /**
-     * A HashSet containing every entry currently registered. Entries are added to this set when something is registered
-     * and removed from the set when they are unregistered. An entry being present in this set indicates that it is
-     * currently registered, can be removed, and should not be reregistered until it is removed.
-     */
-    private final Set<V> registered = new HashSet<>();
-    /**
-     * The collection of entries that are currently in this registry. This is a collection (and not a list) because,
-     * internally, entries are stored in a linked list, which is not the same as a normal list.
-     */
-    public final Collection<V> entries = Collections.unmodifiableCollection(_entries);
-
-    /**
-     * @param entry The entry to check.
-     * @return If this entry is currently registered in this registry.
-     */
-    public boolean registered(V entry) {
-        return registered.contains(entry);
-    }
-
-    /**
-     * Ensures that the entry {@code entry} is registered.
-     *
-     * @param entry The entry to register.
-     * @return A boolean indicating whether or not this is a new registration. No matter the value of this boolean, the
-     * entry is always guaranteed to now be in this registry. This boolean simply indicates if the entry was <i>not</i>
-     * in the map prior to this method call.
-     */
-    public boolean register(V entry) {
-        if (!registered(entry)) {
-            _entries.addFirst(entry);
-            registered.add(entry);
-            return true;
-        }
-        return false;
-    }
-
-    /**
-     * Unregisters this entry from this registry. After this method call, the entry is guaranteed to be removed from the
-     * registry, since each entry only ever appears once.
-     *
-     * @param entry The entry to unregister.
-     */
-    public void unregister(V entry) {
-        if (registered(entry)) {
-            return;
-        }
-        _entries.remove(entry);
-        registered.remove(entry);
-    }
-
-    /**
-     * Returns an iterator that iterates over each entry in this registry, with the newest elements iterated over first.
-     * Internally, as new elements are prepended to the registry rather than appended to the end, this order is the best
-     * way to search through the registry if you want to discover newer items first.
-     */
-    public Iterator<V> iterator() {
-        return _entries.iterator();
-    }
-
-    /**
-     * Returns an iterator that iterates over each entry in this registry, in the order they were added. Internally,
-     * this iterates through the registry backwards, as new elements are prepended to the registry rather than appended
-     * to the end. You should only do this when you need to, for example, list elements in order - it is almost always
-     * fine to simply use {@link Iterable#forEach(Consumer) forEach} on the {@link #entries} collection instead.
-     */
-    public Iterator<V> descendingIterator() {
-        return _entries.descendingIterator();
-    }
-
-    /**
-     * Returns a stream that contains each entry in this registry, with the newest elements ordered first. Internally,
-     * as new elements are prepended to the registry rather than appended to the end, this order is the best way to
-     * search through the registry if you want to discover newer items first.
-     */
-    public Stream<V> stream() {
-        return _entries.stream();
-    }
-
-    /**
-     * Returns a stream that returns each entry in this registry, in the order they were added. Internally, this orders
-     * the registry backwards, as new elements are prepended to the registry rather than appended to the end. You should
-     * only use this when you need to, for example, list elements in order - it is almost always fine to simply use the
-     * regular {@link #stream()} method instead.
-     */
-    public Stream<V> descendingStream() {
-        return StreamSupport.stream(Spliterators.spliterator(
-                descendingIterator(),
-                _entries.size(),
-                Spliterator.SIZED | Spliterator.SUBSIZED
-        ), false);
-    }
-}

src/api/java/baritone/api/event/events/TabCompleteEvent.java

@@ -1,34 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.event.events;
-
-import baritone.api.event.events.type.Cancellable;
-
-/**
- * @author LoganDark
- */
-public class TabCompleteEvent extends Cancellable {
-
-    public final String prefix;
-    public String[] completions;
-
-    public TabCompleteEvent(String prefix) {
-        this.prefix = prefix;
-        this.completions = null;
-    }
-}

src/api/java/baritone/api/event/events/TickEvent.java

@@ -19,20 +19,22 @@ package baritone.api.event.events;
 
 import baritone.api.event.events.type.EventState;
 
-import java.util.function.BiFunction;
-
 public final class TickEvent {
 
-    private static int overallTickCount;
-
     private final EventState state;
     private final Type type;
     private final int count;
 
-    public TickEvent(EventState state, Type type, int count) {
+    private static int overallTickCount;
+
+    public TickEvent(EventState state, Type type) {
         this.state = state;
         this.type = type;
-        this.count = count;
+        this.count = incrementCount();
+    }
+
+    private static synchronized int incrementCount() {
+        return overallTickCount++;
     }
 
     public int getCount() {
@@ -47,10 +49,6 @@ public final class TickEvent {
         return state;
     }
 
-    public static synchronized BiFunction<EventState, Type, TickEvent> createNextProvider() {
-        final int count = overallTickCount++;
-        return (state, type) -> new TickEvent(state, type, count);
-    }
 
     public enum Type {
         /**

src/api/java/baritone/api/event/events/type/Overrideable.java

@@ -1,53 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.event.events.type;
-
-/**
- * @author LoganDark
- */
-public class Overrideable<T> {
-
-    private T value;
-    private boolean modified;
-
-    public Overrideable(T current) {
-        value = current;
-    }
-
-    public T get() {
-        return value;
-    }
-
-    public void set(T newValue) {
-        value = newValue;
-        modified = true;
-    }
-
-    public boolean wasModified() {
-        return modified;
-    }
-
-    @Override
-    public String toString() {
-        return String.format(
-                "Overrideable{modified=%b,value=%s}",
-                modified,
-                value.toString()
-        );
-    }
-}

src/api/java/baritone/api/event/listener/AbstractGameEventListener.java

@@ -39,9 +39,6 @@ public interface AbstractGameEventListener extends IGameEventListener {
     @Override
     default void onSendChatMessage(ChatEvent event) {}
 
-    @Override
-    default void onPreTabComplete(TabCompleteEvent event) {}
-
     @Override
     default void onChunkEvent(ChunkEvent event) {}
 

src/api/java/baritone/api/event/listener/IGameEventListener.java

@@ -18,12 +18,11 @@
 package baritone.api.event.listener;
 
 import baritone.api.event.events.*;
-import io.netty.util.concurrent.GenericFutureListener;
 import net.minecraft.client.Minecraft;
 import net.minecraft.client.entity.EntityPlayerSP;
 import net.minecraft.client.gui.GuiGameOver;
+import net.minecraft.client.gui.GuiScreen;
 import net.minecraft.client.multiplayer.WorldClient;
-import net.minecraft.client.settings.GameSettings;
 import net.minecraft.entity.Entity;
 import net.minecraft.network.Packet;
 
@@ -45,7 +44,7 @@ public interface IGameEventListener {
      * Run once per game tick from before and after the player rotation is sent to the server.
      *
      * @param event The event
-     * @see EntityPlayerSP#onUpdate()
+     * @see EntityPlayerSP#tick()
      */
     void onPlayerUpdate(PlayerUpdateEvent event);
 
@@ -57,25 +56,15 @@ public interface IGameEventListener {
      */
     void onSendChatMessage(ChatEvent event);
 
-    /**
-     * Runs whenever the client player tries to tab complete in chat.
-     *
-     * @param event The event
-     */
-    void onPreTabComplete(TabCompleteEvent event);
-
     /**
      * Runs before and after whenever a chunk is either loaded, unloaded, or populated.
      *
      * @param event The event
-     * @see WorldClient#doPreChunk(int, int, boolean)
      */
     void onChunkEvent(ChunkEvent event);
 
     /**
-     * Runs once per world render pass. Two passes are made when {@link GameSettings#anaglyph} is on.
-     * <p>
-     * <b>Note:</b> {@link GameSettings#anaglyph} has been removed in Minecraft 1.13
+     * Runs once per world render pass.
      *
      * @param event The event
      */
@@ -85,7 +74,7 @@ public interface IGameEventListener {
      * Runs before and after whenever a new world is loaded
      *
      * @param event The event
-     * @see Minecraft#loadWorld(WorldClient, String)
+     * @see Minecraft#loadWorld(WorldClient, GuiScreen)
      */
     void onWorldEvent(WorldEvent event);
 
@@ -94,7 +83,6 @@ public interface IGameEventListener {
      *
      * @param event The event
      * @see Packet
-     * @see GenericFutureListener
      */
     void onSendPacket(PacketEvent event);
 
@@ -103,7 +91,6 @@ public interface IGameEventListener {
      *
      * @param event The event
      * @see Packet
-     * @see GenericFutureListener
      */
     void onReceivePacket(PacketEvent event);
 
@@ -117,10 +104,10 @@ public interface IGameEventListener {
     void onPlayerRotationMove(RotationMoveEvent event);
 
     /**
-     * Called whenever the sprint keybind state is checked in {@link EntityPlayerSP#onLivingUpdate}
+     * Called whenever the sprint keybind state is checked in {@link EntityPlayerSP#livingTick}
      *
      * @param event The event
-     * @see EntityPlayerSP#onLivingUpdate()
+     * @see EntityPlayerSP#livingTick()
      */
     void onPlayerSprintState(SprintStateEvent event);
 

src/api/java/baritone/api/pathing/goals/Goal.java

@@ -54,18 +54,4 @@ public interface Goal {
     default double heuristic(BlockPos pos) {
         return heuristic(pos.getX(), pos.getY(), pos.getZ());
     }
-
-    /**
-     * Returns the heuristic at the goal.
-     * i.e. {@code heuristic() == heuristic(x,y,z)}
-     * when {@code isInGoal(x,y,z) == true}
-     * This is needed by {@code PathingBehavior#estimatedTicksToGoal} because
-     * some Goals actually do not have a heuristic of 0 when that condition is met
-     *
-     * @return The estimate number of ticks to satisfy the goal when the goal
-     * is already satisfied
-     */
-    default double heuristic() {
-        return 0;
-    }
 }

src/api/java/baritone/api/pathing/goals/GoalBlock.java

@@ -17,7 +17,6 @@
 
 package baritone.api.pathing.goals;
 
-import baritone.api.utils.SettingsUtil;
 import baritone.api.utils.interfaces.IGoalRenderPos;
 import net.minecraft.util.math.BlockPos;
 
@@ -68,12 +67,7 @@ public class GoalBlock implements Goal, IGoalRenderPos {
 
     @Override
     public String toString() {
-        return String.format(
-                "GoalBlock{x=%s,y=%s,z=%s}",
-                SettingsUtil.maybeCensor(x),
-                SettingsUtil.maybeCensor(y),
-                SettingsUtil.maybeCensor(z)
-        );
+        return "GoalBlock{x=" + x + ",y=" + y + ",z=" + z + "}";
     }
 
     /**

src/api/java/baritone/api/pathing/goals/GoalComposite.java

@@ -57,16 +57,6 @@ public class GoalComposite implements Goal {
         return min;
     }
 
-    @Override
-    public double heuristic() {
-        double min = Double.MAX_VALUE;
-        for (Goal g : goals) {
-            // just take the highest value that is guaranteed to be inside the goal
-            min = Math.min(min, g.heuristic());
-        }
-        return min;
-    }
-
     @Override
     public String toString() {
         return "GoalComposite" + Arrays.toString(goals);

src/api/java/baritone/api/pathing/goals/GoalGetToBlock.java

@@ -17,7 +17,6 @@
 
 package baritone.api.pathing.goals;
 
-import baritone.api.utils.SettingsUtil;
 import baritone.api.utils.interfaces.IGoalRenderPos;
 import net.minecraft.util.math.BlockPos;
 
@@ -62,11 +61,6 @@ public class GoalGetToBlock implements Goal, IGoalRenderPos {
 
     @Override
     public String toString() {
-        return String.format(
-                "GoalGetToBlock{x=%s,y=%s,z=%s}",
-                SettingsUtil.maybeCensor(x),
-                SettingsUtil.maybeCensor(y),
-                SettingsUtil.maybeCensor(z)
-        );
+        return "GoalGetToBlock{x=" + x + ",y=" + y + ",z=" + z + "}";
     }
 }

src/api/java/baritone/api/pathing/goals/GoalInverted.java

@@ -1,57 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.pathing.goals;
-
-/**
- * Invert any goal.
- * <p>
- * In the old chat control system, #invert just tried to pick a {@link GoalRunAway} that <i>effectively</i> inverted the
- * current goal. This goal just reverses the heuristic to act as a TRUE invert. Inverting a Y level? Baritone tries to
- * get away from that Y level. Inverting a GoalBlock? Baritone will try to make distance whether it's in the X, Y or Z
- * directions. And of course, you can always invert a GoalXZ.
- *
- * @author LoganDark
- */
-public class GoalInverted implements Goal {
-
-    public final Goal origin;
-
-    public GoalInverted(Goal origin) {
-        this.origin = origin;
-    }
-
-    @Override
-    public boolean isInGoal(int x, int y, int z) {
-        return false;
-    }
-
-    @Override
-    public double heuristic(int x, int y, int z) {
-        return -origin.heuristic(x, y, z);
-    }
-
-    @Override
-    public double heuristic() {
-        return Double.NEGATIVE_INFINITY;
-    }
-
-    @Override
-    public String toString() {
-        return String.format("GoalInverted{%s}", origin.toString());
-    }
-}

src/api/java/baritone/api/pathing/goals/GoalNear.java

@@ -17,14 +17,10 @@
 
 package baritone.api.pathing.goals;
 
-import baritone.api.utils.SettingsUtil;
 import baritone.api.utils.interfaces.IGoalRenderPos;
-import it.unimi.dsi.fastutil.doubles.DoubleOpenHashSet;
-import it.unimi.dsi.fastutil.doubles.DoubleIterator;
 import net.minecraft.util.math.BlockPos;
 
 public class GoalNear implements Goal, IGoalRenderPos {
-
     private final int x;
     private final int y;
     private final int z;
@@ -53,34 +49,6 @@ public class GoalNear implements Goal, IGoalRenderPos {
         return GoalBlock.calculate(xDiff, yDiff, zDiff);
     }
 
-    @Override
-    public double heuristic() {// TODO less hacky solution
-        int range = (int) Math.ceil(Math.sqrt(rangeSq));
-        DoubleOpenHashSet maybeAlwaysInside = new DoubleOpenHashSet(); // see pull request #1978
-        double minOutside = Double.POSITIVE_INFINITY;
-        for (int dx = -range; dx <= range; dx++) {
-            for (int dy = -range; dy <= range; dy++) {
-                for (int dz = -range; dz <= range; dz++) {
-                    double h = heuristic(x + dx, y + dy, z + dz);
-                    if (h < minOutside && isInGoal(x + dx, y + dy, z + dz)) {
-                        maybeAlwaysInside.add(h);
-                    } else {
-                        minOutside = Math.min(minOutside, h);
-                    }
-                }
-            }
-        }
-        double maxInside = Double.NEGATIVE_INFINITY;
-        DoubleIterator it = maybeAlwaysInside.iterator();
-        while (it.hasNext()) {
-            double inside = it.nextDouble();
-            if (inside < minOutside) {
-                maxInside = Math.max(maxInside, inside);
-            }
-        }
-        return maxInside;
-    }
-
     @Override
     public BlockPos getGoalPos() {
         return new BlockPos(x, y, z);
@@ -88,12 +56,11 @@ public class GoalNear implements Goal, IGoalRenderPos {
 
     @Override
     public String toString() {
-        return String.format(
-                "GoalNear{x=%s, y=%s, z=%s, rangeSq=%d}",
-                SettingsUtil.maybeCensor(x),
-                SettingsUtil.maybeCensor(y),
-                SettingsUtil.maybeCensor(z),
-                rangeSq
-        );
+        return "GoalNear{" +
+                "x=" + x +
+                ", y=" + y +
+                ", z=" + z +
+                ", rangeSq=" + rangeSq +
+                "}";
     }
 }

src/api/java/baritone/api/pathing/goals/GoalRunAway.java

@@ -17,9 +17,6 @@
 
 package baritone.api.pathing.goals;
 
-import baritone.api.utils.SettingsUtil;
-import it.unimi.dsi.fastutil.doubles.DoubleOpenHashSet;
-import it.unimi.dsi.fastutil.doubles.DoubleIterator;
 import net.minecraft.util.math.BlockPos;
 
 import java.util.Arrays;
@@ -33,7 +30,7 @@ public class GoalRunAway implements Goal {
 
     private final BlockPos[] from;
 
-    private final int distanceSq;
+    private final double distanceSq;
 
     private final Integer maintainY;
 
@@ -46,7 +43,7 @@ public class GoalRunAway implements Goal {
             throw new IllegalArgumentException();
         }
         this.from = from;
-        this.distanceSq = (int) (distance * distance);
+        this.distanceSq = distance * distance;
         this.maintainY = maintainY;
     }
 
@@ -58,7 +55,7 @@ public class GoalRunAway implements Goal {
         for (BlockPos p : from) {
             int diffX = x - p.getX();
             int diffZ = z - p.getZ();
-            int distSq = diffX * diffX + diffZ * diffZ;
+            double distSq = diffX * diffX + diffZ * diffZ;
             if (distSq < distanceSq) {
                 return false;
             }
@@ -67,7 +64,7 @@ public class GoalRunAway implements Goal {
     }
 
     @Override
-    public double heuristic(int x, int y, int z) {// mostly copied from GoalBlock
+    public double heuristic(int x, int y, int z) {//mostly copied from GoalBlock
         double min = Double.MAX_VALUE;
         for (BlockPos p : from) {
             double h = GoalXZ.calculate(p.getX() - x, p.getZ() - z);
@@ -82,56 +79,10 @@ public class GoalRunAway implements Goal {
         return min;
     }
 
-    @Override
-    public double heuristic() {// TODO less hacky solution
-        int distance = (int) Math.ceil(Math.sqrt(distanceSq));
-        int minX = Integer.MAX_VALUE;
-        int minY = Integer.MAX_VALUE;
-        int minZ = Integer.MAX_VALUE;
-        int maxX = Integer.MIN_VALUE;
-        int maxY = Integer.MIN_VALUE;
-        int maxZ = Integer.MIN_VALUE;
-        for (BlockPos p : from) {
-            minX = Math.min(minX, p.getX() - distance);
-            minY = Math.min(minY, p.getY() - distance);
-            minZ = Math.min(minZ, p.getZ() - distance);
-            maxX = Math.max(minX, p.getX() + distance);
-            maxY = Math.max(minY, p.getY() + distance);
-            maxZ = Math.max(minZ, p.getZ() + distance);
-        }
-        DoubleOpenHashSet maybeAlwaysInside = new DoubleOpenHashSet(); // see pull request #1978
-        double minOutside = Double.POSITIVE_INFINITY;
-        for (int x = minX; x <= maxX; x++) {
-            for (int y = minY; y <= maxY; y++) {
-                for (int z = minZ; z <= maxZ; z++) {
-                    double h = heuristic(x, y, z);
-                    if (h < minOutside && isInGoal(x, y, z)) {
-                        maybeAlwaysInside.add(h);
-                    } else {
-                        minOutside = Math.min(minOutside, h);
-                    }
-                }
-            }
-        }
-        double maxInside = Double.NEGATIVE_INFINITY;
-        DoubleIterator it = maybeAlwaysInside.iterator();
-        while (it.hasNext()) {
-            double inside = it.nextDouble();
-            if (inside < minOutside) {
-                maxInside = Math.max(maxInside, inside);
-            }
-        }
-        return maxInside;
-    }
-
     @Override
     public String toString() {
         if (maintainY != null) {
-            return String.format(
-                    "GoalRunAwayFromMaintainY y=%s, %s",
-                    SettingsUtil.maybeCensor(maintainY),
-                    Arrays.asList(from)
-            );
+            return "GoalRunAwayFromMaintainY y=" + maintainY + ", " + Arrays.asList(from);
         } else {
             return "GoalRunAwayFrom" + Arrays.asList(from);
         }

src/api/java/baritone/api/pathing/goals/GoalStrictDirection.java

@@ -17,7 +17,6 @@
 
 package baritone.api.pathing.goals;
 
-import baritone.api.utils.SettingsUtil;
 import net.minecraft.util.EnumFacing;
 import net.minecraft.util.math.BlockPos;
 
@@ -25,7 +24,6 @@ import net.minecraft.util.math.BlockPos;
  * Dig a tunnel in a certain direction, but if you have to deviate from the path, go back to where you started
  */
 public class GoalStrictDirection implements Goal {
-
     public final int x;
     public final int y;
     public final int z;
@@ -64,20 +62,14 @@ public class GoalStrictDirection implements Goal {
         return heuristic;
     }
 
-    @Override
-    public double heuristic() {
-        return Double.NEGATIVE_INFINITY;
-    }
-
     @Override
     public String toString() {
-        return String.format(
-                "GoalStrictDirection{x=%s, y=%s, z=%s, dx=%s, dz=%s}",
-                SettingsUtil.maybeCensor(x),
-                SettingsUtil.maybeCensor(y),
-                SettingsUtil.maybeCensor(z),
-                SettingsUtil.maybeCensor(dx),
-                SettingsUtil.maybeCensor(dz)
-        );
+        return "GoalStrictDirection{" +
+                "x=" + x +
+                ", y=" + y +
+                ", z=" + z +
+                ", dx=" + dx +
+                ", dz=" + dz +
+                "}";
     }
 }

src/api/java/baritone/api/pathing/goals/GoalTwoBlocks.java

@@ -17,7 +17,6 @@
 
 package baritone.api.pathing.goals;
 
-import baritone.api.utils.SettingsUtil;
 import baritone.api.utils.interfaces.IGoalRenderPos;
 import net.minecraft.util.math.BlockPos;
 
@@ -74,11 +73,6 @@ public class GoalTwoBlocks implements Goal, IGoalRenderPos {
 
     @Override
     public String toString() {
-        return String.format(
-                "GoalTwoBlocks{x=%s,y=%s,z=%s}",
-                SettingsUtil.maybeCensor(x),
-                SettingsUtil.maybeCensor(y),
-                SettingsUtil.maybeCensor(z)
-        );
+        return "GoalTwoBlocks{x=" + x + ",y=" + y + ",z=" + z + "}";
     }
 }

src/api/java/baritone/api/pathing/goals/GoalXZ.java

@@ -18,8 +18,6 @@
 package baritone.api.pathing.goals;
 
 import baritone.api.BaritoneAPI;
-import baritone.api.utils.BetterBlockPos;
-import baritone.api.utils.SettingsUtil;
 import net.minecraft.util.math.MathHelper;
 import net.minecraft.util.math.Vec3d;
 
@@ -47,11 +45,6 @@ public class GoalXZ implements Goal {
         this.z = z;
     }
 
-    public GoalXZ(BetterBlockPos pos) {
-        this.x = pos.x;
-        this.z = pos.z;
-    }
-
     @Override
     public boolean isInGoal(int x, int y, int z) {
         return x == this.x && z == this.z;
@@ -66,11 +59,7 @@ public class GoalXZ implements Goal {
 
     @Override
     public String toString() {
-        return String.format(
-                "GoalXZ{x=%s,z=%s}",
-                SettingsUtil.maybeCensor(x),
-                SettingsUtil.maybeCensor(z)
-        );
+        return "GoalXZ{x=" + x + ",z=" + z + "}";
     }
 
     public static double calculate(double xDiff, double zDiff) {
@@ -98,7 +87,7 @@ public class GoalXZ implements Goal {
         float theta = (float) Math.toRadians(yaw);
         double x = origin.x - MathHelper.sin(theta) * distance;
         double z = origin.z + MathHelper.cos(theta) * distance;
-        return new GoalXZ(MathHelper.floor(x), MathHelper.floor(z));
+        return new GoalXZ((int) x, (int) z);
     }
 
     public int getX() {

src/api/java/baritone/api/pathing/goals/GoalYLevel.java

@@ -18,7 +18,6 @@
 package baritone.api.pathing.goals;
 
 import baritone.api.pathing.movement.ActionCosts;
-import baritone.api.utils.SettingsUtil;
 
 /**
  * Useful for mining (getting to diamond / iron level)
@@ -60,9 +59,6 @@ public class GoalYLevel implements Goal, ActionCosts {
 
     @Override
     public String toString() {
-        return String.format(
-                "GoalYLevel{y=%s}",
-                SettingsUtil.maybeCensor(level)
-        );
+        return "GoalYLevel{y=" + level + "}";
     }
 }

src/api/java/baritone/api/process/IBaritoneProcess.java

@@ -75,7 +75,7 @@ public interface IBaritoneProcess {
      * to start eating this tick. {@code PauseForAutoEatProcess} should only actually right click once onTick is called with
      * {@code isSafeToCancel} true though.
      *
-     * @return Whether or not if this control is temporary
+     * @return Whethor or not if this control is temporary
      */
     boolean isTemporary();
 

src/api/java/baritone/api/process/IBuilderProcess.java

@@ -17,14 +17,12 @@
 
 package baritone.api.process;
 
-import baritone.api.schematic.ISchematic;
-import net.minecraft.block.state.IBlockState;
+import baritone.api.utils.ISchematic;
 import net.minecraft.client.Minecraft;
 import net.minecraft.util.math.BlockPos;
 import net.minecraft.util.math.Vec3i;
 
 import java.io.File;
-import java.util.List;
 
 /**
  * @author Brady
@@ -52,24 +50,13 @@ public interface IBuilderProcess extends IBaritoneProcess {
     boolean build(String name, File schematic, Vec3i origin);
 
     default boolean build(String schematicFile, BlockPos origin) {
-        File file = new File(new File(Minecraft.getMinecraft().gameDir, "schematics"), schematicFile);
+        File file = new File(new File(Minecraft.getInstance().gameDir, "schematics"), schematicFile);
         return build(schematicFile, file, origin);
     }
 
-    void buildOpenSchematic();
-
     void pause();
 
-    boolean isPaused();
-
     void resume();
 
     void clearArea(BlockPos corner1, BlockPos corner2);
-
-    /**
-     * @return A list of block states that are estimated to be placeable by this builder process. You can use this in
-     * schematics, for example, to pick a state that the builder process will be happy with, because any variation will
-     * cause it to give up. This is updated every tick, but only while the builder process is active.
-     */
-    List<IBlockState> getApproxPlaceable();
 }

src/api/java/baritone/api/process/IExploreProcess.java

@@ -20,7 +20,6 @@ package baritone.api.process;
 import java.nio.file.Path;
 
 public interface IExploreProcess extends IBaritoneProcess {
-
     void explore(int centerX, int centerZ);
 
     void applyJsonFilter(Path path, boolean invert) throws Exception;

src/api/java/baritone/api/process/IFarmProcess.java

@@ -17,29 +17,6 @@
 
 package baritone.api.process;
 
-import net.minecraft.util.math.BlockPos;
-
 public interface IFarmProcess extends IBaritoneProcess {
-
-    /**
-     * Begin to search for crops to farm with in specified aria
-     * from specified location.
-     *
-     * @param range The distance from center to farm from
-     * @param pos   The center position to base the range from
-     */
-    void farm(int range, BlockPos pos);
-
-    /**
-     * Begin to search for nearby crops to farm.
-     */
-    default void farm() {farm(0, null);}
-
-    /**
-     * Begin to search for crops to farm with in specified aria
-     * from the position the command was executed.
-     *
-     * @param range The distance to search for crops to farm
-     */
-    default void farm(int range) {farm(range, null);}
+    void farm();
 }

src/api/java/baritone/api/process/IGetToBlockProcess.java

@@ -17,7 +17,6 @@
 
 package baritone.api.process;
 
-import baritone.api.utils.BlockOptionalMeta;
 import net.minecraft.block.Block;
 
 /**
@@ -25,11 +24,7 @@ import net.minecraft.block.Block;
  */
 public interface IGetToBlockProcess extends IBaritoneProcess {
 
-    void getToBlock(BlockOptionalMeta block);
-
-    default void getToBlock(Block block) {
-        getToBlock(new BlockOptionalMeta(block));
-    }
+    void getToBlock(Block block);
 
     boolean blacklistClosest();
 }

src/api/java/baritone/api/process/IMineProcess.java

@@ -17,12 +17,8 @@
 
 package baritone.api.process;
 
-import baritone.api.utils.BlockOptionalMeta;
-import baritone.api.utils.BlockOptionalMetaLookup;
 import net.minecraft.block.Block;
 
-import java.util.stream.Stream;
-
 /**
  * @author Brady
  * @since 9/23/2018
@@ -32,9 +28,9 @@ public interface IMineProcess extends IBaritoneProcess {
     /**
      * Begin to search for and mine the specified blocks until
      * the number of specified items to get from the blocks that
-     * are mined.
+     * are mined. This is based on the first target block to mine.
      *
-     * @param quantity The total number of items to get
+     * @param quantity The number of items to get from blocks mined
      * @param blocks   The blocks to mine
      */
     void mineByName(int quantity, String... blocks);
@@ -45,18 +41,9 @@ public interface IMineProcess extends IBaritoneProcess {
      * are mined. This is based on the first target block to mine.
      *
      * @param quantity The number of items to get from blocks mined
-     * @param filter   The blocks to mine
+     * @param blocks   The blocks to mine
      */
-    void mine(int quantity, BlockOptionalMetaLookup filter);
-
-    /**
-     * Begin to search for and mine the specified blocks.
-     *
-     * @param filter The blocks to mine
-     */
-    default void mine(BlockOptionalMetaLookup filter) {
-        mine(0, filter);
-    }
+    void mine(int quantity, Block... blocks);
 
     /**
      * Begin to search for and mine the specified blocks.
@@ -67,38 +54,6 @@ public interface IMineProcess extends IBaritoneProcess {
         mineByName(0, blocks);
     }
 
-    /**
-     * Begin to search for and mine the specified blocks.
-     *
-     * @param boms The blocks to mine
-     */
-    default void mine(int quantity, BlockOptionalMeta... boms) {
-        mine(quantity, new BlockOptionalMetaLookup(boms));
-    }
-
-    /**
-     * Begin to search for and mine the specified blocks.
-     *
-     * @param boms The blocks to mine
-     */
-    default void mine(BlockOptionalMeta... boms) {
-        mine(0, boms);
-    }
-
-    /**
-     * Begin to search for and mine the specified blocks.
-     *
-     * @param quantity The total number of items to get
-     * @param blocks   The blocks to mine
-     */
-    default void mine(int quantity, Block... blocks) {
-        mine(quantity, new BlockOptionalMetaLookup(
-                Stream.of(blocks)
-                        .map(BlockOptionalMeta::new)
-                        .toArray(BlockOptionalMeta[]::new)
-        ));
-    }
-
     /**
      * Begin to search for and mine the specified blocks.
      *

src/api/java/baritone/api/schematic/CompositeSchematic.java

@@ -1,81 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.schematic;
-
-import net.minecraft.block.state.IBlockState;
-
-import java.util.ArrayList;
-import java.util.List;
-
-public class CompositeSchematic extends AbstractSchematic {
-
-    private final List<CompositeSchematicEntry> schematics;
-    private CompositeSchematicEntry[] schematicArr;
-
-    private void recalcArr() {
-        schematicArr = schematics.toArray(new CompositeSchematicEntry[0]);
-        for (CompositeSchematicEntry entry : schematicArr) {
-            this.x = Math.max(x, entry.x + entry.schematic.widthX());
-            this.y = Math.max(y, entry.y + entry.schematic.heightY());
-            this.z = Math.max(z, entry.z + entry.schematic.lengthZ());
-        }
-    }
-
-    public CompositeSchematic(int x, int y, int z) {
-        super(x, y, z);
-        schematics = new ArrayList<>();
-        recalcArr();
-    }
-
-    public void put(ISchematic extra, int x, int y, int z) {
-        schematics.add(new CompositeSchematicEntry(extra, x, y, z));
-        recalcArr();
-    }
-
-    private CompositeSchematicEntry getSchematic(int x, int y, int z, IBlockState currentState) {
-        for (CompositeSchematicEntry entry : schematicArr) {
-            if (x >= entry.x && y >= entry.y && z >= entry.z &&
-                    entry.schematic.inSchematic(x - entry.x, y - entry.y, z - entry.z, currentState)) {
-                return entry;
-            }
-        }
-        return null;
-    }
-
-    @Override
-    public boolean inSchematic(int x, int y, int z, IBlockState currentState) {
-        CompositeSchematicEntry entry = getSchematic(x, y, z, currentState);
-        return entry != null && entry.schematic.inSchematic(x - entry.x, y - entry.y, z - entry.z, currentState);
-    }
-
-    @Override
-    public IBlockState desiredState(int x, int y, int z, IBlockState current, List<IBlockState> approxPlaceable) {
-        CompositeSchematicEntry entry = getSchematic(x, y, z, current);
-        if (entry == null) {
-            throw new IllegalStateException("couldn't find schematic for this position");
-        }
-        return entry.schematic.desiredState(x - entry.x, y - entry.y, z - entry.z, current, approxPlaceable);
-    }
-
-    @Override
-    public void reset() {
-        for (CompositeSchematicEntry entry : schematicArr) {
-            entry.schematic.reset();
-        }
-    }
-}

src/api/java/baritone/api/schematic/CompositeSchematicEntry.java

@@ -1,33 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.schematic;
-
-public class CompositeSchematicEntry {
-
-    public final ISchematic schematic;
-    public final int x;
-    public final int y;
-    public final int z;
-
-    public CompositeSchematicEntry(ISchematic schematic, int x, int y, int z) {
-        this.schematic = schematic;
-        this.x = x;
-        this.y = y;
-        this.z = z;
-    }
-}

src/api/java/baritone/api/schematic/FillSchematic.java

@@ -1,54 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.schematic;
-
-import baritone.api.utils.BlockOptionalMeta;
-import net.minecraft.block.state.IBlockState;
-
-import java.util.List;
-
-public class FillSchematic extends AbstractSchematic {
-
-    private final BlockOptionalMeta bom;
-
-    public FillSchematic(int x, int y, int z, BlockOptionalMeta bom) {
-        super(x, y, z);
-        this.bom = bom;
-    }
-
-    public FillSchematic(int x, int y, int z, IBlockState state) {
-        this(x, y, z, new BlockOptionalMeta(state.getBlock(), state.getBlock().getMetaFromState(state)));
-    }
-
-    public BlockOptionalMeta getBom() {
-        return bom;
-    }
-
-    @Override
-    public IBlockState desiredState(int x, int y, int z, IBlockState current, List<IBlockState> approxPlaceable) {
-        if (bom.matches(current)) {
-            return current;
-        }
-        for (IBlockState placeable : approxPlaceable) {
-            if (bom.matches(placeable)) {
-                return placeable;
-            }
-        }
-        return bom.getAnyBlockState();
-    }
-}

src/api/java/baritone/api/schematic/ISchematicSystem.java

@@ -1,44 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.schematic;
-
-import baritone.api.command.registry.Registry;
-import baritone.api.schematic.format.ISchematicFormat;
-
-import java.io.File;
-import java.util.Optional;
-
-/**
- * @author Brady
- * @since 12/23/2019
- */
-public interface ISchematicSystem {
-
-    /**
-     * @return The registry of supported schematic formats
-     */
-    Registry<ISchematicFormat> getRegistry();
-
-    /**
-     * Attempts to find an {@link ISchematicFormat} that supports the specified schematic file.
-     *
-     * @param file A schematic file
-     * @return The corresponding format for the file, {@link Optional#empty()} if no candidates were found.
-     */
-    Optional<ISchematicFormat> getByFile(File file);
-}

src/api/java/baritone/api/schematic/IStaticSchematic.java

@@ -1,59 +0,0 @@
-/*
- * This file is part of Baritone.
- *
- * Baritone is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Baritone is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with Baritone.  If not, see <https://www.gnu.org/licenses/>.
- */
-
-package baritone.api.schematic;
-
-import net.minecraft.block.state.IBlockState;
-
-/**
- * A static schematic is capable of providing the desired state at a given position without
- * additional context. Schematics of this type are expected to have non-varying contents.
- *
- * @author Brady
- * @see #getDirect(int, int, int)
- * @since 12/24/2019
- */
-public interface IStaticSchematic extends ISchematic {
-
-    /**
-     * Gets the {@link IBlockState} for a given position in this schematic. It should be guaranteed
-     * that the return value of this method will not change given that the parameters are the same.
-     *
-     * @param x The X block position
-     * @param y The Y block position
-     * @param z The Z block position
-     * @return The desired state at the specified position.
-     */
-    IBlockState getDirect(int x, int y, int z);
-
-    /**
-     * Returns an {@link IBlockState} array of size {@link #heightY()} which contains all
-     * desired block states in the specified vertical column. The index of {@link IBlockState}s
-     * in the array are equivalent to their Y position in the schematic.
-     *
-     * @param x The X column position
-     * @param z The Z column position
-     * @return An {@link IBlockState} array
-     */
-    default IBlockState[] getColumn(int x, int z) {
-        IBlockState[] column = new IBlockState[this.heightY()];
-        for (int i = 0; i < this.heightY(); i++) {
-            column[i] = getDirect(x, i, z);
-        }
-        return column;
-    }
-}