Wrong parameter type in schema

[chengyak]

Hi, I have a REST API and in the json, the parameter type is integer
"parameters": [ { "name": "id", "in": "path", "description": "ID of the company.", "required": true, "schema": { "type": "integer", "description": "ID of the company.", "format": "int64" } } ],

however, the 'id' parameter shows as float in the graphql schema in the doc in the playground(company(id: Float!): Company). is t his caused by there is no type in Graphql to handle int64? Thanks

[ardatan]

We are currently rewriting OpenAPI handler and releasing it under @graphql-mesh/new-openapi and newOpenapi names.
So it should have been fixed there. Would you try it out?
https://github.com/Urigo/graphql-mesh/blob/master/packages/handlers/new-openapi/yaml-config.graphql#L5

[chengyak]

the endpoint i am using is an internal endpoint, so i think i cannot reproduce outside. but is this something to do with version dependency?
here is the packages I am using:
"@graphql-mesh/cli": "0.72.1",
"@graphql-mesh/graphql": "0.27.1",
"@graphql-mesh/openapi": "^0.26.1",
"@graphql-mesh/runtime": "0.36.1",
"@graphql-mesh/transform-filter-schema": "0.14.62",
"@graphql-mesh/transform-resolvers-composition": "^0.12.71",
"apollo-server": "3.8.2",
"apollo-server-express": "^3.10.0",
"express": "^4.18.1",
"graphql": "15.3.0",
"graphql-middleware": "^6.1.32",
"graphql-request": "^4.3.0",
"graphql-shield": "^7.5.0",
"graphql-tools": "^8.3.1",

and I notice that if I upgrade @graphql-mesh/openapi to the latest version (0.28) I will get
Failed to generate the schema TypeError: config.fetch is not a function error, very similar error than switching to the new openapi.

[ardatan]

Sorry but this is not helpful so we cannot help you with these information. You should provide a reproduction on CodeSandbox or StackBlitz or a failing test in a PR.

[chengyak]

I create a repo and one example endpoint to reproduce this issue. please git clone the code from https://github.com/chengyak/graphqlmesh and do npm install / npm run dev and you should see this exception. Thanks for the help

[ardatan]

Thank you that is really helpful 🙏 Oh you need to change source to oasFilePath
https://github.com/Urigo/graphql-mesh/blob/master/packages/handlers/new-openapi/yaml-config.graphql#L5
See this configuration schema for the new handler.
But still I'm still testing it :) I'll keep you posted.

[ardatan]

It seems to be working with these canaries. I did some fixes by using your schema;
#4237 (comment)
You can test it until it gets released!
Thanks for all the information! It was really helpful for us to make Mesh better.