From 1a946615e6d28cd02ecae18e51b552f8e14613ab 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:47 -0400 Subject: [PATCH 1/6] fix(ci): make omnibus sync reviewer request non-fatal, point at team-posthog-desktop MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The daily sync-omnibus workflow has failed every run since 2026-06-03: `gh pr create --reviewer "PostHog/team-posthog-ai"` hard-fails because that team slug no longer resolves (renamed away). The PR is created before the reviewer request runs, so the failure lands after creation — PR_URL is never captured, `gh pr merge --auto` never runs, the step exits 1, and the sync kept looking busy while nothing merged. 73 sync PRs have piled up unmerged and skills/omnibus/ is ~3 months stale (agent-skills v0.102.0 vs v0.715.0 upstream). 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. A renamed team can never block a merge again. Generated-By: PostHog Desktop Task-Id: 85436084-530e-4448-9757-b8690214c27c --- .github/workflows/sync-omnibus.yml | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/.github/workflows/sync-omnibus.yml b/.github/workflows/sync-omnibus.yml index 434dbb77..5d8d4382 100644 --- a/.github/workflows/sync-omnibus.yml +++ b/.github/workflows/sync-omnibus.yml @@ -146,6 +146,14 @@ 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 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 ce602cf592b1a01ced160a3649925660c540368e 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/6] 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-omnibus.yml | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/.github/workflows/sync-omnibus.yml b/.github/workflows/sync-omnibus.yml index 5d8d4382..19c6c814 100644 --- a/.github/workflows/sync-omnibus.yml +++ b/.github/workflows/sync-omnibus.yml @@ -148,11 +148,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 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 0202fdd3047781a245f2d2d82358492897f102c7 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/6] Remove the explanatory comment entirely Generated-By: PostHog Desktop Task-Id: 85436084-530e-4448-9757-b8690214c27c --- .github/workflows/sync-omnibus.yml | 2 -- 1 file changed, 2 deletions(-) diff --git a/.github/workflows/sync-omnibus.yml b/.github/workflows/sync-omnibus.yml index 19c6c814..2b32ff37 100644 --- a/.github/workflows/sync-omnibus.yml +++ b/.github/workflows/sync-omnibus.yml @@ -148,8 +148,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 3c94683fad7de7e49893649afff8c01675c2adf3 Mon Sep 17 00:00:00 2001 From: "Vincent (Wen Yu) Ge" Date: Wed, 2 Sep 2026 19:31:27 -0400 Subject: [PATCH 4/6] fix(ci): request review from team-self-driving instead Co-Authored-By: Claude Fable 5 --- .github/workflows/sync-omnibus.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/sync-omnibus.yml b/.github/workflows/sync-omnibus.yml index 2b32ff37..e93dee90 100644 --- a/.github/workflows/sync-omnibus.yml +++ b/.github/workflows/sync-omnibus.yml @@ -148,7 +148,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 From c026e325e812c6be91b0e75dd80573cb572615a8 Mon Sep 17 00:00:00 2001 From: "Vincent (Wen Yu) Ge" Date: Wed, 2 Sep 2026 19:32:02 -0400 Subject: [PATCH 5/6] fix(ci): request review from both desktop and self-driving teams Co-Authored-By: Claude Fable 5 --- .github/workflows/sync-omnibus.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/sync-omnibus.yml b/.github/workflows/sync-omnibus.yml index e93dee90..2e531eb2 100644 --- a/.github/workflows/sync-omnibus.yml +++ b/.github/workflows/sync-omnibus.yml @@ -148,7 +148,7 @@ jobs: --title "$COMMIT_MESSAGE" \ --body "Automated skill sync from PostHog release and context-mill.") - 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 edit "$PR_URL" --add-reviewer "PostHog/team-posthog-desktop" --add-reviewer "PostHog/team-self-driving" \ + || echo "::warning::Could not request reviews; continuing to auto-merge" gh pr merge "$PR_URL" --auto --squash From f18805528ee79fb5643e423c6ee3515c475563dd Mon Sep 17 00:00:00 2001 From: "Vincent (Wen Yu) Ge" Date: Wed, 2 Sep 2026 19:37:55 -0400 Subject: [PATCH 6/6] fix(ci): request review from team-self-driving only Co-Authored-By: Claude Fable 5 --- .github/workflows/sync-omnibus.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/sync-omnibus.yml b/.github/workflows/sync-omnibus.yml index 2e531eb2..e93dee90 100644 --- a/.github/workflows/sync-omnibus.yml +++ b/.github/workflows/sync-omnibus.yml @@ -148,7 +148,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" --add-reviewer "PostHog/team-self-driving" \ - || echo "::warning::Could not request reviews; 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