fix(compliance): disclose email collection in Play Data Safety + align privacy docs (#146)
* fix(compliance): disclose email collection in Play Data Safety + align privacy docs (closes #143) Google Play rejected cc.agentlabs.opencode (2026-07-22) because the Data Safety declaration did not disclose collection of Email Address. Root cause: the optional "OpenCode Connect" waitlist card on the Connect screen (app/connection/add.tsx -> src/lib/waitlist.ts) collects an email and forwards it to Brevo (email marketing/CRM) via the beta-signup backend. Audited all other PII surfaces and confirmed no other undisclosed collection: Chatwoot support reports stay anonymous (no email/name), Sentry strips URLs/tokens and sends no default PII, and PostHog analytics uses only a random anonymous ID with coarse event properties. Updates: - distribution/play-listing.md: Data Safety table now declares Personal info / Email address (collected, shared with Brevo, optional, purpose account management); embedded privacy-policy draft and app description updated to match. - distribution/privacy-policy.md/.html + docs/privacy/index.html: new section 3c discloses the waitlist email collection, third-party services list adds Brevo, retention/rights sections and the Apple Privacy Nutrition Label table updated accordingly. - docs/playstore.md: checklist entry documents the rejection and points to the fix. - PUBLISHING.md: adds exact Play Console resubmission steps (Data types -> Personal info -> Email address -> collected/shared/purpose) plus a note on the earlier unrelated "Missing sign-in details" App access blocker in case it resurfaces. No app code changed; npm test (209 pass) and tsc --noEmit are clean. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> * fix(ci): run required checks on docs-only PRs (unblock branch protection) ios-ci.yml (which emits the required 'Typecheck and unit tests' check) had paths-ignore for docs/**, docs-site/**, distribution/**, **/*.md. A required status check that is path-filtered never runs on docs-only PRs, so those PRs sit permanently in mergeStateStatus=BLOCKED (missing required check). Remove the paths-ignore so required checks always run. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> --------- Co-authored-by: test <test@test.local> Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
This commit is contained in:
@@ -57,6 +57,57 @@ Production is **not** published by CI by default — the service account is scop
|
||||
- **Recommended — Play Console:** Production → Create release → **Add from library** → select the build by its **versionName** (e.g. `0.4.10`) and confirm its `versionCode` (the run_number-derived one, e.g. `142` — not the `app.json` number) → review → roll out. If the "What's new" field is empty, paste from `distribution/whatsnew/whatsnew-en-US`. No rebuild.
|
||||
- **Fully automated (optional):** grant the CI service account **"Release to production"** for this app in Play Console → Users & permissions, then run the workflow's `workflow_dispatch` with `track=production`, `status=completed`. **Without that permission the production dispatch fails with `The caller does not have permission` after building** — so don't dispatch `track=production` until the service account has been granted production access.
|
||||
|
||||
## Resubmitting after a Data Safety rejection (issue #143)
|
||||
|
||||
Google Play rejected `cc.agentlabs.opencode` on 2026-07-22 because the app's Data Safety
|
||||
declaration did not disclose collection of **Email Address**. Root cause: the "OpenCode
|
||||
Connect" waitlist card on the Connect screen (`app/connection/add.tsx` →
|
||||
`src/lib/waitlist.ts` → `POST https://opencode.agentlabs.cc/api/beta-signup`) collects an
|
||||
email address when a user opts in, and the backend forwards it to **Brevo** (email
|
||||
marketing/CRM). This was true collection that the Data Safety form did not declare — Play
|
||||
requires *all* personal-info collection to be declared, even when it's optional and
|
||||
unrelated to the app's core function.
|
||||
|
||||
The repo-side declaration is now fixed (this PR): `distribution/play-listing.md` Data Safety
|
||||
table, `distribution/privacy-policy.md`/`.html`, and `docs/privacy/index.html` all disclose
|
||||
the email collection. To resubmit:
|
||||
|
||||
1. **Play Console → your app → App content → Data safety → Manage**.
|
||||
2. Under **Data types → Personal info**, check **Email address**.
|
||||
- **Is this data collected, shared, or both?** → **Collected and shared**.
|
||||
- **Is this data processed ephemerally?** → No.
|
||||
- **Is data collection required for your app, or can users choose whether this data is
|
||||
collected?** → **Users can choose whether this data is collected** (optional — only
|
||||
collected if the user opts into the waitlist).
|
||||
- **Why is this user data collected?** → check **Account management** (the waitlist is a
|
||||
signup for the not-yet-launched hosted service). Optionally also check **App
|
||||
functionality** if Console requires at least one additional purpose.
|
||||
- Under sharing: declare it is shared with a third party (Brevo) for the same purpose.
|
||||
3. Re-verify the existing declared types are still accurate (unchanged by this fix):
|
||||
**App activity** (PostHog analytics), **App info and performance / Crash logs** (Sentry),
|
||||
and **Diagnostics — user-submitted reports** (Chatwoot) — all opt-in, default OFF, shared
|
||||
with the named third parties. See the full table in `distribution/play-listing.md` →
|
||||
"Data safety form".
|
||||
4. Confirm the **Privacy policy URL** field still points at
|
||||
`https://dzianisv.github.io/opencode-mobile/privacy/` (now updated with the email
|
||||
disclosure — `docs/privacy/index.html`, mirrored from `distribution/privacy-policy.md`).
|
||||
5. Save, then **Send for review** (Play re-reviews Data Safety changes; this is separate from
|
||||
a binary/release review since no code changed).
|
||||
6. Once Data Safety is approved, resume any blocked release rollout (e.g. 0.4.12 and pending
|
||||
retention fixes) — those builds themselves did not need to change, only the Console-side
|
||||
declaration.
|
||||
|
||||
**Known earlier blocker (if it resurfaces):** a prior release (v0.4.5) was blocked by Google
|
||||
with a "Missing sign-in details" rejection under **App access**, not Data Safety — Play
|
||||
reviewers could not exercise the app because it requires the user's own opencode server and
|
||||
Play had no way to sign in / connect one. That was resolved (see `HANDOFF.md`) by providing
|
||||
reviewer instructions plus a temporary demo server URL in the **App access** form (see the
|
||||
reviewer-instructions block in `distribution/play-listing.md` → "App access"). If a future
|
||||
review flags "sign-in details" again, the fix is the same: confirm **App access** still has
|
||||
either working temporary credentials/demo server or an accurate "all functionality available
|
||||
without sign-in" declaration — this is unrelated to the Data Safety fix in this PR, but is
|
||||
the other known rejection mode for this app and worth checking in the same Console pass.
|
||||
|
||||
## Fastlane (Alternative)
|
||||
|
||||
A Fastlane setup is included for local publishing:
|
||||
|
||||
Reference in New Issue
Block a user