BC collect-fees
Check and collect LP fees from PancakeSwap V3 and Infinity (v4) positions. Use when user says "collect my fees", "claim LP fees", "how much fees have I earned", "pending fees", "uncollected fees", "/collect-fees", "harvest LP fees", or asks about fees from a specific token pair position.
As a process C 63/100 · Has gaps — weak spots: inputs and preconditions, consistency, running it twice
What is at stake
Medium-severity findings: the skill is probably honest, but read what alarmed the scanner.
Below is the worst case for this category. The finding here is medium: the guard saw a sign, not a proof.
The skill asks for more than the task needs: broad tool access, credential environment variables, binaries. Every extra permission widens the damage from a mistake or a compromise.
Narrow allowed-tools and the variable list to the minimum; replace binaries with readable sources or scripts.
How to improve
- The SKILL.md body is over 5,000 tokens: move reference detail into references/ and load it when needed.
- Your own cases (evals/evals.json, 4–6 real requests with expected answers): the full check would then run those instead of a model-drafted suite.
- A spec.yaml with trigger phrases and assertions — a behaviour contract for CI; `skilltest init` writes a template.
Guard findings · 12
✓ No critical or high findings
Medium and low: 12
-
medium Broad scope
meta-broad-allowed-toolsSKILL.md:1Broad tool permissions pre-approved: Bash(curl:*)allowed-tools: Read Glob Grep Bash(curl:*) Bash(node:*) Bash(npm:*) Bash(xdg-open:*) Bash(open:*) WebFetch AskUserQuestion
-
low Secrets in code
secret-high-entropy-tokenreferences/fetch-solana.cjs:25High-entropy token-like string (may be an id, hash or a credential) (quoted — discussed, not commanded)const PANCAKE_CLMM_PROGRAM_ID = new PublicKey('HpNf…xFq')quoted -
low Secrets in code
secret-high-entropy-tokenreferences/fetch-v3-positions.mjs:30High-entropy token-like string (may be an id, hash or a credential) (quoted — discussed, not commanded)1: '0x55…59e',
quoted -
low Secrets in code
secret-high-entropy-tokenreferences/fetch-v3-positions.mjs:31High-entropy token-like string (may be an id, hash or a credential) (quoted — discussed, not commanded)56: '0x55…59e',
quoted -
low Secrets in code
secret-high-entropy-tokenreferences/fetch-v3-positions.mjs:32High-entropy token-like string (may be an id, hash or a credential) (quoted — discussed, not commanded)324: '0x4c…463',
quoted -
low Secrets in code
secret-high-entropy-tokenreferences/fetch-v3-positions.mjs:33High-entropy token-like string (may be an id, hash or a credential) (quoted — discussed, not commanded)42161: '0x5e…694',
quoted -
low Secrets in code
secret-high-entropy-tokenreferences/fetch-v3-positions.mjs:34High-entropy token-like string (may be an id, hash or a credential) (quoted — discussed, not commanded)59144: '0x22…c57',
quoted -
low Secrets in code
secret-high-entropy-tokenSKILL.md:127High-entropy token-like string (may be an id, hash or a credential) (detector / deny-list definition; documentation table row)| BNB Smart Chain | 56 | `bsc` | `https://bsc-…org` | `0x46…364` |
detectortable -
low Secrets in code
secret-high-entropy-tokenSKILL.md:128High-entropy token-like string (may be an id, hash or a credential) (detector / deny-list definition; documentation table row)| Ethereum | 1 | `eth` | `https://eth.llamarpc.com` | `0x46…364` |
detectortable -
low Secrets in code
secret-high-entropy-tokenSKILL.md:129High-entropy token-like string (may be an id, hash or a credential) (documentation table row)| Arbitrum One | 42161 | `arb` | `https://arb1….io/rpc` | `0x46…364` |
table -
low Secrets in code
secret-high-entropy-tokenSKILL.md:130High-entropy token-like string (may be an id, hash or a credential) (detector / deny-list definition; documentation table row)| Base | 8453 | `base` | `https://mainnet.base.org` | `0x46…364` |
detectortable -
low Secrets in code
secret-high-entropy-tokenSKILL.md:131High-entropy token-like string (may be an id, hash or a credential) (detector / deny-list definition; documentation table row)| zkSync Era | 324 | `zksync` | `https://mainnet.era.zksync.io` | `0xa8…883` |
detectortable
Files scanned: 5. Evidence is masked. Grey chips explain why severity was lowered.
Against the Agent Skills spec
- warning
body-longSKILL.md body ≈ 5521 tokens (recommended < 5000); move details to references/ - note
frontmatter-keyunknown frontmatter key "slug" - note
frontmatter-keyunknown frontmatter key "homepage"
Process rating: all ten parameters 63/100
- 0Inputs and preconditions. Does not say what the process needs to start
- 0Progress reporting. Says nothing while it works
- 30Running it twice. 2 mutating operations with no state check
- 40Consistency. Frontmatter name (collect-fees) differs from the folder (pcs-collect-fees)
- 50When it triggers. No condition that starts the skill
- 60Result and completion. Output format stated, no completion criterion
- 70Failures and branches. 5 branches
- 70Execution cost. Instruction body is 5521 tokens
- 100Tools and files. Tools declared in frontmatter
- 100Steps. 33 steps
- medium Safety rules and hard prohibitions inside a skill: they belong in the system prompt, here they protect nothing
- low 15 top-level sections: this looks like several domains in one skill
Everything here is measured from the skill text rather than judged by a model, so the numbers are checkable. A parameter weighs more when it is a more common reason for the process to stall.
Quality signals
- +4Description does not say when NOT to use the skill (false activations)
- +2Single-language instructions
- +5Description quotes 7 example trigger phrases
- +3Description length 288: enough signal without eating the budget
- +4Structure: 29 headings
- +3Step-by-step instructions: 33 items
- +3Output format is stated explicitly
- +4Has examples (21 code blocks)
- +1License stated
Quality base 70; lint remarks subtract, signals add up to 100. Result: 81.