Parse ObjectID scalar to Mongo ObjectID

[machi1990]

Currently the ObjectID scalar only checks for conformity with the Mongo ObjectID

We, at Graphback are considering the usage of this scalar type, we are considering the options of parsing the value directly to ObjectID and not return the raw string. In doing so, thought that this is a good feature request in upstream.

Opening for consideration and discussion.

/cc @craicoverflow @Urigo @Wrocki

[ardatan]

@machi1990 I already tried to return ObjectID instead of string but treeshaking didn't work well with mongodb package. That's why, I kept ObjectID scalar as it is.

[machi1990]

@machi1990 I already tried to return ObjectID instead of string but treeshaking didn't work well with mongodb package. That's why, I kept ObjectID scalar as it is.

Hi @ardatan thanks for the reply. Are there plans to have such a support e.g by investigating the treeshaking issue further?

[Urigo]

@machi1990 yes we would be open for this change, would you care to try a PR that might tackle this issue?

[machi1990]

@Urigo @ardatan I gave it a try here but the bundle size more than doubled.

./dist/index.js: 191.88KB > maxSize 85KB (no compression) 

Was this the approach that you considered? @ardatan

[ardatan]

Webpack assumes exported GraphQLScalarType instances have always side effects. For most of them, pure magic comment works;
https://github.com/Urigo/graphql-scalars/blob/master/src/scalars/ObjectID.ts#L5
But it doesn't work well with MongoDB's ObjectID somehow.

[ash0080]

ObjectID should return a fixed type, but now, it could be a string or an ObjectID, A basic principle should be at least that the incoming and outgoing are the same type.

Same case, e.g. URL, incoming is a string, return type is an URL object, The silliest thing is that it's stored in the database and my storage has grown 110% as a result. And really, all one needs is a regex check
besides, PhoneNumber can't work without a '+' (???)
...

Sorry for gushing, but please understand that I struggled for hours and eventually found I had to rewrite resolvers myself, There are so many incredible

[ardatan]

@ash0080 There are multiple subjects here :) First, ObjectID should always string in our scalar (if not this is a bug).
Second, you can easily stringify incoming URL object with 'toString'. The scalar uses URL object like DateTime scalar uses Date object which can be easily formatted with 'toJSON' or 'toString' easily. Those are already mentioned in the docs afaik. If you want to have a validation only and keep the incoming string as-is, you can use RegularExpression factory to create your scalars easily.

[ash0080]

const { GraphQLScalarType, Kind, GraphQLError } = require('graphql')
const MONGODB_OBJECTID_REGEX                    = new RegExp('^[0-9a-fA-F]{24}$')
const URL_REGEX                                 = new RegExp('^(?:http(s)?:\\/\\/)?[\\w.-]+(?:\\.[\\w\\.-]+)+[\\w\\-\\._~:/?#[\\]@!\\$&\'\\(\\)\\*\\+,;=.]+$')
const { ObjectID }                              = require('mongodb')
const validateObjectId = value => {
  if (typeof value !== 'string' || !(value.length === 24 && MONGODB_OBJECTID_REGEX.test(value))) {
    throw new TypeError(`Value is not a valid mongodb object id of form: ${value}`)
  }
  return ObjectID(value)
}
const validateURL      = value => {
  if (!URL_REGEX.test(typeof value === 'string' ? value.trim() : value)) {
    throw new TypeError(`Value is not a valid URL of form: ${value}`)
  }
  return value
}
module.exports         = {
  ObjectID: new GraphQLScalarType({
                                    name: 'ObjectID',
                                    description: 'MongoDB ObjectID type.',
                                    parseValue (value) { // value from the client
                                      return validateObjectId(value)
                                    },
                                    serialize (value) { // value sent to the client
                                      if (ObjectID.isValid(value)) {
                                        return value.toHexString()
                                      }
                                      throw new TypeError(`Value is not a valid ObjectID of form: ${value}`)
                                    },
                                    parseLiteral (ast) { // ast value is always in string format
                                      if (ast.kind !== Kind.STRING) {
                                        throw new GraphQLError(`Can only validate strings as mongodb object id but got a: ${ast.kind}`)
                                      }
                                      return validateObjectId(ast.value)
                                    }
                                  }),
  URL: new GraphQLScalarType({
                               name: 'ObjectID',
                               description: 'A field whose value conforms with the standard mongodb object ID as described here: https://docs.mongodb.com/manual/reference/method/ObjectId/#ObjectId. Example: 5e5677d71bdc2ae76344968c',
                               serialize: validateURL,
                               parseValue: validateURL,
                               parseLiteral (ast) {
                                 if (ast.kind !== Kind.STRING) {
                                   throw new GraphQLError(`Can only validate strings as mongodb object id but got a: ${ast.kind}`)
                                 }
                                 return validateURL(ast.value)
                               }
                             })

I've given up on this. rewrite is faster, besides, ObjectID support auto conversion now

update, fix [ObjectID] support

[ardatan]

My concern about ObjectID is the bundle size. Treeshaking doesn't work well with mongodb package so it increases bundlesize heavily and I don't want to use ObjectID from mongodb directly.
For URL, you can use RegularExpression factory like below;

const MyRegexType = new RegularExpression('MyRegexType', /^ABC$/);

[ash0080]

You can just extract the code from ObjectID or let the user pass in an instance

[ardatan]

What do you mean by extracting code from ObjectID and how do we decide if that is an instance of ObjectID without importing mongodb?

[ash0080]

node_modules/bson/lib/bson/objectid.js 

see? Why do you want to reference the entire mongodb?
You can use bson-objectid, it's 33k,Even if you quote the entire bson, it's only 2.5M

[ardatan]

The entire GraphQL Scalars package is 90 kB and even 33 kB is too much compared to entire size but all scalars are treeshakable so for example if you use only one scalar, you will only have that scalar's bundle size.
If we use any other package for ObjectID, that might not work with MongoDB driver because it uses bson npm library and sometimes it uses bson-ext if available.

[ash0080]

It doesn't make sense, In the end, you can specify mongodb as an external module in webPack ::jack_o_lantern: