fix(ci): improve npm publish error detection in release workflow

- Check for success marker (+ package@version) instead of relying on exit codes
- Handle auth errors gracefully for GitHub Packages (401/403) and continue to npmjs
- Add explicit success notices and better error reporting
- Fixes false positive failures when npm publish returns non-zero despite success

Author: mars167

.git-ai/lancedb.tar.gz

@@ -1,3 +1,3 @@
 version https://git-lfs.github.com/spec/v1
-oid sha256:2398f5dd1592b08f6a39f43e197bf33a1e18d1060157206d441bb1d9c97a4438
-size 314362
+oid sha256:b30e7f67c4cfed10caec590edb722d7394c9dd1fb6d9631f3c4524242fb84bea
+size 314594

.github/workflows/release.yml

@@ -113,12 +113,19 @@ jobs:
         id: publish_github
         run: |
           npm pkg set name="@${{ github.repository_owner }}/git-ai"
-          npm publish --access public 2>&1 | tee publish_output.txt
+          npm publish --access public 2>&1 | tee publish_output.txt || true
           if grep -q "You cannot publish over the previously published versions" publish_output.txt; then
             echo "::notice::Version already exists on GitHub Packages, skipping"
             echo "skipped=true" >> "$GITHUB_OUTPUT"
-          elif [ ${PIPESTATUS[0]} -ne 0 ]; then
+          elif grep -q "^\+ @${{ github.repository_owner }}/git-ai@" publish_output.txt; then
+            echo "::notice::Successfully published to GitHub Packages"
+            echo "skipped=false" >> "$GITHUB_OUTPUT"
+          elif grep -q "npm error 40[13]" publish_output.txt; then
+            echo "::warning::GitHub Packages publish failed with auth error, continuing to npmjs.org"
+            echo "skipped=true" >> "$GITHUB_OUTPUT"
+          else
             echo "::error::Failed to publish to GitHub Packages"
+            cat publish_output.txt
             exit 1
           fi
         env:
@@ -144,11 +151,16 @@ jobs:
           if grep -q "You cannot publish over the previously published versions" publish_output.txt; then
             echo "::notice::Version already exists on npmjs.org, skipping"
             echo "skipped=true" >> "$GITHUB_OUTPUT"
+          elif grep -q "^\+ @mars167/git-ai@" publish_output.txt; then
+            echo "::notice::Successfully published to npmjs.org"
+            echo "skipped=false" >> "$GITHUB_OUTPUT"
           elif grep -q "npm error" publish_output.txt; then
             echo "::error::Failed to publish to npmjs.org"
             cat publish_output.txt
             exit 1
           else
+            echo "::warning::Unexpected npm publish output, check logs"
+            cat publish_output.txt
             echo "skipped=false" >> "$GITHUB_OUTPUT"
           fi
         env: