shll-skills 6.0.2 → 6.0.3

package/dist/{chunk-MPUPO5EJ.mjs → chunk-TH3ENDDM.mjs}

@@ -977,10 +977,14 @@ function checkActionRecipientSafety(action, vault) {
 function policyRejectionHelp(reason, tokenId) {
   const consoleUrl = agentConsoleUrl(tokenId);
   const r = reason ?? "";
+  if (r.includes("Function not allowed"))
+    return { explanation: "The contract method selector is not enabled by the active DeFi policy configuration.", action: "Enable the required protocol pack or selector in Console > Safety, or use a supported transaction path.", consoleUrl };
   if (r.includes("Approve spender not allowed"))
     return { explanation: "The DEX router address is not in the approved spender whitelist.", action: "Contact the Agent owner to approve this router, or use a different DEX.", consoleUrl };
   if (r.includes("Target not allowed"))
     return { explanation: "The contract address is not in the DeFi target whitelist.", action: "Enable the corresponding DeFi Pack in Console > Safety, or contact the Agent owner.", consoleUrl };
+  if (r.includes("Receiver must be vault"))
+    return { explanation: "Swap output recipient must be the agent vault address enforced by ReceiverGuard.", action: "Retry with recipient/to set to the vault address, or use a supported flow that routes proceeds back to the vault.", consoleUrl };
   if (r.includes("Token not in whitelist"))
     return { explanation: "Token restriction is ON and this token is not whitelisted.", action: `Add the token to the whitelist or disable token restriction at: ${consoleUrl}`, consoleUrl };
   if (r.includes("Exceeds per-tx limit"))
@@ -1228,13 +1232,18 @@ function ensureRecipientSafe(result) {
   }
 }
 async function validateActionsOrThrow(policyClient, tokenId, actions) {
-  for (const action of actions) {
+  for (const [index, action] of actions.entries()) {
     const sim = await policyClient.validate(tokenId, action);
     if (!sim.ok) {
       throw new SkillError(
         "POLICY_REJECTED",
         "Policy rejected transaction",
         {
+          failedActionIndex: index,
+          failedActionTarget: action.target,
+          failedActionSelector: getActionSelector(action),
+          failedActionFunction: getActionFunctionName(action),
+          failedActionValue: action.value.toString(),
           reason: sim.reason,
           ...policyRejectionHelp(sim.reason, tokenId.toString())
         },
@@ -1254,6 +1263,37 @@ async function executeActions(policyClient, tokenId, actions) {
   const result = await policyClient.executeBatch(tokenId, actions, true);
   return { hash: result.hash };
 }
+function getActionSelector(action) {
+  return action.data && action.data.length >= 10 ? action.data.slice(0, 10) : "0x";
+}
+function getActionFunctionName(action) {
+  if (action.data === "0x") {
+    return "nativeTransfer";
+  }
+  const candidates = [
+    ERC20_ABI,
+    WBNB_ABI,
+    VTOKEN_ABI,
+    VBNB_MINT_ABI,
+    SWAP_EXACT_ETH_ABI,
+    SWAP_EXACT_TOKENS_ABI,
+    SWAP_EXACT_TOKENS_FOR_ETH_ABI,
+    SWAP_EXACT_ETH_FOR_TOKENS_FEE_ABI,
+    SWAP_EXACT_TOKENS_FOR_TOKENS_FEE_ABI,
+    SWAP_EXACT_TOKENS_FOR_ETH_FEE_ABI,
+    V3_EXACT_INPUT_SINGLE_ABI,
+    V3_EXACT_INPUT_ABI,
+    FOUR_MEME_V1_ABI,
+    FOUR_MEME_V2_ABI
+  ];
+  for (const abi of candidates) {
+    const decoded = tryDecodeCalldata(abi, action.data);
+    if (decoded?.functionName) {
+      return decoded.functionName;
+    }
+  }
+  return "unknown";
+}
 
 // src/services/info.ts
 var ZERO_ADDRESS = "0x0000000000000000000000000000000000000000";
@@ -1692,25 +1732,6 @@ async function executeSwap(input) {
           args: [minOut, path, vault, deadline]
         })
       });
-    } else if (isNativeOut) {
-      actions.push({
-        target: tokenIn.address,
-        value: 0n,
-        data: encodeFunctionData2({
-          abi: ERC20_ABI,
-          functionName: "approve",
-          args: [v2Router, amountIn]
-        })
-      });
-      actions.push({
-        target: v2Router,
-        value: 0n,
-        data: encodeFunctionData2({
-          abi: SWAP_EXACT_TOKENS_FOR_ETH_ABI,
-          functionName: "swapExactTokensForETH",
-          args: [amountIn, minOut, path, vault, deadline]
-        })
-      });
     } else {
       actions.push({
         target: tokenIn.address,
@@ -1725,8 +1746,8 @@ async function executeSwap(input) {
         target: v2Router,
         value: 0n,
         data: encodeFunctionData2({
-          abi: SWAP_EXACT_TOKENS_ABI,
-          functionName: "swapExactTokensForTokens",
+          abi: SWAP_EXACT_TOKENS_FOR_TOKENS_FEE_ABI,
+          functionName: "swapExactTokensForTokensSupportingFeeOnTransferTokens",
           args: [amountIn, minOut, path, vault, deadline]
         })
       });

package/dist/index.js

@@ -979,10 +979,14 @@ function checkActionRecipientSafety(action, vault) {
 function policyRejectionHelp(reason, tokenId) {
   const consoleUrl = agentConsoleUrl(tokenId);
   const r = reason ?? "";
+  if (r.includes("Function not allowed"))
+    return { explanation: "The contract method selector is not enabled by the active DeFi policy configuration.", action: "Enable the required protocol pack or selector in Console > Safety, or use a supported transaction path.", consoleUrl };
   if (r.includes("Approve spender not allowed"))
     return { explanation: "The DEX router address is not in the approved spender whitelist.", action: "Contact the Agent owner to approve this router, or use a different DEX.", consoleUrl };
   if (r.includes("Target not allowed"))
     return { explanation: "The contract address is not in the DeFi target whitelist.", action: "Enable the corresponding DeFi Pack in Console > Safety, or contact the Agent owner.", consoleUrl };
+  if (r.includes("Receiver must be vault"))
+    return { explanation: "Swap output recipient must be the agent vault address enforced by ReceiverGuard.", action: "Retry with recipient/to set to the vault address, or use a supported flow that routes proceeds back to the vault.", consoleUrl };
   if (r.includes("Token not in whitelist"))
     return { explanation: "Token restriction is ON and this token is not whitelisted.", action: `Add the token to the whitelist or disable token restriction at: ${consoleUrl}`, consoleUrl };
   if (r.includes("Exceeds per-tx limit"))
@@ -1249,13 +1253,18 @@ function ensureRecipientSafe(result) {
   }
 }
 async function validateActionsOrThrow(policyClient, tokenId, actions) {
-  for (const action of actions) {
+  for (const [index, action] of actions.entries()) {
     const sim = await policyClient.validate(tokenId, action);
     if (!sim.ok) {
       throw new SkillError(
         "POLICY_REJECTED",
         "Policy rejected transaction",
         {
+          failedActionIndex: index,
+          failedActionTarget: action.target,
+          failedActionSelector: getActionSelector(action),
+          failedActionFunction: getActionFunctionName(action),
+          failedActionValue: action.value.toString(),
           reason: sim.reason,
           ...policyRejectionHelp(sim.reason, tokenId.toString())
         },
@@ -1275,6 +1284,37 @@ async function executeActions(policyClient, tokenId, actions) {
   const result = await policyClient.executeBatch(tokenId, actions, true);
   return { hash: result.hash };
 }
+function getActionSelector(action) {
+  return action.data && action.data.length >= 10 ? action.data.slice(0, 10) : "0x";
+}
+function getActionFunctionName(action) {
+  if (action.data === "0x") {
+    return "nativeTransfer";
+  }
+  const candidates = [
+    ERC20_ABI,
+    WBNB_ABI,
+    VTOKEN_ABI,
+    VBNB_MINT_ABI,
+    SWAP_EXACT_ETH_ABI,
+    SWAP_EXACT_TOKENS_ABI,
+    SWAP_EXACT_TOKENS_FOR_ETH_ABI,
+    SWAP_EXACT_ETH_FOR_TOKENS_FEE_ABI,
+    SWAP_EXACT_TOKENS_FOR_TOKENS_FEE_ABI,
+    SWAP_EXACT_TOKENS_FOR_ETH_FEE_ABI,
+    V3_EXACT_INPUT_SINGLE_ABI,
+    V3_EXACT_INPUT_ABI,
+    FOUR_MEME_V1_ABI,
+    FOUR_MEME_V2_ABI
+  ];
+  for (const abi of candidates) {
+    const decoded = tryDecodeCalldata(abi, action.data);
+    if (decoded?.functionName) {
+      return decoded.functionName;
+    }
+  }
+  return "unknown";
+}
 
 // src/services/info.ts
 var import_viem5 = require("viem");
@@ -1716,25 +1756,6 @@ async function executeSwap(input) {
           args: [minOut, path, vault, deadline]
         })
       });
-    } else if (isNativeOut) {
-      actions.push({
-        target: tokenIn.address,
-        value: 0n,
-        data: (0, import_viem7.encodeFunctionData)({
-          abi: ERC20_ABI,
-          functionName: "approve",
-          args: [v2Router, amountIn]
-        })
-      });
-      actions.push({
-        target: v2Router,
-        value: 0n,
-        data: (0, import_viem7.encodeFunctionData)({
-          abi: SWAP_EXACT_TOKENS_FOR_ETH_ABI,
-          functionName: "swapExactTokensForETH",
-          args: [amountIn, minOut, path, vault, deadline]
-        })
-      });
     } else {
       actions.push({
         target: tokenIn.address,
@@ -1749,8 +1770,8 @@ async function executeSwap(input) {
         target: v2Router,
         value: 0n,
         data: (0, import_viem7.encodeFunctionData)({
-          abi: SWAP_EXACT_TOKENS_ABI,
-          functionName: "swapExactTokensForTokens",
+          abi: SWAP_EXACT_TOKENS_FOR_TOKENS_FEE_ABI,
+          functionName: "swapExactTokensForTokensSupportingFeeOnTransferTokens",
           args: [amountIn, minOut, path, vault, deadline]
         })
       });

package/dist/index.mjs

@@ -27,7 +27,7 @@ import {
   transferFromVault,
   unwrapWbnb,
   wrapBnb
-} from "./chunk-MPUPO5EJ.mjs";
+} from "./chunk-TH3ENDDM.mjs";
 
 // src/index.ts
 import { Command as Command9 } from "commander";

package/dist/mcp.js

@@ -978,10 +978,14 @@ function checkActionRecipientSafety(action, vault) {
 function policyRejectionHelp(reason, tokenId) {
   const consoleUrl = agentConsoleUrl(tokenId);
   const r = reason ?? "";
+  if (r.includes("Function not allowed"))
+    return { explanation: "The contract method selector is not enabled by the active DeFi policy configuration.", action: "Enable the required protocol pack or selector in Console > Safety, or use a supported transaction path.", consoleUrl };
   if (r.includes("Approve spender not allowed"))
     return { explanation: "The DEX router address is not in the approved spender whitelist.", action: "Contact the Agent owner to approve this router, or use a different DEX.", consoleUrl };
   if (r.includes("Target not allowed"))
     return { explanation: "The contract address is not in the DeFi target whitelist.", action: "Enable the corresponding DeFi Pack in Console > Safety, or contact the Agent owner.", consoleUrl };
+  if (r.includes("Receiver must be vault"))
+    return { explanation: "Swap output recipient must be the agent vault address enforced by ReceiverGuard.", action: "Retry with recipient/to set to the vault address, or use a supported flow that routes proceeds back to the vault.", consoleUrl };
   if (r.includes("Token not in whitelist"))
     return { explanation: "Token restriction is ON and this token is not whitelisted.", action: `Add the token to the whitelist or disable token restriction at: ${consoleUrl}`, consoleUrl };
   if (r.includes("Exceeds per-tx limit"))
@@ -1229,13 +1233,18 @@ function ensureRecipientSafe(result) {
   }
 }
 async function validateActionsOrThrow(policyClient, tokenId, actions) {
-  for (const action of actions) {
+  for (const [index, action] of actions.entries()) {
     const sim = await policyClient.validate(tokenId, action);
     if (!sim.ok) {
       throw new SkillError(
         "POLICY_REJECTED",
         "Policy rejected transaction",
         {
+          failedActionIndex: index,
+          failedActionTarget: action.target,
+          failedActionSelector: getActionSelector(action),
+          failedActionFunction: getActionFunctionName(action),
+          failedActionValue: action.value.toString(),
           reason: sim.reason,
           ...policyRejectionHelp(sim.reason, tokenId.toString())
         },
@@ -1255,6 +1264,37 @@ async function executeActions(policyClient, tokenId, actions) {
   const result = await policyClient.executeBatch(tokenId, actions, true);
   return { hash: result.hash };
 }
+function getActionSelector(action) {
+  return action.data && action.data.length >= 10 ? action.data.slice(0, 10) : "0x";
+}
+function getActionFunctionName(action) {
+  if (action.data === "0x") {
+    return "nativeTransfer";
+  }
+  const candidates = [
+    ERC20_ABI,
+    WBNB_ABI,
+    VTOKEN_ABI,
+    VBNB_MINT_ABI,
+    SWAP_EXACT_ETH_ABI,
+    SWAP_EXACT_TOKENS_ABI,
+    SWAP_EXACT_TOKENS_FOR_ETH_ABI,
+    SWAP_EXACT_ETH_FOR_TOKENS_FEE_ABI,
+    SWAP_EXACT_TOKENS_FOR_TOKENS_FEE_ABI,
+    SWAP_EXACT_TOKENS_FOR_ETH_FEE_ABI,
+    V3_EXACT_INPUT_SINGLE_ABI,
+    V3_EXACT_INPUT_ABI,
+    FOUR_MEME_V1_ABI,
+    FOUR_MEME_V2_ABI
+  ];
+  for (const abi of candidates) {
+    const decoded = tryDecodeCalldata(abi, action.data);
+    if (decoded?.functionName) {
+      return decoded.functionName;
+    }
+  }
+  return "unknown";
+}
 
 // src/services/info.ts
 var import_viem5 = require("viem");
@@ -1696,25 +1736,6 @@ async function executeSwap(input) {
           args: [minOut, path, vault, deadline]
         })
       });
-    } else if (isNativeOut) {
-      actions.push({
-        target: tokenIn.address,
-        value: 0n,
-        data: (0, import_viem7.encodeFunctionData)({
-          abi: ERC20_ABI,
-          functionName: "approve",
-          args: [v2Router, amountIn]
-        })
-      });
-      actions.push({
-        target: v2Router,
-        value: 0n,
-        data: (0, import_viem7.encodeFunctionData)({
-          abi: SWAP_EXACT_TOKENS_FOR_ETH_ABI,
-          functionName: "swapExactTokensForETH",
-          args: [amountIn, minOut, path, vault, deadline]
-        })
-      });
     } else {
       actions.push({
         target: tokenIn.address,
@@ -1729,8 +1750,8 @@ async function executeSwap(input) {
         target: v2Router,
         value: 0n,
         data: (0, import_viem7.encodeFunctionData)({
-          abi: SWAP_EXACT_TOKENS_ABI,
-          functionName: "swapExactTokensForTokens",
+          abi: SWAP_EXACT_TOKENS_FOR_TOKENS_FEE_ABI,
+          functionName: "swapExactTokensForTokensSupportingFeeOnTransferTokens",
           args: [amountIn, minOut, path, vault, deadline]
         })
       });

package/dist/mcp.mjs

@@ -26,7 +26,7 @@ import {
   transferFromVault,
   unwrapWbnb,
   wrapBnb
-} from "./chunk-MPUPO5EJ.mjs";
+} from "./chunk-TH3ENDDM.mjs";
 
 // src/mcp.ts
 import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "shll-skills",
-    "version": "6.0.2",
+    "version": "6.0.3",
     "description": "SHLL DeFi Agent - CLI + MCP Server for BSC",
     "main": "dist/index.js",
     "bin": {
@@ -35,4 +35,4 @@
         "SKILL.md"
     ],
     "license": "MIT"
-}
+}