Don't try to detect a campaign if core detected an AI agent #189
Conversation
sgiehl
force-pushed
the
dev-19575
branch
3 times, most recently
from
cc2f9a5 to
7bb13e8
Compare
james-hill-matomo
left a comment
james-hill-matomo
left a comment
There was a problem hiding this comment.
LGTM as a quick glance. I can run & test the code tomorrow.
Please bump the version number and update the changelog - the next release date is 2025-10-13
AltamashShaikh
left a comment
AltamashShaikh
left a comment
There was a problem hiding this comment.
I can see the AIAssistant being tracked correctly after I checkout this PR 👍
|
|
||
| yield 'utm_source param provided by Perplexity is detected as AI assistant' => [ | ||
| 'https://www.example.com/?utm_source=perplexity.ai', | ||
| '', // no referer |
There was a problem hiding this comment.
@sgiehl Should we add 1 more case where utm_source is something else like should_not_be_Recorded and referrer is AI ?
AltamashShaikh
left a comment
AltamashShaikh
left a comment
There was a problem hiding this comment.
left 1 optional comment, as James mentioned update the version and changelog, so that we can release it on coming Monday
AltamashShaikh
left a comment
AltamashShaikh
left a comment
There was a problem hiding this comment.
left 1 optional comment, as James mentioned update the version and changelog, so that we can release it on coming Monday
|
This was a beautiful PR, the comments in the code anticipated any questions or complaints I might have in advance, and it's very clear. I do wonder if "campaigns" should never have been conflated with "referrers", but it works well enough and is completely out of scope for this ticket. |
3 participants
Description
ChatGPT and Perplexity are often providing a
utm_sourcewhen referring to external sites.Matomo meanwhile detects those as AI Assistants. However, this plugin currently would overwrite the detection again as campaign, which needs to be avoided.
The implementation is very hackish, as a proper solution would need so bigger refactoring in core. This will be covered in matomo-org/matomo#20067