Steps to reproduce:

The current document is not up-to-date regarding the category, description, and MACH_COMMANDS requirements of creating a new mach command.

Actual results:

Following the doc leads to errors

Expected results:

Update the doc with the newer category, description, and MACH_COMMANDS requirements. Please see attached file.

I have also updated the code in the Conditionally Filtering Commands section.

The Minimizing Code in Commands section probably needs to be rewritten because using the new MACH_COMMANDS dictionary, I assume, not all mach commands are loaded every time mach is called. Probably best left to another bug report.

We don't use bugzilla for code review. please submit your changes on https://phabricator.services.mozilla.com/
See:
https://firefox-source-docs.mozilla.org/contributing/contribution_quickref.html

Update the doc with the newer category, description, and MACH_COMMANDS requirements.

Remove reference to Mozilla source directory.
Emphasise alphabetical order of MACH_COMMANDS dictionary.

There is an r+ patch which didn't land and no activity in this bug for 2 weeks.
:adhaliwal34, could you have a look please?
If you still have some work to do, you can add an action "Plan Changes" in Phabricator.
For more information, please visit BugBot documentation.

I will commandeer this and get it landed.

https://hg.mozilla.org/mozilla-central/rev/e6460f25f44d