refactor: fold ruby fix into build, streamline update-bindings workflow

- Call scripts/fix-ruby-escaping.sh from bindings/ruby/build.sh so a single
  ./scripts/build.sh always emits valid Ruby; drop the now-redundant call
  from update-bindings.sh
- Workflow: use uv + prek instead of setup-python + pre-commit, run the
  linter once (ruff --fix is single-pass), and only bump the version and
  open a PR when the regenerated bindings actually differ

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: cloudsmith-iduffy

.github/workflows/update-bindings.yml

@@ -27,15 +27,29 @@ jobs:
         with:
           ruby-version: "3.4"
 
-      - name: Set up Python
-        uses: actions/setup-python@v5
-        with:
-          python-version: "3.12"
+      - name: Set up uv
+        uses: astral-sh/setup-uv@v5
 
-      - name: Install pre-commit
-        run: pip install pre-commit
+      - name: Generate bindings
+        run: ./scripts/build.sh
 
-      - name: Determine versions
+      - name: Apply lint autofixes
+        # ruff --fix applies every fixable issue in a single pass; a non-zero exit
+        # only signals that it modified files, which is expected and desired.
+        run: uvx prek run --all-files || true
+
+      - name: Detect changes
+        id: detect
+        run: |
+          if [ -n "${{ inputs.version }}" ] || [ -n "$(git status --porcelain)" ]; then
+            echo "changed=true" >> "$GITHUB_OUTPUT"
+          else
+            echo "changed=false" >> "$GITHUB_OUTPUT"
+            echo "Bindings already match the current API; no PR needed."
+          fi
+
+      - name: Bump version and regenerate
+        if: steps.detect.outputs.changed == 'true'
         id: versions
         run: |
           api_version=$(curl -s --max-time 30 "https://api.cloudsmith.io/status/check/basic/" | jq -r '.version')
@@ -53,30 +67,15 @@ jobs:
             new_version="${major}.${minor}.$((patch + 1))"
           fi
 
+          sed -i -E "s/^package_version=.*/package_version=\"${new_version}\"/" scripts/common.sh
+          ./scripts/build.sh
+          uvx prek run --all-files || true
+
           echo "api_version=$api_version" >> "$GITHUB_OUTPUT"
           echo "new_version=$new_version" >> "$GITHUB_OUTPUT"
 
-      - name: Bump package_version
-        run: |
-          sed -i -E 's/^package_version=.*/package_version="${{ steps.versions.outputs.new_version }}"/' scripts/common.sh
-
-      - name: Generate bindings
-        run: ./scripts/build.sh
-
-      - name: Fix Ruby apostrophe escaping
-        run: ./scripts/fix-ruby-escaping.sh
-
-      - name: Run pre-commit to convergence
-        run: |
-          for attempt in 1 2 3 4 5; do
-            if pre-commit run --all-files; then
-              exit 0
-            fi
-          done
-          echo "pre-commit hooks did not converge after 5 attempts" >&2
-          exit 1
-
       - name: Create pull request
+        if: steps.detect.outputs.changed == 'true'
         uses: peter-evans/create-pull-request@v7
         with:
           branch: automated/update-bindings-v${{ steps.versions.outputs.new_version }}

bindings/ruby/build.sh

@@ -38,3 +38,5 @@ docker container run --rm -v $self_dir:/local "${swagger_codegen_cli_image:?}" g
     -o /local/src \
     -t /local/templates \
     $common_codegen_options
+
+$root_dir/scripts/fix-ruby-escaping.sh

scripts/update-bindings.sh

@@ -142,14 +142,6 @@ generate_bindings() {
         log_error "Failed to generate bindings"
         exit 1
     fi
-
-    log_info "Fixing Ruby apostrophe escaping..."
-    if ./scripts/fix-ruby-escaping.sh; then
-        log_success "Ruby apostrophe escaping fixed"
-    else
-        log_error "Failed to fix Ruby apostrophe escaping"
-        exit 1
-    fi
 }
 
 run_precommit_to_convergence() {