Version 6 doesn't correctly generate swagger-ui-init.js

[Panzer1119]

Is there an existing issue for this?

• I have searched the existing issues

Current behavior

The generated swagger-ui-init.js does not have functions correctly replaced in the swagger options.

For example the previous version generated something like this:

// [...]
  var options = {
  "swaggerDoc": {
    "openapi": "3.0.0",
    "paths": {
      // [...]
    },
    "info": {
      // [...]},
    "tags": [],
    "servers": [],
    "components": {
      "schemas": {}
    }
  },
  "customOptions": {
    "operationsSorter": (a, b) => a.get("id").localeCompare(b.get("id")) // Type of function is kept
  },
  "swaggerUrl": {}
};
// [...]

But the new version generates something like this:

// [...]
  let options = {
  "swaggerDoc": {
    "openapi": "3.0.0",
    "paths": {
      // [...]
    },
    "info": {
      // [...]
    },
    "tags": [],
    "servers": [],
    "components": {
      "schemas": {}
    }
  },
  "customOptions": {
    "operationsSorter": "(a, b) => a.get("id").localeCompare(b.get("id"))" // Almost the same, but has " before and after the function, which cause an Error
  }
};
// [...]

This leads to an empty page when trying to access the generated Swagger API Documention.

Minimum reproduction code

https://github.com/Panzer1119/nestjs-swagger-build-jsinit-options

Steps to reproduce

1. npm i
2. npm start
3. Open the Swagger API Documentation in Browser (URL is printed to console)
4. Inspect the Sources and look at api/swagger-ui-init.js

Expected behavior

Since Version 6 this module provides it's own implementations for the swagger HTML generation etc.

But the newly created lib/swagger-ui/helpers.ts, which basically contains some code from swagger-ui-express index.js, fails to correctly replace the function placeholder, because it misses the " before and after the placeholder in the RegExp.

This can be fixed by adding them back and then the built JSInitOptions will have functions as functions and not strings set.

Original Code (Line 21):

json = json.replace(new RegExp(functionPlaceholder, 'g'), () => fns.shift());

Fixed Code:

json = json.replace(new RegExp('"' + functionPlaceholder + '"', 'g'), () => fns.shift());

Package version

9.0.1

NestJS version

9.0.1

Node.js version

16.13.2

In which operating systems have you tested?

• macOS
• Windows
• Linux

Other

No response