Reformat XLS-65 specification by Tapanito · Pull Request #463 · XRPLF/XRPL-Standards

Tapanito · 2026-02-11T15:08:33Z

High Level Overview of Change

This PR restructures XLS-65 specfication, without functional changes.

Context of Change

Type of Change

New XLS Draft
XLS Update (changes to an existing XLS)
XLS Status Change (e.g., Draft → Final, Draft → Stagnant)
Process/Meta (changes to CONTRIBUTING.md, XLS-1, templates, etc.)
Infrastructure (CI, workflows, scripts, website)
Documentation (README updates, typo fixes)

Tapanito · 2026-02-11T15:10:58Z

The remaining validation error is due to a missing Security section. It's out of scope for this PR to add such a section.

…tion - Add Example JSON sections for Vault ledger entry and all transactions (VaultCreate, VaultSet, VaultDelete, VaultDeposit, VaultWithdraw, VaultClawback, Payment) with real transaction data - Add invariants for the Vault ledger entry (universal checks) and all transaction types derived from the ValidVault invariant checker - Restructure section 10 from "API" to "RPC: vault_info" matching the amendment template format with Request Fields, Response Fields, Failure Conditions, Example Request, and Example Response subsections - Update response fields table with missing fields (Data, Asset.mpt_issuance_id, shares.DomainID, shares.MPTokenMetadata) and correct Always Present values - Update response examples to use proper JSON format with response envelope - Add section 9.1 Fields for Payment transaction - Remove Index section and all Return to Index links Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

- Reorganize top-level sections: Abstract (1), Introduction (2), Specification (3), Rationale (4), Security Considerations (5), Appendix - Move all ledger entry, transaction, and RPC sections under "3. Specification" as subsections (3.1-3.9) - Remove "1.1 Overview" heading, merge content into Introduction body - Renumber Introduction subsections: Terminology (2.1), Actors (2.2), Connecting to the Vault (2.3) - Demote all specification headings by one level with new numbering - Add Rationale section explaining decoupled vault design - Rename FAQ section to "Appendix A: FAQ" with A.x numbering - Fix heading levels for Key Variables and Vault State Update Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Tapanito · 2026-02-24T10:43:53Z

@mvadari , I completely restructured the specification to match the template.

mvadari · 2026-02-24T21:06:17Z


 A protocol connecting to a Vault must track its debt. Furthermore, the updates to the Vault state when funds are removed or added back must be handled in the transactors of the protocol. For an example, please refer to the [Lending Protocol](https://github.com/XRPLF/XRPL-Standards/discussions/190) specification.

-[**Return to Index**](#index)
+## 3. Specification


If you remove the Specification heading and move everything out by one level, you won't run out of Markdown heading depth.
(P.S. AI handles this sort of direction well)

Nope, can't do that. It's part of the mandated spec. format :)

The opposite is part of the mandated spec format :)

I don't see what to fix here. Specification is a mandated section in the XLS_TEMPLATE.md

Tapanito · 2026-02-25T15:52:29Z

Actually, @mvadari, it makes no sense to add error codes in a separate PR. I'm adding them here.

