Merge 33e2364053 into c843ad1268

lib/python/qmk/cli/new/keymap.py

@@ -1,6 +1,5 @@
 """This script automates the copying of the default keymap into your own keymap.
 """
-import re
 import shutil
 
 from milc import cli
@@ -14,13 +13,6 @@ from qmk.keyboard import keyboard_completer, keyboard_folder
 from qmk.userspace import UserspaceDefs
 
 
-def validate_keymap_name(name):
-    """Returns True if the given keymap name contains only a-z, 0-9 and underscore characters.
-    """
-    regex = re.compile(r'^[a-zA-Z0-9][a-zA-Z0-9_]+$')
-    return bool(regex.match(name))
-
-
 def prompt_keyboard():
     prompt = """{fg_yellow}Select Keyboard{style_reset_all}
 If you`re unsure you can view a full list of supported keyboards with {fg_yellow}qmk list-keyboards{style_reset_all}.
@@ -68,10 +60,6 @@ def new_keymap(cli):
         cli.log.error(f'Default keymap {{fg_cyan}}{keymap_path_default}{{fg_reset}} does not exist!')
         return False
 
-    if not validate_keymap_name(user_name):
-        cli.log.error('Keymap names must contain only {fg_cyan}a-z{fg_reset}, {fg_cyan}0-9{fg_reset} and {fg_cyan}_{fg_reset}! Please choose a different name.')
-        return False
-
     if keymap_path_new.exists():
         cli.log.error(f'Keymap {{fg_cyan}}{user_name}{{fg_reset}} already exists! Please choose a different name.')
         return False

lib/python/qmk/cli/via2json.py

@@ -29,7 +29,6 @@ def _convert_macros(via_macros):
     if len(via_macros) == 0:
         return list()
     split_regex = re.compile(r'(}\,)|(\,{)')
-    macro_group_regex = re.compile(r'({.+?})')
     macros = list()
     for via_macro in via_macros:
         # Split VIA macro to its elements
@@ -39,28 +38,13 @@ def _convert_macros(via_macros):
         macro_data = list()
         for m in macro:
             if '{' in m or '}' in m:
-                # Split macro groups
-                macro_groups = macro_group_regex.findall(m)
-                for macro_group in macro_groups:
-                    # Remove whitespaces and curly braces from around group
-                    macro_group = macro_group.strip(' {}')
-
-                    macro_action = 'tap'
-                    macro_keycodes = []
-
-                    if macro_group[0] == '+':
-                        macro_action = 'down'
-                        macro_keycodes.append(macro_group[1:])
-                    elif macro_group[0] == '-':
-                        macro_action = 'up'
-                        macro_keycodes.append(macro_group[1:])
-                    else:
-                        macro_keycodes.extend(macro_group.split(',') if ',' in macro_group else [macro_group])
-
-                    # Remove the KC prefixes
-                    macro_keycodes = list(map(lambda s: s.replace('KC_', ''), macro_keycodes))
-
-                    macro_data.append({"action": macro_action, "keycodes": macro_keycodes})
+                # Found keycode(s)
+                keycodes = m.split(',')
+                # Remove whitespaces and curly braces from around keycodes
+                keycodes = list(map(lambda s: s.strip(' {}'), keycodes))
+                # Remove the KC prefix
+                keycodes = list(map(lambda s: s.replace('KC_', ''), keycodes))
+                macro_data.append({"action": "tap", "keycodes": keycodes})
             else:
                 # Found text
                 macro_data.append(m)
@@ -70,13 +54,13 @@ def _convert_macros(via_macros):
 
 
 def _fix_macro_keys(keymap_data):
-    macro_no = re.compile(r'MACRO0?\(([0-9]{1,2})\)')
+    macro_no = re.compile(r'MACRO0?([0-9]{1,2})')
     for i in range(0, len(keymap_data)):
         for j in range(0, len(keymap_data[i])):
             kc = keymap_data[i][j]
             m = macro_no.match(kc)
             if m:
-                keymap_data[i][j] = f'MC_{m.group(1)}'
+                keymap_data[i][j] = f'MACRO_{m.group(1)}'
     return keymap_data