Deleting all profiles crashes Windows Terminal #17513

Closed
opened 2026-01-31 05:44:45 +00:00 by claunia · 5 comments
Owner

Originally created by @lhecker on GitHub (May 18, 2022).

Windows Terminal version

1.13.11373.0

Windows build number

No response

Other Software

No response

Steps to reproduce

  1. Delete all profiles

Expected Behavior

  • Shouldn't crash

Actual Behavior

  • Crash
Originally created by @lhecker on GitHub (May 18, 2022). ### Windows Terminal version 1.13.11373.0 ### Windows build number _No response_ ### Other Software _No response_ ### Steps to reproduce 1. Delete all profiles ### Expected Behavior * Shouldn't crash ### Actual Behavior * Crash
Author
Owner

@carlos-zamora commented on GitHub (May 19, 2022):

A few quick thoughts...

  1. this is probably relatively new behavior now that you're allowed to delete any profile (instead of just the custom ones, since we blocked deleting CMD before state.json got added)
  2. back in the JSON-only days, we would throw a warning/error and (I believe) fallback to the default settings
  3. Wouldn't this be a case of "play stupid games..."? 100% agree we shouldn't crash, but I feel like solution would be that we fallback to the default settings (like before) until you fix it, but then that might cause a weird state with the SUI. I'm just concerned we'd need major changes to fix a crash that presumably very few (if any) people are accidentally hitting.
@carlos-zamora commented on GitHub (May 19, 2022): A few quick thoughts... 1. this is probably relatively new behavior now that you're allowed to delete _any_ profile (instead of just the custom ones, since we blocked deleting CMD before state.json got added) 2. back in the JSON-only days, we would throw a warning/error and (I believe) fallback to the default settings 3. Wouldn't this be a case of "play stupid games..."? 100% agree we shouldn't crash, but I feel like solution would be that we fallback to the default settings (like before) until you fix it, but then that might cause a weird state with the SUI. I'm just concerned we'd need major changes to fix a crash that presumably very few (if any) people are accidentally hitting.
Author
Owner

@j4james commented on GitHub (May 21, 2022):

Is this perhaps the same thing as #13017?

@j4james commented on GitHub (May 21, 2022): Is this perhaps the same thing as #13017?
Author
Owner

@lhecker commented on GitHub (May 21, 2022):

No those are separate issues as far as I can tell. #13017 occurred when you saved your settings and you deleted a profile afterwards. Due to a bug in the UI code it crashed.

But I noticed that we also crash if you simply delete all profiles.

@lhecker commented on GitHub (May 21, 2022): No those are separate issues as far as I can tell. #13017 occurred when you saved your settings and you deleted a profile afterwards. Due to a bug in the UI code it crashed. But I noticed that we also crash if you simply delete all profiles.
Author
Owner

@ghost commented on GitHub (Jul 6, 2022):

:tada:This issue was addressed in #13242, which has now been successfully released as Windows Terminal v1.14.186.🎉

Handy links:

@ghost commented on GitHub (Jul 6, 2022): :tada:This issue was addressed in #13242, which has now been successfully released as `Windows Terminal v1.14.186`.:tada: Handy links: * [Release Notes](https://github.com/microsoft/terminal/releases/tag/v1.14.186) * [Store Download](https://www.microsoft.com/store/apps/9n8g5rfz9xk3?cid=storebadge&ocid=badge)
Author
Owner

@ghost commented on GitHub (Jul 6, 2022):

:tada:This issue was addressed in #13242, which has now been successfully released as Windows Terminal Preview v1.15.186.🎉

Handy links:

@ghost commented on GitHub (Jul 6, 2022): :tada:This issue was addressed in #13242, which has now been successfully released as `Windows Terminal Preview v1.15.186`.:tada: Handy links: * [Release Notes](https://github.com/microsoft/terminal/releases/tag/v1.15.186) * [Store Download](https://www.microsoft.com/store/apps/9n8g5rfz9xk3?cid=storebadge&ocid=badge)
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#17513