more cleanup

eliminate need to call read_json. the constructor does it now
and silently fails in the case where you are creating a new file

traversal returns item index now in addition to the args

remove run() it needs more work to simplify using save/load item
in it

remove warning for files not in filename arg set

demonstrate read and write patterns in test