Conversation
…riptions, and dfx references - Separate compatibility (env tools) from prerequisites (library deps) across all 16 skills - Move icp-cli and Node.js from prerequisites to compatibility where needed - Standardize Node.js >= 24 across all skills requiring it - Enrich all skill descriptions with "Use when..." / "Do NOT use for..." triggering clauses - Rename wallet skill to cycles-management (fixes misleading name) - Convert all dfx-format icp.yaml configs to proper recipe syntax - Replace all localhost:4943 references with localhost:8000 (icp-cli default) - Replace dfx commands with icp equivalents in wallet-integration and internet-identity - Fix internet-identity to use ic_env cookie pattern instead of legacy CANISTER_ID_* env vars - Fix evm-rpc config to use pre-built WASM pattern (validated against dfinity/examples#1310) - Remove dfx_test_key references from vetkd, recommend test_key_1 (works locally + mainnet) - Fix cycles-management CLI commands: --icp for mint, --cycles for create, canister status - Add icp-cli installation section - Remove stale canister_ids.json references (icp-cli uses .icp/data/mappings/)
raymondk
reviewed
Mar 9, 2026
- Lower Node.js requirement from >= 24 to >= 22 (active LTS) across all skills - Remove Node.js from compatibility where it's not a hard requirement (icp-cli, certified-variables, vetkd) — keep it only for frontend-focused skills (asset-canister, internet-identity, wallet-integration) - Remove redundant Node.js prerequisite from icp-cli (already in Installation)
raymondk
approved these changes
Mar 10, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
@dfinity/motoko@v4.1.0,@dfinity/rust@v3.2.0,@dfinity/asset-canister@v2.1.0).localhost:4943(dfx default) withlocalhost:8000(icp-cli default).dfx start→icp network start -din wallet-integration and internet-identity.CANISTER_ID_*env var pattern withic_envcookie /PUBLIC_CANISTER_ID:pattern.type: pull/type: customwithpre-builtWASM pattern (validated against feat: add icp-cli support to rust/evm_block_explorer examples#1310).dfx_test_keyreferences, recommendtest_key_1(works both locally and on mainnet).--icpfor mint,--cyclesfor create,canister statusinstead ofcanister info).All CLI commands verified against icp-cli source (clap Args structs). All 16 skills pass validation.
Supersedes #66 — those icp-cli prerequisite lines have been removed entirely (moved to the
compatibilityfrontmatter field).Closes #70
Related: #74 (wallet renamed + improved, redundancy question still open), #73 (vetkd key names fixed, full accuracy review still pending), #72 (sns-launch dfx moved to compatibility, content accuracy still pending)