Skip to content

Remove protocols feature from core crate - #9

Merged
KaiCode2 merged 1 commit into
mainfrom
codex/remove-protocols
Jun 18, 2026
Merged

Remove protocols feature from core crate#9
KaiCode2 merged 1 commit into
mainfrom
codex/remove-protocols

Conversation

@KaiCode2

Copy link
Copy Markdown
Owner

Summary

  • remove the old protocols feature and in-crate AMM/protocol modules, tests, fixtures, examples, and storage-key benchmark
  • make immutable metadata token-decimals-only with cache format version 2
  • rename remaining storage purge helpers to contract terminology and update docs, CI, examples, and benchmarks around the generic crate boundary
  • add a release-surface guard test to prevent the removed protocol surface from returning

Validation

  • cargo test --test public_release_surface
  • cargo test --all-targets
  • cargo fmt --all --check
  • cargo clippy --all-targets --no-deps -- -D warnings
  • cargo test --doc
  • RUSTDOCFLAGS="-D warnings" cargo doc --no-deps
  • cargo bench --no-run
  • cargo package --locked --allow-dirty
  • cargo +1.88.0 check --lib --locked

A separate review-agent pass found no core correctness blockers. Its release-facing benchmark/wording findings were addressed before opening this PR.

Move the public surface back to a generic EVM simulation engine by removing the protocols feature, AMM metadata/injectors, V3 event adapter, protocol fixtures/tests/benches, and protocol-only examples.

Make immutable metadata token-decimals-only with a v2 cache format, rename storage purge helpers to contract terminology, and update release docs/CI/benchmarks around the new crate boundary.
@KaiCode2
KaiCode2 merged commit 0e41f7c into main Jun 18, 2026
2 checks passed
@KaiCode2
KaiCode2 deleted the codex/remove-protocols branch June 30, 2026 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant