Exclude `node_modules` by default from JS Code Hints.

See #8646.

This can be overridden by adding a .jscodehints file and setting
excluded-directories in there.
`baseRegExp` is always added to `settings`, so `regExpString` will never be empty, so we'll never execute this code block. Is that right?