fix coverage analysis bug

Signed-off-by: Jeromy Cannon <[email protected]>

Author: jeromy-cannon

.github/workflows/zxc-code-analysis.yaml

@@ -214,14 +214,14 @@ jobs:
         uses: actions/download-artifact@fa0a91b85d4f404e444e00e005971372dc801d16 # v4.1.8
         if: ${{ (inputs.enable-codecov-analysis || inputs.enable-codacy-coverage) && inputs.enable-e2e-coverage-report && !cancelled() && !failure() }}
         with:
-          name: ${{ inputs.e2e-node-local-build-test-coverage-report }}
+          name: ${{ inputs.e2e-node-local-build-coverage-report }}
           path: 'coverage/${{ inputs.e2e-node-local-build-test-subdir }}'
 
       - name: Download E2E Add Coverage Report
         uses: actions/download-artifact@fa0a91b85d4f404e444e00e005971372dc801d16 # v4.1.8
         if: ${{ (inputs.enable-codecov-analysis || inputs.enable-codacy-coverage) && inputs.enable-e2e-coverage-report && !cancelled() && !failure() }}
         with:
-          name: ${{ inputs.e2e-node-add-test-coverage-report }}
+          name: ${{ inputs.e2e-node-add-coverage-report }}
           path: 'coverage/${{ inputs.e2e-node-add-test-subdir }}'
 
       - name: Publish To Codecov

src/core/account_manager.mjs

@@ -623,6 +623,7 @@ export class AccountManager {
 
     // ensure serviceEndpoint.ipAddressV4 value for all nodes in the addressBook is a 4 bytes array instead of string
     // See: https://github.com/hashgraph/hedera-protobufs/blob/main/services/basic_types.proto#L1309
+    // TODO: with v0.53 will mirror node no longer need this and we can remove @hashgraph/proto?
     const addressBook = HashgraphProto.proto.NodeAddressBook.decode(addressBookBytes)
     const hasAlphaRegEx = /[a-zA-Z]+/
     let modified = false