fix: fix scripts

modelcontextprotocol · Jan 13, 2025 · 9f86ee5 · 9f86ee5
1 parent c9b1adf
commit 9f86ee5
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 6 deletions.
diff --git a/.github/workflows/release-check.yml b/.github/workflows/release-check.yml
@@ -30,6 +30,8 @@ jobs:
   check-release:
     needs: prepare
     runs-on: ubuntu-latest
+    outputs:
+      release: ${{ steps.check.outputs.release }}
     strategy:
       matrix:
         directory: ${{ fromJson(needs.prepare.outputs.matrix) }}
@@ -53,8 +55,15 @@ jobs:
         run: uv python install
 
       - name: Check release
+        id: check
         run: |
-          uv run --script scripts/release.py --dry-run "${{ matrix.directory }}" "${{ needs.prepare.outputs.last_release }}" | tee -a "$GITHUB_OUTPUT"
+          output=$(uv run --script scripts/release.py --dry-run "${{ matrix.directory }}" "${{ needs.prepare.outputs.last_release }}" \
+            | grep -o -E "[a-zA-Z0-9\-]+@[0-9]+\.[0-9]+\.[0-9]+" || true)
+          if [ ! -z "$output" ]; then
+            echo "release<<EOF" >> $GITHUB_OUTPUT
+            echo "$output" >> $GITHUB_OUTPUT
+            echo "EOF" >> $GITHUB_OUTPUT
+          fi
 
   check-tag:
     needs: [prepare, check-release]
@@ -64,16 +73,17 @@ jobs:
 
       - name: Simulate tag creation
         run: |
-          if [ -s "$GITHUB_OUTPUT" ]; then
+          echo "${{ needs.check-release.outputs.release }}" > packages.txt
+          if [ -s packages.txt ]; then
             DATE=$(date +%Y.%m.%d)
             echo "🔍 Dry run: Would create tag v${DATE} if this was a real release"
 
             echo "# Release ${DATE}" > notes.md
             echo "" >> notes.md
             echo "## Updated Packages" >> notes.md
             while IFS= read -r line; do
-              echo "- ${line}" >> notes.md
-            done < "$GITHUB_OUTPUT"
+              echo "- $line" >> notes.md
+            done < packages.txt
 
             echo "🔍 Would create release with following notes:"
             cat notes.md

diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
@@ -64,7 +64,7 @@ jobs:
         env:
           NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
           UV_PUBLISH_TOKEN: ${{ secrets.PYPI_TOKEN }}
-        run: uv run --script scripts/release.py "${{ matrix.directory }}" "${{ needs.prepare.outputs.last_release }}" >> "$GITHUB_OUTPUT"
+          run: uv run --script scripts/release.py "${{ matrix.directory }}" "${{ needs.prepare.outputs.last_release }}" >> "$GITHUB_OUTPUT"
 
   create-release:
     needs: [prepare, release]

diff --git a/scripts/release.py b/scripts/release.py
@@ -171,7 +171,7 @@ def main(directory: Path, git_hash: GitHash, dry_run: bool) -> int:
         if not dry_run:
             click.echo(f"{name}@{version}")
         else:
-            click.echo(f"🔍 Dry run: Would have published {name}@{version} if this was a real release")
+            click.echo(f"Dry run: Would have published {name}@{version}")
         return 0
     except Exception as e:
         return 1