Fix JSON syntax errors in .devcontainer/devcontainer.json (removed
invalid JS-style // comments) and .devcontainer/opencode.json (removed
90+ trailing commas). Apply auto-fixes for end-of-file and trailing
whitespace issues across 100+ files. Fix SIM105 ruff violations in
benchmarks/core_circuit_breaker_bench.py (use contextlib.suppress).
Note: The security fix from issue #7478 (validate_path startswith bypass)
was already delivered to master in commit e18ac5f2. This PR as currently
structured is non-atomic (35 commits across 10+ issues) and needs
significant restructure before merge. This commit only addresses the
CI/pre-commit failures.
ISSUES CLOSED: #7478
29 KiB
This file is a mirror of the authoritative root file. For the latest version, see the root CHANGELOG.md.
Changelog
All notable changes to this project will be documented in this file. The format follows Keep a Changelog.
[Unreleased]
Changed
-
CI coverage job now waits for unit_tests (#10714): Added
unit_teststo theneedslist of thecoveragejob inci.yml. Previously the coverage job ran in parallel with unit tests, which could produce misleading pass results when tests were still in-flight or had already failed. Coverage now only starts after unit tests succeed, eliminating redundant parallel test execution and ensuring coverage results are always meaningful. -
Diagnostics spec examples expanded to all 9 providers (#5320): Updated the
agents diagnosticscommand examples in the specification to show all 9 supported providers (OpenAI, Anthropic, Google, Gemini, Azure, OpenRouter, Cohere, Groq, Together), matching the implementation from PR #3469. Rich, plain, JSON, and YAML example outputs now reflect comprehensive provider coverage with accurate warning counts and per-provider recommendations.
Added
-
SubplanExecutionService lazy wiring in CLI (#10268): Wired
subplan_servicefrom the DI container into_get_plan_executor()soPlanExecutorcan spawn child plans during the Execute phase. WhenSubplanServiceis available butSubplanExecutionServiceis not explicitly injected,_execute_subplans()now lazily creates aSubplanExecutionServiceusing the parent plan'ssubplan_configand the_execute_child_plancallback. Added recursion guard and strategize result validation to_execute_child_planto prevent re-entrant or orphaned child plan execution. -
TDD: MCPToolAdapter.infer_resource_slots() TypeError with null properties (#10470): Added a TDD issue-capture Behave scenario that reproduces the bug where
MCPToolAdapter.infer_resource_slots()raisesTypeErrorwhen the input schema contains{"properties": None}. The test is tagged@tdd_expected_failand will pass (by inversion) until the underlying bug is fixed. -
Architecture Pool Supervisor Milestone Assignment (#7521): Added a "PR Workflow for Major Changes" section to the
architecture-pool-supervisoragent definition documenting the milestone assignment step for spec PRs. The agent now hasforgejo_update_pull_requestpermission to assign PRs to the current active milestone after creation, improving traceability of specification changes within project milestone planning. Includes BDD test coverage for the new workflow documentation and permission configuration. -
Git Worktree TOCTOU Race Condition (#7507): Fixed a Time-Of-Check-To-Time-Of-Use (TOCTOU) race condition in
git_worktree.pythat could causegit worktree addoperations to fail under concurrent execution. The fix replaces the unsafemkdtemp()+rmdir()pattern with a parent-directory approach that maintains the OS-level uniqueness guarantee throughout the entire operation. The parent temporary directory is now persisted and properly cleaned up on both success and failure paths. Comprehensive BDD test coverage validates the fix under concurrent execution and confirms proper cleanup behavior.
Fixed
-
Actor namespace/name disambiguation (#11254): When action YAML references actors using
namespace/nameformat (e.g.strategy_actor: local/my-strategist), the_parse_actor_name()functions no longer mistake the namespace prefix for an LLM provider name. A new_is_known_provider()utility checks whether the first slash-separated segment matches a knownProviderType(e.g.openai,anthropic); if not, the input is treated as namespace/name.PlanLifecycleServicenow providesresolve_actor_provider_model()to resolve namespaced references to their underlyingprovider/modelvia the actor registry. All affected call sites —StrategyActor,LLMStrategizeActor,LLMExecuteActor, andSessionWorkflow._resolve_llm()— pre-resolve actor names before passing them to the LLM provider, fixing theValueError: Unknown provider typecrash when using namespaced actor references. -
Built-in actors v3 YAML format (#10883): Fixed
agents actor runfailing for built-in actors (e.g.,openai/gpt-4,anthropic/claude-3-opus) due to missing v3typefield in stored configuration.ActorRegistry.ensure_built_in_actors()now generates and persists v3 YAML text withtype: llmanddescriptionfields, ensuring built-in actors work identically to custom actors. The_generate_builtin_actor_yaml()helper creates spec-compliant YAML that passesReactiveConfigParser._is_v3_format()validation. Includes BDD scenarios and unit tests covering YAML generation, schema validation, and multiple provider handling. -
Atomic
server_connectconfig writes (#993): Fixedserver_connectincli/commands/server.pyto write all three config values (server.url,server.namespace,server.tls-verify) atomically. A snapshot of the config file is taken before any writes; if anyset_value()call fails, the snapshot is restored and compensatingCONFIG_CHANGEDevents are emitted for already-applied keys so the audit trail reflects the rollback. Addedemit_config_changed()helper toConfigServicefor decoupled event emission in rollback flows. Addedclose()method toReactiveEventBusfor proper resource cleanup in tests. Resolved merge conflict inconfig_service.pyintegrating the PR'semit_config_changed()helper with master's scoped config infrastructure. Removed# type: ignore[assignment]by introducing a typed_AutoDiscoversentinel class. BDD regression coverage infeatures/tdd_server_connect_atomic_writes.feature. -
Atomic
load_from_metadatafor Autonomy Guardrails (#7504): FixedAutonomyGuardrailService.load_from_metadata()to validate bothAutonomyGuardrailsandGuardrailAuditTrailmodels before writing either to state, ensuring atomic updates. Previously, a validation failure on the audit trail after guardrails were already written would leave the system in an inconsistent state with partial updates. The method now uses a two-phase validate-then-write approach: all model validation occurs in Phase 1, and state mutations only happen in Phase 2 after all validations succeed. -
ReactiveConfigParser route synthesis for v3 actors (#10807): Fixed
agents actor runsilently returning empty output for v3type:llmactors._build_from_v3()and_build()now synthesise a default single-node graph route when agents are created without explicit routes, ensuringrun_single_shot()can invoke the LLM viaGraphExecutor. The nestedactors:map format also translates the v3actor: "provider/model"key into separateproviderandmodelkeys so the correct LLM provider is instantiated. -
ActorRegistry.add() spec-compliant YAML support (#4466): The registry now accepts actor YAML using the spec's
actors:map format with nestedconfig:blocks, in addition to the legacy top-levelprovider/modelformat. Theunsafeflag and graph descriptor from nested config are now correctly preserved during registration. Multi-actor YAML (>1 entry inactors:/agents:map) is now rejected byadd()with aValidationError. Nestedconfig.optionsare now correctly preserved. Theunsafecoercion now uses strictis True or == 1instead ofbool()to prevent truthy non-boolean YAML values (e.g.unsafe: "no") from being treated as unsafe. -
UKO Runtime Layer 2 (Paradigm) Indexing (#9351): Added missing
rdf:type uko-oo:Classtriple emission inPythonAnalyzer._extract_class()so that Python class definitions are now correctly classified at layer 2 (paradigm/OO) in addition to layer 3 (technology). Added the corresponding Behave scenarioIndexing a Python file populates layer 2 (paradigm)tofeatures/uko_runtime.feature, completing four-layer guarantee verification for the UKO runtime. -
Actor CLI v3 YAML Schema Support (#6283): Fixed three components to add full v3
ActorConfigSchemasupport to the actor CLI registration and execution paths.ActorConfiguration.from_blob()now detects v3 format (top-leveltypekey ofllm/graph/tool) and correctly extracts provider, model, and graph descriptors — includingtype: toolactors without amodelfield.ActorRegistry.add()validates against the full Pydantic v2 schema, persistsskills/lsp/descriptionin the config blob, and compiles graph actors with proper metadata.ReactiveConfigParser._build_from_v3()now uses correctsource/targetedge keys (fixingKeyErrorinto_graph_config()), handlesconfig: nullnodes without crashing, propagatescontext_view/memory/context/env_vars/response_format/lsp_capabilities/lsp_context_enrichmentinto agent configs, and validatesentry_nodeagainst the nodes map. Exception handling narrowed from broadexcept Exceptionto specificNotFoundErrorandActorCompilationError. v3 registration logic extracted tov3_registry.pyto keepregistry.pyunder the 500-line limit. 19 BDD scenarios cover all v3 paths including tool actors, update mode, LSP dict bindings, and field propagation. -
TDD Non-AssertionError Guard Visibility (#8294):
apply_tdd_inversioninfeatures/environment.pynow emits its non-assertion exception guard warning to both the structured logger andstderrvia a new_warning_with_stderrhelper. This makes the guard firing visible in standard Behave console output and CI log snippets where the structured logging sink may not be displayed. BDD infrastructure coverage added: a new scenario intdd_expected_fail_infrastructure.featureasserts that the warning is emitted to stderr when a non-AssertionError exception is encountered in an@tdd_expected_failscenario, and a second scenario asserts the warning is NOT emitted when the exception is anAssertionError. TheCONTRIBUTING.mdnow documents that@tdd_expected_failstep definitions must signal expected failures viaAssertionError. -
Parallel Behave Runner Log Noise Reduction (#8351): The parallel behave runner now suppresses captured stdout/stderr for passing worker chunks and only replays diagnostics for failed, errored, or crashed chunks. This makes failure output significantly easier to spot in CI and local runs. A worker crash (unhandled exception) is detected via an all-zero summary and the captured traceback is always surfaced.
-
Bug Hunt Pool Supervisor Non-Blocking Tracking: Updated
bug-hunt-pool-supervisorto make the automation tracking step non-blocking. Theautomation-tracking-managercall in step 5 is now best-effort — if it does not complete within a reasonable time or fails, the supervisor skips it and continues to the next cycle. Added explicit rule 9 clarifying that tracking must never block the main loop. Core functionality (module scanning and worker dispatch) takes priority over status reporting. -
Name Validator Server-Qualified Format (#9074): Updated actor, skill, and tool name validators to accept the spec-required
[[server:]namespace/]nameformat. Previously, server-qualified names likedev:freemo/custom-analysiswere incorrectly rejected. Added BDD scenarios for server-qualified name acceptance and rejection. All three validators (ActorConfigSchema.validate_name,NAMESPACED_NAME_RE,_TOOL_NAME_PATTERN) now correctly support optional server prefixes while maintaining backward compatibility with existingnamespace/namenames. -
Automation Profile Silent Fallback (#8232):
_resolve_profile_for_planinPlanLifecycleServicenow raises a clearValidationErrorwhen a plan's automation profile name is not a known built-in profile, instead of silently falling back to"manual". Users who configured custom automation profiles (e.g."semi-auto","acme/strict") will now receive an actionable error message listing available built-in profiles. The resolved profile name is also logged at debug level for observability.
Added
-
Wired
StrategyActorinto the real plan execution path:_get_plan_executorinplan.pynow resolves the strategy actor viaresolve_strategy_actor()(reading theactor.default.strategyconfig key) instead of always constructingLLMStrategizeActor.run_strategizeinPlanExecutornow passesresources(derived fromplan.project_links) andproject_contextto the actor so the LLM prompt receives full project context. Strategy decisions are serialised as JSON inplan.error_details["strategy_decisions_json"]so_build_decisionscan reconstruct the full hierarchy (dependency ordering, parent/child structure) during Execute instead of rebuilding fromdefinition_of_done.StrategizeStubActor.executeaccepts**kwargsfor forward-compatibility. Added BDD coverage for the stored-JSON path, corrupt-JSON fallback, resource-passing, and stub extra-kwargs scenarios. (#828) -
TDD Issue-Capture Test Activation (#7025): Replaced 234 bare
@skiptags across 82 Behave feature files with the correct@tdd_expected_fail @tdd_issue @tdd_issue_<N>tag system. Scenarios whose referenced bugs were already fixed had@tdd_expected_failremoved and now run as permanent regression guards. Net result: 629 features active in CI (up from ~545), zero@skiptags remain. -
Git Worktree Sandbox Apply (#4454): The
plan applycommand now merges LLM-generated changes viagit mergefrom an isolated worktree branch instead of flatshutil.copy2. Displays spec-aligned Apply Summary (plan ID, artifacts, insertions/deletions, project, timestamp), Sandbox Cleanup panel, and✓ OK Changes appliedfooter. Non-git projects fall back to the original flat file copy. -
Context Hydration Fix (#4454): Fixed
ContextFragmentmetadata types (detail_depthandrelevance_scoremust be strings, not int/float) that caused Pydantic validation errors during context assembly, resulting in the LLM receiving zero file context. -
Automation Tracking System: Replaced shared session state issue tracking with individual per-agent tracking issues. Each agent now creates its own
[AUTO-<PREFIX>]titled issues with standardized headers, reporting intervals, and health indicators. Agents:session-persister,implementation-orchestrator,system-watchdog,backlog-groomer,human-liaison. Documentation atdocs/development/automation-tracking.md. -
Automated Health Monitoring and Recovery: The
system-watchdognow runsaudit_automation_tracking_health()every 5 minutes, detecting stalled agents when tracking issues are >20% overdue from their declared reporting interval. On detection, it terminates stalled sessions via the OpenCode Server API, performs root-cause analysis, creates high-priority diagnostic issues, and closes stale tracking issues with recovery notes. -
Centralized Label Management (
forgejo-label-manager): A new specialized subagent centralizes all Forgejo label operations across the agent system. Enforces the organization-level label system, prohibits label creation, and validates label compliance. Agentsbacklog-groomer,human-liaison,project-owner,epic-planner,new-issue-creator, andissue-state-updaternow delegate all label operations to this subagent. -
PR-Issue Label Synchronization: PRs now inherit
Priority/,MoSCoW/,Points/, andState/labels from their associated issues at creation time (pr-api-creator). Thebacklog-groomeradds a continuous Pass 19 for ongoing PR-issue label synchronization. Theissue-state-updatersyncs PR state labels whenever issue states change. -
Automation Tracking Announcements: Extended
automation-tracking-managerwith announcement issue support (CREATE_ANNOUNCEMENT_ISSUE,CLOSE_ANNOUNCEMENT_ISSUE,LIST_TRACKING_ISSUES,READ_ANNOUNCEMENTS,REVIEW_OWN_ANNOUNCEMENTS). Supervisors and workers now read critical announcements before each cycle for cross-agent awareness. Priority-based filtering (Critical/High/Medium/Low) reduces noise. Backlog-groomer performs intelligent cleanup with age thresholds by priority. -
PR Agent Reorganization: All PR-related agents renamed and reorganized to follow the
*-pool-supervisornaming pattern. New agents added:pr-editor(safe PR editing with description preservation),pr-manager(unified PR interface), andpr-merge-pool-supervisor(automated PR merging supervisor). Renamed:pr-api-creatortopr-creator,pr-checkertopr-ci-test-fixer,pr-status-checkertopr-status-analyzer,pr-self-reviewertopr-reviewer,pr-fix-orchestratortopr-fix-pool-supervisor. -
Automated PR Merging (
pr-merge-pool-supervisor): New supervisor continuously monitors for merge-ready PRs and merges them automatically when all criteria are met (approvals, CI passing, no conflicts). Supports both formal reviews and comment-based approvals (LGTM, ready to merge, etc.). -
Implementation Worker Workflow Completion:
implementation-workernow implements work claiming protocols with conflict detection, comprehensive review feedback handling with intelligent parsing, sophisticated merge conflict resolution with multiple strategies, and parallel subtask execution with wave-based dependency analysis. Pass rate improved from 48.15% to 84.8%. -
Container Resource Stop Support:
agents resource stopnow correctly stopscontainer-instanceanddevcontainer-instanceresource types. -
Centralized Automation Tracking Manager (
automation-tracking-manager): The manager is now the single interface for all tracking issue operations (CREATE_TRACKING_ISSUE,UPDATE_TRACKING_ISSUE,CLOSE_TRACKING_ISSUE,READ_TRACKING_STATE,GET_NEXT_CYCLE_NUMBER). Agents delegate to the manager rather than calling the Forgejo API directly, ensuring sequential cycle numbers across restarts and preventing duplicate issues. Migrated agents includesystem-watchdog,implementation-pool-supervisor,timeline-update-pool-supervisor,project-owner-pool-supervisor,product-builder, andbacklog-grooming-pool-supervisor. The legacyshared/automation_tracking.mdmodule was removed. -
Documentation Writer Tracking (
docs-writer): The documentation writer now participates in the automation tracking system by creating individual[AUTO-DOCS] Documentation Report (Cycle N)issues every 10 cycles (~3.3 hours). The manager applies the mandatoryAutomation Trackinglabel automatically, while teams may add additional workflow labels as needed. Seedocs/development/automation-tracking.mdand the newdocs/development/docs-writer.mdreference. -
ACMS / UKO API Documentation (
docs/api/acms.md): Added comprehensive API reference for thecleveragents.acmspackage covering the four-layer UKO ontology hierarchy,VocabularyRegistry,ProvenanceInfo,UKOClass,UKOProperty,UKOVocabulary,Layer2Dependency,ParadigmVocabulary,DetailLevelMapBuilder, and all Layer 3 language vocabulary types (Python, TypeScript, Rust, Java). The new page is linked from the API Reference index and the MkDocs navigation.
Changed
-
product-builderWorker Allocation Tier Comments (#8169): Clarified theN_FULLtier comment to explicitly document that PR fixing is handled byimplementation-pool-supervisorvia its PR-First Priority rule. Updated theN_QUARTERcomment to enumerate the pools it covers (UAT, bug hunting, test infra). Prevents confusion about which supervisor handles PR fix work. -
Decision Tree Full ULID Display (#5825): The
agents plan treecommand now displays full 26-character ULIDs for all decisions instead of truncating them to 8 characters. This enables users to copy decision IDs directly from tree output and use them in follow-up CLI commands likeagents plan correctwithout manual ID reconstruction. Added "Decision IDs (for correction)" section with human-readable labels for easy reference. Applies to both table and rich/plain text output formats. -
Automation Tracking Format: All automation tracking issues now use a standardized header format with mandatory
Reporting Interval: <interval> (Next report expected: <ts>)declarations, enabling precise staleness detection. -
PR Review Policy: Reduced PR review requirement from 2 approvals to 1. Self-approval is now permitted including for automated bot PRs. Approval can be a formal review OR an approval comment (LGTM, Approved, ready to merge).
-
Label Delegation Enforcement:
automation-tracking-managernow enforces delegation toforgejo-label-managerfor all label operations, preventing "invalid label ID" errors and ensuring label application uses correct name-to-ID mapping. -
Automation Tracking Label Guidance: Documentation now clarifies that the manager automatically applies the
Automation Trackinglabel and that additional labels such asType/Automation,State/In Progress, orPriority/Mediumremain optional workflow choices rather than mandatory. -
Automation Tracking Agent Prefix Registry: Expanded from 5 agents to 18 agents. New prefixes include
AUTO-DOCS,AUTO-REV-POOL,AUTO-UAT-POOL,AUTO-BUG-POOL,AUTO-INF-POOL,AUTO-ARCH,AUTO-EPIC,AUTO-EVLV,AUTO-GUARD,AUTO-SPEC,AUTO-TIME,AUTO-PROJ-OWN, andAUTO-PROD-BLDR.
Fixed
-
Plan Concurrency Race Condition (#7989): Fixed critical race condition in
execute_plan()andapply_plan()where concurrent CLI/worker sessions could simultaneously modify the same plan, corrupting plan state.LockServiceis now wired into the plan lifecycle with plan-level advisory locking. Each invocation generates a unique caller identity (UUID) to prevent re-entrant lock acquisition by concurrent sessions on the same plan. Concurrent attempts now raiseLockConflictErrorinstead of silently racing. Lock is acquired before phase transition and released in afinallyblock to ensure cleanup even on error. -
--format colorANSI Output (#7910): Fixedformat_outputrouting thecolorformat option to_format_plain, which produced plain uncoloured text instead of ANSI escape sequences. Thecolorformat is now routed toformat_output_sessionwhich uses theColorMaterializerto emit proper ANSI-coloured output.--format plainand all other formats remain unaffected. -
ContextTierService Thread Safety (#7547): Added
threading.RLocktoContextTierServiceto preventRuntimeError: dictionary changed size during iterationand data corruption under concurrent plan execution. All public methods (store,get,promote,demote,evict_lru,enforce_staleness,get_metrics,get_all_fragments,get_hot_fragments,get_for_actor,get_scoped_view,get_scoped_by_resource,get_scoped_metrics) now acquire the reentrant lock before accessing the hot/warm/cold tier dicts. The service was previously documented as single-threaded but registered as a DI Singleton, causing potential data corruption when parallel subplans shared the same instance. TheTierRuntimeMixin.enforce_staleness()andScopedTierMixin.get_scoped_by_resource()/get_scoped_metrics()methods are also protected. The DI container registration asproviders.Singletonis now correct and safe. -
TOCTOU Race Condition in Git Worktree Sandbox (#7507): Fixed Time-Of-Check-To-Time-Of-Use race condition in
GitWorktreeSandbox.create()by replacing unsafe mkdtemp+rmdir pattern with persistent parent directory approach. Parent directory is now held throughout operation lifetime and properly cleaned up in all error paths (timeout, CalledProcessError, OSError) and in the cleanup() method, eliminating race window where another process could claim the worktree path. Comprehensive BDD coverage added for all error-path cleanup branches. -
Validation Gate Empty-Run Guard (#7508): Fixed
ApplyValidationSummary.all_required_passedreturningTruewhen zero validations were run, silently bypassing the apply gate. The property now returnsFalsewhen the validation result set is empty (is_emptyisTrue), ensuring that apply is blocked unless at least one validation was actually executed. Also addedrequired_totalproperty for completeness. Updatedconsolidated_validation.featurescenarios to reflect the corrected blocking behavior for empty summaries and no-attachment runs. -
ACMS context tier hydration:
ContextTierServiceno longer starts empty on every CLI invocation. A newcontext_tier_hydrator.pyreads files from linked project resources (viagit ls-filesoros.walk), createsTieredFragmentobjects, and stores them in the tier service before context assembly inLLMExecuteActor.execute(). The LLM now receives real file context during plan execution. Respects max file size (256 KB), total budget (10 MB), binary file exclusion, and.git/node_modules/__pycache__directory skipping. (#1028) -
Sandbox root wiring:
_get_plan_executor()now passessandbox_root=.cleveragents/sandbox/, so LLM file output (FILE:blocks) is written to disk during the execute phase. (#4222) -
SubplanExecutionService fail_fast cancellation (#7582): Fixed a race condition where already-running parallel subplans were not cancelled when
fail_fastfired. Previously,Future.cancel()only prevented queued futures from starting but had no effect on in-flight futures that completed afterstop_flagwas set -- theirCOMPLETEresults were incorrectly included in the merge output. The fix adds a post-completion guard that overrides any non-ERRORED/non-CANCELLEDresult toCANCELLEDwhenstop_flagis active, and clears the associated output to prevent it from entering the merge. Also replaces the O(n) linearstatuslookup in theas_completed()loop with an O(1)status_mapdict pre-computed before the executor block. -
Robot Framework TDD Listener Guards (#5436): Added three guard conditions to the
tdd_expected_fail_listenerend_test()function to prevent blindly inverting ALL test failures to passes, which was masking infrastructure errors and causing flaky CI behavior. Guards: setup/teardown error detection, non-assertion failure detection (infrastructure errors), and dry-run mode detection. Also fixedVariable Should Existsyntax errors in e2e test files and removedtdd_expected_failfrom 4 context assembly e2e tests where bugs were already fixed. -
PluginLoader entry point prefix validation (#7476): Parse entry point targets before import, enforce the module allowlist ahead of loading, and add Behave plus Robot Framework regression coverage to ensure disallowed prefixes never execute untrusted module-level code in either unit or integration flows.
-
issue-state-updaterBash Script Errors: Removed problematic bash script examples that tried to invoketask forgejo-label-manageras a bash command (the Task tool cannot be invoked from bash). Replaced with clear step-by-step operational instructions and direct label management via API. -
automation-tracking-managerLabel Delegation Syntax: Fixed incorrect delegation syntax when callingforgejo-label-manager. The manager now uses correct natural language requests (e.g., "Apply labels to issue #123: Automation Tracking") instead of structured parameters, ensuring tracking issues receive proper labels. -
product-builderMissing Supervisors: Added missingpr-fix-pool-supervisorandpr-merge-pool-supervisorto the product-builder's supervisor launch list (18 total supervisors). Updated all numeric references, pre-flight checklists, and validation logic. -
ActionRepository.update()now uses explicit bulksa_delete()+session.flush()before re-inserting child rows foraction_argumentsandaction_invariants, fixing asqlite3.IntegrityError: UNIQUE constraint failedcrash whenagents plan usewas called on an action that already had arguments registered viaaction create. (#4197)
[3.8.0] -- 2026-04-05
Added
-
Wired Invariant Reconciliation Actor auto-invocation into
PlanLifecycleServicephase transitions (start_strategize,execute_plan,apply_plan). Reconciliation failures now block the transition withReconciliationBlockedErrorand emitINVARIANT_VIOLATEDevents. Post-correction reconciliation runs viaCORRECTION_APPLIEDevent subscription (best-effort). AddedInvariantServiceSingleton provider in the DI container. -
TUI -- Shell danger detection: The TUI shell mode (
!prefix) now detects dangerous command patterns before execution. A configurable pattern registry classifies commands by danger level (warning, critical) and surfaces a user warning overlay before proceeding. Patterns cover destructive filesystem operations, privilege escalation, network exfiltration, and more. (#1003) -
TUI -- Permission Question Widget: A new inline
PermissionQuestionWidgetrenders permission requests directly in the conversation stream for single-file operations. Users can allow/reject with single-key shortcuts (a/A/r/R), navigate with arrow keys, confirm withEnter, or pressvto open the full