chore: apply suggestion

Author: MicBun

core/contractsapi/bridge_history.go

@@ -18,10 +18,19 @@ func (s *Action) GetHistory(ctx context.Context, input types.GetHistoryInput) ([
 
 	limit := 20
 	if input.Limit != nil {
+		if *input.Limit < 0 {
+			return nil, errors.New("limit must be non-negative")
+		}
 		limit = *input.Limit
+		if limit > 100 {
+			limit = 100
+		}
 	}
 	offset := 0
 	if input.Offset != nil {
+		if *input.Offset < 0 {
+			return nil, errors.New("offset must be non-negative")
+		}
 		offset = *input.Offset
 	}
 

core/types/bridge_types.go

@@ -18,6 +18,6 @@ type BridgeHistory struct {
 type GetHistoryInput struct {
 	BridgeIdentifier string `validate:"required"`
 	Wallet           string `validate:"required"`
-	Limit            *int
-	Offset           *int
+	Limit            *int   `validate:"omitempty,min=1"`
+	Offset           *int   `validate:"omitempty,min=0"`
 }

docs/api-reference.md

@@ -1009,6 +1009,23 @@ history, err := client.GetHistory(ctx, types.GetHistoryInput{
 })
 ```
 
+#### `BridgeHistory` Struct
+
+```go
+type BridgeHistory struct {
+    Type                string  `json:"type"`                // "deposit" or "withdrawal"
+    Amount              string  `json:"amount"`              // NUMERIC(78,0) as string
+    FromAddress         []byte  `json:"from_address"`        // Sender address (if available)
+    ToAddress           []byte  `json:"to_address"`          // Recipient address
+    InternalTxHash      []byte  `json:"internal_tx_hash"`    // Kwil TX hash
+    ExternalTxHash      []byte  `json:"external_tx_hash"`    // Ethereum TX hash
+    Status              string  `json:"status"`              // "completed", "claimed", "pending_epoch"
+    BlockHeight         int64   `json:"block_height"`        // Kwil block height
+    BlockTimestamp      int64   `json:"block_timestamp"`     // Kwil block timestamp
+    ExternalBlockHeight *int64  `json:"external_block_height"` // Ethereum block height
+}
+```
+
 ### Performance Optimization
 
 #### Cache Strategy
@@ -2046,25 +2063,6 @@ type DecodedRow struct {
 }
 ```
 
-#### `BridgeHistory`
-
-Represents a transaction history record from the bridge extension.
-
-```go
-type BridgeHistory struct {
-    Type                string  `json:"type"`                // "deposit" or "withdrawal"
-    Amount              string  `json:"amount"`              // NUMERIC(78,0) as string
-    FromAddress         []byte  `json:"from_address"`        // Sender address (if available)
-    ToAddress           []byte  `json:"to_address"`          // Recipient address
-    InternalTxHash      []byte  `json:"internal_tx_hash"`    // Kwil TX hash
-    ExternalTxHash      []byte  `json:"external_tx_hash"`    // Ethereum TX hash
-    Status              string  `json:"status"`              // "completed", "claimed", "pending_epoch"
-    BlockHeight         int64   `json:"block_height"`        // Kwil block height
-    BlockTimestamp      int64   `json:"block_timestamp"`     // Kwil block timestamp
-    ExternalBlockHeight *int64  `json:"external_block_height"` // Ethereum block height
-}
-```
-
 **For attestation results:**
 - `Values[0]`: Unix timestamp as string (e.g., "1704067200")
 - `Values[1]`: 18-decimal fixed-point value as string (e.g., "77.051806494788211665")

examples/history_example/history_example

@@ -33,7 +33,7 @@ func main() {
 	// Use testnet gateway or local node
 	endpoint := os.Getenv("TN_GATEWAY_URL")
 	if endpoint == "" {
-		endpoint = "https://gateway.testnet.truf.network" // Default to local node
+		endpoint = "https://gateway.testnet.truf.network"
 	}
 
 	client, err := tnclient.NewClient(ctx, endpoint, tnclient.WithSigner(signer))
@@ -77,6 +77,17 @@ func main() {
 		return
 	}
 
+	// Helper to format byte slices
+	formatHexShort := func(b []byte) string {
+		if len(b) == 0 {
+			return "null"
+		}
+		if len(b) > 4 {
+			return fmt.Sprintf("0x%x...", b[:4])
+		}
+		return fmt.Sprintf("0x%x", b)
+	}
+
 	// Use tabwriter for aligned output
 	w := tabwriter.NewWriter(os.Stdout, 0, 0, 2, ' ', 0)
 	fmt.Fprintln(w, "TYPE\tAMOUNT\tFROM\tTO\tINTERNAL TX\tEXTERNAL TX\tSTATUS\tBLOCK\tEXT BLOCK\tTIMESTAMP")
@@ -87,19 +98,6 @@ func main() {
 		tm := time.Unix(rec.BlockTimestamp, 0)
 		timeStr := tm.Format(time.RFC3339)
 
-		// Shorten hashes for display if needed, but printing full for now as per request
-		// Or shortening to keep table readable? The CLI output truncated them.
-		// "0x%x..." logic was used before. I will use a helper to optionally shorten.
-		formatHexShort := func(b []byte) string {
-			if len(b) == 0 {
-				return "null"
-			}
-			if len(b) > 4 {
-				return fmt.Sprintf("0x%x...", b[:4])
-			}
-			return fmt.Sprintf("0x%x", b)
-		}
-
 		// Handle nullable external block height
 		extBlock := "null"
 		if rec.ExternalBlockHeight != nil {