Annotate all internal functions with '@internal'

[IvanGoncharov]

Even though some functions are expored from src/**/*.js files they are private.
The only function exported by src/index.js and src/*/index.js are considered to be part of the public API. Please see:
https://github.com/graphql/graphql-js/blob/master/src/README.md

It would be great to have all private API marked as such, e.g.:

graphql-js/tstypes/validation/validate.d.ts

Lines 32 to 33 in 6f341a3

	// @internal
	export function validateSDL(

Also, it would be great to find all public APIs without JS doc comments.

[wtrocki]

Working on it. Removed assignment by mistake

[wtrocki]

Spent the entire day on this and doing this by hand is really tricky and prone to errors.
I could do something fast but I would not be content about every change there and verification would be also tricky as well.

How to resolve this problem

My approach for this would be to:

• Build a unit test/script (after migration to ts it will be ts-lint rule) that will read the entire exported api from the index.
• Then it will traverse thru the source code and compare individual index.js exports (and individual files)
• Build diff and print it (later it could just produce a warning

There is the interest of having this kind of check-in TypeScript community (Exported member is not available globally) and we can configure comment format that will disable that as a warning.

Advantage

Advantage of this approach is that:

• It could be enforced at review time which safes mental effort to review PR's
• It will be not TypeScript/Flow specific
• It will reduce the tedious job of asking people to do it for every PR - when introducing new changes
• Having ability to replicate this fast will mean that there is a lesser chance of doing the same manual job twice when source code will be in typescript.

What it means that it will require much more work initially, but it will be much better for tracking API changes in the future.
@IvanGoncharov

I have seen some babel tools that can help here, but if you guys know any other tools that can help feel free to post it here.

[IvanGoncharov]

@wtrocki tslint is in deprecated state, you can write a custom ESlint and put it in the resources directory.

[wtrocki]

Yes. I meant ESLint. For the moment I just have node script that need to print out method names so I can find them and manually add comment

[wtrocki]

Got some progress on this but still not ready for PR. I will be off next week so hopefuly I will manage to create PR next next week