Skip to content

[AUD-59] Add SDK workflow rule runtime#6535

Draft
limdauto wants to merge 1 commit intomainfrom
lh/aud-59-sdk-workflow-rule-runtime
Draft

[AUD-59] Add SDK workflow rule runtime#6535
limdauto wants to merge 1 commit intomainfrom
lh/aud-59-sdk-workflow-rule-runtime

Conversation

@limdauto
Copy link
Copy Markdown
Contributor

@limdauto limdauto commented Mar 30, 2026

Summary

  • add internal workflow rule artifact models for SDK-delivered targeting rules
  • add a small native evaluator for the WeatherCat demo subset (and, ==, var)
  • add focused unit tests covering the demo rule bundle and matching behavior

Testing

  • xcodebuild build -workspace RevenueCat.xcworkspace -scheme RevenueCat -destination 'generic/platform=iOS Simulator'
  • xcodebuild test -workspace RevenueCat.xcworkspace -scheme RevenueCat -destination 'platform=iOS Simulator,id=75B6F762-E275-44AA-9762-32B5E62887DD' -only-testing:UnitTests/WorkflowRuleEvaluatorTests

@RevenueCat-Danger-Bot
Copy link
Copy Markdown

1 Error
🚫 Label the PR using one of the change type labels. If you are not sure which label to use, choose pr:other.
1 Warning
⚠️ RevenueCat.xcodeproj is out of sync.

The following Swift files were added but are missing from RevenueCat.xcodeproj:
Sources/Workflows/Rules/WorkflowRuleBundle.swift
Sources/Workflows/Rules/WorkflowRuleEvaluator.swift
Tests/UnitTests/Workflows/WorkflowRuleEvaluatorTests.swift

To fix: open RevenueCat.xcodeproj in Xcode, add/remove the files above in the appropriate target. Check where similar files in the same directory are assigned if you're unsure which target to use.

Label Description
pr:feat A new feature. Use along with pr:breaking to force a major release.
pr:fix A bug fix. Use along with pr:force_minor to force a minor release.
pr:other Other changes. Catch-all for anything that doesn't fit the above categories. Releases that only contain this label will not be released. Use along with pr:force_patch, or pr:force_minor to force a patch or minor release.
pr:RevenueCatUI Use along any other tag to mark a PR that only contains RevenueCatUI changes
pr:next_release Preparing a new release
pr:dependencies Updating a dependency
pr:phc_dependencies Updating purchases-hybrid-common dependency
pr:changelog_ignore The PR will not be included in the changelog. This label doesn't determine the type of bump of the version and must be combined with pr:feat, pr:fix or pr:other.

Generated by 🚫 Danger

@emerge-tools
Copy link
Copy Markdown

emerge-tools Bot commented Mar 31, 2026

📸 Snapshot Test

Base build not found

No build was found for the base commit e6cd64c. This is required to generate a snapshot diff for your pull request.

It's possible that you created a branch off the base commit before all of the CI steps have finished processing, e.g. the one that uploads a build to our system. If that's the case, no problem! Just wait and this will eventually resolve.


🛸 Powered by Emerge Tools

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.

2 participants