Skip to content

Latest commit

 

History

History
87 lines (58 loc) · 4.08 KB

CONTRIBUTING.md

File metadata and controls

87 lines (58 loc) · 4.08 KB
Raw

Contribute

Mineflayer has originally been made mostly by andrewrk but has since then be improved and fixed by many contributors. That's why it is important to know the best ways to contribute to mineflayer.

Issue organization

We have 3 stage labels to try to organize issues:

  • Stage 1: just created by someone new to the project, we don't know yet if it deserves an implementation / a fix
  • Stage 2: promising idea, but needs more thinking before implementation
  • Stage 3: idea is precisely specified, only coding is left to do

Links like https://github.com/PrismarineJS/mineflayer/issues?q=is%3Aopen+is%3Aissue+-label%3AStage1 can be used to filter out stage 1 if you're looking for things that are ready for contribution

Creating tests

Mineflayer has two kind of tests :

  • internal tests : tests that are done against a simple server created with node-minecraft-protocol
  • external tests : tests that are done against the vanilla server

The objective of these tests is to know automatically what works and what doesn't in mineflayer, so it's easier to make mineflayer work.

Creating an external test

In order to add an external test now you only need to create a file in test/externalTests

An example : test/externalTests/digAndBuild.js

That file needs to export a function returning a function or an array of function taking as parameter the bot object and a done callback, it should contain asserts to test if the tested functionality failed.

Creating a third party plugin

Mineflayer is pluggable; anyone can create a plugin that adds an even higher level API on top of Mineflayer.

Several such third party plugins have already been created

In order to create a new one you need to :

  1. create a new repo
  2. in your index.js file, exports an init function taking in argument mineflayer (example)
  3. that function returns a inject function taking in argument the bot object (example)
  4. that inject function add functionalities to the bot object (example)

Since the mineflayer object is passed in parameter, that new package doesn't need to depend on mineflayer (no mineflayer dependency in the package.json)

See a full example here.

Reporting bugs

Mineflayer works well for most usages, but it sometimes still has bugs.

When finding one it's best to report an issue providing these information :

  • what you want to do (the objective in english)
  • what you tried (the code)
  • what happened
  • what you expected to happen

Mineflayer code

Some things to think about when submitting a Pull Request or making a commit :

Error handling

In most cases, mineflayer shouldn't crash the bot. Even if something fails, the bot can take an alternative route to get to its objective.

What that means is we shouldn't use throw(new Error("error")) but instead use the node.js convention of passing the error in the callback.

For example :

function myfunction (param1, callback) {
  // do stuff
  let toDo = 1
  toDo = 2
  if (toDo === 2) { // everything worked
    callback()
  } else {
    callback(new Error('something failed'))
  }
}

See an other example of that in mineflayer code

Updating the documentation

The table of content of docs/api.md is made with doctoc. After updating that file, you should run doctoc docs/api.md to update the table of content.