[charlesvien]

Problem

The tasks-title-ai-gateway flag was checked with a fixed distinct id, so any rollout percentage sent either every task title through the gateway or none (follow-up to #127830).

Changes

• create_task passes the creator's distinct id to generate_task_title and the flag buckets on it.
• Tasks without a creator get a random id per call so the percentage still holds.
• Analytics attribution keeps the fixed task-title-generator id.

How did you test

New parametrized test for the flag distinct id, plus the title, facade and warm facade suites locally.

[trunk-io]

😎 Merged successfully - details.

[charlesvien]

@PostHog review --deep

[parameterai]

Risk: No findings

This change buckets the tasks-title-ai-gateway feature flag on the task creator's distinct id (or a random per-call id when there is no creator) instead of a fixed constant, so percentage rollouts apply per user; the LLM client keeps the fixed task-title-generator attribution id. The distinct id is server-assigned and only reaches the internal PostHog analytics flag call, so no new attacker-controlled input or security boundary is introduced.

_{Sentinel reviewed 63841cc · Review settings}

[posthog]

@charlesvien PostHog Review is reviewing this pull request.

[github-actions]

🤖 CI report

⚠️ Trunk lane — backend Python lane

This PR is assigned to the backend Python lane. It runs backend Python tests and may merge in parallel with PRs in other lanes.

✅ Duplication (Python) — clean

New Python code duplication introduced by this branch. Fails at 70+ tokens in app code, or 150+ tokens when both copies live in test files. Advisory while the gate proves itself: extract a shared helper instead of copying.

✅ Duplication (TypeScript) — clean

New TypeScript code duplication introduced by this branch. Fails at 70+ tokens in app code, or 150+ tokens when both copies live in test files. Advisory while the gate proves itself: extract a shared helper instead of copying.

[posthog]

PostHog Review · Deep · reviewed 63841cc

Nothing to post.

Step	Status	Result
Prepare the diff	done	2 files, 1 chunk
Pick perspectives	done	3 perspectives
Run review passes	done	0 issues
Merge overlapping findings	done	0 → 0
Validate findings	done
Publish	done	Nothing to post

[charlesvien]

@PostHog stamp

[posthog]

@charlesvien Stamphog is reviewing this pull request.

[stamphog]

Approved.

The task write-path change has owning-team authorship and focused test coverage. No unresolved substantive concerns or other refusal grounds were found; the partial automated review does not count as independent review assurance.

• Author wrote 30% of the modified lines and has 41 merged PRs in these paths (familiarity MODERATE).
• PostHog Review (Deep) reviewed the current head (no findings) but skipped 2 changed files, so it does not count as assurance.

Gate mechanics and policy version

Gate		Result
prerequisites	✓	all clear
deny-list	✓	no deny categories matched
size	✓	17L, 2F substantive, 47L/4F incl. docs/generated/snapshots — within ceiling
tier	✓	T1-agent / T1b-small (47L, 4F, single-area, fix)
stamphog 2.7.0		.stamphog/policy.yml @ 63841cc · reviewed head 63841cc

[deployment-status-posthog]

Deploy status

Environment	Status	Deployed At	Workflow
dev	✅ Deployed	2026-11-03 05:09 UTC	Run
prod-us	✅ Deployed	2026-11-03 05:35 UTC	Run
prod-eu	✅ Deployed	2026-11-03 05:35 UTC	Run