Skip to content

Express types should be included in dependencies #2409

@Grmiade

Description

@Grmiade

In the last 4.2.0 version, we encounter some types issues during build:

node_modules/.pnpm/@[email protected]/node_modules/@slack/bolt/dist/receivers/ExpressReceiver.d.ts:9:98 - error TS7016: Could not find a declaration file for module 'express'. '/usr/app/node_modules/.pnpm/[email protected]/node_modules/express/index.js' implicitly has an 'any' type.
node_modules/.pnpm/@[email protected]/node_modules/@slack/bolt/dist/receivers/ParamsIncomingMessage.d.ts:3:39 - error TS2307: Cannot find module 'express-serve-static-core' or its corresponding type declarations.

Indeed, the @types/express package has been moved to devDependencies recently, but these types should have been stayed in dependencies since the generated d.ts files need them.

Metadata

Metadata

Assignees

No one assigned

    Labels

    discussionM-T: An issue where more input is needed to reach a decisionneeds infoAn issue that is claimed to be a bug and hasn't been reproduced, or otherwise needs more info

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions