From 9db21d37ef39c118113b2181ded98dce7b0a6a42 Mon Sep 17 00:00:00 2001
From: Elliot Slusky <44592435+ElliotSlusky@users.noreply.github.com>
Date: Thu, 16 Jul 2026 18:13:36 -0700
Subject: [PATCH] style: apply Ruff formatting to recently merged tests (#644)

Fix the Ruff formatter check on main by formatting tests changed in #639 and #640 with the repository's pinned Ruff 0.15.1. No behavior change.
---
 tests/skills/test_importer.py     | 8 ++------
 tests/skills/test_skills.py       | 4 +---
 tests/tools/test_knowledge_sql.py | 4 +---
 3 files changed, 4 insertions(+), 12 deletions(-)

diff --git a/tests/skills/test_importer.py b/tests/skills/test_importer.py
index 51c0f741..08008d44 100644
--- a/tests/skills/test_importer.py
+++ b/tests/skills/test_importer.py
@@ -255,9 +255,7 @@ class TestDangerousCapabilityGate:
         assert result.success
         assert result.requires_confirmation
         assert any("confirmed by caller" in w for w in result.warnings)
-        content = (
-            tmp_path / "skills" / "hermes" / "my-skill" / ".source"
-        ).read_text()
+        content = (tmp_path / "skills" / "hermes" / "my-skill" / ".source").read_text()
         assert 'trust_tier = "unreviewed"' in content
         assert 'dangerous_capabilities = ["shell:execute"]' in content
 
@@ -269,8 +267,6 @@ class TestDangerousCapabilityGate:
         assert result.success
         assert not result.requires_confirmation
         assert result.dangerous_capabilities == []
-        content = (
-            tmp_path / "skills" / "hermes" / "my-skill" / ".source"
-        ).read_text()
+        content = (tmp_path / "skills" / "hermes" / "my-skill" / ".source").read_text()
         assert 'trust_tier = "unreviewed"' in content
         assert "dangerous_capabilities = []" in content
diff --git a/tests/skills/test_skills.py b/tests/skills/test_skills.py
index fb587923..b13810e7 100644
--- a/tests/skills/test_skills.py
+++ b/tests/skills/test_skills.py
@@ -171,9 +171,7 @@ class TestSkillExecutorCapabilities:
         assert result.context.get("result") == "hello"
 
     def test_policy_blocks_missing_capability(self):
-        executor = SkillExecutor(
-            ToolExecutor([EchoTool()]), allowed_capabilities=set()
-        )
+        executor = SkillExecutor(ToolExecutor([EchoTool()]), allowed_capabilities=set())
         result = executor.run(self._manifest())
         assert not result.success
         assert len(result.step_results) == 1
diff --git a/tests/tools/test_knowledge_sql.py b/tests/tools/test_knowledge_sql.py
index 1b3f8d70..72bec1f1 100644
--- a/tests/tools/test_knowledge_sql.py
+++ b/tests/tools/test_knowledge_sql.py
@@ -70,9 +70,7 @@ def test_allows_select_with_keyword_substring(store: KnowledgeStore) -> None:
     from openjarvis.tools.knowledge_sql import KnowledgeSQLTool
 
     tool = KnowledgeSQLTool(store=store)
-    result = tool.execute(
-        query="SELECT author AS created_author FROM knowledge_chunks"
-    )
+    result = tool.execute(query="SELECT author AS created_author FROM knowledge_chunks")
     assert result.success, result.content
     assert "Alice" in result.content