Skip to content
This repository was archived by the owner on May 11, 2026. It is now read-only.

Commit e036637

Browse files
committed
fix: Address additional Bugbot feedback
1. Document 'skip' recommendation in docstring 2. Restore coupon parameter to convenience function
1 parent 133cf4d commit e036637

1 file changed

Lines changed: 15 additions & 5 deletions

File tree

agents/connectors/cogdx.py

Lines changed: 15 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -162,7 +162,7 @@ def verify_before_trade(
162162
- approved: bool
163163
- validity_score: float
164164
- issues: list of detected problems
165-
- recommendation: 'proceed' | 'review' | 'reject'
165+
- recommendation: 'proceed' | 'review' | 'reject' | 'skip' (on error)
166166
"""
167167
result = self.analyze_reasoning(reasoning)
168168

@@ -198,7 +198,11 @@ def verify_before_trade(
198198
}
199199

200200

201-
def verify_trade_reasoning(reasoning: str, wallet: str = None) -> bool:
201+
def verify_trade_reasoning(
202+
reasoning: str,
203+
coupon: str = None,
204+
wallet: str = None
205+
) -> bool:
202206
"""
203207
Convenience function for quick trade verification.
204208
@@ -210,9 +214,15 @@ def verify_trade_reasoning(reasoning: str, wallet: str = None) -> bool:
210214
else:
211215
print("Reasoning flagged for review")
212216
213-
Returns True if reasoning passes verification, False otherwise.
214-
Note: Returns False if API is unavailable (fails closed).
217+
Args:
218+
reasoning: The reasoning trace to verify
219+
coupon: Optional coupon code for credits
220+
wallet: Optional wallet address for credits
221+
222+
Returns:
223+
True if reasoning passes verification, False otherwise.
224+
Note: Returns False if API is unavailable (fails closed).
215225
"""
216-
client = CogDxClient(wallet=wallet)
226+
client = CogDxClient(coupon=coupon, wallet=wallet)
217227
result = client.verify_before_trade(reasoning)
218228
return result.get("approved", False)

0 commit comments

Comments
 (0)