Skip to content

feat(cli): billing top-up becomes add-credits, top-up stays as an alias - #7105

Closed
miguel-heygen wants to merge 3 commits into
mainfrom
feat/cli-billing-add-credits
Closed

miguel-heygen wants to merge 3 commits into
mainfrom
feat/cli-billing-add-credits

Conversation

@miguel-heygen

Copy link
Copy Markdown
Collaborator

What

hyperframes billing top-up is now hyperframes billing add-credits, HeyGen's product name: add-on credits are spent separately from the plan's monthly credits and can exceed it, so "top-up" was the wrong word. top-up still works as a hidden alias (not listed in help, no deprecation notice), so old docs and agents keep working.

Changes

  • billing: new action name and help text ("Buy add-on credits, used separately from your plan's monthly credits"); top-up maps to add-credits before anything else reads it.
  • The out-of-credits refusals now say "Buy add-on credits" and name billing add-credits.
  • The reverse-any-video and media-use skills, the media-use refusal message and the CLI docs use the new name. Changelog entries are history and stay as written.

Tests

  • billing.test.ts: add-credits and top-up both open the credits page, with nothing printed about a rename.
  • Fails without the fix: removing the alias mapping turns the top-up case red (1 failed, 2 passed); with it, 27 passed across billing.test.ts and v3.test.ts, and the media-use message test passes (20).

Size

A lone rename kept apart from the open sign-in PR on request; nothing else open can carry it.

@mintlify

mintlify Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated
hyperframes 🟢 Ready View Preview Nov 3, 2026, 8:23 PM

💡 Tip: Enable Automations to automatically generate PRs for you.

@heygengenesis heygengenesis Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verdict: APPROVE · 0 blocker · 0 important · 1 nit · head 6125e9d

  • nit: scripts/check-core-skills.test.mjs:40,49 still says "top up" / matches billing top-up --needed <total>; it passes only via the |--needed <total> branch. Worth renaming to add-credits so the pin tracks the new command.
Scope

Read the full diff (9 files). The alias maps top-up → add-credits before any branch reads action, so wait logic, page choice and the timeout message are unchanged. The top-up case in billing.test.ts would fail without the mapping (unknown action → console.error + failCommand). Grepped the tree at head: no remaining billing top-up outside changelogs/releases and the nit above. CI was still running when this was posted; the merge is held by the review gate.

@miguel-heygen
miguel-heygen force-pushed the feat/cli-billing-add-credits branch from 6125e9d to 80a1898 Compare November 3, 2026 20:19

@heygengenesis heygengenesis Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verdict: APPROVE — 0 resolved, 1 open (nit), 0 new · 6125e9d → 80a1898. The only patch change is start.md. Main's rewritten section was kept, its new top-up lines now say add-on credits, and nothing else in the tree still says billing top-up.

@heygengenesis heygengenesis Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verdict: APPROVE — 1 resolved, 0 open, 0 new · 80a1898 → 68bc59c. The core-skills test now pins billing add-credits --needed <total> and checks that top-up is absent; all 9 tests pass locally. The manifest hashes were regenerated, and the alias comment was dropped for the comment check.

@miguel-heygen

Copy link
Copy Markdown
Collaborator Author

Superseded: the add-credits rename merged inside #7068, which was stacked on it.

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