AsyncAPI Studio automated updates broken

[derberg]

Looks to me like https://github.com/asyncapi/studio/blob/master/apps/studio/package.json#L44 is running on very old react component version which is not good.

Its confusing people for very long, look: https://studio.asyncapi.com/?share=3469f1ef-eae6-4e23-a53c-f4d2877fd857 it still runs on version there reverts meaning of action verbs :(

So one thing is to update the component in studio, another is to automate it.

Current automation do not work: https://github.com/asyncapi/asyncapi-react/blob/master/.github/workflows/bump.yml

It is because it uses the default workflow from .github that is using default location of package.json from root, to grab package name. The workflow needs an update here or in .github but then we need to figure out, hot to overwrite default packagejson_path with custom value, like ./library. Probably environment variables.

cc @Shurtu-gal so he is aware of issues in Studio.

[derberg]

Its confusing people for very long, look: https://studio.asyncapi.com/?share=3469f1ef-eae6-4e23-a53c-f4d2877fd857 it still runs on version there reverts meaning of action verbs :(

actually it is a different weird bug. The switch of verbs happens only if reply is present. If I remove reply, the verb shows as SEND not RECEIVE

[Shurtu-gal]

I will take a look into this this weekend.

[derberg]

@Shurtu-gal did you have a chance to have a look into it?

[Shurtu-gal]

Sorry, I was actually travelling to college 2 weeks ago and was down with fever after that.

I will put this on priority. Had totally forgotten about this.

[Shurtu-gal]

Regarding the bump workflow should we make it customisable to make it work in monorepos?

Or we can simply find package.json to max depth of 3 as I suppose that would be enough.

[Shurtu-gal]

actually it is a different weird bug. The switch of verbs happens only if reply is present. If I remove reply, the verb shows as SEND not RECEIVE

I believe that this is a problem in asyncapi-react, as the parsed spec looks alright. I will look into this repo's code and update here.

[Shurtu-gal]

After some digging around found the reason:
https://github.com/asyncapi/asyncapi-react/blob/master/library/src/helpers/common.ts#L21-L28

Whenever a operation has reply according to this it should show REQUEST.

cc: @derberg

[asyncapi-bot]

Hello, @Shurtu-gal! 👋🏼

I'm 🧞🧞🧞 Genie 🧞🧞🧞 from the magic lamp. Looks like somebody needs a hand!

At the moment the following comments are supported in issues:

• /good-first-issue {js | ts | java | go | docs | design | ci-cd} or /gfi {js | ts | java | go | docs | design | ci-cd} - label an issue as a good first issue.
  example: /gfi js or /good-first-issue ci-cd

[derberg]

@Shurtu-gal but the problem if the issue is not that it is done wrong here, but that it is still visible wrong in studio as studio runs on old version. Or?

[Shurtu-gal]

I upgraded to the latest version still the problem is there.

Also tried in the demo url in this repo

[derberg]

I upgraded to the latest version still the problem is there.

you mean on your local right?

[Shurtu-gal]

Yeah

[derberg]

ah, ok, looks like it was changed intentionally.

problem is that verb Request do not imply if it is inbound or outbound, so the reply you have specified, it is hard to identify if reply is to incomming request or a reply from server to outgoing request.

@jonaslagoni do you remember why in #856 you introduced REQUEST label by default?