merge

Author: mattapperson

.github/actions/validate-sdk/action.yaml

@@ -0,0 +1,47 @@
+name: 'Validate SDK'
+description: 'Run build, typecheck, and tests for the SDK'
+
+runs:
+  using: 'composite'
+  steps:
+    - name: Setup Node.js
+      uses: actions/setup-node@v4
+      with:
+        node-version: '22'
+        cache: 'npm'
+
+    - name: Install dependencies
+      shell: bash
+      run: npm install
+
+    - name: Build SDK
+      shell: bash
+      run: npm run build
+
+    - name: Typecheck tests directory
+      shell: bash
+      run: npx tsc --noEmit --skipLibCheck --esModuleInterop --moduleResolution node --module esnext --target es2020 'tests/**/*.ts'
+
+    - name: Install examples dependencies
+      shell: bash
+      working-directory: examples
+      run: npm install
+
+    - name: Typecheck examples root
+      shell: bash
+      working-directory: examples
+      run: npx tsc --noEmit --skipLibCheck --esModuleInterop --moduleResolution node --module esnext --target es2020 '*.ts'
+
+    - name: Install nextjs-example dependencies
+      shell: bash
+      working-directory: examples/nextjs-example
+      run: npm install
+
+    - name: Typecheck nextjs-example
+      shell: bash
+      working-directory: examples/nextjs-example
+      run: npx tsc --noEmit
+
+    - name: Run tests
+      shell: bash
+      run: npm test

.github/workflows/pr-validation.yaml

@@ -0,0 +1,10 @@
+name: PR Validation
+
+on:
+  pull_request:
+    paths-ignore:
+      - .speakeasy/in.openapi.yaml
+
+jobs:
+  validate:
+    uses: ./.github/workflows/validation-checks.yaml

.github/workflows/speakeasy_run_on_pr.yaml

@@ -30,36 +30,8 @@ jobs:
       - name: Run Speakeasy
         run: speakeasy run
 
-      - name: Setup Node.js
-        uses: actions/setup-node@v4
-        with:
-          node-version: '22'
-          cache: 'npm'
-
-      - name: Install dependencies
-        run: npm install
-
-      - name: Build SDK
-        run: npm run build
-
-      - name: Typecheck tests directory
-        run: npx tsc --noEmit --skipLibCheck --esModuleInterop --moduleResolution node --module esnext --target es2020 'tests/**/*.ts'
-
-      - name: Install examples dependencies
-        working-directory: examples
-        run: npm install
-
-      - name: Typecheck examples root
-        working-directory: examples
-        run: npx tsc --noEmit --skipLibCheck --esModuleInterop --moduleResolution node --module esnext --target es2020 '*.ts'
-
-      - name: Install nextjs-example dependencies
-        working-directory: examples/nextjs-example
-        run: npm install
-
-      - name: Typecheck nextjs-example
-        working-directory: examples/nextjs-example
-        run: npx tsc --noEmit
+      - name: Validate SDK
+        uses: ./.github/actions/validate-sdk
 
       - name: Commit changes
         run: |

.github/workflows/validation-checks.yaml

@@ -0,0 +1,14 @@
+name: Validation Checks
+
+on:
+  workflow_call:
+
+jobs:
+  validate:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Validate SDK
+        uses: ./.github/actions/validate-sdk

package.json

