Add hook for GraphQLScalarType

[ab-pm]

Feature description

There is no hook for GraphQLScalarType. We should add one!

Motivating example

I wanted to hook into the creation of the scalar types in pgTypes plugin. In particular, I wanted to rename Date to IsoDate (ok, there's an inflection for that), update the description (would've been possible by hooking into wrapDescription, but it's not really meant for that), and update the parsing methods to validate a strict YYYY-MM-DD format not everything that Postgres allows.

I got a workaround that overwrites build.graphql.GraphQLScalarType before pgTypesPlugin uses it, but it's really ugly.

Supporting development

• I am willing to wait for v5
• I am a Graphile sponsor ❤️

[benjie]

Love bullet 1 🙌

[benjie]

Incidentally if you fancied having a pop at this yourself; you basically need code like this:

graphile-engine/packages/graphile-build/src/makeNewBuild.js

Lines 796 to 806 in 6692c2a

	const commonContext = {
	type: "GraphQLEnumType",
	scope,
	};
	newSpec = builder.applyHooks(
	this,
	"GraphQLEnumType",
	newSpec,
	commonContext,
	`|${newSpec.name}`
	);

Search through that file (and the .d.ts that goes with it) for GraphQLEnumType and add similar code for GraphQLScalarType (except even simpler since you don't have to deal with values). Then find where we've done new GraphQLScalarType( in the codebase and replace it with newWithHooks(GraphQLScalarType, and you're basically done I think.

(It's a bit more effort than I make it sound.)

[benjie]

V5 does indeed have a GraphQLScalarType hook.