Fixes

Author: David Crow

.github/workflows/prep-plugin-releases.yml

@@ -34,7 +34,7 @@ on:
         default: true
 
 env:
-  GITHUB_TOKEN: ${{ secrets.MOBILE_PLUGIN_RELEASE_PAT }}
+  GH_TOKEN: ${{ secrets.MOBILE_PLUGIN_RELEASE_PAT }}
 
 jobs:
   centralized-plugin-releases:
@@ -82,7 +82,7 @@ jobs:
             "${{ github.event.inputs.android_version }}" \
             $PLUGIN_ARGS
         env:
-          GITHUB_TOKEN: ${{ secrets.MOBILE_PLUGIN_RELEASE_PAT }}
+          GH_TOKEN: ${{ secrets.MOBILE_PLUGIN_RELEASE_PAT }}
           GEMINI_API_KEY: ${{ secrets.GEMINI_API_KEY }}
 
       - name: Display release summary

.github/workflows/prep-proxy-release.yml

@@ -4,7 +4,7 @@ on:
   workflow_dispatch:
 
 env:
-  BUNDLE_PATH: vendor/bundle
+  GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 
 jobs:
   prepare-release:
@@ -16,10 +16,7 @@ jobs:
 
       - name: Detect SDK versions
         id: detect
-        run: |
-          ./scripts/detect_sdk_versions.sh
-        env:
-          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+        run: ./scripts/detect_sdk_versions.sh
 
       - name: Display version summary
         run: |
@@ -40,8 +37,7 @@ jobs:
           EOF
 
       - name: Prepare release
-        run: |
-          ./scripts/prep_proxy_release.sh
+        run: ./scripts/prep_proxy_release.sh
 
       - name: Create Pull Request
         id: cpr

scripts/detect_sdk_versions.sh

@@ -1,4 +1,4 @@
-#!/bin/bash
+#!/usr/bin/env bash
 set -e
 
 # Detect SDK versions and calculate proxy version bump
@@ -7,6 +7,9 @@ set -e
 SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
 REPO_ROOT="$(cd "$SCRIPT_DIR/.." && pwd)"
 
+# Source shared utilities
+source "$SCRIPT_DIR/lib/version_utils.sh"
+
 # Colors for output
 RED='\033[0;31m'
 GREEN='\033[0;32m'
@@ -56,25 +59,6 @@ if [ -z "$LATEST_ANDROID_VERSION" ]; then
 fi
 echo -e "Latest Android SDK:    ${BOLD}$LATEST_ANDROID_VERSION${NC}"
 
-# Function to determine bump type (major, minor, patch)
-determine_bump_type() {
-    local old=$1
-    local new=$2
-
-    IFS='.' read -r old_major old_minor old_patch <<< "$old"
-    IFS='.' read -r new_major new_minor new_patch <<< "$new"
-
-    if [ "$new_major" -gt "$old_major" ]; then
-        echo "major"
-    elif [ "$new_minor" -gt "$old_minor" ]; then
-        echo "minor"
-    elif [ "$new_patch" -gt "$old_patch" ]; then
-        echo "patch"
-    else
-        echo "none"
-    fi
-}
-
 # Determine bump types for each SDK
 IOS_BUMP=$(determine_bump_type "$CURRENT_IOS_VERSION" "$LATEST_IOS_VERSION")
 ANDROID_BUMP=$(determine_bump_type "$CURRENT_ANDROID_VERSION" "$LATEST_ANDROID_VERSION")

scripts/lib/changelog_prompt.txt

@@ -44,5 +44,5 @@ IMPORTANT INSTRUCTIONS:
 2. Decide SIMPLE or DETAILED based on criteria above
 3. Return ONLY the formatted changelog (no explanations, no preamble)
 4. If unsure or SDK changelogs lack substance, use SIMPLE
-5. Ensure all version numbers in links match exactly (no 'v' prefix in URLs)
+5. Do not include 'v' prefix in version numbers within URLs
 6. Use release type: "Major" for X.0.0, "Minor" for X.Y.0, "Patch" for X.Y.Z

scripts/lib/generate_changelog.sh

@@ -1,11 +1,14 @@
-#!/bin/bash
+#!/usr/bin/env bash
 # Intelligent changelog generation using Gemini CLI
 # Fetches SDK changelogs, analyzes them, and generates appropriate format
 
 # Get script directory
 LIB_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
 PROMPT_TEMPLATE="$LIB_DIR/changelog_prompt.txt"
 
