joarwilk / flowgen

Generate flowtype definition files from TypeScript
Other
658 stars 78 forks source link

Error when parsing Record type #155

Closed remarkablemark closed 2 years ago

remarkablemark commented 2 years ago

index.d.ts:

declare const foo: Record<string, string>

flowgen:

$ npx flowgen index.d.ts

Error:

(node:315) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'length' of undefined
    at getFullyQualifiedName (/home/runner/flowgen-error/node_modules/flowgen/lib/printers/node.js:127:68)
    at printType (/home/runner/flowgen-error/node_modules/flowgen/lib/printers/node.js:504:41)
    at Object.fn (/home/runner/flowgen-error/node_modules/flowgen/lib/env.js:10:12)
    at Object.propertyDeclaration (/home/runner/flowgen-error/node_modules/flowgen/lib/printers/declarations.js:46:31)
    at printType (/home/runner/flowgen-error/node_modules/flowgen/lib/printers/node.js:534:36)
    at fn (/home/runner/flowgen-error/node_modules/flowgen/lib/env.js:10:12)
    at Array.map (<anonymous>)
    at parseNameFromNode (/home/runner/flowgen-error/node_modules/flowgen/lib/parse/ast.js:41:86)
    at collectNode (/home/runner/flowgen-error/node_modules/flowgen/lib/parse/index.js:70:51)
    at node.statements.forEach.n (/home/runner/flowgen-error/node_modules/flowgen/lib/parse/index.js:111:34)
(node:315) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:315) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

https://replit.com/@remarkablemark/flowgen-error-when-parsing-Record

May be related to latest TypeScript version

remarkablemark commented 2 years ago

No longer seeing the error after upgrading to the latest version

pladaria commented 2 years ago

I am getting this same error on v1.16.0

@remarkablemark you mean upgrading typescript or flowgen?

remarkablemark commented 2 years ago

@pladaria I upgraded both and the issue was resolved for me.

Update: it seems like the error resurfaces in typescript v4.5.3. Try installing typescript@4.5.2 to see if that fixes your error.