Skip to content

chore: bump source submodules and update docs for icp-cli v1.0.0#297

Open
marc0olo wants to merge 3 commits into
mainfrom
infra/bump-source-submodules
Open

chore: bump source submodules and update docs for icp-cli v1.0.0#297
marc0olo wants to merge 3 commits into
mainfrom
infra/bump-source-submodules

Conversation

@marc0olo

@marc0olo marc0olo commented Jun 17, 2026

Copy link
Copy Markdown
Member

Summary

Bumps pinned source submodules and updates docs to match. Driven by the icp-cli v1.0.0 release.

Submodule bumps

  • icp-cli v0.3.1 → v1.0.0 (.sources/VERSIONS updated)
  • icp-cli-recipes, icp-cli-templates, icskills → latest main
  • icp-js-sdk-docs already at tip

icp-cli v1.0.0

  • --set-controller was removed; settings.mdx now uses --remove-all-controllers combined with --add-controller.

Motoko recipe v5.0.0

The recipe now compiles via mops build, so the source file, Candid file, and compiler args move from the recipe configuration: block into mops.toml. Updated in project-structure.mdx, candid.mdx, optimization.md, and ethereum.mdx. The GC-options example now shows a real legacy-persistence alternative (--legacy-persistence + --compacting-gc) rather than the now-default --incremental-gc.

Latest recipes

  • Rust recipe pins → v3.3.0; redundant package dropped where it equals the canister name (v3.3.0 defaults it), with an explanatory note kept in the getting-started page.
  • asset-canister v2.2.1 and prebuilt v2.0.0 already current.

Mainnet domain → icp.net / id.ai

The icp-cli mainnet HTTP gateway domain is now icp.net (was icp0.io). Browser/canister-access URLs, raw.* domains, the custom-domains/v1/ API, live canister links, and the NNS app (nns.ic0.appnns.icp.net) were updated across the docs (all verified resolving). The Internet Identity service link now points to its canonical https://id.ai.

Deliberately left unchanged (these are not the gateway):

  • JS HttpAgent / agent API hosts — the API endpoint is icp-api.io/icp0.io, separate from the gateway.
  • The Internet Identity icp0.ioic0.app delegation-rewriting behavior.
  • The verifiable-credentials iss (issuer) claim — a protocol identifier that must match what II signs (already flagged for human verification).

Reconciling the icskills to icp.net is tracked in dfinity/icskills#224.

Other

  • Fixed a pre-existing broken link in edge-infrastructure.md (http-gateway-spec.mdhttp-gateway-protocol-spec.md) surfaced by validation, plus minor lint cleanup (em-dash, mo:base reference) in files touched here.

Notes

  • npm run build and scripts/validate.js both pass (209 pages).
  • Changes were reviewed by a subagent against .sources/ with no must-fix findings.
  • ⚠️ Possible merge conflict with feat: canister logs extensions #252 (also edits settings.mdx, in a different section).

- Bump icp-cli v0.3.1 -> v1.0.0; icp-cli-recipes, icp-cli-templates, icskills to latest main
- Replace removed --set-controller with --remove-all-controllers/--add-controller
- Migrate Motoko recipe examples to v5.0.0 (mops build / mops.toml config)
- Move mainnet HTTP gateway domain icp0.io/ic0.app -> icp.net across docs (keep API hosts on icp-api.io/icp0.io, named dapps, and II domain-rewriting behavior)
- Advertise Rust recipe v3.3.0 with optional package parameter
@marc0olo marc0olo requested review from a team as code owners June 17, 2026 17:36
@github-actions

github-actions Bot commented Jun 17, 2026

Copy link
Copy Markdown

🤖 Here's your preview: https://uc7il-jaaaa-aaaam-ai2gq-cai.icp0.io

marc0olo added 2 commits June 17, 2026 20:51
…o NNS

- Fix broken http-gateway-protocol-spec link in edge-infrastructure.md (validation failure)
- Motoko GC example: use a real legacy-persistence alternative (--legacy-persistence + --compacting-gc) instead of the now-default --incremental-gc
- Update NNS app references nns.ic0.app -> nns.icp.net (verified serving)
- Clean up pre-existing lint in touched files (mo:base prose reference, em-dash in upstream comment)
identity.ic0.app -> id.ai (canonical; identity.icp.net redirects there). VC issuer (iss) claim values left unchanged as protocol identifiers (already flagged for human verification).
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.

2 participants