refactor(trackerless-network): unify naming and refactor selectParallelTargets

[juslesan]

Summary

No longer nodes that are attempted as neighbors neighbors. Added getExcludedIds helper function to remove duplicate code. Added clearer comments to explain the control flow. Fixed a bug where nodes were pushed to excludedIds too soon.

[Copilot]

Pull Request Overview

This PR refactors the handshake target selection logic in the neighbor discovery module by unifying naming and reducing code duplication.

• Renamed local variables from "neighbors" to "targets" for clarity.
• Introduced the getExcludedIds helper function to consolidate exclusion logic.
• Updated handshake functions to use consistent naming.

Comments suppressed due to low confidence (1)

packages/trackerless-network/src/content-delivery-layer/neighbor-discovery/Handshaker.ts:79

• [nitpick] Consider renaming the helper function 'getExcludedIds' to a more descriptive name (e.g., 'aggregateExcludedIds') to clarify that it combines the initial exclusions with the keys from 'targets'.

const getExcludedIds = () => [...excludedIds, ...Array.from(targets.keys())]