feat(transfer): show client status and trusted height on render page

The Transfer section listed each client by ID only, making it hard to
tell which client is up-to-date or usable when several track the same
chain. Add read-only core.ClientLatestHeight and core.ClientStatus
accessors and surface both next to each client. Non-active clients
(Expired, Frozen) are still listed but without a "send via" link, since
packets can't be sent through them.

Author: tbruyelle

gno.land/r/aib/ibc/apps/transfer/filetests/z0bb_render_home_with_client_filetest.gno

@@ -36,7 +36,7 @@ func main(cur realm) {
 //
 // Trigger an IBC transfer from your wallet. Pick the client of the destination chain:
 //
-// - [send via `07-tendermint-1`](/r/aib/main$help&func=Transfer&amount=&clientID=07-tendermint-1&denom=&memo=&receiver=&timeoutTimestamp=1234571490) ([client details](/r/aib/ibc/core:clients/07-tendermint-1))
+// - [send via `07-tendermint-1`](/r/aib/main$help&func=Transfer&amount=&clientID=07-tendermint-1&denom=&memo=&receiver=&timeoutTimestamp=1234571490) — Active, trusted height `2/2` ([client details](/r/aib/ibc/core:clients/07-tendermint-1))
 //
 // ## Vouchers (0)
 //

gno.land/r/aib/ibc/apps/transfer/filetests/z0bbb_render_home_expired_client_filetest.gno

@@ -0,0 +1,65 @@
+// PKGPATH: gno.land/r/aib/main
+package main
+
+import (
+	"testing"
+	"time"
+
+	tmtesting "gno.land/p/aib/ibc/lightclient/tendermint/testing"
+	"gno.land/p/aib/ibc/types"
+
+	"gno.land/r/aib/ibc/apps/transfer"
+	"gno.land/r/aib/ibc/core"
+)
+
+// renderHome with a non-active (Expired) client: confirms the Transfer section
+// still lists the client with its status and trusted height, but omits the
+// "send via" txlink since packets can't be sent through it. The client is
+// created Active, then block time is advanced past the trusting period to
+// expire it.
+func main(cur realm) {
+	var (
+		chainID        = "chain-id-2"
+		clientState    = tmtesting.NewClientState(chainID, types.NewHeight(2, 2))
+		apphash        = tmtesting.Hash("apphash")
+		trustedValset  = tmtesting.GenValset()
+		consensusState = tmtesting.GenConsensusState(time.Now(), apphash, trustedValset.Hash())
+	)
+	clientID := core.CreateClient(cross(cur), clientState, consensusState)
+	core.RegisterCounterparty(cross(cur), clientID, [][]byte{[]byte("iavlStoreKey"), []byte("prefix2")}, "07-tendermint-2")
+
+	// Advance block time past the trusting period to expire the client.
+	ctx := testing.GetContext()
+	ctx.Time = time.Now().Add(clientState.TrustingPeriod)
+	testing.SetContext(ctx)
+
+	println(transfer.Render(""))
+}
+
+// Output:
+// # IBC transfer
+//
+// ICS-20 style transfer state and voucher token queries.
+//
+// ## Transfer
+//
+// Trigger an IBC transfer from your wallet. Pick the client of the destination chain:
+//
+// - `07-tendermint-1` — Expired, trusted height `2/2` ([client details](/r/aib/ibc/core:clients/07-tendermint-1))
+//
+// ## Vouchers (0)
+//
+// No vouchers yet.
+//
+// ## Escrow (0)
+//
+// No escrow yet.
+//
+// ## JSON endpoints
+//
+// - [`denoms`](/r/aib/ibc/apps/transfer:denoms): list known IBC denoms (`?page`, `?limit`)
+// - `denoms/ibc/{hash}`: get metadata for an IBC denom
+// - `total_escrow/{denom}`: get total escrow tracked for a base denom
+// - [`vouchers`](/r/aib/ibc/apps/transfer:vouchers): list voucher tokens (`?page`, `?limit`)
+// - `voucher/ibc/{hash}`: get voucher token metadata
+// - `voucher/ibc/{hash}/balance/{addr}`: get a voucher balance for an address

gno.land/r/aib/ibc/apps/transfer/render.gno

@@ -6,6 +6,7 @@ import (
 	"strings"
 	"time"
 
+	"gno.land/p/aib/ibc/lightclient"
 	"gno.land/p/aib/jsonpage"
 	"gno.land/p/moul/txlink"
 	"gno.land/p/nt/mux/v0"
@@ -186,10 +187,20 @@ func renderTransferLinks() string {
 
 	out.WriteString("Trigger an IBC transfer from your wallet. Pick the client of the destination chain:\n\n")
 	for _, clientID := range clientIDs {
-		out.WriteString(ufmt.Sprintf(
-			"- [send via `%s`](%s) ([client details](/r/aib/ibc/core:clients/%s))\n",
-			clientID, newTransferLink(clientID, "", "").URL(), clientID,
-		))
+		status := core.ClientStatus(clientID)
+		height := core.ClientLatestHeight(clientID)
+		if status == lightclient.Active {
+			out.WriteString(ufmt.Sprintf(
+				"- [send via `%s`](%s) — %s, trusted height `%s` ([client details](/r/aib/ibc/core:clients/%s))\n",
+				clientID, newTransferLink(clientID, "", "").URL(), status, height, clientID,
+			))
+		} else {
+			// Non-active client: no send link, since packets can't be sent.
+			out.WriteString(ufmt.Sprintf(
+				"- `%s` — %s, trusted height `%s` ([client details](/r/aib/ibc/core:clients/%s))\n",
+				clientID, status, height, clientID,
+			))
+		}
 	}
 
 	// Per-voucher links: send back via IBC, plus local GRC20 send/approve.

gno.land/r/aib/ibc/core/client.gno

@@ -58,6 +58,30 @@ func ClientIDs() []string {
 	return ids
 }
 
+// ClientLatestHeight returns the latest (highest) consensus height the given
+// client has been updated to, formatted as "<revision>/<height>". This is the
+// client's latest trusted height. Returns "" if the client is unknown.
+// Intended for read-only callers (UIs) that want to surface how up-to-date a
+// client is.
+func ClientLatestHeight(clientID string) string {
+	c := store.getClient(clientID)
+	if c == nil {
+		return ""
+	}
+	return c.lightClient.LatestHeight().String()
+}
+
+// ClientStatus returns the status of the given client (Active, Expired,
+// Frozen, ...). Returns "" if the client is unknown. Intended for read-only
+// callers (UIs) that want to surface whether a client is usable.
+func ClientStatus(clientID string) string {
+	c := store.getClient(clientID)
+	if c == nil {
+		return ""
+	}
+	return c.lightClient.Status()
+}
+
 // RegisterCounterparty will register the IBC v2 counterparty info for the
 // given clientID. It must be called by the same relayer that called
 // CreateClient.