cleveragents/cleveragents-core
4
0
Fork 3
Code Issues 6.0k Pull Requests 115 Actions Packages Projects Releases Wiki Activity

fix(cli): implement real actor execution and spec-required output panels in agents session tell #3473

Open
freemo wants to merge 2 commits from fix/session-tell-stub-missing-panels-and-actor-execution into master
pull from: fix/session-tell-stub-missing-panels-and-actor-execution
merge into: cleveragents:master
Conversation 24 Commits 2 Files Changed 15
+936 -62

2 Commits

Author SHA1 Message Date
controller-ci-rerun 939ae2092c chore: re-trigger CI [controller]
CI / lint (pull_request) Successful in 53s
Details
CI / quality (pull_request) Successful in 55s
Details
CI / typecheck (pull_request) Successful in 1m5s
Details
CI / helm (pull_request) Successful in 27s
Details
CI / security (pull_request) Successful in 1m5s
Details
CI / build (pull_request) Successful in 30s
Details
CI / e2e_tests (pull_request) Failing after 17m41s
Details
CI / integration_tests (pull_request) Failing after 18m8s
Details
CI / unit_tests (pull_request) Failing after 18m9s
Details
CI / benchmark-publish (pull_request) Has been skipped
Details
CI / docker (pull_request) Has been skipped
Details
CI / benchmark-regression (pull_request) Failing after 39s
Details
CI / coverage (pull_request) Failing after 15m48s
Details
CI / status-check (pull_request) Has been cancelled
Details
2026-06-18 10:58:29 -04:00
freemo e1ce238985 fix(cli): implement real actor execution and spec-required output panels in agents session tell
CI / lint (pull_request) Successful in 28s
Details
CI / typecheck (pull_request) Successful in 47s
Details
CI / quality (pull_request) Successful in 36s
Details
CI / security (pull_request) Successful in 1m3s
Details
CI / build (pull_request) Successful in 28s
Details
CI / helm (pull_request) Successful in 24s
Details
CI / unit_tests (pull_request) Failing after 6m48s
Details
CI / docker (pull_request) Has been skipped
Details
CI / coverage (pull_request) Successful in 10m36s
Details
CI / e2e_tests (pull_request) Successful in 17m29s
Details
CI / integration_tests (pull_request) Failing after 22m3s
Details
CI / status-check (pull_request) Failing after 1s
Details
CI / benchmark-publish (pull_request) Has been skipped
Details
CI / benchmark-regression (pull_request) Successful in 57m15s
Details 
Replace stub actor execution in `agents session tell` with real LLM invocation
via the provider registry. Implement all four spec-required Rich output panels:
Plan Request, Commands Executed, Result, and Usage.

Key changes:
- Add `_invoke_actor_llm()` helper that invokes the configured LLM actor with
  session history as context, falling back gracefully if no provider is configured
- Implement non-streaming mode with four Rich panels per spec §"agents session tell"
- Implement streaming mode with Session panel, real-time token streaming, Usage panel,
  and "✓ OK Stream complete" success message
- Add `--format` option supporting json/yaml/plain output paths with structured data
- Update existing Behave tests to match new panel-based output
- Add new `session_tell_panels.feature` with comprehensive panel coverage scenarios
- Update Robot Framework helper to verify Plan Request panel instead of stub output
- Update coverage boost and uncovered branches step files to mock `service.get()`
  and `service.update_token_usage()` which are now called by the tell command

ISSUES CLOSED: #3430
 2026-04-05 21:34:35 +00:00