From 7762584fb1f49ae6f9ff2aa44ff063fb3fc54f68 Mon Sep 17 00:00:00 2001
From: Eva Ho <eva.ho@infrahq.com>
Date: Tue, 11 Jul 2023 14:38:38 -0400
Subject: [PATCH] address comments

---
 app/src/index.ts | 23 ++++++++++-------------
 1 file changed, 10 insertions(+), 13 deletions(-)

diff --git a/app/src/index.ts b/app/src/index.ts
index 1d574ff2..79996c72 100644
--- a/app/src/index.ts
+++ b/app/src/index.ts
@@ -13,18 +13,15 @@ require('@electron/remote/main').initialize()
 const store = new Store()
 let tray: Tray | null = null
 
-const logFile = new winston.transports.DailyRotateFile({
-  filename: path.join(app.getPath('home'), '.ollama', 'logs', 'server-%DATE%.log'),
-  datePattern: 'YYYY-MM-DD-HH',
-  zippedArchive: true,
-  maxSize: '20m',
-  maxFiles: '7d',
+const logFile = new winston.transports.File({
+  filename: path.join(app.getPath('home'), '.ollama', 'logs', 'server.log'),
+  maxsize: 1024 * 1024 * 20,
+  maxFiles: 5,
 });
 
 const logger = winston.createLogger({ 
   transports: [logFile],
   format: winston.format.combine(
-    winston.format.colorize(),
     winston.format.timestamp(),
     winston.format.printf(info => `${info.timestamp} ${info.level}: ${info.message}`)
   )
@@ -64,10 +61,10 @@ function server() {
 
   const proc = spawn(binary, ['serve'])
   proc.stdout.on('data', data => {
-    logger.info(`[server] ${data.toString()}`)
+    logger.info(`server: ${data.toString()}`)
   })
   proc.stderr.on('data', data => {
-    logger.error(`[server] ${data.toString()}`)
+    logger.error(`server: ${data.toString()}`)
   })
 
   proc.on('exit', () => {
@@ -107,12 +104,12 @@ function installCLI() {
     `
         exec(`osascript -e '${command}'`, (error: Error | null, stdout: string, stderr: string) => {
           if (error) {
-            logger.error(`[CLI] Failed to install CLI - ${error.message}`)
+            logger.error(`CLI: Failed to install CLI - ${error.message}`)
             return
           }
 
-          logger.info(`[CLI] ${stdout}}`)
-          logger.error(`[CLI] ${stderr}`)
+          logger.info(`CLI: ${stdout}}`)
+          logger.error(`CLI: ${stderr}`)
         })
       }
     })
@@ -206,7 +203,7 @@ if (app.isPackaged) {
 }
 
 autoUpdater.on('error', e => {
-  logger.error(`[auto updater] update check failed - ${e.message}`)
+  logger.error(`auto updater: update check failed - ${e.message}`)
 })
 
 autoUpdater.on('update-downloaded', (event, releaseNotes, releaseName) => {