Skip to content

fix(chat): refresh question status after answer - #460

Merged
Harry19081 merged 1 commit into
developfrom
fix/issue-459-question-answer-ui
Jul 21, 2026
Merged

fix(chat): refresh question status after answer#460
Harry19081 merged 1 commit into
developfrom
fix/issue-459-question-answer-ui

Conversation

@beruro

@beruro beruro commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • immediately complete the owning ask_user_questions event after answer submission succeeds
  • apply the same immediate terminal state after Skip
  • keep agent:interaction_finalized as the authoritative idempotent confirmation
  • scope event lookup and upsert to the owning session
  • add regression coverage for immediate completion and cross-session isolation

Testing

  • pnpm exec vitest run src/engines/ChatPanel/InputArea/AskQuestionCard/__tests__/questionSubmitUtils.test.ts
  • pnpm exec eslint src/engines/ChatPanel/InputArea/AskQuestionCard/questionSubmitUtils.ts src/engines/ChatPanel/InputArea/AskQuestionCard/useQuestionSubmission.ts src/engines/ChatPanel/InputArea/AskQuestionCard/__tests__/questionSubmitUtils.test.ts
  • scoped TypeScript check from the pre-commit hook

Closes #459

@Harry19081
Harry19081 merged commit 9305baf into develop Jul 21, 2026
2 checks passed
Neonforge98 pushed a commit that referenced this pull request Jul 30, 2026
fix(chat): refresh question status after answer
@Harry19081
Harry19081 self-requested a review July 30, 2026 08:45
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.

fix(chat): refresh question status immediately after answer

2 participants