Refactor node connection passing to make it use LocalNodeConnectInfo type

[carbolymer]

Changelog

- description: |
    Refactor node connection passing to make it use `LocalNodeConnectInfo` type
# uncomment types applicable to the change:
  type:
  # - feature        # introduces a new feature
  # - breaking       # the API has changed in a breaking way
  # - compatible     # the API has changed but is non-breaking
  # - optimisation   # measurable performance improvements
   - refactoring    # QoL changes
  # - bugfix         # fixes a defect
  # - test           # fixes/modifies tests
  # - maintenance    # not directly related to the code
  # - release        # related to a new release preparation
  # - documentation  # change in code docs, haddocks...

Context

This is just a compatible refactoring to reduce boilerplate. LocalNodeConnectInfo is now used for carrying node connection info, instead of three separate fields.

The change in golden files, is just a change of the order where --socket-path appears in CLI help.

Checklist

• Commit sequence broadly makes sense and commits have useful messages
• New tests are added if needed and existing tests are updated. See Running tests for more details
• Self-reviewed the diff

[smelc]

Nice 👍