Skip to content

Update default DFX_VERSION to 0.31.0#1312

Open
lwshang wants to merge 2 commits intomasterfrom
update-dfx-0.31.0
Open

Update default DFX_VERSION to 0.31.0#1312
lwshang wants to merge 2 commits intomasterfrom
update-dfx-0.31.0

Conversation

@lwshang
Copy link
Contributor

@lwshang lwshang commented Mar 2, 2026

Summary

  • Update DFX_VERSION from 0.30.2 to 0.31.0 in CI provisioning scripts (provision-darwin.sh and provision-linux.sh)

Test plan

  • CI workflows pass with the new DFX version

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@lwshang lwshang requested a review from a team as a code owner March 2, 2026 23:14
@lwshang lwshang marked this pull request as draft March 2, 2026 23:17
DFX 0.31.0 changed `dfx generate` to emit declarations importing from
`@icp-sdk/core/{agent,principal,candid}` instead of the three separate
`@dfinity/agent`, `@dfinity/candid`, `@dfinity/principal` packages.

Updated all affected examples:
- motoko/random_maze
- motoko/internet_identity_integration
- motoko/vetkd
- rust/face-recognition
- rust/image-classification
- rust/vetkd

Changes per example:
- Replace @dfinity/agent, @dfinity/candid, @dfinity/principal with
  @icp-sdk/core ~5.0.0 in package.json (keeping auth-client, identity,
  vetkeys as separate @dfinity packages where needed)
- Update direct source imports of @dfinity/agent and @dfinity/principal
  to @icp-sdk/core equivalents
- Update webpack.config.js canister env var format from
  <NAME>_CANISTER_ID to CANISTER_ID_<NAME> where applicable
- Regenerate declaration files with new @icp-sdk/core imports

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@lwshang lwshang marked this pull request as ready for review March 3, 2026 16:16
@lwshang lwshang requested review from a team as code owners March 3, 2026 16:16
Copy link
Contributor

@crusso crusso left a comment

Choose a reason for hiding this comment

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

I don't see any moc changes so here's a rubberstamp from us

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.

4 participants