@cascade-fyi/sati-agent0-sdk 0.1.0 → 0.1.1

package/dist/index.mjs

@@ -615,7 +615,7 @@ var SatiAgent = class SatiAgent {
 		const transferIx = getTransferInstruction({
 			source: sourceAta,
 			destination: destAta,
-			authority: { address: ownerAddr },
+			authority: ownerAddr,
 			amount: 1n
 		});
 		return { signature: await access.sender.signAndSend([createAtaIx, transferIx]) };
@@ -652,30 +652,39 @@ var SatiAgent = class SatiAgent {
 		const agentMint = await generateKeyPairSigner();
 		const rpc = this._sdk.sati.getRpc();
 		const [registryConfigAddress] = await findRegistryConfigPda();
-		const registryConfig = await fetchRegistryConfig(rpc, registryConfigAddress);
-		const groupMint = registryConfig.data.groupMint;
 		const ownerAddress = address(access.sender.address);
 		const [agentTokenAccount] = await findAssociatedTokenAddress(agentMint.address, ownerAddress);
-		const [agentIndex] = await findAgentIndexPda(registryConfig.data.totalAgents + 1n);
-		const registerIx = await getRegisterAgentInstructionAsync({
-			payer: { address: ownerAddress },
-			owner: ownerAddress,
-			groupMint,
-			agentMint,
-			agentTokenAccount,
-			agentIndex,
-			name: this._registrationFile.name,
-			symbol: "",
-			uri,
-			additionalMetadata: null,
-			nonTransferable: false
-		});
-		const signature = await access.sender.signAndSend([registerIx], [agentMint]);
-		const finalMemberNumber = (await fetchRegistryConfig(rpc, registryConfigAddress)).data.totalAgents;
-		return {
-			signature,
-			agentId: this._storeIdentity(agentMint.address, ownerAddress, uri, finalMemberNumber)
-		};
+		const MAX_RETRIES = 3;
+		const RETRY_DELAY_MS = 1500;
+		let lastError;
+		for (let attempt = 0; attempt < MAX_RETRIES; attempt++) try {
+			const registryConfig = await fetchRegistryConfig(rpc, registryConfigAddress);
+			const groupMint = registryConfig.data.groupMint;
+			const memberNumber = registryConfig.data.totalAgents + 1n;
+			const [agentIndex] = await findAgentIndexPda(memberNumber);
+			const registerIx = await getRegisterAgentInstructionAsync({
+				payer: { address: ownerAddress },
+				owner: ownerAddress,
+				groupMint,
+				agentMint,
+				agentTokenAccount,
+				agentIndex,
+				name: this._registrationFile.name,
+				symbol: "",
+				uri,
+				additionalMetadata: null,
+				nonTransferable: false
+			});
+			return {
+				signature: await access.sender.signAndSend([registerIx], [agentMint]),
+				agentId: this._storeIdentity(agentMint.address, ownerAddress, uri, memberNumber)
+			};
+		} catch (error) {
+			lastError = error;
+			if (!isRegistrationCollisionError(error)) throw error;
+			if (attempt < MAX_RETRIES - 1) await new Promise((r) => setTimeout(r, RETRY_DELAY_MS));
+		}
+		throw new Error(`Registration failed after ${MAX_RETRIES} attempts`, { cause: lastError });
 	}
 	_storeIdentity(mint, owner, uri, memberNumber) {
 		this._identity = {
@@ -708,6 +717,11 @@ var SatiAgent = class SatiAgent {
 		return oasfEndpoint;
 	}
 };
+/** Detect PDA collision errors from concurrent registrations. */
+function isRegistrationCollisionError(error) {
+	const msg = error instanceof Error ? error.message : String(error);
+	return msg.includes("already in use") || msg.includes("already been initialized") || msg.includes("0x0");
+}
 
 //#endregion
 //#region src/sdk.ts
@@ -850,7 +864,7 @@ var SatiSDK = class {
 						sasSchema: schema,
 						agentMint: agent.mint
 					})).items.map((item) => {
-						if (item.data.contentType === ContentType$1.JSON && item.data.content.length > 0) return JSON.parse(new TextDecoder().decode(item.data.content))?.score;
+						return this._parseContentJson(item.data.content, item.data.contentType)?.score;
 					}).filter((s) => s !== void 0);
 					const count = scores.length;
 					const averageValue = count > 0 ? scores.reduce((a, b) => a + b, 0) / count : 0;
@@ -858,7 +872,14 @@ var SatiSDK = class {
 						count,
 						averageValue
 					});
-				} catch {}
+				} catch (error) {
+					this._warn({
+						code: "RPC_ERROR",
+						message: "Failed to fetch feedback stats",
+						context: agent.mint,
+						cause: error
+					});
+				}
 			}));
 		}
 		const results = [];
