Monitor print filter doesn't work with System Time Log timstamps (IDFGH-14852)

[spahijakov]

Checklist

• Checked the issue tracker for similar issues to ensure this is not a duplicate
• Read the documentation to confirm the issue is not addressed there and your configuration is set correctly
• Tested with the latest version to ensure the issue hasn't been fixed

How often does this bug occurs?

always

Expected behavior

--- esp-idf-monitor 1.5.0 on /dev/ttyUSB0 115200
--- Quit: Ctrl+] | Menu: Ctrl+T | Help: Ctrl+T followed by Ctrl+H
--- Print filter: "main_task:I"
I (00:00:00.106) main_task: Started on CPU0
I (00:00:00.117) main_task: Calling app_main()

Actual behavior (suspected bug)

--- esp-idf-monitor 1.5.0 on /dev/ttyUSB0 115200
--- Quit: Ctrl+] | Menu: Ctrl+T | Help: Ctrl+T followed by Ctrl+H
--- Print filter: "main_task:I"

The issue is that the regex responsible for checking the log level and tag doesn't account for it:
https://github.com/espressif/esp-idf-monitor/blob/master/esp_idf_monitor/base/line_matcher.py#L26

Error logs or terminal output

--- esp-idf-monitor 1.5.0 on /dev/ttyUSB0 115200
--- Quit: Ctrl+] | Menu: Ctrl+T | Help: Ctrl+T followed by Ctrl+H
--- Print filter: "main_task:I"

Steps to reproduce the behavior

Go to hello_world example and build it with CONFIG_LOG_TIMESTAMP_SOURCE_SYSTEM=y:

cd examples/get-started/hello_world
idf.py menuconfig
idf.py flash monitor --print-filter "main_task:I"

Project release version

master, latest, v1.5.0

System architecture

Intel/AMD 64-bit (modern PC, older Mac)

Operating system

Linux

Operating system version

Linux 6.12.17-1-MANJARO

Shell

ZSH

Additional context

idf5.2_py3.13_env environment

[spahijakov]

Add this regex to line matcher, it should work:

        self._re = re.compile(r'^(?:\033\[[01];?[0-9]+m?)?([EWIDV]) (?:\([0-9]+\)|\([0-9]{2}:[0-9]{2}:[0-9]{2}\.[0-9]{3}\)) ([^:]+): ')

[peterdragun]

Hi @spahijakov, thank you for opening the issue. If you are interested, you can open a pull request with the change, or I will implement this fix.

One more note if you would like to contribute, the regex should also account for the new Log v2, which is going to be introduced in ESP-IDF 5.5 (already available on the master branch), which also allows to skip the log level, skip the timestamps and also one new format for timestamp System Time (YY-MM-DD HH:MM:SS.sss)

[spahijakov]

Hi @spahijakov, thank you for opening the issue. If you are interested, you can open a pull request with the change, or I will implement this fix.

One more note if you would like to contribute, the regex should also account for the new Log v2, which is going to be introduced in ESP-IDF 5.5 (already available on the master branch), which also allows to skip the log level, skip the timestamps and also one new format for timestamp System Time (YY-MM-DD HH:MM:SS.sss)

I can look into it, I've opened a PR, so feel free to add commentary, I forgot to include the permission from maintainers 💢