From e83c6bea68c54f4bac804410addaaae44ef899fe Mon Sep 17 00:00:00 2001 From: "Vincent (Wen Yu) Ge" <29069505+gewenyu99@users.noreply.github.com> Date: Wed, 26 Aug 2026 11:46:57 -0400 Subject: [PATCH 1/4] fix(ci): make skill sync reviewer request non-fatal, point at team-posthog-desktop This workflow has the exact same failure mode that silently broke the PostHog/skills sync-omnibus workflow from 2026-06-03: `gh pr create --reviewer "PostHog/team-posthog-ai"` hard-fails because that team slug no longer resolves, after the PR is already created, so `gh pr merge --auto` never runs. Here it has been failing most days as well; sync PRs only land when a human merges them by hand. Fix: drop --reviewer from gh pr create, request the review in a separate non-fatal gh pr edit --add-reviewer step pointed at the live owning team (PostHog/team-posthog-desktop), then always run gh pr merge --auto --squash. Generated-By: PostHog Desktop Task-Id: 85436084-530e-4448-9757-b8690214c27c --- .github/workflows/sync-skills.yml | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/.github/workflows/sync-skills.yml b/.github/workflows/sync-skills.yml index 0a5b233..f800da9 100644 --- a/.github/workflows/sync-skills.yml +++ b/.github/workflows/sync-skills.yml @@ -165,6 +165,15 @@ jobs: --base "$BASE_REF" \ --head "$HEAD_REF" \ --title "$COMMIT_MESSAGE" \ - --body "Automated skill sync from PostHog release and context-mill." \ - --reviewer "PostHog/team-posthog-ai") + --body "Automated skill sync from PostHog release and context-mill.") + + # Request review separately and non-fatally. A reviewer that no + # longer resolves must NOT take down the auto-merge: that is exactly + # how the PostHog/skills sync silently broke from 2026-06-03 to + # 2026-08-26 (team 'team-posthog-ai' was renamed, `gh pr create + # --reviewer` hard-failed after creating the PR, and ~70 sync PRs + # piled up unmerged). + gh pr edit "$PR_URL" --add-reviewer "PostHog/team-posthog-desktop" \ + || echo "::warning::Could not request review from PostHog/team-posthog-desktop; continuing to auto-merge" + gh pr merge "$PR_URL" --auto --squash From 6b6f80786fb6cb344a1833f1482761c2e2af4584 Mon Sep 17 00:00:00 2001 From: "Vincent (Wen Yu) Ge" <29069505+gewenyu99@users.noreply.github.com> Date: Wed, 26 Aug 2026 11:51:05 -0400 Subject: [PATCH 2/4] Shrink the inline incident comment; it lives in the PR description now Generated-By: PostHog Desktop Task-Id: 85436084-530e-4448-9757-b8690214c27c --- .github/workflows/sync-skills.yml | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/.github/workflows/sync-skills.yml b/.github/workflows/sync-skills.yml index f800da9..57c6b72 100644 --- a/.github/workflows/sync-skills.yml +++ b/.github/workflows/sync-skills.yml @@ -167,12 +167,8 @@ jobs: --title "$COMMIT_MESSAGE" \ --body "Automated skill sync from PostHog release and context-mill.") - # Request review separately and non-fatally. A reviewer that no - # longer resolves must NOT take down the auto-merge: that is exactly - # how the PostHog/skills sync silently broke from 2026-06-03 to - # 2026-08-26 (team 'team-posthog-ai' was renamed, `gh pr create - # --reviewer` hard-failed after creating the PR, and ~70 sync PRs - # piled up unmerged). + # The review request is deliberately non-fatal: a renamed team must + # never block auto-merge again. See PR description for the incident. gh pr edit "$PR_URL" --add-reviewer "PostHog/team-posthog-desktop" \ || echo "::warning::Could not request review from PostHog/team-posthog-desktop; continuing to auto-merge" From e1ed4f65ce0be5d4e436a39f1334c57dcd981e65 Mon Sep 17 00:00:00 2001 From: "Vincent (Wen Yu) Ge" <29069505+gewenyu99@users.noreply.github.com> Date: Wed, 26 Aug 2026 11:53:22 -0400 Subject: [PATCH 3/4] Remove the explanatory comment entirely Generated-By: PostHog Desktop Task-Id: 85436084-530e-4448-9757-b8690214c27c --- .github/workflows/sync-skills.yml | 2 -- 1 file changed, 2 deletions(-) diff --git a/.github/workflows/sync-skills.yml b/.github/workflows/sync-skills.yml index 57c6b72..03144d1 100644 --- a/.github/workflows/sync-skills.yml +++ b/.github/workflows/sync-skills.yml @@ -167,8 +167,6 @@ jobs: --title "$COMMIT_MESSAGE" \ --body "Automated skill sync from PostHog release and context-mill.") - # The review request is deliberately non-fatal: a renamed team must - # never block auto-merge again. See PR description for the incident. gh pr edit "$PR_URL" --add-reviewer "PostHog/team-posthog-desktop" \ || echo "::warning::Could not request review from PostHog/team-posthog-desktop; continuing to auto-merge" From ad4b44d2b3a2f62c135a37a4a66db29df07c9ffe Mon Sep 17 00:00:00 2001 From: "Vincent (Wen Yu) Ge" Date: Wed, 2 Sep 2026 19:21:35 -0400 Subject: [PATCH 4/4] fix(ci): request review from team-self-driving instead Co-Authored-By: Claude Fable 5 --- .github/workflows/sync-skills.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/sync-skills.yml b/.github/workflows/sync-skills.yml index 03144d1..ffc0714 100644 --- a/.github/workflows/sync-skills.yml +++ b/.github/workflows/sync-skills.yml @@ -167,7 +167,7 @@ jobs: --title "$COMMIT_MESSAGE" \ --body "Automated skill sync from PostHog release and context-mill.") - gh pr edit "$PR_URL" --add-reviewer "PostHog/team-posthog-desktop" \ - || echo "::warning::Could not request review from PostHog/team-posthog-desktop; continuing to auto-merge" + gh pr edit "$PR_URL" --add-reviewer "PostHog/team-self-driving" \ + || echo "::warning::Could not request review from PostHog/team-self-driving; continuing to auto-merge" gh pr merge "$PR_URL" --auto --squash