Repository navigation
Conversation
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: trueComment |
Member
Author
|
Closing as superseded by #175, which adapted this branch's billing quote/confirmation and payment-required changes from 7514677 onto the unified session-configuration contract. The remaining diff against dev would reintroduce the retired direct session create method and |
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.
Funded session starts now surface a typed payment-required error without retrying the financial operation. Number-tier changes use a reviewed quote followed by explicit confirmation, preserve fractional credit quantities, and expose paid-window expiry. Session and client-rule methods use the current Platform routes instead of the removed Messaging paths.
Rebased on SDK dev a7db39e. The nine billing operations and sixteen Platform-control operations are pinned to API 0d62656002fe7274f4e86c47f334429d5f1d4800, in the API/Admin integration review: https://github.com/polymorfa/polymorfa/pull/164. The reviewed operation bodies were compared before repinning. This is a scoped integration checkpoint, not a full SDK parity or deployed-availability claim; the existing repository ledger still records 37 uncovered operations outside this work.
Validation: 415 tests, TypeScript typecheck, build, ESLint and formatting pass, including clean package imports and HTTP-boundary tests. Hosted CI and matching API release remain required.
API is pinned rather than changed here. SDK methods/types/errors and SDK documentation are updated. The CLI consumer review will pin this reachable commit and link back here. No production flags, deployment, npm release or pricing cutover occurs in this PR.
Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.