Lee-W commented on code in PR #74449: URL: https://github.com/apache/airflow/pull/74449#discussion_r4227104076
########## providers/common/ai/src/airflow/providers/common/ai/toolsets/__init__.py: ########## @@ -27,8 +27,11 @@ from airflow.providers.common.ai.toolsets.sandbox import SandboxToolset __all__ = [ + "AgentSkillsToolset", Review Comment: Yep, update it to every toolset now imports fro mthe root. `MCPToolset` doesn't raise on access, since `toolsets/mcp.py` has no optional import at module level. The missing dependency raises `ImportError` on first connection (in `MCPHook.get_conn`), so the docs say that instead. `SQLToolset` and `DataFusionToolset` are documented as raising `AirflowOptionalProviderFeatureException` on first access. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
