Skip to content

Remove obsolete historical scripts and artifacts - #2997

Open
clement-ux wants to merge 1 commit into
masterfrom
chore/remove-obsolete-historical-artifacts
Open

Remove obsolete historical scripts and artifacts#2997
clement-ux wants to merge 1 commit into
masterfrom
chore/remove-obsolete-historical-artifacts

Conversation

@clement-ux

@clement-ux clement-ux commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

Plain-English summary

This repository still contains many files from workflows that are no longer used: old staking and liquidity scripts, generated diagrams, unused ABIs, copied libraries, and empty placeholders.

This PR removes only that historical baggage. It does not replace Hardhat or change how active commands and local forks run. Keeping this cleanup separate makes the three functional migration steps much easier to review.

Context

Foundry already handles contract compilation, testing, deployment, and ABI generation. Hardhat remains temporarily for two active jobs:

  1. running operational commands;
  2. running the local/forked node.

The next three PRs replace those jobs one at a time, then remove Hardhat.

What changes in this PR

  • remove retired Resolution, staking, compensation, and liquidity scripts;
  • remove obsolete Hardhat-only ABIs and the historical localhost.json snapshot;
  • remove orphaned diagrams and their generator entries;
  • remove unused Rooster library copies and stale analysis configuration;
  • remove empty test/deployment placeholders and dead utility modules;
  • remove documentation and package-script references to those deleted files.

What does not change yet

  • active operational commands still run through Hardhat;
  • Hardhat still provides the local/forked node;
  • no runtime replacement happens in this layer.

Stack and merge order

Base: master

  1. Remove obsolete historical scripts and artifacts #2997 — remove obsolete historical scripts and artifacts (this PR)
  2. Introduce standalone ops CLI #2998 — introduce standalone ops CLI
  3. Migrate local forks to Anvil #2999 — migrate local forks to Anvil
  4. Remove the Hardhat runtime #3000 — remove the Hardhat runtime

Merge in cascade: #3000#2999#2998#2997master.

What to review

The important question is not the implementation of a new runtime; there is none here. Please check that every deleted category is genuinely retired and that no surviving file still points to a deleted path.

This layer changes 80 files: 24,782 deletions and 8 small documentation/configuration additions.

Validation

@clement-ux
clement-ux force-pushed the chore/remove-obsolete-historical-artifacts branch from 54f509b to 567bcc6 Compare September 2, 2026 07:57

@sparrowDom sparrowDom 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.

Some comments inline

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.

With this .gitkeep gone, tests/invariant/ (and likewise tests/unit/{bridges,harvest,oracle}/) contain no files and stop existing in git, but CLAUDE.md:214 still lists invariant/ in the test-tree diagram. Either drop that line or keep this one .gitkeep.

@@ -1,26 +0,0 @@
# multi-abi: true

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.

Two echidna leftovers survive this deletion: .gitignore:75 still has contracts/echidna-corpus, and contracts/slither.config.json:2 filter_paths still names echidna|crytic (and solidity-bytes-utils, which #2998 removes). There is no contract Echidna anywhere in the repo, so both can go.

@@ -1,289 +0,0 @@
/* Reimbursements csv: https://docs.google.com/spreadsheets/d/1lCzDmmLV73rwRSnAKVGdsUkhuelxBvH4fxYNAUAd69U/edit?usp=sharing

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.

contracts/.prettierignore:1 still ignores scripts/compensation/forkDeployment.js — a pre-existing dead path, but it belongs to the compensation family removed here. Fine to fold into #2998, which touches .prettierignore anyway.

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