chore: remove SelfSerializable from NodeId

[abies]

Description:
Part of the bigger task of getting rid of SelfSerialiable. NodeId had the issue with slightly wrong deserialization, so removing it early helps with buggy node scans.

Related issue(s):

Fixes #26756

Checklist

• Documented (Code comments, README, etc.)
• Tested (unit, integration, etc.)

[lfdt-bot]

✅ Snyk checks have passed. No issues have been found so far.

Status	Scan Engine	Critical	High	Medium	Low	Total (0)
✅	Open Source Security	0	0	0	0	0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

@@             Coverage Diff              @@
##               main   #26805      +/-   ##
============================================
+ Coverage     70.45%   70.47%   +0.01%     
+ Complexity    11674    11672       -2     
============================================
  Files          2582     2582              
  Lines        108331   108316      -15     
  Branches      12117    12114       -3     
============================================
+ Hits          76330    76332       +2     
+ Misses        28017    27998      -19     
- Partials       3984     3986       +2     

Files with missing lines	Coverage Δ	Complexity Δ
...in/java/org/hiero/consensus/model/node/NodeId.java	68.00% <ø> (+3.89%)	0.00 <0.00> (ø)
...ensus/constructable/ConstructableRegistration.java	87.50% <ø> (-0.74%)	0.00 <0.00> (ø)

... and 10 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[testlens-app]

✅ All tests passed ✅

🏷️ Commit: fd0ee6e
▶️ Tests: 125 executed
⚪️ Checks: 50/50 completed

---

Learn more about TestLens at testlens.app.