4 Commits

Author	SHA1	Message	Date
HAL9000	02e51b9298	fix(actor): fix lint errors and add regression tests for lsp_binding field (#1432) ... - Remove trailing whitespace from blank lines in _extract_lsp_bindings() (W293 at lines 160, 164, 167, 172, 183, 188, 203) - Simplify over-long line 180 (E501): auto_detect=node.lsp_binding.auto - Add Behave scenario verifying NodeLspBinding typed field populates CompilationMetadata.lsp_bindings (features/actor_lsp_binding_field.feature) - Add corresponding @given step using NodeLspBinding in actor_compiler_steps.py - Add Robot Framework regression test case and lsp-binding-field helper command confirming node.lsp_binding is read by compile_actor() - Update CHANGELOG.md and CONTRIBUTORS.md ISSUES CLOSED: #1432	2026-05-30 00:36:20 -04:00
HAL9000	5db663cb63	fix(actor): read actor_ref from NodeDefinition field in _detect_subgraph_cycles ... Fix cross-actor subgraph cycle detection in the actor compiler by reading actor_ref from the top-level NodeDefinition field instead of the config dict. The _detect_subgraph_cycles(), _map_node(), and compile_actor() functions were all reading node.config.get("actor_ref", "") instead of node.actor_ref. Because actor_ref is a typed, validated Pydantic field on NodeDefinition (not a key inside the untyped config dict), the old code always returned an empty string, causing cross-actor cycle detection to silently fail and leaving the system vulnerable to infinite recursion at runtime. Changes: - Fix all three call sites in src/cleveragents/actor/compiler.py - Add Behave regression tests (features/actor_subgraph_cycle_detection.feature) with @tdd_issue and @tdd_issue_1431 tags on all scenarios - Add Robot Framework integration test (robot/actor_compiler.robot) - Fix existing test helpers to use actor_ref as top-level field - Add CHANGELOG entry ISSUES CLOSED: #1431	2026-05-06 04:08:25 +00:00
hurui200320	8953449dc2	fix(actor): validate v3 YAML via ActorConfigSchema in agents actor add CLI ... - schema.py: provider field changed to Optional[str] with model validator validate_provider_required_for_llm_graph() that requires it only for LLM and GRAPH actor types; TOOL actors do not require provider - schema.py: is_v3_yaml() uses version_str == "3" or version_str.startswith("3.") to avoid false positives from "30" or "300" version strings - schema.py: tool namespace validation uses strict 2-part split to reject empty namespace or empty name (e.g. "/tool", "ns/", "a/b/c") - cli/commands/actor.py: schema_version extraction uses raw_version pattern (no # type: ignore[assignment]) for clean static typing - actor/__init__.py: is_v3_yaml removed from __all__ and _LAZY_IMPORTS since it is a module-private helper, not a public API - robot/actor_add_v3_schema_validation.robot: YAML fixtures for 'Reject v3 LLM Actor Without Model Field' and 'Reject v3 TOOL Actor Without Tools Field' now include required provider field (and model for TOOL fixture) - robot/helper_actor_add_v3_schema_validation.py: except clauses unified to catch (subprocess.TimeoutExpired, FileNotFoundError) in both add_actor() and update_actor() functions - features/actor_add_v3_schema_validation.feature: 'Update a v3 actor with valid YAML succeeds' scenario now includes 'And the actor should be validated via ActorConfigSchema'; error assertions tightened to exact messages (e.g. "Input should be 'llm', 'tool' or 'graph'", "Node ID must be alphanumeric", "must be namespaced") - features/steps/actor_add_v3_schema_validation_steps.py: step_run_actor_update now spies on ActorConfigSchema.model_validate; failure paths assert isinstance(result.exception, SystemExit) ISSUES CLOSED: #5869	2026-04-17 16:28:45 +08:00
freemo	c47e6445d0	feat(actor): compile hierarchical actor configs to LangGraph ... Add ActorCompiler module that translates GRAPH-type ActorConfigSchema definitions into LangGraph NodeConfig/Edge structures with LSP binding metadata. Includes subgraph resolution with cross-actor cycle detection, entry/exit validation, and CompilationMetadata for diagnostics. New files: - src/cleveragents/actor/compiler.py: Core compiler with compile_actor() - features/actor_compiler.feature: 13 Behave scenarios - features/steps/actor_compiler_steps.py: Step definitions - robot/actor_compiler.robot: 4 Robot smoke tests - benchmarks/actor_compiler_bench.py: ASV performance benchmarks - docs/reference/actor_compiler.md: Compilation pipeline reference Modified: - src/cleveragents/actor/__init__.py: Export compiler types - vulture_whitelist.py: Whitelist new public API ISSUES CLOSED: #158	2026-02-24 17:57:18 +00:00