Skip to content

wallet-api: rename STRK20 sub-accounts to shadow accounts - #406

Merged
odednaor merged 1 commit into
masterfrom
odedn/rename-subaccount-to-shadow-account
Aug 11, 2026
Merged

odednaor merged 1 commit into
masterfrom
odedn/rename-subaccount-to-shadow-account

Conversation

@odednaor

@odednaor odednaor commented Aug 11, 2026 •

Copy link
Copy Markdown
Collaborator

Description

Renames the STRK20 sub-account concept to shadow account across the wallet API spec. Terminology only — no semantic or structural change.

Breaking (wire-visible)

Before After
wallet_strk20SubaccountCommitment wallet_strk20ShadowAccountCommitment
STRK20_SUBACCOUNT_INVOKE_ACTION STRK20_SHADOW_ACCOUNT_INVOKE_ACTION
"subaccount_invoke" (action discriminator) "shadow_account_invoke"

The remaining 16 changed lines are summaries, descriptions and titles.

Checklist:

  • Validated the specification files - npm run validate_all
  • Applied formatting - npm run format
  • Performed code self-review
  • Checked if this PR resolves any issues
  • If making a new release, check out the guidelines

🤖 Generated with Claude Code

Renames the STRK20 sub-account concept to "shadow account" throughout the
wallet API spec. Terminology only - no semantic or structural change.

Breaking (wire-visible):
- method wallet_strk20SubaccountCommitment -> wallet_strk20ShadowAccountCommitment
- schema STRK20_SUBACCOUNT_INVOKE_ACTION -> STRK20_SHADOW_ACCOUNT_INVOKE_ACTION
- action discriminator "subaccount_invoke" -> "shadow_account_invoke"

The remaining changes are summaries, descriptions and titles. Prose uses
"shadow account" unhyphenated, since "shadow" is an adjective rather than a
prefix like "sub-".

Sub-accounts were introduced after v0.10.3 and exist only in the unreleased
v0.10.4-rc.0, so no deprecation alias is needed. Version left untouched here;
bumps are made in separate repo-wide PRs.

Note: this also renames the "sub-account anonymizer" to "shadow account
anonymizer" in two descriptions. That names an on-chain contract defined
outside this repo and needs confirmation from its owner.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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