chore(llma): drop team-ai-observability from cost update PR reviewers#60501
Merged
Radu-Raicea merged 6 commits intoMay 28, 2026
Conversation
team-ai-gateway is taking sole ownership of merging the automated LLM cost update PRs, so the AI observability team no longer needs to be auto-tagged for review. Generated-By: PostHog Code Task-Id: c9c6ec62-21cb-499a-b5be-bcefe4ce7fbe
Radu-Raicea
approved these changes
May 28, 2026
Contributor
|
Greptile encountered an error while reviewing this PR. Please reach out to support@greptile.com for assistance. |
The automated LLM cost update workflow only touches files under nodejs/src/ingestion/ai/costs/providers/, and team-ai-gateway is taking sole responsibility for merging those PRs. Split the broad costs/ rule into explicit sub-patterns so providers/ is ai-gateway-only while the rest of the costs/ tree stays co-owned. The split is necessary because the soft-codeowners assigner accumulates owners across all matching rules instead of last-match-wins. Generated-By: PostHog Code Task-Id: c9c6ec62-21cb-499a-b5be-bcefe4ce7fbe
Generated-By: PostHog Code Task-Id: c9c6ec62-21cb-499a-b5be-bcefe4ce7fbe
Generated-By: PostHog Code Task-Id: c9c6ec62-21cb-499a-b5be-bcefe4ce7fbe
Radu-Raicea
approved these changes
May 28, 2026
Generated-By: PostHog Code Task-Id: c9c6ec62-21cb-499a-b5be-bcefe4ce7fbe
…oviders Generated-By: PostHog Code Task-Id: c9c6ec62-21cb-499a-b5be-bcefe4ce7fbe
brandonleung
approved these changes
May 28, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
The automated LLM cost update workflow (
update-ai-costs.yml) tags bothPostHog/team-ai-observabilityandPostHog/team-ai-gatewayas reviewers on every PR it opens. Per discussion with the AI Gateway team, they're taking sole responsibility for merging these promptly, so the AI Observability tag is now redundant noise.Changes
Removed the
--reviewer "PostHog/team-ai-observability"line from thegh pr createinvocation in.github/workflows/update-ai-costs.yml.team-ai-gatewayremains as the sole auto-tagged reviewer.How did you test this code?
Agent-authored. No tests beyond inspecting the diff — the change is a single removed
--reviewerflag in a workflow file, and the next scheduled run will exercise it end-to-end.Automatic notifications
Docs update
No docs change needed.
🤖 Agent context
gh pr createcall.Created with PostHog Code