Skip to content

Conversation

@Twixes
Copy link
Member

@Twixes Twixes commented Nov 20, 2025

Problem

We had to revert #35070 due to an issue no unit or integration test caught. We're entirely missing E2E coverage of PostHog AI.

Changes

Adding basic tests to ensure PostHog AI's integrity end-to-end.

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

15 files reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

@github-actions
Copy link
Contributor

github-actions bot commented Nov 20, 2025

Size Change: 0 B

Total Size: 3.4 MB

ℹ️ View Unchanged
Filename Size
frontend/dist/toolbar.js 3.4 MB

compressed-size-action

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@Twixes Twixes force-pushed the ai-e2e-test branch 2 times, most recently from 74514ee to 9f248d9 Compare November 21, 2025 13:21
@posthog-bot
Copy link
Contributor

⏭️ Skipped snapshot commit because branch advanced to 9f248d9 while workflow was testing 74514ee.

The new commit will trigger its own snapshot update workflow.

If you expected this workflow to succeed: This can happen due to concurrent commits. To get a fresh workflow run, either:

  • Merge master into your branch, or
  • Push an empty commit: git commit --allow-empty -m 'trigger CI' && git push

@posthog-bot
Copy link
Contributor

Visual regression: Storybook UI snapshots updated

Mode: UPDATE (triggered by human commit 15f0880)

Changes: 7 snapshots (7 modified, 0 added, 0 deleted)

What this means:

  • Snapshots have been automatically updated to match current rendering
  • Next CI run will switch to CHECK mode to verify stability
  • If snapshots change again, CHECK mode will fail (indicates flapping)

Next steps:

  • Review the changes to ensure they're intentional
  • Approve if changes match your expectations
  • If unexpected, investigate component rendering

Review snapshot changes →

@Twixes Twixes closed this Nov 24, 2025
@Twixes
Copy link
Member Author

Twixes commented Nov 24, 2025

Merged with #41850

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants