fix: remove unrelated changes from GFI label PR

bhuvan-somisetty · bhuvan-somisetty · commit 905e101a9cc8 · 2026-04-30T02:49:21.000+05:30
Signed-off-by: Bhuvan Somisetty &lt;somisettybhuvan5@gmail.com&gt;
diff --git a/.github/scripts/bot-next-issue-recommendation.js b/.github/scripts/bot-next-issue-recommendation.js
@@ -59,7 +59,7 @@ module.exports = async ({ github, context, core }) => {
     // Determine issue difficulty level
     const difficultyLevels = {
       beginner: labelSet.has('beginner'),
-      goodFirstIssue: labelSet.has('Good First Issue'),
+      goodFirstIssue: labelSet.has('good first issue'),
       intermediate: labelSet.has('intermediate'),
       advanced: labelSet.has('advanced'),
     };
@@ -85,7 +85,7 @@ module.exports = async ({ github, context, core }) => {
 
     if (recommendedIssues.length === 0) {
       isFallback = true;
-      recommendedIssues = await searchIssues(github, core, repoOwner, repoName, 'Good First Issue');
+      recommendedIssues = await searchIssues(github, core, repoOwner, repoName, 'good first issue');
       recommendedLabel = 'Good First Issue';
     }
 
@@ -170,7 +170,7 @@ async function generateAndPostComment(github, context, core, prNumber, recommend
       `org:${context.repo.owner}`,
       'archived:false',
       'no:assignee',
-      '(label:"Good First Issue" OR label:"skill: Good First Issue")',
+      '(label:"good first issue" OR label:"skill: good first issue")',
       `(${repoQuery})`,
     ].join(' ');
 
diff --git a/.github/scripts/coderabbit_plan_trigger.js b/.github/scripts/coderabbit_plan_trigger.js
@@ -2,14 +2,9 @@
 
 const CODERABBIT_MARKER = '<!-- CodeRabbit Plan Trigger -->';
 
-async function triggerCodeRabbitPlan(github, owner, repo, issue, marker = CODERABBIT_MARKER, isDryRun = false) {
+async function triggerCodeRabbitPlan(github, owner, repo, issue, marker = CODERABBIT_MARKER) {
   const comment = `${marker} @coderabbitai plan`;
 
-  if (isDryRun) {
-    console.log(`[DRY RUN] Would trigger CodeRabbit plan for issue #${issue.number}`);
-    return true;
-  }
-
   try {
     await github.rest.issues.createComment({
       owner,
@@ -105,11 +100,8 @@ async function main({ github, context }) {
       return console.log(`CodeRabbit plan already triggered for #${issue.number}`);
     }
 
-    // Check for dry run (default to true if not specified, for safety)
-    const isDryRun = (process.env.DRY_RUN || 'true').toLowerCase() === 'true';
-
     // Post CodeRabbit plan trigger
-    await triggerCodeRabbitPlan(github, owner, repo, issue, CODERABBIT_MARKER, isDryRun);
+    await triggerCodeRabbitPlan(github, owner, repo, issue, CODERABBIT_MARKER);
 
     logSummary(owner, repo, issue);
   } catch (err) {
diff --git a/.github/workflows/bot-coderabbit-plan-trigger.yml b/.github/workflows/bot-coderabbit-plan-trigger.yml
@@ -5,16 +5,6 @@ name: CodeRabbit Plan Trigger
 on:
   issues:
     types: [labeled]
-  workflow_dispatch:
-    inputs:
-      dry_run:
-        description: "Run without posting comments"
-        required: false
-        default: "true"
-        type: choice
-        options:
-          - "true"
-          - "false"
 
 permissions:
   issues: write
@@ -28,10 +18,9 @@ jobs:
       cancel-in-progress: false
     # Only run when the newly added label is beginner, intermediate, or advanced (case-insensitive)
     if: >
-      github.event_name == 'workflow_dispatch' ||
-      (github.event.action == 'labeled' &&
+      github.event.action == 'labeled' &&
       github.event.issue.state == 'open' &&
-      contains(fromJson('["beginner","intermediate","advanced","Beginner","Intermediate","Advanced"]'), github.event.label.name))
+      contains(fromJson('["beginner","intermediate","advanced","Beginner","Intermediate","Advanced"]'), github.event.label.name)
 
     steps:
       - name: Harden the runner
@@ -45,7 +34,6 @@ jobs:
       - name: Trigger CodeRabbit Plan
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-          DRY_RUN: ${{ github.event_name == 'workflow_dispatch' && inputs.dry_run || 'false' }}
         uses: actions/github-script@3a2844b7e9c422d3c10d287c895573f7108da1b3 #v9.0.0
         with:
           script: |
diff --git a/.github/workflows/publish.yml b/.github/workflows/publish.yml
@@ -15,7 +15,7 @@ jobs:
 
     steps:
       - name: Harden the runner (Audit all outbound calls)
-        uses: step-security/harden-runner@6c3c2f2c1c457b00c10c4848d6f5491db3b629df # v2.18.0
+        uses: step-security/harden-runner@8d3c67de8e2fe68ef647c8db1e6a09f647780f40 # v2.19.0
         with:
           egress-policy: audit
 
@@ -62,7 +62,7 @@ jobs:
 
     steps:
       - name: Harden the runner (Audit all outbound calls)
-        uses: step-security/harden-runner@6c3c2f2c1c457b00c10c4848d6f5491db3b629df # v2.18.0
+        uses: step-security/harden-runner@8d3c67de8e2fe68ef647c8db1e6a09f647780f40 # v2.19.0
         with:
           egress-policy: audit
 
diff --git a/docs/sdk_developers/training/workflow/04_assigning_issues.md b/docs/sdk_developers/training/workflow/04_assigning_issues.md
@@ -1,6 +1,6 @@
 ## Getting Assigned to an Issue
 
-Claim an issue as yours to work on by commenting on a Good First Issue with exactly: /assign
+Claim an issue as yours to work on by commenting on a good first issue with exactly: /assign
 
 We recommend Good First Issues for developers new to the Python SDK. These are easier and better documented tasks.
 
diff --git a/src/hiero_sdk_python/consensus/topic_create_transaction.py b/src/hiero_sdk_python/consensus/topic_create_transaction.py
@@ -11,6 +11,7 @@
 
 from hiero_sdk_python.account.account_id import AccountId
 from hiero_sdk_python.channels import _Channel
+from hiero_sdk_python.crypto.key import Key
 from hiero_sdk_python.Duration import Duration
 from hiero_sdk_python.executable import _Method
 from hiero_sdk_python.hapi.services import consensus_create_topic_pb2, transaction_pb2
@@ -19,7 +20,7 @@
 )
 from hiero_sdk_python.tokens.custom_fixed_fee import CustomFixedFee
 from hiero_sdk_python.transaction.transaction import Transaction
-from hiero_sdk_python.utils.key_utils import Key, key_to_proto
+from hiero_sdk_python.utils.key_utils import key_to_proto
 
 
 class TopicCreateTransaction(Transaction):
diff --git a/src/hiero_sdk_python/query/transaction_get_receipt_query.py b/src/hiero_sdk_python/query/transaction_get_receipt_query.py
@@ -277,17 +277,22 @@ def _map_status_error(self, response: response_pb2.Response) -> PrecheckError |
             TransactionReceipt._from_proto(response.transactionGetReceipt.receipt, self.transaction_id),
         )
 
-    def _map_receipt_list(self, receipts: list[transaction_receipt_pb2.TransactionReceipt]) -> list[TransactionReceipt]:
+    def _map_receipt_list(
+        self, receipts: list[transaction_receipt_pb2.TransactionReceipt], include_parent_tx_id: bool = False
+    ) -> list[TransactionReceipt]:
         """
         Maps a list of protobuf transaction receipts to TransactionReceipt objects.
 
         Args:
             receipts: A list of protobuf TransactionReceipt objects
+            include_parent_tx_id: If True, pass parent transaction_id to mapped receipts (for duplicates).
+                                 If False, pass None (for child receipts).
 
         Returns:
             A list of TransactionReceipt objects
         """
-        return [TransactionReceipt._from_proto(receipt_proto, self.transaction_id) for receipt_proto in receipts]
+        transaction_id = self.transaction_id if include_parent_tx_id else None
+        return [TransactionReceipt._from_proto(receipt_proto, transaction_id) for receipt_proto in receipts]
 
     def execute(self, client: Client, timeout: int | float | None = None) -> TransactionReceipt:
         """
@@ -315,12 +320,18 @@ def execute(self, client: Client, timeout: int | float | None = None) -> Transac
         parent = TransactionReceipt._from_proto(response.transactionGetReceipt.receipt, self.transaction_id)
 
         if self.include_children:
-            children = self._map_receipt_list(response.transactionGetReceipt.child_transaction_receipts)
+            # Child receipts are sub-transactions; they don't need parent transaction_id
+            children = self._map_receipt_list(
+                response.transactionGetReceipt.child_transaction_receipts, include_parent_tx_id=False
+            )
 
             parent._set_children(children)
 
         if self.include_duplicates:
-            duplicates = self._map_receipt_list(response.transactionGetReceipt.duplicateTransactionReceipts)
+            # Duplicate receipts are related to parent; keep parent transaction_id for context
+            duplicates = self._map_receipt_list(
+                response.transactionGetReceipt.duplicateTransactionReceipts, include_parent_tx_id=True
+            )
 
             parent._set_duplicates(duplicates)
 
diff --git a/src/hiero_sdk_python/tokens/token_create_transaction.py b/src/hiero_sdk_python/tokens/token_create_transaction.py
@@ -18,6 +18,7 @@
 
 from hiero_sdk_python.account.account_id import AccountId
 from hiero_sdk_python.channels import _Channel
+from hiero_sdk_python.crypto.key import Key
 from hiero_sdk_python.Duration import Duration
 from hiero_sdk_python.executable import _Method
 from hiero_sdk_python.hapi.services import token_create_pb2, transaction_pb2
@@ -29,7 +30,7 @@
 from hiero_sdk_python.tokens.supply_type import SupplyType
 from hiero_sdk_python.tokens.token_type import TokenType
 from hiero_sdk_python.transaction.transaction import Transaction
-from hiero_sdk_python.utils.key_utils import Key, key_to_proto
+from hiero_sdk_python.utils.key_utils import key_to_proto
 
 
 AUTO_RENEW_PERIOD = Duration(7890000)  # around 90 days in seconds
diff --git a/src/hiero_sdk_python/tokens/token_update_transaction.py b/src/hiero_sdk_python/tokens/token_update_transaction.py
@@ -14,6 +14,7 @@
 
 from hiero_sdk_python.account.account_id import AccountId
 from hiero_sdk_python.channels import _Channel
+from hiero_sdk_python.crypto.key import Key
 from hiero_sdk_python.Duration import Duration
 from hiero_sdk_python.executable import _Method
 from hiero_sdk_python.hapi.services import token_update_pb2, transaction_pb2
@@ -25,7 +26,7 @@
 from hiero_sdk_python.tokens.token_id import TokenId
 from hiero_sdk_python.tokens.token_key_validation import TokenKeyValidation
 from hiero_sdk_python.transaction.transaction import Transaction
-from hiero_sdk_python.utils.key_utils import Key, key_to_proto
+from hiero_sdk_python.utils.key_utils import key_to_proto
 
 
 @dataclass
diff --git a/src/hiero_sdk_python/transaction/transaction.py b/src/hiero_sdk_python/transaction/transaction.py
@@ -5,6 +5,7 @@
 
 from hiero_sdk_python.account.account_id import AccountId
 from hiero_sdk_python.client.client import Client
+from hiero_sdk_python.crypto.key import Key
 from hiero_sdk_python.exceptions import PrecheckError
 from hiero_sdk_python.executable import _Executable, _ExecutionState
 from hiero_sdk_python.hapi.services import basic_types_pb2, transaction_contents_pb2, transaction_pb2
@@ -15,7 +16,7 @@
 from hiero_sdk_python.transaction.transaction_id import TransactionId
 from hiero_sdk_python.transaction.transaction_receipt import TransactionReceipt
 from hiero_sdk_python.transaction.transaction_response import TransactionResponse
-from hiero_sdk_python.utils.key_utils import Key, key_to_proto
+from hiero_sdk_python.utils.key_utils import key_to_proto
 
 
 if TYPE_CHECKING:
diff --git a/src/hiero_sdk_python/transaction/transaction_receipt.py b/src/hiero_sdk_python/transaction/transaction_receipt.py
@@ -88,10 +88,15 @@ def account_id(self) -> AccountId | None:
         """
         Retrieves the AccountId associated with the transaction receipt, if available.
 
+        Unlike other ID properties (token_id, file_id, etc.), this returns the AccountId
+        even when accountNum is 0. This is intentional to support EVM auto-account creation
+        scenarios where the protobuf may contain an AccountID with num=0 to identify the
+        newly created account before it's fully initialized on-chain.
+
         Returns:
-            AccountId or None: The AccountId if present; otherwise, None.
+            AccountId or None: The AccountId if present (including num=0); otherwise, None.
         """
-        if self._receipt_proto.HasField("accountID") and self._receipt_proto.accountID.accountNum != 0:
+        if self._receipt_proto.HasField("accountID"):
             return AccountId._from_proto(self._receipt_proto.accountID)
         return None
 
@@ -107,7 +112,9 @@ def serial_numbers(self) -> list[int]:
 
     @property
     def file_id(self) -> FileId | None:
-        """Returns the file ID associated with this receipt."""
+        """
+        Returns the file ID associated with this receipt.
+        """
         if self._receipt_proto.HasField("fileID") and self._receipt_proto.fileID.fileNum != 0:
             return FileId._from_proto(self._receipt_proto.fileID)
         return None
@@ -243,14 +250,14 @@ def _to_proto(self):
 
     @classmethod
     def _from_proto(
-        cls, proto: transaction_receipt_pb2.TransactionReceipt, transaction_id: TransactionId
+        cls, proto: transaction_receipt_pb2.TransactionReceipt, transaction_id: TransactionId | None
     ) -> TransactionReceipt:
         """
         Creates a TransactionReceipt instance from a protobuf TransactionReceipt object.
 
         Args:
             proto (transaction_receipt_pb2.TransactionReceipt): The protobuf TransactionReceipt object.
-            transaction_id (TransactionId): The transaction ID associated with this receipt.
+            transaction_id (TransactionId | None): The transaction ID associated with this receipt. Can be None for child receipts.
 
         Returns:
             TransactionReceipt: A new instance of TransactionReceipt populated with data from the protobuf object.
diff --git a/tests/integration/account_records_query_e2e_test.py b/tests/integration/account_records_query_e2e_test.py
@@ -20,9 +20,10 @@ def test_integration_account_record_query_can_execute(env):
     account = env.create_account()
 
     # Perform transfer transaction
+    transfer_tx = TransferTransaction()
+    transfer_tx.transaction_fee = Hbar(2).to_tinybars()
     transfer_receipt = (
-        TransferTransaction()
-        .add_hbar_transfer(account.id, Hbar(1).to_tinybars())
+        transfer_tx.add_hbar_transfer(account.id, Hbar(1).to_tinybars())
         .add_hbar_transfer(env.operator_id, -Hbar(1).to_tinybars())
         .execute(env.client)
     )
@@ -50,9 +51,10 @@ def test_integration_account_record_query_get_cost(env):
     """Test that AccountRecordsQuery can calculate query costs."""
     account = env.create_account()
 
+    transfer_tx = TransferTransaction()
+    transfer_tx.transaction_fee = Hbar(2).to_tinybars()
     transfer_receipt = (
-        TransferTransaction()
-        .add_hbar_transfer(account.id, Hbar(1).to_tinybars())
+        transfer_tx.add_hbar_transfer(account.id, Hbar(1).to_tinybars())
         .add_hbar_transfer(env.operator_id, -Hbar(1).to_tinybars())
         .execute(env.client)
     )
@@ -75,9 +77,10 @@ def test_integration_account_record_query_insufficient_payment(env):
     """Test that AccountRecordsQuery fails with insufficient payment."""
     account = env.create_account()
 
+    transfer_tx = TransferTransaction()
+    transfer_tx.transaction_fee = Hbar(2).to_tinybars()
     transfer_receipt = (
-        TransferTransaction()
-        .add_hbar_transfer(account.id, Hbar(1).to_tinybars())
+        transfer_tx.add_hbar_transfer(account.id, Hbar(1).to_tinybars())
         .add_hbar_transfer(env.operator_id, -Hbar(1).to_tinybars())
         .execute(env.client)
     )
diff --git a/tests/integration/transaction_get_receipt_query_e2e_test.py b/tests/integration/transaction_get_receipt_query_e2e_test.py
diff --git a/tests/unit/get_receipt_query_test.py b/tests/unit/get_receipt_query_test.py
