Skip to content

Add cursor context for EVM#3013

Open
codchen wants to merge 2 commits intomainfrom
tony/knowledge
Open

Add cursor context for EVM#3013
codchen wants to merge 2 commits intomainfrom
tony/knowledge

Conversation

@codchen
Copy link
Collaborator

@codchen codchen commented Mar 4, 2026

Describe your changes and provide context

Add markdown contexts that are automatically attached whenever an agent works on relevant EVM files.

Testing performed to validate your change

n/a

@github-actions
Copy link

github-actions bot commented Mar 4, 2026

The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedMar 5, 2026, 11:02 AM

@Kbhat1
Copy link
Contributor

Kbhat1 commented Mar 4, 2026

Only question (discussing with other folks) is if we want this in a Agents.md vs cursor rules (so that it’s ai tool independent - cursor, Claude code, codex etc)

@codecov
Copy link

codecov bot commented Mar 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 58.40%. Comparing base (0fea76f) to head (26d1658).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #3013   +/-   ##
=======================================
  Coverage   58.40%   58.40%           
=======================================
  Files        2110     2110           
  Lines      175009   175009           
=======================================
  Hits       102213   102213           
  Misses      63767    63767           
  Partials     9029     9029           
Flag Coverage Δ
sei-db 70.41% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Collaborator

@masih masih left a comment

Choose a reason for hiding this comment

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

My recommendation is to:

  • Use AGENTS.md instead.
  • Have one in root of the repo.
  • Have specialised MD files in each corresponding root package. then mention them in root.

evmrpc/RPC.md Outdated
@@ -0,0 +1,21 @@
---
globs: ["evmrpc/**"]
Copy link
Collaborator

Choose a reason for hiding this comment

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

Name this file (annd EVM.md) AGENTS.md? similar to all other files in package level.

Essentially, the file name AGENTS.md in any hierarchy captures the agentic context, where the hierarchy makes it progressively specialised to the context itself.

AGENTS.md Outdated
Comment on lines +1 to +5
# EVM
For context about EVM on Sei, refer to `x/evm/EVM.md`.

# EVM RPC
For EVM RPC specs, refer to `evmrpc/RPC.md`.
Copy link
Collaborator

@masih masih Mar 5, 2026

Choose a reason for hiding this comment

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

The beauty of convention means less explicit config needed; which fits the fuzzy world of agentic interaction very well (and won't go out of date quickly if we forget to update it):

Suggested change
# EVM
For context about EVM on Sei, refer to `x/evm/EVM.md`.
# EVM RPC
For EVM RPC specs, refer to `evmrpc/RPC.md`.
When working within a specific package, always check for and read any AGENTS.md file in that package directory before making changes. These contain domain-specific architecture decisions, conventions, and constraints that supplement this top-level guide.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants