Re-raise LLM exceptions instead of raising new exception

[bruvduroiu]

In Python, re-raising exceptions (raise instead of raise Exception(e) preserves the original stack trace, helping with debugging.

Change the LLM __call__ exception handling to re-raise errors happening during generation

semantic-router/semantic_router/llms/openai.py

Line 53 in f8fcd43

raise Exception(f"LLM error: {e}")

[simjak]

@bruvduroiu so it should raise instead of raise Exception(e) ?

[bruvduroiu]

@simjak yes, raise Exception(e) raises a new exception, thus losing the stack of the original exception

[alongadot]

Hey, great project. I'm looking to contribute (and generally getting started with contributing to OS), is this issue still unassigned?

[simjak]

Hey, great project. I'm looking to contribute (and generally getting started with contributing to OS), is this issue still unassigned?

Yes, you could contribute if you can 🙏