diff options
Diffstat (limited to 'lib/python/qmk/cli')
| -rw-r--r-- | lib/python/qmk/cli/__init__.py | 5 | ||||
| -rw-r--r-- | lib/python/qmk/cli/doctor/check.py | 2 | ||||
| -rwxr-xr-x | lib/python/qmk/cli/doctor/main.py | 2 | ||||
| -rw-r--r-- | lib/python/qmk/cli/flash.py | 46 | ||||
| -rwxr-xr-x | lib/python/qmk/cli/generate/api.py | 9 | ||||
| -rw-r--r-- | lib/python/qmk/cli/generate/keycodes.py | 55 | ||||
| -rw-r--r-- | lib/python/qmk/cli/generate/keycodes_tests.py | 39 | ||||
| -rw-r--r-- | lib/python/qmk/cli/git/__init__.py | 0 | ||||
| -rw-r--r-- | lib/python/qmk/cli/git/submodule.py | 22 | ||||
| -rw-r--r-- | lib/python/qmk/cli/list/keyboards.py | 3 | ||||
| -rw-r--r-- | lib/python/qmk/cli/migrate.py | 81 | ||||
| -rw-r--r-- | lib/python/qmk/cli/new/keyboard.py | 10 | ||||
| -rwxr-xr-x | lib/python/qmk/cli/new/keymap.py | 56 | 
13 files changed, 281 insertions, 49 deletions
| diff --git a/lib/python/qmk/cli/__init__.py b/lib/python/qmk/cli/__init__.py index 1da4d25741..16edf88ad6 100644 --- a/lib/python/qmk/cli/__init__.py +++ b/lib/python/qmk/cli/__init__.py @@ -57,9 +57,11 @@ subcommands = [      'qmk.cli.generate.keyboard_c',      'qmk.cli.generate.keyboard_h',      'qmk.cli.generate.keycodes', +    'qmk.cli.generate.keycodes_tests',      'qmk.cli.generate.rgb_breathe_table',      'qmk.cli.generate.rules_mk',      'qmk.cli.generate.version_h', +    'qmk.cli.git.submodule',      'qmk.cli.hello',      'qmk.cli.import.kbfirmware',      'qmk.cli.import.keyboard', @@ -67,11 +69,12 @@ subcommands = [      'qmk.cli.info',      'qmk.cli.json2c',      'qmk.cli.lint', +    'qmk.cli.kle2json',      'qmk.cli.list.keyboards',      'qmk.cli.list.keymaps',      'qmk.cli.list.layouts', -    'qmk.cli.kle2json',      'qmk.cli.mass_compile', +    'qmk.cli.migrate',      'qmk.cli.multibuild',      'qmk.cli.new.keyboard',      'qmk.cli.new.keymap', diff --git a/lib/python/qmk/cli/doctor/check.py b/lib/python/qmk/cli/doctor/check.py index 8a0422ba72..a368242fdb 100644 --- a/lib/python/qmk/cli/doctor/check.py +++ b/lib/python/qmk/cli/doctor/check.py @@ -119,10 +119,8 @@ def check_submodules():      """      for submodule in submodules.status().values():          if submodule['status'] is None: -            cli.log.error('Submodule %s has not yet been cloned!', submodule['name'])              return CheckStatus.ERROR          elif not submodule['status']: -            cli.log.warning('Submodule %s is not up to date!', submodule['name'])              return CheckStatus.WARNING      return CheckStatus.OK diff --git a/lib/python/qmk/cli/doctor/main.py b/lib/python/qmk/cli/doctor/main.py index 1600ab8dd4..d55a11e5fd 100755 --- a/lib/python/qmk/cli/doctor/main.py +++ b/lib/python/qmk/cli/doctor/main.py @@ -142,7 +142,7 @@ def doctor(cli):      if sub_ok == CheckStatus.OK:          cli.log.info('Submodules are up to date.')      else: -        if yesno('Would you like to clone the submodules?', default=True): +        if git_check_repo() and yesno('Would you like to clone the submodules?', default=True):              submodules.update()              sub_ok = check_submodules() diff --git a/lib/python/qmk/cli/flash.py b/lib/python/qmk/cli/flash.py index 40bfbdab56..52defb5f0d 100644 --- a/lib/python/qmk/cli/flash.py +++ b/lib/python/qmk/cli/flash.py @@ -11,12 +11,14 @@ import qmk.path  from qmk.decorators import automagic_keyboard, automagic_keymap  from qmk.commands import compile_configurator_json, create_make_command, parse_configurator_json, build_environment  from qmk.keyboard import keyboard_completer, keyboard_folder +from qmk.keymap import keymap_completer  from qmk.flashers import flasher -def print_bootloader_help(): +def _list_bootloaders():      """Prints the available bootloaders listed in docs.qmk.fm.      """ +    cli.print_help()      cli.log.info('Here are the available bootloaders:')      cli.echo('\tavrdude')      cli.echo('\tbootloadhid') @@ -36,14 +38,29 @@ def print_bootloader_help():      cli.echo('\tuf2-split-left')      cli.echo('\tuf2-split-right')      cli.echo('For more info, visit https://docs.qmk.fm/#/flashing') +    return False + + +def _flash_binary(filename, mcu): +    """Try to flash binary firmware +    """ +    cli.echo('Flashing binary firmware...\nPlease reset your keyboard into bootloader mode now!\nPress Ctrl-C to exit.\n') +    try: +        err, msg = flasher(mcu, filename) +        if err: +            cli.log.error(msg) +            return False +    except KeyboardInterrupt: +        cli.log.info('Ctrl-C was pressed, exiting...') +    return True  @cli.argument('filename', nargs='?', arg_only=True, type=qmk.path.FileType('r'), completer=FilesCompleter('.json'), help='A configurator export JSON to be compiled and flashed or a pre-compiled binary firmware file (bin/hex) to be flashed.')  @cli.argument('-b', '--bootloaders', action='store_true', help='List the available bootloaders.')  @cli.argument('-bl', '--bootloader', default='flash', help='The flash command, corresponding to qmk\'s make options of bootloaders.')  @cli.argument('-m', '--mcu', help='The MCU name. Required for HalfKay, HID, USBAspLoader and ISP flashing.') -@cli.argument('-km', '--keymap', help='The keymap to build a firmware for. Use this if you dont have a configurator file. Ignored when a configurator file is supplied.') -@cli.argument('-kb', '--keyboard', type=keyboard_folder, completer=keyboard_completer, help='The keyboard to build a firmware for. Use this if you dont have a configurator file. Ignored when a configurator file is supplied.') +@cli.argument('-kb', '--keyboard', type=keyboard_folder, completer=keyboard_completer, help='The keyboard to build a firmware for. Ignored when a configurator export is supplied.') +@cli.argument('-km', '--keymap', completer=keymap_completer, help='The keymap to build a firmware for. Ignored when a configurator export is supplied.')  @cli.argument('-n', '--dry-run', arg_only=True, action='store_true', help="Don't actually build, just show the make command to be run.")  @cli.argument('-j', '--parallel', type=int, default=1, help="Set the number of parallel make jobs; 0 means unlimited.")  @cli.argument('-e', '--env', arg_only=True, action='append', default=[], help="Set a variable to be passed to make. May be passed multiple times.") @@ -56,30 +73,17 @@ def flash(cli):      If a binary firmware is supplied, try to flash that. -    If a Configurator JSON export is supplied this command will create a new keymap. Keymap and Keyboard arguments -    will be ignored. +    If a Configurator export is supplied this command will create a new keymap, overwriting an existing keymap if one exists. -    If no file is supplied, keymap and keyboard are expected. +    If a keyboard and keymap are provided this command will build a firmware based on that.      If bootloader is omitted the make system will use the configured bootloader for that keyboard.      """ -    if cli.args.filename and cli.args.filename.suffix in ['.bin', '.hex']: -        # Try to flash binary firmware -        cli.echo('Flashing binary firmware...\nPlease reset your keyboard into bootloader mode now!\nPress Ctrl-C to exit.\n') -        try: -            err, msg = flasher(cli.args.mcu, cli.args.filename) -            if err: -                cli.log.error(msg) -                return False -        except KeyboardInterrupt: -            cli.log.info('Ctrl-C was pressed, exiting...') -        return True +    if cli.args.filename and cli.args.filename.suffix in ['.bin', '.hex', '.uf2']: +        return _flash_binary(cli.args.filename, cli.args.mcu)      if cli.args.bootloaders: -        # Provide usage and list bootloaders -        cli.print_help() -        print_bootloader_help() -        return False +        return _list_bootloaders()      # Build the environment vars      envs = build_environment(cli.args.env) diff --git a/lib/python/qmk/cli/generate/api.py b/lib/python/qmk/cli/generate/api.py index 8650a36b84..dd4830f543 100755 --- a/lib/python/qmk/cli/generate/api.py +++ b/lib/python/qmk/cli/generate/api.py @@ -11,7 +11,7 @@ from qmk.info import info_json  from qmk.json_encoders import InfoJSONEncoder  from qmk.json_schema import json_load  from qmk.keyboard import find_readme, list_keyboards -from qmk.keycodes import load_spec, list_versions +from qmk.keycodes import load_spec, list_versions, list_languages  DATA_PATH = Path('data')  TEMPLATE_PATH = DATA_PATH / 'templates/api/' @@ -44,6 +44,13 @@ def _resolve_keycode_specs(output_folder):          output_file = output_folder / f'constants/keycodes_{version}.json'          output_file.write_text(json.dumps(overall, indent=4), encoding='utf-8') +    for lang in list_languages(): +        for version in list_versions(lang): +            overall = load_spec(version, lang) + +            output_file = output_folder / f'constants/keycodes_{lang}_{version}.json' +            output_file.write_text(json.dumps(overall, indent=4), encoding='utf-8') +      # Purge files consumed by 'load_spec'      shutil.rmtree(output_folder / 'constants/keycodes/') diff --git a/lib/python/qmk/cli/generate/keycodes.py b/lib/python/qmk/cli/generate/keycodes.py index 29b7db3c80..2ed84cd589 100644 --- a/lib/python/qmk/cli/generate/keycodes.py +++ b/lib/python/qmk/cli/generate/keycodes.py @@ -8,6 +8,24 @@ from qmk.path import normpath  from qmk.keycodes import load_spec +def _render_key(key): +    width = 7 +    if 'S(' in key: +        width += len('S()') +    if 'A(' in key: +        width += len('A()') +    if 'RCTL(' in key: +        width += len('RCTL()') +    if 'ALGR(' in key: +        width += len('ALGR()') +    return key.ljust(width) + + +def _render_label(label): +    label = label.replace("\\", "(backslash)") +    return label + +  def _generate_ranges(lines, keycodes):      lines.append('')      lines.append('enum qk_keycode_ranges {') @@ -67,6 +85,23 @@ def _generate_helpers(lines, keycodes):          lines.append(f'#define IS_{ group.upper() }_KEYCODE(code) ((code) >= {lo} && (code) <= {hi})') +def _generate_aliases(lines, keycodes): +    lines.append('') +    lines.append('// Aliases') +    for key, value in keycodes["aliases"].items(): +        define = _render_key(value.get("key")) +        val = _render_key(key) +        if 'label' in value: +            lines.append(f'#define {define} {val} // {_render_label(value.get("label"))}') +        else: +            lines.append(f'#define {define} {val}') + +    lines.append('') +    for key, value in keycodes["aliases"].items(): +        for alias in value.get("aliases", []): +            lines.append(f'#define {alias} {value.get("key")}') + +  @cli.argument('-v', '--version', arg_only=True, required=True, help='Version of keycodes to generate.')  @cli.argument('-o', '--output', arg_only=True, type=normpath, help='File to write to')  @cli.argument('-q', '--quiet', arg_only=True, action='store_true', help="Quiet mode, only output error messages") @@ -86,3 +121,23 @@ def generate_keycodes(cli):      # Show the results      dump_lines(cli.args.output, keycodes_h_lines, cli.args.quiet) + + +@cli.argument('-v', '--version', arg_only=True, required=True, help='Version of keycodes to generate.') +@cli.argument('-l', '--lang', arg_only=True, required=True, help='Language of keycodes to generate.') +@cli.argument('-o', '--output', arg_only=True, type=normpath, help='File to write to') +@cli.argument('-q', '--quiet', arg_only=True, action='store_true', help="Quiet mode, only output error messages") +@cli.subcommand('Used by the make system to generate keymap_{lang}.h from keycodes_{lang}_{version}.json', hidden=True) +def generate_keycode_extras(cli): +    """Generates the header file. +    """ + +    # Build the header file. +    keycodes_h_lines = [GPL2_HEADER_C_LIKE, GENERATED_HEADER_C_LIKE, '#pragma once', '#include "keymap.h"', '// clang-format off'] + +    keycodes = load_spec(cli.args.version, cli.args.lang) + +    _generate_aliases(keycodes_h_lines, keycodes) + +    # Show the results +    dump_lines(cli.args.output, keycodes_h_lines, cli.args.quiet) diff --git a/lib/python/qmk/cli/generate/keycodes_tests.py b/lib/python/qmk/cli/generate/keycodes_tests.py new file mode 100644 index 0000000000..453b4693a7 --- /dev/null +++ b/lib/python/qmk/cli/generate/keycodes_tests.py @@ -0,0 +1,39 @@ +"""Used by the make system to generate a keycode lookup table from keycodes_{version}.json +""" +from milc import cli + +from qmk.constants import GPL2_HEADER_C_LIKE, GENERATED_HEADER_C_LIKE +from qmk.commands import dump_lines +from qmk.path import normpath +from qmk.keycodes import load_spec + + +def _generate_defines(lines, keycodes): +    lines.append('') +    lines.append('std::map<uint16_t, std::string> KEYCODE_ID_TABLE = {') +    for key, value in keycodes["keycodes"].items(): +        lines.append(f'    {{{value.get("key")}, "{value.get("key")}"}},') +    lines.append('};') + + +@cli.argument('-v', '--version', arg_only=True, required=True, help='Version of keycodes to generate.') +@cli.argument('-o', '--output', arg_only=True, type=normpath, help='File to write to') +@cli.argument('-q', '--quiet', arg_only=True, action='store_true', help="Quiet mode, only output error messages") +@cli.subcommand('Used by the make system to generate a keycode lookup table from keycodes_{version}.json', hidden=True) +def generate_keycodes_tests(cli): +    """Generates a keycode to identifier lookup table for unit test output. +    """ + +    # Build the keycodes.h file. +    keycodes_h_lines = [GPL2_HEADER_C_LIKE, GENERATED_HEADER_C_LIKE, '// clang-format off'] +    keycodes_h_lines.append('extern "C" {\n#include <keycode.h>\n}') +    keycodes_h_lines.append('#include <map>') +    keycodes_h_lines.append('#include <string>') +    keycodes_h_lines.append('#include <cstdint>') + +    keycodes = load_spec(cli.args.version) + +    _generate_defines(keycodes_h_lines, keycodes) + +    # Show the results +    dump_lines(cli.args.output, keycodes_h_lines, cli.args.quiet) diff --git a/lib/python/qmk/cli/git/__init__.py b/lib/python/qmk/cli/git/__init__.py new file mode 100644 index 0000000000..e69de29bb2 --- /dev/null +++ b/lib/python/qmk/cli/git/__init__.py diff --git a/lib/python/qmk/cli/git/submodule.py b/lib/python/qmk/cli/git/submodule.py new file mode 100644 index 0000000000..bcc3868a52 --- /dev/null +++ b/lib/python/qmk/cli/git/submodule.py @@ -0,0 +1,22 @@ +import shutil +from qmk.path import normpath + +from milc import cli + +REMOVE_DIRS = [ +    'lib/ugfx', +    'lib/pico-sdk', +    'lib/chibios-contrib/ext/mcux-sdk', +    'lib/lvgl', +] + + +@cli.subcommand('Git Submodule actions.') +def git_submodule(cli): +    for folder in REMOVE_DIRS: +        if normpath(folder).is_dir(): +            print(f"Removing '{folder}'") +            shutil.rmtree(folder) + +    cli.run(['git', 'submodule', 'sync', '--recursive'], capture_output=False) +    cli.run(['git', 'submodule', 'update', '--init', '--recursive', '--progress'], capture_output=False) diff --git a/lib/python/qmk/cli/list/keyboards.py b/lib/python/qmk/cli/list/keyboards.py index 8b6c451673..405b9210e4 100644 --- a/lib/python/qmk/cli/list/keyboards.py +++ b/lib/python/qmk/cli/list/keyboards.py @@ -5,9 +5,10 @@ from milc import cli  import qmk.keyboard +@cli.argument('--no-resolve-defaults', arg_only=True, action='store_false', help='Ignore any "DEFAULT_FOLDER" within keyboards rules.mk')  @cli.subcommand("List the keyboards currently defined within QMK")  def list_keyboards(cli):      """List the keyboards currently defined within QMK      """ -    for keyboard_name in qmk.keyboard.list_keyboards(): +    for keyboard_name in qmk.keyboard.list_keyboards(cli.args.no_resolve_defaults):          print(keyboard_name) diff --git a/lib/python/qmk/cli/migrate.py b/lib/python/qmk/cli/migrate.py new file mode 100644 index 0000000000..4164f9c8ad --- /dev/null +++ b/lib/python/qmk/cli/migrate.py @@ -0,0 +1,81 @@ +"""Migrate keyboard configuration to "Data Driven" +""" +import json +from pathlib import Path +from dotty_dict import dotty + +from milc import cli + +from qmk.keyboard import keyboard_completer, keyboard_folder, resolve_keyboard +from qmk.info import info_json, find_info_json +from qmk.json_encoders import InfoJSONEncoder +from qmk.json_schema import json_load + + +def _candidate_files(keyboard): +    kb_dir = Path(resolve_keyboard(keyboard)) + +    cur_dir = Path('keyboards') +    files = [] +    for dir in kb_dir.parts: +        cur_dir = cur_dir / dir +        files.append(cur_dir / 'config.h') +        files.append(cur_dir / 'rules.mk') + +    return [file for file in files if file.exists()] + + +@cli.argument('-f', '--filter', arg_only=True, action='append', default=[], help="Filter the performed migrations based on the supplied value. Supported format is 'KEY' located from 'data/mappings'. May be passed multiple times.") +@cli.argument('-kb', '--keyboard', arg_only=True, type=keyboard_folder, completer=keyboard_completer, required=True, help='The keyboard\'s name') +@cli.subcommand('Migrate keyboard config to "Data Driven".', hidden=True) +def migrate(cli): +    """Migrate keyboard configuration to "Data Driven" +    """ +    # Merge mappings as we do not care to where "KEY" is found just that its removed +    info_config_map = json_load(Path('data/mappings/info_config.hjson')) +    info_rules_map = json_load(Path('data/mappings/info_rules.hjson')) +    info_map = {**info_config_map, **info_rules_map} + +    # Parse target info.json which will receive updates +    target_info = Path(find_info_json(cli.args.keyboard)[0]) +    info_data = dotty(json_load(target_info)) + +    # Already parsed used for updates +    kb_info_json = dotty(info_json(cli.args.keyboard)) + +    # List of candidate files +    files = _candidate_files(cli.args.keyboard) + +    # Filter down keys if requested +    keys = info_map.keys() +    if cli.args.filter: +        keys = list(set(keys) & set(cli.args.filter)) + +    cli.log.info(f'{{fg_green}}Migrating keyboard {{fg_cyan}}{cli.args.keyboard}{{fg_green}}.{{fg_reset}}') + +    # Start migration +    for file in files: +        cli.log.info(f'  Migrating file {file}') +        file_contents = file.read_text(encoding='utf-8').split('\n') +        for key in keys: +            for num, line in enumerate(file_contents): +                if line.startswith(f'{key} =') or line.startswith(f'#define {key} '): +                    cli.log.info(f'    Migrating {key}...') + +                    while line.rstrip().endswith('\\'): +                        file_contents.pop(num) +                        line = file_contents[num] +                    file_contents.pop(num) + +                    update_key = info_map[key]["info_key"] +                    if update_key in kb_info_json: +                        info_data[update_key] = kb_info_json[update_key] + +        file.write_text('\n'.join(file_contents), encoding='utf-8') + +    # Finally write out updated info.json +    cli.log.info(f'  Updating {target_info}') +    target_info.write_text(json.dumps(info_data.to_dict(), cls=InfoJSONEncoder)) + +    cli.log.info(f'{{fg_green}}Migration of keyboard {{fg_cyan}}{cli.args.keyboard}{{fg_green}} complete!{{fg_reset}}') +    cli.log.info(f"Verify build with {{fg_yellow}}qmk compile -kb {cli.args.keyboard} -km default{{fg_reset}}.") diff --git a/lib/python/qmk/cli/new/keyboard.py b/lib/python/qmk/cli/new/keyboard.py index 251ad919dd..cdd3919168 100644 --- a/lib/python/qmk/cli/new/keyboard.py +++ b/lib/python/qmk/cli/new/keyboard.py @@ -195,11 +195,6 @@ def new_keyboard(cli):      cli.echo('')      kb_name = cli.args.keyboard if cli.args.keyboard else prompt_keyboard() -    user_name = cli.config.new_keyboard.name if cli.config.new_keyboard.name else prompt_user() -    real_name = cli.args.realname or cli.config.new_keyboard.name if cli.args.realname or cli.config.new_keyboard.name else prompt_name(user_name) -    default_layout = cli.args.layout if cli.args.layout else prompt_layout() -    mcu = cli.args.type if cli.args.type else prompt_mcu() -      if not validate_keyboard_name(kb_name):          cli.log.error('Keyboard names must contain only {fg_cyan}lowercase a-z{fg_reset}, {fg_cyan}0-9{fg_reset}, and {fg_cyan}_{fg_reset}! Please choose a different name.')          return 1 @@ -208,6 +203,11 @@ def new_keyboard(cli):          cli.log.error(f'Keyboard {{fg_cyan}}{kb_name}{{fg_reset}} already exists! Please choose a different name.')          return 1 +    user_name = cli.config.new_keyboard.name if cli.config.new_keyboard.name else prompt_user() +    real_name = cli.args.realname or cli.config.new_keyboard.name if cli.args.realname or cli.config.new_keyboard.name else prompt_name(user_name) +    default_layout = cli.args.layout if cli.args.layout else prompt_layout() +    mcu = cli.args.type if cli.args.type else prompt_mcu() +      # Preprocess any development_board presets      if mcu in dev_boards:          defaults_map = json_load(Path('data/mappings/defaults.hjson')) diff --git a/lib/python/qmk/cli/new/keymap.py b/lib/python/qmk/cli/new/keymap.py index 60cb743cb6..e7823bc46d 100755 --- a/lib/python/qmk/cli/new/keymap.py +++ b/lib/python/qmk/cli/new/keymap.py @@ -1,12 +1,32 @@  """This script automates the copying of the default keymap into your own keymap.  """  import shutil -from pathlib import Path -import qmk.path +from milc import cli +from milc.questions import question + +from qmk.path import is_keyboard, keymap +from qmk.git import git_get_username  from qmk.decorators import automagic_keyboard, automagic_keymap  from qmk.keyboard import keyboard_completer, keyboard_folder -from milc import cli + + +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}. + +Keyboard Name? """ + +    return question(prompt) + + +def prompt_user(): +    prompt = """ +{fg_yellow}Name Your Keymap{style_reset_all} +Used for maintainer, copyright, etc + +Your GitHub Username? """ +    return question(prompt, default=git_get_username())  @cli.argument('-kb', '--keyboard', type=keyboard_folder, completer=keyboard_completer, help='Specify keyboard name. Example: 1upkeyboards/1up60hse') @@ -17,32 +37,34 @@ from milc import cli  def new_keymap(cli):      """Creates a new keymap for the keyboard of your choosing.      """ -    # ask for user input if keyboard or keymap was not provided in the command line -    keyboard = cli.config.new_keymap.keyboard if cli.config.new_keymap.keyboard else input("Keyboard Name: ") -    keymap = cli.config.new_keymap.keymap if cli.config.new_keymap.keymap else input("Keymap Name: ") +    cli.log.info('{style_bright}Generating a new keymap{style_normal}') +    cli.echo('') -    # generate keymap paths -    kb_path = Path('keyboards') / keyboard -    keymap_path = qmk.path.keymap(keyboard) -    keymap_path_default = keymap_path / 'default' -    keymap_path_new = keymap_path / keymap +    # ask for user input if keyboard or keymap was not provided in the command line +    kb_name = cli.config.new_keymap.keyboard if cli.config.new_keymap.keyboard else prompt_keyboard() +    user_name = cli.config.new_keymap.keymap if cli.config.new_keymap.keymap else prompt_user()      # check directories -    if not kb_path.exists(): -        cli.log.error('Keyboard %s does not exist!', kb_path) +    if not is_keyboard(kb_name): +        cli.log.error(f'Keyboard {{fg_cyan}}{kb_name}{{fg_reset}} does not exist! Please choose a valid name.')          return False +    # generate keymap paths +    km_path = keymap(kb_name) +    keymap_path_default = km_path / 'default' +    keymap_path_new = km_path / user_name +      if not keymap_path_default.exists(): -        cli.log.error('Keyboard default %s does not exist!', keymap_path_default) +        cli.log.error(f'Default keymap {{fg_cyan}}{keymap_path_default}{{fg_reset}} does not exist!')          return False      if keymap_path_new.exists(): -        cli.log.error('Keymap %s already exists!', keymap_path_new) +        cli.log.error(f'Keymap {{fg_cyan}}{user_name}{{fg_reset}} already exists! Please choose a different name.')          return False      # create user directory with default keymap files      shutil.copytree(keymap_path_default, keymap_path_new, symlinks=True)      # end message to user -    cli.log.info("%s keymap directory created in: %s", keymap, keymap_path_new) -    cli.log.info("Compile a firmware with your new keymap by typing: \n\n\tqmk compile -kb %s -km %s\n", keyboard, keymap) +    cli.log.info(f'{{fg_green}}Created a new keymap called {{fg_cyan}}{user_name}{{fg_green}} in: {{fg_cyan}}{keymap_path_new}.{{fg_reset}}') +    cli.log.info(f"Compile a firmware with your new keymap by typing: {{fg_yellow}}qmk compile -kb {kb_name} -km {user_name}{{fg_reset}}.") | 
