Simplify prompt and remove redundancy

[kmesiab]

Repetition of You will... on each line of the prompt is superflous. The prompt should be optimized for both token spend and instruction quality.

ai-code-critic/internal/openai.go

Lines 29 to 45 in a055e66

	prompt := `
	You are a helpful software developer.
	Attached below is a diff of a pull request.
	You will review the diff for bugs and common mistakes.
	You will review the diff for best practices according to its language.
	You will review the diff for security vulnerabilities.
	You will review the diff for performance issues.
	You will review the diff for readability.
	You will review the diff for maintainability.
	You will review the diff for testability.
	You will review the diff for scalability.
	You will review the diff for extensibility.
	You will review the diff for reusability.
	You will review the diff for modularity.
	You will review the diff for simplicity.
	You will produce a concise code review and output it in markdown.
	Break all lines before 60 characters.

ChatGPT 3.5 suggests this change:

 Please review the pull request's diff for the following aspects:

Bugs and common mistakes
Best practices specific to the language used
Security vulnerabilities
Performance issues
Readability
Maintainability
Testability
Scalability
Extensibility
Reusability
Modularity
Simplicity
Once you've completed the review, provide a concise code review summary in markdown format. Remember to break lines before 60 characters.

ChatGPT 4 suggests:

Please review the attached pull request diff with the following considerations in mind:

Bug Detection: Identify any bugs or common mistakes.
Best Practices: Evaluate adherence to language-specific best practices.
Security: Check for potential security vulnerabilities.
Performance: Assess any performance issues.
Readability: Ensure the code is easily understandable.
Maintainability: Consider long-term maintainability.
Testability: Evaluate ease of testing.
Scalability: Assess scalability potential.
Extensibility: Check for extensibility features.
Reusability: Determine code reusability.
Modularity: Evaluate the modularity of the code.
Simplicity: Ensure the code is not overly complex.
Provide a comprehensive code review in markdown format, ensuring line breaks before 60 characters for readability.

A better prompt altogether:

## Code Review Request

**Pull Request Overview**:
- Brief description of changes:
- Purpose and scope of the PR:

**Specific Concerns**:
- Highlight areas needing detailed review:
- Any complex logic or new implementations:

**Security Aspects**:
- Security-sensitive sections of the code:

**Performance Considerations**:
- Performance-critical sections:

**Readability and Maintainability**:
- Feedback on naming conventions, structure, and comments:

**Testing**:
- Description of testing done:
- Suggestions for additional tests:

**Dependencies**:
- New or changed dependencies:

**Compliance with Best Practices**:
- Review of compliance with language/framework best practices:

**Documentation**:
- Review of updated/added documentation:

**Questions for Reviewers**:
- Any specific questions or areas where feedback is sought:

%s