Add simulate-before-send pattern to all skill examples

[critesjosh]

Summary

• Add .simulate() before every .send() call across all 15 TypeScript skill files (~60 instances)
• Document the pattern in CLAUDE.md as a critical section
• Without simulation, failing transactions hang for up to 600s with opaque errors; .simulate() surfaces revert reasons instantly

Transformation rules applied

Rule	Pattern	Files affected
Standard sends	Add .simulate({ from }) before .send()	All 15 files
Deploy chains	Break .deploy().send() → variable, simulate, send	8 files
Account deploys	Break getDeployMethod().send() → variable, simulate, send	7 files
Error tests	Replace .send() with .simulate() entirely	test-patterns.md
Helper functions	Add .simulate() inside before .send()	3 files
Parallel sends	Simulate all upfront, then send in parallel	transaction-patterns.md
AuthWit	Simulate without authWitnesses (only from)	authwit-frontend.md

Key constraints followed

• .simulate() only uses { from } — no fee, wait, or authWitnesses
• "Common Errors" examples in authwit-frontend.md intentionally left unmodified
• Variable naming: deployRequest for contract deploys, deployMethod for account deploys

Motivation

Since this plugin is referenced by the Aztec MCP server, these examples directly influence how AI agents write Aztec code. Ref: aztec-starter#245

Test plan

• grep -r '\.deploy(.*).send(' skills/ returns 0 results (all deploy chains broken)
• grep -r 'getDeployMethod()).send(' skills/ returns 0 results (all inline chains broken)
• No .simulate() calls include fee, wait, or authWitnesses params
• Error examples in authwit-frontend.md (lines 120-157) unchanged
• All error test patterns use .simulate() instead of .send()

🤖 Generated with Claude Code