Replaced ctrl.legacy w/ key prefix

Author: adamlui

remove-json-keys/src/remove_json_keys/lib/settings.py

@@ -26,24 +26,24 @@
         args=['--docs'], action='store_true', exit=True, handler=lambda cli: url.open(cli.urls.docs)),
     debug=sn(
         args=['-V', '--debug'], nargs='?', const=True, metavar='TARGET_KEY' ),
-    no_wizard_legacy=sn(
-        legacy=True, args=['-W',])
+    legacy_no_wizard=sn(
+        replaced_by='no_wizard', args=['-W'])
 )
 
 def load(cli):
     cli.config = sn()
 
     # Assign help tips from cli.msgs
     for ctrl_key, ctrl in vars(controls).items():
-        if getattr(ctrl, 'legacy', False) : continue
+        if ctrl_key.startswith('legacy') : continue
         if not hasattr(ctrl, 'help') : ctrl.help = getattr(cli.msgs, f'help_{ctrl_key.upper()}')
 
     # Parse CLI args
     argp = argparse.ArgumentParser(description=cli.description, add_help=False)
     for ctrl_key, ctrl in vars(controls).items(): # add args to argp
         kwargs = ctrl.__dict__.copy()
         args = kwargs.pop('args')
-        if getattr(ctrl, 'legacy', False):
+        if ctrl_key.startswith('legacy'):
             for arg in args:
                 if arg in sys.argv:
                     log.warn(f'{cli.msgs.warn_OPTION} {arg} {cli.msgs.warn_NO_LONGER_HAS_ANY_EFFECT}.')
@@ -54,8 +54,8 @@ def load(cli):
         argp.add_argument(*args, **argparse_kwargs)
     parsed_args, unknown_args = argp.parse_known_args()
     exempt_flags = [] # exempt dashless + legacy args from validation
-    for ctrl in vars(controls).values():
-        if getattr(ctrl, 'subcmd', False) or getattr(ctrl, 'legacy', False):
+    for ctrl_key, ctrl in vars(controls).items():
+        if getattr(ctrl, 'subcmd', False) or ctrl_key.startswith('legacy'):
             for arg in ctrl.args : exempt_flags.append(arg)
     if unknown_args and not all(any(arg.startswith(exempt) for exempt in exempt_flags) for arg in unknown_args):
         log.cmd_docs_url_exit(cli, f"{cli.msgs.err_UNRECOGNIZED_ARGS}: {' '.join(unknown_args)}", cmd='help')

translate-messages/src/translate_messages/lib/settings.py

@@ -36,24 +36,24 @@
         args=['--docs'], action='store_true', exit=True, handler=lambda cli: url.open(cli.urls.docs)),
     debug=sn(
         args=['-V', '--debug'], nargs='?', const=True, metavar='TARGET_KEY' ),
-    no_wizard_legacy=sn(
-        legacy=True, args=['-W',])
+    legacy_no_wizard=sn(
+        replaced_by='no_wizard', args=['-W'])
 )
 
 def load(cli):
     cli.config = sn()
 
     # Assign help tips from cli.msgs
     for ctrl_key, ctrl in vars(controls).items():
-        if getattr(ctrl, 'legacy', False) : continue
+        if ctrl_key.startswith('legacy') : continue
         if not hasattr(ctrl, 'help') : ctrl.help = getattr(cli.msgs, f'help_{ctrl_key.upper()}')
 
     # Parse CLI args
     argp = argparse.ArgumentParser(description=cli.description, add_help=False)
     for ctrl_key, ctrl in vars(controls).items(): # add args to argp
         kwargs = ctrl.__dict__.copy()
         args = kwargs.pop('args')
-        if getattr(ctrl, 'legacy', False):
+        if ctrl_key.startswith('legacy'):
             for arg in args:
                 if arg in sys.argv:
                     log.warn(f'{cli.msgs.warn_OPTION} {arg} {cli.msgs.warn_NO_LONGER_HAS_ANY_EFFECT}.')
@@ -64,8 +64,8 @@ def load(cli):
         argp.add_argument(*args, **argparse_kwargs)
     parsed_args, unknown_args = argp.parse_known_args()
     exempt_flags = [] # exempt dashless + legacy args from validation
-    for ctrl in vars(controls).values():
-        if getattr(ctrl, 'subcmd', False) or getattr(ctrl, 'legacy', False):
+    for ctrl_key, ctrl in vars(controls).items():
+        if getattr(ctrl, 'subcmd', False) or ctrl_key.startswith('legacy'):
             for arg in ctrl.args : exempt_flags.append(arg)
     if unknown_args and not all(any(arg.startswith(exempt) for exempt in exempt_flags) for arg in unknown_args):
         log.cmd_docs_url_exit(cli, f"{cli.msgs.err_UNRECOGNIZED_ARGS}: {' '.join(unknown_args)}", cmd='help')