feat: set default max concurrency to 100 for web-api client

docs/_packages/web_api.md

@@ -642,8 +642,8 @@ Each of the API method calls the client starts are happening **concurrently**, o
 to perform a lot of method calls, let's say 100 of them, at the same time, each one of them would be competing for the
 same network resources (such as bandwidth). By competing, they might negatively affect the performance of all the rest,
 and therefore negatively affect the performance of your app. This is one of the reasons why the `WebClient` limits the
-**concurrency** of requests by default to ten, which means it keeps track of how many requests are waiting, and only
-starts an eleventh request when one of them completes. The exact number of requests the client allows at the same time
+**concurrency** of requests by default to one hundred, which means it keeps track of how many requests are waiting, and only
+starts an additional request when one of them completes. The exact number of requests the client allows at the same time
 can be set using the `maxRequestConcurrency` option.
 
 ```javascript

packages/web-api/src/WebClient.spec.js

@@ -1040,25 +1040,23 @@ describe('WebClient', function () {
         });
     });
 
-    it('should have a default conncurrency of 3', function () {
+    it('should have a default conncurrency of 100', function () {
       const client = new WebClient(token);
-      const requests = [
-        client.apiCall('1'),
-        client.apiCall('2'),
-        client.apiCall('3'),
-        client.apiCall('4'),
-      ];
+      const requests = [];
+      for (let i = 0; i < 101; i++) {
+        requests.push(client.apiCall(`${i}`));
+      }
       return Promise.all(requests)
         .then((responses) => {
           // verify all responses are present
-          assert.lengthOf(responses, 4);
+          assert.lengthOf(responses, 101);
 
           // verify that maxRequestConcurrency requests were all sent concurrently
-          const concurrentResponses = responses.slice(0, 3); // the first 3 responses
+          const concurrentResponses = responses.slice(0, 100); // the first 100 responses
           concurrentResponses.forEach((r) => assert.isBelow(r.diff, responseDelay));
 
           // verify that any requests after maxRequestConcurrency were delayed by the responseDelay
-          const queuedResponses = responses.slice(3);
+          const queuedResponses = responses.slice(100);
           const minDiff = concurrentResponses[concurrentResponses.length - 1].diff + responseDelay;
           queuedResponses.forEach((r) => assert.isAtLeast(r.diff, minDiff));
         });

packages/web-api/src/WebClient.ts

@@ -163,7 +163,7 @@ export class WebClient extends Methods {
     slackApiUrl = 'https://slack.com/api/',
     logger = undefined,
     logLevel = undefined,
-    maxRequestConcurrency = 3,
+    maxRequestConcurrency = 100,
     retryConfig = tenRetriesInAboutThirtyMinutes,
     agent = undefined,
     tls = undefined,