`add_options` is annotated to take `OptionManager`, but the tests pass a spy object. With `mypy --strict` running on `tests/`, this will be flagged as an incompatible argument type. Consider typing `parser` as a small `Protocol` (e.g., something with `add_option(...)`) or `Any`/`object` + runtime duck-typing to keep both plugin correctness and test type-checking.

[steven-cutting]

add_options is annotated to take OptionManager, but the tests pass a spy object. With mypy --strict running on tests/, this will be flagged as an incompatible argument type. Consider typing parser as a small Protocol (e.g., something with add_option(...)) or Any/object + runtime duck-typing to keep both plugin correctness and test type-checking.

Originally posted by @copilot-pull-request-reviewer in #34 (comment)