fix: Pass LawAndOrderValidator to BasicFeatureGenerator

leonidb · leonidb · commit bfdc6cef0a2f · 2026-01-01T13:13:03.000+02:00
After changing BasicFeatureGenerator's validators field to default to an
empty tuple (from LawAndOrderValidator), SemanticInsightsGenerator needs
to explicitly pass the validator. The docstring already stated it uses
LawAndOrderValidator, now the implementation matches.
diff --git a/agentune/analyze/feature/gen/semantic_insights_generator/generator.py b/agentune/analyze/feature/gen/semantic_insights_generator/generator.py
@@ -12,6 +12,7 @@
     BasicFeatureGenerator,
 )
 from agentune.analyze.feature.problem import Problem
+from agentune.analyze.feature.validate.law_and_order import LawAndOrderValidator
 from agentune.analyze.join.base import TablesWithJoinStrategies
 from agentune.core.dataset import Dataset
 from agentune.core.sercontext import LLMWithSpec
@@ -59,6 +60,7 @@ async def agenerate(
             generation_model=self.generation_model,
             repair_model=self.repair_model,
             seed=self.seed,
+            validators=(LawAndOrderValidator(),),
         )
 
         async for gen_feature in basic_gen.agenerate(

Original file line number	Diff line number	Diff line change
`@@ -12,6 +12,7 @@`
`12`	`12`	`BasicFeatureGenerator,`
`13`	`13`	`)`
`14`	`14`	`from agentune.analyze.feature.problem import Problem`
	`15`	`+from agentune.analyze.feature.validate.law_and_order import LawAndOrderValidator`
`15`	`16`	`from agentune.analyze.join.base import TablesWithJoinStrategies`
`16`	`17`	`from agentune.core.dataset import Dataset`
`17`	`18`	`from agentune.core.sercontext import LLMWithSpec`
`@@ -59,6 +60,7 @@ async def agenerate(`
`59`	`60`	`generation_model=self.generation_model,`
`60`	`61`	`repair_model=self.repair_model,`
`61`	`62`	`seed=self.seed,`
	`63`	`+ validators=(LawAndOrderValidator(),),`
`62`	`64`	`)`
`63`	`65`
`64`	`66`	`async for gen_feature in basic_gen.agenerate(`