Skip to content

Add Makefile, document fee configuration differences, consolidate genesis source of truth#136

Draft
Copilot wants to merge 2 commits intomainfrom
copilot/add-makefile-document-fee-configurations
Draft

Add Makefile, document fee configuration differences, consolidate genesis source of truth#136
Copilot wants to merge 2 commits intomainfrom
copilot/add-makefile-document-fee-configurations

Conversation

Copy link

Copilot AI commented Mar 15, 2026

Several developer experience gaps: no task runner, undocumented fee discrepancies across environments, silent feeRecipient misconfiguration on testnet, and duplicated genesis files with no canonical source indicator.

Task Runner

  • Makefile added at repo root with make help, make test-rpc, make test-ws, make backup
  • requirements.txt added for Python test deps (requests, websockets)

Fee Configuration

  • ## Fee Configuration section added to README with per-environment table:

    Parameter Mainnet (Jade) Testnet (Snow) Devnet
    minBaseFee 100 Gwei 1 Gwei 100 Gwei
    allowFeeRecipients true true not set
  • allowFeeRecipients: true added to chains/testnet/genesis.json — previously absent, causing feeRecipient to be silently ignored and fees burned

  • Fee recipient addresses documented (mainnet 0xe49a..., testnet 0xE021...) with notes on burn behavior when flag is absent

Genesis Consolidation

  • Root genesis/*.json files (byte-identical copies of chains/testnet/) replaced with symlinks to chains/testnet/ as the canonical source, eliminating silent drift risk

README Fixes

  • "Archieve Node" → "Archive Node" (TOC + heading)
  • --whitelisted-subnets--track-subnets (deprecated flag, two occurrences)
Original prompt

This section details on the original issue you should resolve

<issue_title>[Feature][Medium] Add Makefile, document fee configuration differences, consolidate genesis source of truth</issue_title>
<issue_description>## Summary

Multiple medium-priority improvements for developer experience and configuration clarity:

1. No Makefile or task runner exists

The repository has no unified task runner. Developers must manually navigate directories and execute scripts individually. No requirements.txt for Python test dependencies (requests, websockets).

Fix: Add a Makefile with targets like make test-rpc, make test-ws, make backup, make help. Add requirements.txt for Python dependencies.

2. Mainnet minBaseFee is 100x higher than testnet (undocumented)

  • chains/mainnet/genesis.json: "minBaseFee": 100000000000 (100 Gwei)
  • chains/testnet/genesis.json: "minBaseFee": 1000000000 (1 Gwei)
  • chains/devnet/genesis.json: "minBaseFee": 100000000000 (100 Gwei, matches mainnet)

Developers testing on testnet cannot predict mainnet gas costs. Devnet matches mainnet rather than testnet, despite being a development environment.

Fix: Document fee differences. Consider aligning devnet with testnet for progressive testing.

3. allowFeeRecipients config mismatch between environments

  • chains/mainnet/genesis.json: "allowFeeRecipients": true
  • chains/testnet/genesis.json: field absent (defaults to false)

Testnet validators using the feeRecipient config have fees silently burned instead of received.

Fix: Clarify which testnet genesis variant is active. Align allowFeeRecipients for consistent testing.

4. feeRecipient addresses undocumented

  • Mainnet config: 0xe49a1220eE09Fbf0D25CA9e3BB8D5fD356Fc67FF
  • Testnet config: 0xE021c9B8DC3953f4f7f286C44a63f5fF001EF481

No documentation about address type (EOA/multisig), ownership, or fee usage policy.

Fix: Add fee recipient documentation for community transparency.

5. Genesis files duplicated without canonical source indicator

genesis/ root directory contains byte-identical copies of chains/testnet/ files. subnet-cli/subnet-cli-wizard.sh references ../genesis/ while README references chains/ paths.

Fix: Designate chains/{env}/ as canonical. Replace root genesis/ with symlinks or remove entirely.

6. README typo "Archieve Node" and deprecated --whitelisted-subnets

  • README.md line 37, 867: "Archieve" should be "Archive"
  • --whitelisted-subnets is deprecated; current flag is --track-subnets

Fix: Fix typo and update deprecated flags.

Expected Impact

  • Makefile improves developer onboarding significantly
  • Fee documentation prevents cost surprises between environments
  • Genesis consolidation prevents silent configuration drift

Generated by Health Monitor with Omni</issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

…te genesis

Co-authored-by: numbers-official <181934381+numbers-official@users.noreply.github.com>
Copilot AI changed the title [WIP] Add Makefile and document fee configuration differences Add Makefile, document fee configuration differences, consolidate genesis source of truth Mar 15, 2026
Copilot AI requested a review from numbers-official March 15, 2026 10:56
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.

[Feature][Medium] Add Makefile, document fee configuration differences, consolidate genesis source of truth

2 participants