Heads up, and this one is not what it looks like from the outside.
The comments say the Assistants path is legacy and being phased out for the Agents SDK, and openai-agents is already a dependency, so it reads like the migration is done. I don't think it is. The routing check in raven/ai/ai.py L24 and L201 is:
if bot.model_provider in ["OpenAI", "Local LLM"] and not bot.openai_assistant_id:
but before_insert in raven_bot.py L120 calls create_openai_assistant() for every bot where model_provider == "OpenAI", and that sets openai_assistant_id via db_set at L168. So not bot.openai_assistant_id is already False by the time any message gets routed. As far as I can tell the Agents SDK branch is only reachable for Local LLM bots, and every OpenAI bot, including one created on a fresh install today, still runs on beta.assistants / beta.threads.
On Aug 26 that means: creating or editing an AI bot throws from before_insert, DMs to a bot fail at beta.threads.create (ai.py L120/L138), follow-ups fail at beta.threads.messages.create (L270/L288), and tool calls die at submit_tool_outputs_stream (handler.py L345).
There's stored state too, not just code. openai_assistant_id on Raven Bot and openai_thread_id on Raven Channel point at objects that stop existing, so flipping the routing alone leaves existing bots broken.
I found this with a scanner I'm building for exactly this kind of rot, dead model ids and retired endpoints. Happy to send a PR for the routing fix if useful, though you know the stored bot remapping far better than I do. Either way I thought you'd want to know before the 26th.
Heads up, and this one is not what it looks like from the outside.
The comments say the Assistants path is legacy and being phased out for the Agents SDK, and openai-agents is already a dependency, so it reads like the migration is done. I don't think it is. The routing check in raven/ai/ai.py L24 and L201 is:
but before_insert in raven_bot.py L120 calls create_openai_assistant() for every bot where model_provider == "OpenAI", and that sets openai_assistant_id via db_set at L168. So
not bot.openai_assistant_idis already False by the time any message gets routed. As far as I can tell the Agents SDK branch is only reachable for Local LLM bots, and every OpenAI bot, including one created on a fresh install today, still runs on beta.assistants / beta.threads.On Aug 26 that means: creating or editing an AI bot throws from before_insert, DMs to a bot fail at beta.threads.create (ai.py L120/L138), follow-ups fail at beta.threads.messages.create (L270/L288), and tool calls die at submit_tool_outputs_stream (handler.py L345).
There's stored state too, not just code. openai_assistant_id on Raven Bot and openai_thread_id on Raven Channel point at objects that stop existing, so flipping the routing alone leaves existing bots broken.
I found this with a scanner I'm building for exactly this kind of rot, dead model ids and retired endpoints. Happy to send a PR for the routing fix if useful, though you know the stored bot remapping far better than I do. Either way I thought you'd want to know before the 26th.