+# Source shared utilities
+source "$LIB_DIR/version_utils.sh"
+
 # Fetch SDK changelog from GitHub release notes
 fetch_github_changelog() {
     local repo="$1"
@@ -131,7 +134,7 @@ validate_and_fix() {
 
     # 1. Fix version format (remove 'v' prefix if present)
     if echo "$changelog_content" | grep -q "Version v[0-9]"; then
-        sed -i '' 's/Version v\([0-9]\)/Version \1/g' "$changelog_file"
+        sedi 's/Version v\([0-9]\)/Version \1/g' "$changelog_file"
         echo "    ✓ Fixed version format (removed 'v' prefix)" >&2
         fixes_made=true
     fi
@@ -145,14 +148,14 @@ validate_and_fix() {
     # 3. Fix malformed GitHub links
     if echo "$changelog_content" | grep -q "github.com/urbanairship/.*releases/tag/[^)]*[^0-9.)]"; then
         # Remove any trailing characters after version number in links
-        sed -i '' -E 's|(https://github.com/urbanairship/[^/]+/releases/tag/[0-9.]+)[^)]*|\1|g' "$changelog_file"
+        sedi -E 's|(https://github.com/urbanairship/[^/]+/releases/tag/[0-9.]+)[^)]*|\1|g' "$changelog_file"
         echo "    ✓ Fixed malformed GitHub release links" >&2
         fixes_made=true
     fi
 
     # 4. Remove trailing whitespace
     if grep -q "[[:space:]]$" "$changelog_file"; then
-        sed -i '' 's/[[:space:]]*$//' "$changelog_file"
+        sedi 's/[[:space:]]*$//' "$changelog_file"
         echo "    ✓ Removed trailing whitespace" >&2
         fixes_made=true
     fi
@@ -254,10 +257,11 @@ validate_pr() {
     local pr_title=$(echo "$pr_data" | jq -r '.title')
     local pr_body=$(echo "$pr_data" | jq -r '.body')
 
-    # Expected title format: "Release X.Y.Z"
+    # Expected title format: "Release X.Y.Z" or "[TEST] Release X.Y.Z"
     local expected_title="Release ${plugin_version}"
+    local expected_test_title="[TEST] Release ${plugin_version}"
 
-    if [ "$pr_title" != "$expected_title" ]; then
+    if [ "$pr_title" != "$expected_title" ] && [ "$pr_title" != "$expected_test_title" ]; then
         echo "    ⚠️  WARNING: PR title is '$pr_title', expected '$expected_title'" >&2
     fi
 

scripts/prep_proxy_release.sh

@@ -1,4 +1,4 @@
-#!/bin/bash
+#!/usr/bin/env bash
 set -e
 
 # Prepare proxy release: update versions, dependencies, changelog
@@ -7,6 +7,9 @@ set -e
 SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
 REPO_ROOT="$(cd "$SCRIPT_DIR/.." && pwd)"
 
+# Source shared utilities
+source "$SCRIPT_DIR/lib/version_utils.sh"
+
 # Parse arguments
 TEST_MODE=false
 if [ "$1" = "--test" ]; then
@@ -44,8 +47,8 @@ echo -e "${BLUE}Updating files...${NC}"
 
 # Update AirshipFrameworkProxy.podspec
 if [ "$TEST_MODE" = "false" ]; then
-    sed -i '' "s/s.version[[:space:]]*=[[:space:]]*\"[0-9]*\.[0-9]*\.[0-9]*\"/s.version                 = \"${PROXY_VERSION}\"/" "$REPO_ROOT/AirshipFrameworkProxy.podspec"
-    sed -i '' "s/s.dependency[[:space:]]*'Airship',[[:space:]]*\"[0-9]*\.[0-9]*\.[0-9]*\"/s.dependency                'Airship', \"${IOS_VERSION}\"/" "$REPO_ROOT/AirshipFrameworkProxy.podspec"
+    sedi "s/s.version[[:space:]]*=[[:space:]]*\"[0-9]*\.[0-9]*\.[0-9]*\"/s.version                 = \"${PROXY_VERSION}\"/" "$REPO_ROOT/AirshipFrameworkProxy.podspec"
+    sedi "s/s.dependency[[:space:]]*'Airship',[[:space:]]*\"[0-9]*\.[0-9]*\.[0-9]*\"/s.dependency                'Airship', \"${IOS_VERSION}\"/" "$REPO_ROOT/AirshipFrameworkProxy.podspec"
     echo "✓ Updated AirshipFrameworkProxy.podspec"
 else
     echo "  Would update AirshipFrameworkProxy.podspec:"
@@ -55,7 +58,7 @@ fi
 
 # Update Package.swift
 if [ "$TEST_MODE" = "false" ]; then
-    sed -i '' "s/from: \"[0-9]*\.[0-9]*\.[0-9]*\"/from: \"${IOS_VERSION}\"/" "$REPO_ROOT/Package.swift"
+    sedi "s/from: \"[0-9]*\.[0-9]*\.[0-9]*\"/from: \"${IOS_VERSION}\"/" "$REPO_ROOT/Package.swift"
     echo "✓ Updated Package.swift"
 else
     echo "  Would update Package.swift:"
@@ -64,8 +67,8 @@ fi
 
 # Update android/gradle/libs.versions.toml
 if [ "$TEST_MODE" = "false" ]; then
-    sed -i '' "s/airshipProxy = '[0-9]*\.[0-9]*\.[0-9]*'/airshipProxy = '${PROXY_VERSION}'/" "$REPO_ROOT/android/gradle/libs.versions.toml"
-    sed -i '' "s/airship = '[0-9]*\.[0-9]*\.[0-9]*'/airship = '${ANDROID_VERSION}'/" "$REPO_ROOT/android/gradle/libs.versions.toml"
+    sedi "s/airshipProxy = '[0-9]*\.[0-9]*\.[0-9]*'/airshipProxy = '${PROXY_VERSION}'/" "$REPO_ROOT/android/gradle/libs.versions.toml"
+    sedi "s/airship = '[0-9]*\.[0-9]*\.[0-9]*'/airship = '${ANDROID_VERSION}'/" "$REPO_ROOT/android/gradle/libs.versions.toml"
     echo "✓ Updated android/gradle/libs.versions.toml"
 else
     echo "  Would update android/gradle/libs.versions.toml:"