Skip to content

[docs] add missing pattern entries for account_based and key_request#465

Open
Danny-Devs wants to merge 1 commit intoMystenLabs:mainfrom
Danny-Devs:danny/docs-missing-patterns
Open

[docs] add missing pattern entries for account_based and key_request#465
Danny-Devs wants to merge 1 commit intoMystenLabs:mainfrom
Danny-Devs:danny/docs-missing-patterns

Conversation

@Danny-Devs
Copy link

@Danny-Devs Danny-Devs commented Feb 7, 2026

Description

ExamplePatterns.md documents 5 of the 7 access patterns in move/patterns/sources/, but account_based and key_request are missing. This adds both entries following the existing style (use-case-first, single paragraph, link to Move source).

  • account_based — stateless, address-derived encryption for messaging and notifications
  • key_request — two-step pattern that separates policy evaluation from the Seal approval step

Placement: account_based is inserted between Private data and Allowlist to follow a complexity gradient (single owner → specific address → group). key_request is placed last because it's a meta-pattern that wraps other patterns rather than defining its own access policy.

Also fixes "ay" → "at" in CONTRIBUTING.md line 27.

Written with the help of Claude Code.

Test plan

  • Verified both new sections render correctly with mkdocs serve
  • Cross-checked descriptions against Move source code
  • Docs only, no code changes

@Danny-Devs Danny-Devs changed the title [docs] Add missing pattern documentation for account_based and key_request [docs] add missing pattern entries for account_based and key_request Feb 7, 2026
@Danny-Devs Danny-Devs force-pushed the danny/docs-missing-patterns branch from da23c6b to e3bb408 Compare February 20, 2026 22:18
@Danny-Devs
Copy link
Author

Updated to target the new docs/content/ExamplePatterns.mdx after the mkDocs → Docusaurus migration in #468. Same two entries, matched to the new file format.

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