I was not talking specifically about this issue but in general, it would save him a lot of troubles to have a test cases script.
I think you misunderstood what I was saying because we both know what happened with this issue.
This test script would have been run on the version in which recursion was removed (let's say 5.5), not now or for the recent beta, and since calling /join inside that custom /join wouldn't have produced the "error recursive call" message, the script would have warned that it's not working as expected. (whatever what was expected at that time, this script might not be accurate)
I took this to mean that Khaled realized he was breaking backwards compatiblity
He changed it because trying to do recursion by calling an alias as an identifier and as a command would produce differents behaviors, but regardless, calling the built-in was not correct in this situation.
