This code used a "safeList" global variable to cache the result of loading the safe list.
This has the unintuitive behavior that the value of the safeListPath parameter isn't observed after the first time discoverTypings is called.
Instead, we require the caller of discoverTypings to pass in the safe list itself instead of a path, and require the caller to handle caching of the safe list.

There is also a private static safelist in editorServices.ts that looks suspicious, but I'll leave that for later.