improve argparse usage

Use --fatcat-api-url instead of (ambiguous) --host-url for commands that
aren't deployed/running via systemd.

TODO: update the other --host-url usage, and either roll-out change
consistently or support the old arg as an alias during cut-over

Use argparse.ArgumentDefaultsHelpFormatter (thanks Martin!)

Add help messages for all sub-commands, both as documentation and as a
way to get argparse to print available commands in a more readable
format.

1 files changed, 12 insertions, 10 deletions

diff --git a/python/fatcat_util.py b/python/fatcat_util.py
index 08224dce..d6e76697 100755
--- a/python/fatcat_util.py
+++ b/python/fatcat_util.py
@@ -32,31 +32,33 @@ def run_editgroup_submit(args):
     args.api.update_editgroup(args.editgroup_id, eg, submit=True)
 
 def main():
-    parser = argparse.ArgumentParser()
-    parser.add_argument('--debug',
-        action='store_true',
-        help="enable debugging interface")
-    parser.add_argument('--host-url',
+    parser = argparse.ArgumentParser(
+        formatter_class=argparse.ArgumentDefaultsHelpFormatter)
+    parser.add_argument('--fatcat-api-url',
         default="http://localhost:9411/v0",
         help="connect to this host/port")
     subparsers = parser.add_subparsers()
 
-    sub_uuid2fcid = subparsers.add_parser('uuid2fcid')
+    sub_uuid2fcid = subparsers.add_parser('uuid2fcid',
+        help="convert a standard UUID (as string) to fatcat ident format")
     sub_uuid2fcid.set_defaults(func=run_uuid2fcid)
     sub_uuid2fcid.add_argument('uuid',
         help="UUID to transform")
 
-    sub_fcid2uuid = subparsers.add_parser('fcid2uuid')
+    sub_fcid2uuid = subparsers.add_parser('fcid2uuid',
+        help="convert a fatcat ident string to standard UUID format")
     sub_fcid2uuid.set_defaults(func=run_fcid2uuid)
     sub_fcid2uuid.add_argument('fcid',
         help="FCID to transform (into UUID)")
 
-    sub_editgroup_accept = subparsers.add_parser('editgroup-accept')
+    sub_editgroup_accept = subparsers.add_parser('editgroup-accept',
+        help="accept an editgroup (by ident)")
     sub_editgroup_accept.set_defaults(func=run_editgroup_accept)
     sub_editgroup_accept.add_argument('editgroup_id',
         help="editgroup to accept")
 
-    sub_editgroup_submit = subparsers.add_parser('editgroup-submit')
+    sub_editgroup_submit = subparsers.add_parser('editgroup-submit',
+        help="submit an editgroup for review (by ident)")
     sub_editgroup_submit.set_defaults(func=run_editgroup_submit)
     sub_editgroup_submit.add_argument('editgroup_id',
         help="editgroup to submit")
@@ -66,7 +68,7 @@ def main():
         print("tell me what to do!")
         sys.exit(-1)
 
-    args.api = authenticated_api(args.host_url)
+    args.api = authenticated_api(args.fatcat_api_url)
     args.func(args)
 
 if __name__ == '__main__':