Skip to content

Conversation

@stwiname
Copy link
Contributor

@stwiname stwiname commented Nov 5, 2025

  • Update @subql deps
  • [release] 20251105

Summary by CodeRabbit

  • Chores
    • Bumped common-algorand package version from 4.4.4 to 4.4.5.
    • Bumped node package version from 4.0.4 to 4.0.5.
    • Updated internal/shared dependencies to compatible newer versions.
    • Updated changelogs and release links to reflect the new versions.

@coderabbitai
Copy link

coderabbitai bot commented Nov 5, 2025

Walkthrough

This PR bumps package versions and updates dependencies: packages/common-algorand 4.4.4 → 4.4.5 and packages/node 4.0.4 → 4.0.5. Corresponding CHANGELOG.md entries and comparison links were added/updated; @subql/common and @subql/node-core dependency ranges were incremented.

Changes

Cohort / File(s) Change summary
Changelogs
packages/common-algorand/CHANGELOG.md, packages/node/CHANGELOG.md
Added new release entries (4.4.5 and 4.0.5), updated Unreleased comparison refs and added compare links for the new versions; kept existing entries unchanged.
Package manifests
packages/common-algorand/package.json, packages/node/package.json
Bumped package versions (common-algorand: 4.4.44.4.5; node: 4.0.44.0.5); updated dependency @subql/common ^5.8.0^5.8.1 in both; updated @subql/node-core ^18.5.0^18.5.1 in packages/node/package.json.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Mechanical changelog and manifest/version bumps.
  • Files to spot-check: packages/*/package.json dependency ranges and packages/*/CHANGELOG.md compare links.

Possibly related PRs

Poem

🐇
I hopped through lines of changelog light,
Pushed version carrots into the night,
Common and Node now fresh and new,
A tiny bump — a joyful chew! 🥕

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'Release 20251105' is vague and uses a generic date format that doesn't convey meaningful information about the specific changes (version bumps for common-algorand and node packages, dependency updates). Consider using a more descriptive title like 'Bump packages to 4.4.5/4.0.5 and update dependencies' to better communicate the primary changes.
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch release-20251105-algo

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 2aa03fe and 411b40a.

📒 Files selected for processing (2)
  • packages/common-algorand/CHANGELOG.md (2 hunks)
  • packages/node/CHANGELOG.md (2 hunks)
🚧 Files skipped from review as they are similar to previous changes (2)
  • packages/node/CHANGELOG.md
  • packages/common-algorand/CHANGELOG.md

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 5, 2025

Coverage report

Caution

Test run failed

St.
Category Percentage Covered / Total
🔴 Statements 33.57% 899/2678
🔴 Branches 55.22% 74/134
🔴 Functions 29.85% 40/134
🔴 Lines 33.57% 899/2678

Test suite run failed

