[ERROR] on-window-detected[37].on-window-detected: Unknown key

[kniziol]

Body

Steps to reproduce

1. Navigate to https://nikitabobko.github.io/AeroSpace/guide#on-window-detected-callback
2. Scroll to Examples of automations
3. Copy the following to your existing aerospace.toml

on-window-detected = [
    {
        if = 'test %{app-bundle-id} = org.alacritty',
        run = 'move-node-to-workspace T', # mnemonics T - Terminal
    },
    {
        if = 'test %{app-bundle-id} = com.google.Chrome || test %{app-bundle-id} = org.mozilla.firefox',
        run = 'move-node-to-workspace W', # mnemonics W - Web browser
    },
    {
        if = 'test %{app-bundle-id} = com.jetbrains.intellij',
        run = 'move-node-to-workspace I', # mnemonics I - IDE
    }
]

4. Reload configuration of Aerospace

Expected result

No error

Actual result

Failed to parse '~/.config/aerospace/aerospace.toml'. 1 error(s). 0 warning(s)
[ERROR] on-window-detected[37].on-window-detected: Unknown key

Additional info 1

#660 (comment)

Additional info 2

$ aerospace -v

aerospace CLI client version: 0.21.0-Beta dd6b927a299c3af3e5760c4c3fc6012b984f9e51
AeroSpace.app server version: 0.21.0-Beta dd6b927a299c3af3e5760c4c3fc6012b984f9e51

[nikitabobko]

You are having a TOML moment. It's not AeroSpace problem. Please exclusively use either [[on-window-detected]] or on-window-detected = [...]. Both won't work

[iandol]

Even if you only use on-window-detected = [], now this key must be placed above other [tables], as TOML cannot use root table keys below sub tables (there is no terminator on a TOML table so it absorbs a following key). See #2148

[kniziol]

@nikitabobko @iandol I’ve modified my configuration based on your information, the default configuration, and the TOML documentation. Now, it works. Thanks.

[kniziol]

It's not AeroSpace problem

@nikitabobko IMHO, it’s partially an AeroSpace issue because some users are still using the "old" TOML format in their configuration files. They’re unaware of the latest changes implemented to the configuration, including those made to the default config.

Updating AeroSpace isn’t the only thing that needs to be done for the end user. They also need to update (or even convert) their AeroSpace configuration. Speaking of that, it would be great if there was a tool or some information in the documentation on how to migrate AeroSpace configuration to use all the latest features and the fresh/latest TOML configuration format.

[kniziol]

Finally, I've solved the issue. It’s beneficial to compare your configuration with the default configuration, and review TOML v1.1.0 documentation.

Here’s part of my configuration (just for reference)

# Config version for compatibility and deprecations
# See: https://nikitabobko.github.io/AeroSpace/guide#config-version
# Fallback value (if you omit the key): config-version = 1
config-version = 2

# Start AeroSpace at login
start-at-login = true

# Normalizations. See: https://nikitabobko.github.io/AeroSpace/guide#normalization
enable-normalization-flatten-containers = true
enable-normalization-opposite-orientation-for-nested-containers = true

# Windows arrangement per workspace
#
on-window-detected = [
    # Floating windows - without workspace assigned
    {
        if = '''
        test %{app-bundle-id} = com.apple.calculator \
            || test %{app-bundle-id} = com.apple.clock \
            || test %{app-bundle-id} = com.apple.systempreferences \
            || test %{app-bundle-id} = com.apple.mail
        ''',
        run = ['layout floating'],
    },
    # Per workspace location
    # See: https://nikitabobko.github.io/AeroSpace/commands#move-node-to-workspace
    #
    # Workspace 1 (Private)
    {
        if = '''
        test %{app-bundle-id} = com.apple.mail \
            || test %{app-bundle-id} = com.apple.Music \
            || test %{app-bundle-id} = com.apple.MobileSMS \
            || test %{app-bundle-id} = com.apple.TV \
            || test %{app-bundle-id} = com.apple.Photos \
        ''',
        run = ['move-node-to-workspace 1', 'layout floating'],
    },
    # Workspace 2 (Files and Directories, Finder)
    {
        if = 'test %{app-bundle-id} = com.apple.finder',
        run = ['move-node-to-workspace 2'],
    },
]

focus-follows-mouse.enabled = true

# 'main' binding mode declaration
# See: https://nikitabobko.github.io/AeroSpace/guide#binding-modes
# 'main' binding mode must be always presented
# Fallback value (if you omit the key): mode.main.binding = {}
[mode.main.binding]

    # All possible keys:
    # - Letters.        a, b, c, ..., z
    # - Numbers.        0, 1, 2, ..., 9
    # - Keypad numbers. keypad0, keypad1, keypad2, ..., keypad9
    # - F-keys.         f1, f2, ..., f20
    # - Special keys.   minus, equal, period, comma, slash, backslash, quote, semicolon,
    #                   backtick, leftSquareBracket, rightSquareBracket, space, enter, esc,
    #                   backspace, tab, pageUp, pageDown, home, end, forwardDelete,
    #                   sectionSign (ISO keyboards only, european keyboards only)
    # - Keypad special. keypadClear, keypadDecimalMark, keypadDivide, keypadEnter, keypadEqual,
    #                   keypadMinus, keypadMultiply, keypadPlus
    # - Arrows.         left, down, up, right

    # All possible modifiers: cmd, alt, ctrl, shift

    # All possible commands: https://nikitabobko.github.io/AeroSpace/commands

    # See: https://nikitabobko.github.io/AeroSpace/commands#exec-and-forget
    # You can uncomment the following lines to open up terminal with alt + enter shortcut
    # (like in i3)
    # alt-enter = '''exec-and-forget osascript -e '
    # tell application "Terminal"
    #     do script
    #     activate
    # end tell'
    # '''

    # See: https://nikitabobko.github.io/AeroSpace/commands#layout
    alt-slash = 'layout tiles horizontal vertical'
    alt-comma = 'layout accordion horizontal vertical'

    # See: https://nikitabobko.github.io/AeroSpace/commands#focus
    ctrl-h = 'focus --boundaries-action wrap-around-the-workspace left'
    ctrl-j = 'focus --boundaries-action wrap-around-the-workspace down'
    ctrl-k = 'focus --boundaries-action wrap-around-the-workspace up'
    ctrl-l = 'focus --boundaries-action wrap-around-the-workspace right'

    # See: https://nikitabobko.github.io/AeroSpace/commands#move
    alt-ctrl-h = 'move left'
    alt-ctrl-j = 'move down'
    alt-ctrl-k = 'move up'
    alt-ctrl-l = 'move right'

# ...