fix(cli): fix session tell output to return spec-required structured response #6729

Open
HAL9000 wants to merge 5 commits from fix/issue-6452-session-tell-output into master

5 Commits

Author SHA1 Message Date
controller-ci-rerun 4e49209090 chore: re-trigger CI [controller]
CI / lint (pull_request) Failing after 45s
CI / quality (pull_request) Successful in 44s
CI / typecheck (pull_request) Successful in 56s
CI / push-validation (pull_request) Successful in 33s
CI / helm (pull_request) Successful in 38s
CI / build (pull_request) Successful in 45s
CI / e2e_tests (pull_request) Successful in 3m21s
CI / integration_tests (pull_request) Failing after 15m9s
CI / unit_tests (pull_request) Failing after 15m43s
CI / security (pull_request) Failing after 15m47s
CI / benchmark-publish (pull_request) Has been skipped
CI / coverage (pull_request) Has been skipped
CI / docker (pull_request) Has been skipped
CI / benchmark-regression (pull_request) Has been skipped
CI / status-check (pull_request) Failing after 3s
2026-06-18 10:58:57 -04:00
HAL9000 8ec556bd77 fix(cli): move in-function imports to module level for PR #6729
CI / benchmark-publish (pull_request) Has been skipped
CI / integration_tests (pull_request) Failing after 14s
CI / e2e_tests (pull_request) Failing after 11s
CI / lint (pull_request) Failing after 18s
CI / typecheck (pull_request) Failing after 18s
CI / unit_tests (pull_request) Failing after 16s
CI / quality (pull_request) Failing after 16s
CI / security (pull_request) Failing after 16s
CI / coverage (pull_request) Has been skipped
CI / docker (pull_request) Has been skipped
CI / benchmark-regression (pull_request) Has been skipped
CI / push-validation (pull_request) Successful in 31s
CI / helm (pull_request) Successful in 38s
CI / build (pull_request) Successful in 1m2s
CI / status-check (pull_request) Failing after 3s
- Moved 'import typer' from render_tell_output() to top of session_tell.py
- Moved 'import json' from build_export_content() to top of session_helpers.py
- Fixes CI lint failure: in-function imports violate CONTRIBUTING.md rule
- All imports now at module level per project standards

ISSUES CLOSED: #6452
2026-05-08 16:00:16 +00:00
HAL9000 5c7d613cc6 chore(contributors): add HAL9000 contribution entry for PR #6729
CI / helm (pull_request) Successful in 44s
CI / push-validation (pull_request) Successful in 31s
CI / build (pull_request) Successful in 57s
CI / lint (pull_request) Failing after 1m13s
CI / typecheck (pull_request) Successful in 1m22s
CI / quality (pull_request) Successful in 1m26s
CI / security (pull_request) Successful in 1m37s
CI / coverage (pull_request) Has been skipped
CI / e2e_tests (pull_request) Successful in 3m29s
CI / integration_tests (pull_request) Successful in 4m29s
CI / unit_tests (pull_request) Failing after 4m42s
CI / docker (pull_request) Has been skipped
CI / status-check (pull_request) Failing after 2s
CI / benchmark-publish (pull_request) Has been skipped
CI / benchmark-regression (pull_request) Has been skipped
- Added HAL9000 to Contributor list with session tell structured output (#6452) reference
- Documented spec-compliant CLI panels contribution details

ISSUES CLOSED: #6452
2026-05-08 10:53:18 +00:00
HAL9000 75f067d71f fix(cli): fix session tell output to return spec-required structured response
CI / benchmark-publish (pull_request) Has been skipped
CI / helm (pull_request) Successful in 41s
CI / lint (pull_request) Failing after 1m47s
CI / build (pull_request) Successful in 47s
CI / push-validation (pull_request) Successful in 50s
CI / quality (pull_request) Successful in 1m55s
CI / security (pull_request) Successful in 2m2s
CI / typecheck (pull_request) Successful in 2m9s
CI / coverage (pull_request) Has been skipped
CI / benchmark-regression (pull_request) Has been skipped
CI / e2e_tests (pull_request) Successful in 3m42s
CI / integration_tests (pull_request) Successful in 4m26s
CI / unit_tests (pull_request) Failing after 6m52s
CI / docker (pull_request) Has been skipped
CI / status-check (pull_request) Failing after 3s
- Extract tell helpers into session_tell.py and session_helpers.py to keep
  session.py under the 500-line project limit (497 lines)
- Fix import sort order (Ruff I001) and remove line length violations (E501)
- Fix step_tell_nonexistent to mock service.get instead of service.append_message
- Add @tdd_issue @tdd_issue_6452 regression tags to all three Tell Behave scenarios
- Add Commands Executed and Result panel assertions to Tell scenario
- Replace "Stub" in status_text with spec-aligned "Orchestrator acknowledged request"
- Update robot/session_cli.robot: rename test to Session Tell Renders Structured Panels
- Add Session Tell JSON Envelope robot integration test
- Add CHANGELOG.md entry under [Unreleased] ### Fixed
- Fix streaming mode string from "stream" to "streaming" per spec

ISSUES CLOSED: #6452
2026-05-05 19:54:21 +00:00
HAL9000 dc435aa66e fix(cli): fix session tell output to return spec-required structured response (#6452)
CI / benchmark-publish (pull_request) Has been skipped
CI / push-validation (pull_request) Successful in 17s
CI / build (pull_request) Successful in 27s
CI / helm (pull_request) Successful in 30s
CI / lint (pull_request) Failing after 39s
CI / quality (pull_request) Successful in 39s
CI / typecheck (pull_request) Successful in 53s
CI / security (pull_request) Successful in 59s
CI / coverage (pull_request) Has been skipped
CI / benchmark-regression (pull_request) Has been skipped
CI / e2e_tests (pull_request) Successful in 3m36s
CI / integration_tests (pull_request) Failing after 4m0s
CI / unit_tests (pull_request) Failing after 8m17s
CI / docker (pull_request) Has been skipped
CI / status-check (pull_request) Failing after 2s
ISSUES CLOSED: #6452
2026-04-09 23:47:38 +00:00