Skip to content

docs(mcp-analytics): forbid the standalone $identify event - #134

Merged
gesh merged 2 commits into
mainfrom
feat/mcp-drop-identify
Oct 20, 2026
Merged

gesh merged 2 commits into
mainfrom
feat/mcp-drop-identify

Conversation

@gesh

@gesh gesh commented Oct 19, 2026 •

Copy link
Copy Markdown
Member

Problem

Changes

  • The Person identity requirement now says the SDK SHALL NOT send a standalone $identify, for any MCP revision or session mode.
  • Per-event identity does not change: $set and $groups on identified events, $process_person_profile: false on anonymous ones.
  • A new scenario checks that an identified call sends no $identify.
  • The identified-call scenario in acceptance/public/mcp-analytics.feature now asserts no $identify. It reuses the harness phrase no event named "…" should be enqueued.
  • The change is proposed, applied and archived under openspec/changes/archive/2026-10-19-drop-mcp-identify-event/.

Note

Insights built on $identify from MCP servers stop receiving that event. Person properties are unaffected.

Testing

  • openspec validate --specs --strict --no-interactive with CI's pinned 1.4.1 passes all 64 specs.
  • Not run: the acceptance scenario. PostHog/posthog-sdk-test-harness has no MCP step bindings yet, so no MCP scenario in that feature runs today.

🤖 Agent context

  • Autonomy: human-driven, agent-assisted. The author reviewed the diff before the push.
  • Tool and model: Claude Code, claude-opus-5-5.
  • Skills: writing-pr-descriptions.

🤖 Generated with Claude Code

gesh and others added 2 commits October 19, 2026 17:20
The JS and Python SDKs no longer send $identify from MCP servers. It
carried no $anon_distinct_id, so it merged no person, and every event
already carries distinct_id, $set and $groups.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
@gesh
gesh marked this pull request as ready for review October 19, 2026 14:44
@gesh
gesh requested a review from a team as a code owner October 19, 2026 14:44
@gesh
gesh requested review from a team and lucasheriques and removed request for a team October 19, 2026 14:44
@gesh gesh self-assigned this Oct 19, 2026
@gesh
gesh merged commit d53af67 into main Oct 20, 2026
7 checks passed
@gesh
gesh deleted the feat/mcp-drop-identify branch October 20, 2026 08:43
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