@@ -55,7 +55,10 @@
   "scripts": {
     "lint": "eslint --cache --max-warnings=0 src",
     "build": "tsc",
-    "prepublishOnly": "npm run build"
+    "prepublishOnly": "npm run build",
+    "test": "vitest run",
+    "test:e2e": "vitest run tests/e2e",
+    "test:watch": "vitest"
   },
   "peerDependencies": {
     "@tanstack/react-query": "^5",

tests/e2e/chat.test.ts

@@ -1,6 +1,5 @@
 import { beforeAll, describe, expect, it } from "vitest";
 import { OpenRouter } from "../../src/sdk/sdk.js";
-import { ChatError } from "../../src/models/errors/chaterror.js";
 
 describe("Chat E2E Tests", () => {
   let client: OpenRouter;
@@ -115,6 +114,7 @@ describe("Chat E2E Tests", () => {
       const chunks: any[] = [];
 
       for await (const chunk of response) {
+        expect(chunk).toBeDefined();
         chunks.push(chunk);
       }
 
@@ -154,7 +154,7 @@ describe("Chat E2E Tests", () => {
 
       expect(chunkCount).toBeGreaterThan(0);
       expect(fullContent.length).toBeGreaterThan(0);
-    });
+    }, 10000)
 
     it("should include finish_reason in final chunk", async () => {
       const response = await client.chat.send({
@@ -181,134 +181,6 @@ describe("Chat E2E Tests", () => {
       }
 
       expect(foundFinishReason).toBe(true);
-    });
-  });
-
-  describe("Error Handling", () => {
-    it("should throw ErrorResponse with 401 status code for invalid API key", async () => {
-      const invalidClient = new OpenRouter({
-        apiKey: "invalid-api-key-12345",
-      });
-
-      try {
-        await invalidClient.chat.send({
-          model: "meta-llama/llama-3.2-1b-instruct",
-          messages: [
-            {
-              role: "user",
-              content: "Hello",
-            },
-          ],
-          stream: false,
-        });
-
-        // If we reach here, the test should fail
-        expect.fail("Expected an error to be thrown for invalid API key");
-      } catch (error) {
-        // Verify it's an ErrorResponse
-        expect(error).toBeInstanceOf(ChatError);
-
-        if (error instanceof ChatError) {
-          // Verify status code is 401 (Unauthorized)
-          expect(error.statusCode).toBe(401);
-
-          // Verify error structure
-          expect(error.error).toBeDefined();
-          expect(error.error.code).toBe(401);
-          expect(error.error.message).toBeDefined();
-          expect(typeof error.error.message).toBe("string");
-
-          // Verify the error message contains relevant information
-          expect(error.message.toLowerCase()).toMatch(/invalid|unauthorized|api key/i);
-        }
-      }
-    });
-
-    it("should throw ErrorResponse with 400 status code for invalid model", async () => {
-      try {
-        await client.chat.send({
-          model: "this-model-does-not-exist/invalid-model-name-12345",
-          messages: [
-            {
-              role: "user",
-              content: "Hello",
-            },
-          ],
-          stream: false,
-        });
-
-        // If we reach here, the test should fail
-        expect.fail("Expected an error to be thrown for invalid model");
-      } catch (error) {
-        // Verify it's an ErrorResponse
-        expect(error).toBeInstanceOf(ChatError);
-
-        if (error instanceof ChatError) {
-          // Verify status code is 400 (Bad Request)
-          expect(error.statusCode).toBe(400);
-
-          // Verify error structure
-          expect(error.error).toBeDefined();
-          expect(error.error.code).toBe(400);
-          expect(error.error.message).toBeDefined();
-          expect(typeof error.error.message).toBe("string");
-
-          // Verify the error message contains relevant information about invalid model
-          expect(error.message.toLowerCase()).toMatch(/model|invalid|not found/i);
-        }
-      }
-    });
-
-    it("should throw ErrorResponse with proper structure for invalid model in streaming mode", async () => {
-      await expect(async () => {
-        const response = await client.chat.send({
-          model: "this-model-does-not-exist/invalid-model-name-streaming",
-          messages: [
-            {
-              role: "user",
-              content: "Hello",
-            },
-          ],
-          stream: true,
-        });
-
-        // Consume the stream - error may be thrown here or during iteration
-        for await (const _chunk of response) {
-          // If we get chunks, that's unexpected
-        }
-      }).rejects.toThrow();
-
-      // Now test the actual error details
-      try {
-        const response = await client.chat.send({
-          model: "this-model-does-not-exist/invalid-model-name-streaming",
-          messages: [
-            {
-              role: "user",
-              content: "Hello",
-            },
-          ],
-          stream: true,
-        });
-
-        for await (const _chunk of response) {
-          // Continue consuming stream
-        }
-      } catch (error) {
-        // Verify it's an ErrorResponse
-        expect(error).toBeInstanceOf(ChatError);
-
-        if (error instanceof ChatError) {
-          // Verify status code is 400 (Bad Request)
-          expect(error.statusCode).toBe(400);
-
-          // Verify error structure
-          expect(error.error).toBeDefined();
-          expect(error.error.code).toBe(400);
-          expect(error.error.message).toBeDefined();
-          expect(typeof error.error.message).toBe("string");
-        }
-      }
-    });
+    }, 10000)
   });
 });