fix: cortex models migration should be done once

louis-jan · louis-jan · commit d62006b33cac · 2025-07-24T14:01:19.000+07:00
diff --git a/extensions/llamacpp-extension/src/index.ts b/extensions/llamacpp-extension/src/index.ts
@@ -801,6 +801,9 @@ export default class llamacpp_extension extends AIEngine {
   }
 
   private async migrateLegacyModels() {
+    // Attempt to migrate only once
+    if (localStorage.getItem('cortex_models_migrated') === 'true') return
+
     const janDataFolderPath = await getJanDataFolderPath()
     const modelsDir = await joinPath([janDataFolderPath, 'models'])
     if (!(await fs.existsSync(modelsDir))) return
@@ -886,6 +889,7 @@ export default class llamacpp_extension extends AIEngine {
         stack.push(child)
       }
     }
+    localStorage.setItem('cortex_models_migrated', 'true')
   }
 
   override async import(modelId: string, opts: ImportOptions): Promise<void> {

Original file line number	Diff line number	Diff line change
`@@ -801,6 +801,9 @@ export default class llamacpp_extension extends AIEngine {`
`801`	`801`	`}`
`802`	`802`
`803`	`803`	`private async migrateLegacyModels() {`
	`804`	`+ // Attempt to migrate only once`
	`805`	`+ if (localStorage.getItem('cortex_models_migrated') === 'true') return`
	`806`	`+`
`804`	`807`	`const janDataFolderPath = await getJanDataFolderPath()`
`805`	`808`	`const modelsDir = await joinPath([janDataFolderPath, 'models'])`
`806`	`809`	`if (!(await fs.existsSync(modelsDir))) return`
`@@ -886,6 +889,7 @@ export default class llamacpp_extension extends AIEngine {`
`886`	`889`	`stack.push(child)`
`887`	`890`	`}`
`888`	`891`	`}`
	`892`	`+ localStorage.setItem('cortex_models_migrated', 'true')`
`889`	`893`	`}`
`890`	`894`
`891`	`895`	`override async import(modelId: string, opts: ImportOptions): Promise<void> {`