Skip to content

Conversation

@lucarin91
Copy link
Contributor

@lucarin91 lucarin91 commented Nov 12, 2025

Motivation

Add a monitor command for interacting with the microcontroller monitor.

Change description

You can test it by:

  1. flashing the monitor examples
  2. Install this version of arduno-app-cli with task board:install
  3. adb shell and run arduino-app-cli monitor

You should see the monitor print from the microcontroller, and you should be able to send in message too.

Additional Notes

Reviewer checklist

  • PR addresses a single concern.
  • PR title and description are properly filled.
  • Changes will be merged in main.
  • Changes are covered by tests.
  • Logging is meaningful in case of troubleshooting.

@lucarin91 lucarin91 requested a review from cmaglie November 12, 2025 09:31
@per1234 per1234 added the enhancement New feature or request label Nov 12, 2025
@lucarin91 lucarin91 marked this pull request as ready for review November 28, 2025 18:03
@lucarin91 lucarin91 requested a review from a team December 1, 2025 10:27
@lucarin91 lucarin91 changed the title feat: add bridge monitor command feat: add monitor command Dec 1, 2025
Co-authored-by: Cristian Maglie <c.maglie@bug.st>
@lucarin91 lucarin91 requested a review from dido18 December 1, 2025 11:08
dido18
dido18 previously requested changes Dec 1, 2025
@lucarin91 lucarin91 requested a review from dido18 December 1, 2025 11:23
@lucarin91 lucarin91 requested a review from dido18 December 1, 2025 14:33
lucarin91 and others added 3 commits December 2, 2025 10:50
Co-authored-by: Davide <davideneri18@gmail.com>
Co-authored-by: Davide <davideneri18@gmail.com>
Co-authored-by: Davide <davideneri18@gmail.com>
@lucarin91
Copy link
Contributor Author

I will try to add some tests, I did a lot of changes, and I am not sure about myself 😄

@lucarin91 lucarin91 requested a review from dido18 December 5, 2025 14:30
@lucarin91 lucarin91 dismissed dido18’s stale review December 9, 2025 10:04

implement suggestions

@lucarin91 lucarin91 requested a review from cmaglie December 9, 2025 10:04
@lucarin91 lucarin91 merged commit ed4e465 into arduino:main Dec 9, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement the monitor command in app-cli

4 participants