Skip to content

Correct ledger header units and document the extension field - #2930

Merged
ElliotFriend merged 2 commits into
mainfrom
2907-ledger-header-units-and-ext
Nov 2, 2026
Merged

ElliotFriend merged 2 commits into
mainfrom
2907-ledger-header-units-and-ext

Conversation

@kaankacar

Copy link
Copy Markdown
Contributor

Closes #2907

What

  • docs/learn/fundamentals/stellar-data-structures/ledgers.mdx:
    • Total coins and fee pool: the header stores both as integers in stroops. Horizon shows them in XLM.
    • Base reserve: give the unit (stroops).
    • Add the header extension field (ext).

Verification

  • Stellar-ledger.x (stellar-xdr curr): totalCoins and baseReserve are in stroops. ext has arm 0 (void) and arm 1 (LedgerHeaderExtensionV1 with flags).
  • stellar-core adds the fee in stroops to feePool with no conversion.
  • @stellar/stellar-sdk 17.2.1 decoded the header_xdr of Mainnet ledger 65043333. Raw feePool and totalCoins equal Horizon fee_pool and total_coins times 10,000,000. Raw baseReserve is 5000000. The ext discriminant is 0.
  • CAP-38 defines the three liquidity pool flags and the LEDGER_UPGRADE_FLAGS vote for emergencies.
  • pnpm check:mdx, scripts/check-relative-links.sh and an MDX compile pass.

Notes

The ledgers page said the fee pool is in lumens. The header stores the
fee pool and total coins as integers in stroops. Horizon shows both in
XLM. The page now states these units and the base reserve unit. It also
adds the header extension field to the field list.

Closes #2907
@kaankacar

Copy link
Copy Markdown
Contributor Author

/triage

@github-actions github-actions Bot added the preview Preview builds for PRs by SDF employees. label Oct 21, 2026
@github-actions

Copy link
Copy Markdown
Contributor

@stellar-triage-bot stellar-triage-bot Bot added the bot:ready-to-merge Bot verified and approved; waiting for a maintainer to merge label Oct 21, 2026
@stellar-triage-bot

Copy link
Copy Markdown
Contributor

🤖 Automated message from the Stellar Triage Bot.

Reviewing because @kaankacar asked with /triage.

Verified: Stellar-ledger.x and a decoded Mainnet header (ledger 65061604) store totalCoins, feePool and baseReserve in stroops. CAP-38 matches the ext flags text. The preview renders the page. Ready to merge.

@mintlify

mintlify Bot commented Nov 2, 2026 •

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated
stellar-docs 🟢 Ready View Preview Nov 2, 2026, 6:23 PM

@ElliotFriend
ElliotFriend merged commit 378944a into main Nov 2, 2026
9 checks passed
@ElliotFriend
ElliotFriend deleted the 2907-ledger-header-units-and-ext branch November 2, 2026 18:27
kalepail added a commit to stellar-experimental/stellar-raven that referenced this pull request Nov 2, 2026
stellar/stellar-docs#2930 is deployed. Claude Fable 5.1 (high) re-derived
the fix live: RETIRE, no residual. Resolution comment posted on
stellar/stellar-docs#2907.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
kalepail added a commit to stellar-experimental/stellar-raven that referenced this pull request Nov 2, 2026
* improvements: mark sd-056 fixed-upstream after the live recheck

stellar/stellar-docs#2930 is deployed. Refs #316.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* improvements: retire sd-056 after the distinct review

stellar/stellar-docs#2930 is deployed. Claude Fable 5.1 (high) re-derived
the fix live: RETIRE, no residual. Resolution comment posted on
stellar/stellar-docs#2907.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>

This branch was successfully deployed

1 active deployment
staging — 3cc2062e Deployed Nov 2, 2026 by mintlify[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bot:ready-to-merge Bot verified and approved; waiting for a maintainer to merge preview Preview builds for PRs by SDF employees.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Correct ledger-header fee units and document its extension member

2 participants