hey Marcus! many thanks for your contributions.

I've left some comments but if you don't have time to address them, I'd be happy to - I just wanted to leave some bookmarks of places I'd need to revisit.

superb work, many thanks! let me know if you want me do handle the comments.

Excellent feedback, thanks! I figured I'd get some visibility on the work so I can get your direction so I'm in line with project conventions. As you can probably tell, I'm in the semicolons-meh camp, but I'm trying hard to adjust! ;o)

I'll try to get a couple of hours in on this tonight to fix the things you mention, as well as add the missing dirs functionality, or remove it from exports altogether as you suggest if I don't have time to get around to implementing it properly.

I've changed the return values to be an array representing the stack now. Should the commands also log the stack when invoked, like their shell counterparts do? Currently they only log errors (by invoking error().)

thanks Marcus - ready to merge, unless you're still working on the tests

Some more tests wouldn't hurt I think. I still need to make sure the options have tests as well as invalid usage (e.g. pushd("/some/invalid/dir") should fail.) Feel free to merge if you'd like. In that case I'll just make another pull request with the added tests later – your call!