docs: show how to keep identity calls under a registered personless setting by eli-r-ph · Pull Request #1053 · PostHog/posthog-ios

eli-r-ph · 2026-11-10T22:23:06Z

💡 Motivation and Context

In 4.0, a registered $process_person_profile: false takes effect, and it also reaches the events identify, alias, group and setPersonProperties send. PostHog drops those identity events. The migration guide didn't say this changed from 3.x, or show the per-call option that keeps them.

Event options: say a registered false had no effect in 3.x and applies in 4.0, also after identify and with personProfiles set to .always.
Event options: show how to keep identifying by passing process_person_profile: true on the call or in beforeSend, with an example, and point to personProfiles = .never for no person profiles at all.
Checklist: add apps that register $process_person_profile as false and still call identity methods.

💚 How did you test it?

Docs-only. Rendered the changed list item with GitHub's markdown API (the code block stays inside the bullet), checked the #event-options link resolves, ran git diff --check, and checked the example against the identify(_:userProperties:userPropertiesSetOnce:options:) signature.

📝 Checklist

All commits are signed (see Commit signing)
I reviewed the submitted code.
I added tests to verify the changes.
I updated the docs if needed.
No breaking change or entry added to the changelog.

If releasing new changes

Ran pnpm changeset to generate a changeset file

No changeset: this only changes MIGRATION.md, which isn't released.

🤖 Agent context

Autonomy: Human-driven (agent-assisted)

Written with Claude Code (Claude Opus 5.5). The guide text was drafted and approved before it was applied as written.

🤖 Generated with Claude Code

…etting Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

veria-ai · 2026-11-10T22:33:16Z

PR overview

No security issues were found in this pull request.

Reviews

4a00fba — No new issues

turnipdabeets

We pushed a couple of additions in d46fcbe: a pointer to .identifiedOnly (the default) for apps that registered false to skip person profiles for anonymous users, and a line that events sent with false after identify are still linked to the person and billed as identified. Feel free to tweak or revert. Approving, thanks for picking these up!

…lling

…on guide shows the identity call option #146 at 15501ca requires migration guides to show a per-call process_person_profile: true on identity calls; MIGRATION.md at e27cbe4cb lacks it until PostHog/posthog-ios#1053 merges. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

