chore: Replace print statements with logger

[iron-prog]

Description:
Replace print() and traceback.print_exc() with structured logging across query and network modules.

• Add module-level loggers to query and network modules
• Replace print()/traceback.print_exc() with logger.error(..., exc_info=True) or logger.warning(...)
• Remove unused traceback imports
• Add regression tests to verify no print() calls remain, logs use the correct module logger, and tracebacks are preserved

Related issue(s):
Fixes #2403

Notes for reviewer:

• No public API changes; internal logging only
• Docstring print(...) examples were intentionally left unchanged
• TransactionGetReceiptQuery uses a different mock target because it builds its header inline

Checklist

• Documented
• Tested

[coderabbitai]

Walkthrough

Query request error paths and mirror-node discovery now use module-level logging instead of direct console output, while preserving re-raises and fallback results. New unit tests verify logger names, levels, exception information, and removal of direct traceback printing.

Changes

Structured error logging

Layer / File(s)	Summary
Query request error logging src/hiero_sdk_python/account/*, src/hiero_sdk_python/contract/*, src/hiero_sdk_python/file/*, src/hiero_sdk_python/query/*, src/hiero_sdk_python/schedule/*	Query _make_request handlers use module loggers with exc_info=True instead of print and traceback.print_exc(), then re-raise exceptions.
Mirror-node diagnostic logging src/hiero_sdk_python/client/network.py	Mirror-node URL, request, and parsing diagnostics use warning and error logger calls while returning the existing fallback values.
Logging behavior validation tests/unit/query_error_logging_test.py	Parameterized tests verify query logging, absence of direct traceback output, and mirror-node warning/error records.

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

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly summarizes the main change: replacing print-based output with logging.
Description check	✅ Passed	The description matches the changeset and accurately describes the logging updates and tests.
Linked Issues check	✅ Passed	The PR addresses #2403 by removing direct print/traceback output from the listed query and network paths while preserving behavior.
Out of Scope Changes check	✅ Passed	The added tests and logging-only code changes stay within the stated issue scope, with no obvious unrelated changes.
Docstring Coverage	✅ Passed	Docstring coverage is 88.46% which is sufficient. The required threshold is 80.00%.

📋 Issue Planner

Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).

View plan for ticket: #2403

✨ 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]

@iron-prog, thanks for the PR, just a small change request

[coderabbitai]

Actionable comments posted: 4

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

src/hiero_sdk_python/query/token_info_query.py (1)

69-70: 🎯 Functional Correctness | 🟠 Major | ⚡ Quick win

Restore required token-ID validation.

if self.token_id now allows None, so _make_request() silently builds a request with an empty/default token instead of raising the documented ValueError. Restore the guard and serialize only after validation.

Suggested fix

         try:
+            if self.token_id is None:
+                raise ValueError("Token ID must be set before making the request.")
             query_header = self._make_request_header()

             token_info_query = token_get_info_pb2.TokenGetInfoQuery()
             token_info_query.header.CopyFrom(query_header)
-            if self.token_id:
-                token_info_query.token.CopyFrom(self.token_id._to_proto())
+            token_info_query.token.CopyFrom(self.token_id._to_proto())

Add a regression test for TokenInfoQuery()._make_request() without a token ID.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: be976864-47ab-41c2-be81-e9bcded72eb1

📥 Commits

Reviewing files that changed from the base of the PR and between 6afbe47 and 24472d2.

📒 Files selected for processing (15)

• src/hiero_sdk_python/account/account_records_query.py
• src/hiero_sdk_python/client/network.py
• src/hiero_sdk_python/contract/contract_bytecode_query.py
• src/hiero_sdk_python/contract/contract_call_query.py
• src/hiero_sdk_python/contract/contract_info_query.py
• src/hiero_sdk_python/file/file_contents_query.py
• src/hiero_sdk_python/file/file_info_query.py
• src/hiero_sdk_python/query/account_balance_query.py
• src/hiero_sdk_python/query/account_info_query.py
• src/hiero_sdk_python/query/token_info_query.py
• src/hiero_sdk_python/query/token_nft_info_query.py
• src/hiero_sdk_python/query/topic_info_query.py
• src/hiero_sdk_python/query/transaction_get_receipt_query.py
• src/hiero_sdk_python/schedule/schedule_info_query.py
• tests/unit/query_error_logging_test.py

resolved

[exploreriii]

Hi @iron-prog this is ready to merge, can you click update branch at the bottom please

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

@@            Coverage Diff             @@
##             main    #2452      +/-   ##
==========================================
+ Coverage   95.07%   95.27%   +0.20%     
==========================================
  Files         164      164              
  Lines       10474    10485      +11     
==========================================
+ Hits         9958     9990      +32     
+ Misses        516      495      -21     

🚀 New features to boost your workflow:

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

[github-actions]

👋 Hi @iron-prog! 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