mirror of
https://github.com/austintgriffith/ethskills.git
synced 2026-09-14 20:07:04 +08:00
1128ed80c2
* qa: mark page-level CTA network check as explicit FAIL Strengthens the wrong-network rule in qa/SKILL.md so auditors don't pass an app just because SE-2's header shows a WrongNetworkDropdown. The primary onchain CTA (Sign/Stake/Deposit/etc.) must itself become a "Switch to [Chain]" button when the connected wallet is on the wrong chain — otherwise the user clicks through and eats a silent wagmi error. - Adds explicit FAIL: main CTA rendered on wrong network - Adds PASS criterion: CTA slot branches on chainId === targetNetwork.id - Adds grep snippet auditors can run to catch missing chainId checks - Tightens Ship-Blocking checklist to require switch button in the primary CTA slot, not only the header dropdown Caught this on a SE-2 build whose audit passed three cycles while the main Sign button rendered unconditionally — user on wrong chain got a silent failure. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * qa: drop prescriptive grep snippet — rule lines carry the check Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * docs: point sub-skills that mention Uniswap/Aave to addresses/SKILL.md Bots landing on testing/, security/, orchestration/, or frontend-playbook/ via web search had no breadcrumb to the verified-addresses skill. --------- Co-authored-by: clawdbotatg <clawdbotatg@users.noreply.github.com> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>