node-vk-bot-api - npm Package Compare versions

Comparing version 3.3.3 to 3.3.4

lib/context.js

@@ -14,3 +14,3 @@ class Context {
   reply(...args) {
-    this.bot.sendMessage(this.message.peer_id || this.message.user_id, ...args);
+    return this.bot.sendMessage(this.message.peer_id || this.message.user_id, ...args);
   }
@@ -17,0 +17,0 @@ }

lib/utils/execute.js

 const api = require('../api');
+const ApiError = require('../errors/ApiError');
 
 module.exports = (methods, token) => {
-  // eslint-disable-next-line no-param-reassign
-  methods = methods.map((item, index) => ({
-    ...item,
-    code: `"${index + 1}": ${item.code}`,
-  }));
+  const promises = [];
+  const calls = [];
 
-  for (let i = 0, j = Math.ceil(methods.length / 25); i < j; i++) {
-    const code = methods
+  methods.forEach((item, index) => {
+    promises.push({
+      resolve: item.resolve,
+      reject: item.reject,
+    });
+
+    calls.push(`"${index + 1}": ${item.code}`);
+  });
+
+  for (let i = 0, j = Math.ceil(calls.length / 25); i < j; i++) {
+    const code = calls
       .slice(i * 25, i * 25 + 25)
-      .map(item => item.code)
       .join(',');
@@ -21,10 +27,9 @@
       .then(({ response, execute_errors: errors = [] }) => {
-        // eslint-disable-next-line no-param-reassign
-        errors = errors.filter(item => item.method !== 'execute');
+        const apiErrors = errors.filter(item => item.method !== 'execute');
 
         Object.entries(response).forEach(([index, response]) => {
-          const { resolve, reject } = methods[index - 1];
+          const { resolve, reject } = promises[index - 1];
 
           if (!response) {
-            reject(errors.shift());
+            reject(new ApiError(apiErrors.shift()));
           } else {
@@ -31,0 +36,0 @@ resolve(response);

package.json

 {
   "name": "node-vk-bot-api",
-  "version": "3.3.3",
+  "version": "3.3.4",
   "description": "🤖 VK bot framework for Node.js, based on Bots Long Poll API and Callback API",
@@ -5,0 +5,0 @@ "main": "lib/index.js",

test/execute.test.js

@@ -20,6 +20,5 @@ const { expect } = require('chai');
     bot.execute('wall.post', {}).catch((err) => {
-      expect(err).to.be.an('object');
-      expect(err.method).to.be.a('string');
-      expect(err.error_code).to.be.a('number');
-      expect(err.error_msg).to.be.a('string');
+      expect(err).to.be.a('error');
+      expect(err.message).to.be.equal('ApiError');
+      expect(err.response).to.be.an('object');
 
@@ -26,0 +25,0 @@ done();

New alerts

Environment variable access

Supply chain risk

Package accesses environment variables, which may be a sign of credential stuffing or data theft.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

44202

increased by0.04%

Lines of code

1125

increased by0.18%

Worsened metrics

Number of low supply chain risk alerts

14

increased by7.69%

No dependency changes