Skip to content

Parse single_step_fallback_id in PublishedWorkflow#6733

Merged
facumenzella merged 2 commits into
mainfrom
worktree-effervescent-strolling-steele
May 5, 2026
Merged

Parse single_step_fallback_id in PublishedWorkflow#6733
facumenzella merged 2 commits into
mainfrom
worktree-effervescent-strolling-steele

Conversation

@facumenzella

@facumenzella facumenzella commented May 5, 2026

Copy link
Copy Markdown
Member

Summary

  • Adds singleStepFallbackId: String? to PublishedWorkflow to decode the new single_step_fallback_id field exposed by the backend
  • Field is optional and decoded automatically via JSONDecoder.default's .convertFromSnakeCase strategy
  • Adds test coverage for present and absent field cases

Backend PR

RevenueCat/khepri#20018 — exposes single_step_fallback_id in the workflow SDK response so clients can determine which step's default package to use as context for early packageless screens in a multipage paywall.

Test Plan

  • swift build passes
  • WorkflowResponseTests added for both present and nil cases
  • SwiftLint clean

Note

Low Risk
Low risk: adds an optional decoded field and unit tests, with no behavioral changes beyond exposing new backend data.

Overview
Adds optional singleStepFallbackId to PublishedWorkflow so the SDK decodes the backend’s single_step_fallback_id field.

Extends WorkflowResponseTests to assert the property is nil when absent and correctly populated when present.

Reviewed by Cursor Bugbot for commit 4bb4f5d. Bugbot is set up for automated code reviews on this repo. Configure here.

Adds `singleStepFallbackId: String?` to `PublishedWorkflow` to decode the
field exposed by the backend in RevenueCat/khepri#20018.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@facumenzella facumenzella requested a review from vegaro May 5, 2026 13:02
@facumenzella facumenzella marked this pull request as ready for review May 5, 2026 13:02
@facumenzella facumenzella requested a review from a team as a code owner May 5, 2026 13:02
@facumenzella facumenzella merged commit 79d837d into main May 5, 2026
16 of 18 checks passed
@facumenzella facumenzella deleted the worktree-effervescent-strolling-steele branch May 5, 2026 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants