Skip to content
Open
Show file tree
Hide file tree
Changes from 19 commits
Commits
Show all changes
34 commits
Select commit Hold shift + click to select a range
9f3ec99
replaced branchname with latest SHA commit within github links in di…
edna-harriet Oct 23, 2025
1c4951d
modified github link with commits to use SHA commit
edna-harriet Oct 24, 2025
7fa9494
updated github commit links in files to use SHA commit
edna-harriet Oct 24, 2025
8aac441
updated files with SHA commit
edna-harriet Oct 24, 2025
143cb91
modified files with github links to use SHA commit
edna-harriet Oct 27, 2025
5e9c988
updated documents and code reference links to use SHA commit
edna-harriet Oct 27, 2025
aa1c82e
reversed the spec reference to use the tag it had before and also rem…
edna-harriet Oct 28, 2025
4fe0945
changed the SHA commit on line 836, to previous SHA commit
edna-harriet Oct 28, 2025
8d3b160
Replaced SHA commit references with their release tags
edna-harriet Oct 28, 2025
dcfde24
replaced references with SHA commit to use release tags
edna-harriet Oct 28, 2025
73e8790
removed extra comments and replaced SHA commits with release tags
edna-harriet Oct 28, 2025
590a352
removed extra comments and changed SHA commits to release tags
edna-harriet Oct 28, 2025
9a171b0
modified SHA commits to instead use release tags and removed extra co…
edna-harriet Oct 28, 2025
71ff424
added release tags in place of SHA commit
edna-harriet Oct 28, 2025
6c5c8be
replaced SHA commits with release tags
edna-harriet Oct 28, 2025
d4919ca
removed extra comment
edna-harriet Oct 29, 2025
4e2a633
modified reference to the initial SHA commit value
edna-harriet Oct 29, 2025
05b89de
removed extra comments
edna-harriet Oct 29, 2025
71e5ff2
removed extra comments
edna-harriet Oct 29, 2025
4bf3777
removed extra lines and returned comments removed erroneously
edna-harriet Nov 1, 2025
cbed098
removed extra lines, comments and returned initial SHA commit that ha…
edna-harriet Nov 1, 2025
d42d80a
removed extra lines and comments
edna-harriet Nov 1, 2025
4f2c588
modified extra lines and changes previously made
edna-harriet Nov 1, 2025
fb37d15
removed extra comments
edna-harriet Nov 1, 2025
bcd4928
moved suggested tags back to initial SHA commits or branchnames
edna-harriet Nov 1, 2025
bc56748
removed added ln77
edna-harriet Nov 2, 2025
dc073df
undo:removed added ln77
edna-harriet Nov 2, 2025
a5cc926
replaced branchnames with tags
edna-harriet Nov 3, 2025
47f6ddf
used tags in place of branchnames
edna-harriet Nov 3, 2025
7d9f7f7
replaced branch names with tags
edna-harriet Nov 4, 2025
d6c40a2
use tags in place of branch names
edna-harriet Nov 4, 2025
e11fd75
replaced branch names with tag
edna-harriet Nov 4, 2025
adae416
returned comment that had been remove
edna-harriet Nov 4, 2025
5cfbd50
reversed SHA commit back to the initial SHA commit value
edna-harriet Nov 4, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -192,7 +192,7 @@ export async function verifyBlocksExecutionPayload(
// a valid terminal PoW block.
//
// However specs define this check to be run inside forkChoice's onBlock
// (https://github.com/ethereum/consensus-specs/blob/dev/specs/bellatrix/fork-choice.md#on_block)
// (https://github.com/ethereum/consensus-specs/blob/v1.6.0-alpha.6/specs/bellatrix/fork-choice.md#on_block)
// but we perform the check here (as inspired from the lighthouse impl)
//
// Reasons:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -131,6 +131,7 @@ const MAX_ATTESTATIONS_PER_GROUP_ELECTRA = Math.min(
MAX_ATTESTATIONS_ELECTRA
);


/** Same to https://github.com/ethereum/consensus-specs/blob/v1.5.0/specs/altair/beacon-chain.md#has_flag */
const TIMELY_SOURCE = 1 << TIMELY_SOURCE_FLAG_INDEX;
const TIMELY_TARGET = 1 << TIMELY_TARGET_FLAG_INDEX;
Expand Down Expand Up @@ -831,6 +832,7 @@ export function aggregateInto(attestation1: AttestationWithIndex, attestation2:
/**
* Electra and after: Block proposer consolidates attestations with the same
* attestation data from different committee into a single attestation
*
* https://github.com/ethereum/consensus-specs/blob/aba6345776aa876dad368cab27fbbb23fae20455/specs/_features/eip7549/validator.md?plain=1#L39
*/
export function aggregateConsolidation({byCommittee, attData}: AttestationsConsolidation): electra.Attestation {
Expand Down
3 changes: 2 additions & 1 deletion packages/beacon-node/src/chain/opPools/utils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,8 @@ export function isValidBlsToExecutionChangeForBlockInclusion(
state: CachedBeaconStateAllForks,
signedBLSToExecutionChange: capella.SignedBLSToExecutionChange
): boolean {
// For each condition from https://github.com/ethereum/consensus-specs/blob/dev/specs/capella/beacon-chain.md#new-process_bls_to_execution_change

// For each condition from https://github.com/ethereum/consensus-specs/blob/v1.6.0-alpha.6/specs/capella/beacon-chain.md#new-process_bls_to_execution_change
//
// 1. assert address_change.validator_index < len(state.validators):
// If valid before will always be valid in the future, no need to check
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -315,7 +315,9 @@ export async function produceBlockBody<T extends BlockType>(
parentBlockRoot: toRootHex(parentBlockRoot),
feeRecipient,
});
// https://github.com/ethereum/consensus-specs/blob/dev/specs/deneb/validator.md#constructing-the-beaconblockbody


// https://github.com/ethereum/consensus-specs/blob/v1.6.0-alpha.6/specs/deneb/validator.md#constructing-the-beaconblockbody
const prepareRes = await prepareExecutionPayload(
this,
this.logger,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ import {kzg} from "../../util/kzg.js";

/**
* Optionally sanity-check that the KZG commitments match the versioned hashes in the transactions
* https://github.com/ethereum/consensus-specs/blob/11a037fd9227e29ee809c9397b09f8cc3383a8c0/specs/eip4844/validator.md#blob-kzg-commitments
* https://github.com/ethereum/consensus-specs/blob/v1.2.0/specs/eip4844/validator.md#blob-kzg-commitments
*/
export async function validateBlobsAndKzgCommitments(
commitments: deneb.KZGCommitment[],
Expand Down
2 changes: 1 addition & 1 deletion packages/beacon-node/src/chain/validation/attestation.ts
Original file line number Diff line number Diff line change
Expand Up @@ -765,7 +765,7 @@ function verifyAttestationTargetRoot(headBlock: ProtoBlock, targetRoot: Root, at
// Whilst this attestation *technically* could be used to add value to a block, it is
// invalid in the spirit of the protocol. Here we choose safety over profit.
//
// Reference:

// https://github.com/ethereum/consensus-specs/pull/2001#issuecomment-699246659
throw new AttestationError(GossipAction.REJECT, {
code: AttestationErrorCode.INVALID_TARGET_ROOT,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,8 @@ import {LightClientError, LightClientErrorCode} from "../errors/lightClientError
import {IBeaconChain} from "../interface.js";
import {updateReceivedTooEarly} from "./lightClientOptimisticUpdate.js";

// https://github.com/ethereum/consensus-specs/blob/dev/specs/altair/light-client/p2p-interface.md#light_client_finality_update

// https://github.com/ethereum/consensus-specs/blob/v1.6.0-alpha.6/specs/altair/light-client/p2p-interface.md#light_client_finality_update
export function validateLightClientFinalityUpdate(
config: ChainForkConfig,
chain: IBeaconChain,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,8 @@ import {GossipAction} from "../errors/index.js";
import {LightClientError, LightClientErrorCode} from "../errors/lightClientError.js";
import {IBeaconChain} from "../interface.js";

// https://github.com/ethereum/consensus-specs/blob/dev/specs/altair/light-client/p2p-interface.md#light_client_optimistic_update

// https://github.com/ethereum/consensus-specs/blob/v1.6.0-alpha.6/specs/altair/light-client/p2p-interface.md#light_client_optimistic_update
export function validateLightClientOptimisticUpdate(
config: ChainForkConfig,
chain: IBeaconChain,
Expand Down
3 changes: 2 additions & 1 deletion packages/beacon-node/src/chain/validatorMonitor.ts
Original file line number Diff line number Diff line change
Expand Up @@ -839,7 +839,8 @@ function renderAttestationSummary(
summary: AttestationSummary | undefined,
flags: ParticipationFlags
): string {
// Reference https://github.com/ethereum/consensus-specs/blob/dev/specs/altair/beacon-chain.md#get_attestation_participation_flag_indices

// https://github.com/ethereum/consensus-specs/blob/v1.6.0-alpha.6/specs/altair/beacon-chain.md#get_attestation_participation_flag_indices
//
// is_matching_source = data.source == justified_checkpoint
// is_matching_target = is_matching_source and data.target.root == get_block_root(state, data.target.epoch)
Expand Down
2 changes: 1 addition & 1 deletion packages/beacon-node/src/constants/network.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
/**
* For more info on some of these constants:
* https://github.com/ethereum/consensus-specs/blob/v1.1.10/specs/phase0/p2p-interface.md#configuration
* https://github.com/ethereum/consensus-specs/blob/v1.6.0-alpha.6/specs/phase0/p2p-interface.md#configuration
*/

// Gossip constants
Expand Down
3 changes: 2 additions & 1 deletion packages/beacon-node/src/execution/engine/interface.ts
Original file line number Diff line number Diff line change
Expand Up @@ -132,7 +132,8 @@ export interface IExecutionEngine {
* Returns ``True`` iff ``execution_payload`` is valid with respect to ``self.execution_state``.
*
* Required for block processing in the beacon state transition function.
* https://github.com/ethereum/consensus-specs/blob/0eb0a934a3/specs/merge/beacon-chain.md#on_payload
* Reference : on_payload
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

The Reference : on_payload prefix seems unnecessary. The subsequent line provides the direct link to the specification, which serves as the reference. Removing the Reference prefix would improve readability.

Suggested change
* Reference : on_payload
* https://github.com/ethereum/consensus-specs/blob/4c1156d504279069799924dbbc79cf13d9454ae3/specs/merge/beacon-chain.md#on_payload

* https://github.com/ethereum/consensus-specs/blob/4c1156d504279069799924dbbc79cf13d9454ae3/specs/merge/beacon-chain.md#on_payload
*
* Should be called in advance before, after or in parallel to block processing
*/
Expand Down
6 changes: 4 additions & 2 deletions packages/beacon-node/src/network/network.ts
Original file line number Diff line number Diff line change
Expand Up @@ -690,7 +690,9 @@ export class Network implements INetwork {

try {
// messages SHOULD be broadcast after SYNC_MESSAGE_DUE_BPS of slot has transpired
// https://github.com/ethereum/consensus-specs/blob/dev/specs/altair/light-client/p2p-interface.md#sync-committee


// https://github.com/ethereum/consensus-specs/blob/v1.6.0-alpha.6/specs/altair/light-client/p2p-interface.md#sync-committee
await this.waitForSyncMessageCutoff(finalityUpdate.signatureSlot);
await this.publishLightClientFinalityUpdate(finalityUpdate);
} catch (e) {
Expand All @@ -707,7 +709,7 @@ export class Network implements INetwork {

try {
// messages SHOULD be broadcast after SYNC_MESSAGE_DUE_BPS of slot has transpired
// https://github.com/ethereum/consensus-specs/blob/dev/specs/altair/light-client/p2p-interface.md#sync-committee
// https://github.com/ethereum/consensus-specs/blob/v1.6.0-alpha.6/specs/altair/light-client/p2p-interface.md#sync-committee
await this.waitForSyncMessageCutoff(optimisticUpdate.signatureSlot);
await this.publishLightClientOptimisticUpdate(optimisticUpdate);
} catch (e) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,8 @@ export type ReqRespBeaconNodeOpts = ReqRespOpts & {disableLightClientServer?: bo
* Implementation of Ethereum Consensus p2p Req/Resp domain.
* For the spec that this code is based on, see:
* https://github.com/ethereum/consensus-specs/blob/v1.1.10/specs/phase0/p2p-interface.md#the-reqresp-domain
* https://github.com/ethereum/consensus-specs/blob/dev/specs/altair/light-client/p2p-interface.md#the-reqresp-domain
* Reference : the-reqresp-domain
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

The Reference : the-reqresp-domain prefix seems unnecessary. The subsequent line provides the direct link to the specification, which serves as the reference. Removing the Reference prefix would improve readability.

Suggested change
* Reference : the-reqresp-domain
* https://github.com/ethereum/consensus-specs/blob/03c4ea240fccfa78469aa039a4a7177beca54074/specs/altair/light-client/p2p-interface.md#the-reqresp-domain

* https://github.com/ethereum/consensus-specs/blob/v1.6.0-alpha.6/specs/altair/light-client/p2p-interface.md#the-reqresp-domain
*/
export class ReqRespBeaconNode extends ReqResp {
private readonly metadataController: MetadataController;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ export async function* onBlobSidecarsByRange(
// Note: Here the forkChoice head may change due to a re-org, so the headChain reflects the canonical chain
// at the time of the start of the request. Spec is clear the chain of blobs must be consistent, but on
// re-org there's no need to abort the request
// Spec: https://github.com/ethereum/consensus-specs/blob/a1e46d1ae47dd9d097725801575b46907c12a1f8/specs/eip4844/p2p-interface.md#blobssidecarsbyrange-v1
// Spec: https://github.com/ethereum/consensus-specs/blob/v1.2.0/specs/eip4844/p2p-interface.md#blobssidecarsbyrange-v1

const blobSideCarsBytesWrapped = await unfinalized.getBinary(fromHex(block.blockRoot));
if (!blobSideCarsBytesWrapped) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ export async function* onBlobSidecarsByRoot(

// NOTE: Only support non-finalized blocks.
// SPEC: Clients MUST support requesting blocks and sidecars since the latest finalized epoch.
// https://github.com/ethereum/consensus-specs/blob/11a037fd9227e29ee809c9397b09f8cc3383a8c0/specs/eip4844/p2p-interface.md#beaconblockandblobssidecarbyroot-v1
// https://github.com/ethereum/consensus-specs/blob/v1.2.0/specs/eip4844/p2p-interface.md#beaconblockandblobssidecarbyroot-v1
if (!block || block.slot <= finalizedSlot) {
continue;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ export async function* onDataColumnSidecarsByRange(
// Note: Here the forkChoice head may change due to a re-org, so the headChain reflects the canonical chain
// at the time of the start of the request. Spec is clear the chain of columns must be consistent, but on
// re-org there's no need to abort the request
// Spec: https://github.com/ethereum/consensus-specs/blob/ad36024441cf910d428d03f87f331fbbd2b3e5f1/specs/fulu/p2p-interface.md#L425-L429
// Spec: https://github.com/ethereum/consensus-specs/blob/v1.6.0-alpha.6/specs/fulu/p2p-interface.md#L425-L429
const dataColumnSidecars = await unfinalized.getManyBinary(fromHex(block.blockRoot), availableColumns);

const unavailableColumnIndices: ColumnIndex[] = [];
Expand Down
3 changes: 2 additions & 1 deletion packages/beacon-node/src/sync/backfill/backfill.ts
Original file line number Diff line number Diff line change
Expand Up @@ -128,7 +128,8 @@ export class BackfillSync extends (EventEmitter as {new (): BackfillSyncEmitter}
private wsValidated = false;

/**
* From https://github.com/ethereum/consensus-specs/blob/dev/specs/phase0/weak-subjectivity.md
* Reference :
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

The Reference : prefix seems unnecessary. The subsequent line provides the direct link to the specification, which serves as the reference. Removing the Reference prefix would improve readability.

Suggested change
* Reference :
* From https://github.com/ethereum/consensus-specs/blob/46306c4f8addef64defc2a52d85f46992ba2421b/specs/phase0/weak-subjectivity.md

* From https://github.com/ethereum/consensus-specs/blob/46306c4f8addef64defc2a52d85f46992ba2421b/specs/phase0/weak-subjectivity.md
*
*
* If
Expand Down
2 changes: 1 addition & 1 deletion packages/beacon-node/src/sync/constants.ts
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ export const BATCH_BUFFER_SIZE = Math.ceil(10 / EPOCHS_PER_BATCH);

/**
* Maximum number of concurrent requests to perform with a SyncChain.
* This is according to the spec https://github.com/ethereum/consensus-specs/blob/dev/specs/phase0/p2p-interface.md
* This is according to the spec https://github.com/ethereum/consensus-specs/blob/46306c4f8addef64defc2a52d85f46992ba2421b/specs/phase0/p2p-interface.md
*/
export const MAX_CONCURRENT_REQUESTS = 2;

Expand Down
12 changes: 8 additions & 4 deletions packages/beacon-node/test/spec/bls/bls.ts
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,8 @@ export const testFnByType: Record<string, "skip" | ((data: any) => any)> = {
};

/**
* https://github.com/ethereum/bls12-381-tests/blob/master/formats/aggregate_verify.md
* Reference : aggregate_verify
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

The Reference : aggregate_verify prefix seems unnecessary. The subsequent line provides the direct link to the specification, which serves as the reference. Removing the Reference prefix would improve readability.

Suggested change
* Reference : aggregate_verify
* https://github.com/ethereum/bls12-381-tests/blob/7a1e2f64adf524d7c53c33c7bfbed6afcc73f664/formats/aggregate_verify.md

* https://github.com/ethereum/bls12-381-tests/blob/7a1e2f64adf524d7c53c33c7bfbed6afcc73f664/formats/aggregate_verify.md
* ```
* input:
* pubkeys: List[bytes48] -- the pubkeys
Expand Down Expand Up @@ -89,7 +90,8 @@ function fast_aggregate_verify(input: {pubkeys: string[]; message: string; signa
* signatures: List[bytes96] -- the signatures to verify against pubkeys and messages
* output: bool -- VALID or INVALID
* ```
* https://github.com/ethereum/bls12-381-tests/blob/master/formats/batch_verify.md
* Reference : batch_verify
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

The Reference : batch_verify prefix seems unnecessary. The subsequent line provides the direct link to the specification, which serves as the reference. Removing the Reference prefix would improve readability.

Suggested change
* Reference : batch_verify
* https://github.com/ethereum/bls12-381-tests/blob/2e357bf380a3eb72883f529d0e303ae03251767c/formats/batch_verify.md

* https://github.com/ethereum/bls12-381-tests/blob/2e357bf380a3eb72883f529d0e303ae03251767c/formats/batch_verify.md
*/
function batch_verify(input: {pubkeys: string[]; messages: string[]; signatures: string[]}): boolean | null {
const {pubkeys, messages, signatures} = input;
Expand All @@ -113,7 +115,8 @@ function batch_verify(input: {pubkeys: string[]; messages: string[]; signatures:
* message: bytes32 -- input message to sign (a hash)
* output: BLS Signature -- expected output, single BLS signature or empty.
* ```
* https://github.com/ethereum/bls12-381-tests/blob/master/formats/sign.md
* Reference - sign.md
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

The Reference - sign.md prefix seems unnecessary. The subsequent line provides the direct link to the specification, which serves as the reference. Removing the Reference prefix would improve readability.

Suggested change
* Reference - sign.md
* https://github.com/ethereum/bls12-381-tests/blob/c01854d47b936b65404e5f08181f05db48792679/formats/sign.md

* https://github.com/ethereum/bls12-381-tests/blob/c01854d47b936b65404e5f08181f05db48792679/formats/sign.md
*/
function sign(input: {privkey: string; message: string}): string | null {
const {privkey, message} = input;
Expand Down Expand Up @@ -145,7 +148,8 @@ function verify(input: {pubkey: string; message: string; signature: string}): bo
* input: pubkey: bytes48 -- the pubkey
* output: bool -- VALID or INVALID
* ```
* https://github.com/ethereum/bls12-381-tests/blob/master/formats/deserialization_G1.md
* Reference : deserialization_G1
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

The Reference : deserialization_G1 prefix seems unnecessary. The subsequent line provides the direct link to the specification, which serves as the reference. Removing the Reference prefix would improve readability.

Suggested change
* Reference : deserialization_G1
* https://github.com/ethereum/bls12-381-tests/blob/f61f5f01e41c0e422d812d1254f06e89c2531c5b/formats/deserialization_G1.md

* https://github.com/ethereum/bls12-381-tests/blob/f61f5f01e41c0e422d812d1254f06e89c2531c5b/formats/deserialization_G1.md
*/
function deserialization_G1(input: {pubkey: string}): boolean {
try {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ const epochTransitionFns: Record<string, EpochTransitionFn> = {
};

/**
* https://github.com/ethereum/consensus-specs/blob/dev/tests/formats/epoch_processing/README.md
* https://github.com/ethereum/consensus-specs/blob/v1.6.0-alpha.6/tests/formats/epoch_processing/README.md
*/
type EpochTransitionCacheingTestCase = {
meta?: {bls_setting?: bigint};
Expand Down Expand Up @@ -86,7 +86,7 @@ const epochProcessing =

if (testcase.post === undefined) {
// If post.ssz_snappy is not value, the sub-transition processing is aborted
// https://github.com/ethereum/consensus-specs/blob/dev/tests/formats/epoch_processing/README.md#postssz_snappy
// https://github.com/ethereum/consensus-specs/blob/v1.6.0-alpha.6/tests/formats/epoch_processing/README.md#postssz_snappy
expect(() => epochTransitionFn(state, epochTransitionCache)).toThrow();
} else {
epochTransitionFn(state, epochTransitionCache);
Expand Down Expand Up @@ -121,7 +121,7 @@ specTestIterator(path.join(ethereumConsensusSpecsTests.outputDir, "tests", ACTIV
fn: epochProcessing([
// TODO: invalid_large_withdrawable_epoch asserts an overflow on a u64 for its exit epoch.
// Currently unable to reproduce in Lodestar, skipping for now
// https://github.com/ethereum/consensus-specs/blob/3212c419f6335e80ed825b4855a071f76bef70c3/tests/core/pyspec/eth2spec/test/phase0/epoch_processing/test_process_registry_updates.py#L349
// https://github.com/ethereum/consensus-specs/blob/v1.6.0-alpha.6/tests/core/pyspec/eth2spec/test/phase0/epoch_processing/test_process_registry_updates.py#L349
"invalid_large_withdrawable_epoch",
]),
},
Expand Down
3 changes: 2 additions & 1 deletion packages/beacon-node/test/spec/presets/finality.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,8 @@ export function generateBlocksSZZTypeMapping(fork: ForkName, n: number): BlocksS
* ```
* {blocks_count: 16}
* ```
* https://github.com/ethereum/consensus-specs/blob/dev/tests/formats/finality/README.md
*
* https://github.com/ethereum/consensus-specs/blob/v1.6.0-alpha.6/tests/formats/finality/README.md
*/
type FinalityTestCase = {
[k: string]: altair.SignedBeaconBlock | unknown | null | undefined;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ import {RootHex, ssz} from "@lodestar/types";
import {toHex} from "@lodestar/utils";
import {TestRunnerFn} from "../../utils/types.js";

// https://github.com/ethereum/consensus-specs/blob/da3f5af919be4abb5a6db5a80b235deb8b4b5cba/tests/formats/light_client/single_merkle_proof.md
// https://github.com/ethereum/consensus-specs/blob/v1.6.0-alpha.6/tests/formats/light_client/single_merkle_proof.md
type SingleMerkleProofTestCase = {
meta?: any;
object: TreeViewDU<any>;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ import {fromHex, toHex} from "@lodestar/utils";
import {testLogger} from "../../../utils/logger.js";
import {TestRunnerFn} from "../../utils/types.js";

// https://github.com/ethereum/consensus-specs/blob/da3f5af919be4abb5a6db5a80b235deb8b4b5cba/tests/formats/light_client/single_merkle_proof.md
// https://github.com/ethereum/consensus-specs/blob/v1.6.0-alpha.6/tests/formats/light_client/single_merkle_proof.md
type SyncTestCase = {
meta: {
genesis_validators_root: RootHex;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ import {InputType} from "@lodestar/spec-test-util";
import {LightClientUpdate, altair, ssz, sszTypesFor} from "@lodestar/types";
import {TestRunnerFn} from "../../utils/types.js";

// https://github.com/ethereum/consensus-specs/blob/da3f5af919be4abb5a6db5a80b235deb8b4b5cba/tests/formats/light_client/update_ranking.md
// https://github.com/ethereum/consensus-specs/blob/v1.6.0-alpha.6/tests/formats/light_client/update_ranking.md
type UpdateRankingTestCase = {
meta: {
updates_count: bigint;
Expand Down
4 changes: 3 additions & 1 deletion packages/beacon-node/test/spec/presets/ssz_static.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,9 @@ import {RunnerType} from "../utils/types.js";
// | serialized.ssz_snappy
// | value.yaml
//
// Docs: https://github.com/ethereum/consensus-specs/blob/master/tests/formats/ssz_static/core.md
//
//
// Docs: https://github.com/ethereum/consensus-specs/blob/v1.6.0-alpha.6/tests/formats/ssz_static/core.md

type Types = Record<string, Type<any>>;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -55,8 +55,6 @@ export function isVotingKeystore(filename: string): boolean {
// However, that's too restrictive. Guide left here as a reference
//
// The format exported by the `staking-deposit-cli` library.
//
// Reference to function that generates keystores:
// https://github.com/ethereum/staking-deposit-cli/blob/7cebff15eac299b3b1b090c896dd3410c8463450/eth2deposit/credentials.py#L58-L62
//
// Since we include the key derivation path of `m/12381/3600/x/0/0` this should only ever match
Expand Down
4 changes: 2 additions & 2 deletions packages/config/src/chainConfig/configs/mainnet.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@ import {PresetName} from "@lodestar/params";
import {fromHex as b} from "@lodestar/utils";
import {ChainConfig} from "../types.js";

// Mainnet config
// https://github.com/ethereum/consensus-specs/blob/dev/configs/mainnet.yaml

// https://github.com/ethereum/consensus-specs/blob/v1.6.0-alpha.6/configs/mainnet.yaml

export const chainConfig: ChainConfig = {
// Extends the mainnet preset
Expand Down
4 changes: 2 additions & 2 deletions packages/config/src/chainConfig/configs/minimal.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@ import {PresetName} from "@lodestar/params";
import {fromHex as b} from "@lodestar/utils";
import {ChainConfig} from "../types.js";

// Minimal config
// https://github.com/ethereum/consensus-specs/blob/dev/configs/minimal.yaml

// https://github.com/ethereum/consensus-specs/blob/v1.6.0-alpha.6/configs/minimal.yaml

export const chainConfig: ChainConfig = {
// Extends the minimal preset
Expand Down
2 changes: 1 addition & 1 deletion packages/config/src/chainConfig/networks/chiado.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ import {ChainConfig} from "../types.js";
import {gnosisChainConfig as gnosis} from "./gnosis.js";

// Chiado beacon chain config:
// https://github.com/gnosischain/configs/blob/main/chiado/config.yaml
// https://github.com/gnosischain/configs/blob/3f44fafa888439f56fc60d2f6173bfe63f31c962/chiado/config.yaml

export const chiadoChainConfig: ChainConfig = {
...gnosis,
Expand Down
3 changes: 2 additions & 1 deletion packages/config/src/chainConfig/networks/ephemery.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,8 @@ import {chainConfig as mainnet} from "../configs/mainnet.js";
import {ChainConfig} from "../types.js";

// Ephemery dynamic beacon chain config:
// https://github.com/ephemery-testnet/ephemery-genesis/blob/master/cl-config.yaml

// https://github.com/ephemery-testnet/ephemery-genesis/blob/dbc3377143c6fc1a688d4a9c648f89d8812ec7aa/cl-config.yaml

// Ephemery specification:
// https://eips.ethereum.org/EIPS/eip-6916
Expand Down
3 changes: 2 additions & 1 deletion packages/config/src/chainConfig/networks/gnosis.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,8 @@ import {chainConfig as mainnet} from "../configs/mainnet.js";
import {ChainConfig} from "../types.js";

// Gnosis beacon chain config:
// https://github.com/gnosischain/configs/blob/main/mainnet/config.yaml

// https://github.com/gnosischain/configs/blob/d447a94bbe1bafbef4fe5a2fbb2e5469e7b494bb/mainnet/config.yaml

export const gnosisChainConfig: ChainConfig = {
...mainnet,
Expand Down
Loading