From 2160d0c6df7320c85263b871ead774296daa31bd Mon Sep 17 00:00:00 2001 From: Michael Assraf Date: Mon, 7 Sep 2026 23:10:45 -0400 Subject: [PATCH] fix: test_clustering_forced.py passes kwargs Config.from_cli does not accept Config.from_cli has no dependency_graph_dir or docs_dir parameter, so the call was a TypeError. It reached main because #35 and #46 changed the same construction differently and git auto-merged to #35's from_cli form without raising a conflict, and my verification pass only checked direct Config(...) calls - it did not look at Config.from_* factories at all. Restore the direct Config(...) construction #46 landed, which takes all sixteen kwargs including the two output directories this test relies on. Every Config construction site in the repo is now checked both ways - direct construction against the dataclass fields, and each from_* call against the factory's real signature. Co-Authored-By: Claude Opus 5 --- test_clustering_forced.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test_clustering_forced.py b/test_clustering_forced.py index 833ad075..6070bffa 100644 --- a/test_clustering_forced.py +++ b/test_clustering_forced.py @@ -40,7 +40,7 @@ def print_summary(self): test_repo = os.getenv("TEST_REPO_PATH", os.path.dirname(os.path.abspath(__file__))) -config = Config.from_cli( +config = Config( repo_path=test_repo, output_dir="/tmp/test", dependency_graph_dir="/tmp/test/deps", docs_dir="/tmp/test/docs", max_depth=2, main_model=os.getenv("MAIN_MODEL", "gpt-4o"),