Skip to content

Commit 47d754b

Browse files
visahakclaude
andcommitted
fix(formatting): apply ruff formatting to mcp_server and entity sharing tests
Fixes failing CI check: check-formatting (3.12) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 229d228 commit 47d754b

3 files changed

Lines changed: 16 additions & 10 deletions

File tree

altk_evolve/frontend/mcp/mcp_server.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -332,6 +332,7 @@ def publish_entity(entity_id: str, user_id: str | None = None) -> str:
332332
logger.info(f"publish entity={entity_id} owner={user_id} namespace={evolve_config.namespace_id}")
333333
try:
334334
from datetime import datetime, UTC
335+
335336
updated = get_client().patch_entity_metadata(
336337
namespace_id=evolve_config.namespace_id,
337338
entity_id=entity_id,

tests/unit/test_entity_sharing.py

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@ def mock_get_client():
2222

2323
# ── create_entity ─────────────────────────────────────────────────────────────
2424

25+
2526
def test_create_entity_stores_visibility(mock_get_client):
2627
mock_get_client.update_entities.return_value = [
2728
EntityUpdate(id="1", type="guideline", content="test", event="ADD", metadata={"visibility": "private"})
@@ -48,16 +49,15 @@ def test_create_entity_invalid_visibility(mock_get_client):
4849

4950

5051
def test_create_entity_no_owner_id_omitted(mock_get_client):
51-
mock_get_client.update_entities.return_value = [
52-
EntityUpdate(id="1", type="note", content="x", event="ADD", metadata={})
53-
]
52+
mock_get_client.update_entities.return_value = [EntityUpdate(id="1", type="note", content="x", event="ADD", metadata={})]
5453
create_entity(content="x", entity_type="note")
5554
entities = mock_get_client.update_entities.call_args[1]["entities"]
5655
assert "owner_id" not in entities[0].metadata
5756

5857

5958
# ── publish_entity ─────────────────────────────────────────────────────────────
6059

60+
6161
def _make_entity(entity_id="42", visibility="private", owner_id=None):
6262
meta = {"visibility": visibility}
6363
if owner_id:
@@ -92,6 +92,7 @@ def test_publish_entity_no_user_id(mock_get_client):
9292

9393
def test_publish_entity_not_found(mock_get_client):
9494
from altk_evolve.schema.exceptions import EvolveException
95+
9596
mock_get_client.patch_entity_metadata.side_effect = EvolveException("Entity '99' not found")
9697

9798
result = json.loads(publish_entity(entity_id="99", user_id="alice"))
@@ -100,6 +101,7 @@ def test_publish_entity_not_found(mock_get_client):
100101

101102
# ── unpublish_entity ───────────────────────────────────────────────────────────
102103

104+
103105
def test_unpublish_entity_reverts_to_private(mock_get_client):
104106
updated = _make_entity(visibility="private")
105107
mock_get_client.patch_entity_metadata.return_value = updated
@@ -114,6 +116,7 @@ def test_unpublish_entity_reverts_to_private(mock_get_client):
114116

115117
# ── get_entities with include_public ──────────────────────────────────────────
116118

119+
117120
def test_get_entities_include_public_annotates_results(mock_get_client):
118121
private = _make_entity(entity_id="1", visibility="private")
119122
public = _make_entity(entity_id="2", visibility="public", owner_id="bob")
@@ -149,11 +152,10 @@ def test_get_entities_deduplicates_public_already_in_private(mock_get_client):
149152

150153
# ── backward compatibility ─────────────────────────────────────────────────────
151154

155+
152156
def test_create_entity_backward_compat_no_visibility_args(mock_get_client):
153157
"""Calling create_entity without new args should still work and default to private."""
154-
mock_get_client.update_entities.return_value = [
155-
EntityUpdate(id="1", type="fact", content="x", event="ADD", metadata={})
156-
]
158+
mock_get_client.update_entities.return_value = [EntityUpdate(id="1", type="fact", content="x", event="ADD", metadata={})]
157159
result = json.loads(create_entity(content="x", entity_type="fact"))
158160
assert "error" not in result
159161
entities = mock_get_client.update_entities.call_args[1]["entities"]

tests/unit/test_entity_sharing_e2e.py

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@ def ns(client):
2727

2828
# ── get_entity_by_id ──────────────────────────────────────────────────────────
2929

30+
3031
def test_get_entity_by_id_returns_entity(client, ns):
3132
updates = client.update_entities(ns, [Entity(type="guideline", content="be concise")], enable_conflict_resolution=False)
3233
entity_id = updates[0].id
@@ -43,9 +44,11 @@ def test_get_entity_by_id_missing_returns_none(client, ns):
4344

4445
# ── patch_entity_metadata ─────────────────────────────────────────────────────
4546

47+
4648
def test_patch_entity_metadata_merges_without_touching_content(client, ns):
4749
updates = client.update_entities(
48-
ns, [Entity(type="guideline", content="original content", metadata={"creation_mode": "manual"})],
50+
ns,
51+
[Entity(type="guideline", content="original content", metadata={"creation_mode": "manual"})],
4952
enable_conflict_resolution=False,
5053
)
5154
entity_id = updates[0].id
@@ -76,6 +79,7 @@ def test_patch_entity_metadata_raises_for_missing_entity(client, ns):
7679

7780
# ── publish / unpublish round-trip ────────────────────────────────────────────
7881

82+
7983
def test_publish_makes_entity_appear_in_public_search(client, ns):
8084
updates = client.update_entities(ns, [Entity(type="guideline", content="use context managers")], enable_conflict_resolution=False)
8185
entity_id = updates[0].id
@@ -106,13 +110,12 @@ def test_private_entities_not_returned_by_get_public(client, ns):
106110

107111
# ── cross-namespace public discovery ──────────────────────────────────────────
108112

113+
109114
def test_public_entity_in_one_namespace_visible_from_another(client):
110115
ns_a = client.create_namespace("user_a").id
111116
ns_b = client.create_namespace("user_b").id
112117

113-
updates = client.update_entities(
114-
ns_a, [Entity(type="guideline", content="always write tests")], enable_conflict_resolution=False
115-
)
118+
updates = client.update_entities(ns_a, [Entity(type="guideline", content="always write tests")], enable_conflict_resolution=False)
116119
entity_id = updates[0].id
117120
client.patch_entity_metadata(ns_a, entity_id, {"visibility": "public", "owner_id": "alice"})
118121

0 commit comments

Comments
 (0)