Skip to content

fix: Queue messages during conversation compaction#1546

Open
charlesvien wants to merge 5 commits into04-07-prevent_permission_request_from_stealing_focus_across_cellsfrom
04-07-queue_messages_during_conversation_compaction
Open

fix: Queue messages during conversation compaction#1546
charlesvien wants to merge 5 commits into04-07-prevent_permission_request_from_stealing_focus_across_cellsfrom
04-07-queue_messages_during_conversation_compaction

Conversation

@charlesvien
Copy link
Copy Markdown
Member

@charlesvien charlesvien commented Apr 8, 2026

Problem

Messages sent during compaction bypass the queue and are lost because isCompacting was only a derived UI property, invisible to the service layer.

Closes #1272

Changes

  1. Add isCompacting state to AgentSession and gate sendPrompt on it
  2. Track compaction lifecycle via STATUS and COMPACT_BOUNDARY notifications in service
  3. Drain queued messages when compaction completes
  4. Replace all _posthog/ string literals with POSTHOG_NOTIFICATIONS constants
  5. Add isNotification() utility in agent package to handle potential double-prefix
  6. Remove duplicate isPosthogMethod helpers from renderer files

How did you test this?

Maually

@charlesvien charlesvien force-pushed the 04-07-queue_messages_during_conversation_compaction branch from 160431f to aa26669 Compare April 8, 2026 04:17
@charlesvien charlesvien force-pushed the 04-07-prevent_permission_request_from_stealing_focus_across_cells branch from 4cb808f to 1c97223 Compare April 8, 2026 04:17
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.

1 participant