Skip to content

fix(ios): gate Sign in with Apple off for v1 (avoid App Store 5.1.1(v))#205

Merged
oratis merged 1 commit into
mainfrom
fix/ios-hide-siwa-v1
Jul 1, 2026
Merged

fix(ios): gate Sign in with Apple off for v1 (avoid App Store 5.1.1(v))#205
oratis merged 1 commit into
mainfrom
fix/ios-hide-siwa-v1

Conversation

@oratis

@oratis oratis commented Jul 1, 2026

Copy link
Copy Markdown
Owner

SIWA = account creation ⇒ Apple requires in-app account deletion (Guideline 5.1.1(v)). The cloud is still single-tenant M0 (SIWA isn't per-user yet), so it's not worth the review risk for the first submission.

  • Gate the SIWA button + handler (SettingsView + OnboardingFlow) behind #if LISA_ENABLE_SIWA (undefined ⇒ compiled out). The paste-cloud-URL (?token=) path — what the reviewer uses — stays.
  • Drop the com.apple.developer.applesignin entitlement from project.yml.

Re-enable via LISA_ENABLE_SIWA in SWIFT_ACTIVE_COMPILATION_CONDITIONS once C3 (per-uid) + account deletion ship.

Verified: build.sh BUILD SUCCEEDED (SIWA excluded); generated entitlements have no applesignin.

🤖 Generated with Claude Code

Offering Sign in with Apple = account creation, which requires in-app account
deletion (Guideline 5.1.1(v)) — and the cloud is still single-tenant M0 (SIWA
isn't per-user yet), so it's not worth the review risk for the first submission.

- Gate the SIWA button + handler in SettingsView and OnboardingFlow behind
  `#if LISA_ENABLE_SIWA` (undefined ⇒ compiled out). The "paste your cloud URL
  (with ?token=)" path stays as the cloud connect flow the reviewer uses.
- Drop the `com.apple.developer.applesignin` entitlement from project.yml so the
  binary declares no SIWA capability.

Re-enable (button + handler + entitlement) by adding LISA_ENABLE_SIWA to
SWIFT_ACTIVE_COMPILATION_CONDITIONS once C3 (per-uid) + account deletion ship.

Verified: build.sh BUILD SUCCEEDED (SIWA excluded); generated entitlements have
no applesignin key.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@oratis oratis merged commit b75dcbf into main Jul 1, 2026
1 check passed
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