Skip to content

tools: rewrite spec build in go#751

Open
jshufro wants to merge 4 commits intoethereum:mainfrom
jshufro:jms/go-rewrite
Open

tools: rewrite spec build in go#751
jshufro wants to merge 4 commits intoethereum:mainfrom
jshufro:jms/go-rewrite

Conversation

@jshufro
Copy link
Contributor

@jshufro jshufro commented Feb 9, 2026

The four commits present are each fairly atomic.

1a0daa3 moves rpctestgen into tools/cmd/rpctestgen and updates CI accordingly
0c09476 runs gofmt to fix a lint error
ced5a4d introduces a new tool in tools/cmd/specgen that generates the refs-openrpc.json file
7578428 updates the build script + ci to use the new tool, and deletes the superceded spec build logic from build.js

I used jd to ensure there's no functional diff between openrpc.json as generated by main and the new form:

$ jd -set openrpc.json openrpc.json.bak
$ echo $?
0

Somewhat annoyingly, the new files are in a completely different order, since it isn't feasible to roundtrip raw json in go in a way that retains the order. If we need deterministic output, we can sort the output in build.js

@jshufro jshufro force-pushed the jms/go-rewrite branch 2 times, most recently from 9557247 to 1aa5170 Compare February 10, 2026 21:32
@jshufro jshufro marked this pull request as ready for review February 10, 2026 21:34
@fjl fjl changed the title Rewrite build in go tools: rewrite spec build in go Feb 11, 2026
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.

1 participant