Work on feedback

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

@@ -478,14 +478,16 @@ export function getValidatorApi({
       // configurations could cause a validator pubkey to have builder disabled with builder selection builder only
       // (TODO: independently make sure such an options update is not successful for a validator pubkey)
       //
-      // So if builder is disabled ignore builder selection of builderonly if caused by user mistake
+      // So if builder is disabled ignore builder selection of builder only if caused by user mistake
+      // https://github.com/ChainSafe/lodestar/issues/6338
       const isEngineEnabled = !isBuilderEnabled || builderSelection !== routes.validator.BuilderSelection.BuilderOnly;
 
       const loggerContext = {
         fork,
         builderSelection,
         slot,
         isBuilderEnabled,
+        isEngineEnabled,
         strictFeeRecipientCheck,
         // winston logger doesn't like bigint
         builderBoostFactor: `${builderBoostFactor}`,
@@ -520,9 +522,8 @@ export function getValidatorApi({
               })
             : Promise.reject<ReturnType<typeof produceBuilderBlindedBlock>>(builderDisabledError),
 
-          isEngineEnabled // TODO deneb: builderSelection needs to be figured out if to be done beacon side
-            ? // || builderSelection !== BuilderSelection.BuilderOnly
-              produceEngineFullBlockOrContents(slot, randaoReveal, graffiti, {
+          isEngineEnabled
+            ? produceEngineFullBlockOrContents(slot, randaoReveal, graffiti, {
                 feeRecipient,
                 strictFeeRecipientCheck,
                 // skip checking and recomputing head in these individual produce calls

packages/utils/src/promise.ts

@@ -117,6 +117,11 @@ export class MutedPromise<T> implements Promise<T> {
   }
 }
 
+/**
+ * ArrayToTuple converts an `Array<T>` to `[T, ...T]`
+ *
+ * eg: `[1, 2, 3]` from type `number[]` to `[number, number, number]`
+ */
 type ReturnPromiseWithTuple<Tuple extends NonEmptyArray<unknown>> = {
   [Index in keyof ArrayToTuple<Tuple>]: MutedPromise<Awaited<Tuple[Index]>>;
 };