Skip to content

chore: migrate rust/query_stats to icp-cli#1400

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

chore: migrate rust/query_stats to icp-cli#1400
marc0olo wants to merge 1 commit into
masterfrom
chore/migrate-rust-query-stats-to-icp-cli

Conversation

@marc0olo

Copy link
Copy Markdown
Member

Summary

  • Replace dfx.json with icp.yaml using @dfinity/rust@v3.3.0 recipe (no package field, backend-only)
  • Move src/lib.rsbackend/lib.rs; restructure as workspace with backend/ member
  • Update ic-cdk to 0.20 and replace deprecated ic_cdk::api::management_canister with ic-cdk-management-canister = "0.1.1"
  • Add Makefile with test (fast API shape check) and test-stats (polling loop for non-zero stats) targets
  • Add rust-toolchain.toml with wasm32-unknown-unknown target
  • Update README.md with icp-cli prerequisites and preserve the query stats aggregation explanation from the Motoko variant
  • Add rust-query_stats job to .github/workflows/query_stats.yml using icp-dev-env-rust:1.0.0
  • Delete dfx.json, BUILD.md, query-stats.did, .devcontainer/

Test plan

  • icp network start -d && icp deploy && make test passes (API shape check)
  • make test-stats polls and verifies non-zero query stats after ~13 load queries
  • CI rust-query_stats job passes on push

🤖 Generated with Claude Code

- Replace dfx.json with icp.yaml using @dfinity/rust@v3.3.0 recipe
- Move src/lib.rs to backend/lib.rs; add workspace Cargo.toml
- Update ic-cdk to 0.20 and replace deprecated ic_cdk::api::management_canister
  with ic-cdk-management-canister = "0.1.1"
- Add Makefile with test and test-stats targets (fast API check + polling loop)
- Add rust-toolchain.toml with wasm32-unknown-unknown target
- Update README with icp-cli prerequisites and query stats explanation
- Add rust-query_stats job to .github/workflows/query_stats.yml
- Delete dfx.json, BUILD.md, query-stats.did, .devcontainer/

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