Skip to content

feat(tooling): add consume-hive skill - #3612

Merged
danceratopz merged 6 commits into
ethereum:forks/amsterdamfrom
danceratopz:hive-sim-skill
Oct 1, 2026
Merged

danceratopz merged 6 commits into
ethereum:forks/amsterdamfrom
danceratopz:hive-sim-skill

Conversation

@danceratopz

@danceratopz danceratopz commented Sep 17, 2026 •

Copy link
Copy Markdown
Member

Description

Add /consume-hive to run local fixtures against fresh client images for a specified network using one selected simulator: engine, enginex, rlp, or sync. The skill resolves client YAMLs from hive-tests, preserves the Nimbus exception-matching exemption, and includes the Claude symlink.

execute hive is intentionally left out for a separate skill.

Related Issues or PRs

Checklist

  • Ran fast static checks to avoid CI fails, see Code Standards & Verifying Changes: just static
  • PR title has the form <type>(<area>): <title>, where <type> and <area> come from an appropriate C-<type>, respectively A-<area>, label. The title should match the target squash commit message.

Cute Animal Picture

image

Photo by Scottmliddell, CC BY 2.0 UK.

@danceratopz danceratopz added C-feat Category: an improvement or new feature A-tooling Area: Improvements or changes to auxiliary tooling such as uv, ruff, mypy, ... labels Sep 17, 2026
@codecov

codecov Bot commented Sep 17, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.62%. Comparing base (1c8b05b) to head (7c95d25).
⚠️ Report is 41 commits behind head on forks/amsterdam.

Additional details and impacted files
@@                 Coverage Diff                 @@
##           forks/amsterdam    #3612      +/-   ##
===================================================
+ Coverage            94.44%   94.62%   +0.18%     
===================================================
  Files                  624      624              
  Lines                36928    36976      +48     
  Branches              3326     3349      +23     
===================================================
+ Hits                 34875    34988     +113     
+ Misses                1450     1397      -53     
+ Partials               603      591      -12     
Flag Coverage Δ
unittests 94.62% <ø> (+0.18%) ⬆️

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

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@marioevz marioevz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, approving now. I just left one comment I did not verify whether the current PR already does this or not, might be worth to check. Thanks!


```bash
./hive --dev --dev.addr "127.0.0.1:$HIVE_PORT" \
--client-file "$CLIENT_YAML" \

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

If we could add how to build these YAML files to this skill I think it would be very valuable. Earlier today I instructed Claude to build one with all the sepolia images of all clients, and it did well, so it should not be too difficult. Maybe the skill already does this but worth verifying?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Ah, thanks! You specify a network and it tries to grab the network config client.yaml file from the hive-tests repo: https://github.com/ethpandaops/hive-tests/tree/master/.github/configs/hive

This was a bit too specific/limiting - I just updated to enable the agent to create its own client.yaml if need be!

Let's see how it behaves, we should tweak if need be!

@danceratopz
danceratopz merged commit 8667dac into ethereum:forks/amsterdam Oct 1, 2026
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-tooling Area: Improvements or changes to auxiliary tooling such as uv, ruff, mypy, ... C-feat Category: an improvement or new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants