feat(ops): fix Github action to publish on NPM (#45)

Author: Dam-Buty

.github/workflows/publish.yaml

@@ -12,7 +12,6 @@ jobs:
     needs: [tests]
     steps:
       - uses: actions/checkout@v4
-      # Setup .npmrc file to publish to npm
       - uses: actions/setup-node@v3
         with:
           node-version: '19.x'

package.json

@@ -15,7 +15,8 @@
   "scripts": {
     "build": "tsup ./src",
     "test": "jest --detectOpenHandles --runInBand --watchAll=false",
-    "prepare": "husky install"
+    "prepare": "husky install",
+    "release": "./release.sh"
   },
   "author": "Literal AI",
   "license": "Apache-2.0",

release.sh

@@ -0,0 +1,27 @@
+#! /bin/sh
+
+GREEN='\033[0;32m'
+YELLOW='\033[0;33m'
+NC='\033[0m'
+
+echo "🪜 Incrementing version number"
+npm version patch
+
+NEW_VERSION=$(npm pkg get version | tr -d '"')
+
+echo "🌿 Creating release branch for v$NEW_VERSION"
+git checkout -b release/$NEW_VERSION
+git add .
+git commit -m "chore: bump to $NEW_VERSION"
+git push origin release/$NEW_VERSION
+
+echo
+echo
+echo -e "----------------------------------------"
+echo -e "|| ${GREEN} Your release is ready for review ${NC} ||"
+echo -e "----------------------------------------"
+echo -e "1️⃣  Go to ${YELLOW}https://github.com/Chainlit/typescript-client/compare/main...release/$NEW_VERSION${NC}"
+echo -e "2️⃣  Create a pull request and merge it to main"
+echo -e "3️⃣  Go to ${YELLOW}https://github.com/Chainlit/typescript-client/actions/workflows/publish.yaml${NC}"
+echo -e "4️⃣  Click on 'Run workflow'"
+echo -e "5️⃣  Rock'n'roll 🎸"

tests/api.test.ts

@@ -182,6 +182,8 @@ describe('End to end tests for the SDK', function () {
       })
       .send();
 
+    await new Promise((resolve) => setTimeout(resolve, 1000));
+
     const fetchedStep = await client.api.getStep(step.id!);
     expect(fetchedStep?.id).toBe(step.id);
     expect(fetchedStep?.error).toBe('test');
@@ -214,6 +216,8 @@ describe('End to end tests for the SDK', function () {
       })
       .send();
 
+    await new Promise((resolve) => setTimeout(resolve, 1000));
+
     const fetchedStep = await client.api.getStep(step.id!);
     expect(fetchedStep?.id).toBe(step.id);
     expect(fetchedStep?.error).toBe('test');
@@ -241,6 +245,8 @@ describe('End to end tests for the SDK', function () {
       throw new Error('Step id is null');
     }
 
+    await new Promise((resolve) => setTimeout(resolve, 1000));
+
     const steps = await client.api.getSteps({
       filters: [
         {

tests/integration/openai.test.ts

@@ -290,6 +290,8 @@ describe('OpenAI Instrumentation', () => {
         });
       });
 
+      await new Promise((resolve) => setTimeout(resolve, 1000));
+
       const {
         data: [step]
       } = await client.api.getSteps({
@@ -305,7 +307,7 @@ describe('OpenAI Instrumentation', () => {
 
       expect(step?.threadId).toBe(threadId);
       expect(step?.parentId).toBe(parentId);
-    });
+    }, 30_000);
 
     it("doesn't mix up threads and steps", async () => {
       const testId = uuidv4();
@@ -399,6 +401,6 @@ describe('OpenAI Instrumentation', () => {
       expect(firstGeneration?.parentId).toEqual(firstStep?.id);
       expect(secondGeneration?.threadId).toEqual(secondThreadId);
       expect(secondGeneration?.parentId).toEqual(secondStep?.id);
-    }, 30000);
+    }, 30_000);
   });
 });