Failed tests: 3/19. Failed suites: 2/4.
  ● dictionary v1 › successfully validates metatada

    ApolloError: Failed to parse URL from /graphql

      at new ApolloError (node_modules/@apollo/client/errors/index.js:59:28)
      at node_modules/@apollo/client/core/QueryManager.js:823:71
      at both (node_modules/@apollo/client/utilities/observables/asyncMap.js:35:31)
      at node_modules/@apollo/client/utilities/observables/asyncMap.js:24:72
      at Object.then (node_modules/@apollo/client/utilities/observables/asyncMap.js:24:24)
      at Object.error (node_modules/@apollo/client/utilities/observables/asyncMap.js:37:49)
      at notifySubscription (node_modules/zen-observable/lib/Observable.js:140:18)
      at onNotify (node_modules/zen-observable/lib/Observable.js:179:3)
      at SubscriptionObserver.error (node_modules/zen-observable/lib/Observable.js:240:7)
      at node_modules/@apollo/client/utilities/observables/iteration.js:11:68
          at Array.forEach (<anonymous>)
      at iterateObserversSafely (node_modules/@apollo/client/utilities/observables/iteration.js:11:25)
      at Object.error (node_modules/@apollo/client/utilities/observables/Concast.js:76:21)
      at notifySubscription (node_modules/zen-observable/lib/Observable.js:140:18)
      at onNotify (node_modules/zen-observable/lib/Observable.js:179:3)
      at SubscriptionObserver.error (node_modules/zen-observable/lib/Observable.js:240:7)
      at handleError (node_modules/@apollo/client/link/http/parseAndCheckHttpResponse.js:170:14)
      at node_modules/@apollo/client/link/http/createHttpLink.js:148:17

    Cause:
    TypeError: Failed to parse URL from /graphql



    Cause:
    TypeError: Invalid URL

      146 |       chainId,
      147 |     );
    > 148 |     await dictionary.init();
          |                      ^
      149 |     return dictionary;
      150 |   }
      151 |

      at node_modules/@apollo/client/link/http/createHttpLink.js:130:13
      at new Subscription (node_modules/zen-observable/lib/Observable.js:197:34)
      at Observable.subscribe (node_modules/zen-observable/lib/Observable.js:279:14)
      at Object.complete (node_modules/@apollo/client/utilities/observables/Concast.js:111:43)
      at Concast.Object.<anonymous>.Concast.start (node_modules/@apollo/client/utilities/observables/Concast.js:152:23)
      at new Concast (node_modules/@apollo/client/utilities/observables/Concast.js:137:19)
      at QueryManager.Object.<anonymous>.QueryManager.getObservableFromLink (node_modules/@apollo/client/core/QueryManager.js:744:37)
      at QueryManager.Object.<anonymous>.QueryManager.getResultsFromLink (node_modules/@apollo/client/core/QueryManager.js:786:30)
      at resultsFromLink (node_modules/@apollo/client/core/QueryManager.js:1114:26)
      at QueryManager.Object.<anonymous>.QueryManager.fetchQueryByPolicy (node_modules/@apollo/client/core/QueryManager.js:1177:52)
      at fromVariables (node_modules/@apollo/client/core/QueryManager.js:856:41)
      at QueryManager.Object.<anonymous>.QueryManager.fetchConcastWithInfo (node_modules/@apollo/client/core/QueryManager.js:898:35)
      at QueryManager.Object.<anonymous>.QueryManager.fetchQuery (node_modules/@apollo/client/core/QueryManager.js:383:21)
      at QueryManager.Object.<anonymous>.QueryManager.query (node_modules/@apollo/client/core/QueryManager.js:481:21)
      at ApolloClient.Object.<anonymous>.ApolloClient.query (node_modules/@apollo/client/core/ApolloClient.js:276:34)
      at AlgorandDictionaryV1.init (node_modules/@subql/node-core/src/indexer/dictionary/v1/dictionaryV1.ts:54:21)
      at AlgorandDictionaryV1.create (packages/node/src/indexer/dictionary/v1/algorandDictionaryV1.ts:148:22)
      at Object.<anonymous> (packages/node/src/indexer/dictionary/v1/algorandDictionaryV1.spec.ts:72:45)

  ● dictionary v1 › build correct dictionary entries and remove undefined fields in filter

    ApolloError: Failed to parse URL from /graphql

      at new ApolloError (node_modules/@apollo/client/errors/index.js:59:28)
      at node_modules/@apollo/client/core/QueryManager.js:823:71
      at both (node_modules/@apollo/client/utilities/observables/asyncMap.js:35:31)
      at node_modules/@apollo/client/utilities/observables/asyncMap.js:24:72
      at Object.then (node_modules/@apollo/client/utilities/observables/asyncMap.js:24:24)
      at Object.error (node_modules/@apollo/client/utilities/observables/asyncMap.js:37:49)
      at notifySubscription (node_modules/zen-observable/lib/Observable.js:140:18)
      at onNotify (node_modules/zen-observable/lib/Observable.js:179:3)
      at SubscriptionObserver.error (node_modules/zen-observable/lib/Observable.js:240:7)
      at node_modules/@apollo/client/utilities/observables/iteration.js:11:68
          at Array.forEach (<anonymous>)
      at iterateObserversSafely (node_modules/@apollo/client/utilities/observables/iteration.js:11:25)
      at Object.error (node_modules/@apollo/client/utilities/observables/Concast.js:76:21)
      at notifySubscription (node_modules/zen-observable/lib/Observable.js:140:18)
      at onNotify (node_modules/zen-observable/lib/Observable.js:179:3)
      at SubscriptionObserver.error (node_modules/zen-observable/lib/Observable.js:240:7)
      at handleError (node_modules/@apollo/client/link/http/parseAndCheckHttpResponse.js:170:14)
      at node_modules/@apollo/client/link/http/createHttpLink.js:148:17

    Cause:
    TypeError: Failed to parse URL from /graphql



    Cause:
    TypeError: Invalid URL

      146 |       chainId,
      147 |     );
    > 148 |     await dictionary.init();
          |                      ^
      149 |     return dictionary;
      150 |   }
      151 |

      at node_modules/@apollo/client/link/http/createHttpLink.js:130:13
      at new Subscription (node_modules/zen-observable/lib/Observable.js:197:34)
      at Observable.subscribe (node_modules/zen-observable/lib/Observable.js:279:14)
      at Object.complete (node_modules/@apollo/client/utilities/observables/Concast.js:111:43)
      at Concast.Object.<anonymous>.Concast.start (node_modules/@apollo/client/utilities/observables/Concast.js:152:23)
      at new Concast (node_modules/@apollo/client/utilities/observables/Concast.js:137:19)
      at QueryManager.Object.<anonymous>.QueryManager.getObservableFromLink (node_modules/@apollo/client/core/QueryManager.js:744:37)
      at QueryManager.Object.<anonymous>.QueryManager.getResultsFromLink (node_modules/@apollo/client/core/QueryManager.js:786:30)
      at resultsFromLink (node_modules/@apollo/client/core/QueryManager.js:1114:26)
      at QueryManager.Object.<anonymous>.QueryManager.fetchQueryByPolicy (node_modules/@apollo/client/core/QueryManager.js:1177:52)
      at fromVariables (node_modules/@apollo/client/core/QueryManager.js:856:41)
      at QueryManager.Object.<anonymous>.QueryManager.fetchConcastWithInfo (node_modules/@apollo/client/core/QueryManager.js:898:35)
      at QueryManager.Object.<anonymous>.QueryManager.fetchQuery (node_modules/@apollo/client/core/QueryManager.js:383:21)
      at QueryManager.Object.<anonymous>.QueryManager.query (node_modules/@apollo/client/core/QueryManager.js:481:21)
      at ApolloClient.Object.<anonymous>.ApolloClient.query (node_modules/@apollo/client/core/ApolloClient.js:276:34)
      at AlgorandDictionaryV1.init (node_modules/@subql/node-core/src/indexer/dictionary/v1/dictionaryV1.ts:54:21)
      at AlgorandDictionaryV1.create (packages/node/src/indexer/dictionary/v1/algorandDictionaryV1.ts:148:22)
      at Object.<anonymous> (packages/node/src/indexer/dictionary/v1/algorandDictionaryV1.spec.ts:72:45)


  ● ApiService › waits on pending block to fetch hash

    Unable to get block hash for round 55259274

      250 |       }
      251 |     }
    > 252 |     throw new Error(`Unable to get block hash for round ${round}`);
          |           ^
      253 |   }
      254 | }
      255 |

      at AlgorandApi.getBlockHash (packages/node/src/algorand/api.algorand.ts:252:11)
      at packages/node/src/algorand/api.algorand.ts:205:22
          at async Promise.all (index 0)
      at AlgorandApi.fetchBlocks (packages/node/src/algorand/api.algorand.ts:203:29)
      at fetchLatestBlock (packages/node/src/indexer/api.service.spec.ts:87:8)

  ● ApiService › waits on pending block to fetch hash

    Unable to get block hash for round 55259274

      250 |       }
      251 |     }
    > 252 |     throw new Error(`Unable to get block hash for round ${round}`);
          |           ^
      253 |   }
      254 | }
      255 |

      at AlgorandApi.getBlockHash (packages/node/src/algorand/api.algorand.ts:252:11)
      at packages/node/src/algorand/api.algorand.ts:205:22
          at async Promise.all (index 0)
      at AlgorandApi.fetchBlocks (packages/node/src/algorand/api.algorand.ts:203:29)
      at fetchLatestBlock (packages/node/src/indexer/api.service.spec.ts:87:8)
      at Object.<anonymous> (packages/node/src/indexer/api.service.spec.ts:91:19)

