fix: address version bump review feedback

Author: erichare

Makefile

@@ -642,7 +642,7 @@ patch: ## Update version across all projects. Usage: make patch v=1.5.0 [sdk_v=0
 	if ! grep -q "^version = \"$$LANGFLOW_VERSION\"" src/lfx/pyproject.toml; then echo "$(RED)✗ LFX pyproject.toml version validation failed$(NC)"; exit 1; fi; \
 	if ! grep -q "^version = \"$$SDK_VERSION\"" src/sdk/pyproject.toml; then echo "$(RED)✗ SDK pyproject.toml version validation failed$(NC)"; exit 1; fi; \
 	if ! grep -q "\"langflow-sdk>=$$SDK_VERSION\"" src/lfx/pyproject.toml; then echo "$(RED)✗ LFX SDK dependency validation failed$(NC)"; exit 1; fi; \
-	if ! grep -q "\"version\": \"$$LANGFLOW_VERSION\"" src/lfx/src/lfx/_assets/component_index.json; then echo "$(RED)✗ Component index version validation failed$(NC)"; exit 1; fi; \
+	if ! python -c 'import json, sys; index = json.load(open(sys.argv[1], encoding="utf-8")); raise SystemExit(index.get("version") != sys.argv[2])' src/lfx/src/lfx/_assets/component_index.json "$$LANGFLOW_VERSION"; then echo "$(RED)✗ Component index version validation failed$(NC)"; exit 1; fi; \
 	if ! grep -q "\"version\": \"$$LANGFLOW_VERSION\"" src/frontend/package.json; then echo "$(RED)✗ Frontend package.json version validation failed$(NC)"; exit 1; fi; \
 	echo "$(GREEN)✓ All versions updated successfully$(NC)"; \
 	\

src/backend/tests/unit/test_bundle_lfx_pin.py

@@ -15,7 +15,11 @@
 from pathlib import Path
 
 import pytest
-import tomllib
+
+try:
+    import tomllib
+except ModuleNotFoundError:  # Python 3.10: tomllib is stdlib only on 3.11+
+    import tomli as tomllib
 
 REPO_ROOT = Path(__file__).resolve().parents[4]
 _SCRIPT = REPO_ROOT / "scripts" / "ci" / "sync_bundle_lfx_pin.py"
@@ -145,7 +149,10 @@ def test_bundle_manifests_match_distribution_versions():
     """Keep published extension identity aligned with wheel metadata."""
     for pyproject in sorted((REPO_ROOT / "src" / "bundles").glob("*/pyproject.toml")):
         project_version = tomllib.loads(pyproject.read_text(encoding="utf-8"))["project"]["version"]
-        for manifest in pyproject.parent.glob("src/*/extension.json"):
+        manifests = sorted(pyproject.parent.glob("src/*/extension.json"))
+        if pyproject.parent.name != "lfx-bundles":
+            assert manifests, f"{pyproject.relative_to(REPO_ROOT)} is missing extension.json"
+        for manifest in manifests:
             manifest_version = json.loads(manifest.read_text(encoding="utf-8"))["version"]
             assert manifest_version == project_version, (
                 f"{manifest.relative_to(REPO_ROOT)} has version {manifest_version}, "

src/lfx/tests/unit/test_patch_regexes.py

@@ -7,6 +7,7 @@
 
 from __future__ import annotations
 
+import json
 import re
 
 # ---------------------------------------------------------------------------
@@ -45,6 +46,18 @@ def _patch_lfx_pyproject(txt: str, langflow_version: str) -> str:
     return re.sub(r'^version = ".*"', f'version = "{langflow_version}"', txt, flags=re.MULTILINE)
 
 
+def _patch_sdk_pyproject(txt: str, sdk_version: str) -> str:
+    return re.sub(r'^version = ".*"', f'version = "{sdk_version}"', txt, flags=re.MULTILINE)
+
+
+def _patch_lfx_sdk_dependency(txt: str, sdk_version: str) -> str:
+    return re.sub(r'"langflow-sdk(?:==|>=|~=)[^"]*"', f'"langflow-sdk>={sdk_version}"', txt)
+
+
+def _component_index_version_matches(txt: str, langflow_version: str) -> bool:
+    return json.loads(txt).get("version") == langflow_version
+
+
 # ---------------------------------------------------------------------------
 # langflow-core pins in main pyproject.toml
 # ---------------------------------------------------------------------------
@@ -220,6 +233,43 @@ def test_realistic_lfx_fragment(self):
         assert "Lightweight executor" in result
 
 
+# ---------------------------------------------------------------------------
+# SDK version and LFX SDK dependency
+# ---------------------------------------------------------------------------
+
+
+class TestSdkVersionSubstitution:
+    def test_updates_sdk_version(self):
+        txt = '[project]\nname = "langflow-sdk"\nversion = "0.3.0"\n'
+        result = _patch_sdk_pyproject(txt, "0.4.0")
+        assert 'version = "0.4.0"' in result
+        assert 'name = "langflow-sdk"' in result
+
+    def test_updates_lfx_sdk_dependency(self):
+        txt = 'dependencies = ["langflow-sdk~=0.3.0", "orjson>=3.10.0"]'
+        result = _patch_lfx_sdk_dependency(txt, "0.4.0")
+        assert '"langflow-sdk>=0.4.0"' in result
+        assert '"orjson>=3.10.0"' in result
+
+
+# ---------------------------------------------------------------------------
+# component-index version validation
+# ---------------------------------------------------------------------------
+
+
+class TestComponentIndexVersionValidation:
+    def test_accepts_matching_top_level_version(self):
+        index = {"entries": [], "version": "1.12.0"}
+        assert _component_index_version_matches(json.dumps(index), "1.12.0")
+
+    def test_rejects_nested_match_when_top_level_version_is_stale(self):
+        index = {
+            "entries": [["example", {"metadata": {"dependencies": [{"name": "langflow", "version": "1.12.0"}]}}]],
+            "version": "1.11.0",
+        }
+        assert not _component_index_version_matches(json.dumps(index), "1.12.0")
+
+
 # ---------------------------------------------------------------------------
 # release.yml: major.minor extraction from the current lfx constraint
 #