fix(web-analytics): Fix using a cohort internal test filter in web overview

[robbie-c]

Problem

See https://posthoghelp.zendesk.com/agent/tickets/22891

If you have a cohort as an internal test filter, the web overview query errors, because the inner query defines person_id, but the cohort filter also tries to use person_id to mean events.person_id.

Changes

Rename the person_id to session_person_id to avoid the collision

Does this work well for both Cloud and self-hosted?

Yes

How did you test this code?

Added a previously-failing test, which passes with this fix.

[greptile-apps]

PR Summary

This PR fixes a naming collision in web overview queries when using cohort-based test account filters, renaming person_id to session_person_id to prevent conflicts.

• Fixed collision between inner query's person_id and cohort filter's events.person_id by renaming to session_person_id in web_overview.py
• Added new test case test_filter_cohort that verifies cohort-based test account filtering works correctly
• The bug was causing web overview queries to fail when cohorts were used as internal test filters
• The fix ensures proper distinction between session person IDs and event person IDs in query construction
• Solution maintains compatibility with existing functionality while resolving the specific naming conflict

_{2 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile}