fix: max prompt length per model

[mattzcarey]

Fixes #31

[github-actions]

LOGAF Level 4 - /home/runner/work/code-review-gpt/code-review-gpt/src/review/prompt/getMaxPromptLength.ts

---

LOGAF Level 2 - /home/runner/work/code-review-gpt/code-review-gpt/src/review/prompt/constructPrompt.ts

1. The splitFileContentsIntoChunks function could be more efficient by using a while loop instead of a for loop. This would avoid unnecessary iterations.

const splitFileContentsIntoChunks = (fileContents: string, maxChunkSize: number): string[] => {
  const chunks = [];
  let position = 0;
  while (position < fileContents.length) {
    chunks.push(fileContents.slice(position, position + maxChunkSize));
    position += maxChunkSize;
  }
  return chunks;
};

2. The constructPromptsArray function is quite large and does a lot of things. Consider breaking it down into smaller, more manageable functions.

---

LOGAF Level 3 - /home/runner/work/code-review-gpt/code-review-gpt/src/review/index.ts

1. Consider adding type annotations to the ReviewArgs interface for better type safety. For example, if ci is always a boolean, specify it as such.

interface ReviewArgs {
  ci: boolean;
  _: (string | number)[];
  $0: string;
}

2. The modelName variable is cast as a string without any checks. This could lead to runtime errors if yargs.model is not a string. Consider adding a check or handling for this scenario.

---

🔄🔧🔍

---

Powered by Code Review GPT