Merge #3770

3770: [backport] fix(rpc): return the cycles of the first non-cellbase transaction as cellbase's cycles r=quake a=yangby-cryptape

### What problem does this PR solve?

Backport #3763.

### Check List

Tests

- Unit test
- Integration test
- Manual test (add detailed scripts or steps below)
- No code ci-runs-only: [ quick_checks,linters ]

### Release note

```release-note
Title Only: Include only the PR title in the release note.
```

Co-authored-by: Boyu Yang <[email protected]>

rpc/src/module/chain.rs

@@ -1890,13 +1890,17 @@ impl ChainRpcImpl {
     ) -> Result<Option<TransactionWithStatus>> {
         let snapshot = self.shared.snapshot();
         if let Some(tx_info) = snapshot.get_transaction_info(&tx_hash) {
-            let cycles = snapshot
-                .get_block_ext(&tx_info.block_hash)
-                .and_then(|block_ext| {
-                    block_ext
-                        .cycles
-                        .and_then(|v| v.get(tx_info.index.saturating_sub(1)).copied())
-                });
+            let cycles = if tx_info.is_cellbase() {
+                None
+            } else {
+                snapshot
+                    .get_block_ext(&tx_info.block_hash)
+                    .and_then(|block_ext| {
+                        block_ext
+                            .cycles
+                            .and_then(|v| v.get(tx_info.index.saturating_sub(1)).copied())
+                    })
+            };
 
             return Ok(Some(TransactionWithStatus::with_committed(
                 None,
@@ -1929,13 +1933,17 @@ impl ChainRpcImpl {
     ) -> Result<Option<TransactionWithStatus>> {
         let snapshot = self.shared.snapshot();
         if let Some((tx, tx_info)) = snapshot.get_transaction_with_info(&tx_hash) {
-            let cycles = snapshot
-                .get_block_ext(&tx_info.block_hash)
-                .and_then(|block_ext| {
-                    block_ext
-                        .cycles
-                        .and_then(|v| v.get(tx_info.index.saturating_sub(1)).copied())
-                });
+            let cycles = if tx_info.is_cellbase() {
+                None
+            } else {
+                snapshot
+                    .get_block_ext(&tx_info.block_hash)
+                    .and_then(|block_ext| {
+                        block_ext
+                            .cycles
+                            .and_then(|v| v.get(tx_info.index.saturating_sub(1)).copied())
+                    })
+            };
 
             return Ok(Some(TransactionWithStatus::with_committed(
                 Some(tx),