Skip to content

Update dependencies in variables.yml#1481

Open
brunopgalvao wants to merge 11 commits intomasterfrom
update-dependencies-variables-yml
Open

Update dependencies in variables.yml#1481
brunopgalvao wants to merge 11 commits intomasterfrom
update-dependencies-variables-yml

Conversation

@brunopgalvao
Copy link
Contributor

@brunopgalvao brunopgalvao commented Feb 5, 2026

Summary

Updates dependencies in variables.yml to resolve open GitHub issues starting with "Update needed:".

Closes #1467, closes #1468, closes #1472, closes #1473, closes #1474, closes #1475, closes #1477

Subdependency Issues

These are pinned to the polkadot_sdk_parachain_template release (v0.0.4) and will be updated when a new template version is released:

Closes #1469, closes #1470, closes #1471

Dependencies Updated

Dependency Old New Issue
chopsticks 1.2.5 1.2.7 #1473
moonwall 5.18.2 5.18.3 #1474
polkadot_api 1.23.2 1.23.3 #1475
paraspell_sdk 12.3.0 12.5.1 #1477
polkadot_omni_node 0.7.0 0.13.0 #1472
open_zeppelin_contracts v5.4.0 v5.5.0 #1468
polkadot_sdk polkadot-stable2512 polkadot-stable2512-1 #1467

Additional Changes

  • Added polkadot_sdk_version: polkadot-stable2503-3 subdependency to polkadot_sdk_parachain_template
  • Added YAML comment explaining subdependencies are pinned to template release
  • Fixed hardcoded version in reference/tools/chopsticks.md (now uses variable)

⚠️ resolc NOT Updated (#1476)

The resolc dependency was not updated and ignore_updates: true was added.

Bug Description

The @parity/resolc npm package has a bug where compile() only requests ['abi'] in outputSelection, not bytecode. This breaks:

  1. The Ethers.js tutorial in .deploy-basic-pvm.md which expects evm.bytecode.object
  2. The CLI --bin flag (produces no output)

To Reproduce

npm install @parity/resolc@1.0.0
node -e "
  const {compile} = require('@parity/resolc');
  compile({'Test.sol': {content: 'pragma solidity ^0.8.9; contract T {}'}})
    .then(o => console.log('bytecode:', o.contracts['Test.sol']['T'].evm));
"
# Output: bytecode: undefined (should contain bytecode object)

Root Cause

In js-revive/src/index.ts, the compile() function hardcodes:

outputSelection: {
    '*': {
        '*': ['abi'],  // Missing 'evm.bytecode'!
    },
},

Action Needed

An issue should be filed at https://github.com/paritytech/js-revive to fix this bug. Once fixed, the ignore_updates flag can be removed and resolc can be updated.


Test plan

  • Verified all npm packages exist with correct versions
  • Checked release notes for breaking changes
  • Tested paraspell_sdk 12.5.1 - imports and basic functions work
  • Tested polkadot_api 1.23.3 - imports work correctly
  • Verified tutorials use version variables correctly

@brunopgalvao brunopgalvao self-assigned this Feb 5, 2026
@brunopgalvao brunopgalvao added B0 - Needs Review Pull request is ready for review C1 - Medium Medium priority task A1 - Maintenance Major Pull request contains major updates to an existing page (i.e., adding a new section, reorgs, etc.) labels Feb 5, 2026
@brunopgalvao brunopgalvao force-pushed the update-dependencies-variables-yml branch from a0d24a4 to 4be73c3 Compare February 5, 2026 07:32
@brunopgalvao brunopgalvao marked this pull request as ready for review February 5, 2026 07:33
@brunopgalvao brunopgalvao requested a review from a team as a code owner February 5, 2026 07:33
@brunopgalvao
Copy link
Contributor Author

@nhussein11 will take a look at #1476 and resolve it in a separate PR.

Copy link
Collaborator

@nhussein11 nhussein11 left a comment

Choose a reason for hiding this comment

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

thanksssss

Copy link
Contributor

@0xlukem 0xlukem left a comment

Choose a reason for hiding this comment

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

LGTM!

@0xlukem 0xlukem added B1 - Ready to Merge Pull request is ready to be merged and removed B0 - Needs Review Pull request is ready for review labels Feb 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A1 - Maintenance Major Pull request contains major updates to an existing page (i.e., adding a new section, reorgs, etc.) B1 - Ready to Merge Pull request is ready to be merged C1 - Medium Medium priority task

Projects

None yet

3 participants