Skip to content

docs: add Solver7702Delegate docs#627

Open
igorroncevic wants to merge 3 commits into
mainfrom
feat/solver-7702-delegate
Open

docs: add Solver7702Delegate docs#627
igorroncevic wants to merge 3 commits into
mainfrom
feat/solver-7702-delegate

Conversation

@igorroncevic
Copy link
Copy Markdown
Contributor

@igorroncevic igorroncevic commented May 22, 2026

Description

Adds a solver-facing guide for using Solver7702Delegate to keep the existing solver EOA while using auxiliary EOAs for parallel settlement submission.

Changes

  • Add a new Solver7702Delegate tutorial under solver docs.
  • Document the direct vs delegated transaction shape and packed calldata format.
  • Cover setup, verification, monitoring, caller replacement, revocation, and compromised key handling.
  • Link the guide from solver onboarding and driver submission docs.

Summary by CodeRabbit

  • Documentation
    • Reformatted solver driver tutorial for clearer auction preprocessing steps.
    • Added guidance in onboarding and driver docs for solvers that need parallel settlement submission using Solver7702Delegate.
    • Published a comprehensive Solver7702Delegate tutorial covering setup, verification, deployment, funding of auxiliary accounts, operational procedures, and security considerations.

Review Change Stack

@vercel
Copy link
Copy Markdown

vercel Bot commented May 22, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview May 25, 2026 3:03pm

Request Review

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 22, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 89a99e9d-813c-4624-a592-dc3b4f4fcf6d

📥 Commits

Reviewing files that changed from the base of the PR and between b8dcd6e and 0f19379.

📒 Files selected for processing (2)
  • docs/cow-protocol/tutorials/arbitrate/solver/driver.md
  • docs/cow-protocol/tutorials/solvers/solver-7702-delegate.md
✅ Files skipped from review due to trivial changes (2)
  • docs/cow-protocol/tutorials/arbitrate/solver/driver.md
  • docs/cow-protocol/tutorials/solvers/solver-7702-delegate.md

📝 Walkthrough

Walkthrough

Adds a new Solver7702Delegate tutorial describing parallel settlement submission under ERC-7702 and updates solver onboarding and driver tutorials with cross-references and minor formatting changes.

Changes

Solver7702Delegate Documentation

Layer / File(s) Summary
New Solver7702Delegate tutorial page
docs/cow-protocol/tutorials/solvers/solver-7702-delegate.md
Complete new tutorial covering parallel submission model with auxiliary EOAs, calldata packing (`bytes20(target)
Cross-references and formatting updates
docs/cow-protocol/tutorials/arbitrate/solver/driver.md, docs/cow-protocol/tutorials/solvers/onboard.md
Reformatted the “Preprocessing auctions” steps to hyphen-prefixed list items and inserted a note in the driver tutorial’s Submitting settlements section linking to Solver7702Delegate; onboarding tutorial added a brief API-spec note about using Solver7702Delegate to enable parallel submission nonce lanes while preserving the solver EOA.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

I hop and write in tidy pace,
A delegate to speed the race.
Parallel lanes for settlements bright,
Solver7702 leading the night. 🐇✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately and concisely describes the primary change: adding documentation for Solver7702Delegate.
Description check ✅ Passed The description follows the repository template with clear sections for Description and Changes, including specific tasks that align with the actual changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/solver-7702-delegate

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@igorroncevic igorroncevic changed the title docs: add docs docs: add Solver7702Delegate docs May 22, 2026
@igorroncevic igorroncevic marked this pull request as ready for review May 25, 2026 09:11
@igorroncevic igorroncevic requested a review from a team as a code owner May 25, 2026 09:11
Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@docs/cow-protocol/tutorials/arbitrate/solver/driver.md`:
- Line 58: Update the wording in the sentence fragment "discarding orders that
can definitely not be settled (e.g. user is missing balances)" to improve flow
by replacing "can definitely not be settled" with "definitely cannot be settled"
or "cannot be settled" (e.g., change to "discarding orders that definitively
cannot be settled (e.g. user is missing balances)" or "discarding orders that
cannot be settled (e.g. user is missing balances)").
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 0b6db3ef-bbe4-447c-94a1-6b2c9ddc530b

📥 Commits

Reviewing files that changed from the base of the PR and between 978e255 and b8dcd6e.

📒 Files selected for processing (3)
  • docs/cow-protocol/tutorials/arbitrate/solver/driver.md
  • docs/cow-protocol/tutorials/solvers/onboard.md
  • docs/cow-protocol/tutorials/solvers/solver-7702-delegate.md

Comment thread docs/cow-protocol/tutorials/arbitrate/solver/driver.md Outdated
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.

2 participants