feat: Update PublicKey To Key In Node Create And Update

[aceppaluni]

Description:

This PR updates node create/update to use Key instead of PublicKey as per issue #2337.

Related issue(s):

Fixes #2337

Notes for reviewer:

Checklist

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

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

@@            Coverage Diff             @@
##             main    #2457      +/-   ##
==========================================
+ Coverage   95.01%   95.03%   +0.01%     
==========================================
  Files         164      164              
  Lines       10464    10464              
==========================================
+ Hits         9942     9944       +2     
+ Misses        522      520       -2     

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[coderabbitai]

Walkthrough

Node create and update transactions now accept Key values for admin_key, serialize them with to_proto_key(), deserialize them with Key.from_proto_key(), and test private-key inputs produce public key protobuf bytes.

Changes

Node admin key support

Layer / File(s)	Summary
Admin key type contracts src/hiero_sdk_python/nodes/node_create_transaction.py, src/hiero_sdk_python/nodes/node_update_transaction.py	Parameter models, constructors, documentation, and setters now use Key for admin_key.
Protobuf conversion and private-key tests src/hiero_sdk_python/nodes/node_create_transaction.py, src/hiero_sdk_python/nodes/node_update_transaction.py, tests/unit/node_create_transaction_test.py, tests/unit/node_update_transaction_test.py	Protobuf construction and parsing use generic Key conversion methods; tests verify private keys serialize public key bytes and round-trip through protobuf.

Estimated code review effort: 2 (Simple) | ~10 minutes

Sequence Diagram(s)

sequenceDiagram
  participant KeyInput
  participant NodeTransaction
  participant TransactionBody
  KeyInput->>NodeTransaction: set_admin_key(Key)
  NodeTransaction->>KeyInput: to_proto_key()
  NodeTransaction->>TransactionBody: serialize admin_key
  TransactionBody->>NodeTransaction: provide admin_key
  NodeTransaction->>KeyInput: Key.from_proto_key()

Loading

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly summarizes the main change: switching node create/update admin keys from PublicKey to Key.
Description check	✅ Passed	The description is directly related to the code changes and references the linked issue correctly.
Linked Issues check	✅ Passed	The PR matches #2337 by updating admin_key types, serialization, and tests to support generic Key values.
Out of Scope Changes check	✅ Passed	The changes stay within the node transaction update and corresponding tests, with no obvious unrelated additions.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

📋 Issue Planner

Built with CodeRabbit's Coding Plans for faster development and fewer bugs.

View plan used: #2337

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[manishdait]

@aceppaluni, please rebase the branch

[coderabbitai]

Actionable comments posted: 1

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: f45a0ebb-9d42-458c-a8e8-dcb6366d0eb9

📥 Commits

Reviewing files that changed from the base of the PR and between 714f833 and c7808ea.

📒 Files selected for processing (2)

• tests/unit/node_create_transaction_test.py
• tests/unit/node_update_transaction_test.py

[github-actions]

👋 Hi @aceppaluni! Great work completing a Intermediate issue! 🎉

Thanks for your contribution! 🚀

Here are some issues you might want to explore next:

• [Advanced]: Add CI Pipeline for Example Validation
• [Advanced]: Integrate Code Coverage for C++ SDK
• [Advanced]: Upgrade gRPC and protobuf dependencies

🌟 Stay connected:

• ⭐ Star this repository
• 👀 Watch for new issues
• 💬 Join us on Discord

Happy coding! 🚀
— Hiero Python SDK Team