@@ -975,7 +996,7 @@ var SatiSDK = class {
 		const transferIx = getTransferInstruction({
 			source: sourceAta,
 			destination: destAta,
-			authority: { address: ownerAddr },
+			authority: ownerAddr,
 			amount: 1n
 		});
 		return { signature: await access.sender.signAndSend([createAtaIx, transferIx]) };
@@ -1199,13 +1220,19 @@ var SatiSDK = class {
 			satiFilter.agentMint = knownIdentity.mint;
 		}
 		if (filters.reviewers?.length) satiFilter.counterparty = address(filters.reviewers[0]);
+		const reviewerSet = filters.reviewers?.length ? new Set(filters.reviewers) : null;
+		const agentMintSet = filters.agents?.length && filters.agents.length > 1 ? new Set(await Promise.all(filters.agents.map(async (id) => {
+			return (await this._resolveIdentity(id)).mint;
+		}))) : null;
 		const result = await this._sati.listFeedbacks(satiFilter);
 		const currentSlot = await this._sati.getRpc().getSlot({ commitment: "confirmed" }).send();
 		const nowSec = Math.floor(Date.now() / 1e3);
 		const feedbacks = [];
 		for (let i = 0; i < result.items.length; i++) {
 			const item = result.items[i];
-			const rawContent = item.data.contentType === ContentType$1.JSON && item.data.content.length > 0 ? JSON.parse(new TextDecoder().decode(item.data.content)) : null;
+			if (reviewerSet && !reviewerSet.has(item.data.counterparty)) continue;
+			if (agentMintSet && !agentMintSet.has(item.data.agentMint)) continue;
+			const rawContent = this._parseContentJson(item.data.content, item.data.contentType);
 			const score = rawContent?.score;
 			const tags = rawContent?.tags ?? [];
 			const text = rawContent?.m;
@@ -1263,7 +1290,14 @@ var SatiSDK = class {
 				if (!addr) return;
 				try {
 					fb.txHash = (await photon.getCompressionSignaturesForAddress(address(addr), { limit: 1 })).items[0]?.signature;
-				} catch {}
+				} catch (error) {
+					this._warn({
+						code: "SIGNATURE_LOOKUP_FAILED",
+						message: "Failed to fetch tx signature",
+						context: addr,
+						cause: error
+					});
+				}
 			}));
 		}
 		return feedbacks;
@@ -1296,7 +1330,7 @@ var SatiSDK = class {
 		let sum = 0;
 		let count = 0;
 		for (const item of result.items) {
-			const rawContent = item.data.contentType === ContentType$1.JSON && item.data.content.length > 0 ? JSON.parse(new TextDecoder().decode(item.data.content)) : null;
+			const rawContent = this._parseContentJson(item.data.content, item.data.contentType);
 			const score = rawContent?.score;
 			const tags = rawContent?.tags ?? [];
 			if (tag1 && !tags.includes(tag1)) continue;
@@ -1336,7 +1370,13 @@ var SatiSDK = class {
 	async getCreationSignature(compressedAddress) {
 		try {
 			return (await this._sati.getLightClient().getRpc().getCompressionSignaturesForAddress(address(compressedAddress), { limit: 1 })).items[0]?.signature ?? null;
-		} catch {
+		} catch (error) {
+			this._warn({
+				code: "RPC_ERROR",
+				message: "Failed to fetch creation signature",
+				context: compressedAddress,
+				cause: error
+			});
 			return null;
 		}
 	}
@@ -1370,6 +1410,25 @@ var SatiSDK = class {
 		});
 	}
 	/**
+	* Revoke (close) a feedback by its compressed account address.
+	*
+	* Preferred over `revokeFeedback()` - uses a stable identifier instead of
+	* a fragile array index. Get the address from `feedback.context.satiCompressedAddress`.
+	*
+	* @param compressedAddress - Base58 compressed account address
+	*/
+	async revokeFeedbackByAddress(compressedAddress) {
+		const sasConfig = this._requireSASConfig();
+		const signer = this._requireSigner();
+		return this._sati.closeCompressedAttestation({
+			payer: signer,
+			counterparty: signer,
+			sasSchema: sasConfig.schemas.feedbackPublic ?? sasConfig.schemas.feedback,
+			attestationAddress: address(compressedAddress),
+			lookupTableAddress: sasConfig.lookupTable
+		});
+	}
+	/**
 	* Search validation attestations for an agent.
 	*
 	* Validations are on-chain attestations from validators (TEE, zkML, re-execution, etc.)
@@ -1418,6 +1477,19 @@ var SatiSDK = class {
 	get config() {
 		return this._config;
 	}
+	/** Safely parse JSON content from an attestation. Returns null on invalid JSON. */
+	_parseContentJson(content, contentType) {
+		if (contentType !== ContentType$1.JSON || content.length === 0) return null;
+		try {
+			return JSON.parse(new TextDecoder().decode(content));
+		} catch {
+			return null;
+		}
+	}
+	/** Fire a non-fatal warning via the optional onWarning callback. */
+	_warn(warning) {
+		this._config.onWarning?.(warning);
+	}
 	_requireSigner() {
 		if (!this._config.signer) throw new Error("This operation requires a KeyPairSigner. Initialize SatiSDK with a signer for server-side write operations.");
 		return this._config.signer;