refactor: rename db.disable_agent() to db.revoke_agent() for consistency

Author: jackharrhy

src/where_the_plow/admin_routes.py

@@ -163,7 +163,7 @@ async def revoke_agent(
         return JSONResponse({"error": "Unauthorized"}, status_code=401)
 
     db: Database = request.app.state.db
-    db.disable_agent(agent_id)
+    db.revoke_agent(agent_id)
     return {"ok": True}
 
 

src/where_the_plow/db.py

@@ -563,7 +563,7 @@ def approve_agent(self, agent_id: str) -> None:
             [agent_id],
         )
 
-    def disable_agent(self, agent_id: str) -> None:
+    def revoke_agent(self, agent_id: str) -> None:
         """Revoke an agent by setting status='revoked'."""
         self._cursor().execute(
             "UPDATE agents SET status = 'revoked' WHERE agent_id = ?",

tests/test_agent_routes.py

@@ -90,7 +90,7 @@ def test_checkin_unknown_agent(agent_client):
 def test_checkin_revoked_agent(agent_client):
     db = app.state.db
     private_pem, _, agent_id, _ = _register_agent(db)
-    db.disable_agent(agent_id)
+    db.revoke_agent(agent_id)
 
     body = b""
     headers = _sign_request(private_pem, agent_id, body)

tests/test_agents_db.py

@@ -61,9 +61,9 @@ def test_list_agents(db):
     assert agents[1]["agent_id"] == "agent-2"
 
 
-def test_disable_agent(db):
+def test_revoke_agent(db):
     db.create_agent("agent-1", "My Agent", "pk_abc123")
-    db.disable_agent("agent-1")
+    db.revoke_agent("agent-1")
     agent = db.get_agent("agent-1")
     assert agent is not None
     assert agent["status"] == "revoked"