Skip to content

CLI-5: Restore user input required notifications#483

Merged
dwash96 merged 18 commits intocecli-dev:v0.99.2from
szmania:cli-5-notifications
Apr 15, 2026
Merged

CLI-5: Restore user input required notifications#483
dwash96 merged 18 commits intocecli-dev:v0.99.2from
szmania:cli-5-notifications

Conversation

@szmania
Copy link
Copy Markdown

@szmania szmania commented Apr 12, 2026

Summary

This PR restores the user input required notifications that were inadvertently removed or disabled in recent changes.

Changes Made

  • Restored user input required notification functionality
  • Fixed notification triggering for user input prompts
  • Added proper control over notification triggering in agent coder
  • Implemented notification triggering based on agent reflection
  • Removed unnecessary ring_bell calls to prevent duplicate notifications
  • Ensured single notification for confirmations
  • Refactored notification logic to separate user input notification concerns
  • Added TUI mode support for user input notifications

Testing

  • All existing tests continue to pass
  • Notifications now properly trigger when user input is required
  • No duplicate notifications observed during testing

Related Issues

Fixes CLI-5: User input notifications not appearing

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature
  • Breaking change
  • Documentation update

@dwash96 dwash96 changed the base branch from main to v0.99.2 April 15, 2026 11:37
@dwash96 dwash96 merged commit 8ad46fd into cecli-dev:v0.99.2 Apr 15, 2026
12 checks passed
@dwash96 dwash96 mentioned this pull request Apr 15, 2026
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