mvadari · 2026-05-11T21:08:23Z


 The key of the `Vault` object is the result of [`SHA512-Half`](https://xrpl.org/docs/references/protocol/data-types/basic-data-types/#hashes) of the following values concatenated in order:

 - The `Vault` space key `0x0056` (capital V)
- The [`AccountID`](https://xrpl.org/docs/references/protocol/binary-format/#accountid-fields) of the account submitting the `VaultSet`transaction, i.e.`VaultOwner`.
+- The [`ACCOUNTID`](https://xrpl.org/docs/references/protocol/binary-format/#accountid-fields) of the account submitting the `VaultSet`transaction, i.e.`VaultOwner`.


Case should be changed back here too

mvadari · 2026-05-11T21:09:37Z

+| `Asset`            |   Yes    | `string or object` |    `ISSUE`    |          `N/A`           | The asset (`XRP`, `IOU` or `MPT`) of the Vault.                                                                                                        |
+| `AssetsMaximum`    |    No    |      `number`      |   `NUMBER`    |            0             | The maximum asset amount that can be held in a vault.                                                                                                  |
+| `MPTokenMetadata`  |    No    |      `string`      |    `BLOB`     |                          | Arbitrary metadata about the share `MPT`, in hex format, limited to 1024 bytes.                                                                        |
+| `WithdrawalPolicy` |    No    |      `number`      |    `UINT8`    | `strFirstComeFirstServe` | Indicates the withdrawal strategy used by the Vault.                                                                                                   |


Is there supposed to be a str prefix?

mvadari · 2026-05-12T15:12:48Z

@@ -287,14 +256,14 @@ Because the share amount is rounded down, the actual assets taken from the depos

 $$\Delta_{assets'} = \frac{\Delta_{shares} \times \Gamma_{assets}}{\Gamma_{shares}}$$

-#### Vault State Update
+##### Vault State Update


This needs a number

Tapanito added 4 commits February 11, 2026 15:15

reformats failure and state changes to use numerical indexes

a4d1d0b

re-indexes transaction sections

5d8a3a5

adds fields section to each tx

82cd4e4

minor formattign changes

22edf42

Tapanito requested a review from mvadari February 11, 2026 15:10

Tapanito added 4 commits February 11, 2026 16:22

moves header levels left by one

2922ade

renames column names and emojis to strings

da9a789

formatting

866caa4

Merge branch 'master' into tapanito/vault-enhanced

9ae1b15

Tapanito changed the title ~~Tapanito/vault enhanced~~ Reformat XLS-65 specification Feb 19, 2026

Tapanito added 2 commits February 19, 2026 10:47

Merge branch 'master' into tapanito/vault-enhanced

17068a6

Merge branch 'master' into tapanito/vault-enhanced

3ce8514

mvadari reviewed Feb 23, 2026

View reviewed changes

Comment thread XLS-0065-single-asset-vault/README.md Outdated

mvadari reviewed Feb 23, 2026

View reviewed changes

Comment thread XLS-0065-single-asset-vault/README.md Outdated

mvadari reviewed Feb 23, 2026

View reviewed changes

Comment thread XLS-0065-single-asset-vault/README.md Outdated

Tapanito and others added 4 commits February 24, 2026 10:27

addressees review comments

6b30e85

adds example JON

d8381d6

mvadari reviewed Feb 24, 2026

View reviewed changes

Comment thread XLS-0065-single-asset-vault/README.md Outdated

mvadari reviewed Feb 24, 2026

View reviewed changes

Comment thread XLS-0065-single-asset-vault/README.md Outdated

mvadari reviewed Feb 24, 2026

View reviewed changes

Comment thread XLS-0065-single-asset-vault/README.md

mvadari reviewed Feb 24, 2026

View reviewed changes

Comment thread XLS-0065-single-asset-vault/README.md Outdated

mvadari reviewed Feb 24, 2026

View reviewed changes

Comment thread XLS-0065-single-asset-vault/README.md Outdated

mvadari reviewed Feb 24, 2026

View reviewed changes

This was referenced Feb 24, 2026

adds VaultDepositBlock flag #469

Open

Add Data field to VaultDelete #470

Open

Merge branch 'master' into tapanito/vault-enhanced

3a1afbc

Tapanito added 2 commits February 25, 2026 14:15

addresses review feedback

29d603e

formatting

2ec8b80

adds error codes to failure conditions

72b7ab3

mvadari self-requested a review February 26, 2026 02:55

Tapanito added 2 commits February 26, 2026 14:33

Merge branch 'master' into tapanito/vault-enhanced

b8d7999

Merge branch 'master' into tapanito/vault-enhanced

c2fc663

mvadari reviewed Mar 16, 2026

View reviewed changes

Comment thread XLS-0065-single-asset-vault/README.md Outdated

mvadari reviewed Mar 16, 2026

View reviewed changes

Comment thread XLS-0065-single-asset-vault/README.md Outdated

mvadari reviewed Mar 16, 2026

View reviewed changes

Comment thread XLS-0065-single-asset-vault/README.md Outdated

mvadari reviewed Mar 17, 2026

View reviewed changes

Comment thread XLS-0065-single-asset-vault/README.md Outdated

Tapanito added 3 commits April 27, 2026 11:55

address review comments

e3ce4a2

clean up

746d4f1

address PR review

f2dbe60

mvadari reviewed May 11, 2026

View reviewed changes

mvadari reviewed May 12, 2026

View reviewed changes

Conversation

Tapanito commented Feb 11, 2026

High Level Overview of Change

Context of Change

Type of Change

Uh oh!

Tapanito commented Feb 11, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Tapanito commented Feb 24, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

mvadari Feb 24, 2026

Choose a reason for hiding this comment

Uh oh!

Tapanito Feb 25, 2026

Choose a reason for hiding this comment

Uh oh!

mvadari Mar 16, 2026

Choose a reason for hiding this comment

Uh oh!

Tapanito Apr 27, 2026

Choose a reason for hiding this comment

Uh oh!

Tapanito commented Feb 25, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

mvadari May 11, 2026

Choose a reason for hiding this comment

Uh oh!

mvadari May 11, 2026

Choose a reason for hiding this comment

Uh oh!

mvadari May 12, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants