agent0-sdk 1.2.0__py3-none-any.whl → 1.4.1__py3-none-any.whl

agent0_sdk/core/contracts.py

@@ -159,7 +159,7 @@ IDENTITY_REGISTRY_ABI = [
             {"internalType": "uint256", "name": "agentId", "type": "uint256"}
         ],
         "name": "getAgentWallet",
-        "outputs": [{"internalType": "bytes", "name": "", "type": "bytes"}],
+        "outputs": [{"internalType": "address", "name": "", "type": "address"}],
         "stateMutability": "view",
         "type": "function"
     },
@@ -231,7 +231,7 @@ REPUTATION_REGISTRY_ABI = [
     {
         "inputs": [
             {"internalType": "uint256", "name": "agentId", "type": "uint256"},
-            {"internalType": "int256", "name": "value", "type": "int256"},
+            {"internalType": "int128", "name": "value", "type": "int128"},
             {"internalType": "uint8", "name": "valueDecimals", "type": "uint8"},
             {"internalType": "string", "name": "tag1", "type": "string"},
             {"internalType": "string", "name": "tag2", "type": "string"},
@@ -285,7 +285,7 @@ REPUTATION_REGISTRY_ABI = [
         ],
         "name": "readFeedback",
         "outputs": [
-            {"internalType": "int256", "name": "value", "type": "int256"},
+            {"internalType": "int128", "name": "value", "type": "int128"},
             {"internalType": "uint8", "name": "valueDecimals", "type": "uint8"},
             {"internalType": "string", "name": "tag1", "type": "string"},
             {"internalType": "string", "name": "tag2", "type": "string"},
@@ -304,7 +304,7 @@ REPUTATION_REGISTRY_ABI = [
         "name": "getSummary",
         "outputs": [
             {"internalType": "uint64", "name": "count", "type": "uint64"},
-            {"internalType": "int256", "name": "summaryValue", "type": "int256"},
+            {"internalType": "int128", "name": "summaryValue", "type": "int128"},
             {"internalType": "uint8", "name": "summaryValueDecimals", "type": "uint8"}
         ],
         "stateMutability": "view",
@@ -322,7 +322,7 @@ REPUTATION_REGISTRY_ABI = [
         "outputs": [
             {"internalType": "address[]", "name": "clients", "type": "address[]"},
             {"internalType": "uint64[]", "name": "feedbackIndexes", "type": "uint64[]"},
-            {"internalType": "int256[]", "name": "values", "type": "int256[]"},
+            {"internalType": "int128[]", "name": "values", "type": "int128[]"},
             {"internalType": "uint8[]", "name": "valueDecimals", "type": "uint8[]"},
             {"internalType": "string[]", "name": "tag1s", "type": "string[]"},
             {"internalType": "string[]", "name": "tag2s", "type": "string[]"},
@@ -358,7 +358,7 @@ REPUTATION_REGISTRY_ABI = [
             {"indexed": True, "internalType": "uint256", "name": "agentId", "type": "uint256"},
             {"indexed": True, "internalType": "address", "name": "clientAddress", "type": "address"},
             {"indexed": False, "internalType": "uint64", "name": "feedbackIndex", "type": "uint64"},
-            {"indexed": False, "internalType": "int256", "name": "value", "type": "int256"},
+            {"indexed": False, "internalType": "int128", "name": "value", "type": "int128"},
             {"indexed": False, "internalType": "uint8", "name": "valueDecimals", "type": "uint8"},
             {"indexed": True, "internalType": "string", "name": "indexedTag1", "type": "string"},
             {"indexed": False, "internalType": "string", "name": "tag1", "type": "string"},
@@ -514,6 +514,11 @@ VALIDATION_REGISTRY_ABI = [
 # Contract registry for different chains
 # Updated addresses from: https://github.com/erc-8004/erc-8004-contracts
 DEFAULT_REGISTRIES: Dict[int, Dict[str, str]] = {
+    1: {  # Ethereum Mainnet
+        "IDENTITY": "0x8004A169FB4a3325136EB29fA0ceB6D2e539a432",
+        "REPUTATION": "0x8004BAa17C55a88189AE136b182e5fdA19dE9b63",
+        # "VALIDATION": "0x...",  # Set when deployed/enabled
+    },
     11155111: {  # Ethereum Sepolia
         "IDENTITY": "0x8004A818BFB912233c491871b3d84c89A494BD9e",
         "REPUTATION": "0x8004B663056A597Dffe9eCcC1965A193B7388713",
@@ -540,8 +545,9 @@ DEFAULT_REGISTRIES: Dict[int, Dict[str, str]] = {
 # Default subgraph URLs for different chains
 # Note: Subgraph URLs may need to be updated when new contracts are deployed
 DEFAULT_SUBGRAPH_URLS: Dict[int, str] = {
+    1: "https://gateway.thegraph.com/api/7fd2e7d89ce3ef24cd0d4590298f0b2c/subgraphs/id/FX78UzofJFr5h2Udznv7pZ2uLG1JBbYsPm7eecRSYnty",  # Ethereum Mainnet
     11155111: "https://gateway.thegraph.com/api/00a452ad3cd1900273ea62c1bf283f93/subgraphs/id/6wQRC7geo9XYAhckfmfo8kbMRLeWU8KQd3XsJqFKmZLT",  # Ethereum Sepolia
     # Other chains temporarily disabled - subgraphs to be updated
     # 84532: "https://gateway.thegraph.com/api/...",  # Base Sepolia - To be updated
     # 80002: "https://gateway.thegraph.com/api/...",  # Polygon Amoy - To be updated
-}
+}

agent0_sdk/core/feedback_manager.py

@@ -17,6 +17,7 @@ from .models import (
 from .web3_client import Web3Client
 from .ipfs_client import IPFSClient
 from .value_encoding import encode_feedback_value, decode_feedback_value
+from .transaction_handle import TransactionHandle
 
 logger = logging.getLogger(__name__)
 
@@ -73,7 +74,7 @@ class FeedbackManager:
         tag2: Optional[str] = None,
         endpoint: Optional[str] = None,
         feedbackFile: Optional[Dict[str, Any]] = None,
-    ) -> Feedback:
+    ) -> TransactionHandle[Feedback]:
         """Give feedback (maps 8004 endpoint)."""
         # Parse agentId into (chainId, tokenId)
         agent_chain_id: Optional[int] = None
@@ -185,7 +186,7 @@ class FeedbackManager:
                     "clientAddress": f"eip155:{agent_chain_id}:{clientAddress}",
                     "createdAt": created_at,
                     # On-chain fields (store raw+decimals for precision)
-                    "value": str(value_raw),
+                    "value": int(value_raw),
                     "valueDecimals": int(value_decimals),
 
                     # OPTIONAL fields that mirror on-chain
@@ -220,35 +221,34 @@ class FeedbackManager:
                 feedbackUri,
                 feedbackHash
             )
-            
-            # Wait for transaction confirmation
-            receipt = self.web3_client.wait_for_transaction(txHash)
-            
         except Exception as e:
             raise ValueError(f"Failed to submit feedback to blockchain: {e}")
-        
-        # Create feedback object (address normalization happens in Feedback.create_id)
+
+        # Create a tx handle; build the Feedback object on confirmation.
         feedbackId = Feedback.create_id(agentId, clientAddress, feedbackIndex)
-        
         ff: Dict[str, Any] = feedback_file or {}
-        return Feedback(
-            id=feedbackId,
-            agentId=agentId,
-            reviewer=clientAddress,  # create_id normalizes the ID; reviewer field can remain as-is
-            value=decode_feedback_value(value_raw, value_decimals),
-            tags=[tag1, tag2] if tag1 or tag2 else [],
-            text=ff.get("text"),
-            context=ff.get("context"),
-            proofOfPayment=ff.get("proofOfPayment"),
-            fileURI=feedbackUri if feedbackUri else None,
-            endpoint=endpoint_onchain if endpoint_onchain else None,
-            createdAt=int(time.time()),
-            isRevoked=False,
-            # Off-chain only fields
-            capability=ff.get("capability"),
-            name=ff.get("name"),
-            skill=ff.get("skill"),
-            task=ff.get("task")
+
+        return TransactionHandle(
+            web3_client=self.web3_client,
+            tx_hash=txHash,
+            compute_result=lambda _receipt: Feedback(
+                id=feedbackId,
+                agentId=agentId,
+                reviewer=clientAddress,
+                value=decode_feedback_value(value_raw, value_decimals),
+                tags=[tag1, tag2] if tag1 or tag2 else [],
+                text=ff.get("text"),
+                context=ff.get("context"),
+                proofOfPayment=ff.get("proofOfPayment"),
+                fileURI=feedbackUri if feedbackUri else None,
+                endpoint=endpoint_onchain if endpoint_onchain else None,
+                createdAt=int(time.time()),
+                isRevoked=False,
+                capability=ff.get("capability"),
+                name=ff.get("name"),
+                skill=ff.get("skill"),
+                task=ff.get("task"),
+            ),
         )
 
     def getFeedback(
@@ -412,7 +412,7 @@ class FeedbackManager:
 
     def searchFeedback(
         self,
-        agentId: AgentId,
+        agentId: Optional[AgentId] = None,
         clientAddresses: Optional[List[Address]] = None,
         tags: Optional[List[str]] = None,
         capabilities: Optional[List[str]] = None,
@@ -424,25 +424,68 @@ class FeedbackManager:
         include_revoked: bool = False,
         first: int = 100,
         skip: int = 0,
+        agents: Optional[List[AgentId]] = None,
     ) -> List[Feedback]:
-        """Search feedback for an agent - uses subgraph if available."""
+        """Search feedback.
+        
+        Backwards compatible:
+        - `agentId` was previously required; it is now optional.
+        
+        New:
+        - `agents` supports searching across multiple agents.
+        - If neither `agentId` nor `agents` are provided, the query can still run via subgraph
+          using other filters like `clientAddresses` (reviewers), tags, etc.
+        """
         # Use indexer for subgraph queries (unified search interface)
         if self.indexer and self.subgraph_client:
             # Indexer handles subgraph queries for unified search architecture
             # This enables future semantic search capabilities
             return self.indexer.search_feedback(
-                agentId, clientAddresses, tags, capabilities, skills, tasks, names,
-                minValue, maxValue, include_revoked, first, skip
+                agentId,
+                clientAddresses,
+                tags,
+                capabilities,
+                skills,
+                tasks,
+                names,
+                minValue,
+                maxValue,
+                include_revoked,
+                first,
+                skip,
+                agents=agents,
             )
         
         # Fallback: direct subgraph access (if indexer not available)
         if self.subgraph_client:
             return self._search_feedback_subgraph(
-                agentId, clientAddresses, tags, capabilities, skills, tasks, names,
-                minValue, maxValue, include_revoked, first, skip
+                agentId,
+                clientAddresses,
+                tags,
+                capabilities,
+                skills,
+                tasks,
+                names,
+                minValue,
+                maxValue,
+                include_revoked,
+                first,
+                skip,
+                agents=agents,
             )
         
-        # Fallback to blockchain
+        # Fallback to blockchain (requires a specific agent)
+        if not agentId and not agents:
+            raise ValueError(
+                "searchFeedback requires a subgraph when searching without agentId/agents."
+            )
+        if not agentId and agents and len(agents) == 1:
+            agentId = agents[0]
+        if not agentId:
+            raise ValueError(
+                "Blockchain fallback only supports searching a single agent; provide agentId or a single-item agents=[...]."
+            )
+
         # Parse agent ID
         if ":" in agentId:
             tokenId = int(agentId.split(":")[-1])
@@ -505,7 +548,7 @@ class FeedbackManager:
     
     def _search_feedback_subgraph(
         self,
-        agentId: AgentId,
+        agentId: Optional[AgentId],
         clientAddresses: Optional[List[Address]],
         tags: Optional[List[str]],
         capabilities: Optional[List[str]],
@@ -517,11 +560,18 @@ class FeedbackManager:
         include_revoked: bool,
         first: int,
         skip: int,
+        agents: Optional[List[AgentId]] = None,
     ) -> List[Feedback]:
         """Search feedback using subgraph."""
+        merged_agents: Optional[List[AgentId]] = None
+        if agents:
+            merged_agents = list(agents)
+        if agentId:
+            merged_agents = (merged_agents or []) + [agentId]
+
         # Create SearchFeedbackParams
         params = SearchFeedbackParams(
-            agents=[agentId],
+            agents=merged_agents,
             reviewers=clientAddresses,
             tags=tags,
             capabilities=capabilities,
@@ -613,7 +663,7 @@ class FeedbackManager:
         self,
         agentId: AgentId,
         feedbackIndex: int,
-    ) -> Dict[str, Any]:
+    ) -> TransactionHandle[Feedback]:
         """Revoke feedback."""
         # Parse agent ID
         if ":" in agentId:
@@ -630,17 +680,11 @@ class FeedbackManager:
                 tokenId,
                 feedbackIndex
             )
-            
-            receipt = self.web3_client.wait_for_transaction(txHash)
-            
-            return {
-                "txHash": txHash,
-                "agentId": agentId,
-                "clientAddress": clientAddress,
-                "feedbackIndex": feedbackIndex,
-                "status": "revoked"
-            }
-            
+            return TransactionHandle(
+                web3_client=self.web3_client,
+                tx_hash=txHash,
+                compute_result=lambda _receipt: self.getFeedback(agentId, clientAddress, feedbackIndex),
+            )
         except Exception as e:
             raise ValueError(f"Failed to revoke feedback: {e}")
 
@@ -650,7 +694,7 @@ class FeedbackManager:
         clientAddress: Address,
         feedbackIndex: int,
         response: Dict[str, Any],
-    ) -> Feedback:
+    ) -> TransactionHandle[Feedback]:
         """Append a response/follow-up to existing feedback."""
         # Parse agent ID
         if ":" in agentId:
@@ -681,12 +725,11 @@ class FeedbackManager:
                 responseUri,  # Note: contract uses responseURI but variable name kept for compatibility
                 responseHash
             )
-            
-            receipt = self.web3_client.wait_for_transaction(txHash)
-            
-            # Read updated feedback
-            return self.getFeedback(agentId, clientAddress, feedbackIndex)
-            
+            return TransactionHandle(
+                web3_client=self.web3_client,
+                tx_hash=txHash,
+                compute_result=lambda _receipt: self.getFeedback(agentId, clientAddress, feedbackIndex),
+            )
         except Exception as e:
             raise ValueError(f"Failed to append response: {e}")
 

agent0_sdk/core/indexer.py

@@ -433,7 +433,7 @@ class AgentIndexer:
                 mcpPrompts=reg_file.get('mcpPrompts', []),
                 mcpResources=reg_file.get('mcpResources', []),
                 active=reg_file.get('active', True),
-                x402support=reg_file.get('x402support', False),
+                x402support=reg_file.get('x402Support', reg_file.get('x402support', False)),
                 extras={}
             )
             
@@ -724,7 +724,7 @@ class AgentIndexer:
                 "mcpPrompts": reg_file.get('mcpPrompts', []),
                 "mcpResources": reg_file.get('mcpResources', []),
                 "active": reg_file.get('active', True),
-                "x402support": reg_file.get('x402support', False),
+                "x402support": reg_file.get('x402Support', reg_file.get('x402support', False)),
                 "totalFeedback": agent_data.get('totalFeedback', 0),
                 "lastActivity": agent_data.get('lastActivity'),
                 "updatedAt": agent_data.get('updatedAt'),
@@ -871,7 +871,7 @@ class AgentIndexer:
                     "mcpPrompts": reg_file.get('mcpPrompts', []),
                     "mcpResources": reg_file.get('mcpResources', []),
                     "active": reg_file.get('active', True),
-                    "x402support": reg_file.get('x402support', False),
+                    "x402support": reg_file.get('x402Support', reg_file.get('x402support', False)),
                     "totalFeedback": agent.get('totalFeedback', 0),
                     "lastActivity": agent.get('lastActivity'),
                     "updatedAt": agent.get('updatedAt'),
@@ -1106,7 +1106,7 @@ class AgentIndexer:
     
     def search_feedback(
         self,
-        agentId: AgentId,
+        agentId: Optional[AgentId] = None,
         clientAddresses: Optional[List[Address]] = None,
         tags: Optional[List[str]] = None,
         capabilities: Optional[List[str]] = None,
@@ -1118,29 +1118,77 @@ class AgentIndexer:
         include_revoked: bool = False,
         first: int = 100,
         skip: int = 0,
+        agents: Optional[List[AgentId]] = None,
     ) -> List[Feedback]:
-        """Search feedback for an agent - uses subgraph if available."""
-        # Parse chainId from agentId
-        chain_id, token_id = self._parse_agent_id(agentId)
+        """Search feedback via subgraph.
+        
+        Backwards compatible:
+        - Previously required `agentId`; it is now optional.
+        
+        New:
+        - `agents` supports searching across multiple agents.
+        - If neither `agentId` nor `agents` is provided, subgraph search can still run using
+          other filters (e.g., reviewers / tags).
+        """
+
+        merged_agents: Optional[List[AgentId]] = None
+        if agents:
+            merged_agents = list(agents)
+        if agentId:
+            merged_agents = (merged_agents or []) + [agentId]
+
+        # Determine chain/subgraph client based on first specified agent (if any)
+        chain_id = None
+        if merged_agents and len(merged_agents) > 0:
+            first_agent = merged_agents[0]
+            chain_id, token_id = self._parse_agent_id(first_agent)
         
         # Get subgraph client for the chain
         subgraph_client = None
-        full_agent_id = agentId
-        
+
         if chain_id is not None:
             subgraph_client = self._get_subgraph_client_for_chain(chain_id)
         else:
-            # No chainId in agentId, use SDK's default
-            # Construct full agentId format for subgraph query
-            default_chain_id = self.web3_client.chain_id
-            full_agent_id = f"{default_chain_id}:{token_id}"
+            # If no explicit chainId, use SDK's default subgraph client (if configured).
             subgraph_client = self.subgraph_client
+
+        # If we have agent ids but they weren't chain-prefixed, prefix them with default chain id for the subgraph.
+        if merged_agents and chain_id is None:
+            default_chain_id = self.web3_client.chain_id
+            normalized: List[AgentId] = []
+            for aid in merged_agents:
+                if isinstance(aid, str) and ":" in aid:
+                    normalized.append(aid)
+                else:
+                    normalized.append(f"{default_chain_id}:{int(aid)}")
+            merged_agents = normalized
+        elif merged_agents and chain_id is not None:
+            # Ensure all agent ids are chain-prefixed for the chosen chain
+            normalized = []
+            for aid in merged_agents:
+                if isinstance(aid, str) and ":" in aid:
+                    normalized.append(aid)
+                else:
+                    normalized.append(f"{chain_id}:{int(aid)}")
+            merged_agents = normalized
         
         # Use subgraph if available (preferred)
         if subgraph_client:
             return self._search_feedback_subgraph(
-                full_agent_id, clientAddresses, tags, capabilities, skills, tasks, names,
-                minValue, maxValue, include_revoked, first, skip, subgraph_client
+                agentId=None,
+                agents=merged_agents,
+                clientAddresses=clientAddresses,
+                tags=tags,
+                capabilities=capabilities,
+                skills=skills,
+                tasks=tasks,
+                names=names,
+                minValue=minValue,
+                maxValue=maxValue,
+                include_revoked=include_revoked,
+                first=first,
+                skip=skip,
+                subgraph_client=subgraph_client,
             )
         
         # Fallback not implemented (would require blockchain queries)
@@ -1149,7 +1197,8 @@ class AgentIndexer:
     
     def _search_feedback_subgraph(
         self,
-        agentId: AgentId,
+        agentId: Optional[AgentId],
+        agents: Optional[List[AgentId]],
         clientAddresses: Optional[List[Address]],
         tags: Optional[List[str]],
         capabilities: Optional[List[str]],
@@ -1169,9 +1218,15 @@ class AgentIndexer:
         if not client:
             return []
         
+        merged_agents: Optional[List[AgentId]] = None
+        if agents:
+            merged_agents = list(agents)
+        if agentId:
+            merged_agents = (merged_agents or []) + [agentId]
+
         # Create SearchFeedbackParams
         params = SearchFeedbackParams(
-            agents=[agentId],
+            agents=merged_agents,
             reviewers=clientAddresses,
             tags=tags,
             capabilities=capabilities,
@@ -1305,7 +1360,7 @@ class AgentIndexer:
                 token_id
             )
             
-            # Get agentWallet using new dedicated function
+            # Get on-chain verified wallet (IdentityRegistry.getAgentWallet)
             wallet_address = None
             try:
                 wallet_address = self.web3_client.call_contract(
@@ -1316,7 +1371,6 @@ class AgentIndexer:
                 if wallet_address == "0x0000000000000000000000000000000000000000":
                     wallet_address = None
             except Exception:
-                # Fallback to registration file if getAgentWallet not available
                 pass
             
             # Create agent ID

agent0_sdk/core/models.py

@@ -89,7 +89,7 @@ class RegistrationFile:
             endpoints.append(endpoint_dict)
         
         # Note: agentWallet is no longer included in endpoints array.
-        # It's now a reserved on-chain metadata key managed via setAgentWallet().
+        # It's now a reserved on-chain metadata key managed via Agent.setWallet().
         
         # Build registrations array
         registrations = []
@@ -106,7 +106,7 @@ class RegistrationFile:
             "name": self.name,
             "description": self.description,
             "image": self.image,
-            "endpoints": endpoints,
+            "services": endpoints,
             "registrations": registrations,
             "supportedTrust": [tm.value if isinstance(tm, TrustModel) else tm for tm in self.trustModels],
             "active": self.active,
@@ -118,7 +118,8 @@ class RegistrationFile:
     def from_dict(cls, data: Dict[str, Any]) -> RegistrationFile:
         """Create from dictionary."""
         endpoints = []
-        for ep_data in data.get("endpoints", []):
+        raw_services = data.get("services", data.get("endpoints", []))
+        for ep_data in raw_services:
             name = ep_data["name"]
             # Special handling for agentWallet - it's not a standard endpoint type
             if name == "agentWallet":

agent0_sdk/core/sdk.py

@@ -27,6 +27,7 @@ from .agent import Agent
 from .indexer import AgentIndexer
 from .ipfs_client import IPFSClient
 from .feedback_manager import FeedbackManager
+from .transaction_handle import TransactionHandle
 from .subgraph_client import SubgraphClient
 
 
@@ -273,6 +274,8 @@ class SDK:
             name=name,
             description=description,
             image=image,
+            # Default trust model: reputation (if caller doesn't set one explicitly).
+            trustModels=[TrustModel.REPUTATION],
             updatedAt=int(time.time())
         )
         return Agent(sdk=self, registration_file=registration_file)
@@ -590,7 +593,7 @@ class SDK:
                     mcpPrompts=reg_file.get('mcpPrompts', []),
                     mcpResources=reg_file.get('mcpResources', []),
                     active=reg_file.get('active', True),
-                    x402support=reg_file.get('x402support', False),
+                    x402support=reg_file.get('x402Support', reg_file.get('x402support', False)),
                     extras={'averageValue': agent_data.get('averageValue')}
                 )
                 results.append(agent_summary)
@@ -747,7 +750,7 @@ class SDK:
                 mcpPrompts=reg_file.get('mcpPrompts', []),
                 mcpResources=reg_file.get('mcpResources', []),
                 active=reg_file.get('active', True),
-                x402support=reg_file.get('x402support', False),
+                x402support=reg_file.get('x402Support', reg_file.get('x402support', False)),
                 extras={'averageValue': agent_data.get('averageValue')}
             )
             results.append(agent_summary)
@@ -797,7 +800,7 @@ class SDK:
         tag2: Optional[str] = None,
         endpoint: Optional[str] = None,
         feedbackFile: Optional[Dict[str, Any]] = None,
-    ) -> "Feedback":
+    ) -> "TransactionHandle[Feedback]":
         """Give feedback (on-chain first; optional off-chain file upload).
         
         - If feedbackFile is None: submit on-chain only (no upload even if IPFS is configured).
@@ -825,7 +828,7 @@ class SDK:
 
     def searchFeedback(
         self,
-        agentId: "AgentId",
+        agentId: Optional["AgentId"] = None,
         reviewers: Optional[List["Address"]] = None,
         tags: Optional[List[str]] = None,
         capabilities: Optional[List[str]] = None,
@@ -837,10 +840,38 @@ class SDK:
         include_revoked: bool = False,
         first: int = 100,
         skip: int = 0,
+        agents: Optional[List["AgentId"]] = None,
     ) -> List["Feedback"]:
-        """Search feedback for an agent."""
+        """Search feedback.
+        
+        Backwards compatible:
+        - Previously required `agentId`; it is now optional.
+        
+        New:
+        - `agents` can be used to search feedback across multiple agents in one call.
+        - `reviewers` can now be used without specifying any agent, enabling "all feedback given by a wallet".
+        """
+        has_any_filter = any([
+            bool(agentId),
+            bool(agents),
+            bool(reviewers),
+            bool(tags),
+            bool(capabilities),
+            bool(skills),
+            bool(tasks),
+            bool(names),
+            minValue is not None,
+            maxValue is not None,
+        ])
+        if not has_any_filter:
+            raise ValueError(
+                "searchFeedback requires at least one filter "
+                "(agentId/agents/reviewers/tags/capabilities/skills/tasks/names/minValue/maxValue)."
+            )
+
         return self.feedback_manager.searchFeedback(
             agentId=agentId,
+            agents=agents,
             clientAddresses=reviewers,
             tags=tags,
             capabilities=capabilities,
@@ -857,13 +888,10 @@ class SDK:
     def revokeFeedback(
         self,
         agentId: "AgentId",
-        clientAddress: "Address",
         feedbackIndex: int,
-    ) -> "Feedback":
-        """Revoke feedback."""
-        return self.feedback_manager.revokeFeedback(
-            agentId, clientAddress, feedbackIndex
-        )
+    ) -> "TransactionHandle[Feedback]":
+        """Revoke feedback (submitted-by-default)."""
+        return self.feedback_manager.revokeFeedback(agentId, feedbackIndex)
     
     def appendResponse(
         self,
@@ -871,8 +899,8 @@ class SDK:
         clientAddress: "Address",
         feedbackIndex: int,
         response: Dict[str, Any],
-    ) -> "Feedback":
-        """Append a response/follow-up to existing feedback."""
+    ) -> "TransactionHandle[Feedback]":
+        """Append a response/follow-up to existing feedback (submitted-by-default)."""
         return self.feedback_manager.appendResponse(
             agentId, clientAddress, feedbackIndex, response
         )
@@ -890,7 +918,7 @@ class SDK:
         self,
         agentId: "AgentId",
         newOwnerAddress: str,
-    ) -> Dict[str, Any]:
+    ) -> "TransactionHandle[Dict[str, Any]]":
         """Transfer agent ownership to a new address.
         
         Convenience method that loads the agent and calls transfer().