Config file gets loaded when using 'pypr --help'

[dantefromhell]

Not sure if this is a bug, hence discussion first. But in case this is a bug, plz convert accordingly.

With version 2.2.7, when running pypr --help a stacktrace will be printed in case the config file has errors in it:

❯ pypr --help
Unhandled exception:
Traceback (most recent call last):
  File "/nix/store/c09z7x20rjsbjiz7fkdkwaw8vra8gggv-python3.11-pyprland-2.2.7/lib/python3.11/site-packages/pyprland/command.py", line 464, in main
    asyncio.run(run_daemon() if invoke_daemon else run_client())
  File "/nix/store/gd3shnza1i50zn8zs04fa729ribr88m9-python3-3.11.8/lib/python3.11/asyncio/runners.py", line 190, in run
    return runner.run(main)
           ^^^^^^^^^^^^^^^^
  File "/nix/store/gd3shnza1i50zn8zs04fa729ribr88m9-python3-3.11.8/lib/python3.11/asyncio/runners.py", line 118, in run
    return self._loop.run_until_complete(task)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/nix/store/gd3shnza1i50zn8zs04fa729ribr88m9-python3-3.11.8/lib/python3.11/asyncio/base_events.py", line 654, in run_until_complete
    return future.result()
           ^^^^^^^^^^^^^^^
  File "/nix/store/c09z7x20rjsbjiz7fkdkwaw8vra8gggv-python3.11-pyprland-2.2.7/lib/python3.11/site-packages/pyprland/command.py", line 408, in run_client
    await manager.load_config(init=False)
  File "/nix/store/c09z7x20rjsbjiz7fkdkwaw8vra8gggv-python3.11-pyprland-2.2.7/lib/python3.11/site-packages/pyprland/command.py", line 170, in load_config
    await self.__open_config()
  File "/nix/store/c09z7x20rjsbjiz7fkdkwaw8vra8gggv-python3.11-pyprland-2.2.7/lib/python3.11/site-packages/pyprland/command.py", line 81, in __open_config
    merge(config, await self.__open_config(extra_config))
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/nix/store/c09z7x20rjsbjiz7fkdkwaw8vra8gggv-python3.11-pyprland-2.2.7/lib/python3.11/site-packages/pyprland/command.py", line 77, in __open_config
    config = self.__load_config_file(fname)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/nix/store/c09z7x20rjsbjiz7fkdkwaw8vra8gggv-python3.11-pyprland-2.2.7/lib/python3.11/site-packages/pyprland/command.py", line 92, in __load_config_file
    config = tomllib.load(f)
             ^^^^^^^^^^^^^^^
  File "/nix/store/gd3shnza1i50zn8zs04fa729ribr88m9-python3-3.11.8/lib/python3.11/tomllib/_parser.py", line 66, in load
    return loads(s, parse_float=parse_float)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/nix/store/gd3shnza1i50zn8zs04fa729ribr88m9-python3-3.11.8/lib/python3.11/tomllib/_parser.py", line 102, in loads
    pos = key_value_rule(src, pos, out, header, parse_float)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/nix/store/gd3shnza1i50zn8zs04fa729ribr88m9-python3-3.11.8/lib/python3.11/tomllib/_parser.py", line 326, in key_value_rule
    pos, key, value = parse_key_value_pair(src, pos, parse_float)
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/nix/store/gd3shnza1i50zn8zs04fa729ribr88m9-python3-3.11.8/lib/python3.11/tomllib/_parser.py", line 369, in parse_key_value_pair
    pos, value = parse_value(src, pos, parse_float)
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/nix/store/gd3shnza1i50zn8zs04fa729ribr88m9-python3-3.11.8/lib/python3.11/tomllib/_parser.py", line 649, in parse_value
    raise suffixed_err(src, pos, "Invalid value")
tomllib.TOMLDecodeError: Invalid value (at line 2, column 11)

IMO printing the help screen should not trigger a loading of the config file, no?

Also it could IMO be good if the config parsing throws a better error message when config parsing failures happen, especially stating in which file the config parsing error occured. WDYT?

[fdev31]

The plugins need to be loaded to get their commands & documentation...

In you case it's a parsing error, meaning not even the plugin list can be loaded...
I can only "improve" the error (it doesn't look to bad at the moment, the last line summarizes the problem... but feel free to suggest something else, adding the filename sounds like a good idea but it could be at the cost of the (useless?) traceback).