Skip to content

ovm skill#192

Merged
HananINouman merged 8 commits intomainfrom
Hanan/ovm-skills
Feb 20, 2026
Merged

ovm skill#192
HananINouman merged 8 commits intomainfrom
Hanan/ovm-skills

Conversation

@HananINouman
Copy link
Contributor

@HananINouman HananINouman commented Feb 17, 2026

Summary

Details

i did a couple of write operations with claude code and tested with cursor opus model

Screenshot 2026-02-19 at 5 06 31 PM

How to test it

ticket: #000

@HananINouman HananINouman changed the title ovm mcp server ovm skill Feb 19, 2026
Copy link
Contributor

@pinebit pinebit left a comment

Choose a reason for hiding this comment

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

I like this, but now we have two sets of pretty much identical scripts: the forge's one (./script/ovm/*.s.sol) and these new shell scripts. It would be ideal to have just one set, either now or eventually.
Also, I think that these scripts might be useful outside of Claude's ecosystem, so power users could reuse them in CLI or higher level scripting. People who don't use Claude, may not guess these scripts exist at all. Moving them to ./script folder would be reasonable, but this would lead us to the above statement of having two sets of identical scripts... I am fine to remove forge's script, but then we will need to add the missing one - "deploy ovm factory".

@HananINouman
Copy link
Contributor Author

I like this, but now we have two sets of pretty much identical scripts: the forge's one (./script/ovm/*.s.sol) and these new shell scripts. It would be ideal to have just one set, either now or eventually. Also, I think that these scripts might be useful outside of Claude's ecosystem, so power users could reuse them in CLI or higher level scripting. People who don't use Claude, may not guess these scripts exist at all. Moving them to ./script folder would be reasonable, but this would lead us to the above statement of having two sets of identical scripts... I am fine to remove forge's script, but then we will need to add the missing one - "deploy ovm factory".

i think the goal eventually is to move this obol-ovm skill folder to the obolnetowrks/skills repo and to be used outside of the obol-splits, this is why i added all needed scripts in obol-ovm instead of using .sol scripts,  and I didnt add deploy-factory because we only want ovm capabilities,
we dont want users to deploy factories

@HananINouman HananINouman merged commit f4b0b74 into main Feb 20, 2026
4 of 5 checks passed
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