8 Commits

Author	SHA1	Message	Date
hurui200320	87a7ce35d7	feat(session): implement real LLM actor invocation in session tell ... Replace the M3 stub in `agents session tell` with real orchestrator actor invocation via `SessionWorkflow.tell()`. The stub echoed a canned `Acknowledged: ...` response without calling any LLM or actor; this commit wires up the full pipeline: Architecture: - New `SessionWorkflow` (Application layer) orchestrates LLM invocation for session tell. Accepts a `ProviderRegistry` and `ToolRegistry`; falls back to `FakeListLLM` when no provider is configured. - `LangChainSessionCaller` implements the `LLMCaller` protocol, building history-aware LangChain message lists from the session conversation and invoking the LLM via `ToolCallingRuntime.run_tool_loop()`. - `TellResult` (Pydantic BaseModel) carries the assistant response plus token usage (input_tokens, output_tokens, cost_usd, duration_ms). Domain / service layer: - `SessionActorNotConfiguredError` added to the session domain model; raised when tell is invoked with no actor on the session and no `--actor` override. Clear message; CLI exits with code 1. - `SessionService.get_messages()` abstract method added (+ implementation in `PersistentSessionService`) to load ordered message history. A2A facade: - `A2aLocalFacade` gains `message/send` and `message/stream` standard A2A operation handlers that route to `SessionWorkflow.tell()`. Total supported operations count: 42 → 44. CLI: - `session tell` command delegates to `_build_session_workflow()` (patchable factory) instead of directly calling `SessionService`. - Non-streaming: `SessionWorkflow.tell()` returns `TellResult`; output includes a Usage panel (Rich/Plain) or a `usage` object (JSON/YAML). - Streaming: `SessionWorkflow.tell_stream()` yields tokens; CLI prints them via `console.print` (not raw `sys.stdout.write`). - Token usage recorded via `SessionService.update_token_usage()` in both paths. Tests: - New Behave feature `session_tell_llm.feature` (4 scenarios): real LLM response persisted; streaming yields tokens; no-actor exits code 1; `--actor` override resolves correctly. - New Robot suite `session_tell_llm.robot` (4 tests): end-to-end with stub LLM injected via monkey-patching `_resolve_llm`. - Updated all existing `session tell` test steps to patch `_build_session_workflow` returning a mock `TellResult` so tests remain isolated from the LLM layer. - Updated operation-count assertions (42 → 44) in `a2a_cli_facade_integration`, `consolidated_misc`, `m6_autonomy_acceptance` feature files and steps. Cycle 7 (Review ID 8088) fixes: - Blocker 4: Split `session_workflow.py` (was 801 lines) into two files: `session_workflow.py` (467 lines) and `session_caller.py` (318 lines). Extracted: `LangChainSessionCaller`, `extract_content`, `extract_token_usage`, `estimate_cost`, `estimate_tokens`, `history_to_langchain_messages`, and stub classes. - Blocker 5: Route CLI streaming path through `_facade_dispatch("message/stream", ...)` instead of directly calling `workflow.tell_stream()`. The facade's `_handle_message_stream` falls back to non-streaming with `streamed: false` (acceptable for this milestone per the spec). - Updated streaming test assertion to validate full response presence (no longer checks token-by-token word positions, since the facade fallback returns a complete message). ISSUES CLOSED: #5784	2026-05-11 04:39:29 +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	f2b60cc3d6	fix(cli): restore session list json contract ... Ensure the session list helper only emits the documented fields so downstream consumers stay compliant, and tighten the Behave coverage to enforce the contract.\n\nISSUES CLOSED: #6436	2026-04-26 19:40:00 +00:00
HAL9000	b92f0270c2	fix(cli): fix session show/list JSON output to not redact input messages (#6436) ... ISSUES CLOSED: #6436	2026-04-26 19:40:00 +00:00
freemo	54285e0f33	fix(cli): correct session show rich output Session Summary panel field labels and fields ... Reviewed and APPROVED. Closes #3040.	2026-04-05 21:12:13 +00:00
freemo	a0df5a4cd0	fix(cli): wrap format_output() in spec-required JSON/YAML envelope across all CLI commands ... Implements the spec-required JSON/YAML output envelope for all CLI commands that use format_output(). The envelope structure is: { "command": "<command that was run>", "status": "ok" | "warn" | "error", "exit_code": 0, "data": { ... command-specific payload ... }, "timing": { "duration_ms": 123 }, "messages": [{ "level": "ok", "text": "..." }] } Changes: - Add _build_envelope() helper to construct the spec-required envelope - Add optional command, status, exit_code, messages parameters to format_output() - Wrap json/yaml output in the envelope; plain/table/rich/color unchanged - Add timing measurement (duration_ms) to all json/yaml outputs - Add new BDD feature file (cli_json_envelope.feature) with 14 scenarios testing envelope field presence, values, and data payload - Update 14 existing step files to unwrap the envelope when checking specific data keys (backward-compatible via _unwrap_envelope() helper) Closes #3431	2026-04-05 19:48:40 +00:00
freemo	e4493e8978	fix(cli): correct session show rich output Session Summary panel field labels and fields ... Fixes three deviations from docs/specification.md in the agents session show rich output Session Summary panel: 1. Renamed 'Session ID:' label to 'ID:' to match spec exactly. 2. Removed 'Namespace:' field which is not part of the spec. 3. Added 'Automation:' field sourced from session.metadata['automation_profile'], defaulting to '(none)' when not set. 4. Field order now matches spec: ID → Actor → Messages → Created → Updated → Automation. Updated features/session_cli.feature to assert the corrected field labels are present ('ID:', 'Actor:', 'Messages:', 'Created:', 'Updated:', 'Automation:') and that the removed fields ('Session ID:', 'Namespace:') are absent. Added 'the session CLI output should not contain' step definition to features/steps/session_cli_steps.py to support the new negative assertions. ISSUES CLOSED: #3040	2026-04-05 19:44:08 +00:00
brent.edwards	04c8a886db	feat(cli): add session commands	2026-02-20 01:10:19 +00:00