Skip to content

chore: migrate motoko/threshold-ecdsa to icp-cli#1368

Merged
marc0olo merged 12 commits into
masterfrom
chore/migrate-threshold-ecdsa-to-icp-cli
Jun 12, 2026
Merged

chore: migrate motoko/threshold-ecdsa to icp-cli#1368
marc0olo merged 12 commits into
masterfrom
chore/migrate-threshold-ecdsa-to-icp-cli

Conversation

@marc0olo

@marc0olo marc0olo commented Jun 10, 2026

Copy link
Copy Markdown
Member

Summary

Migrates motoko/threshold-ecdsa from dfx to icp-cli.

  • Replace dfx.json with icp.yaml (@dfinity/motoko@v5.0.0)
  • Move src/ecdsa_example_motoko/backend/; rename utility modules to backend/Hex.mo, backend/SHA256.mo
  • Replace inline IC actor type with import { ic } "mo:ic"
  • Update mops.toml: moc 1.9.0, core 2.5.0, ic 4.0.0; apply mops check --fix
  • Add Makefile with 3 tests (format checks + secp256k1 cryptographic verification via test.sh)
  • Add test.sh with secp256k1 ecdsaVerify end-to-end signature check; add package.json
  • Remove dfx_test_key references
  • CI workflow (icp-dev-env-motoko:0.3.2)
  • README: link to chain-key cryptography concept page; remove outdated building-apps URL; remove inapplicable "Updating the Candid interface" section (backend-only, no frontend)

Test plan

  • motoko-threshold-ecdsa CI passes

🤖 Generated with Claude Code

marc0olo and others added 12 commits June 10, 2026 17:17
Replace dfx-based tooling with icp-cli: swap dfx.json for icp.yaml,
move source files to backend/ layout, replace inline IC actor type with
import { ic } "mo:ic" (v4.0.0), update mops.toml to moc 1.9.0 / core
2.5.0, add Makefile test target, add CI workflow, and remove dfx
artifacts (BUILD.md, .devcontainer, package.json, test.sh).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…frontend)

No backend.did is committed and no frontend needs bindings, so this
section is not applicable.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…ng-apps URL

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@marc0olo marc0olo marked this pull request as ready for review June 11, 2026 20:32
@marc0olo marc0olo requested review from a team as code owners June 11, 2026 20:32
@marc0olo marc0olo merged commit ec5bd21 into master Jun 12, 2026
6 checks passed
@marc0olo marc0olo deleted the chore/migrate-threshold-ecdsa-to-icp-cli branch June 12, 2026 10:45
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.

3 participants