chore: Added missing timeout support for some execute() implementations

[manishdait]

Description:
Support for passing a timeout to execute(client, timeout) was introduced earlier PR #1499, but some query sub-classes that override execute() did not forward the parameter when calling _execute().

This PR updates the affected query classes to correctly propagate the timeout argument to ensure execute(client, timeout) behaves consistently across queries.

Related issue(s):

Closes #1735

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    #1736   +/-   ##
=======================================
  Coverage   93.25%   93.25%           
=======================================
  Files         141      141           
  Lines        9061     9061           
=======================================
  Hits         8450     8450           
  Misses        611      611           

🚀 New features to boost your workflow:

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

[coderabbitai]

Walkthrough

Adds optional timeout propagation to several Query subclass execute() methods and corrects docstring type-annotation typos; updates CHANGELOG to note timeout propagation.

Changes

Cohort / File(s)	Summary
CHANGELOG CHANGELOG.md	Added entry documenting that Query sub-class execute() methods now propagate the optional timeout parameter.
Query subclasses (timeout propagation) src/hiero_sdk_python/account/account_records_query.py, src/hiero_sdk_python/contract/contract_bytecode_query.py, src/hiero_sdk_python/file/file_contents_query.py, src/hiero_sdk_python/file/file_info_query.py, src/hiero_sdk_python/schedule/schedule_info_query.py	Added optional timeout: Optional[Union[int, float]] = None to execute() signatures, updated typing imports to include Union, forwarded timeout into internal _execute() calls, and updated docstrings.
Docstring/type fixes (no behavioral change) src/hiero_sdk_python/contract/contract_call_query.py, src/hiero_sdk_python/contract/contract_info_query.py, src/hiero_sdk_python/query/account_info_query.py, src/hiero_sdk_python/query/account_balance_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/query/transaction_record_query.py	Fixed/corrected timeout type annotations and minor docstring formatting in execute() docstrings. No signature or runtime behavior changes.

Sequence Diagram(s)

sequenceDiagram
    participant Caller as Caller
    participant Query as Query Subclass
    participant Client as Client
    participant Exec as _execute / Transport

    Caller->>Query: execute(client, timeout)
    Query->>Client: prepare/validate request
    Query->>Exec: _execute(client, timeout)
    Exec->>Client: send request (network)
    Client-->>Exec: response
    Exec-->>Query: return response
    Query-->>Caller: return parsed result

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly identifies the main change: adding missing timeout support to execute() implementations in Query sub-classes, which matches the changeset content.
Description check	✅ Passed	The description is fully related to the changeset, explaining that timeout support was introduced in PR #1499 but not consistently propagated, and this PR fixes that.
Linked Issues check	✅ Passed	The PR fully addresses issue #1735 by adding timeout parameter propagation to all affected Query sub-class execute() methods: AccountRecordsQuery, ContractBytecodeQuery, FileContentsQuery, FileInfoQuery, and ScheduleInfoQuery.
Out of Scope Changes check	✅ Passed	All changes are in-scope: timeout parameter additions to query classes, docstring improvements, and import updates (Union type). The docstring fixes in multiple files are minor corrections related to the timeout implementation.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

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 and usage tips.}

[coderabbitai]

Actionable comments posted: 3

[Adityarya11]

LGTM!

[exploreriii]

Request step security help please @hiero-ledger/security-maintainers

[coderabbitai]

Actionable comments posted: 2

[coderabbitai]

⚠️ Potential issue | 🟡 Minor

Minor grammar issue and missing issue reference in changelog entry.

The entry reads slightly awkwardly. Consider rewording for grammatical consistency with other entries in the "Fixed" section, and adding the issue reference for traceability.

Suggested fix

-- Ensure all Query sub-class `execute()` function to correctly propagate the optional `timeout` parameter.
+- Fixed all Query sub-class `execute()` functions to correctly propagate the optional `timeout` parameter. (`#1735`)

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

	- Ensure all Query sub-class `execute()` function to correctly propagate the optional `timeout` parameter.
	- Fixed all Query sub-class `execute()` functions to correctly propagate the optional `timeout` parameter. (`#1735`)

[coderabbitai]

Actionable comments posted: 1

[coderabbitai]

🧹 Nitpick | 🔵 Trivial

Nit: typing.List is deprecated in favor of built-in list (Python 3.9+).

Ruff UP035 flags this. However, since the entire codebase consistently uses typing.List, this is fine to keep for now. Consider a codebase-wide migration as a separate task.

🧰 Tools

🪛 Ruff (0.14.14)

[warning] 6-6: typing.List is deprecated, use list instead

(UP035)

[github-actions]

Hi, this is MergeConflictBot.
Your pull request cannot be merged because it contains merge conflicts.

Please resolve these conflicts locally and push the changes.

Quick Fix for CHANGELOG.md Conflicts

If your conflict is only in CHANGELOG.md, you can resolve it easily using the GitHub web editor:

1. Click on the "Resolve conflicts" button in the PR
2. Accept both changes (keep both changelog entries)
3. Click "Mark as resolved"
4. Commit the merge

For all other merge conflicts, please read:

• Resolving Merge Conflicts
• Rebasing Guide

Thank you for contributing!

[exploreriii]

@manishdait please rebase, i think step security is now passing