swagger: @nestjs/swagger and bson lib collision problem

I’m submitting a Bug report


[ ] Regression 
[x] Bug report
[ ] Feature request
[ ] Documentation issue or request
[ ] Support request => Please do not submit support request here, instead post your question on Stack Overflow.

import { PrimaryKey, Property, SerializedPrimaryKey } from '@mikro-orm/core';

import { ObjectId } from 'bson';

export abstract class CommonEntity {
  @PrimaryKey()
  _id: ObjectId;

  @SerializedPrimaryKey()
  id!: string; // string variant of PK, will be handled automatically

  @Property()
  createdAt = new Date();

  @Property({ onUpdate: () => new Date() })
  updatedAt = new Date();
}

{
  "collection": "@nestjs/schematics",
  "sourceRoot": "src",
  "projects": {},
  "compilerOptions": {
    "webpack": false,
    "plugins": [
      {
        "name": "@nestjs/swagger",
        "options": {
          "classValidatorShim": true,
          "introspectComments": true
        }
      }
    ]
  }
}

Current behavior

Error: Cannot find module ‘bson/bson’ Require stack:

/Volumes/HD/workspace/_temp/hello-nest/dist/src/entities/common.entity.js
/Volumes/HD/workspace/_temp/hello-nest/dist/src/modules/users/entities/user.entity.js
/Volumes/HD/workspace/_temp/hello-nest/dist/src/modules/users/users.service.js
/Volumes/HD/workspace/_temp/hello-nest/dist/src/modules/users/users.module.js
/Volumes/HD/workspace/_temp/hello-nest/dist/src/app.module.js
/Volumes/HD/workspace/_temp/hello-nest/dist/src/main.js at Function.Module._resolveFilename (node:internal/modules/cjs/loader:927:15)

Expected behavior

Expected import bson，dont bson/bson

Minimal reproduction of the problem with instructions

https://github.com/WumaCoder/hello-nest/blob/a27ab9a6eb82d99890a6a2ce56ca2e71057579b9/src/entities/common.entity.ts

What is the motivation / use case for changing the behavior?

npm i npm run start

Environment


Nest version: 8.0

 
For Tooling issues:
- Node version: node 16  
- Platform:  mac 15

Others:

About this issue

Original URL
State: open
Created 3 years ago
Reactions: 7
Comments: 20 (3 by maintainers)

Most upvoted comments

This appears to be an issue with the CLI Plugin. I was able to solve this issue by removing the @nestjs/swagger plugin declaration from the nest-cli.json file. We have declared decorators manually so the plugin was not needed. As soon as it was removed, this problem went away.

shwaddell28 on Dec 21, 2022

Also i noted

    @ApiHideProperty()
    @PrimaryKey()
    _id: ObjectId;

Doing this does not produce any error . However on other cases swagger plugin tries to introspect the ObjectId type and thats when bson/bson lib collision occurs

rubiin on Aug 8, 2022

One workaround is to install ‘bson’ version 1.1.5 directly and use:

import {ObjectId} from 'bson'; instead of import {ObjectId} from 'mongodb'}

Another workaround is to not have prefix in the entity filename.

So instead of common.entity.js just use entity.js not ideal I know but it works

KolevDarko on Jul 19, 2023

Did you run mongodb instance locally? Without that the orm wont bootup You can see here the logs https://imgur.com/a/K7VbNCL @micalevisk

rubiin on Nov 7, 2022

I managed to fix this issue by adding @agravelot’s custom script in package.json and forcing it to run after nest build script like so.

{
    "scripts": {
        "build": "nest build",
        "postbuild": "npm run fix:bson",
        "start": "nest start --exec 'npm run fix:bson; node'",
        "fix:bson": "grep -rl bson/bson . | grep dist | xargs sed -i -e 's/bson\\/bson/bson/g'"
    },
}

mrshaq on Sep 1, 2022

@micalevisk here is a reproduction https://github.com/rubiin/nest-swagger-bug . Basically problem from file common.entity.ts

rubiin on Aug 8, 2022

Yeah. I will try to provide a minimal reproduction asap

rubiin on Aug 6, 2022

@rubiin can you share a minimum reproduction of that?

why reproductions are required

micalevisk on Aug 6, 2022

I am experiencing the same issue, when using abstract or just extending another class.

niklaszantner on Jan 1, 2022