Skip to content

Add cursor context for EVM#3013

Merged
masih merged 5 commits intomainfrom
tony/knowledge
Mar 12, 2026
Merged

Add cursor context for EVM#3013
masih merged 5 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 12, 2026, 1:58 PM

@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.39%. Comparing base (04faa6c) to head (dedf4cf).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3013      +/-   ##
==========================================
+ Coverage   58.26%   58.39%   +0.12%     
==========================================
  Files        2072     2109      +37     
  Lines      171414   173946    +2532     
==========================================
+ Hits        99882   101575    +1693     
- Misses      62602    63352     +750     
- Partials     8930     9019      +89     
Flag Coverage Δ
sei-db 70.41% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 308 files with indirect coverage changes

🚀 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.

@masih masih enabled auto-merge March 12, 2026 11:30
@masih masih added this pull request to the merge queue Mar 12, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Mar 12, 2026
@masih masih enabled auto-merge March 12, 2026 13:04
@masih masih added this pull request to the merge queue Mar 12, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Mar 12, 2026
@masih masih enabled auto-merge March 12, 2026 13:57
@masih masih added this pull request to the merge queue Mar 12, 2026
Merged via the queue into main with commit 73ebf8a Mar 12, 2026
52 of 54 checks passed
@masih masih deleted the tony/knowledge branch March 12, 2026 14:57
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