Skip to content

chore: reassign product owners after team split#61357

Merged
joshsny merged 1 commit into
masterfrom
posthog-code/reassign-product-owners-team-split
Jun 3, 2026
Merged

chore: reassign product owners after team split#61357
joshsny merged 1 commit into
masterfrom
posthog-code/reassign-product-owners-team-split

Conversation

@joshsny
Copy link
Copy Markdown
Contributor

@joshsny joshsny commented Jun 3, 2026

Problem

PostHog/posthog.com#17202 updates feature ownership to reflect a team change: posthog-ai is being split into signals and posthog-code. The products/*/product.yaml owners: fields in this repo (which drive automatic reviewer assignment via .github/scripts/assign-reviewers.js) still point at the retired team-posthog-ai, so PRs touching these products would be routed to the wrong team.

Changes

Reassign every product.yaml still owned by team-posthog-ai:

Product Old owner New owner Source
slack_app team-posthog-ai team-signals "Slack app" → signals
mcp_store team-posthog-ai team-signals "MCP server" → signals
posthog_ai team-posthog-ai team-signals "PostHog AI platform" → signals
tasks team-posthog-ai team-posthog-code "Background agents" → "Cloud agents" → posthog-code
mcp_analytics team-posthog-ai team-signals Not a posthog.com feature; grouped with the other MCP product since its owner team is retired

After this change no product.yaml references team-posthog-ai.

How did you test this code?

I'm an agent (PostHog Code). No automated tests were run locally. The repo's owner-lint (hogli product:lint:owners, gated on products/*/product.yaml changes) validates the new owner slugs against PostHog org teams in CI — team-signals is already used by signals/tracing, and team-posthog-code will be validated there.

Automatic notifications

  • Publish to changelog?
  • Alert Sales and Marketing teams?

Docs update

No docs impact — this only changes reviewer-routing metadata.

🤖 Agent context

Authored by PostHog Code at Josh Snyder's request, following Julian's note to adjust the product.yamls to the right GitHub teams after the posthog.com feature-ownership PR.

The mapping was derived from the diff of PostHog/posthog.com#17202 (src/hooks/useFeatureOwnership.tsx), translating each posthog.com owner slug to the repo's team-<slug> convention. mcp_analytics was the one judgement call: it has no matching posthog.com feature, but since team-posthog-ai is being fully retired it can't stay, so it was grouped with the other MCP product under team-signals. Flag if it should instead go to team-posthog-code or elsewhere.


Created with PostHog Code

team-posthog-ai is being split into team-signals and team-posthog-code, mirroring the feature-ownership update in PostHog/posthog.com#17202. Reassign every product.yaml owner accordingly so reviewer routing points at the new teams.

- slack_app, mcp_store, posthog_ai -> team-signals (per the PR)
- tasks (Cloud agents) -> team-posthog-code (per the PR)
- mcp_analytics -> team-signals (not a posthog.com feature, but its owner team is retired; grouped with the other MCP product)

Generated-By: PostHog Code
Task-Id: 4e4f2d82-3215-4ac3-b5f7-ce9830fd1082
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 2026

Hey @joshsny! 👋

It looks like your git author email on this PR isn't your @posthog.com address (joshua@timenavi.com). Since you're on the PostHog team, it's worth pointing your local git author email at your @posthog.com address. Why it matters:

  • Consistent work identity in git history — internal tooling that attributes commits to team members keys off your @posthog.com address.
  • Keeps team contributions easy to tell apart from external community ones when scanning history.

You can fix it for this repo with:

git config user.email "you@posthog.com"

Or set it globally with git config --global user.email "you@posthog.com". No need to redo this PR — just a nudge for next time. 🙂

@joshsny joshsny marked this pull request as ready for review June 3, 2026 11:02
Copilot AI review requested due to automatic review settings June 3, 2026 11:02
@joshsny joshsny enabled auto-merge (squash) June 3, 2026 11:02
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Jun 3, 2026

Reviews (1): Last reviewed commit: "chore: reassign product owners after tea..." | Re-trigger Greptile

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates products/*/product.yaml ownership metadata to reflect the split of the retired team-posthog-ai into team-signals and team-posthog-code, ensuring automatic reviewer assignment routes to the correct teams.

Changes:

  • Reassigned Tasks product ownership to team-posthog-code.
  • Reassigned Slack app, PostHog AI, MCP servers, and MCP analytics product ownership to team-signals.
  • Removed remaining team-posthog-ai references from products/*/product.yaml.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
products/tasks/product.yaml Updates product owners to team-posthog-code for Tasks.
products/slack_app/product.yaml Updates product owners to team-signals for Slack app.
products/posthog_ai/product.yaml Updates product owners to team-signals for PostHog AI.
products/mcp_store/product.yaml Updates product owners to team-signals for MCP servers.
products/mcp_analytics/product.yaml Updates product owners to team-signals for MCP analytics.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@joshsny joshsny merged commit 417e116 into master Jun 3, 2026
180 of 181 checks passed
@joshsny joshsny deleted the posthog-code/reassign-product-owners-team-split branch June 3, 2026 11:11
@deployment-status-posthog
Copy link
Copy Markdown

deployment-status-posthog Bot commented Jun 3, 2026

Deploy status

Environment Status Deployed At Workflow
dev ✅ Deployed 2026-06-03 12:03 UTC Run
prod-us ✅ Deployed 2026-06-03 12:21 UTC Run
prod-eu ✅ Deployed 2026-06-03 12:22 UTC Run

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