docs: add UTM tracking to aictrl.dev links - #107
Conversation
Code reviewVerdict: Address the major findings before merging. · 🔴 0 · 🟠 1 · 🟡 0 · ⚪ 0 · 0/1 resolved
🤖 Fix all 1 open findings with your agent📋 Out-of-diff findings (1)
Reviewed 2 files · 0 inline · view all 1 findings ↗ aictrl · AI code review for fast-moving teams · aictrl.dev |
Review response — PR #107Kept referral tracking in repository documentation while restoring the legal notice to its canonical text. Issues addressed (pushed to this PR)
Review claims verified false (no change needed)
Not addressed here
|
|
|
||
| --- | ||
| [aictrl.dev](https://aictrl.dev) | ||
| [aictrl.dev](https://aictrl.dev/?utm_medium=referral&utm_source=github&utm_campaign=cli&utm_content=readme) |
There was a problem hiding this comment.
🟡 UTM rollout: other product links in repo remain untagged.
🤖 Fix with your agent
Fix this code review finding (aictrl-dev/cli PR #107, README.md:125):
Problem: UTM rollout: other product links in repo remain untagged
Detail: PR title and issue #106 describe UTM tracking for aictrl.dev "links", but this edits the repo's only aictrl.dev hyperlink while every other user-facing product link remains untagged on the bare https://aictrl.ai domain: github/README.md:3 (GitHub Action marketplace readme), sdks/vscode/README.md:3 and :7 (VS Code marketplace readme), packages/sdk/README.md:64 (rendered as the npm readme of @aictrl/sdk), packages/cli/package.json:11 (homepage), and install:23/:458 (install script output). If aictrl.ai and aictrl.dev feed the same analytics property, issue #106's attribution goal misses all of these higher-traffic surfaces; if they are separate properties, the repo now promotes two canonical product domains and this PR deepens the split. Domain equivalence could not be confirmed from the repo alone.
Suggested fix: Either extend the same campaign (utm_medium=referral&utm_source=github&utm_campaign=cli with a per-surface utm_content, e.g. action-readme / vscode-readme / sdk-readme) to the product links in github/README.md, sdks/vscode/README.md, and packages/sdk/README.md, or rescope the PR title/description to the single README link and file the aictrl.dev vs aictrl.ai domain split as a follow-up issue.
Implement the fix on the PR head branch and add a regression test that fails before the fix and passes after.
Why this matters
PR title and issue #106 describe UTM tracking for aictrl.dev "links", but this edits the repo's only aictrl.dev hyperlink while every other user-facing product link remains untagged on the bare https://aictrl.ai domain: github/README.md:3 (GitHub Action marketplace readme), sdks/vscode/README.md:3 and :7 (VS Code marketplace readme), packages/sdk/README.md:64 (rendered as the npm readme of @aictrl/sdk), packages/cli/package.json:11 (homepage), and install:23/:458 (install script output). If aictrl.ai and aictrl.dev feed the same analytics property, issue #106's attribution goal misses all of these higher-traffic surfaces; if they are separate properties, the repo now promotes two canonical product domains and this PR deepens the split. Domain equivalence could not be confirmed from the repo alone.
Aictrl is a fork of the [OpenCode](https://opencode.ai) project and is licensed under the MIT License.
---
[aictrl.dev](https://aictrl.dev/?utm_medium=referral&utm_source=github&utm_campaign=cli&utm_content=readme)
Code reviewVerdict: Looks good — only minor / nit comments below. · 🔴 0 · 🟠 0 · 🟡 1 · ⚪ 0 · 0/1 resolved
🤖 Fix all 1 open findings with your agent📋 Out-of-diff findings (1)
Reviewed 1 files · 0 inline · view all 1 findings ↗ aictrl · AI code review for fast-moving teams · aictrl.dev |
Closes #106
Problem
Traffic from this repository to aictrl.dev cannot currently be attributed in analytics.
Behavior
The README link now includes the shared GitHub referral campaign and
utm_content=readme. There is no runtime behavior change.Scope
utm_medium=referral,utm_source=github,utm_campaign=cli, andutm_content=readmeon the README link.LICENSEcopyright notice unchanged so legal-text matching remains stable.Rollout and risks
Documentation-only change. The destination is unchanged; rollback is removing the README query parameters.
Verification
bun turbo typecheck(pre-push hook) — 6/6 tasks successful.verify, CodeQL analyses, and Aictrl AI Review — passed.6c572e5bb.git diff --check origin/main...HEAD— passed.