Skip to content

chore: migrate motoko/classes to icp-cli#1385

Draft
marc0olo wants to merge 1 commit into
masterfrom
chore/migrate-classes-to-icp-cli
Draft

chore: migrate motoko/classes to icp-cli#1385
marc0olo wants to merge 1 commit into
masterfrom
chore/migrate-classes-to-icp-cli

Conversation

@marc0olo

Copy link
Copy Markdown
Member

Summary

  • Replaces dfx.json with icp.yaml using the @dfinity/motoko@v5.0.0 recipe
  • Moves source files: src/map/Map.mobackend/app.mo, src/map/Buckets.mobackend/Buckets.mo
  • Removes src/test/Test.mo (test canister); testing moved to Makefile via icp canister call
  • Updates mops.toml to moc = "1.9.0", core = "2.5.0", adds --default-persistent-actors flag
  • Removes persistent keyword from main actor Map (covered by --default-persistent-actors); keeps persistent on actor class Bucket (actor class sub-WASMs require explicit keyword)
  • Adds Makefile with test and topup targets; icp deploy --cycles 30t used because the example dynamically creates child Bucket canisters
  • Adds .github/workflows/classes.yml with icp-dev-env-motoko:0.3.2 container and --cycles 30t deploy
  • Updates README to icp-cli instructions with security best practices link
  • Deletes BUILD.md, dfx.json, and .devcontainer/

Test plan

  • icp network start -d && icp deploy --cycles 30t deploys the backend canister
  • make test passes all 4 tests (put/get, null for missing key, bucket distribution)
  • make topup works if cycles run out
  • CI workflow (classes.yml) runs on push/PR

🤖 Generated with Claude Code

Replaces dfx.json with icp.yaml, moves src/map/ → backend/ (Map.mo →
app.mo, Buckets.mo), updates mops.toml to moc 1.9.0 / core 2.5.0 with
--default-persistent-actors, adds Makefile with test/topup targets, adds
CI workflow with --cycles 30t (example creates child Bucket canisters),
updates README to icp-cli instructions, and removes BUILD.md, dfx.json,
and .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