This repository was archived by the owner on Jul 27, 2026. It is now read-only.
fix: identify PostHog users with Cloud identity - #1948
Closed
neubig wants to merge 2 commits into
Closed
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
✅ Mock-LLM E2E Tests60/60 passed Commit: Details
Posted by the Mock-LLM E2E workflow · results are deterministic (scripted LLM responses) |
Contributor
✅ Mock-LLM Docker E2E Test Results60/60 passed Commit: Details
Posted by the Mock-LLM E2E workflow · results are deterministic (scripted LLM responses) |
neubig
marked this pull request as ready for review
July 27, 2026 13:01
Member
Author
|
This repository has moved to https://github.com/OpenHands/OpenHands. We’d appreciate it if you re-opened this pull request there. Thank you! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
HUMAN:
AGENT:
Why
Canvas currently attaches
cloud_user_idas an event property but deliberately keeps a browser/installdistinct_id. That makes person-level analysis depend on expensivecoalesce(cloud_user_id, distinct_id)HogQL across every insight and leaves Cloud and local Canvas activity fragmented.PostHog recommends calling
identifywith a unique, stable user ID when the user logs in so pre-login anonymous activity and later identified activity resolve to the same person: https://posthog.com/docs/product-analytics/identifyFixes OpenHands/OpenHands#15417.
Summary
Issue Number
OpenHands/OpenHands#15417
How to Test
Run:
Observed: 78 focused tests passed; ESLint, Prettier, and TypeScript completed without errors; the production client and SPA builds completed successfully.
The focused tests exercise anonymous-to-identified transition, account switching, logout, denial/re-grant, pre-reset consent revocation, privacy clear, credential replacement, Cloud disconnect, unresolved startup, and intentional local-backend identity preservation. Cloud OAuth was not exercised manually.
Video/Screenshots
Not applicable; this changes telemetry identity behavior without a visual UI change.
Type
Notes
The companion Automations change carries the same actor identity into asynchronous local automation events and rejects browser identity at the Cloud trust boundary: OpenHands/automation#265
Existing dashboard queries can be simplified after both changes are deployed and historical data ages out.
🐳 Docker images for this PR
• GHCR package: https://github.com/OpenHands/agent-canvas/pkgs/container/agent-canvas
ghcr.io/openhands/agent-canvasghcr.io/openhands/agent-server:1.37.0-pythonopenhands-automation==1.3.1afeb4725847995d5ea99faf851047f858af97617Pull (multi-arch manifest)
# Multi-arch manifest — Docker automatically pulls the correct architecture docker pull ghcr.io/openhands/agent-canvas:sha-afeb472Run
All tags pushed for this build
About Multi-Architecture Support
sha-afeb472) is a multi-arch manifest supporting both amd64 and arm64sha-afeb472-amd64) are also available if needed