chore: remove 'active' from validator status type (#7146)

* chore: remove 'active' from validator status type

* Clean up usage

Author: nflaig

packages/beacon-node/src/api/impl/validator/index.ts

@@ -1363,7 +1363,6 @@ export function getValidatorApi(
         const validator = headState.validators.getReadonly(validatorIndex);
         const status = getValidatorStatus(validator, currentEpoch);
         return (
-          status === "active" ||
           status === "active_exiting" ||
           status === "active_ongoing" ||
           status === "active_slashed" ||

packages/types/src/utils/validatorStatus.ts

@@ -1,8 +1,10 @@
 import {FAR_FUTURE_EPOCH} from "@lodestar/params";
 import {Epoch, phase0} from "../types.js";
 
+/**
+ * [Validator status specification](https://hackmd.io/ofFJ5gOmQpu1jjHilHbdQQ)
+ */
 export type ValidatorStatus =
-  | "active"
   | "pending_initialized"
   | "pending_queued"
   | "active_ongoing"

packages/validator/src/services/indices.ts

@@ -20,7 +20,6 @@ type SimpleValidatorStatus = "pending" | "active" | "exited" | "withdrawn";
 
 const statusToSimpleStatusMapping = (status: routes.beacon.ValidatorStatus): SimpleValidatorStatus => {
   switch (status) {
-    case "active":
     case "active_exiting":
     case "active_slashed":
     case "active_ongoing":

packages/validator/test/unit/services/attestationDuties.test.ts

@@ -33,7 +33,7 @@ describe("AttestationDutiesService", function () {
   const defaultValidator: routes.beacon.ValidatorResponse = {
     index,
     balance: 32e9,
-    status: "active",
+    status: "active_ongoing",
     validator: ssz.phase0.Validator.defaultValue(),
   };
 

packages/validator/test/unit/services/syncCommitteDuties.test.ts

@@ -36,7 +36,7 @@ describe("SyncCommitteeDutiesService", function () {
   const defaultValidator: routes.beacon.ValidatorResponse = {
     index: indices[0],
     balance: 32e9,
-    status: "active",
+    status: "active_ongoing",
     validator: ssz.phase0.Validator.defaultValue(),
   };