This content has been marked as final. Show 2 replies
In general it looks good to me. One thing that I think is missing is unit tests. Would be good to prove that new commands are working as expected.
Thank you for reviewing, I uploaded a new patch with testcase.