bug: /compact is not interruptable #606

Activity

1. thdxr

   assigned

   adamdotdevin

   on Jul 2, 2025
2. morgvanny commented on Jul 2, 2025

   

   morgvanny

   on Jul 2, 2025

   Last edited by morgvanny

   More actions

   Looking into this a bit, it seems the issue is that CompactSession spawns a goroutine to call the summarize API but doesn't use the context parameter or store a cancel function. The interrupt handler in tui.go calls a.app.Cancel() which only aborts the main session operations, not the compact operation running in its own goroutine. I think to make compact operations interruptable like regular session operations we'd need to:

   • Store a context.CancelFunc in the App struct when starting a compact operation
   • Pass a cancellable context to the Session.Summarize call
   • Have the interrupt handler also cancel this context if it exists
3. thdxr

   added

   bugSomething isn't workingSomething isn't working

   bad

   and removed

   bugSomething isn't workingSomething isn't working

   on Jul 3, 2025
4. adamdotdevin

   closed this as completedin #691on Jul 8, 2025
5. bussard76

   added a commit that references this issue on May 12, 2026

   feat: pulse Soul sidebar icon when soul mode is active (anomalyco#606)

   2824919