Make `express` an optional peer dependency
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 2.9k
- Forks
- 445
- Avg merge
- 1d 3h
- Merged PRs (30d)
- 28
Description
express is a very heavy library with 67 dependencies: https://npmgraph.js.org/?q=express
It doesn't seem to be used in a required functionality here, but in optional functionality:
https://github.com/slackapi/bolt-js/blob/7c0ed4fae4d82e6c45b9b8afd50d931a19b7140f/src/receivers/ExpressReceiver.ts#L26
Users who are using express will have it installed. Those who are using a different web server should not have express installed
Requirements
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Read src/receivers/ExpressReceiver.ts to confirm how Express is used, then inspect the package metadata and dependency setup. Done means Express remains available for users of ExpressReceiver while installations that use another web server do not install it as a required dependency.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- express, node.js, typescript
- Domain
- backend
- Issue type
- Feature
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 68/100