Skip to content

chore: migrate rust/canister-info to icp-cli#1415

Draft
marc0olo wants to merge 1 commit into
masterfrom
chore/migrate-rust-canister-info-to-icp-cli
Draft

chore: migrate rust/canister-info to icp-cli#1415
marc0olo wants to merge 1 commit into
masterfrom
chore/migrate-rust-canister-info-to-icp-cli

Conversation

@marc0olo

Copy link
Copy Markdown
Member

Summary

  • Replaces dfx.json with icp.yaml using @dfinity/rust@v3.3.0 recipe
  • Moves src/lib.rsbackend/lib.rs; converts root Cargo.toml to a workspace manifest pointing at backend/
  • Upgrades from the removed ic_cdk::api::management_canister API (ic-cdk 0.13) to ic-cdk-management-canister = "0.1.1" — adapts to renamed types (CanisterInfoRequestCanisterInfoArgs, CanisterInfoResponseCanisterInfoResult, CanisterChangeChange) and the now-optional Change::details field
  • Rewrites Makefile with five numbered tests calling all public functions via icp canister call, using icp canister status backend -i to resolve the backend's own canister ID
  • Adds CI workflow at .github/workflows/canister-info.yml with job rust-canister-info using ghcr.io/dfinity/icp-dev-env-rust:1.0.0
  • Updates README.md to icp-cli prerequisites and deploy instructions; preserves the link to the IC Interface Specification
  • Deletes dfx.json, BUILD.md, canister-info.did, test.did, test.wasm, .devcontainer/

Test plan

  • CI job rust-canister-info passes (icp network start -d && icp deploy && make test)
  • All five Makefile tests pass: info, reflexive_transitive_controllers, canister_controllers, canister_module_hash, canister_deployment_chain
  • cargo check --target wasm32-unknown-unknown passes locally (verified)

🤖 Generated with Claude Code

Replaces dfx.json with icp.yaml, moves src/ → backend/, updates
Cargo.toml to a workspace root, upgrades to ic-cdk-management-canister
0.1.1 (replacing the removed ic_cdk::api::management_canister API),
rewrites Makefile and README for icp-cli, and adds CI workflow.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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.

1 participant