Code rabbit remarks fixes

Author: PycKamil

action.yml

@@ -146,10 +146,10 @@ inputs:
     default: 'false'
   azure_api_instance_name:  
     required: false
-    description: 'For example, if your Azure instance is hosted under https://{INSTANCE_NAME}.openai.azure.com/openai/deployments/{DEPLOYMENT_NAME} use INSTANCE_NAME'
+    description: 'Instance name of your Azure API. For example, if your Azure instance is hosted under https://{INSTANCE_NAME}.openai.azure.com/openai/deployments/{DEPLOYMENT_NAME}, use INSTANCE_NAME'
   azure_api_deployment_name:  
     required: false
-    description: 'For example, if your Azure instance is hosted under https://{INSTANCE_NAME}.openai.azure.com/openai/deployments/{DEPLOYMENT_NAME} use DEPLOYMENT_NAME'
+    description: 'Deployment name of your Azure API. For example, if your Azure instance is hosted under https://{INSTANCE_NAME}.openai.azure.com/openai/deployments/{DEPLOYMENT_NAME}, use DEPLOYMENT_NAME'
   azure_api_version:
     required: false
     description: 'Api version like "2023-07-01-preview"'

dist/index.js

@@ -52,7 +52,7 @@ export class AzureBot implements BotProtocol {
       })
     } else {
       const err =
-        "Unable to initialize the OpenAI API, both 'AZURE_OPENAI_API_KEY' environment variable are not available"
+        "Unable to initialize the OpenAI API, ensure 'AZURE_OPENAI_API_KEY', 'azureApiDeployment', 'apiBaseUrl', and 'azureApiInstance' are properly set"
       throw new Error(err)
     }
   }
@@ -63,6 +63,7 @@ export class AzureBot implements BotProtocol {
       res = await this.chat_(message)
       return res
     } catch (e: unknown) {
+      warning(`Failed to chat: ${e}`)
       return res
     }
   }

src/azure-bot.ts

@@ -13,6 +13,7 @@ import {OpenAIOptions, Options} from './options'
 import {Prompts} from './prompts'
 import {codeReview} from './review'
 import {handleReviewComment} from './review-comment'
+import {TokenLimits} from './limits'
 
 async function run(): Promise<void> {
   const options: Options = new Options(
@@ -48,43 +49,33 @@ async function run(): Promise<void> {
 
   // Create two bots, one for summary and one for review
 
-  let lightBot: BotProtocol | null = null
-  try {
-    if (options.azureApiDeployment.length > 0) {
-      lightBot = new AzureBot(
-        options,
-        new OpenAIOptions(options.openaiLightModel, options.lightTokenLimits)
-      )
-    } else {
-      lightBot = new Bot(
-        options,
-        new OpenAIOptions(options.openaiLightModel, options.lightTokenLimits)
+  function createBot(
+    model: string,
+    tokenLimits: TokenLimits
+  ): BotProtocol | null {
+    try {
+      if (options.azureApiDeployment.length > 0) {
+        return new AzureBot(options, new OpenAIOptions(model, tokenLimits))
+      } else {
+        return new Bot(options, new OpenAIOptions(model, tokenLimits))
+      }
+    } catch (e: any) {
+      warning(
+        `Skipped: failed to create bot, please check your openai_api_key: ${e}, backtrace: ${e.stack}`
       )
+      return null
     }
-  } catch (e: any) {
-    warning(
-      `Skipped: failed to create summary bot, please check your openai_api_key: ${e}, backtrace: ${e.stack}`
-    )
-    return
   }
 
-  let heavyBot: BotProtocol | null = null
-  try {
-    if (options.azureApiDeployment.length > 0) {
-      heavyBot = new AzureBot(
-        options,
-        new OpenAIOptions(options.openaiHeavyModel, options.heavyTokenLimits)
-      )
-    } else {
-      heavyBot = new Bot(
-        options,
-        new OpenAIOptions(options.openaiHeavyModel, options.heavyTokenLimits)
-      )
-    }
-  } catch (e: any) {
-    warning(
-      `Skipped: failed to create review bot, please check your openai_api_key: ${e}, backtrace: ${e.stack}`
-    )
+  const lightBot: BotProtocol | null = createBot(
+    options.openaiLightModel,
+    options.lightTokenLimits
+  )
+  const heavyBot: BotProtocol | null = createBot(
+    options.openaiHeavyModel,
+    options.heavyTokenLimits
+  )
+  if (lightBot == null || heavyBot == null) {
     return
   }
 

src/main.ts

@@ -1,7 +1,7 @@
 import {info, warning} from '@actions/core'
 // eslint-disable-next-line camelcase
 import {context as github_context} from '@actions/github'
-import {type BotProtocol} from './bot-interface'
+import {BotProtocol} from './bot-interface'
 import {
   Commenter,
   COMMENT_REPLY_TAG,

src/review-comment.ts

@@ -2,7 +2,7 @@ import {error, info, warning} from '@actions/core'
 // eslint-disable-next-line camelcase
 import {context as github_context} from '@actions/github'
 import pLimit from 'p-limit'
-import {type BotProtocol} from './bot-interface'
+import {BotProtocol} from './bot-interface'
 import {
   Commenter,
   COMMENT_REPLY_TAG,