docs: add consolidated trust-model page (admin authority, key custody) #1492
ci.yml
on: pull_request
Lint & Typecheck
54s
Verify Vercel Build
36s
Test
1m 11s
Commit Messages
22s
PR Title
3s
Dependency Review
5s
E2E Tests
1m 10s
Soroban Contract Tests
39s
Annotations
1 error and 1 notice
|
Commit Messages
You have commit messages with errors
⧗ input: fix(docs): correct citation attribution and prettier formatting in trust-model.md
collinsezedike caught two things:
1. pnpm format:check failed: the page used single-asterisk markdown
italics (*word*) throughout; Prettier normalizes these to
underscores (_word_). Fixed with prettier --write.
2. The quoted sentence 'the ~1-day timelock ... is the only thing
standing between a leaked key and the vault's entire position moving
to an address the attacker controls' was attributed to
migrate_adapter's doc comment, but it's actually verbatim from
begin_migration's doc comment (lib.rs lines 781-783).
migrate_adapter's own doc comment makes a related but
differently-worded point (citing the same MIN_LEDGER_GAP mechanism,
and additionally naming the two concrete reaction options:
transfer_admin/accept_admin rotation, or pausing deposits). Fixed by
citing each doc comment separately and accurately, rather than
merging them under one attribution.
Verified: prettier --check passes, vitepress build succeeds, and
confirmed the corrected content (not a stray encoding artifact from my
own edit, which I introduced and caught before pushing) renders
correctly in the built HTML output.
✖ header must not be longer than 72 characters, current length is 81 [header-max-length]
✖ found 1 problems, 0 warnings
ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint
|
|
🎭 Playwright Run Summary
12 passed (18.3s)
|