flow-typed v2.2.0 fails on windows

npx flow-typed
...\node_modules\flow-typed\dist\cli.js
Invalid regular expression: /definitions\npm\(@[^\]*\)?[^\]*\?/: Unterminated character class

It doesn’t matter what commands I run and what folder I run them in - end result is the same.

• node v8.1.2
• Windows 10 x64

[kyrylkov]

@marudor any update?

[kulmajaba]

Same issue for me on Windows.

The issue is in \dist\commands\runTests.js line 70:

var basePathRegex = new RegExp('definitions' + _node.path.sep + 'npm' + _node.path.sep + '(@[^' + _node.path.sep + ']*' + _node.path.sep + ')?[^' + _node.path.sep + ']*' + _node.path.sep + '?');

On Windows, _node.path.sep is \ but it’s also the escape symbol, so the symbols have to be double-escaped: \\\\

Another option is to use / for Windows as well. From Node.js docs:

Note: On Windows, both the forward slash (/) and backward slash (\) are accepted as path segment separators; however, the path methods only add backward slashes (\).

Both options worked for me in Windows, testing in Command Prompt, Node.js Command Prompt and Git Bash.

Which approach should we take? And if we go for the double-escaped backslashes, how can we do it neatly in the code?

If someone can point me to the right direction I can submit a PR.