Switch <base/cmd> to use <base/json> rather than <starsha/note> for args file parsing

This removes the only non-starsha user of \<starsha/note>.

All args files inside the codebase at present are updated in this commit.
They __might__ be. With just leaving at `std::move(temp)` the chain of implicit constructors is less clearly correct to me. By moving it into a temporary TJson, I know that TJson is going to move construct, and I know that the temporary TJson will get move assigned. Previously the implicit was keeping us from noticing there were  copies there, so I wanted to get as far away from that as possible.