Skip to content

feat(chat): integrate ask-user-question HITL flow - #482

Merged
jubaoliang merged 2 commits into
developfrom
feature/ask-user-question-hitl
Aug 31, 2026
Merged

feat(chat): integrate ask-user-question HITL flow#482
jubaoliang merged 2 commits into
developfrom
feature/ask-user-question-hitl

Conversation

@Pandakingxbc

Copy link
Copy Markdown
Collaborator

Summary

  • add a Dashboard question card that stays above the composer, asks bundled questions one at a time, and collapses after answering
  • resume ask_user_question through the existing HITL respond path with validation, usage tracking, and history persistence
  • support sequential question/answer collection in IM channels, including letter/number option shortcuts and follow-up HITL pauses
  • expose the built-in interaction tool in the tool catalog and add Chinese/English UI and IM copy

Merge prerequisite

  • Do not merge until a harness-agent release containing ask_user_question is published and installable.
  • Verified on 2026-08-30: orcakit-harness-agent==1.0.1 is published on PyPI, its wheel contains harness_agent/builtin/tools/ask_user.py, and current develop already requires orcakit-harness-agent[all]>=1.0.1.
  • If CI or an internal package mirror cannot resolve 1.0.1 yet, keep this PR unmerged until propagation completes.

Validation

  • make all: 2717 passed, 15 skipped; Ruff, formatting, and mypy passed
  • Dashboard focused tests: 2 files, 5 tests passed
  • Dashboard production build: passed

Notes

  • The implementation reuses Octop's existing HITL coordinator and persistence flow; it does not add a second question database or require a harness-gateway change.
  • /skip is intentionally not part of the IM ask-user flow; a user's next ordinary text message answers the current question.

@Pandakingxbc

Copy link
Copy Markdown
Collaborator Author
Clipboard_Screenshot_1788097016 octop red style

@jubaoliang
jubaoliang merged commit f7c5040 into develop Aug 31, 2026
3 checks passed
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