Skip to content

docs(skills): add the upstream EIP adoption skill - #50

Merged
pdobacz merged 3 commits into
forks/monad_ninefrom
eip-skill
Aug 27, 2026
Merged

docs(skills): add the upstream EIP adoption skill#50
pdobacz merged 3 commits into
forks/monad_ninefrom
eip-skill

Conversation

@pdobacz

@pdobacz pdobacz commented Aug 25, 2026

Copy link
Copy Markdown
Collaborator

Capture the process for adopting one upstream EIP into a WIP Monad fork and releasing its fixtures.

Tested with 7976 and 2780 (because why not), seemed to work rather ok

Greptile Summary

The PR adds a reusable workflow for adopting an upstream EIP into a work-in-progress Monad fork and releasing its fixtures.

  • Documents EIP scoping, branch setup, mixin-based fork construction, and explicit fork succession through follows().
  • Covers spec porting, test selection and adaptation, fixture validation and release, and merge-time cleanup.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
.claude/commands/adopt-upstream-eip.md Adds the end-to-end EIP adoption guide and replaces the unsupported class keyword with a classmethod-based succession mechanism, including instructions to add the supporting trait when the target branch lacks it.

Reviews (2): Last reviewed commit: "docs(skills): declare fork succession th..." | Re-trigger Greptile

Capture the process for adopting one upstream EIP into a WIP Monad fork and releasing its fixtures.

Co-Authored-By: Claude <claude-opus-5>
@pdobacz
pdobacz requested review from QEDK and mijovic as code owners August 25, 2026 14:40
A bundle declares `monad_eip<a>+<b>`, so the rehearsal can deduce it.

Co-Authored-By: Claude <claude-opus-5>
Comment thread .claude/commands/adopt-upstream-eip.md
@pdobacz
pdobacz merged commit daf910c into forks/monad_nine Aug 27, 2026
4 checks passed
@pdobacz
pdobacz deleted the eip-skill branch August 27, 2026 15:34
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