Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions .github/workflows/publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,12 @@ jobs:
# Get version from main package
VERSION=$(node -p "require('./packages/connect/package.json').version")

# Skip if release already exists
if gh release view "v$VERSION" > /dev/null 2>&1; then
echo "Release v$VERSION already exists, skipping"
exit 0
fi
Comment on lines +63 to +67
Copy link

Copilot AI Feb 2, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Consider checking for the existing release at the start of the job, before the npm publish step. This would provide better fail-fast behavior and clearer feedback. Additionally, consider checking if the npm package version already exists to provide a complete idempotency check. The current placement after npm publish means the workflow will still execute build, test, and npm publish steps even when the release already exists, wasting CI resources and potentially causing confusing errors if the npm version was also already published.

Copilot uses AI. Check for mistakes.

# Create release
gh release create "v$VERSION" \
--title "v$VERSION" \
Expand Down