[PR #8156] Update active profiles when loading defaults #27124

Open
opened 2026-01-31 09:20:09 +00:00 by claunia · 0 comments
Owner

Original Pull Request: https://github.com/microsoft/terminal/pull/8156

State: closed
Merged: Yes


Summary of the Pull Request

When we get a serialization error, we "catch" it in AppLogic and only
LoadDefaults(). Since LoadDefaults() doesn't perform a full
validation of CascadiaSettings, we need to manually update our list of
active profiles (similar to how we manually resolve the default
profile).

Validation Steps Performed

Repro steps fixed:

  1. add deserialization error to settings.json (i.e. "fontWeight": "wumbo")
  2. launch WT
  3. verify that dropdown is populated with active profiles

Closes #8146

**Original Pull Request:** https://github.com/microsoft/terminal/pull/8156 **State:** closed **Merged:** Yes --- ## Summary of the Pull Request When we get a serialization error, we "catch" it in `AppLogic` and only `LoadDefaults()`. Since `LoadDefaults()` doesn't perform a full validation of `CascadiaSettings`, we need to manually update our list of active profiles (similar to how we manually resolve the default profile). ## Validation Steps Performed Repro steps fixed: 1. add deserialization error to settings.json (i.e. "fontWeight": "wumbo") 2. launch WT 3. verify that dropdown is populated with active profiles Closes #8146
claunia added the pull-request label 2026-01-31 09:20:09 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#27124