Docgen: Some types omitted

[sirreal]

Some types may be omitted from docgen output. For example:

/** @type {Record<string,true|undefined>} */

gutenberg/packages/deprecated/src/index.js

Lines 6 to 12 in 0545e7d

	/**
	* Object map tracking messages which have been logged, for use in ensuring a
	* message is only logged once.
	*
	* @type {Record<string,true|undefined>}
	*/
	export const logged = Object.create( null );

Was rendered as:

_Type_

-   `Record<string,(|undefined)>` 

Originally posted by @sirreal in #26429 (comment)

[sarayourfriend]

@sirreal @gziolo What do you think about us trying to re-write docgen using the TypeScript parser? We'd ditch babel and doctrine altogether and just use TS to parse the source and the JSDoc comments. It does a really good job on JavaScript files: https://astexplorer.net/#/gist/5931456cb14f4dfbeef6a824c04cacce/2ae696cccaedc016a032ff636f4b2aaf977f2514

It'd be a lot of work (basically a full re-write of the entire docgen utility) but it would solve this problem and let us ditch the deprecated doctrine library.

If this a desirable path forward for the docgen package, I'm super excited to work on this and would be happy to do so.

Update: Actually it looks like this exact solution was already proposed in #18045. Shall we close this issue and revive the topic over there?

[gziolo]

Update: Actually it looks like this exact solution was already proposed in #18045. Shall we close this issue and revive the topic over there?

I think that @nosolosw would be the best person to answer your question but I believe that you are correct. #28615 should resolve two issues at once 🎉