feat: warn if no bak file exists in patched extension when loaded

Author: subframe7536

src/index.ts

@@ -9,28 +9,24 @@ import { debounce, showMessage } from './utils'
 const changedMsg = 'UI Style changed.'
 const rollbackMsg = 'UI Style rollback.'
 
+const configChangedMsg = 'Configuration changed, apply now?'
+const newVersionMsg = 'Seems like first time use or new version is installed, initialize and reload config now?'
+const extensionUpdatedMsg = 'Seems like extensions are updated, reload config now?'
 const { activate, deactivate } = defineExtension(() => {
-  const { hasBakFile, reload, rollback } = createFileManagers()
+  const { hasBakFile, hasBakExtFiles, reload, rollback } = createFileManagers()
+
+  const requestReload = (msg: string) => showMessage(msg, 'Yes', 'No')
+    .then<any>(item => item === 'Yes' && reload(changedMsg))
 
   if (!hasBakFile()) {
-    showMessage(
-      'Seems like first time use or new version is installed, initialize and reload config now?',
-      'Yes',
-      'No',
-    )
-      .then<any>(item => item === 'Yes' && reload(changedMsg))
+    requestReload(newVersionMsg)
+  } else if (!hasBakExtFiles()) {
+    requestReload(extensionUpdatedMsg)
   }
 
   useCommand(Meta.commands.reload, () => reload(changedMsg))
   useCommand(Meta.commands.rollback, () => rollback(rollbackMsg))
 
-  const notifyChanged = () => showMessage(
-    'Configuration changed, apply now?',
-    'Yes',
-    'No',
-  )
-    .then<any>(item => item === 'Yes' && reload(changedMsg))
-
   useDisposable(
     workspace.onDidChangeConfiguration(
       debounce(
@@ -39,14 +35,14 @@ const { activate, deactivate } = defineExtension(() => {
             return
           }
           if (e.affectsConfiguration(Meta.name)) {
-            notifyChanged()
+            requestReload(configChangedMsg)
           } else if (e.affectsConfiguration(ffKey) && !config['font.monospace']) {
             const {
               globalValue,
               workspaceValue,
             } = workspace.getConfiguration().inspect<string>(ffKey)!
             if (globalValue === workspaceValue) {
-              notifyChanged()
+              requestReload(configChangedMsg)
             }
           }
         },

src/manager/extension.ts

@@ -61,15 +61,20 @@ class ExtensionFileManager extends BaseFileManager {
   }
 }
 
-export function createExtensionFileManagers() {
+export function createExtensionFileManagers(skipWarnIfExtensionNotExist = false) {
   if (!config['extensions.enable'] || !config['extensions.config']) {
     return []
   }
   const managers = []
   for (const [extensionId, patchConfig] of Object.entries(config['extensions.config'])) {
     const rootPath = extensions.getExtension(extensionId)?.extensionPath
     if (!rootPath) {
-      promptWarn(`No such extension: ${extensionId}, skip patch`)
+      const msg = `No such extension: ${extensionId}, skip`
+      if (skipWarnIfExtensionNotExist) {
+        log.warn(msg)
+      } else {
+        promptWarn(msg)
+      }
       continue
     }
     if (!Array.isArray(patchConfig)) {
@@ -88,7 +93,9 @@ export function createExtensionFileManagers() {
       }
     }
     if (warningArray.length > 0) {
-      promptWarn(`Configs of ${extensionId} are invalid: ${JSON.stringify(warningArray, null, 2)}`)
+      promptWarn(
+        `Configs of ${extensionId} are invalid: ${JSON.stringify(warningArray, null, 2)}`,
+      )
     }
   }
   return managers

src/manager/index.ts

@@ -32,6 +32,7 @@ export function createFileManagers() {
 
   return {
     hasBakFile: () => builtinManagers.every(m => m.hasBakFile),
+    hasBakExtFiles: () => createExtensionFileManagers(true).every(m => m.hasBakFile),
     reload: async (text: string) => {
       await runAndRestart(
         text,