Typecheck (BREAKING CHANGES)

[goodmind]

Probably experimental and shouldn't be merged?

CLI

• Add --no-module-exports options, it disables declare module.exports syntax in favor of declare export default
• Add --flow-typed-export [folder] instead of exporting to ../exports folder, it now exports to ./exports and you can change folder name
• Remove --string-enums flag

Compiler (breaking change?)

• Use TypeScript typechecker for name resolving

Parser

• Use transformers to remove legacy syntax with new

  • ImportEqualsTransformer
    Replace import x = require('y') with import * as x from 'y'

  • LegacyModules
    Replace module Identifier {} with namespace Identifier {}
    (don't confuse with declare module 'package-name' {})

• Use recursive addChildren for nested namespaces

• Print enums, classes, and nested namespaces identifiers inside namespaces

• Use Symbol.for('nodejs.util.inspect') for printing nodes in console instead of cloning them

Printers

• Print empty line for import 'side-effect' in declare module
• Move literalType printer
• Move typeParameter printer
• Move propertyDeclaration printer
• Replace class enums with simplified object enums, resolve enum and enum members references with typechecker
• Qualify identifiers inside class and interface (resolve namespaces)
• Add support for unnamed default export
  export default function(): void;
• Add support for renaming module identifiers (@types/react should be installed)

From	To
import * as React from 'react' type S = React.ReactNode	import * as React from 'react' type S = React.Node
import {ReactNode} from 'react' type S = ReactNode	import type {Node} from 'react' type S = Node

• Error on BigInt (I actually added parsing of BigInts to Flow in Add support for BigInt parsing facebook/flow#7471)

• Error on unique symbol

• Convert typeof import('module') type to $Exports<'module'>

[orta]

If it's useful, I'm fine with bumping this lib in major semver for it?

[goodmind]

@orta, yeah, probably would be ok, i'm just sure that tests doesn't cover everything and this looks like major shift in implementation

[goodmind]

@orta anything preventing merging this?

[orta]

Nope, just me coming back and merging it :D

[orta]

OK, shipped as 1,6,0

[goodmind]

@orta do you mind if I had rights to commit and label issues directly in this repo?

[orta]

I'd prefer it, but I don't have access to give to that access I'm afraid

[FezVrasta]

Was this ever released? --flow-typed-export tells me it's not a valid option.

[ozyman42]

Use --flow-typed-format instead of --flow-typed-export