Keep CLI parser clean

Author: clemsgrs

slide2vec/cli.py

@@ -17,12 +17,8 @@ def get_args_parser(add_help: bool = True):
 
 def parse_args(argv=None):
     parser = get_args_parser(add_help=True)
-    if hasattr(parser, "parse_known_args"):
-        args, opts = parser.parse_known_args(argv)
-        args.opts = opts
-    else:
-        args = parser.parse_args(argv)
-        args.opts = getattr(args, "opts", [])
+    args, opts = parser.parse_known_args(argv)
+    args.opts = opts
     return args
 
 

tasks/lessons.md

@@ -2,7 +2,7 @@
 
 ## 2026-04-12
 
-- When refactoring CLI parsing to support `parse_known_args()`, keep existing test doubles in mind: if a fake parser only implements `parse_args()`, add a small compatibility fallback instead of forcing every monkeypatch to mirror the full parser API.
+- When refactoring CLI parsing to support `parse_known_args()`, prefer updating the test double to match the real parser API instead of adding a production fallback for mocks. Keep the runtime code clean unless the fallback is genuinely needed by real callers.
 
 ## 2026-04-10
 

tests/test_progress.py

@@ -112,6 +112,9 @@ class FakeParser:
         def parse_args(self, argv=None):
             return SimpleNamespace(tiling_only=False)
 
+        def parse_known_args(self, argv=None):
+            return self.parse_args(argv), []
+
     monkeypatch.setattr(cli, "get_args_parser", lambda add_help=True: FakeParser())
     monkeypatch.setattr(
         cli,