Skip to content

Conversation

@peterj
Copy link
Collaborator

@peterj peterj commented Jan 22, 2025

No description provided.

Signed-off-by: Peter Jausovec <peter.jausovec@solo.io>
Signed-off-by: Peter Jausovec <peter.jausovec@solo.io>
@peterj peterj closed this Jan 22, 2025
mukundkumarjha pushed a commit to mukundkumarjha/kagent that referenced this pull request May 6, 2025
apexlnc added a commit to apexlnc/kagent that referenced this pull request Oct 21, 2025
Address maintainer feedback by standardizing HITL (Human-in-the-Loop)
functionality in kagent-core to enable uniform interrupt handling across
all executor types (LangGraph, CrewAI, ADK).

Changes:
- Add kagent-core/a2a/_hitl.py with framework-agnostic HITL types and utilities
- Add HITL constants to kagent-core/a2a/_consts.py
- Refactor kagent-langgraph executor to use core HITL utilities
- Extract backtick escaping to dedicated function
- Implement two-tier decision detection (DataPart priority, TextPart fallback)
- Change security default from approve to deny
- Fix: Use JSON for checkpoint metadata serialization (LangGraph 1.0 compatibility)
- Add 10 comprehensive tests for HITL functionality

Addresses: kagent-dev#1025 (comments kagent-dev#2, kagent-dev#4, kagent-dev#5, kagent-dev#6, kagent-dev#8)
Signed-off-by: apexlnc <43242113+apexlnc@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants