From 6e10d004e7e08600c98302979c334d11d682c57a Mon Sep 17 00:00:00 2001
From: David Gidwani <david.gidwani@atomweight.io>
Date: Wed, 18 Mar 2026 19:09:04 -0400
Subject: [PATCH] ci(publish): switch trigger from pull_request to push

The pull_request event sets GITHUB_REF to refs/pull/N/merge,
which doesn't match the release environment's branch policy
(main, create-pull-request/patch), causing the publish job
to be rejected before execution with no logs.

Switching to push trigger on main so GITHUB_REF is
refs/heads/main, which satisfies the branch policy.

- Replace pull_request closed trigger with push to main
- Gate build job on commit message starting with 'Release '
- Remove PR-specific env vars and checks from release_check
---
 .github/workflows/publish.yml | 40 ++++++++++++++---------------------
 1 file changed, 16 insertions(+), 24 deletions(-)

diff --git a/.github/workflows/publish.yml b/.github/workflows/publish.yml
index f4c7de5..d8df3d0 100644
--- a/.github/workflows/publish.yml
+++ b/.github/workflows/publish.yml
@@ -1,17 +1,22 @@
 name: Publish
 
 on:
-  pull_request:
+  push:
     branches:
       - main
-    types: [closed]
   workflow_dispatch:
 
 jobs:
   build:
     name: Build source distribution and wheels
     uses: ./.github/workflows/build.yml
-    if: github.event_name == 'workflow_dispatch' || (github.event_name == 'pull_request' && github.event.pull_request.merged == true && github.repository == 'darvid/python-hyperscan' && startsWith(github.event.pull_request.head.ref, vars.RELEASE_PR_BRANCH || 'create-pull-request/patch'))
+    if: >
+      github.event_name == 'workflow_dispatch' ||
+      (
+        github.event_name == 'push' &&
+        github.repository == 'darvid/python-hyperscan' &&
+        startsWith(github.event.head_commit.message, 'Release ')
+      )
     permissions:
       contents: read
       actions: write
@@ -38,9 +43,6 @@ jobs:
         id: release_check
         env:
           EVENT_NAME: ${{ github.event_name }}
-          PR_MERGED: ${{ github.event.pull_request.merged }}
-          PR_HEAD_REF: ${{ github.event.pull_request.head.ref }}
-          RELEASE_PR_BRANCH: ${{ vars.RELEASE_PR_BRANCH || 'create-pull-request/patch' }}
           REPOSITORY: ${{ github.repository }}
         run: |
           if [[ "${REPOSITORY}" != "darvid/python-hyperscan" ]]; then
@@ -49,27 +51,17 @@ jobs:
             exit 0
           fi
 
-          # For workflow_dispatch, skip PR checks - just verify PyPI status
-          if [[ "${EVENT_NAME}" != "workflow_dispatch" ]]; then
-            if [[ "${PR_MERGED}" != "true" ]]; then
-              echo "Pull request not merged, skipping release"
+          if [[ "${EVENT_NAME}" == "workflow_dispatch" ]]; then
+            echo "workflow_dispatch triggered - skipping commit message check"
+          else
+            # For push events, the build job already gates on 'Release ' prefix,
+            # but double-check here for safety
+            HEAD_MSG=$(git log -1 --format=%s)
+            if ! [[ "${HEAD_MSG}" =~ ^Release\  ]]; then
+              echo "HEAD commit '${HEAD_MSG}' is not a release commit, skipping"
               echo "should_release=false" >> "$GITHUB_OUTPUT"
               exit 0
             fi
-
-            if [[ -n "${RELEASE_PR_BRANCH}" ]]; then
-              case "${PR_HEAD_REF}" in
-                "${RELEASE_PR_BRANCH}"*)
-                  ;;
-                *)
-                  echo "Head ref ${PR_HEAD_REF} does not match expected release branch prefix ${RELEASE_PR_BRANCH}"
-                  echo "should_release=false" >> "$GITHUB_OUTPUT"
-                  exit 0
-                  ;;
-              esac
-            fi
-          else
-            echo "workflow_dispatch triggered - skipping PR checks"
           fi
 
           # Get the version we're about to release