fix: duplicated find argument

Author: hfudev

idf_build_apps/main.py

@@ -395,12 +395,17 @@ def main():
     else:
         arguments = BuildArguments(**kwargs_without_none)
 
+    if isinstance(arguments, FindArguments):
+        find_arguments = arguments
+    else:
+        find_arguments = FindArguments.model_validate(arguments.model_dump(exclude_none=True))
+
     # real call starts here
     # build also needs to find first
     apps = find_apps(
         arguments.paths,
         arguments.target,
-        find_arguments=FindArguments.model_validate(arguments.model_dump(exclude_none=True)),
+        find_arguments=find_arguments,
     )
 
     if isinstance(arguments, FindArguments):  # find only

tests/test_cmd.py

@@ -137,6 +137,43 @@ def test_manifest_patterns(tmp_path, monkeypatch, capsys):
     assert f'"{os.path.join(tmp_path, "bar")}" does not exist' in err
 
 
+def test_find_with_manifest_patterns_from_config_does_not_duplicate_manifest(tmp_path, monkeypatch, capsys):
+    (tmp_path / '.idf_build_apps.toml').write_text(
+        dedent(
+            """\
+            manifest_filepatterns = [
+                '**/.build-test-rules.yml',
+            ]
+            """
+        )
+    )
+    (tmp_path / '.build-test-rules.yml').write_text(
+        dedent(
+            """\
+            app-one:
+              enable:
+                - if: IDF_TARGET == "esp32"
+
+            app-two:
+              enable:
+                - if: IDF_TARGET == "esp32"
+            """
+        )
+    )
+    (tmp_path / 'app-one').mkdir()
+    (tmp_path / 'app-two').mkdir()
+
+    with monkeypatch.context() as m:
+        m.chdir(tmp_path)
+        m.setattr(sys, 'argv', ['idf-build-apps', 'find'])
+        with pytest.raises(SystemExit) as exc_info:
+            main()
+
+    assert exc_info.value.code == 0
+    _, err = capsys.readouterr()
+    assert 'is already defined in' not in err
+
+
 def test_build_include_all_apps(tmp_path, monkeypatch, capsys):
     create_project('foo', tmp_path)