hyperforge-nucliadb 1.0.0.post105__tar.gz → 1.0.0.post108__tar.gz

{hyperforge_nucliadb-1.0.0.post105 → hyperforge_nucliadb-1.0.0.post108}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: hyperforge_nucliadb
-Version: 1.0.0.post105
+Version: 1.0.0.post108
 Summary: NucliaDB Hyperforge agent
 Author-email: Nuclia <nucliadb@nuclia.com>
 License-Expression: Apache-2.0

{hyperforge_nucliadb-1.0.0.post105 → hyperforge_nucliadb-1.0.0.post108}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "hyperforge_nucliadb"
-version = "1.0.0.post105"
+version = "1.0.0.post108"
 license = "Apache-2.0"
 description = "NucliaDB Hyperforge agent"
 authors = [{ name = "Nuclia", email = "nucliadb@nuclia.com" }]

{hyperforge_nucliadb-1.0.0.post105 → hyperforge_nucliadb-1.0.0.post108}/src/hyperforge_nucliadb/sync/agent.py

@@ -398,6 +398,8 @@ class SyncAskAgent(BasicAskAgent):
                 connection_id,
                 connection_credentials,
             ) in existing_credentials.items():
+                if connection_id not in connections_by_resource:
+                    continue
                 source = connections[kb_source_id]
                 if (
                     connection_id not in source.sync_configs

{hyperforge_nucliadb-1.0.0.post105 → hyperforge_nucliadb-1.0.0.post108}/src/hyperforge_nucliadb.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: hyperforge_nucliadb
-Version: 1.0.0.post105
+Version: 1.0.0.post108
 Summary: NucliaDB Hyperforge agent
 Author-email: Nuclia <nucliadb@nuclia.com>
 License-Expression: Apache-2.0

{hyperforge_nucliadb-1.0.0.post105 → hyperforge_nucliadb-1.0.0.post108}/tests/test_sync.py

@@ -311,6 +311,60 @@ async def test_hybrid_dynamic_connection_is_authorized():
     assert result == {"source": {SYNC_CONFIG_ID: ["connected-resource"]}}
 
 
+async def test_hybrid_dynamic_connection_ignores_unrequested_credentials():
+    agent = SyncAskAgent(SyncAskAgentConfig(sources=["source"]))
+    driver = SimpleNamespace(
+        config=SimpleNamespace(connection_ids=[]),
+        sync_configs={SYNC_CONFIG_ID: [EXTERNAL_CONNECTION_ID]},
+        information={
+            EXTERNAL_CONNECTION_ID: SimpleNamespace(provider="sharefile_oauth")
+        },
+        validate_resources=AsyncMock(return_value=["connected-resource"]),
+    )
+    resource = SimpleNamespace(
+        id="connected-resource",
+        origin=SimpleNamespace(
+            source_id=f"sync_config_{SYNC_CONFIG_ID}",
+            sync_metadata=SimpleNamespace(model_dump=MagicMock(return_value={})),
+        ),
+    )
+    ndb = SimpleNamespace(
+        config=SimpleNamespace(kbid="kb"),
+        driver=SimpleNamespace(get_resource_by_id=AsyncMock(return_value=resource)),
+    )
+    manager = SimpleNamespace(
+        drivers=SimpleNamespace(get=MagicMock(return_value=driver))
+    )
+    memory = MagicMock()
+    memory.get_session_id.return_value = "session"
+    memory.send_feedback = AsyncMock(
+        return_value=UserToAgentInteraction(
+            request_id="session",
+            response=json.dumps(
+                {
+                    "existing_credentials": {
+                        SYNC_CONFIG_ID: {EXTERNAL_CONNECTION_ID: "credential"},
+                        "00000000-0000-0000-0000-000000000002": {
+                            "unrequested-connection": "credential"
+                        },
+                    }
+                }
+            ),
+        )
+    )
+
+    with patch("hyperforge_nucliadb.sync.agent.get_ndb_driver", return_value=ndb):
+        result = await agent._post_filter_configured_resources(
+            memory=memory,
+            manager=manager,  # type: ignore[arg-type]
+            resources={"source": [resource.id]},
+            allowed_connection_ids={SYNC_CONFIG_ID},
+        )
+
+    assert result == {"source": {SYNC_CONFIG_ID: ["connected-resource"]}}
+    driver.validate_resources.assert_awaited_once()
+
+
 async def test_hybrid_resources_without_connections_skip_authorization():
     agent = SyncAskAgent(SyncAskAgentConfig(sources=["source"]))
     driver = SimpleNamespace(config=SimpleNamespace(connection_ids=[]))