feat(customers): rename wallet to balances - #168
Open
joshdholtz wants to merge 3 commits into
Open
Conversation
'wallet' was a name the CLI invented — the product is In-App Currency and nothing else calls this a wallet. The command is now 'rc customers balances' with 'wallet' kept as an alias for old scripts. The api layer methods follow their endpoint names (VirtualCurrencies*) instead of the CLI concept. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
… in help 'wallet' shows in help and schema output, so it isn't hidden; and the help now names the old Virtual Currency term so grepping for it still finds the command. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…esource list Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
rc customers walletused a name the CLI made up: the API endpoint isvirtual_currenciesand the product is called In-App Currency, so "wallet" appeared nowhere else. The command is nowrc customers balances, which says what you get without inventing a noun.walletstill works as an alias so nothing breaks.The help copy now says In-App Currency (the product was renamed from Virtual Currency), and the API-layer methods are named after their endpoints (
VirtualCurrencies*) instead of the CLI concept, per the two-layer rule.🤖 Generated with Claude Code
Note
Low Risk
User-facing rename with backward-compatible alias; same endpoints and behavior, no auth or data-path changes.
Overview
Renames the customer command from
rc customers wallettorc customers balances, aligning the CLI with the In-App Currency product name and thevirtual_currenciesAPI instead of the invented “wallet” noun.walletremains a cobra alias so existing scripts keep working.Help text and
docs/command-surface.mdare updated accordingly (including resource sidebar wording and build-order checklists). The API client methods are renamed to match endpoints—VirtualCurrencies,VirtualCurrenciesUpdateBalance, andVirtualCurrenciesTransaction—with the path test updated to call the new names.Reviewed by Cursor Bugbot for commit 467c759. Bugbot is set up for automated code reviews on this repo. Configure here.