Skip to content

chore(deps): update posthoganalytics to 7.83.1 - #128369

Merged
trunk-io[bot] merged 1 commit into
masterfrom
andehen/posthoganalytics-7.83.1
Nov 3, 2026
Merged

trunk-io[bot] merged 1 commit into
masterfrom
andehen/posthoganalytics-7.83.1

Conversation

@andehen

@andehen andehen commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

Problem

The app runs posthoganalytics 7.82.2. 7.83.0 moves remote flag evaluation to /flags?v=4 (PostHog/posthog-python#1031), and 7.83.1 adds an MCP fix (PostHog/posthog-python#1166). The app should run the current SDK.

Changes

  • Mechanical: bumps posthoganalytics (and posthog in the clitools requirements) from 7.82.2 to 7.83.1 in the same files the release bot updates, and both uv.lock files. Only posthoganalytics moves in the locks.
  • Nothing a user can see changes: existing flag getters return the same values. Only flags in the new rules format report a different reason text.

How did you test this code?

  • Ran the app's flag code against a local stack with 7.82.2 and with 7.83.1 and compared the results: the persons-on-events backfill resolver, the HogQL move-notice flag, the query-scan flag, and every local flag through get_flags_decision, all flags and payloads, snapshots, single-flag getters and posthoganalytics.feature_enabled, for three identities, against a flags service with the v4 response off and one with it on. Values, payloads and $feature_flag_called events were identical, except the reason text of one new-format flag. Events were compared in memory, not sent.
  • pytest posthog/query_scan/test/test_flag.py posthog/dags/tests/test_backfill_persons_on_events_mode.py posthog/hogql/test/test_metadata.py passes.
  • Not checked: production traffic.

Test rationale: No new test: a dependency bump with no code change; the existing tests of the SDK call sites above cover it.

Release status

  • No feature flag controls this change

Docs update

  • None.

🤖 Agent context

Autonomy: Human-driven (agent-assisted)

Agent: Claude Code, Claude Opus 5.5

Skills: None

More context for AI agents
  • Duplicate check: the release bot's chore(deps): update posthoganalytics to 7.83.0 #128223 bumps the same files to 7.83.0; this PR takes 7.83.1, the version tested.
  • posthoganalytics is exempt from the repo's 7-day exclude-newer cutoff, so uv lock --upgrade-package posthoganalytics resolves it.

🤖 Generated with Claude Code

@andehen andehen added the stamphog Request AI approval (no full review) label Nov 3, 2026
@andehen andehen self-assigned this Nov 3, 2026
@trunk-io

trunk-io Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

😎 Merged successfully - details.

@parameterai

parameterai Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

Risk: No findings

A pure dependency bump of posthoganalytics 7.82.2 → 7.83.1 across pyproject pins, uv.lock files, and two requirements files; no application code changes. I found no security issue: the lock diffs move only this package, and the SDK's flag-evaluation changes touch no authorization boundary in this repo.

Sentinel reviewed 313ddf9 · Review settings

@github-actions github-actions Bot added the feature/agent-infra Feature Tag: Agent infra label Nov 3, 2026
@stamphog stamphog Bot removed the stamphog Request AI approval (no full review) label Nov 3, 2026

@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.

Not approved — this change needs a human reviewer.

Re-adding the stamphog label gives the same result unless the changed files change.

stamphog can't auto-review this pull request because two gates refused it. The deny-list gate matched the deps_toolchain category, since the PR changes dependency pins and lockfiles (pyproject.toml, services/llm-gateway/pyproject.toml, both uv.lock files, and the two requirements.txt files). The tier gate classified it as T2-never because it is a cross-cutting chore touching six files across several services and tools.

The size and prerequisites gates passed, but they don't override the refusal. Please ask a human reviewer to look at this bump, and mention the existing release-bot PR for 7.83.0 so they can decide which one to take.

Gate mechanics and policy version
Gate Result
prerequisites ✓ all clear
deny-list ✗ matches: deps_toolchain
size ✓ 4L, 2F substantive, 24L/6F incl. docs/generated/snapshots — within ceiling
tier ✗ classified as T2-never: T2-never (24L, 6F, cross-cutting, chore)
stamphog 2.7.0 .stamphog/policy.yml @ unknown · reviewed head 313ddf9

@pr-assigner-resolver-posthog
pr-assigner-resolver-posthog Bot requested a review from a team November 3, 2026 16:12
@github-actions

github-actions Bot commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

🤖 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.

✅ Playwright — all passed

All tests passed.

View test results →

✅ Hobby preview — passed

Hobby deployment smoke test passed successfully.


Run 41671725533

@trunk-io
trunk-io Bot merged commit 6b6408b into master Nov 3, 2026
320 checks passed
@trunk-io
trunk-io Bot deleted the andehen/posthoganalytics-7.83.1 branch November 3, 2026 17:33
@deployment-status-posthog

deployment-status-posthog Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

Deploy status

Environment Status Deployed At Workflow
dev ✅ Deployed 2026-11-03 21:02 UTC Run
prod-us ✅ Deployed 2026-11-03 21:26 UTC Run
prod-eu ✅ Deployed 2026-11-03 21:18 UTC Run

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

Labels

feature/agent-infra Feature Tag: Agent infra

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants