fix: do not cleanup api and repo.lock files

src/index.js

@@ -55,17 +55,23 @@ function handleError (e) {
 
 async function setupConnection () {
   let config = store.get('config')
+  let updateCfg = false
 
   if (config === null) {
-    config = {
-      type: 'go',
-      path: join(app.getPath('home'), '.ipfs')
-    }
+    config = { type: 'go' }
+    updateCfg = true
+  }
+
+  const ipfsd = await createDaemon(config)
 
+  // createDaemon has changed the config object,
+  // but didn't add the repo variable.
+  if (updateCfg) {
+    config.path = ipfsd.repoPath
     store.set('config', config)
   }
 
-  return createDaemon(config)
+  return ipfsd
 }
 
 async function run () {

src/utils/daemon.js

@@ -1,7 +1,4 @@
 import IPFSFactory from 'ipfsd-ctl'
-import { join } from 'path'
-import fs from 'fs-extra'
-import logger from './logger'
 
 export default async function createDaemon (opts) {
   opts.type = opts.type || 'go'
@@ -13,24 +10,16 @@ export default async function createDaemon (opts) {
     throw new Error(`${opts.type} connection is not supported yet`)
   }
 
-  const init = !(await fs.pathExists(opts.path)) || fs.readdirSync(opts.path).length === 0
-
-  if (!init) {
-    await cleanLocks(opts.path)
-  }
-
   const factory = IPFSFactory.create({ type: opts.type })
 
   const ipfsd = await new Promise((resolve, reject) => {
     factory.spawn({
-      init: false,
-      start: false,
       disposable: false,
       defaultAddrs: true,
       repoPath: opts.path
     }, (e, ipfsd) => {
       if (e) return reject(e)
-      if (ipfsd.initialized || !init) {
+      if (ipfsd.initialized) {
         return resolve(ipfsd)
       }
 
@@ -65,28 +54,3 @@ export default async function createDaemon (opts) {
 
   return ipfsd
 }
-
-function cleanLocks (path) {
-  // This fixes a bug on Windows, where the daemon seems
-  // not to be exiting correctly, hence the file is not
-  // removed.
-  logger.info('Cleaning repo.lock and api files')
-  const lockPath = join(path, 'repo.lock')
-  const apiPath = join(path, 'api')
-
-  if (fs.existsSync(lockPath)) {
-    try {
-      fs.unlinkSync(lockPath)
-    } catch (_) {
-      logger.warn('Could not remove repo.lock. Daemon might be running')
-    }
-  }
-
-  if (fs.existsSync(apiPath)) {
-    try {
-      fs.unlinkSync(apiPath)
-    } catch (_) {
-      logger.warn('Could not remove api. Daemon might be running')
-    }
-  }
-}