feat: add getBalance and getAccountInfo tools

[monafrau24-wq]

Summary by cubic

Add Stellar account tools to fetch balances and account details, plus AgentClient methods to access them easily. Exposes both tools in stellarTools for use across the app.

• New Features
  • Added stellar_get_balance and stellar_get_account_info tools. Balance returns XLM and asset balances; account info includes sequence, signers, thresholds (excluding _links and balances).
  • Added AgentClient.getBalance(publicKey?) and AgentClient.getAccountInfo(publicKey?), defaulting to the client’s publicKey. Results return JSON when possible, otherwise a readable error message.
  • Tools validate public keys and support testnet and mainnet. Exported via stellarTools in index.ts.

^{Written for commit 40bf265. Summary will update on new commits. Review in cubic}

[cubic-dev-ai]

2 issues found across 3 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="tools/stellar.ts">

<violation number="1" location="tools/stellar.ts:97">
P2: Non-native balance formatting assumes `asset_code`/`asset_issuer` always exist, misreporting `liquidity_pool_shares` balances as `undefined:undefined`.</violation>
</file>

<file name="agent.ts">

<violation number="1" location="agent.ts:166">
P2: Using `publicKey || this.publicKey` silently falls back on empty-string input, which can query the wrong account instead of rejecting invalid caller input.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[cubic-dev-ai]

P2: Non-native balance formatting assumes asset_code/asset_issuer always exist, misreporting liquidity_pool_shares balances as undefined:undefined.

Prompt for AI agents

Check if this issue is valid — if so, understand the root cause and fix it. At tools/stellar.ts, line 97:

<comment>Non-native balance formatting assumes `asset_code`/`asset_issuer` always exist, misreporting `liquidity_pool_shares` balances as `undefined:undefined`.</comment>

<file context>
@@ -64,4 +64,82 @@ export const stellarSendPaymentTool = new DynamicStructuredTool({
+          };
+        } else {
+          return {
+            asset: `${balance.asset_code}:${balance.asset_issuer}`,
+            balance: balance.balance,
+          };
</file context>

Suggested change

	asset: `${balance.asset_code}:${balance.asset_issuer}`,
	asset:
	balance.asset_type === "liquidity_pool_shares"
	? `LP:${balance.liquidity_pool_id}`
	: `${balance.asset_code}:${balance.asset_issuer}`,

[cubic-dev-ai]

P2: Using publicKey || this.publicKey silently falls back on empty-string input, which can query the wrong account instead of rejecting invalid caller input.

Prompt for AI agents

Check if this issue is valid — if so, understand the root cause and fix it. At agent.ts, line 166:

<comment>Using `publicKey || this.publicKey` silently falls back on empty-string input, which can query the wrong account instead of rejecting invalid caller input.</comment>

<file context>
@@ -157,6 +158,51 @@ export class AgentClient {
+   * @param publicKey Optional Stellar public key (defaults to client's publicKey)
+   */
+  async getBalance(publicKey?: string) {
+    const targetPublicKey = publicKey || this.publicKey;
+    if (!targetPublicKey) {
+      throw new Error("Public key is required to fetch balance.");
</file context>