Skip to content

docs: Batch 8 — tokens, ledgers, and cycles concepts#215

Open
marc0olo wants to merge 2 commits intoinfra/learn-hub-migration-prepfrom
docs/concepts-tokens-ledgers
Open

docs: Batch 8 — tokens, ledgers, and cycles concepts#215
marc0olo wants to merge 2 commits intoinfra/learn-hub-migration-prepfrom
docs/concepts-tokens-ledgers

Conversation

@marc0olo
Copy link
Copy Markdown
Member

@marc0olo marc0olo commented May 6, 2026

Summary

  • New page: docs/concepts/tokenomics.md — ICP token uses (governance, cycles, node rewards, SNS swaps), voting rewards and maturity mechanism (10%→5% over 8 years), supply dynamics (inflation/deflation), SNS tokenomics configuration
  • New page: docs/concepts/token-ledgers.md — ICP ledger (AccountIdentifier), ICRC ledgers (principal + subaccount), append-only transaction log, archive canisters, index canisters, fee models
  • Expanded: docs/concepts/cycles.md — added "How charging works" (memory at intervals, computation per-instruction with DTS, messaging prepayment + refund, special features) and "Cycles ledger" section (canister IDs, ICRC-1/2/3, deposit/withdraw/create_canister, 100M fee per block, replaces cycles wallets)
  • Updated: docs/concepts/governance.md — replaced external Learn Hub link with internal tokenomics.md link
  • Updated: docs/concepts/index.md — added tokenomics and token-ledgers entries; fixed chain-fusion link to subdirectory
  • Deleted: 6 consumed Learn Hub staging files from .migration/learn-hub/

Sync recommendation

tokenomics.md: informed by learn-hub staging — tokens-governance/tokens-governance.md, tokens-governance/tokenomics.md, sns/tokenomics.md
token-ledgers.md: informed by learn-hub staging — tokens-governance/how-token-ledgers-work-on-the-internet-computer.md
cycles.md additions: informed by learn-hub staging — canister-smart-contracts/cycles.md, canister-smart-contracts/cycles-ledger.md

- Add tokenomics.md: ICP token uses, governance rewards, supply dynamics, SNS tokenomics
- Add token-ledgers.md: ICP ledger, ICRC ledgers, archives, index canisters, fees
- Expand cycles.md: add "How charging works" and "Cycles ledger" sections
- Update governance.md: replace Learn Hub link with internal tokenomics.md link
- Update concepts/index.md: add tokenomics and token-ledgers entries; fix chain-fusion link
- Remove 6 consumed Learn Hub staging files
@marc0olo marc0olo requested a review from a team as a code owner May 6, 2026 15:21
@marc0olo
Copy link
Copy Markdown
Member Author

marc0olo commented May 6, 2026

Review notes

Two blocking issues and one minor fix needed:

1. Broken links (blocking, dependency on #213): Two links point into concepts/chain-fusion/ which only exists once PR #213 merges:

  • index.md links to chain-fusion/index.md
  • token-ledgers.md links to chain-fusion/chain-key-tokens.md

Options: batch-merge after #213, or temporarily link to the flat concepts/chain-fusion.md that currently exists on main.

2. Em-dash in token-ledgers.md (banned per CLAUDE.md): The AccountIdentifier definition uses an em-dash: **AccountIdentifier** — a 32-byte hash.... Replace with a colon: **AccountIdentifier**: a 32-byte hash....

3. Minor: guides/digital-assets/ledgers.md is actually ledgers.mdx. Astro resolves .md links to .mdx files correctly, so this is not a hard error, but worth noting for consistency.

No other issues: frontmatter complete on all new pages, governance.md and cycles.md cross-links correct, <!-- Upstream: --> comment present.

@marc0olo
Copy link
Copy Markdown
Member Author

marc0olo commented May 6, 2026

Three issues to address before merging:

1. Em-dash in token-ledgers.md line 29

Em-dashes are banned project-wide (CLAUDE.md). Please replace:

**AccountIdentifier** — a 32-byte hash derived from...

with:

**AccountIdentifier**: a 32-byte hash derived from...

2. Broken link in cycles.md — "How charging works" section

The new section links to protocol/execution.md#deterministic-time-slicing, but docs/concepts/protocol/ does not exist yet (it is added by PR #209). Until that PR merges, this link will be broken. Two options:

3. Chain-fusion link dependencies in concepts/index.md and token-ledgers.md

concepts/index.md links to chain-fusion/index.md, and token-ledgers.md links to chain-fusion/chain-key-tokens.md (twice). Neither path exists until PR #213 merges. Two options:

Suggested merge order: #209#213#215.

…ers pages

- Fix AccountIdentifier em-dash in token-ledgers.md
- Revert chain-fusion/ subdirectory links to chain-fusion.md (flat page on main)
  until PR #213 merges; affects token-ledgers.md and concepts/index.md
- Remove broken link to protocol/execution.md#deterministic-time-slicing
  in cycles.md until PR #209 merges
@marc0olo
Copy link
Copy Markdown
Member Author

marc0olo commented May 6, 2026

Feedback addressed:

  • Fixed **AccountIdentifier** — **AccountIdentifier**: in token-ledgers.md line 29
  • Reverted chain-fusion/chain-key-tokens.md links to chain-fusion.md (the flat page on main) in token-ledgers.md (lines 21, 71) and concepts/index.md (line 27) — will update to the proper subdirectory paths after PR docs: Chain Fusion deep dives #213 merges
  • Removed broken link to protocol/execution.md#deterministic-time-slicing in cycles.md (plain text for now, link can be restored after PR docs: add protocol stack concept pages (Batch 1) #209 merges)

Suggested merge order: #209#213#215

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