sqs-consumer 5.4.0 → 5.7.0

package/.eslintignore

@@ -0,0 +1,4 @@
+node_modules
+coverage
+bake-scripts
+dist

package/.github/CODEOWNERS

@@ -0,0 +1,2 @@
+# All changes should be reviewed by iBL
+* @bbc/ibl

package/.github/CONTRIBUTING.md

@@ -0,0 +1,39 @@
+# Contributing
+
+Thank you for your interest in contributing to the sqs-consumer. 
+
+* If you're unsure if a feature would make a good addition, you can always [create an issue](https://github.com/bbc/sqs-consumer/issues/new) first. Raising an issue before creating a pull request is recommended. 
+* We aim for 100% test coverage. Please write tests for any new functionality or changes.
+* Any API changes should be fully documented.
+* Make sure your code meets our linting standards. Run `npm run lint` to check your code.
+* Maintain the existing coding style. There are some settings in `.jsbeautifyrc` to help.
+* Be mindful of others when making suggestions and/or code reviewing.
+
+
+
+## Reporting Issues
+Before opening a new issue, first check that there is not already an [open issue or Pull Request](https://github.com/bbc/sqs-consumer/issues?utf8=%E2%9C%93&q=is%3Aopen) that addresses it.
+
+If there is, make relevant comments and add your reaction. Use a reaction in place of a "+1" comment:
+* 👍 - upvote
+* 👎 - downvote
+
+If you cannot find an existing issue that describes your bug or feature, create a new issue using the guidelines below.
+
+1. Pick an appropriate template for the type of issue [from here](https://github.com/bbc/sqs-consumer/issues/choose)
+2. Provide as much detail as possible
+3. Follow your issue in the issue tracking workflow
+
+## Contributing Code
+If you do not have push access to the repository, please [fork it](https://help.github.com/en/articles/fork-a-repo). You should then work on your own `master` branch.
+
+Otherwise, you may clone this repository and create a working branch with a _kebab-case_ name reflecting what you are working on (e.g. `fix-the-thing`).
+
+Follow the setup instructions in the [README](../README.md).
+
+Ensure all your code is thoroughly tested and that this testing is detailed in the pull request.
+
+## Pull Request Process
+1. Make sure you have opened an issue and it was approved by a project maintainer before working on a PR
+2. Read and complete all relevant sections of the PR template
+3. Wait for the PR get approved

package/.github/ISSUE_TEMPLATE/bug-report.md

@@ -0,0 +1,27 @@
+---
+name: Bug report
+about: Create a report to help us improve
+title: ''
+labels: bug
+assignees: ''
+
+---
+
+**Describe the bug**
+A clear and concise description of what the bug is.
+
+**To Reproduce**
+Steps to reproduce the behaviour:
+1. Go to '...'
+2. Select '....'
+3. Scroll down to '....'
+4. See error
+
+**Expected behaviour**
+A clear and concise description of what you expected to happen.
+
+**screenshots**
+If applicable, add screenshots to help explain your problem.
+
+**Additional context**
+Add any other context about the problem here, such as specific device information.

package/.github/ISSUE_TEMPLATE/feature-request.md

@@ -0,0 +1,20 @@
+---
+name: Feature request
+about: Suggest an idea for this project
+title: ''
+labels: feature-request
+assignees: ''
+
+---
+
+**The problem**
+A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
+
+**Suggested solution**
+A clear and concise description of what you want to happen.
+
+**Alternatives considered**
+A clear and concise description of any alternative solutions or features you've considered.
+
+**Additional context**
+Add any other context or screenshots about the feature request here.

package/.github/ISSUE_TEMPLATE/technical-question.md

@@ -0,0 +1,17 @@
+---
+name: Technical question
+about: Want to ask a technical question about the project
+title: ''
+labels: question
+assignees: ''
+
+---
+
+**Question**
+A clear technical question could be asked here.
+
+**screenshots**
+If applicable, add screenshots to help explain your question.
+
+**Additional context**
+Add any other context about the question here, such as specific device information, technology choice etc.

package/.github/pull_request_template.md

@@ -0,0 +1,20 @@
+<!--- Provide a general summary of your changes in the Title above -->
+## Description
+<!--- Describe your changes in detail -->
+## Motivation and Context
+<!--- Why is this change required? What problem does it solve? -->
+<!--- If it fixes an open issue, please link to the issue here. -->
+## Types of changes
+<!--- What types of changes does your code introduce? Put an `x` in all the boxes that apply: -->
+- [ ] Bug fix (non-breaking change which fixes an issue)
+- [ ] New feature (non-breaking change which adds functionality)
+- [ ] Breaking change (fix or feature that would cause existing functionality to change)
+## Checklist:
+<!--- Go over all the following points, and put an `x` in all the boxes that apply. -->
+<!--- If you're unsure about any of these, don't hesitate to ask. We're here to help! -->
+- [ ] My code follows the code style of this project.
+- [ ] My change requires a change to the documentation.
+- [ ] I have updated the documentation accordingly.
+- [ ] I have read the **CONTRIBUTING** document.
+- [ ] I have added tests to cover my changes.
+- [ ] All new and existing tests passed.

package/.travis.yml

@@ -1,7 +1,7 @@
 language: node_js
 node_js:
   - "10"
-  - "8"
+  - "12"
 addons:
   code_climate:
     repo_token: 2d851f8f3a9348ac4f43262305037f80a730c2660fda50af8ae4d445fd89333b

package/README.md

@@ -40,6 +40,36 @@ app.start();
 * Messages are deleted from the queue once the handler function has completed successfully.
 * Throwing an error (or returning a rejected promise) from the handler function will cause the message to be left on the queue. An [SQS redrive policy](http://docs.aws.amazon.com/AWSSimpleQueueService/latest/SQSDeveloperGuide/SQSDeadLetterQueue.html) can be used to move messages that cannot be processed to a dead letter queue.
 * By default messages are processed one at a time – a new message won't be received until the first one has been processed. To process messages in parallel, use the `batchSize` option [detailed below](#options).
+* By default, the default Node.js HTTP/HTTPS SQS agent creates a new TCP connection for every new request ([AWS SQS documentation](https://docs.aws.amazon.com/sdk-for-javascript/v2/developer-guide/node-reusing-connections.html)). To avoid the cost of establishing a new connection, you can reuse an existing connection by passing a new SQS instance with `keepAlive: true`.
+```js
+const { Consumer } = require('sqs-consumer');
+const AWS = require('aws-sdk');
+const https = require('https');
+
+const app = Consumer.create({
+  queueUrl: 'https://sqs.eu-west-1.amazonaws.com/account-id/queue-name',
+  handleMessage: async (message) => {
+    // do some work with `message`
+  },
+  sqs: new AWS.SQS({
+    httpOptions: {
+      agent: new https.Agent({
+        keepAlive: true
+      })
+    }
+  })
+});
+
+app.on('error', (err) => {
+  console.error(err.message);
+});
+
+app.on('processing_error', (err) => {
+  console.error(err.message);
+});
+
+app.start();
+```
 
 ### Credentials
 
@@ -98,17 +128,18 @@ Creates a new SQS consumer.
 * `region` - _String_ - The AWS region (default `eu-west-1`)
 * `handleMessage` - _Function_ - An `async` function (or function that returns a `Promise`) to be called whenever a message is received. Receives an SQS message object as it's first argument.
 * `handleMessageBatch` - _Function_ - An `async` function (or function that returns a `Promise`) to be called whenever a batch of messages is received. Similar to `handleMessage` but will receive the list of messages, not each message individually. **If both are set, `handleMessageBatch` overrides `handleMessage`**.
-* `handleMessageTimeout` - _String_ - Time in ms to wait for `handleMessage` to process a message before timing out. Emits `timeout_error` on timeout. By default, if `handleMessage` times out, the unprocessed message returns to the end of the queue.
+* `handleMessageTimeout` - _Number_ - Time in ms to wait for `handleMessage` to process a message before timing out. Emits `timeout_error` on timeout. By default, if `handleMessage` times out, the unprocessed message returns to the end of the queue.
 * `attributeNames` - _Array_ - List of queue attributes to retrieve (i.e. `['All', 'ApproximateFirstReceiveTimestamp', 'ApproximateReceiveCount']`).
 * `messageAttributeNames` - _Array_ - List of message attributes to retrieve (i.e. `['name', 'address']`).
 * `batchSize` - _Number_ - The number of messages to request from SQS when polling (default `1`). This cannot be higher than the AWS limit of 10.
 * `visibilityTimeout` - _Number_ - The duration (in seconds) that the received messages are hidden from subsequent retrieve requests after being retrieved by a ReceiveMessage request.
+* `heartbeatInterval` - _Number_ - The interval (in seconds) between requests to extend the message visibility timeout. On each heartbeat the visibility is extended by adding `visibilityTimeout` to the number of seconds since the start of the handler function. This value must less than `visibilityTimeout`.
 * `terminateVisibilityTimeout` - _Boolean_ - If true, sets the message visibility timeout to 0 after a `processing_error` (defaults to `false`).
 * `waitTimeSeconds` - _Number_ - The duration (in seconds) for which the call will wait for a message to arrive in the queue before returning.
 * `authenticationErrorTimeout` - _Number_ - The duration (in milliseconds) to wait before retrying after an authentication error (defaults to `10000`).
 * `pollingWaitTimeMs` - _Number_ - The duration (in milliseconds) to wait before repolling the queue (defaults to `0`).
 * `sqs` - _Object_ - An optional [AWS SQS](http://docs.aws.amazon.com/AWSJavaScriptSDK/latest/AWS/SQS.html) object to use if you need to configure the client manually
-
+* `shouldDeleteMessages` - _Boolean_ - Default to `true`, if you don't want the package to delete messages from sqs set this to `false`
 ### `consumer.start()`
 
 Start polling the queue for messages.
@@ -127,7 +158,7 @@ Each consumer is an [`EventEmitter`](http://nodejs.org/api/events.html) and emit
 
 |Event|Params|Description|
 |-----|------|-----------|
-|`error`|`err`, `[message]`|Fired when an error occurs interacting with the queue. If the error correlates to a message, that error is included in Params|
+|`error`|`err`, `[message]`|Fired when an error occurs interacting with the queue. If the error correlates to a message, that message is included in Params|
 |`processing_error`|`err`, `message`|Fired when an error occurs processing the message.|
 |`timeout_error`|`err`, `message`|Fired when `handleMessageTimeout` is supplied as an option and if `handleMessage` times out.|
 |`message_received`|`message`|Fired when a message is received.|
@@ -138,8 +169,7 @@ Each consumer is an [`EventEmitter`](http://nodejs.org/api/events.html) and emit
 
 ### AWS IAM Permissions
 
-Consumer will receive and delete messages from the SQS queue. Ensure `sqs:ReceiveMessage` and `sqs:DeleteMessage` access is granted on the queue being consumed.
-
+Consumer will receive and delete messages from the SQS queue. Ensure `sqs:ReceiveMessage`, `sqs:DeleteMessage`, `sqs:DeleteMessageBatch`, `sqs:ChangeMessageVisibility` and `sqs:ChangeMessageVisibilityBatch` access is granted on the queue being consumed.
 
 ### Contributing 
-See contributing [guildlines](https://github.com/bbc/sqs-consumer/blob/master/CONTRIBUTING.md)
+See contributing [guidelines](https://github.com/bbc/sqs-consumer/blob/master/.github/CONTRIBUTING.md).

package/dist/bind.js

@@ -1,5 +1,6 @@
-'use strict';
+"use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.autoBind = void 0;
 function isMethod(propertyName, value) {
     return propertyName !== 'constructor' && typeof value === 'function';
 }

package/dist/consumer.d.ts

@@ -1,6 +1,7 @@
+/// <reference types="node" />
 import * as SQS from 'aws-sdk/clients/sqs';
 import { EventEmitter } from 'events';
-declare type SQSMessage = SQS.Types.Message;
+export declare type SQSMessage = SQS.Types.Message;
 export interface ConsumerOptions {
     queueUrl?: string;
     attributeNames?: string[];
@@ -12,12 +13,24 @@ export interface ConsumerOptions {
     authenticationErrorTimeout?: number;
     pollingWaitTimeMs?: number;
     terminateVisibilityTimeout?: boolean;
+    heartbeatInterval?: number;
     sqs?: SQS;
     region?: string;
     handleMessageTimeout?: number;
+    shouldDeleteMessages?: boolean;
     handleMessage?(message: SQSMessage): Promise<void>;
     handleMessageBatch?(messages: SQSMessage[]): Promise<void>;
 }
+interface Events {
+    'response_processed': [];
+    'empty': [];
+    'message_received': [SQSMessage];
+    'message_processed': [SQSMessage];
+    'error': [Error, void | SQSMessage | SQSMessage[]];
+    'timeout_error': [Error, SQSMessage];
+    'processing_error': [Error, SQSMessage];
+    'stopped': [];
+}
 export declare class Consumer extends EventEmitter {
     private queueUrl;
     private handleMessage;
@@ -32,9 +45,14 @@ export declare class Consumer extends EventEmitter {
     private authenticationErrorTimeout;
     private pollingWaitTimeMs;
     private terminateVisibilityTimeout;
+    private heartbeatInterval;
     private sqs;
+    private shouldDeleteMessages;
     constructor(options: ConsumerOptions);
-    readonly isRunning: boolean;
+    emit<T extends keyof Events>(event: T, ...args: Events[T]): boolean;
+    on<T extends keyof Events>(event: T, listener: (...args: Events[T]) => void): this;
+    once<T extends keyof Events>(event: T, listener: (...args: Events[T]) => void): this;
+    get isRunning(): boolean;
     static create(options: ConsumerOptions): Consumer;
     start(): void;
     stop(): void;
@@ -43,12 +61,13 @@ export declare class Consumer extends EventEmitter {
     private receiveMessage;
     private deleteMessage;
     private executeHandler;
-    private terminateVisabilityTimeout;
+    private changeVisibilityTimeout;
     private emitError;
     private poll;
     private processMessageBatch;
     private deleteMessageBatch;
     private executeBatchHandler;
-    private terminateVisabilityTimeoutBatch;
+    private changeVisabilityTimeoutBatch;
+    private startHeartbeat;
 }
 export {};

package/dist/consumer.js

@@ -1,5 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.Consumer = void 0;
 const SQS = require("aws-sdk/clients/sqs");
 const Debug = require("debug");
 const events_1 = require("events");
@@ -30,6 +31,9 @@ function assertOptions(options) {
     if (options.batchSize > 10 || options.batchSize < 1) {
         throw new Error('SQS batchSize option must be between 1 and 10.');
     }
+    if (options.heartbeatInterval && !(options.heartbeatInterval < options.visibilityTimeout)) {
+        throw new Error('heartbeatInterval must be less than visibilityTimeout.');
+    }
 }
 function isConnectionError(err) {
     if (err instanceof errors_1.SQSError) {
@@ -52,6 +56,7 @@ function hasMessages(response) {
 }
 class Consumer extends events_1.EventEmitter {
     constructor(options) {
+        var _a, _b, _c, _d;
         super();
         assertOptions(options);
         this.queueUrl = options.queueUrl;
@@ -64,14 +69,25 @@ class Consumer extends events_1.EventEmitter {
         this.batchSize = options.batchSize || 1;
         this.visibilityTimeout = options.visibilityTimeout;
         this.terminateVisibilityTimeout = options.terminateVisibilityTimeout || false;
-        this.waitTimeSeconds = options.waitTimeSeconds || 20;
-        this.authenticationErrorTimeout = options.authenticationErrorTimeout || 10000;
-        this.pollingWaitTimeMs = options.pollingWaitTimeMs || 0;
+        this.heartbeatInterval = options.heartbeatInterval;
+        this.waitTimeSeconds = (_a = options.waitTimeSeconds) !== null && _a !== void 0 ? _a : 20;
+        this.authenticationErrorTimeout = (_b = options.authenticationErrorTimeout) !== null && _b !== void 0 ? _b : 10000;
+        this.pollingWaitTimeMs = (_c = options.pollingWaitTimeMs) !== null && _c !== void 0 ? _c : 0;
+        this.shouldDeleteMessages = (_d = options.shouldDeleteMessages) !== null && _d !== void 0 ? _d : true;
         this.sqs = options.sqs || new SQS({
             region: options.region || process.env.AWS_REGION || 'eu-west-1'
         });
         bind_1.autoBind(this);
     }
+    emit(event, ...args) {
+        return super.emit(event, ...args);
+    }
+    on(event, listener) {
+        return super.on(event, listener);
+    }
+    once(event, listener) {
+        return super.once(event, listener);
+    }
     get isRunning() {
         return !this.stopped;
     }
@@ -110,7 +126,13 @@ class Consumer extends events_1.EventEmitter {
     }
     async processMessage(message) {
         this.emit('message_received', message);
+        let heartbeat;
         try {
+            if (this.heartbeatInterval) {
+                heartbeat = this.startHeartbeat(async () => {
+                    return this.changeVisibilityTimeout(message, this.visibilityTimeout);
+                });
+            }
             await this.executeHandler(message);
             await this.deleteMessage(message);
             this.emit('message_processed', message);
@@ -118,14 +140,12 @@ class Consumer extends events_1.EventEmitter {
         catch (err) {
             this.emitError(err, message);
             if (this.terminateVisibilityTimeout) {
-                try {
-                    await this.terminateVisabilityTimeout(message);
-                }
-                catch (err) {
-                    this.emit('error', err, message);
-                }
+                await this.changeVisibilityTimeout(message, 0);
             }
         }
+        finally {
+            clearInterval(heartbeat);
+        }
     }
     async receiveMessage(params) {
         try {
@@ -138,6 +158,10 @@ class Consumer extends events_1.EventEmitter {
         }
     }
     async deleteMessage(message) {
+        if (!this.shouldDeleteMessages) {
+            debug('Skipping message delete since shouldDeleteMessages is set to false');
+            return;
+        }
         debug('Deleting message %s', message.MessageId);
         const deleteParams = {
             QueueUrl: this.queueUrl,
@@ -171,7 +195,7 @@ class Consumer extends events_1.EventEmitter {
             if (err instanceof errors_1.TimeoutError) {
                 err.message = `Message handler timed out after ${this.handleMessageTimeout}ms: Operation timed out.`;
             }
-            else {
+            else if (err instanceof Error) {
                 err.message = `Unexpected message handler failure: ${err.message}`;
             }
             throw err;
@@ -180,14 +204,19 @@ class Consumer extends events_1.EventEmitter {
             clearTimeout(timeout);
         }
     }
-    async terminateVisabilityTimeout(message) {
-        return this.sqs
-            .changeMessageVisibility({
-            QueueUrl: this.queueUrl,
-            ReceiptHandle: message.ReceiptHandle,
-            VisibilityTimeout: 0
-        })
-            .promise();
+    async changeVisibilityTimeout(message, timeout) {
+        try {
+            return await this.sqs
+                .changeMessageVisibility({
+                QueueUrl: this.queueUrl,
+                ReceiptHandle: message.ReceiptHandle,
+                VisibilityTimeout: timeout
+            })
+                .promise();
+        }
+        catch (err) {
+            this.emit('error', toSQSError(err, `Error changing visibility timeout: ${err.message}`), message);
+        }
     }
     emitError(err, message) {
         if (err.name === errors_1.SQSError.name) {
@@ -234,7 +263,13 @@ class Consumer extends events_1.EventEmitter {
         messages.forEach((message) => {
             this.emit('message_received', message);
         });
+        let heartbeat;
         try {
+            if (this.heartbeatInterval) {
+                heartbeat = this.startHeartbeat(async () => {
+                    return this.changeVisabilityTimeoutBatch(messages, this.visibilityTimeout);
+                });
+            }
             await this.executeBatchHandler(messages);
             await this.deleteMessageBatch(messages);
             messages.forEach((message) => {
@@ -244,20 +279,22 @@ class Consumer extends events_1.EventEmitter {
         catch (err) {
             this.emit('error', err, messages);
             if (this.terminateVisibilityTimeout) {
-                try {
-                    await this.terminateVisabilityTimeoutBatch(messages);
-                }
-                catch (err) {
-                    this.emit('error', err, messages);
-                }
+                await this.changeVisabilityTimeoutBatch(messages, 0);
             }
         }
+        finally {
+            clearInterval(heartbeat);
+        }
     }
     async deleteMessageBatch(messages) {
+        if (!this.shouldDeleteMessages) {
+            debug('Skipping message delete since shouldDeleteMessages is set to false');
+            return;
+        }
         debug('Deleting messages %s', messages.map((msg) => msg.MessageId).join(' ,'));
         const deleteParams = {
             QueueUrl: this.queueUrl,
-            Entries: messages.map(message => ({
+            Entries: messages.map((message) => ({
                 Id: message.MessageId,
                 ReceiptHandle: message.ReceiptHandle
             }))
@@ -280,18 +317,28 @@ class Consumer extends events_1.EventEmitter {
             throw err;
         }
     }
-    async terminateVisabilityTimeoutBatch(messages) {
+    async changeVisabilityTimeoutBatch(messages, timeout) {
         const params = {
             QueueUrl: this.queueUrl,
             Entries: messages.map((message) => ({
                 Id: message.MessageId,
                 ReceiptHandle: message.ReceiptHandle,
-                VisibilityTimeout: 0
+                VisibilityTimeout: timeout
             }))
         };
-        return this.sqs
-            .changeMessageVisibilityBatch(params)
-            .promise();
+        try {
+            return await this.sqs
+                .changeMessageVisibilityBatch(params)
+                .promise();
+        }
+        catch (err) {
+            this.emit('error', toSQSError(err, `Error changing visibility timeout: ${err.message}`), messages);
+        }
+    }
+    startHeartbeat(heartbeatFn) {
+        return setInterval(() => {
+            heartbeatFn();
+        }, this.heartbeatInterval * 1000);
     }
 }
 exports.Consumer = Consumer;

package/dist/errors.js

@@ -1,5 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.TimeoutError = exports.SQSError = void 0;
 class SQSError extends Error {
     constructor(message) {
         super(message);

package/dist/index.d.ts

@@ -1 +1 @@
-export { Consumer, ConsumerOptions } from './consumer';
+export { SQSMessage, Consumer, ConsumerOptions } from './consumer';

package/dist/index.js

@@ -1,4 +1,4 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 var consumer_1 = require("./consumer");
-exports.Consumer = consumer_1.Consumer;
+Object.defineProperty(exports, "Consumer", { enumerable: true, get: function () { return consumer_1.Consumer; } });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sqs-consumer",
-  "version": "5.4.0",
+  "version": "5.7.0",
   "description": "Build SQS-based Node applications without the boilerplate",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -10,8 +10,9 @@
     "clean": "rm -fr dist/*",
     "prepublish": "npm run build",
     "pretest": "npm run build",
-    "test": "mocha",
-    "lint": "tslint --project tsconfig.json",
+    "test": "mocha --recursive --full-trace --exit",
+    "lint": "eslint . --ext .ts",
+    "lint:fix": "eslint . --fix",
     "coverage": "nyc mocha && nyc report --reporter=html && nyc report --reporter=json-summary",
     "lcov": "nyc mocha && nyc report --reporter=lcov",
     "posttest": "npm run lint"
@@ -31,27 +32,31 @@
   ],
   "license": "Apache-2.0",
   "devDependencies": {
-    "@types/chai": "^4.1.4",
-    "@types/debug": "^4.1.3",
-    "@types/mocha": "^2.2.43",
-    "@types/node": "^10.12.18",
-    "@types/sinon": "^4.0.0",
+    "@types/chai": "^4.2.11",
+    "@types/debug": "^4.1.5",
+    "@types/mocha": "^7.0.2",
+    "@types/node": "^14.0.13",
+    "@types/sinon": "^9.0.4",
     "@types/typescript": "^2.0.0",
     "chai": "^4.2.0",
     "codeclimate-test-reporter": "^0.5.1",
-    "mocha": "^5.2.0",
-    "nyc": "^14.1.1",
-    "p-event": "^2.1.0",
-    "sinon": "^7.2.2",
-    "ts-node": "^3.3.0",
-    "tslint": "^5.17.0",
-    "tslint-config-airbnb": "^5.3.1",
-    "tslint-microsoft-contrib": "^5.0.3",
-    "typescript": "^2.6.1"
+    "eslint": "^7.2.0",
+    "eslint-config-iplayer-ts": "^2.0.0",
+    "mocha": "^8.0.1",
+    "nyc": "^15.1.0",
+    "p-event": "^4.2.0",
+    "sinon": "^9.0.2",
+    "ts-node": "^8.10.2",
+    "tslint-config-airbnb": "^5.11.2",
+    "tslint-microsoft-contrib": "^6.2.0",
+    "typescript": "^3.9.5",
+    "aws-sdk": "^2.1114.0"
   },
   "dependencies": {
-    "aws-sdk": "^2.443.0",
-    "debug": "^4.1.1"
+    "debug": "^4.3.1"
+  },
+  "peerDependencies": {
+    "aws-sdk": "^2.1114.0"
   },
   "nyc": {
     "include": [
@@ -65,5 +70,16 @@
     ],
     "sourceMap": true,
     "instrument": true
+  },
+  "eslintConfig": {
+    "extends": "iplayer-ts",
+    "parserOptions": {
+      "ecmaVersion": 2017,
+      "sourceType": "module"
+    }
+  },
+  "mocha": {
+    "spec": "test/**/**/*.ts",
+    "require": "ts-node/register"
   }
 }

package/src/bind.ts

@@ -1,5 +1,3 @@
-'use strict';
-
 function isMethod(propertyName: string, value: any): boolean {
   return propertyName !== 'constructor' && typeof value === 'function';
 }