Merge branch 'main' into FedeZara/refactor/rename-fern-cli-error-to-task-abort-signal

Author: FedeZara

.claude/settings.json

@@ -41,6 +41,16 @@
     ]
   },
   "hooks": {
+    "WorktreeCreate": [
+      {
+        "hooks": [
+          {
+            "type": "command",
+            "command": "bash \"$(git rev-parse --show-toplevel)/scripts/worktree-create-hook.sh\""
+          }
+        ]
+      }
+    ],
     "PostToolUse": [
       {
         "matcher": "Edit|MultiEdit|Write",

.cursorindexingignore

@@ -34,8 +34,6 @@ seed/php-model/*
 !seed/php-model/seed.yml
 seed/php-sdk/*
 !seed/php-sdk/seed.yml
-seed/postman/*
-!seed/postman/seed.yml
 seed/pydantic/*
 !seed/pydantic/seed.yml
 seed/pydantic-v2/*
@@ -54,7 +52,5 @@ seed/rust-sdk/*
 !seed/rust-sdk/seed.yml
 seed/swift-sdk/*
 !seed/swift-sdk/seed.yml
-seed/ts-express/*
-!seed/ts-express/seed.yml
 seed/ts-sdk/*
-!seed/ts-sdk/seed.yml
+!seed/ts-sdk/seed.yml

.devin/settings.md

@@ -8,7 +8,7 @@ PR titles are validated by CI (`.github/workflows/lint-pr-title.yml`) using [sem
 
 **Allowed types**: `fix`, `feat`, `revert`, `break`, `chore`
 
-**Allowed scopes**: `docs`, `changelog`, `internal`, `cli`, `typescript`, `python`, `java`, `csharp`, `go`, `php`, `ruby`, `seed`, `postman`, `ci`, `lint`, `fastapi`, `spring`, `express`, `openapi`, `deps`, `deps-dev`, `fiber`, `pydantic`, `ai-search`, `swift`, `rust`
+**Allowed scopes**: `docs`, `changelog`, `internal`, `cli`, `typescript`, `python`, `java`, `csharp`, `go`, `php`, `ruby`, `seed`, `ci`, `lint`, `fastapi`, `spring`, `openapi`, `deps`, `deps-dev`, `pydantic`, `ai-search`, `swift`, `rust`
 
 **Examples**:
 - `chore(docs): update guidelines`

.github/CODEOWNERS

@@ -6,7 +6,6 @@ generators/go-v2/* @amckinney
 generators/java/* @tstanmay13
 generators/openapi/* @dsinghvi @amckinney
 generators/php/* @amckinney
-generators/postman/* @dsinghvi @amckinney
 generators/python/* @amckinney
 generators/python-v2/* @amckinney
 generators/ruby-v2/* @iamnamananand996

.github/actions/auto-update-seed/action.yaml

@@ -5,7 +5,7 @@ description: Run seed and then automatically create and merge PR with changes
 # However, we don't need to validate git diff since we are always updating the seed
 inputs:
   generator-name:
-    description: Generator to use (e.g., go-sdk, ruby-sdk)
+    description: Generator to use (e.g., go-sdk)
     required: true
   generator-path:
     description: The path to the source code of the generator (e.g., generators/go, generators/ruby)
@@ -25,6 +25,14 @@ inputs:
     description: The matrix strategy index of the job (to differentiate when parallelizing jobs)
     required: false
     default: "0"
+  dockerhub-username:
+    description: Docker Hub username for authenticated pulls (avoids rate limits)
+    required: false
+    default: ""
+  dockerhub-token:
+    description: Docker Hub token for authenticated pulls (avoids rate limits)
+    required: false
+    default: ""
 
 runs:
   using: "composite"
@@ -40,6 +48,8 @@ runs:
         allow-unexpected-failures: ${{ inputs.allow-unexpected-failures }}
         skip-scripts: ${{ inputs.skip-scripts }}
         skip-git-diff-check: true
+        dockerhub-username: ${{ inputs.dockerhub-username }}
+        dockerhub-token: ${{ inputs.dockerhub-token }}
 
     # Add changes first to simplify git diff checks, no changes will NOT error here
     - name: Check for changes

.github/actions/cached-seed/action.yaml

@@ -3,7 +3,7 @@ description: Run seed generation with caching for a specific generator.
 
 inputs:
   generator-name:
-    description: Generator to use (e.g., go-sdk, ruby-sdk)
+    description: Generator to use (e.g., go-sdk)
     required: true
   generator-path:
     description: The path to the source code of the generator (e.g., generators/go, generators/ruby)
@@ -20,13 +20,52 @@ inputs:
     description: Whether to allow unexpected failures during seed generation
     required: false
     default: "false"
+  dockerhub-username:
+    description: Docker Hub username for authenticated pulls (avoids rate limits)
+    required: false
+    default: ""
+  dockerhub-token:
+    description: Docker Hub token for authenticated pulls (avoids rate limits)
+    required: false
+    default: ""
 
 runs:
   using: "composite"
   steps:
     - name: Install
       uses: ./.github/actions/install
 
+    - name: Log in to Docker Hub
+      if: ${{ inputs.dockerhub-username != '' }}
+      uses: docker/login-action@v3
+      with:
+        username: ${{ inputs.dockerhub-username }}
+        password: ${{ inputs.dockerhub-token }}
+
+    - name: Clear stale Docker image cache
+      shell: bash
+      run: rm -rf /tmp/docker-cache
+
+    - name: Cache Docker images
+      id: docker-cache
+      uses: actions/cache@v4
+      with:
+        path: /tmp/docker-cache
+        key: ${{ runner.os }}-docker-${{ inputs.generator-name }}-${{ hashFiles('**/Dockerfile*') }}
+        restore-keys: |
+          ${{ runner.os }}-docker-${{ inputs.generator-name }}-
+
+    - name: Load cached Docker images
+      shell: bash
+      run: |
+        for f in /tmp/docker-cache/*.tar; do
+          [ -f "$f" ] && docker load -i "$f" || true
+        done
+
+    - name: Record pre-existing Docker images
+      shell: bash
+      run: docker images -q | sort -u > /tmp/docker-images-before.txt
+
     - name: Seed Test
       shell: bash
       env:
@@ -55,3 +94,19 @@ runs:
           echo "Seed test failed (attempt $attempt/3), retrying in 10s..."
           sleep 10
         done
+
+    - name: Save Docker images for cache
+      if: always()
+      shell: bash
+      run: |
+        if [ ! -f /tmp/docker-images-before.txt ]; then
+          echo "Skipping cache save: docker-images-before.txt not found"
+          exit 0
+        fi
+        mkdir -p /tmp/docker-cache
+        comm -13 /tmp/docker-images-before.txt <(docker images -q | sort -u) | while read -r id; do
+          img=$(docker inspect --format '{{index .RepoTags 0}}' "$id" 2>/dev/null || true)
+          [ -z "$img" ] && continue
+          fname=$(echo "$img" | tr '/:' '_')
+          docker save "$img" -o "/tmp/docker-cache/${fname}.tar" 2>/dev/null || true
+        done

.github/actions/fetch-benchmark-specs/action.yaml

@@ -0,0 +1,57 @@
+name: Fetch Benchmark Specs
+description: Download large public OpenAPI specs for SDK generation benchmarking
+
+outputs:
+  specs-fetched:
+    description: "Space-separated list of spec names that were successfully fetched"
+    value: ${{ steps.summary.outputs.specs-fetched }}
+
+runs:
+  using: composite
+  steps:
+    - name: Fetch Square OpenAPI spec
+      id: fetch-square
+      shell: bash
+      run: |
+        # Pinned to a specific commit SHA for reproducibility.
+        # A live endpoint would be preferable but Square does not publish one.
+        FETCH_OK=false
+        for ATTEMPT in 1 2 3; do
+          echo "Attempt ${ATTEMPT}/3: Fetching Square OpenAPI spec..."
+          if curl -sSfL --retry 2 --retry-delay 5 \
+            "https://raw.githubusercontent.com/square/connect-api-specification/ba11c0a1f006b81dcd3633adb9090e9b2f03fef7/api.json" \
+            -o benchmarks/fern/apis/square/openapi.json; then
+            BYTES=$(wc -c < benchmarks/fern/apis/square/openapi.json)
+            echo "Downloaded Square OpenAPI spec (${BYTES} bytes)"
+            if [ "$BYTES" -ge 100000 ]; then
+              FETCH_OK=true
+              break
+            fi
+            echo "::warning::Square spec unexpectedly small (${BYTES} bytes) on attempt ${ATTEMPT}"
+          else
+            echo "::warning::curl failed on attempt ${ATTEMPT}"
+          fi
+          [ "$ATTEMPT" -lt 3 ] && sleep 5
+        done
+        if [ "$FETCH_OK" = "true" ]; then
+          echo "square-ok=true" >> $GITHUB_OUTPUT
+        else
+          echo "::warning::Failed to fetch Square spec after 3 attempts. Skipping Square benchmarks."
+          rm -f benchmarks/fern/apis/square/openapi.json
+          echo "square-ok=false" >> $GITHUB_OUTPUT
+        fi
+
+    - name: Summarize fetched specs
+      id: summary
+      shell: bash
+      run: |
+        SPECS=""
+        if [ "${{ steps.fetch-square.outputs.square-ok }}" = "true" ]; then
+          SPECS="square"
+        fi
+        if [ -z "$SPECS" ]; then
+          echo "::error::No benchmark specs could be fetched. Benchmark will be skipped."
+          exit 1
+        fi
+        echo "specs-fetched=$SPECS" >> $GITHUB_OUTPUT
+        echo "Successfully fetched specs: $SPECS"

.github/actions/get-test-matrix/action.yaml

@@ -60,13 +60,11 @@ runs:
         if [[ "${{inputs.package-alphabetically}}" == "false" || \
         "${{ inputs.sdk-name }}" == "fastapi" || \
         "${{ inputs.sdk-name }}" == "go-model" || \
-        "${{ inputs.sdk-name }}" == "java-spring" || \
         "${{ inputs.sdk-name }}" == "openapi" || \
         "${{ inputs.sdk-name }}" == "php-model" || \
-        "${{ inputs.sdk-name }}" == "postman" || \
-        "${{ inputs.sdk-name }}" == "ruby-sdk" || \
+        "${{ inputs.sdk-name }}" == "ruby-sdk-v2" || \
         "${{ inputs.sdk-name }}" == "rust-model" || \
-        "${{ inputs.sdk-name }}" == "ts-express" ]]; then
+        false ]]; then
           echo "split-tests=false" >> $GITHUB_OUTPUT
         else
           echo "split-tests=true" >> $GITHUB_OUTPUT

.github/actions/install/action.yaml

@@ -24,11 +24,6 @@ runs:
           ${{ runner.os }}-turbo-${{ github.job }}-
           ${{ runner.os }}-turbo-
 
-    # Ensure npm 11.5.1 or later is installed
-    - name: ⎔ Update npm
-      shell: bash
-      run: npm install -g npm@latest
-
     - name: 📥 Install dependencies
       shell: bash
       run: |

.github/actions/run-benchmark/action.yaml

@@ -0,0 +1,66 @@
+name: Run SDK Generation Benchmark
+description: Run SDK generation for a single generator against benchmark specs and record timing
+
+inputs:
+  generator:
+    description: "Generator name (e.g. ts-sdk, python-sdk)"
+    required: true
+  skip-scripts:
+    description: "If true, skip post-generation build/test scripts (--skip-scripts). Measures generator-only time."
+    required: false
+    default: "false"
+
+runs:
+  using: composite
+  steps:
+    - name: Build seed CLI
+      shell: bash
+      run: pnpm seed:build
+
+    - name: Run benchmark for ${{ inputs.generator }}
+      shell: bash
+      run: |
+        GENERATOR="${{ inputs.generator }}"
+        RESULTS_DIR="benchmark-results"
+        mkdir -p "$RESULTS_DIR"
+
+        for SPEC in square; do
+          # Skip specs whose OAS file was not successfully fetched
+          if [ ! -f "benchmarks/fern/apis/${SPEC}/openapi.json" ]; then
+            echo "::warning::Skipping ${GENERATOR} x ${SPEC} — OpenAPI spec not available"
+            echo "{\"generator\":\"${GENERATOR}\",\"spec\":\"${SPEC}\",\"duration_seconds\":0,\"exit_code\":-1,\"skipped\":true}" \
+              >> "${RESULTS_DIR}/${GENERATOR}.jsonl"
+            continue
+          fi
+
+          echo "::group::Benchmarking ${GENERATOR} against ${SPEC}"
+
+          # $EPOCHSECONDS requires bash 5+ (available on GitHub Actions Ubuntu runners)
+          START_TIME=$EPOCHSECONDS
+          EXIT_CODE=0
+
+          # Run generation (--skip-scripts omits post-gen build/test for cleaner signal)
+          SKIP_FLAG=""
+          if [ "${{ inputs.skip-scripts }}" = "true" ]; then
+            SKIP_FLAG="--skip-scripts"
+          fi
+          pnpm seed:local run \
+            --generator "${GENERATOR}" \
+            --path "benchmarks/fern/apis/${SPEC}" \
+            ${SKIP_FLAG} 2>&1 || EXIT_CODE=$?
+
+          END_TIME=$EPOCHSECONDS
+          DURATION=$(( END_TIME - START_TIME ))
+
+          if [ "$EXIT_CODE" -ne 0 ]; then
+            echo "::warning::Benchmark ${GENERATOR} x ${SPEC} exited with code ${EXIT_CODE} after ${DURATION}s"
+          fi
+
+          echo "Benchmark: ${GENERATOR} x ${SPEC} = ${DURATION}s (exit ${EXIT_CODE})"
+
+          # Write result as JSON (includes exit code so failures are distinguishable)
+          echo "{\"generator\":\"${GENERATOR}\",\"spec\":\"${SPEC}\",\"duration_seconds\":${DURATION},\"exit_code\":${EXIT_CODE}}" \
+            >> "${RESULTS_DIR}/${GENERATOR}.jsonl"
+
+          echo "::endgroup::"
+        done