Commit 28684ab
fix(setup): cancel wizard session before disconnect to prevent stale-session retry errors
When wizard.next timed out (e.g. Teams channel selection hanging),
EnterWizardErrorAsync called DisconnectAsync which nulled _client, then
showed "Start wizard again" / "Skip wizard" buttons. CancelCurrentSessionAsync
checked _client != null and skipped the wizard.cancel call — leaving the
server-side session active. Subsequent "Start wizard again" clicks then hit a
gateway "wizard already running" error.
Fix: replace await DisconnectAsync() with await CancelCurrentSessionAsync()
in both EnterWizardErrorAsync and StartWizardAsync. CancelCurrentSessionAsync
sends wizard.cancel (best-effort, catch ignored) then calls DisconnectAsync,
so the disconnect still happens. The session cancel is a no-op when _client
is already null or _sessionId is empty, so the first-start path is unaffected.
Closes #709
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>1 parent d1b1363 commit 28684ab
1 file changed
Lines changed: 8 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
56 | 56 | | |
57 | 57 | | |
58 | 58 | | |
59 | | - | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
60 | 63 | | |
61 | 64 | | |
62 | 65 | | |
| |||
757 | 760 | | |
758 | 761 | | |
759 | 762 | | |
760 | | - | |
| 763 | + | |
| 764 | + | |
| 765 | + | |
| 766 | + | |
761 | 767 | | |
762 | 768 | | |
763 | 769 | | |
| |||
0 commit comments