Merge pull request #17 from WickrInc/await-get-client-state

fix: await getClientState, update github actions

Author: dwickr

.github/workflows/npm-publish.yml

@@ -10,22 +10,26 @@ on:
 jobs:
   build:
     runs-on: ubuntu-latest
+    permissions:
+      contents: read
     steps:
       - uses: actions/checkout@v2
       - uses: actions/setup-node@v1
         with:
-          node-version: 16
+          node-version: 20
       - run: npm ci
       - run: npm test
 
   publish-npm:
     needs: build
     runs-on: ubuntu-latest
+    permissions:
+      contents: read
     steps:
       - uses: actions/checkout@v2
       - uses: actions/setup-node@v1
         with:
-          node-version: 16
+          node-version: 20
           registry-url: https://registry.npmjs.org/
       - run: npm ci
       - run: npm publish

lib/bot.js

@@ -21,7 +21,7 @@ class WickrBot extends EventEmitter {
   }
 
   async start() {
-    await this.wickr.clientInit(this.username)
+    this.wickr.clientInit(this.username)
     await this.waitForState('RUNNING')
     this.emit('start')
   }
@@ -341,7 +341,7 @@ class WickrBot extends EventEmitter {
   }
 
   async waitForState(state, retries=10, interval=1000, exponential=true) {
-    const curState = this.wickr.getClientState()
+    const curState = await this.wickr.getClientState()
 
     if (curState === state) return
     else if (retries) {

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "wickrbot",
-  "version": "0.4.0",
+  "version": "0.4.1",
   "description": "Framework for WickrIO bots",
   "main": "lib/bot.js",
   "types": "types/index.d.ts",