Skip to content

Tutorial: Make your own AI trading agent#17253

Merged
wackerow merged 16 commits into
ethereum:devfrom
qbzzt:260126-agent
Feb 10, 2026
Merged

Tutorial: Make your own AI trading agent#17253
wackerow merged 16 commits into
ethereum:devfrom
qbzzt:260126-agent

Conversation

@qbzzt
Copy link
Copy Markdown
Contributor

@qbzzt qbzzt commented Feb 6, 2026

Description

A new tutorial.

Related Issue

Closes #17118

@netlify
Copy link
Copy Markdown

netlify Bot commented Feb 6, 2026

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit 04c5c9d
🔍 Latest deploy log https://app.netlify.com/projects/ethereumorg/deploys/698ba0207a9990000864788e
😎 Deploy Preview https://deploy-preview-17253.ethereum.it
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
7 paths audited
Performance: 58 (🟢 up 3 from production)
Accessibility: 94 (no change from production)
Best Practices: 100 (no change from production)
SEO: 100 (no change from production)
PWA: 59 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions Bot added the content 🖋️ This involves copy additions or edits label Feb 6, 2026
Comment thread public/content/developers/tutorials/ai-trading-agent/index.md Outdated
@mnelsonBT
Copy link
Copy Markdown
Contributor

Thanks @qbzzt ! LGTM, @wackerow are you able to review?

qbzzt and others added 4 commits February 10, 2026 09:28
Copy link
Copy Markdown
Member

@wackerow wackerow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good @qbzzt! Thanks again... Pushed a couple tiny formatting patches, fixing a few typos as well. Pulling in =)

@wackerow wackerow merged commit 9f2f9a2 into ethereum:dev Feb 10, 2026
5 of 6 checks passed
@wackerow
Copy link
Copy Markdown
Member

@qbzzt Only thing the LLM review flagged btw was the following:

Slippage protection: Both SELL_PARAMS (line 873) and make_buy_params (line 886) set amountOutMinimum: 0. Combined with deadline: 2**256 - 1, this means zero slippage protection and no transaction expiry. The author clearly knows this is demo code, but since the "To actually use it" section (lines 779-783) walks readers toward real usage, a brief warning about sandwich attacks / MEV when using amountOutMinimum: 0 on mainnet could save someone real money. Even just a one-liner in the "Possible improvements" section would do.

Didn't seem critical, but wanted to note. If you agree, feel free to open a patch for it and ping me, otherwise feel free to ignore. Thanks!

@qbzzt
Copy link
Copy Markdown
Contributor Author

qbzzt commented Feb 10, 2026

@qbzzt Only thing the LLM review flagged btw was the following:

Slippage protection: Both SELL_PARAMS (line 873) and make_buy_params (line 886) set amountOutMinimum: 0. Combined with deadline: 2**256 - 1, this means zero slippage protection and no transaction expiry. The author clearly knows this is demo code, but since the "To actually use it" section (lines 779-783) walks readers toward real usage, a brief warning about sandwich attacks / MEV when using amountOutMinimum: 0 on mainnet could save someone real money. Even just a one-liner in the "Possible improvements" section would do.

Didn't seem critical, but wanted to note. If you agree, feel free to open a patch for it and ping me, otherwise feel free to ignore. Thanks!

I agree, I should have thought about it. Can you merge #17266 ?

@pettinarip pettinarip mentioned this pull request Feb 19, 2026
@pettinarip pettinarip mentioned this pull request Feb 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

content 🖋️ This involves copy additions or edits

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tutorial proposal: Make your own AI trading agent

3 participants