fix(ui): address remaining migration bug and test snapshot

This commit is contained in:
jacob314
2026-03-10 13:59:15 -07:00
parent 2e6e622893
commit ed65a2ff4e
3 changed files with 4 additions and 10 deletions

View File

@@ -932,14 +932,14 @@ export function migrateDeprecatedSettings(
}
}
// Migrate enableLoadingPhrases: false → loadingPhraseLayout: 'none'
// Migrate enableLoadingPhrases: false → loadingPhrases: 'off'
const enableLP = newAccessibility['enableLoadingPhrases'];
if (
typeof enableLP === 'boolean' &&
newUi['loadingPhraseLayout'] === undefined
newUi['loadingPhrases'] === undefined
) {
if (!enableLP) {
newUi['loadingPhraseLayout'] = 'none';
newUi['loadingPhrases'] = 'off';
loadedSettings.setValue(scope, 'ui', newUi);
if (!settingsFile.readOnly) {
anyModified = true;

View File

@@ -18,12 +18,6 @@ Spinner Connecting to MCP servers... (0/5) - Waiting for: s1, s2, s3, +2 more
"
`;
exports[`ConfigInitDisplay > truncates list of waiting servers if too many 2`] = `
"
Spinner Connecting to MCP servers... (0/5) - Waiting for: s1, s2, s3, +2 more
"
`;
exports[`ConfigInitDisplay > updates message on McpClientUpdate event 1`] = `
"
Spinner Connecting to MCP servers... (1/2) - Waiting for: server2

View File

@@ -11,7 +11,7 @@ exports[`StatusDisplay > renders ContextSummaryDisplay by default 1`] = `
`;
exports[`StatusDisplay > renders HookStatusDisplay when hooks are active 1`] = `
"Mock Context Summary Display (Skills: 2, Shells: 0)
"Mock Hook Status Display
"
`;