Skip to content

docs: drop the --no-match-contract Chain selection nobody runs - #38

Merged
thedavidmeister merged 1 commit into
mainfrom
2026-08-14-drop-chain-selection-docs
Aug 14, 2026
Merged

docs: drop the --no-match-contract Chain selection nobody runs#38
thedavidmeister merged 1 commit into
mainfrom
2026-08-14-drop-chain-selection-docs

Conversation

@thedavidmeister

Copy link
Copy Markdown
Contributor

forge test --no-match-contract Chain was documented in three places as a way to run the suite without endpoints. Nothing invokes it.

  • no CI step, no script, no flake package, no Makefile
  • rainix CI runs forge test -vvv, the whole suite, forks and all
  • it appears in my own session history only in the two sessions spent building and checking it

Documenting a capability with no caller creates work to keep it true — which is what #34 turned into: an 800-line test split serving a command nobody runs.

What is removed

Three references. Nothing else.

site was
CLAUDE.md RPC section "and the snapshot contracts run regardless: forge test --no-match-contract Chain"
CLAUDE.md group 4 "forge test --no-match-contract Chain is the whole snapshot gate"
AddressRegistryDeployChain.t.sol natspec "forge test --no-match-contract Chain still runs every snapshot assertion"

What survives

The design each site was attached to is real and stays stated: group 4 lives in its own contract so an unreachable endpoint fails there and not on the snapshot assertions. That separation is a property of the contract split, not of any command.

QA

  • Discriminating tests: n/a — comment and markdown only, no executable line changes and no behaviour to assert. The differential check is git grep -- 'no-match-contract', which returns three hits on main and nothing on this branch; it fails on base by construction.
  • Mutations applied: n/a — the diff contains no executable line to mutate. forge fmt --check exits 0, which is what establishes the edited natspec still parses.
  • Oracle: the absence of callers, established independently of this change — grepping the repo for the string across yaml/yml/toml/sh/nix/Makefile returns only self-referential prose, and rainlanguage/rainix/.github/workflows/rainix-sol-test.yaml runs forge test -vvv with no contract filter.
  • Category check: the ask was to remove this part of the docs. All three sites are removed, not only the one that prompted it. The contract separation the sites described is deliberately kept, because it is a property of the code rather than of the command.

Closes nothing. #32 stays open for the RPC failures, which this does not address.

🤖 Generated with Claude Code

Three sites documented a command with no caller: no CI step, no script, no
flake package invokes it, and CI runs the whole suite. Documenting a capability
nobody uses invites work to keep it true.

What the sites were attached to survives: group 4 is still its own contract so
an endpoint failure lands there and not on the snapshot assertions.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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