Fix missing imports in sync-openai-agents template

[danielmillerp]

Summary

• The sync-openai-agents template (agentex init → Sync ACP + OpenAI Agents SDK) uses StreamTaskMessageFull and TextContent in the no-API-key error handling path but never imports them, causing a NameError at runtime
• Added the missing imports for StreamTaskMessageFull and TextContent
• Added a missing return after the error yield so the function doesn't fall through to the OpenAI Runner code when there's no API key

Test plan

• Verified the fix locally by running agentex init → Sync ACP + OpenAI Agents SDK and confirming the agent starts without errors
• Checked all other templates — this is the only one with missing imports

🤖 Generated with Claude Code

Greptile Summary

This PR fixes three bugs in the sync-openai-agents template and makes a related import-hygiene improvement to utils.py.

• Missing imports fixed: StreamTaskMessageFull (from agentex.types.task_message_update) and TextContent (from agentex.types.text_content) were referenced in the no-API-key error path of handle_message_send but never imported, causing a NameError at runtime for any user of the generated template.
• Missing return added: Without the return after the error yield, the generator would fall through into the OpenAI Runner code even when no API key is present, causing a secondary runtime failure.
• Lazy import in utils.py: OpenAIAgentsPlugin is moved from a top-level import to a lazy import inside get_temporal_client, avoiding eager loading of opentelemetry-sdk for agents that never call into Temporal. The explicit opentelemetry-sdk and opentelemetry-api dependencies added to pyproject.toml formalize what was previously a transitive dependency.

Confidence Score: 5/5

• Safe to merge — fixes genuine runtime bugs without introducing new risk.
• All three code changes are straightforward and clearly correct: the missing imports prevent a NameError, the return prevents silent fall-through, and the lazy import is a low-risk refactor with no behaviour change. The pyproject.toml/lock changes are consistent with the existing transitive dependency. No new logic is introduced and no existing behaviour is altered.
• No files require special attention.

Important Files Changed

Filename	Overview
src/agentex/lib/cli/templates/sync-openai-agents/project/acp.py.j2	Adds missing imports for StreamTaskMessageFull and TextContent, and adds a return after the no-API-key error yield to prevent fall-through — all three fixes are correct and necessary.
src/agentex/lib/core/clients/temporal/utils.py	Moves OpenAIAgentsPlugin import from module-level to a lazy import inside get_temporal_client to avoid eagerly pulling in opentelemetry-sdk for agents that don't use Temporal; logic is otherwise unchanged.
pyproject.toml	Adds opentelemetry-sdk and opentelemetry-api as explicit direct dependencies, consistent with their previously-transitive usage via temporalio.
uv.lock	Lock file updated to reflect new opentelemetry-sdk and opentelemetry-api direct dependencies and version bump from 0.9.3 → 0.9.4.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[handle_message_send called] --> B{OPENAI_API_KEY set?}
    B -- No --> C[yield StreamTaskMessageFull\nerror message]
    C --> D[return ✅]
    B -- Yes --> E[Extract user_prompt\nfrom params]
    E --> F[Get or create\ntask state]
    F --> G[Build SyncStreamingProvider\n& RunConfig]
    G --> H[Initialize Agent\nwith tools]
    H --> I[Runner.run_streamed]
    I --> J[convert_openai_to_agentex_events]
    J --> K[yield agentex_event]
    K --> J
    J -- done --> L[Update state\nwith full conversation history]

Loading

_{Reviews (5): Last reviewed commit: "Add opentelemetry-sdk as a dependency" | Re-trigger Greptile}

[socket-security]

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff	Package	Supply Chain Security	Vulnerability	Quality	Maintenance	License
	opentelemetry-sdk@​1.37.0

View full report