Implement multi-scope agent skill discovery for global, project, and local tiers #9454

Closed
HAL9000 wants to merge 2 commits from fix/agent-skill-multi-scope-discovery into master

2 Commits

Author SHA1 Message Date
HAL9000 2cb92bd102 fix(actor): move namespace filter inside lock in ActorLoader.list_actors
CI / benchmark-publish (pull_request) Has been skipped
CI / build (pull_request) Successful in 1m11s
CI / lint (pull_request) Failing after 1m47s
CI / typecheck (pull_request) Failing after 2m2s
CI / quality (pull_request) Successful in 2m0s
CI / benchmark-regression (pull_request) Failing after 1m42s
CI / helm (pull_request) Successful in 51s
CI / push-validation (pull_request) Successful in 56s
CI / security (pull_request) Successful in 2m40s
CI / e2e_tests (pull_request) Successful in 4m45s
CI / unit_tests (pull_request) Failing after 4m54s
CI / coverage (pull_request) Has been skipped
CI / docker (pull_request) Has been skipped
CI / integration_tests (pull_request) Failing after 6m4s
CI / status-check (pull_request) Failing after 4s
The `list_actors()` method applied its namespace filter outside the
`RLock`, creating a race condition where `self._actors` could be mutated
(concurrent discover calls) between reading and filtering. The entire
filter logic is now performed within the lock to guarantee thread-safe
snapshot consistency when concurrent threads are adding/removing actors.

Also added Behave tests to verify atomic namespace filtering under load.

ISSUES CLOSED: #8660
2026-05-09 12:09:06 +00:00
HAL9000 eb72589260 feat(projects): implement plan correct --mode=revert and --mode=append correction engine (#9599)
CI / benchmark-publish (pull_request) Has been skipped
CI / helm (pull_request) Successful in 42s
CI / lint (pull_request) Failing after 58s
CI / build (pull_request) Successful in 59s
CI / quality (pull_request) Successful in 1m8s
CI / benchmark-regression (pull_request) Failing after 1m19s
CI / security (pull_request) Successful in 1m29s
CI / typecheck (pull_request) Successful in 1m36s
CI / push-validation (pull_request) Successful in 19s
CI / integration_tests (pull_request) Successful in 3m9s
CI / unit_tests (pull_request) Failing after 3m39s
CI / coverage (pull_request) Has been skipped
CI / docker (pull_request) Has been skipped
CI / e2e_tests (pull_request) Successful in 5m9s
CI / status-check (pull_request) Failing after 5s
Add comprehensive BDD tests for the full correction engine supporting both
revert (invalidate subtree + archival) and append (spawn child plan) modes,
including dry-run impact analysis, CLI output formatting (rich/plain/json),
decision tree and influence DAG propagation, validation error handling,
and idempotent dry-run behavior.

Fixes: #9599
Epic: plan-correct-engine

ISSUES CLOSED: #9599
2026-05-08 22:08:29 +00:00