Report generated by 🧪jest coverage report action from 411b40a

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 2

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7e66390 and 2aa03fe.

⛔ Files ignored due to path filters (1)
  • yarn.lock is excluded by !**/yarn.lock, !**/*.lock
📒 Files selected for processing (4)
  • packages/common-algorand/CHANGELOG.md (2 hunks)
  • packages/common-algorand/package.json (2 hunks)
  • packages/node/CHANGELOG.md (2 hunks)
  • packages/node/package.json (2 hunks)
🔇 Additional comments (4)
packages/common-algorand/CHANGELOG.md (1)

153-154: Reference links updated correctly.

The version reference links follow the correct pattern and are properly updated for the new release.

packages/common-algorand/package.json (1)

3-3: Version and dependency updates are consistent.

The package version matches the CHANGELOG entry (4.4.5), and the @subql/common dependency update (^5.8.0 → ^5.8.1) aligns with the changelog note to update this dependency.

Also applies to: 17-17

packages/node/package.json (1)

3-3: Version and dependency updates are consistent.

The package version matches the CHANGELOG entry (4.0.5), and both dependencies are updated consistently:

  • @subql/common: ^5.8.0 → ^5.8.1 (aligned with common-algorand package)
  • @subql/node-core: ^18.5.0 → ^18.5.1 (aligns with changelog note)

All updates are patch-level and forward-compatible.

Also applies to: 27-27, 29-29

packages/node/CHANGELOG.md (1)

324-325: Reference links updated correctly.

The version reference links follow the correct pattern and are properly updated for the new release.

@stwiname stwiname merged commit 80ce0a1 into main Nov 5, 2025
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants