Tim Lai

@tekeroth-snapcode Thanks for the idea and detailed description. The UX challenge I see is that "what" should be refreshed depends on the user and how they are using the tool...

@luckydonald Yes, please create a PR for your proposed solution. Thanks!

@frantuma @char0n Here's another set of reproducible steps, to get the same error, e.g. not specific to the "hovering over references" 1. click on a position within the editor 2....

@char0n @frantuma Is this still an active issue? We do already try to parse with AsyncAPI's parser first here: https://github.com/swagger-api/swagger-editor/blob/b524bc055e8baad80069d5edcc716e9fb38a594a/src/plugins/editor-preview-asyncapi/components/AsyncAPIEditorPreviewPane.jsx#L83

> This requires switching jenkins Node.js runtime to 16.x.y first. Is `jsdom` docs wrong? Docs and `package.json` states it's still Node.js v14. > The latest versions of jsdom require Node.js...

Suggestion: wrap SwaggerUI's `InfoUrl` and possibly also `InfoBasePath` component, to return null. Could also provide an override option for the wrapped component(s) whether or not to return null, which lets...

> are you saying that the AsyncAPI definition isn't embedded? I think it would make a good plugin (one or more) for those definitions. So it's not embedded in core...

Although not enabled, this is now possible using the `REACT_APP_DEFINITION_FILE` environment flag. There exists a sample setting in `.env` as well as a fixture `/public/static/petstore-oas3.yml`.

These swagger-ui most likely related, with file reference issues: https://github.com/swagger-api/swagger-ui/issues/4678 https://github.com/swagger-api/swagger-ui/issues/4739 (also a separate circular reference on itself) https://github.com/swagger-api/swagger-ui/issues/5820 https://github.com/swagger-api/swagger-ui/issues/5705 Note that the $ref error 'Cannot read property '0' of...

@char0n I updated this PR that fixes merge conflicts, test and lint errors on a new branch [mathis-m-ft/external_value](https://github.com/swagger-api/swagger-js/tree/mathis-m-ft/external_value). (mostly b/c I couldn't push to this PR directly). It looks like...