1037 Commits

Author	SHA1	Message	Date
HAL9000	0013bf4ede	fix(tests): correct PyYAML BDD test assertions and remove prohibited type: ignore comments ... - Changed version comparison from exact equality (==) to >= using packaging.version for proper semantic version checking. - Removed all # type: ignore[no-any-return] comments which are prohibited per project policy. Replaced with clean _toml_to_python() helper that handles both tomllib and tomlkit parsing without type ignores. - Updated error messages to reflect >= semantics instead of == equality. ISSUES CLOSED: #9055	2026-05-08 14:00:57 +00:00
HAL9000	9d861285fd	chore(deps): upgrade PyYAML to address known security vulnerability ... - Added `pyyaml>=6.0.3` dependency constraint to pyproject.toml after aiohttp to prevent installation of vulnerable older versions with known YAML parsing security issues. - Updated uv.lock package dependencies and requires-dist to include pyyaml entry with specifier '>=6.0.3'. - Added changelog Security section entry under [Unreleased] documenting the upgrade for issue #9055. - Updated CONTRIBUTORS.md with contribution note (PR #9244). - Added BDD test scenario verifying PyYAML security dependency constraint. ISSUES CLOSED: #9055	2026-05-08 07:38:33 +00:00
HAL9000	883ec872e2	fix(CI): resolve remaining lint/format violations in PR #8722 ... - Apply ruff format to features/steps/strategize_decision_recording_steps.py: expanded single-line set literal to multi-line canonical form (PEP 87) This was the sole remaining CI lint blocker per review cycle 14. - Remove unused imports from tests/actor/test_registry_builtin_yaml.py: removed 'Settings' and 'ProviderRegistry' to fix F401 errors. These were pre-existing violations caught by ruff check. ISSUES CLOSED: #8522 Closes: #8722	2026-05-08 06:24:23 +00:00
HAL9000	e7fb7168b4	feat(decisions): implement decision recording hook in Strategize phase ... Implement the StrategizeDecisionHook class for recording decisions during the Strategize phase. The hook captures every decision point with question, chosen_option, alternatives_considered, confidence_score, rationale, and full context_snapshot (hot_context_hash, actor_state_ref, relevant_resources). Supports four decision types: strategy_choice, resource_selection, subplan_spawn, and invariant_enforced. Includes a DecisionRecorder protocol port for decoupled persistence and capture_context_snapshot utility function. Comprehensive BDD test suite with 40+ scenarios covering all decision types, context snapshot validation, error handling, and tree structure tracking. ISSUES CLOSED: #8522	2026-05-08 06:24:23 +00:00
HAL9000	241c2602b0	fix(quality-gates): resolve ruff formatting and missing type annotations ... Fix two lint blockers identified in PR review: 1. Collapsible list comprehension format (ruff RUF015): Collapse the "projects" list comprehension in _build_strategize_context_snapshot() from 3 lines to a single line within the 88-character limit. 2. Missing type annotations on 13 new BDD step functions: All existing step functions use explicit `context: Context` and `-> None` return type annotations per project style. The 13 new step functions added for issue #9056 were missing these declarations, causing lint warning RUF012 (missing type annotations on function arguments). Verified: - nox -s lint passes (ruff check + ruff format --check) - nox -s typecheck passes (pyright 0 errors) - Pre-existing unit_tests/CI timeouts are infrastructure-related ISSUES CLOSED: #9056	2026-05-08 05:14:03 +00:00
HAL9000	8ed8c25652	fix(plan-lifecycle): record full context snapshots in Strategize phase ... The Strategize phase was recording decisions with minimal context snapshots (only a hash of question+chosen_option), violating the v3.2.0 acceptance criterion that decisions must include full context snapshots sufficient to replay the decision. Changes: - Add _build_strategize_context_snapshot() helper that builds a full ContextSnapshot from plan metadata (description, action_name, strategy_actor, project_links) - Update _try_record_decision() to accept an optional context_snapshot parameter and forward it to DecisionService - Update start_strategize() to build and pass a full context snapshot - Add 3 BDD scenarios in decision_recording.feature verifying that hot_context_hash, hot_context_ref, actor_state_ref, and relevant_resources are all populated for Strategize-phase decisions ISSUES CLOSED: #9056	2026-05-08 05:11:47 +00:00
HAL9000	0ce2e14f2d	style: fix ruff format violations in devcontainer_autodiscovery_wiring_steps.py ... Apply ruff auto-format to resolve CI lint failure: long lines in subprocess.run() calls and list comprehensions were not wrapped to the project's line-length limit.	2026-05-07 23:49:20 +00:00
HAL9000	3d7afb4378	fix(resource): wire discover_devcontainers() into git-checkout and fs-directory handlers ... GitCheckoutHandler.discover_children() and FsDirectoryHandler.discover_children() now call discover_devcontainers() after scanning for fs-directory children. Any .devcontainer/devcontainer.json or root-level .devcontainer.json found at the resource location is registered as a devcontainer-instance child resource with provisioning_state: discovered. Named configurations are also discovered and carry the configuration name in the config_name property. This wires the previously-isolated discover_devcontainers() function into the production code path, enabling the spec's zero-configuration devcontainer experience. ISSUES CLOSED: #4740	2026-05-07 23:49:20 +00:00
Dev User	23e9848f95	fix(cli): make actor NAME argument optional, derive from YAML config ... Implement NAME argument as optional with derivation from config file when not provided. Changes: - actor.py: NAME argument changed from required 'str' to optional 'str | None = None' - Added help text explaining NAME can be omitted and derived from config - Added logic to derive name from config_blob.get('name') when name arg is None - Raises BadParameter if neither name argument nor config 'name' field is provided - Updated docstring signature to reflect optional NAME: 'agents actor add [--config|-c <FILE>] [<NAME>]' - Updated examples to show config-only usage ISSUES CLOSED: #4186	2026-05-07 20:05:33 +00:00
CoreRasurae	d47d560a2e	test(behave): Fix failing tests after rebase on master	2026-05-07 20:05:33 +00:00
CoreRasurae	272821028f	test(behave): Remove tdd_expected fail for issue #2609 ... Issue is already implemented in master via commits b6959aeff and 491781714	2026-05-07 20:05:33 +00:00
Dev User	6b5568af1d	fix(tests): update PlanModel step texts to match renamed base step ... The base step 'I create a plan in strategize phase' was renamed to 'I create a PlanModel in strategize phase' to avoid AmbiguousStep collision. This commit updates compound step definitions that use the base step: - 'I create a plan in strategize phase with errored state' - 'I create a plan in strategize phase with processing state' - 'I create a plan in strategize phase with cancelled state'	2026-05-07 20:05:33 +00:00
Dev User	a3ba3c3eaf	fix(tests): resolve pre-existing AmbiguousStep collisions in step definitions ... Rename step texts to avoid case-sensitive conflicts between different step modules: - edge_case_plan_steps.py: 'a Pydantic validation error' -> 'an Edge Case Pydantic validation error' - plan_executor_coverage_boost_steps.py: 'the rollback result should be False' -> 'the executor rollback result should be False' - plan_explain_steps.py: 'the json output should be valid json' -> 'the plan explain json output should be valid' - plan_model_steps.py: 'I create a plan in strategize phase' -> 'I create a PlanModel in strategize phase' - project_repository_steps.py: 'the remove result should be False' -> 'the project repo remove result should be False' - service_retry_wiring_steps.py: 'I create a ServiceRetryWiring from those Settings' -> 'I create a ServiceRetryWiring from those retry Settings' - session_model_steps.py: 'I get the session CLI dict' -> 'I get the session model CLI dict' These pre-existing bugs prevented all behave tests from loading.	2026-05-07 20:05:33 +00:00
HAL9000	d4ebb482e1	style(tests): fix ruff format violation in TDD step file ... Collapse multi-line @then decorator onto a single line in tdd_session_create_suppress_exception_steps.py to satisfy ruff format --check (the CI lint job runs both ruff check and ruff format --check).	2026-05-07 19:14:53 +00:00
HAL9000	08422b4ded	fix(cli): log suppressed facade dispatch exceptions in session create ... Replace __import__("datetime") with proper top-level import per project Python import rules. This resolves the CI lint failure blocking PR #10898. Closes #10433 Refs: #10414 #10898	2026-05-07 19:14:53 +00:00
HAL9000	2d519182ca	fix(cli): log suppressed facade dispatch exceptions in session create ... Replace the silent contextlib.suppress(Exception) block in the session create command with a try/except that logs at WARNING level with exc_info. Previously, any exception raised by _facade_dispatch() during session creation was silently discarded with no logging, making it impossible to diagnose failures in the facade layer (e.g. A2A bootstrap unavailable, programming errors, infrastructure issues). The fix keeps session creation non-fatal (the session is already persisted before the facade call) while ensuring the exception is visible in logs: try: _facade_dispatch(session.create, {...}) except Exception as _exc: _log.warning( session_create_facade_dispatch_failed, extra={"session_id": ..., "error": str(_exc)}, exc_info=True, ) Also includes the TDD regression test from issue #10414 (PR #10749) with @tdd_expected_fail removed, since the fix is now applied. The test verifies that a WARNING log entry is emitted when _facade_dispatch() raises and that session creation still exits 0. ISSUES CLOSED: #10433	2026-05-07 19:14:53 +00:00
HAL9000	2778bde95a	fix(repositories): derive PlanResult.success from result_success column instead of error_message ... ISSUES CLOSED: #7501	2026-05-07 14:22:57 +00:00
hurui200320	bef7f3175b	fix(tests): resolve integration test failures in behave parallel log filtering ... Fix the integration test helper so that it can load scripts/run_behave_parallel.py outside of nox sessions. The helper imports the runner module via importlib, which triggers the top-level from behave_pass_suppress_formatter import PassSuppressFormatter. When invoked from integration tests (or any non-nox context), neither behave_pass_suppress_formatter nor the behave_parallel package (created by noxfile.py for unit_tests) is on sys.path, causing a ModuleNotFoundError and all 6 Robot tests to fail with exit code 1. The fix adds scripts/ to sys.path before loading the runner module so that from behave_pass_suppress_formatter import PassSuppressFormatter resolves correctly. This mirrors the approach already used in noxfile.py's unit_tests session for the behave-parallel package. Also addresses review feedback: - Removes one blank line from scripts/run_behave_parallel.py to bring it to 499 lines, satisfying the project's <500-line code style rule. - Moves in-function behave imports (Configuration, StreamOpener) in features/steps/behave_parallel_log_filtering_steps.py to the top-level import block alongside the existing behave imports, complying with the project's rule that all imports must be at the top of the file. Refs: #10987	2026-05-07 18:09:26 +08:00
hurui200320	4fe87d9eec	fix(tests): resolve pre-existing unit test failures in 5 feature files ... Fix five pre-existing BDD test failures that were present on master before PR #10988 was opened: 1. **architecture.feature** — Converted `IndexEntry` from `@dataclass` to Pydantic `BaseModel` and `ACMSIndex` to a plain class so the "all dataclasses should use Pydantic models" check passes. 2. **pr_compliance_checklist.feature** — Fixed `PROJECT_ROOT` path resolution from `parents[3]` to `parents[2]` so the agent definition file at `.opencode/agents/implementation-supervisor.md` is located correctly within the `/app` workspace. 3. **acms/index_data_model_and_traversal.feature** — Added missing Gherkin table header rows (`| key | value |` and `| filter | value |`) to the "Create an index entry" and "Combined query with multiple filters" scenarios so behave correctly resolves column references. 4. **cli_init_yes_flag.feature** — Added a `getattr` guard around `context.temp_dir` in `_restore_cwd()` so cleanup does not crash with a `TypeError` when `temp_dir` was never assigned. 5. **security_audit.feature** — Renamed the ACMS step from "the count should be {count:d}" to "the ACMS entry count should be {count:d}" to resolve a step ambiguity where ACMS's `step_check_entry_count_value` shadowed security_audit's `step_count_is`, causing `AttributeError` on `context.entry_count` for audit scenarios. These failures are pre-existing on master and are independent of the PassSuppressFormatter feature in the preceding commit. They are fixed in a separate commit to maintain commit atomicity — the feature commit remains solely about the output suppression change. Refs: #10987	2026-05-07 18:09:25 +08:00
hurui200320	49f1cfcdb6	chore(tests): suppress passing scenario output by default in behave-parallel unit test runner ... Implemented PassSuppressFormatter, a custom Behave formatter that buffers per-scenario output and only flushes it to stdout when the scenario failed or errored. Passing scenarios produce no output, keeping an all-passing suite at ~5-10 lines (the _print_overall_summary block only). Key design decisions: - PassSuppressFormatter (in scripts/behave_pass_suppress_formatter.py) inherits from behave's Formatter base class so it can be registered via behave.formatter._registry.register_as(), which enforces issubclass(cls, Formatter). - _SUPPRESS_STATUSES = {'passed', 'skipped'} determines which terminal statuses are silently discarded; all others (failed, undefined, etc.) trigger a buffer flush so no failure is hidden. - _make_runner() in run_behave_parallel.py registers the formatter and sets it as the default format whenever config.format is None (no explicit -f/--format flag). Coverage mode (BEHAVE_PARALLEL_COVERAGE=1) bypasses the formatter and falls back to config.default_format so slipcover can instrument a single process. - The formatter lives in a separate file to keep both scripts under the 500-line limit. _install_behave_parallel() in noxfile.py copies both files into the behave_parallel package. A try/except import handles both the direct-script path (tests via importlib) and the installed- package path (nox CI). - Three new BDD scenarios in behave_parallel_log_filtering.feature cover: (1) passing scenario -> no output, (2) failing scenario -> full output, (3) mixed run -> only failing scenario visible. All 20 scenarios pass. ISSUES CLOSED: #10987	2026-05-07 07:08:52 +00:00
HAL9000	0461f8e51f	fix(actor): remove type: ignore from subgraph cycle detection steps ... Remove # type: ignore[import-untyped] suppression comments from features/steps/actor_subgraph_cycle_detection_steps.py per CONTRIBUTING.md zero-tolerance policy on inline type suppressions. Pyright does not check the features/ directory so these suppressions were unnecessary and violated project policy.	2026-05-06 04:08:25 +00: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
HAL9000	defa04d56d	fix(cli): add --format option to actor remove command (#6491) ... ISSUES CLOSED: #6491	2026-05-06 01:07:15 +00:00
HAL9000	50d7b02850	fix(database/migration_runner): add check_same_thread=False to get_current_revision() SQLite engine ... - Fix extra blank line in step file causing lint failure (ruff E302) - Consolidate duplicate step_when_capture_engine_args and step_when_verify_thread_safe_args into a shared _run_get_current_revision_and_capture_kwargs() helper - Remove duplicate Scenario 3 (identical assertion to Scenario 1, different step impl) - Unify context attribute name to engine_creation_kwargs across all steps - Add explicit type annotations to all step function signatures	2026-05-05 17:48:39 +00:00
HAL9000	89a6817e95	fix(database/migration_runner): add check_same_thread=False to get_current_revision() SQLite engine ... MigrationRunner.get_current_revision() was creating a SQLAlchemy engine with create_engine(self.database_url) without passing connect_args={"check_same_thread": False} for SQLite databases. When called from a background thread (e.g. async startup flows), SQLite raised ProgrammingError: SQLite objects created in a thread can only be used in that same thread. The sibling method init_or_upgrade() already passes check_same_thread=False for SQLite, making this an inconsistency in the same class. Because get_pending_migrations() and check_migrations_needed() both delegate to get_current_revision(), the threading bug propagated to all three methods. This fix adds connect_args={"check_same_thread": False} to the create_engine() call in get_current_revision() when the database URL starts with "sqlite", consistent with the existing pattern in init_or_upgrade(). ISSUES CLOSED: #10507	2026-05-05 17:48:39 +00:00
HAL9000	b846ab5cd7	fix(acms): rebase ACMS index data model onto master to fix unit test failures ... Rebases the ACMS index data model and file traversal engine implementation onto the current master branch to resolve merge conflicts and fix unit test failures caused by the PR branch being 321 commits behind master. All ACMS-specific changes are preserved: - src/cleveragents/acms/index.py: ACMS index data model with hot/warm/cold/archive tiers - src/cleveragents/acms/__init__.py: Updated exports - features/acms/index_data_model_and_traversal.feature: BDD feature file - features/steps/acms_index_data_model_traversal_steps.py: Step definitions - features/environment.py: Added temp_dir cleanup hook - CHANGELOG.md: ACMS entry preserved ISSUES CLOSED: #9579	2026-05-05 17:48:37 +00:00
HAL9000	44f9abe5d1	style(test): use PROJECT_ROOT constant for clearer path resolution in pr_compliance_checklist_steps ... Applies reviewer suggestion from PR #10071 review #7500 (comment #249234): replace chained .parent calls with a named PROJECT_ROOT constant for improved readability and maintainability. ISSUES CLOSED: #9824	2026-05-05 17:18:16 +00:00
HAL9000	c8e713e50b	style(test): fix ruff format in pr_compliance_checklist_steps.py ... Collapse multi-line assert into single line to satisfy ruff format check. The CI lint job runs both ruff check and ruff format --check; the format check was failing because the assert statement used unnecessary parentheses across three lines. ISSUES CLOSED: #9824	2026-05-05 17:18:16 +00:00
HAL9000	63241f1859	fix(agents): add mandatory PR compliance checklist to implementation-supervisor ... Workers were systematically omitting CHANGELOG.md, CONTRIBUTORS.md, and commit footer (ISSUES CLOSED: #N), causing all PRs to be blocked from merge. Added a mandatory 8-item PR Compliance Checklist to the worker prompt body in implementation-supervisor.md that supervisors must pass to every worker. The checklist covers: 1. CHANGELOG.md update under [Unreleased] 2. CONTRIBUTORS.md update 3. Commit footer with ISSUES CLOSED: #N 4. CI verification (all quality gates green) 5. BDD/Behave test coverage 6. Epic reference in PR description 7. Labels applied via forgejo-label-manager 8. Milestone assignment Also removed @tdd_expected_fail tag from PlanContextInheritance test in depth_breadth_projection.feature (bug #4198 is fixed). Added BDD tests in features/pr_compliance_checklist.feature with 10 scenarios covering all 8 checklist items. ISSUES CLOSED: #9824	2026-05-05 17:18:16 +00:00
HAL9000	6fc294b24b	fix(database/migration_runner): add check_same_thread=False to get_current_revision() SQLite engine ... MigrationRunner.get_current_revision() was creating a SQLite engine without connect_args={'check_same_thread': False}, causing ProgrammingError when called from a different thread than the one that created the engine. This is now consistent with init_or_upgrade() which correctly passes check_same_thread=False for all SQLite engines. Added a Behave scenario to verify that get_current_revision() passes check_same_thread=False when creating a SQLite engine. ISSUES CLOSED: #10952	2026-05-05 11:05:07 +00:00
hurui200320	85c579b51f	fix(cli): display full session IDs in session list output ... Remove the [:8] truncation from session IDs in the Rich table row, Most Recent summary, and Oldest summary fields of list_sessions() and _session_list_dict(). Session IDs are 26-character ULIDs and must be usable directly for copy-paste into session tell, session show, and other session commands. The structured output (JSON/YAML) already used full IDs in the sessions[*].id field, but the summary panel leaked the truncation into those formats as well. Added Behave scenarios: - Rich table displays full 26-character ULIDs (scoped to table region) - Summary panel shows full ULIDs for unnamed sessions (scoped to panel) - Summary panel shows session names for named sessions - Full ULID from list output works with session tell (round-trip, uses parsed ULID, not hardcoded constant) Review Cycle 2 fixes: - docs/specification.md: Updated YAML output example to full ULIDs - docs/showcase/*.md: Updated all example output blocks to full ULIDs - docs/reference/session_cli.md: Replaced placeholder with full ULID - features/session_cli.feature: Consecutive When steps -> And - features/steps/session_cli_steps.py: Summary panel asserts both IDs, 8-char negative guard in table output, ULID capture scoped to table region with fixture verification, named-session absence check for second session - CHANGELOG.md: Added [Unreleased] entry for the behavioral change ISSUES CLOSED: #10970	2026-05-05 10:56:18 +00:00
HAL9000	876a2c6916	fix(data-integrity): Replace unconditional commit with flush in LLMTraceRepository.save() ... Implement dual-path session management in LLMTraceRepository.save(): - UoW mode (explicit session provided): flush only, caller controls commit - Standalone mode (no session): flush + commit + close for durable persistence This resolves three data-integrity violations: 1. Premature commit of outer UoW transactions 2. Loss of rollback capability for subsequent failures 3. Mismatch between class docstring and implementation Also adds: - Input validation: trace must not be None - Updated BDD step definitions to pass session explicitly in UoW scenarios - close() method to _BrokenSession mock for proper cleanup path coverage - CHANGELOG.md entry for issue #7505 - CONTRIBUTORS.md credit for HAL 9000 ISSUES CLOSED: #7505	2026-05-05 09:57:46 +00:00
HAL9000	2240a028c8	fix(tests): replace ThreadPoolExecutor with threading.Thread in RLock concurrency steps ... ThreadPoolExecutor holds internal locks (e.g. _global_shutdown_lock) that can be in a locked state after fork(). The behave-parallel runner uses multiprocessing.Pool with the fork start method, so forked worker processes inherit these locked states and deadlock when trying to create or shut down a ThreadPoolExecutor. Replace both concurrent step implementations with direct threading.Thread usage, following the pattern established in context_tier_thread_safety_steps.py and other step files throughout the codebase. This eliminates the fork+lock deadlock while preserving the deterministic concurrency-counter approach for proving the RLock is released during transport calls.	2026-05-05 09:37:22 +00:00
HAL9000	236499f086	fix(tests): replace timing-based concurrency assertions with deterministic counter approach	2026-05-05 09:37:22 +00:00
HAL9000	60671306ea	fix(tests): fix format and reduce timing thresholds in RLock concurrency tests ... Fixed ruff format violation (single quotes to double quotes) in step definitions file. Reduced SlowMCPTransport delay from 0.3s to 0.15s and tightened the wall-clock assertion from 0.9s to 0.40s to improve CI reliability while still proving concurrent execution. ISSUES CLOSED: #10512	2026-05-05 09:37:22 +00:00
HAL9000	79ecaea18f	fix(tests): use getattr for time._original_sleep fallback in RLock concurrency tests	2026-05-05 09:37:22 +00:00
HAL9000	69b7cd3cb2	fix(mcp): release RLock before transport call in MCPToolAdapter ... MCPToolAdapter.invoke() and discover_tools() held the RLock during the entire transport.call(), blocking all concurrent operations on the adapter. The fix splits both methods into three phases: 1. Acquire lock — validate connection state and inputs 2. Release lock — make the transport call without holding the lock 3. Re-acquire lock (discover_tools only) — update shared state Added Behave BDD tests verifying: - Concurrent invocations complete in parallel (wall-clock < serial time) - Concurrent discoveries complete in parallel - Lock is provably not held during transport calls (cross-thread check) - Validation still occurs under lock before transport call - Disconnected adapter still raises RuntimeError under lock ISSUES CLOSED: #10512	2026-05-05 09:37:22 +00:00
HAL9000	908e53ea85	fix(mcp): release RLock before transport call in MCPToolAdapter ... MCPToolAdapter.invoke() and discover_tools() held the RLock during the entire transport.call(), blocking all concurrent operations on the adapter. The fix splits both methods into three phases: 1. Acquire lock — validate connection state and inputs 2. Release lock — make the transport call without holding the lock 3. Re-acquire lock (discover_tools only) — update shared state Added Behave BDD tests verifying: - Concurrent invocations complete in parallel (wall-clock < serial time) - Concurrent discoveries complete in parallel - Lock is provably not held during transport calls (cross-thread check) - Validation still occurs under lock before transport call - Disconnected adapter still raises RuntimeError under lock ISSUES CLOSED: #10512	2026-05-05 09:37:22 +00:00
hurui200320	7164b04019	refactor(providers): unify provider factory behind single source of truth ... Extract _get_api_key() and _create_provider_instance() as a single internal factory that handles all provider types, including MOCK. Both create_llm() and create_ai_provider() now delegate to _create_provider_instance(), eliminating the duplicated if/elif provider-switching chains and the divergence between the two code paths. Adding a new provider now requires touching exactly one place. Key changes: - New _get_api_key(provider_type) consolidates API key lookup and validation that was previously duplicated across create_llm(), _create_provider_llm(), and every branch of create_ai_provider(). - New _create_provider_instance() is the unified raw-LLM factory, replacing _create_provider_llm() and the specialised adapter constructors in create_ai_provider(). MOCK is now handled here via FakeListLLM. - create_ai_provider() wraps all providers uniformly in LangChainChatProvider instead of using specialised adapters (OpenAIChatProvider, AnthropicChatProvider, GoogleChatProvider, OpenRouterChatProvider) for the four primary providers. - create_llm() drops its own API key validation block; validation now happens inside _create_provider_instance() via _get_api_key(). - All BDD scenarios updated: _create_provider_llm references become _create_provider_instance, and provider-type assertions on create_ai_provider results are updated to LangChainChatProvider. Review fixes (Cycle 2): - Add _coerce_env_bool() for consistent boolean env-var coercion in CLEVERAGENTS_ALLOW_MOCK_PROVIDER checks. - Introduce _ApiKeyMissing sentinel to distinguish 'not provided' from None in provider configuration. - Set MOCK supports_streaming=False in DEFAULT_CAPABILITIES to align with FakeListLLM semantics. - Make MOCK is_configured conditional on CLEVERAGENTS_ALLOW_MOCK_PROVIDER. - Use FakeListLLM(responses=['mock response'] * 10) to prevent IndexError in multi-step workflows. - Strip max_retries from kwargs before passing to LangChain constructors. - Extract _resolve_provider_type() helper shared by create_llm() and create_ai_provider() to reduce duplication and improve readability. - Ensure __api_key_sentinel flows through factory_kwargs even when api_key is None, avoiding double _get_api_key() calls. - Add CHANGELOG env-var documentation and providers.md env-var table. - Resolve template DB lock issues by persisting in-memory SQLite via VACUUM INTO instead of direct file creation on tmpfs. ISSUES CLOSED: #10949	2026-05-05 06:16:46 +00:00
HAL9000	39175dd284	fix(git_tools): eliminate TOCTOU race in _get_base_env() with double-checked locking ... Add module-level _BASE_ENV_LOCK: threading.Lock and replace the bare if _BASE_ENV is None assignment with double-checked locking. The outer check keeps the warm-cache path lock-free; the inner check inside with _BASE_ENV_LOCK prevents duplicate initialisation when two threads race on the very first call. Add three BDD scenarios in features/git_tools.feature (step definitions in features/steps/git_tools_thread_safety_steps.py) verifying caching identity, content correctness, and thread safety under 20 concurrent threads using threading.Barrier. Update CHANGELOG.md and CONTRIBUTORS.md per contribution guidelines. ISSUES CLOSED: #7619	2026-05-05 05:25:28 +00:00
HAL9000	6273fcb6e7	style: fix ruff format quote style in tdd_langgraph_execute_closed_state_steps.py ... Apply ruff format to fix single-quote vs double-quote inconsistency in f-string in step assertion, resolving CI lint/format check failure.	2026-05-05 04:04:13 +00:00
HAL9000	331bf343dc	fix(langgraph): use update_state() in LangGraph.execute() instead of direct state assignment ... Add is_closed guard to StateManager.replace_state() so that LangGraph.execute() — which delegates to replace_state() — raises RuntimeError when the StateManager has been closed. This prevents silent state corruption after StateManager.close() is called. Add Behave BDD scenarios verifying the guard and the happy path. ISSUES CLOSED: #9994	2026-05-05 04:04:13 +00:00
CoreRasurae	cecca72b8e	test(behave): Reduce the coverage level to 96.5%	2026-05-05 02:01:49 +00:00
CoreRasurae	cc24d8c8ac	fix(cli): remove legacy plan commands from help output ... Completely removed all legacy plan commands from the CLI: - Removed tell, build, new, current, cd, continue CLI commands - Removed programmatic wrapper functions (tell_command, build_command, etc.) - Removed legacy deprecation message - Updated help text to indicate V3 Plan Lifecycle exclusively - Removed stale references to tell/build in help output and command validation Removes the legacy 'tell' and 'build' CLI shortcuts from main.py: - Removed echo lines advertising tell/build commands - Removed tell/build from valid_cmds list - Removed tell/build from _LIGHTWEIGHT_COMMANDS frozenset - These dead entries were preventing helpful error messages Test infrastructure improvements: - Event bus exception test: Patch the module-level logger during emit() so that structlog.testing.capture_logs() can capture the logs. Without patching, the module-level logger created at import time is not captured by the context manager. - Session create/list commands: Suppress cleveragents.mcp logger to CRITICAL level during JSON/YAML output formatting to prevent health check warnings with ANSI codes from being written to stdout before JSON output, which breaks JSON parsing. - Extended plan_cli_coverage_boost with scenarios for estimation_result, invariants, execution_environment, validation_summary, and checkpoint coverage in _plan_spec_dict Documentation: - Created docs/Legacy_to_V3_Guide.md with comprehensive migration instructions - Updated CONTRIBUTING.md to document removal of legacy workflow - Updated CHANGELOG.md to reference issue #4181 instead of PR #10800 ISSUES CLOSED: #4181	2026-05-05 02:01:49 +00:00
CoreRasurae	c7208d8a18	test(behave): Create new tests for coverage increase	2026-05-05 02:01:49 +00:00
HAL9000	d567b2e911	feat(cli): add actor context clear command (#6370) ... ISSUES CLOSED: #6370	2026-05-05 01:33:31 +00:00
brent.edwards	b483ee4786	Merge branch 'master' into bugfix/m3-actor-run-response	2026-05-04 11:39:57 -07:00
HAL9000	a998c5a0bf	fix(test): add root_plan_id to raw SQL in plan_phase_migration constraint tests ... The step_try_insert_plan_with_phase_and_state function was using raw SQL to insert a plan with an invalid phase value to test the CHECK constraint. However, the raw SQL was missing the root_plan_id column, which is NOT NULL in the schema. This caused the insert to fail with a NOT NULL violation instead of the intended CHECK constraint violation on the phase column. This fix adds root_plan_id to the raw SQL INSERT statement, setting it to the same value as plan_id (for a root plan). This allows the test to properly exercise the CHECK constraint on the phase column, ensuring the test fails for the correct reason. ISSUES CLOSED: #9411	2026-05-03 23:11:39 +00:00
hurui200320	9b7a0543d0	fix(providers): add OpenRouter support to _create_provider_llm ... Add a ProviderType.OPENROUTER branch to ProviderRegistry._create_provider_llm() so that create_llm("openrouter") returns a configured LangChain BaseLanguageModel instead of raising ValueError("Unsupported provider type: openrouter"). The new branch creates a ChatOpenAI instance with openai_api_base set to the OpenRouter gateway URL and openai_api_key from settings, matching the credentials used by create_ai_provider(). Optional default_headers are sanitized to dict[str, str] (string-coerced keys and values) via inline dict comprehension. Also includes review fixes: - M1: Added openrouter_organization header support (HTTP-Referer, X-Title) - m1: Added explicit ValueError when openrouter_api_key is empty/missing - M2: Added openai_api_key assertions to all OpenRouter BDD scenarios - m2: Added default model fallback scenario for model_id=None - m4: Added empty default_headers edge case scenario - m5: Added extra kwargs forwarding scenario (temperature) - m6: Added empty API key error scenario - m7: Added negative assertion verifying original integer key 123 is absent - m8: Added openai_api_key assertion to sanitized headers scenario ISSUES CLOSED: #10948	2026-05-03 01:37:49 +00:00
HAL9000	f74393bcd5	style(checkpoint-cli): auto-format checkpoint CLI commands and step definitions ... Ruff format was not applied to 2 files added by the checkpoint CLI implementation commit. This ensures CI format check passes. ISSUES CLOSED: #8559	2026-05-03 00:39:15 +00:00