Skip to content

docs: add CLAUDE.md with commands, architecture, and conventions#554

Open
louise-poole wants to merge 1 commit intomainfrom
lp/claude_md
Open

docs: add CLAUDE.md with commands, architecture, and conventions#554
louise-poole wants to merge 1 commit intomainfrom
lp/claude_md

Conversation

@louise-poole
Copy link
Copy Markdown
Contributor

No description provided.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-project-automation github-project-automation bot moved this from Todo to In Progress in Tycho Mar 11, 2026
Comment on lines +30 to +39
### Key Modules (`src/`)
- **`protocol/`**: Core traits and models (`ProtocolComponent`, `Update`)
- **`evm/simulation.rs`**: `SimulationEngine` — runs EVM transactions via `revm`
- **`evm/engine_db/`**: Database backends (in-memory, RPC-backed)
- **`evm/stream.rs`**: Tycho feed integration — decodes live state into protocol instances
- **`evm/protocol/`**: Protocol implementations
- **Native**: Pure Rust math (`uniswap_v2`, `uniswap_v3`, `uniswap_v4`, `ekubo`, `cowamm`, `fluid`, etc.)
- **VM** (`vm/`): Generic Solidity adapter executed in `revm` for protocols without native impls
- Shared: `cpmm.rs`, `clmm.rs`, `safe_math.rs`, `u256_num.rs`
- **`rfq/`**: RFQ client for off-chain market makers (Bebop, Hashflow)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Happy to start like this but I think it's not optimal, we should probably do like Turbine did; trim it to the strict minimum and point to other files (e.g they used ./claude.CODEBASE.md` for the modules descriptions

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Do you know if they felt it had much of an impact on their Claude sessions? This felt very compact already.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yes I think it helped a lot to keep the context small

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

3 participants