The identity-call migration-guide example #146 requires lands in 4.0 with PostHog/posthog-ios#1053, which targets the release branch. This audit tracks 4.0, so a pending release-branch PR is noted rather than scored as a gap. Reverts 85d4acd. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* chore(compliance): audit posthog-ios 4.0 Re-audit all 64 posthog-ios rows at 3460e534a (4.0 release branch) against sdk-specs dd11a6b, add the five rows added to the README table since the last audit, and update the iOS roll-up and gap entries. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * fix(compliance): score posthog-ios HTTP Client partial compression = .none leaves captureAi requests gzipped through the separate captureAiCompression setting, so iOS 4.0 has no whole-client compression opt-out. Roll-up becomes 40/17/1/11 (58%), 18 gaps. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * docs(compliance): list iOS capture v1 retries as matching #146 Draft #146 already permits the client-SDK durable queue, unlimited attempts, 1-30 s backoff and multi-event 413 halving for iOS 4.0. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * fix(compliance): score posthog-ios Session Replay Privacy partial SwiftUI's postHogMask() wins over an enclosing postHogNoMask(), the reverse of the spec's unmask-over-mask precedence. It over-masks rather than leaks. Roll-up becomes 39/18/1/11 (57%), 19 gaps. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * docs(compliance): re-check posthog-ios 4.0 at 1f91c5b2 against main and #146 Re-checks the iOS audit at 1f91c5b2 and sdk-specs 8306698. #1036 keeps $lib/$lib_version, which now matches #146. Opt In meets the #138 replay restart requirements. Surveys gains a #102 gap: setSurveySeen moves the last-seen date on submit and dismiss. HTTP Client goes back to ✅: its compression gap comes from conflicting http-client and capture-ai rules in #146, raised on that PR. Tally 40/17/1/11. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * docs(compliance): score posthog-ios 4.0 capture v1 rows against #146 as merged Scores Capture, Capture AI, Before Send Hook, Event Batcher, HTTP Client and Retry Queue against #146 at 5c26b9b, treated as merged. The capture v1 wire, transport and retry policy conform. Capture AI applies to iOS 4.0 and is 🟡 for the same NSNull serialization gap as Capture; starting the AI queue at setup is what #146 asks of client SDKs. Tally 40/18/1/10. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * docs(compliance): treat the #146 $lib precedence conflict as resolved for clients Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * docs(compliance): re-check posthog-ios 4.0 at e27cbe4cb against main 752f899 and #146 d387ab7 Capture, Capture Exception and Capture AI move to Pass now that null-valued properties are omitted (posthog-ios#1047) and the SDK fills its own $lib pair after the caller's properties (posthog-ios#1050). On Feature Flags moves to Pass: loads in flight across reset() are dropped (posthog-ios#1048) and empty variants are left out of listener flags. HTTP Client is scored Pass on its merits, since #146 now leaves a lane with its own compression setting out of the whole-client opt-out. Surveys stays Partial only for the popup-delay eligibility re-check (SHOULD); posthog-ios#1049 fixed the rest. Opt In meets the new deferred-crash consent requirement (#199); the empty variant (#193), variant key rename (#203) and job-retry (#205) changes need no other status change. Line references move to e27cbe4cb. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * docs(compliance): score posthog-ios Surveys as pass The popup-delay and wait-period SHALLs are met since posthog-ios#1049. The one unmet requirement is a SHOULD (re-check eligibility when the delay elapses), which is recorded but not scored. Totals become 45/13/1/10. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * docs(compliance): score iOS Exception Steps partial on NSNull normalization Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * docs(compliance): score posthog-ios Capture partial until the migration guide shows the identity call option #146 at 15501ca requires migration guides to show a per-call process_person_profile: true on identity calls; MIGRATION.md at e27cbe4cb lacks it until PostHog/posthog-ios#1053 merges. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * docs(compliance): note the README's 69 contracts and the iOS row's 69-row basis Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * docs(compliance): keep posthog-ios Capture passing while tracking 4.0 The identity-call migration-guide example #146 requires lands in 4.0 with PostHog/posthog-ios#1053, which targets the release branch. This audit tracks 4.0, so a pending release-branch PR is noted rather than scored as a gap. Reverts 85d4acd. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * docs(compliance): make the 69-contract README note SDK-neutral The same paragraph lands from both the iOS and Android 4.0 audits, so it merges cleanly in either order and stays true after both. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>

docs: show how to keep identity calls under a registered personless s…

4a00fba

…etting Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

eli-r-ph marked this pull request as ready for review November 10, 2026 22:33

eli-r-ph requested a review from a team as a code owner November 10, 2026 22:33

eli-r-ph self-assigned this Nov 10, 2026

turnipdabeets approved these changes Nov 10, 2026

View reviewed changes

docs: point registered personless users to identifiedOnly and note bi…

d46fcbe

…lling

eli-r-ph merged commit 3d0b455 into chore/v4-remove-deprecated Nov 11, 2026
35 checks passed

eli-r-ph deleted the eli/v1-guide-identity-opt-in branch November 11, 2026 01:10

turnipdabeets mentioned this pull request Nov 13, 2026

chore!: iOS Major Release 4.0 #873

Draft

6 of 18 tasks

Navigation Menu

Repository navigation

docs: show how to keep identity calls under a registered personless setting - #1053

docs: show how to keep identity calls under a registered personless setting#1053
eli-r-ph merged 2 commits into
chore/v4-remove-deprecatedfrom
eli/v1-guide-identity-opt-in

eli-r-ph commented Nov 10, 2026

Uh oh!

veria-ai Bot commented Nov 10, 2026 •

edited

Loading

Uh oh!

turnipdabeets left a comment •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

eli-r-ph commented Nov 10, 2026

💡 Motivation and Context

💚 How did you test it?

📝 Checklist

If releasing new changes

🤖 Agent context

Uh oh!

veria-ai Bot commented Nov 10, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR overview

Reviews

Uh oh!

turnipdabeets left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

veria-ai Bot commented Nov 10, 2026 •

edited

Loading

turnipdabeets left a comment •

edited

Loading