mirror of
https://github.com/xtr-dev/payload-mailing.git
synced 2025-12-10 00:03:23 +00:00
Simplify workflow to basic NPM publish only
### 🔧 Improvements - Removed all version bumping, changelog generation, and git manipulation - Simplified to core workflow: checkout → setup → install → test → build → publish - Removed complex conditions, branches, and notifications - Uses whatever version is currently in package.json ### 🐛 Bug Fixes - Eliminated potential workflow failures from complex logic - Removed git force push operations and version conflicts - Streamlined to reliable, standard publish pattern
This commit is contained in:
286
.github/workflows/version-and-publish.yml
vendored
286
.github/workflows/version-and-publish.yml
vendored
@@ -1,30 +1,16 @@
|
|||||||
name: Version and Publish
|
name: Publish to NPM
|
||||||
|
|
||||||
on:
|
on:
|
||||||
push:
|
push:
|
||||||
branches:
|
branches:
|
||||||
- main
|
- main
|
||||||
pull_request:
|
|
||||||
branches:
|
|
||||||
- main
|
|
||||||
types: [closed]
|
|
||||||
|
|
||||||
jobs:
|
jobs:
|
||||||
version-and-publish:
|
publish:
|
||||||
if: github.event_name == 'push' || (github.event.pull_request.merged == true && (startsWith(github.event.pull_request.head.ref, 'major/') || startsWith(github.event.pull_request.head.ref, 'minor/') || startsWith(github.event.pull_request.head.ref, 'patch/')))
|
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
outputs:
|
|
||||||
new-version: ${{ steps.version-bump.outputs.new-version }}
|
|
||||||
current-version: ${{ steps.version-bump.outputs.current-version }}
|
|
||||||
version-type: ${{ steps.version-type.outputs.type }}
|
|
||||||
package-name: ${{ steps.prerequisites.outputs.package-name }}
|
|
||||||
|
|
||||||
steps:
|
steps:
|
||||||
- name: Checkout code
|
- name: Checkout code
|
||||||
uses: actions/checkout@v4
|
uses: actions/checkout@v4
|
||||||
with:
|
|
||||||
fetch-depth: 0
|
|
||||||
token: ${{ secrets.GITHUB_TOKEN }}
|
|
||||||
|
|
||||||
- name: Setup Node.js
|
- name: Setup Node.js
|
||||||
uses: actions/setup-node@v4
|
uses: actions/setup-node@v4
|
||||||
@@ -34,44 +20,6 @@ jobs:
|
|||||||
|
|
||||||
- name: Setup pnpm
|
- name: Setup pnpm
|
||||||
uses: pnpm/action-setup@v4
|
uses: pnpm/action-setup@v4
|
||||||
with:
|
|
||||||
version: 8
|
|
||||||
|
|
||||||
- name: Get pnpm store directory
|
|
||||||
shell: bash
|
|
||||||
run: echo "STORE_PATH=$(pnpm store path --silent)" >> $GITHUB_ENV
|
|
||||||
|
|
||||||
- name: Setup pnpm cache
|
|
||||||
uses: actions/cache@v4
|
|
||||||
with:
|
|
||||||
path: ${{ env.STORE_PATH }}
|
|
||||||
key: ${{ runner.os }}-pnpm-store-${{ hashFiles('**/pnpm-lock.yaml') }}
|
|
||||||
restore-keys: |
|
|
||||||
${{ runner.os }}-pnpm-store-
|
|
||||||
|
|
||||||
- name: Validate prerequisites and setup
|
|
||||||
id: prerequisites
|
|
||||||
run: |
|
|
||||||
# Extract package info
|
|
||||||
PACKAGE_NAME=$(node -p "require('./package.json').name")
|
|
||||||
CURRENT_VERSION=$(node -p "require('./package.json').version")
|
|
||||||
|
|
||||||
echo "package-name=$PACKAGE_NAME" >> $GITHUB_OUTPUT
|
|
||||||
echo "current-version=$CURRENT_VERSION" >> $GITHUB_OUTPUT
|
|
||||||
|
|
||||||
echo "📦 Package: $PACKAGE_NAME"
|
|
||||||
echo "📌 Current Version: $CURRENT_VERSION"
|
|
||||||
|
|
||||||
# Validate Node.js version matches package.json engines
|
|
||||||
NODE_VERSION=$(node --version)
|
|
||||||
echo "🔍 Validating Node.js version: $NODE_VERSION"
|
|
||||||
|
|
||||||
# Basic validation - check if we're on Node 20+
|
|
||||||
if ! echo "$NODE_VERSION" | grep -qE "^v(20|21|22)\."; then
|
|
||||||
echo "⚠️ WARNING: Node.js version $NODE_VERSION may not match package.json engines requirements"
|
|
||||||
fi
|
|
||||||
|
|
||||||
echo "✅ Prerequisites validation complete"
|
|
||||||
|
|
||||||
- name: Install dependencies
|
- name: Install dependencies
|
||||||
run: pnpm install --frozen-lockfile
|
run: pnpm install --frozen-lockfile
|
||||||
@@ -82,235 +30,7 @@ jobs:
|
|||||||
- name: Run build
|
- name: Run build
|
||||||
run: pnpm build
|
run: pnpm build
|
||||||
|
|
||||||
- name: Determine version bump type
|
|
||||||
id: version-type
|
|
||||||
run: |
|
|
||||||
if [[ "${{ github.event.pull_request.head.ref }}" =~ ^major/ ]]; then
|
|
||||||
echo "type=major" >> $GITHUB_OUTPUT
|
|
||||||
elif [[ "${{ github.event.pull_request.head.ref }}" =~ ^minor/ ]]; then
|
|
||||||
echo "type=minor" >> $GITHUB_OUTPUT
|
|
||||||
elif [[ "${{ github.event.pull_request.head.ref }}" =~ ^patch/ ]]; then
|
|
||||||
echo "type=patch" >> $GITHUB_OUTPUT
|
|
||||||
elif [[ "${{ github.event_name }}" == "push" ]]; then
|
|
||||||
# Default to patch for direct pushes to main
|
|
||||||
echo "type=patch" >> $GITHUB_OUTPUT
|
|
||||||
else
|
|
||||||
echo "type=none" >> $GITHUB_OUTPUT
|
|
||||||
fi
|
|
||||||
|
|
||||||
- name: Configure git
|
|
||||||
if: steps.version-type.outputs.type != 'none'
|
|
||||||
run: |
|
|
||||||
git config --global user.name "github-actions[bot]"
|
|
||||||
git config --global user.email "github-actions[bot]@users.noreply.github.com"
|
|
||||||
|
|
||||||
- name: Generate changelog from commits
|
|
||||||
if: steps.version-type.outputs.type != 'none'
|
|
||||||
id: changelog
|
|
||||||
run: |
|
|
||||||
# Get commits since last tag
|
|
||||||
LAST_TAG=$(git describe --tags --abbrev=0 2>/dev/null || echo "")
|
|
||||||
if [ -z "$LAST_TAG" ]; then
|
|
||||||
COMMITS=$(git log --oneline --no-merges --since="1 week ago" || echo "No recent commits")
|
|
||||||
else
|
|
||||||
COMMITS=$(git log --oneline --no-merges ${LAST_TAG}..HEAD || echo "No new commits")
|
|
||||||
fi
|
|
||||||
|
|
||||||
echo "📝 Generating changelog from commits"
|
|
||||||
echo "Commits to process:"
|
|
||||||
echo "$COMMITS"
|
|
||||||
|
|
||||||
# Simple changelog from commit messages
|
|
||||||
CHANGELOG="## Changes
|
|
||||||
|
|
||||||
### 🔧 Improvements
|
|
||||||
$(echo "$COMMITS" | sed 's/^[a-f0-9]* /- /' | head -10)"
|
|
||||||
|
|
||||||
echo "Generated changelog:"
|
|
||||||
echo "$CHANGELOG"
|
|
||||||
|
|
||||||
# Save changelog to output
|
|
||||||
echo "changelog<<EOF" >> $GITHUB_OUTPUT
|
|
||||||
echo "$CHANGELOG" >> $GITHUB_OUTPUT
|
|
||||||
echo "EOF" >> $GITHUB_OUTPUT
|
|
||||||
|
|
||||||
- name: Create version bump script
|
|
||||||
if: steps.version-type.outputs.type != 'none'
|
|
||||||
run: |
|
|
||||||
# Create Node.js script to safely bump version
|
|
||||||
cat > bump-version.js << 'VERSION_SCRIPT_EOF'
|
|
||||||
const fs = require('fs');
|
|
||||||
const path = require('path');
|
|
||||||
|
|
||||||
const versionType = process.argv[2];
|
|
||||||
const packagePath = path.join(process.cwd(), 'package.json');
|
|
||||||
|
|
||||||
try {
|
|
||||||
const packageJson = JSON.parse(fs.readFileSync(packagePath, 'utf8'));
|
|
||||||
const currentVersion = packageJson.version;
|
|
||||||
const [major, minor, patch] = currentVersion.split('.').map(Number);
|
|
||||||
|
|
||||||
let newVersion;
|
|
||||||
switch (versionType) {
|
|
||||||
case 'major':
|
|
||||||
newVersion = `${major + 1}.0.0`;
|
|
||||||
break;
|
|
||||||
case 'minor':
|
|
||||||
newVersion = `${major}.${minor + 1}.0`;
|
|
||||||
break;
|
|
||||||
case 'patch':
|
|
||||||
newVersion = `${major}.${minor}.${patch + 1}`;
|
|
||||||
break;
|
|
||||||
default:
|
|
||||||
throw new Error(`Invalid version type: ${versionType}`);
|
|
||||||
}
|
|
||||||
|
|
||||||
packageJson.version = newVersion;
|
|
||||||
fs.writeFileSync(packagePath, JSON.stringify(packageJson, null, 2) + '\n');
|
|
||||||
|
|
||||||
console.log(`${currentVersion}:${newVersion}`);
|
|
||||||
} catch (error) {
|
|
||||||
console.error('Error bumping version:', error.message);
|
|
||||||
process.exit(1);
|
|
||||||
}
|
|
||||||
VERSION_SCRIPT_EOF
|
|
||||||
|
|
||||||
- name: Create single atomic commit with all changes
|
|
||||||
if: steps.version-type.outputs.type != 'none'
|
|
||||||
id: version-bump
|
|
||||||
run: |
|
|
||||||
# Extract version type for commit title
|
|
||||||
VERSION_TYPE="${{ steps.version-type.outputs.type }}"
|
|
||||||
|
|
||||||
# Create clean commit message title
|
|
||||||
if [ "$VERSION_TYPE" = "major" ]; then
|
|
||||||
COMMIT_TITLE="🚀 Major Release"
|
|
||||||
elif [ "$VERSION_TYPE" = "minor" ]; then
|
|
||||||
COMMIT_TITLE="✨ Minor Release"
|
|
||||||
else
|
|
||||||
COMMIT_TITLE="🐛 Patch Release"
|
|
||||||
fi
|
|
||||||
|
|
||||||
# Get the safe base commit (last release tag or fallback)
|
|
||||||
LAST_TAG=$(git describe --tags --abbrev=0 2>/dev/null || echo "")
|
|
||||||
if [ -n "$LAST_TAG" ]; then
|
|
||||||
BASE_COMMIT=$(git rev-list -n 1 $LAST_TAG)
|
|
||||||
else
|
|
||||||
# Fallback: get commit from 1 week ago or first commit
|
|
||||||
BASE_COMMIT=$(git rev-list --since="1 week ago" --reverse HEAD | head -1 || git rev-list --max-parents=0 HEAD)
|
|
||||||
fi
|
|
||||||
|
|
||||||
echo "Base commit for squash: $BASE_COMMIT"
|
|
||||||
|
|
||||||
# Safely reset to base (soft reset preserves working directory)
|
|
||||||
git reset --soft $BASE_COMMIT
|
|
||||||
|
|
||||||
# Bump version using our safe Node.js script
|
|
||||||
VERSION_OUTPUT=$(node bump-version.js $VERSION_TYPE)
|
|
||||||
CURRENT_VERSION=$(echo $VERSION_OUTPUT | cut -d: -f1)
|
|
||||||
NEW_VERSION=$(echo $VERSION_OUTPUT | cut -d: -f2)
|
|
||||||
|
|
||||||
echo "current-version=$CURRENT_VERSION" >> $GITHUB_OUTPUT
|
|
||||||
echo "new-version=$NEW_VERSION" >> $GITHUB_OUTPUT
|
|
||||||
|
|
||||||
# Synchronize pnpm lockfile if it exists
|
|
||||||
if [ -f pnpm-lock.yaml ]; then
|
|
||||||
echo "Synchronizing pnpm lockfile..."
|
|
||||||
pnpm install --frozen-lockfile --ignore-scripts || echo "Lockfile sync completed with warnings"
|
|
||||||
fi
|
|
||||||
|
|
||||||
# Create single atomic commit with everything
|
|
||||||
git add -A
|
|
||||||
COMMIT_MSG="$COMMIT_TITLE
|
|
||||||
|
|
||||||
${{ steps.changelog.outputs.changelog }}"
|
|
||||||
|
|
||||||
git commit -m "$COMMIT_MSG" || {
|
|
||||||
echo "Commit failed, checking git status:"
|
|
||||||
git status
|
|
||||||
git diff --cached
|
|
||||||
exit 1
|
|
||||||
}
|
|
||||||
|
|
||||||
# Create git tag
|
|
||||||
git tag -a "v$NEW_VERSION" -m "Version $NEW_VERSION
|
|
||||||
|
|
||||||
${{ steps.changelog.outputs.changelog }}"
|
|
||||||
|
|
||||||
echo "Created single atomic commit with version $CURRENT_VERSION → $NEW_VERSION"
|
|
||||||
|
|
||||||
# Clean up temporary files
|
|
||||||
rm -f bump-version.js
|
|
||||||
|
|
||||||
- name: Push version changes safely
|
|
||||||
if: steps.version-type.outputs.type != 'none'
|
|
||||||
run: |
|
|
||||||
# Check if we need to force push (if git history was rewritten)
|
|
||||||
CURRENT_REMOTE_HEAD=$(git ls-remote origin main | cut -f1)
|
|
||||||
LOCAL_REMOTE_HEAD=$(git rev-parse origin/main)
|
|
||||||
|
|
||||||
if [ "$CURRENT_REMOTE_HEAD" != "$LOCAL_REMOTE_HEAD" ]; then
|
|
||||||
echo "⚠️ Remote main has been updated by another process"
|
|
||||||
echo "Current remote HEAD: $CURRENT_REMOTE_HEAD"
|
|
||||||
echo "Local remote HEAD: $LOCAL_REMOTE_HEAD"
|
|
||||||
echo "❌ ABORTING: Cannot safely force push. Another workflow may be running."
|
|
||||||
exit 1
|
|
||||||
fi
|
|
||||||
|
|
||||||
# Safe force push with lease (only if remote hasn't changed)
|
|
||||||
echo "🚀 Pushing squashed commit to main..."
|
|
||||||
git push --force-with-lease origin main || {
|
|
||||||
echo "❌ ERROR: Force push failed. This likely means:"
|
|
||||||
echo "1. Another workflow pushed to main after this one started"
|
|
||||||
echo "2. Branch protection rules are preventing the push"
|
|
||||||
echo "3. Insufficient permissions"
|
|
||||||
exit 1
|
|
||||||
}
|
|
||||||
|
|
||||||
echo "🏷️ Pushing tags..."
|
|
||||||
git push origin --tags
|
|
||||||
|
|
||||||
- name: Publish to NPM
|
- name: Publish to NPM
|
||||||
if: steps.version-type.outputs.type != 'none'
|
|
||||||
run: pnpm publish --access public --no-git-checks
|
run: pnpm publish --access public --no-git-checks
|
||||||
env:
|
env:
|
||||||
NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
|
NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
|
||||||
|
|
||||||
- name: Create GitHub Release
|
|
||||||
if: steps.version-type.outputs.type != 'none'
|
|
||||||
run: |
|
|
||||||
gh release create "v${{ steps.version-bump.outputs.new-version }}" \
|
|
||||||
--title "Release v${{ steps.version-bump.outputs.new-version }}" \
|
|
||||||
--notes "# Release v${{ steps.version-bump.outputs.new-version }}
|
|
||||||
|
|
||||||
${{ steps.changelog.outputs.changelog }}
|
|
||||||
|
|
||||||
---
|
|
||||||
|
|
||||||
**Version Info**: ${{ steps.version-type.outputs.type }} release (v${{ steps.version-bump.outputs.current-version }} → v${{ steps.version-bump.outputs.new-version }})
|
|
||||||
|
|
||||||
### Installation
|
|
||||||
|
|
||||||
\`\`\`bash
|
|
||||||
npm install ${{ steps.prerequisites.outputs.package-name }}@${{ steps.version-bump.outputs.new-version }}
|
|
||||||
\`\`\`
|
|
||||||
|
|
||||||
### Documentation
|
|
||||||
|
|
||||||
See the [README](https://github.com/${{ github.repository }}#readme) for usage instructions and full documentation."
|
|
||||||
env:
|
|
||||||
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
|
|
||||||
|
|
||||||
notify-success:
|
|
||||||
if: github.event_name == 'push' || (github.event.pull_request.merged == true && (startsWith(github.event.pull_request.head.ref, 'major/') || startsWith(github.event.pull_request.head.ref, 'minor/') || startsWith(github.event.pull_request.head.ref, 'patch/')))
|
|
||||||
needs: version-and-publish
|
|
||||||
runs-on: ubuntu-latest
|
|
||||||
steps:
|
|
||||||
- name: Success notification
|
|
||||||
if: needs.version-and-publish.outputs.new-version != ''
|
|
||||||
run: |
|
|
||||||
echo "🎉 Successfully published version ${{ needs.version-and-publish.outputs.new-version }} to NPM!"
|
|
||||||
echo "📦 Package: https://www.npmjs.com/package/${{ needs.version-and-publish.outputs.package-name }}"
|
|
||||||
echo "🏷️ GitHub Release: https://github.com/${{ github.repository }}/releases/tag/v${{ needs.version-and-publish.outputs.new-version }}"
|
|
||||||
echo "🔄 Version Type: ${{ needs.version-and-publish.outputs.version-type }}"
|
|
||||||
echo "📈 Version Change: v${{ needs.version-and-publish.outputs.current-version }} → v${{ needs.version-and-publish.outputs.new-version }}"
|
|
||||||
Reference in New Issue
Block a user