Python: [BREAKING] Cleanup of dependencies (#1803)

eavanvalkenburg · web-flow · commit 434ffb8ab8e2 · 2025-10-30T15:24:29.000Z
* removed extra and non-released deps

* added comments

* added space and lock

* fix pyright config
diff --git a/python/packages/a2a/pyproject.toml b/python/packages/a2a/pyproject.toml
@@ -55,8 +55,7 @@ omit = [
 ]
 
 [tool.pyright]
-extend = "../../pyproject.toml"
-exclude = ['tests']
+extends = "../../pyproject.toml"
 
 [tool.mypy]
 plugins = ['pydantic.mypy']
diff --git a/python/packages/azure-ai/pyproject.toml b/python/packages/azure-ai/pyproject.toml
@@ -25,7 +25,6 @@ dependencies = [
     "agent-framework-core",
     "azure-ai-projects >= 1.0.0b11",
     "azure-ai-agents == 1.2.0b5",
-    "aiohttp ~= 3.8",
 ]
 
 [tool.uv]
@@ -55,8 +54,7 @@ omit = [
 ]
 
 [tool.pyright]
-extend = "../../pyproject.toml"
-exclude = ['tests']
+extends = "../../pyproject.toml"
 
 [tool.mypy]
 plugins = ['pydantic.mypy']
diff --git a/python/packages/copilotstudio/pyproject.toml b/python/packages/copilotstudio/pyproject.toml
@@ -55,8 +55,8 @@ omit = [
 ]
 
 [tool.pyright]
-extend = "../../pyproject.toml"
-exclude = ['tests']
+extends = "../../pyproject.toml"
+
 
 [tool.mypy]
 plugins = ['pydantic.mypy']
diff --git a/python/packages/core/agent_framework/_workflows/_viz.py b/python/packages/core/agent_framework/_workflows/_viz.py
@@ -80,7 +80,8 @@ def export(self, format: Literal["svg", "png", "pdf", "dot"] = "svg", filename:
             import graphviz  # type: ignore
         except ImportError as e:
             raise ImportError(
-                "viz extra is required for export. Install it with: pip install agent-framework[viz] --pre. "
+                "viz extra is required for export. Install it with: pip install graphviz>=0.20.0 "
+                "The version needs to be at least 0.20.0. "
                 "You also need to install graphviz separately. E.g., sudo apt-get install graphviz on Debian/Ubuntu "
                 "or brew install graphviz on macOS. See https://graphviz.org/download/ for details."
             ) from e
diff --git a/python/packages/core/agent_framework/observability.py b/python/packages/core/agent_framework/observability.py
@@ -279,11 +279,17 @@ def _get_azure_monitor_exporters(
     credential: "TokenCredential | None" = None,
 ) -> list["LogExporter | SpanExporter | MetricExporter"]:
     """Create Azure Monitor Exporters, based on the connection strings and optionally the credential."""
-    from azure.monitor.opentelemetry.exporter import (
-        AzureMonitorLogExporter,
-        AzureMonitorMetricExporter,
-        AzureMonitorTraceExporter,
-    )
+    try:
+        from azure.monitor.opentelemetry.exporter import (
+            AzureMonitorLogExporter,
+            AzureMonitorMetricExporter,
+            AzureMonitorTraceExporter,
+        )
+    except ImportError as e:
+        raise ImportError(
+            "azure-monitor-opentelemetry-exporter is required for Azure Monitor exporters. "
+            "Install it with: pip install azure-monitor-opentelemetry-exporter>=1.0.0b41"
+        ) from e
 
     exporters: list["LogExporter | SpanExporter | MetricExporter"] = []
     for conn_string in connection_strings:
diff --git a/python/packages/core/pyproject.toml b/python/packages/core/pyproject.toml
@@ -22,33 +22,29 @@ classifiers = [
   "Typing :: Typed",
 ]
 dependencies = [
-    "openai>=1.99.0,<2",
+    # utilities
+    "typing-extensions",
     "pydantic>=2,<3",
     "pydantic-settings>=2,<3",
-    "typing-extensions",
+    # telemetry
     "opentelemetry-api>=1.24",
     "opentelemetry-sdk>=1.24",
-    "mcp[ws]>=1.13",
-    "azure-monitor-opentelemetry>=1.7.0",
-    "azure-monitor-opentelemetry-exporter>=1.0.0b41",
     "opentelemetry-exporter-otlp-proto-grpc>=1.36.0",
     "opentelemetry-semantic-conventions-ai>=0.4.13",
-    "aiofiles>=24.1.0",
+    # connectors and functions
+    "openai>=1.99.0,<2",
     "azure-identity>=1,<2",
+    "mcp[ws]>=1.13",
 ]
 
 [project.optional-dependencies]
-viz = [
-    "graphviz>=0.20.0"
-]
 all = [
     "agent-framework-a2a",
     "agent-framework-azure-ai",
     "agent-framework-copilotstudio",
     "agent-framework-mem0",
     "agent-framework-redis",
     "agent-framework-devui",
-    "graphviz>=0.20.0"
 ]
 
 [tool.uv]
@@ -87,8 +83,7 @@ omit = [
 extend = "../../pyproject.toml"
 
 [tool.pyright]
-extend = "../../pyproject.toml"
-exclude = ['tests']
+extends = "../../pyproject.toml"
 
 [tool.mypy]
 plugins = ['pydantic.mypy']
diff --git a/python/packages/devui/pyproject.toml b/python/packages/devui/pyproject.toml
@@ -62,8 +62,8 @@ omit = [
 ]
 
 [tool.pyright]
-extend = "../../pyproject.toml"
-exclude = ['tests']
+extends = "../../pyproject.toml"
+
 
 [tool.mypy]
 plugins = ['pydantic.mypy']
diff --git a/python/packages/lab/pyproject.toml b/python/packages/lab/pyproject.toml
@@ -119,7 +119,7 @@ omit = [
 ]
 
 [tool.pyright]
-extend = "../../pyproject.toml"
+extends = "../../pyproject.toml"
 exclude = ['gaia/tests', 'lightning/tests', 'tau2/tests', 'namespace', '**/samples']
 
 [tool.mypy]
diff --git a/python/packages/mem0/pyproject.toml b/python/packages/mem0/pyproject.toml
@@ -55,8 +55,8 @@ omit = [
 ]
 
 [tool.pyright]
-extend = "../../pyproject.toml"
-exclude = ['tests']
+extends = "../../pyproject.toml"
+
 
 [tool.mypy]
 plugins = ['pydantic.mypy']
diff --git a/python/packages/purview/pyproject.toml b/python/packages/purview/pyproject.toml
@@ -54,8 +54,8 @@ omit = [
 ]
 
 [tool.pyright]
-extend = "../../pyproject.toml"
-exclude = ['tests']
+extends = "../../pyproject.toml"
+
 
 [tool.mypy]
 plugins = ['pydantic.mypy']
diff --git a/python/packages/redis/pyproject.toml b/python/packages/redis/pyproject.toml
@@ -57,8 +57,7 @@ omit = [
 ]
 
 [tool.pyright]
-extend = "../../pyproject.toml"
-exclude = ['tests']
+extends = "../../pyproject.toml"
 
 [tool.mypy]
 plugins = ['pydantic.mypy']
diff --git a/python/pyproject.toml b/python/pyproject.toml
@@ -177,8 +177,8 @@ omit = [
 ]
 
 [tool.pyright]
-include = ["agent_framework", "samples"]
-exclude = ["**/tests/**", "docs", "**/.venv/**", "samples/semantic-kernel-migration/**", "samples/autogen-migration/**", "packages/devui/frontend/**"]
+include = ["agent_framework*"]
+exclude = ["**/tests/**", "docs", "**/.venv/**", "packages/devui/frontend/**"]
 typeCheckingMode = "strict"
 reportUnnecessaryIsInstance = false
 reportMissingTypeStubs = false
@@ -206,7 +206,6 @@ executor.type = "uv"
 
 [tool.poe.tasks]
 markdown-code-lint = """uv run python check_md_code_blocks.py README.md ./packages/**/README.md ./samples/**/*.md --exclude cookiecutter-agent-framework-lab --exclude tau2 --exclude packages/devui/frontend"""
-samples-code-check = """pyright ./samples"""
 docs-install = "uv sync --all-packages --all-extras --dev -U --prerelease=if-necessary-or-explicit --group=docs"
 docs-clean = "rm -rf docs/build"
 docs-build = "uv run python ./docs/generate_docs.py"
@@ -230,8 +229,8 @@ build-meta = "python -m flit build"
 build = ["build-packages", "build-meta"]
 publish = "uv publish"
 # combined checks
-check = ["fmt", "lint", "pyright", "mypy", "test", "markdown-code-lint", "samples-code-check"]
-pre-commit-check = ["fmt", "lint", "pyright", "markdown-code-lint", "samples-code-check"]
+check = ["fmt", "lint", "pyright", "mypy", "test", "markdown-code-lint"]
+pre-commit-check = ["fmt", "lint", "pyright", "markdown-code-lint"]
 
 [tool.poe.tasks.all-tests-cov]
 cmd = """
@@ -330,4 +329,4 @@ name = "agent_framework_meta"
 
 [build-system]
 requires = ["flit-core >= 3.11,<4.0"]
-build-backend = "flit_core.buildapi"
+build-backend = "flit_core.buildapi"
diff --git a/python/samples/getting_started/workflows/parallelism/map_reduce_and_visualization.py b/python/samples/getting_started/workflows/parallelism/map_reduce_and_visualization.py
@@ -37,7 +37,10 @@
 - aiofiles installed for async file I/O.
 - Write access to a tmp directory next to this script.
 - A source text at resources/long_text.txt.
-- Optional for SVG export: install the viz extra for agent framework workflow.
+- Optional for SVG export: install graphviz.
+
+Installation:
+    pip install agent-framework aiofiles graphviz
 """
 
 # Define the temporary directory for storing intermediate results
diff --git a/python/uv.lock b/python/uv.lock

Original file line number	Diff line number	Diff line change
`@@ -55,8 +55,7 @@ omit = [`
`55`	`55`	`]`
`56`	`56`
`57`	`57`	`[tool.pyright]`
`58`		`-extend = "../../pyproject.toml"`
`59`		`-exclude = ['tests']`
	`58`	`+extends = "../../pyproject.toml"`
`60`	`59`
`61`	`60`	`[tool.mypy]`
`62`	`61`	`plugins = ['pydantic.mypy']`
Original file line number	Diff line number	Diff line change
`@@ -25,7 +25,6 @@ dependencies = [`
`25`	`25`	`"agent-framework-core",`
`26`	`26`	`"azure-ai-projects >= 1.0.0b11",`
`27`	`27`	`"azure-ai-agents == 1.2.0b5",`
`28`		`- "aiohttp ~= 3.8",`
`29`	`28`	`]`
`30`	`29`
`31`	`30`	`[tool.uv]`
`@@ -55,8 +54,7 @@ omit = [`
`55`	`54`	`]`
`56`	`55`
`57`	`56`	`[tool.pyright]`
`58`		`-extend = "../../pyproject.toml"`
`59`		`-exclude = ['tests']`
	`57`	`+extends = "../../pyproject.toml"`
`60`	`58`
`61`	`59`	`[tool.mypy]`
`62`	`60`	`plugins = ['pydantic.mypy']`
Original file line number	Diff line number	Diff line change
`@@ -55,8 +55,8 @@ omit = [`
`55`	`55`	`]`
`56`	`56`
`57`	`57`	`[tool.pyright]`
`58`		`-extend = "../../pyproject.toml"`
`59`		`-exclude = ['tests']`
	`58`	`+extends = "../../pyproject.toml"`
	`59`	`+`
`60`	`60`
`61`	`61`	`[tool.mypy]`
`62`	`62`	`plugins = ['pydantic.mypy']`
Original file line number	Diff line number	Diff line change
`@@ -62,8 +62,8 @@ omit = [`
`62`	`62`	`]`
`63`	`63`
`64`	`64`	`[tool.pyright]`
`65`		`-extend = "../../pyproject.toml"`
`66`		`-exclude = ['tests']`
	`65`	`+extends = "../../pyproject.toml"`
	`66`	`+`
`67`	`67`
`68`	`68`	`[tool.mypy]`
`69`	`69`	`plugins = ['pydantic.mypy']`
Original file line number	Diff line number	Diff line change
`@@ -119,7 +119,7 @@ omit = [`
`119`	`119`	`]`
`120`	`120`
`121`	`121`	`[tool.pyright]`
`122`		`-extend = "../../pyproject.toml"`
	`122`	`+extends = "../../pyproject.toml"`
`123`	`123`	`exclude = ['gaia/tests', 'lightning/tests', 'tau2/tests', 'namespace', '**/samples']`
`124`	`124`
`125`	`125`	`[tool.mypy]`
Original file line number	Diff line number	Diff line change
`@@ -54,8 +54,8 @@ omit = [`
`54`	`54`	`]`
`55`	`55`
`56`	`56`	`[tool.pyright]`
`57`		`-extend = "../../pyproject.toml"`
`58`		`-exclude = ['tests']`
	`57`	`+extends = "../../pyproject.toml"`
	`58`	`+`
`59`	`59`
`60`	`60`	`[tool.mypy]`
`61`	`61`	`plugins = ['pydantic.mypy']`