fix(insights): fix legacy transformation of properties

[thmsobrmlr]

Problem

The HogQL funnels feature flag breaks person modals in experiments. This is because the experiment response returns the insight filters in a way that we haven't encountered before:

• Global properties can contain "precalculated-" and "static cohorts", which are now just "cohorts"
• Entity properties can contain a property group filter, whereas only property filters are allowed

Changes

This PR:

• Converts "precalculated-" and "static cohorts" to "cohorts".
• Simplifies the property group filter to a list of property filters.

The fix for cohorts had already been implemented backend side and there have also been other deviations. Thus I've refactored the code a bit that both version do roughly the same thing and changes/additions should easy to add to both frontend and backend.

How did you test this code?

Added more tests and verified that the experiment persons modal works now

[posthog-bot]

📸 UI snapshots have been updated

84 snapshot changes in total. 0 added, 84 modified, 0 deleted:

• chromium: 0 added, 42 modified, 0 deleted (diff for shard 2)
• webkit: 0 added, 42 modified, 0 deleted (diff for shard 2)

Triggered by this commit.

👉 Review this PR's diff of snapshots.

[github-actions]

Size Change: +54 B (0%)

Total Size: 824 kB

ℹ️ View Unchanged

Filename	Size	Change
frontend/dist/toolbar.js	824 kB	+54 B (0%)

_{compressed-size-action}

[posthog-bot]

📸 UI snapshots have been updated

2 snapshot changes in total. 0 added, 2 modified, 0 deleted:

• chromium: 0 added, 2 modified, 0 deleted (diff for shard 1)
• webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

[posthog-bot]

📸 UI snapshots have been updated

2 snapshot changes in total. 0 added, 2 modified, 0 deleted:

• chromium: 0 added, 2 modified, 0 deleted (diff for shard 1)
• webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.