Skip to content

fix(ci): baseline llm_prompt and early_access startup receivers#72102

Merged
eli-r-ph merged 1 commit into
masterfrom
eli/ci-fix-receivers-baseline
Jul 17, 2026
Merged

fix(ci): baseline llm_prompt and early_access startup receivers#72102
eli-r-ph merged 1 commit into
masterfrom
eli/ci-fix-receivers-baseline

Conversation

@eli-r-ph

Copy link
Copy Markdown
Contributor

Problem

Master is red on test_setup_receivers_match_baseline. Three signal receivers connect at django.setup() but are not recorded in posthog/test/setup_receivers_baseline.txt, so every PR that merges with master fails the Django startup-budget gate through no fault of its own.

Changes

Regenerated the baseline to record the three receivers:

Both merged after the baseline was last regenerated (#70613, 2026-07-14), so master drifted. This supersedes the stale partial fix in #71940, which recorded only 1 of the 3 (it was generated before #65180 landed).

How did you test this code?

Ran UPDATE_SETUP_RECEIVERS_BASELINE=1 pytest posthog/test/test_startup_import_budget.py -k receivers_match on a clean master checkout. The diff is exactly the three lines above (no removals, no other additions), and -k receivers_match then passes.

🤖 Agent context

Autonomy: Human-driven (agent-assisted)

I (Eli, via Cursor agent) hit this while getting an unrelated GRL PR stack green. That stack's Django PR failed CI on these three orphan receivers, which turned out to be pre-existing master breakage, not mine. I cut this isolated fix off master so the baseline drift is fixed on its own rather than absorbed into the feature PR. The regen was authoritative (ran the documented UPDATE_SETUP_RECEIVERS_BASELINE=1 command), not hand-edited.

Master is red on test_setup_receivers_match_baseline: three receivers
connect at django.setup() but are not recorded in the baseline.

- llm_prompt.invalidate_llm_prompt_label_cache (post_save + post_delete), from #71509
- early_access_features.signals.create_waitlist_survey_on_concept_stage, from #65180

Both merged after the baseline was last regenerated (#70613, 2026-07-14).
Regenerated via UPDATE_SETUP_RECEIVERS_BASELINE=1. Supersedes the stale
partial fix in #71940, which recorded only 1 of the 3.
@eli-r-ph eli-r-ph added the skip-agent-review Save $$$, skip auto agent reviews (Greptile) — use for trivial or chore PRs label Jul 17, 2026
@eli-r-ph eli-r-ph self-assigned this Jul 17, 2026
@eli-r-ph eli-r-ph added stamphog Request AI approval (no full review) and removed skip-agent-review Save $$$, skip auto agent reviews (Greptile) — use for trivial or chore PRs labels Jul 17, 2026
@eli-r-ph
eli-r-ph marked this pull request as ready for review July 17, 2026 23:38

@stamphog stamphog Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Trivial, purely additive fix to a test baseline file recording three receivers that were already merged into master via other PRs; no production code or risky territory touched.

  • 👍 on the PR from hex-security-app[bot].
Gate mechanics and policy version
Gate Result
prerequisites all clear
deny-list no deny categories matched
size 0L, 0F substantive, 3L/1F incl. docs/generated/snapshots — within ceiling
tier T0 auto-approve: T0-deterministic (3L, 1F, single-area, fix)
stamphog 2.0.0b3 .stamphog/policy.yml @ 4a381aa · reviewed head 70c4d61

@eli-r-ph
eli-r-ph enabled auto-merge (squash) July 17, 2026 23:44
@github-actions

Copy link
Copy Markdown
Contributor

🤖 CI report

Playwright — all passed

All tests passed.

View test results →

@eli-r-ph
eli-r-ph merged commit 24a1cd9 into master Jul 17, 2026
371 of 473 checks passed
@eli-r-ph
eli-r-ph deleted the eli/ci-fix-receivers-baseline branch July 17, 2026 23:58
@deployment-status-posthog

deployment-status-posthog Bot commented Jul 18, 2026

Copy link
Copy Markdown

Deploy status

Environment Status Deployed At Workflow
dev ✅ Deployed 2026-07-18 00:53 UTC Run
prod-us ✅ Deployed 2026-07-18 01:08 UTC Run
prod-eu ✅ Deployed 2026-07-18 01:07 UTC Run

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stamphog Request AI approval (no full review)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant