Greptile Summary

This PR fixes a CI breakage caused by openai 2.34.0 rejecting construction with an empty credential. It pins openai to <2.34.0 and reorders the test setup so the env var is set before OpenAILLMClient is instantiated.

The test reordering is correct and necessary regardless of the version pin. The pin itself is a valid short-term fix, but the underlying issue is that OpenAILLMClient.__init__ eagerly calls OpenAI(...) with a potentially empty credential — the existing _ensure_client method already handles lazy/re-keyed creation correctly, so deferring construction there would be the permanent fix that removes the need for the cap.

Confidence Score: 4/5

Safe to merge as a short-term CI fix; the version cap and test fix are both correct, with a known follow-up needed in the production client.

Only P2 findings present. The changes are small, targeted, and correct. The pin is acknowledged as temporary and the test fix is unambiguously right. No regressions introduced.

app/services/llm_client.py — the eager OpenAI(...) call in __init__ is the root cause and should be addressed in a follow-up.

Important Files Changed

_{Reviews (1): Last reviewed commit: "fix(test): set TEST_KEY before OpenAILLM..." | Re-trigger Greptile}

🎻 "The diff was clean, the tests did pass, the reviewer wept." That poem was about @muddlebee's PR. 🥹

👋 Join us on Discord - OpenSRE : hang out, contribute, or hunt for features and issues. Everyone's welcome.