Feature/serial buffer options

[JPHutchins]

Long overdue feature to allow setting the "line length" and "line buffers". "mtu" is deprecated, but will continue being interpreted as it always has: line_length = mtu, line_buffers = 1.

Many investigations and conversations have confirmed that the line length never should have been configurable in the first place. Including in Zephyr, MCUboot etc. But, we need to continue supporting non-standard line lengths since there are many units in the field that changed the line length as a speed up.

Also committed is an LLM generated report regarding naming and defaults.

[Copilot]

Pull request overview

This PR adds explicit serial transport framing/buffering controls to the smpmgr CLI while keeping --mtu working for backwards compatibility (now deprecated for serial).

Changes:

• Introduces --line-length and --line-buffers CLI options and threads them into the runtime Options.
• Updates serial transport initialization to derive max_smp_encoded_frame_size = line_length * line_buffers, with legacy --mtu mapped to line_length=mtu, line_buffers=1 and guarded against mixed usage.
• Adds an LLM-generated research note on SMP serial parameter naming/defaults and updates VS Code workspace settings for Poetry env/package management.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File	Description
smpmgr/main.py	Adds CLI flags for serial line length/buffer count; updates --mtu help text and passes new values into Options.
smpmgr/common.py	Adds defaults and implements serial transport argument selection/validation (including --mtu deprecation behavior).
docs/llm/smp-serial-naming-research.md	Adds a research document supporting naming/default decisions for SMP serial parameters.
.vscode/settings.json	Sets Poetry as the default env/package manager for VS Code’s Python environment tooling.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[JPHutchins]

Tested with some HW that has line length 4096 and line buffers 1, and it seems good.

 >  poetry run smpmgr --port /dev/ttyACM0 os echo hello
[10:44:35] WARNING  Error reading MCUMgr parameters: header=Header(op=<OP.READ_RSP:  __init__.py:534
                    1>, version=<Version.V2: 1>, flags=<Flag.UNUSED: 0>, length=6,
                    group_id=0, sequence=0, command_id=6) version=<Version.V2: 1>
                    sequence=0
                    smp_data=b'\t\x00\x00\x06\x00\x00\x00\x06\xbfbrc\x08\xff'
                    rc=<MGMT_ERR.ENOTSUP: 8> rsn=None - __init__:534
⠋ Connecting to /dev/ttyACM0... OK
⠋ Waiting for response to EchoWrite... OK
EchoWriteResponse(
    header=Header(
        op=<OP.WRITE_RSP: 3>,
        version=<Version.V2: 1>,
        flags=<Flag.UNUSED: 0>,
        length=10,
        group_id=0,
        sequence=1,
        command_id=0
    ),
    version=<Version.V2: 1>,
    sequence=1,
    smp_data=b'\x0b\x00\x00\n\x00\x00\x01\x00\xbfarehello\xff',
    r='hello'
)
   ~/repos/smpmgr   feature/serial-buffer-opt ≡  1  1.914s   
 >  poetry run smpmgr --port=/dev/ttyACM0 --mtu=4096 os echo hello
Warning: --mtu is deprecated for serial transport. Use --line-length and --line-buffers instead. --mtu 4096 has been applied as --line-length 4096 --line-buffers 1.
[10:44:52] WARNING  Error reading MCUMgr parameters: header=Header(op=<OP.READ_RSP:  __init__.py:534
                    1>, version=<Version.V2: 1>, flags=<Flag.UNUSED: 0>, length=6,
                    group_id=0, sequence=0, command_id=6) version=<Version.V2: 1>
                    sequence=0
                    smp_data=b'\t\x00\x00\x06\x00\x00\x00\x06\xbfbrc\x08\xff'
                    rc=<MGMT_ERR.ENOTSUP: 8> rsn=None - __init__:534
⠋ Connecting to /dev/ttyACM0... OK
⠋ Waiting for response to EchoWrite... OK
EchoWriteResponse(
    header=Header(
        op=<OP.WRITE_RSP: 3>,
        version=<Version.V2: 1>,
        flags=<Flag.UNUSED: 0>,
        length=10,
        group_id=0,
        sequence=1,
        command_id=0
    ),
    version=<Version.V2: 1>,
    sequence=1,
    smp_data=b'\x0b\x00\x00\n\x00\x00\x01\x00\xbfarehello\xff',
    r='hello'
)
   ~/repos/smpmgr   feature/serial-buffer-opt ≡  1  1.954s   
 >  poetry run smpmgr --port=/dev/ttyACM0 --line-length=4096 os echo hello
[10:45:13] WARNING  Error reading MCUMgr parameters: header=Header(op=<OP.READ_RSP:  __init__.py:534
                    1>, version=<Version.V2: 1>, flags=<Flag.UNUSED: 0>, length=6,
                    group_id=0, sequence=0, command_id=6) version=<Version.V2: 1>
                    sequence=0
                    smp_data=b'\t\x00\x00\x06\x00\x00\x00\x06\xbfbrc\x08\xff'
                    rc=<MGMT_ERR.ENOTSUP: 8> rsn=None - __init__:534
⠋ Connecting to /dev/ttyACM0... OK
⠋ Waiting for response to EchoWrite... OK
EchoWriteResponse(
    header=Header(
        op=<OP.WRITE_RSP: 3>,
        version=<Version.V2: 1>,
        flags=<Flag.UNUSED: 0>,
        length=10,
        group_id=0,
        sequence=1,
        command_id=0
    ),
    version=<Version.V2: 1>,
    sequence=1,
    smp_data=b'\x0b\x00\x00\n\x00\x00\x01\x00\xbfarehello\xff',
    r='hello'
)
   ~/repos/smpmgr   feature/serial-buffer-opt ≡  1  1.83s   
 >  poetry run smpmgr --port=/dev/ttyACM0 --line-length=4096 --line-buffers=1 os echo hello
[10:45:22] WARNING  Error reading MCUMgr parameters: header=Header(op=<OP.READ_RSP:  __init__.py:534
                    1>, version=<Version.V2: 1>, flags=<Flag.UNUSED: 0>, length=6,
                    group_id=0, sequence=0, command_id=6) version=<Version.V2: 1>
                    sequence=0
                    smp_data=b'\t\x00\x00\x06\x00\x00\x00\x06\xbfbrc\x08\xff'
                    rc=<MGMT_ERR.ENOTSUP: 8> rsn=None - __init__:534
⠋ Connecting to /dev/ttyACM0... OK
⠋ Waiting for response to EchoWrite... OK
EchoWriteResponse(
    header=Header(
        op=<OP.WRITE_RSP: 3>,
        version=<Version.V2: 1>,
        flags=<Flag.UNUSED: 0>,
        length=10,
        group_id=0,
        sequence=1,
        command_id=0
    ),
    version=<Version.V2: 1>,
    sequence=1,
    smp_data=b'\x0b\x00\x00\n\x00\x00\x01\x00\xbfarehello\xff',
    r='hello'
)