Merge branch 'hiero-ledger:main' into main

Author: MrMadHatt

.coderabbit.yaml

@@ -19,6 +19,20 @@ reviews:
   poem: false # Do not write a literal poem (spammy)
   enable_prompt_for_ai_agents: false # Disable prompts for AI agents (spammy)
 
+  # ============================================================
+  # GLOBAL REVIEW INSTRUCTIONS (APPLY TO ALL FILES)
+  # ============================================================
+  instructions: |
+    You are a code reviewer whose primary responsibility is to verify that the code changes in this pull request fully address the specific requirements outlined in the associated issue description or pull request description.
+    **ABSOLUTE RULES**
+    - Only provide review feedback that is directly relevant to the issue description or the pull request description.
+    - Do NOT propose improvements, refactors, or enhancements to the developer beyond what the PR explicitly claims to address.
+    **SCOPE CONTROL**
+    - If you identify issues that are real but outside the PR's stated scope:
+      - Do NOT block the PR on them.
+      - Do NOT suggest fixes inline.
+      - Instead, aggregate all out-of-scope issues into a single comment with a list of recommendations for one or more follow-up issues that can be created.
+
   path_instructions:
     # --- CUSTOM INSTRUCTIONS FOR EXAMPLES DIRECTORY ---
     - path: "examples/**/*"

.github/ISSUE_TEMPLATE/06_intermediate_issue.yml

@@ -14,31 +14,26 @@ body:
         We truly appreciate your time and effort. 
         This template is designed to help you create an Intermediate issue.
 
-        ⚠️ **Important:**  
-        Intermediate contributors are still building **conceptual awareness and responsibility skills**.
-        Issues created using this template **must be selected and framed carefully** to avoid breaking
-        changes or unintended behavior.
-
         The goal is to create an issue for users that have:
-        - familiarity with the Hiero Python SDK codebase
+        - basic familiarity with the Hiero Python SDK codebase
         - experience following our contribution workflow
         - confidence navigating existing source code and examples
         ---
 
   - type: textarea
     id: intro
     attributes:
-      label: 🛠️ Intermediate Contributors
+      label: 🧩 Intermediate Contributors
       description: Who is this issue for?
       value: |
         This issue is intended for contributors who already have some familiarity with the
         [Hiero Python SDK](https://hiero.org) codebase and contribution workflow.
-
+  
         You should feel comfortable:
         - navigating existing source code and examples
         - understanding SDK concepts without step-by-step guidance
         - following the standard PR workflow without additional onboarding
-
+  
         If this is your very first contribution to the project, we recommend starting with a few
         **Good First Issues** before working on this one.
     validations:
@@ -58,7 +53,6 @@ body:
         > - May involve **refactors or small feature additions**
         > - Provides **enough documentation or examples** to reason about a solution
         > - Often has **similar patterns elsewhere in the codebase**
-        > - Involves unit and integration testing
         >
         > **What this issue is NOT:**
         > - A beginner-friendly onboarding task
@@ -91,7 +85,7 @@ body:
     attributes:
       value: |
         <!-- Example for Problem (hidden in submission) -->
-        ## 📌 Problem – Example
+        ## 🐞 Problem – Example
 
         The `TransactionGetReceiptQuery` currently exposes the `get_children()` method,
         but the behavior is inconsistent with how child receipts are returned by the Mirror Node.

.github/scripts/bot-community-calls.sh

@@ -0,0 +1,119 @@
+#!/usr/bin/env bash
+set -euo pipefail
+
+DRY_RUN="${DRY_RUN:-true}"
+
+ANCHOR_DATE="2025-11-12"
+MEETING_LINK="https://zoom-lfx.platform.linuxfoundation.org/meeting/92041330205?password=2f345bee-0c14-4dd5-9883-06fbc9c60581"
+CALENDAR_LINK="https://zoom-lfx.platform.linuxfoundation.org/meetings/hiero?view=week"
+
+CANCELLED_DATES=(
+  "2025-12-24"
+)
+
+EXCLUDED_AUTHORS=(
+  "rbair23"
+  "nadineloepfe"
+  "exploreriii"
+  "manishdait"
+  "Dosik13"
+  "hendrikebbers"
+)
+
+if [ "$DRY_RUN" = "true" ]; then
+  echo "=== DRY RUN MODE ENABLED ==="
+  echo "No comments will be posted."
+fi
+
+TODAY=$(date -u +"%Y-%m-%d")
+for CANCELLED in "${CANCELLED_DATES[@]}"; do
+  if [ "$TODAY" = "$CANCELLED" ]; then
+    echo "Community Call cancelled on $TODAY. Exiting."
+    exit 0
+  fi
+done
+
+IS_MEETING_WEEK=$(python3 - <<EOF
+from datetime import datetime, date, timezone
+d1 = date.fromisoformat("$ANCHOR_DATE")
+d2 = datetime.now(timezone.utc).date()
+print("true" if (d2 - d1).days % 14 == 0 else "false")
+EOF
+)
+
+if [ "$IS_MEETING_WEEK" = "false" ]; then
+  echo "Not a fortnightly meeting week. Exiting."
+  exit 0
+fi
+
+if [ -z "${GITHUB_REPOSITORY:-}" ]; then
+  echo "ERROR: GITHUB_REPOSITORY is not set."
+  exit 1
+fi
+
+REPO="$GITHUB_REPOSITORY"
+
+ISSUE_DATA=$(gh issue list \
+  --repo "$REPO" \
+  --state open \
+  --json number,author,createdAt)
+
+if [ -z "$ISSUE_DATA" ] || [ "$ISSUE_DATA" = "[]" ]; then
+  echo "No open issues found."
+  exit 0
+fi
+
+COMMENT_BODY=$(cat <<EOF
+Hello, this is CommunityCallBot.
+
+This is a reminder that the Hiero Python SDK Community Call will begin in approximately 4 hours (14:00 UTC).
+
+The call is an open forum where contributors and users can discuss topics, raise issues, and influence the direction of the Python SDK.
+
+Details:
+- Time: 14:00 UTC
+- Join Link: [Zoom Meeting]($MEETING_LINK)
+
+Disclaimer: This is an automated reminder. Please verify the schedule [here]($CALENDAR_LINK) for any changes.
+EOF
+)
+
+echo "$ISSUE_DATA" |
+  jq -r '
+    group_by(.author.login)
+    | .[]
+    | sort_by(.createdAt)
+    | reverse
+    | .[0]
+    | "\(.number) \(.author.login)"
+  ' |
+  while read ISSUE_NUM AUTHOR; do
+    for EXCLUDED in "${EXCLUDED_AUTHORS[@]}"; do
+      if [ "$AUTHOR" = "$EXCLUDED" ]; then
+        echo "Skipping issue #$ISSUE_NUM by excluded author @$AUTHOR"
+        continue 2
+      fi
+    done
+
+    ALREADY_COMMENTED=$(gh issue view "$ISSUE_NUM" \
+      --repo "$REPO" \
+      --json comments \
+      --jq '.comments[].body' | grep -F "CommunityCallBot" || true)
+
+    if [ -n "$ALREADY_COMMENTED" ]; then
+      echo "Issue #$ISSUE_NUM already notified. Skipping."
+      continue
+    fi
+
+    if [ "$DRY_RUN" = "true" ]; then
+      echo "----------------------------------------"
+      echo "[DRY RUN] Would comment on issue #$ISSUE_NUM"
+      echo "[DRY RUN] Author: @$AUTHOR"
+      echo "----------------------------------------"
+      echo "$COMMENT_BODY"
+      echo "----------------------------------------"
+    else
+      gh issue comment "$ISSUE_NUM" --repo "$REPO" --body "$COMMENT_BODY"
+      echo "Reminder posted to issue #$ISSUE_NUM"
+    fi
+  done

…scripts/bot-inactivity-unassign-phase.sh …ithub/scripts/bot-inactivity-unassign.sh.github/scripts/bot-inactivity-unassign-phase.sh renamed to .github/scripts/bot-inactivity-unassign.sh .github/scripts/bot-inactivity-unassign-phase.sh renamed to .github/scripts/bot-inactivity-unassign.sh

@@ -1,28 +1,46 @@
 module.exports = async ({ github, context }) => {
-  const body = context.payload.pull_request.body || "";
-  const regex = /\bFixes\s*:?\s*(#\d+)(\s*,\s*#\d+)*/i;
+  let prNumber;
+  try {
+    const isDryRun = process.env.DRY_RUN === 'true';
+    prNumber = parseInt(process.env.PR_NUMBER) || context.payload.pull_request.number;
+    
+    console.log(`Processing PR #${prNumber} (Dry run: ${isDryRun})`);
+    
+    // For workflow_dispatch, we need to fetch PR details
+    let prData;
+    if (context.payload.pull_request) {
+      prData = context.payload.pull_request;
+    } else {
+      // workflow_dispatch case - fetch PR data
+      const prResponse = await github.rest.pulls.get({
+        owner: context.repo.owner,
+        repo: context.repo.repo,
+        pull_number: prNumber,
+      });
+      prData = prResponse.data;
+    }
 
-  const comments = await github.rest.issues.listComments({
-  owner: context.repo.owner,
-  repo: context.repo.repo,
-  issue_number: context.payload.pull_request.number,
-  });
+    const body = prData.body || "";
+    const regex = /\bFixes\s*:?\s*(#\d+)(\s*,\s*#\d+)*/i;
 
-  const alreadyCommented = comments.data.some(comment =>
-    comment.body.includes("this is LinkBot")
-  );
-
-  if (alreadyCommented) {
-    return;
-  }
-
-  if (!regex.test(body)) {
-    await github.rest.issues.createComment({
+    const comments = await github.rest.issues.listComments({
       owner: context.repo.owner,
       repo: context.repo.repo,
-      issue_number: context.payload.pull_request.number,
-      body: [
-        `Hi @${context.payload.pull_request.user.login}, this is **LinkBot** 👋`,
+      issue_number: prNumber,
+    });
+
+    const alreadyCommented = comments.data.some(comment =>
+      comment.body.includes("this is LinkBot")
+    );
+
+    if (alreadyCommented) {
+      console.log('LinkBot already commented on this PR');
+      return;
+    }
+
+    if (!regex.test(body)) {
+      const commentBody = [
+        `Hi @${prData.user.login}, this is **LinkBot** 👋`,
         ``,
         `Linking pull requests to issues helps us significantly with reviewing pull requests and keeping the repository healthy.`,
         ``,
@@ -38,8 +56,30 @@ module.exports = async ({ github, context }) => {
         `[docs/sdk_developers/creating_issues.md](https://github.com/${context.repo.owner}/${context.repo.repo}/blob/main/docs/sdk_developers/creating_issues.md)`,
         ``,
         `Thanks!`
-      ].join('\n')
-    });
+      ].join('\n');
+
+      if (isDryRun) {
+        console.log('DRY RUN: Would post the following comment:');
+        console.log('---');
+        console.log(commentBody);
+        console.log('---');
+      } else {
+        await github.rest.issues.createComment({
+          owner: context.repo.owner,
+          repo: context.repo.repo,
+          issue_number: prNumber,
+          body: commentBody,
+        });
+        console.log('LinkBot comment posted successfully');
+      }
+    } else {
+      console.log('PR has linked issue - no comment needed');
+    }
+  } catch (error) {
+    console.error('Error processing PR:', error);
+    console.error('PR number:', prNumber);
+    console.error('Repository:', `${context.repo.owner}/${context.repo.repo}`);
+    throw error;
   }
 };
 

.github/scripts/bot-pr-missing-linked-issue.js

@@ -0,0 +1,83 @@
+// Script to trigger CodeRabbit plan for intermediate and advanced issues
+
+const marker = '<!-- CodeRabbit Plan Trigger -->';
+
+async function triggerCodeRabbitPlan(github, owner, repo, issue, marker) {
+  const comment = `${marker} @coderabbitai plan`;
+
+  try {
+    await github.rest.issues.createComment({
+      owner,
+      repo,
+      issue_number: issue.number,
+      body: comment,
+    });
+    console.log(`Triggered CodeRabbit plan for issue #${issue.number}`);
+    return true;
+  } catch (commentErr) {
+    console.log(`Failed to trigger CodeRabbit plan for issue #${issue.number}:`, commentErr.message || commentErr);
+    return false;
+  }
+}
+
+function hasIntermediateOrAdvancedLabel(issue, label) {
+  // Check if issue has intermediate or advanced label (case-insensitive)
+  const hasIntermediateLabel = issue.labels?.some(l => l?.name?.toLowerCase() === 'intermediate');
+  const hasAdvancedLabel = issue.labels?.some(l => l?.name?.toLowerCase() === 'advanced');
+  
+  // Also check if newly added label is intermediate/advanced
+  const isNewLabelIntermediate = label?.name?.toLowerCase() === 'intermediate';
+  const isNewLabelAdvanced = label?.name?.toLowerCase() === 'advanced';
+  
+  return hasIntermediateLabel || hasAdvancedLabel || isNewLabelIntermediate || isNewLabelAdvanced;
+}
+
+async function hasExistingCodeRabbitPlan(github, owner, repo, issueNumber) {
+  // Check for existing CodeRabbit plan comment (limited to first 500 comments)
+  const comments = [];
+  const iterator = github.paginate.iterator(github.rest.issues.listComments, {
+    owner, repo, issue_number: issueNumber, per_page: 100
+  });
+  
+  let count = 0;
+  for await (const { data: page } of iterator) {
+    comments.push(...page);
+    count += page.length;
+    if (count >= 500) break; // Hard upper bound to prevent unbounded pagination
+  }
+  
+  return comments.some(c => c.body?.includes('@coderabbitai plan'));
+}
+
+function logSummary(owner, repo, issue) {
+  console.log('=== Summary ===');
+  console.log(`Repository: ${owner}/${repo}`);
+  console.log(`Issue Number: ${issue.number}`);
+  console.log(`Issue Title: ${issue.title || '(no title)'}`);
+  console.log(`Labels: ${issue.labels?.map(l => l.name).join(', ') || 'none'}`);
+}
+
+module.exports = async ({ github, context }) => {
+  try {
+    const { owner, repo } = context.repo;
+    const { issue, label } = context.payload;
+
+    // Validations
+    if (!issue?.number) return console.log('No issue in payload');
+    
+    if (!hasIntermediateOrAdvancedLabel(issue, label)) {
+      return console.log('Issue does not have intermediate or advanced label');
+    }
+
+    if (await hasExistingCodeRabbitPlan(github, owner, repo, issue.number)) {
+      return console.log(`CodeRabbit plan already triggered for #${issue.number}`);
+    }
+
+    // Post CodeRabbit plan trigger
+    await triggerCodeRabbitPlan(github, owner, repo, issue, marker);
+
+    logSummary(owner, repo, issue);
+  } catch (err) {
+    console.log('❌ Error:', err.message);
+  }
+};