Skip to content

feat: add support to custom providers - #1

Merged
ramonpaolo merged 5 commits into
mainfrom
feat/add-support-to-custom-providers
Jan 14, 2026
Merged

feat: add support to custom providers#1
ramonpaolo merged 5 commits into
mainfrom
feat/add-support-to-custom-providers

Conversation

@ramonpaolo

@ramonpaolo ramonpaolo commented Jan 14, 2026

Copy link
Copy Markdown
Owner

Description

This PR add the functionality to use custom providers/models to do the code review.

This PR allow the integration with local models, like Ollama, llama.cpp; and another providers/models like MiniMax M2.1

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Refactoring
  • Other (please describe)

Checklist

  • I have tested the changes
  • My code follows the project's style guidelines
  • I have commented my code where necessary
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings

Related Issues

N/A

Additional Context

N/A

@ramonpaolo ramonpaolo self-assigned this Jan 14, 2026
@ramonpaolo ramonpaolo changed the title Feat/add support to custom providers feat: add support to custom providers Jan 14, 2026
@ramonpaolo
ramonpaolo force-pushed the feat/add-support-to-custom-providers branch from fa6ca20 to 14f241a Compare January 14, 2026 03:14

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

undefined


Code review performed by CUSTOM - MiniMax-M2.1.

@ramonpaolo
ramonpaolo force-pushed the feat/add-support-to-custom-providers branch from 14f241a to ccb43f5 Compare January 14, 2026 03:20

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎯 Core Changes

This PR adds support for custom AI providers (like MiniMax M2.1, Ollama, llama.cpp) to the AI Code Review action. The changes include:

  1. .github/workflows/code_review.yml: Updated workflow to use the new custom provider branch with MiniMax configuration instead of the local action with OpenAI
  2. action.yml:
    • Added AI_BASE_URL input for custom provider endpoints
    • Made AI_API_KEY optional (previously required)
    • Updated AI_PROVIDER description to include "custom" option
    • Upgraded Node.js runtime from node20 to node24

⚠️ Concerns

  • Missing documentation: The PR description doesn't mention that users need to add the AI_API_KEY secret when using custom providers. Consider documenting this requirement.
  • API key optionality: Making AI_API_KEY optional in action.yml may cause confusing errors if a provider requires authentication but the user doesn't provide a key. Consider adding validation or clearer error messages.
  • Node 24 runtime: Using node24 (which may not be fully stable) instead of the widely-used node20 could cause compatibility issues in some runner environments.

Verdict

The changes implement the stated feature correctly. The concerns above are minor and don't prevent merging. The PR author should consider documenting the new AI_API_KEY secret requirement for custom providers.

Suggested Action: comment

Confidence: 95


Code review performed by CUSTOM - MiniMax-M2.1.

@ramonpaolo
ramonpaolo force-pushed the feat/add-support-to-custom-providers branch from ccb43f5 to 180ad95 Compare January 14, 2026 03:22
@ramonpaolo
ramonpaolo merged commit a71612b into main Jan 14, 2026
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant