diff --git a/Auto Run Docs/PR-Triage/PR-Triage-12.md b/Auto Run Docs/PR-Triage/PR-Triage-12.md index 3d153094..314e5b68 100644 --- a/Auto Run Docs/PR-Triage/PR-Triage-12.md +++ b/Auto Run Docs/PR-Triage/PR-Triage-12.md @@ -10,7 +10,7 @@ Multiple community members want to add provider support. Evaluate whether the pr - [x] Evaluate PR #786 (`feat: add GLM provider and custom Anthropic-compatible API support` by @Zorglub4242, 13 files). **CLOSED.** Three issues: (1) `process.env` mutation is a concurrency bug — env vars leak between sessions since the worker handles multiple sessions on one process. (2) GLM preset is YAGNI — a generic custom provider option would cover GLM users without a dedicated preset. (3) Custom Anthropic-compatible API support is a good concept but needs subprocess-scoped env vars, not global mutation. Invited contributor to re-submit a focused custom-provider-only PR. -- [ ] Evaluate PR #644 (`feat: Add OpenAI provider support` by @niteeshm, 10 files). OpenAI as an alternative provider. Decision: How many providers should the project support? Currently: Claude SDK, Gemini, OpenRouter. Adding OpenAI directly may duplicate OpenRouter (which already supports OpenAI models). If OpenRouter covers this: `gh pr close 644 --comment "OpenAI models are accessible via the OpenRouter provider. We'd prefer to keep the provider count manageable. Thank you!"` +- [x] Evaluate PR #644 (`feat: Add OpenAI provider support` by @niteeshm, 10 files). **CLOSED.** OpenAI models are already accessible via the OpenRouter provider, which uses the same OpenAI-compatible chat/completions API format. Adding a dedicated OpenAI agent (491 lines) would create significant code duplication with OpenRouterAgent. A community commenter (@kiwina) independently flagged the same overlap. Suggested contributor could instead PR a configurable base URL for OpenRouter to support Azure/custom endpoints. - [ ] Evaluate PR #680 (`feat(openrouter): multi-model configuration with automatic fallback` by @RyderFreeman4Logos, 28 files). Large PR adding multi-model fallback to OpenRouter. Decision: Is multi-model fallback needed now? 28 files is a large surface area. If too complex: request scope reduction.